Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ebook reader crashes Nextcloud 17 #125

Open
Wotisrv opened this issue Oct 22, 2019 · 10 comments
Open

Ebook reader crashes Nextcloud 17 #125

Wotisrv opened this issue Oct 22, 2019 · 10 comments

Comments

@Wotisrv
Copy link

Wotisrv commented Oct 22, 2019

When I download and enable ebook reader I get the internal error screen.

Removing the files_reader directory fixes the problem.

Nextcloud 17.0.0
Php 7.3.10
Fedora 30 Server Edition

@pkostelnik
Copy link

Same here, but just disabling the app with php occ app:disable files_reader returns the nextcloud back to work.

  • Nextcloud 17.0.0
  • PHP 7.2.24
  • Ubuntu 18.04.3 LTS

@AetherCollective
Copy link

same here

@Wotisrv
Copy link
Author

Wotisrv commented Feb 5, 2020

Still present with NC18 and upgraded Fedora server. I have disabled the app via occ command
Is there any reason to use this app by the way?

@AetherCollective
Copy link

AetherCollective commented Feb 5, 2020 via email

@SimJoSt
Copy link

SimJoSt commented Feb 20, 2020

I have contacted @Yetangitu about this via email to get an update.

@Wotisrv
Copy link
Author

Wotisrv commented Mar 3, 2020

Is there something happening here?

@noci2012
Copy link

noci2012 commented Mar 6, 2020

The reader still uses OCP\Config which has been removed...
I am trying to get a hold of the Nextcloud API and it is not as trivial as things look like.
The API documentation is probably correct but hard to get into when it's all new. And not all things described are necessarily the most recent state of things (yet).

Although this is a small fix ... there is a huge difference between the API when the reader was build and the current APIs.. Since V15 old stuff is getting removed ==> breaking things that are not updated.
A lot of developpers gave up on rewriting most of their application to accomodate to the new API.. between V11 - V16.

@noci2012
Copy link

noci2012 commented Mar 6, 2020

@yeupou
Copy link

yeupou commented Apr 6, 2020

Same on nextcloud 18.

@e-alfred
Copy link

I forked the files_reader app and will publish a new release to the App Store soon:

https://github.com/e-alfred/epubreader

If anybody wants to help me out with development, any contributions are welcome!

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

No branches or pull requests

7 participants