Skip to content

Adjust files_opds for Nextcloud 16#121

Open
noci2012 wants to merge 1 commit intoYetangitu:masterfrom
noci2012:origin/HEAD
Open

Adjust files_opds for Nextcloud 16#121
noci2012 wants to merge 1 commit intoYetangitu:masterfrom
noci2012:origin/HEAD

Conversation

@noci2012
Copy link
Copy Markdown

These fixes seem to work.
wost thing was the template handling wire file => $file caused a clash.

See also: #120

@guoguoguilai
Copy link
Copy Markdown

guoguoguilai commented Nov 12, 2019

These fixes seem to work.
wost thing was the template handling wire file => $file caused a clash.

See also: #120

Still not work. After adding the file_opds into Nextcloud, an item called Additonal settings was created. However, when I clicked Additonal settings as admin, it just got "HTTP ERROR 500"; when I clicked Additonal settings as personal user, it just got "Directory does not exist!"

And the logging panel shows:
Cannot redeclare checkBox() (previously declared in /var/snap/nextcloud/16739/nextcloud/extra-apps/files_opds/templates/admin.php:15) at /var/snap/nextcloud/16739/nextcloud/extra-apps/files_opds/templates/admin.php#15
How can I fix it?

@noci2012
Copy link
Copy Markdown
Author

Note this PR is nt merged in the "official" here yet.
You need the tree from my repo.

@guoguoguilai
Copy link
Copy Markdown

guoguoguilai commented Nov 24, 2019 via email

@noci2012
Copy link
Copy Markdown
Author

The chosen directory in the user context should already exist. (AFAICT).
(I already had a nextcloud directory, Library, where the documents were) so that why i never ran into this.)
I will look into this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants