[WIP] Move to new backend registration #7
Conversation
|
By analyzing the blame information on this pull request, we identified @icewind1991 to be a potential reviewer |
| ], | ||
| ]); | ||
| \OC::$server->getEventDispatcher()->addListener( | ||
| 'OCA\\Files_External::loadAdditionalBackends', function($event) { |
There was a problem hiding this comment.
@xenopathic ftp is never registered
There was a problem hiding this comment.
If files_external_ftp is loaded after files_external, the event dispatcher trigger will have already fired, so this listener won't be activated. You probably need to check if files_external is available yet, and if not, then go via the event dispatcher. This is a consequence of not knowing which order apps are loaded in, and not having the core bits of files_external in core.
There was a problem hiding this comment.
Well ... i know what is happening. The question is how this was supposed to ever work .
|
Ah, damn, you already did ? I did the same work here #9 and it does work, so maybe use my PR instead ? |
No description provided.