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

Compatibility with Nextcloud 14 #94

Closed
juliusknorr opened this issue Aug 10, 2018 · 21 comments
Closed

Compatibility with Nextcloud 14 #94

juliusknorr opened this issue Aug 10, 2018 · 21 comments

Comments

@juliusknorr
Copy link

Hi,

as you may have already seen, we have entered the beta phase for the release of Nextcloud 14. We try to keep up compatibility with older apps, but in some cases apps need some adjustments to properly work on new Nextcloud releases. With Nextcloud 14 there have been quite some changes in that regard:

You can find a list of all critical changes for app developers and admins at nextcloud/server#7827 You can run occ app:check-code to validate that there are no issues with deprecated or private message calls.

One critical change is the overall restructuring of the main template (nextcloud/server#9982) that might require some adjustments in your app.

We would be very pleased if you can make your app compatible with Nextcloud 14 and release a new version to the app store, so that users can properly test it and the upgrade experience to Nextcloud 14 will be even better than before.

Feel free to ask, if you have any questions regarding making your app compatible with Nextcloud 14.

Btw. you might have seen that starting August 25, the Nextcloud Conference takes place at the familiar TU Berlin. (https://nextcloud.com/conf/)! We'd like all contributors to be there and that most certainly includes you. You might already have registered but if you haven't - please consider joining us!

Thanks for being part of the Nextcloud community and for all the effort you put into providing this app 🚀

@e-alfred
Copy link
Owner

e-alfred commented Aug 14, 2018

Thanks for the heads-up, there are two major roadblocks toward Nextcloud 14 compatibility:

  • Rewrite of database queries because of the deprecated and removed OCP\DBinterface, started already in branches db-rework1 and db-rework.
  • Making the UI compatible with the restructured design, not started (yet).

If somebody wants to speed up the process, pull requests are welcome!

@brandan-schmitz
Copy link

Is there a time estimate to when this could be released for Nextcloud 14? I could really use this plugin but cannot as my instance is NC 14.

@e-alfred
Copy link
Owner

e-alfred commented Oct 7, 2018

No ETA, but I am working on moving all the SQL queries to Querybuilder, just as I said above. If you want to help out, it can be done faster probably.

@Goeste
Copy link

Goeste commented Nov 14, 2018

any help with translations needed?

@lsnow99
Copy link

lsnow99 commented Nov 24, 2018

Hey I'm happy to help with the SQL migration. Do you have any info on where I can start or any progress you've already made?

@e-alfred
Copy link
Owner

@Goeste Sure, if you have any new translations of find any mistakes please submit a PR!

@lsnow2017 Here is my PR with all the fixes: #79
I also created a branch with all the changes: https://github.com/e-alfred/ocdownloader/tree/db-rework

I am very happy with any help because I am currently a bit overstretched with other projects.

@bekon16
Copy link

bekon16 commented Dec 14, 2018

Hi, any progress with ocdownloader on nextcloud 14? It was most important app for me. Its almost half year since i have to downloading files using putty - pain in the ass. Should i downgrade to nextcloud 13? :(

@CHazz
Copy link
Contributor

CHazz commented Dec 21, 2018

Hi, any progress with ocdownloader on nextcloud 14? It was most important app for me. Its almost half year since i have to downloading files using putty - pain in the ass. Should i downgrade to nextcloud 13? :(

i downgrade to NC 13 .. but still waiting and waiting and nothing :(

@blackcrack
Copy link

blackcrack commented Dec 21, 2018

hmm, sometimes i use wget in ssh over "screen" (screen --help) in own "screen" task ,
but if could be possible to have ocdownloader again,
would it great :)
@e-alfred So why are/Where it depends ? *smile*
edit:
@e-alfred you should add a couple of members in this ocdownloader project, where makes able to collaborating. If something with a member, to be possible to goes on further ..
i have no time, but i guess, @jayarc have the possible ..

@jayarc
Copy link

jayarc commented Dec 26, 2018

I have forked the latest db-rework2 branch and tried to fix some of the UI things going on there. I also modified the youtube.php file to use the linux which command to find the proper path, it was failing for me to get the Youtube URL because it wasn't getting the youtube-dl path right.

To use the fork on NC 14+ issue this command inside your nextcloud apps directory:
git clone --single-branch --branch nc-14 https://github.com/jayarc/ocdownloader.git

@bugsyb
Copy link

bugsyb commented Dec 30, 2018

@jayarc - many thanks for that.
I've tried it on 14.0.4 stable. Once app is enabled (from CLI) and opened it unfortunately raises below message.
Had to disable it as couldn't even open generic settings page.
Maybe I'm missing something at my end?

/apps/ocdownloader/add","message":{"Exception":"Error","Message":"Class 'OCP\DB' not found","Code":0,"Trace":[{"file":"/nextcloud/apps/ocdownloader/controller/lib/tools.php","line":187,"function":"getValue","class":"OCA\ocDownloader\Controller\Lib\Settings","type":"->","args":[]},{"file":"/nextcloud/apps/ocdownloader/controller/index.php","line":47,"function":"canCheckForUpdate","class":"OCA\ocDownloader\Controller\Lib\Tools","type":"::","args":[]},{"file":"/nextcloud/apps/ocdownloader/appinfo/application.php","line":44,"function":"__construct","class":"OCA\ocDownloader\Controller\Index","type":"->","args":["ocdownloader",{"class":"OC\AppFramework\Http\Request"},"mypooruser",{"class":"OC\L10N\L10N"}]},{"file":"/nextcloud/3rdparty/pimple/pimple/src/Pimple/Container.php","line":118,"function":"OCA\ocDownloader\AppInfo\{closure}","class":"OCA\ocDownloader\AppInfo\Application","type":"->","args":["*** sensitive parameters replaced "]},{"file":"/nextcloud/lib/private/AppFramework/Utility/SimpleContainer.php","line":117,"function":"offsetGet","class":"Pimple\Container","type":"->","args":["IndexController"]},{"file":"/nextcloud/lib/private/AppFramework/DependencyInjection/DIContainer.php","line":446,"function":"query","class":"OC\AppFramework\Utility\SimpleContainer","type":"->","args":["IndexController"]},{"file":"/nextcloud/lib/private/AppFramework/DependencyInjection/DIContainer.php","line":424,"function":"queryNoFallback","class":"OC\AppFramework\DependencyInjection\DIContainer","type":"->","args":["IndexController"]},{"file":"/nextcloud/lib/private/AppFramework/App.php","line":95,"function":"query","class":"OC\AppFramework\DependencyInjection\DIContainer","type":"->","args":["IndexController"]},{"file":"/nextcloud/lib/private/AppFramework/Routing/RouteActionHandler.php","line":47,"function":"main","class":"OC\AppFramework\App","type":"::","args":["IndexController","Add"," sensitive parameter replaced ***",{"_route":"ocdownloader.Index.Add"}]},{"function":"__invoke","class":"OC\AppFramework\Routing\RouteActionHandler","type":"->","args":[{"_route":"ocdownloader.Index.Add"}]},{"file":"/nextcloud/lib/private/Route/Router.php","line":297,"function":"call_user_func","args":[{"class":"OC\AppFramework\Routing\RouteActionHandler"},{"_route":"ocdownloader.Index.Add"}]},{"file":"/nextcloud/lib/base.php","line":987,"function":"match","class":"OC\Route\Router","type":"->","args":["/apps/ocdownloader/add"]},{"file":"/nextcloud/index.php","line":42,"function":"handleRequest","class":"OC","type":"::","args":[]}],"File":"/nextcloud/apps/ocdownloader/controller/lib/settings.php","Line":80,"CustomMessage":"--"},"userAgent":"Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:64.0) Gecko/20100101 Firefox/64.0","version":"14.0.4.2"}

@bekon16
Copy link

bekon16 commented Dec 31, 2018

I have this error:
`Error: Call to undefined function style()
/var/www/nextcloud/lib/private/legacy/app.php - line 622:

include()

/var/www/nextcloud/lib/private/Settings/Manager.php - line 365:

OC_App::getForms("personal")

/var/www/nextcloud/settings/Controller/CommonSettingsTrait.php - line 93:

OC\Settings\Manager->getPersonalSections()

/var/www/nextcloud/settings/Controller/CommonSettingsTrait.php - line 46:

OC\Settings\Controller\PersonalSettingsController->formatPersonalSections("personal", "personal-info")

/var/www/nextcloud/settings/Controller/CommonSettingsTrait.php - line 125:

OC\Settings\Controller\PersonalSettingsController->getNavigationParameters("personal", "personal-info")

/var/www/nextcloud/settings/Controller/PersonalSettingsController.php - line 57:

OC\Settings\Controller\PersonalSettingsController->getIndexResponse("personal", "personal-info")

/var/www/nextcloud/lib/private/AppFramework/Http/Dispatcher.php - line 166:

OC\Settings\Controller\PersonalSettingsController->index("personal-info")

/var/www/nextcloud/lib/private/AppFramework/Http/Dispatcher.php - line 99:

OC\AppFramework\Http\Dispatcher->executeController(OC\Settings\ ... {}, "index")

/var/www/nextcloud/lib/private/AppFramework/App.php - line 118:

OC\AppFramework\Http\Dispatcher->dispatch(OC\Settings\ ... {}, "index")

/var/www/nextcloud/lib/private/AppFramework/Routing/RouteActionHandler.php - line 47:

OC\AppFramework\App::main("OC\Setting ... r", "index", OC\AppFramew ... {}, { section: " ... "})

OC\AppFramework\Routing\RouteActionHandler->__invoke({ section: " ... "})

/var/www/nextcloud/lib/private/Route/Router.php - line 297:

call_user_func(OC\AppFramew ... {}, { section: " ... "})

/var/www/nextcloud/lib/base.php - line 987:

OC\Route\Router->match("/settings/user")

/var/www/nextcloud/index.php - line 42:

OC::handleRequest()`

@Goeste
Copy link

Goeste commented Jan 2, 2019

I can second @bugsyb error: #108

@e-alfred e-alfred added the bug label Jan 5, 2019
@e-alfred
Copy link
Owner

e-alfred commented Jan 5, 2019

I created a new testing release with all the UI fixes by @jayarc that is compatible with NC14+:

https://github.com/e-alfred/ocdownloader/releases/tag/1.6.0-Alpha1

Currently there are two things that don't work:

  • Personal settings
  • API for extensions

@bekon16
Copy link

bekon16 commented Jan 5, 2019

Hi, not only Personal settings are not working. Also admin settings page is unavailable so without this (admin page settings) this app is useless, but overall good job.
I'm wating for version with working settings page.

@e-alfred
Copy link
Owner

e-alfred commented Jan 9, 2019

@bekon16 I created a new release that should fix the personal settings problem for now.

@jtibrewala
Copy link

Can you please repost the link to the release?

@e-alfred
Copy link
Owner

e-alfred commented Jan 9, 2019

Just go to the releases page on Gihthub or in the Nextcloud App Store:

https://github.com/e-alfred/ocdownloader/releases/tag/1.6.0-Alpha2

@bekon16
Copy link

bekon16 commented Jan 9, 2019

@e-alfred Thank you very much! It works! I appreciate very much!
There are still issues to fix:

  • In settings i can't set download folder. There is some "loading" icon which does not disappear. I can type new destination folder but it remember previous setting.
  • For a while i was able to see additional settings for ocDownloader on admin page (cURL, Aria2 and other options), after refresh (i didn't catch exactly this moment) settings disappeared - THIS ISSUE IS OBSOLETE - i set limitations ocDownloader for groups, there was no admin group and that was a problem.

Here is some output from console related to first issue (destination folder):

index.php/settings/ajax/checksetup 403 (Forbidden)

@bekon16
Copy link

bekon16 commented Feb 19, 2019

Hi, any progress (update to version 15 and error with destination folder)?

@e-alfred
Copy link
Owner

e-alfred commented Apr 3, 2019

This is solved so far, for further issues please refer to the existing ones or create a new one if needed.

@e-alfred e-alfred closed this as completed Apr 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests