epubreader icon indicating copy to clipboard operation
epubreader copied to clipboard

Published 20 hours ago • verified publisher icon e-alfred

Cannot de-select PDF from file types for which Reader should be the default viewer

Open dysonsphere-startmail opened this issue 4 years ago • 10 comments

In Settings > Additional Settings I try to de-select Reader from opening PDFs. I get a "saving..." message that hangs. The setting never sticks.

Nextcloud: 20.0.4 EPUB/CBZ/PDF ebook reader: 1.4.5

dysonsphere-startmail avatar Dec 20 '20 15:12 dysonsphere-startmail

Same issue for me here also using Nextcloud 20.0.4, EPUB/CBZ/PDF ebook reader: 1.4.5

JazMTaz avatar Dec 28 '20 04:12 JazMTaz

Same here! I have several shared PDFs and when accessing them it loads the default view (which is fine) but also a pop-up window with the reader!

m0yP avatar Dec 28 '20 17:12 m0yP

Would love this, too :) Actually, this is the only issue that is preventing me from installing the app currently

szaimen avatar Jan 11 '21 12:01 szaimen

This is actually the same bug as issue #20

rhein avatar Jan 13 '21 17:01 rhein

Hi.

Just want to add my "me too" on this problem. Running EPUB/CBZ/PDF ebook reader 1.4.6 on NextCloud 20.0.9 and I can't de-select the PDF option.

Also it wouldn't be a bad idea to have a global option to prevent EPUB/CBZ/PDF ebook reader of "overtake" the PDF on the whole server and for the whole users.

Thanks Pedro

D1reW0lf avatar May 06 '21 10:05 D1reW0lf

Still having this issue in 1.4.7 with NC 23. I am able to deselect it, but it will just say "Saving..." and when I reload, it is selected again.

Kevin8675 avatar Dec 23 '21 23:12 Kevin8675

bump (is any dev here around and reading any comments?)

manuelkamp avatar Jan 01 '22 21:01 manuelkamp

yeah, such a useful app, it should be part of the core actually, but I had to uninstall again because of this issue.

TheNomad11 avatar Jan 13 '22 09:01 TheNomad11

I searched a little bit why this does not work anymore.

In the App metadata, the route to save the settings is not defined like in this documentation : https://docs.nextcloud.com/server/latest/developer_manual/basics/routing.html

I don't know how the settings are actually displayed, because there is no metadata set in this app, like in : https://docs.nextcloud.com/server/latest/developer_manual/basics/setting.html

This must have been changed at some point in some version of Nextcloud.

The code to handle the settings change is not done in a "Controller", but in a simple file : ajax/personal.php

This app need a rework to work fully in recent version of Nextcloud.

I may give it a try, but it will be a big challenge for me :o

sysadminstory avatar Jan 20 '22 22:01 sysadminstory

My PR #41 should fix this issue.

I'm not a Nextcloud app developper, so this PR needs to be reviewed before being production ready !

sysadminstory avatar Jan 22 '22 01:01 sysadminstory