summaryrefslogtreecommitdiffstats
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Use the proper IAppContainer and IServerContainer type hints to know which ↵Morris Jobke2020-07-213-11/+9
| | | | | | code runs with which container Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Make the bootstrap context return ContainerInterface instancesChristoph Wurst2020-07-214-15/+16
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #21875 from nextcloud/techdebt/deprecate-iloggerMorris Jobke2020-07-202-0/+52
|\ | | | | Deprecate ILogger in favor of the PSR-3 logger
| * Deprecate ILoggerChristoph Wurst2020-07-162-0/+52
| | | | | | | | | | | | | | | | | | | | | | | | The logger service was always intended to follow the PSR-3 interface. It's time to embrace this and switch over to the "official" API, hence this custom interface can be slowly phased out. With Nextcloud 20 the logger also got support for * App id filled out automatically * Exceptions handling (as replacement for logException) Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Merge pull request #21850 from ↵Morris Jobke2020-07-206-2/+145
|\ \ | | | | | | | | | | | | nextcloud/techdebt/noid/register-alternative-logins Allow to register AlternativeLogin on RegistrationContext
| * | Allow to register AlternativeLogin on RegistrationContextJoas Schilling2020-07-166-2/+145
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Merge pull request #21880 from nextcloud/webauthn/discourage_user_authMorris Jobke2020-07-201-2/+7
|\ \ \ | | | | | | | | Discourage webauthn user interaction
| * | | Discourage webauthn user interactionRoeland Jago Douma2020-07-161-2/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Else people might have the feeling this is also doing 2FA. And since it is only prefered it can be ignored and hacked around. Once we have proper 2FA with webauthn in one go this probably needs to be revisted. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | Merge pull request #21891 from ↵Morris Jobke2020-07-192-19/+20
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/techdebt/undeprecate-iappcontainer-iservercontainer Undeprecate IAppContainer and IServerContainer
| * | | | Undeprecate IAppContainer and IServerContainerChristoph Wurst2020-07-172-19/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With 4152216bd8cf9d49e6749d26bb8b491dd49b089b these two interfaces got deprecated with the reasoning that we only need the base PSR interface. However, there are cases where in Nextcloud you still want to have a specific container (the one for the app vs the one for the server) when you either have a container injected or query one from a container. With a single interface that would not be possible. So it's probably better if we leave the two interfaces, but only have them extend the PSR interface. IContainer – with the custom methods – shall still be phased out, but the two other sub interfaces can stay for tagging purposes. Tagging means that no methods shall be added. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | | [tx-robot] updated from transifexNextcloud bot2020-07-192-4/+4
| | | | |
* | | | | Deprecate the query mothod on the server containerChristoph Wurst2020-07-171-0/+3
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The interface method has already been deprecated, but if some code uses the concrete type instead, the deprecation is not shown (by phpstorm), so I think it's better to have this method tagged as well. The "fix" for this deprecation is to simply use `get` instead of `query`. Right now this will work 100% the same, but the goal is to slim down the interface and only use what PSR-11 offers. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | [tx-robot] updated from transifexNextcloud bot2020-07-174-2/+20
| | | |
* | | | Merge pull request #21869 from ↵Joas Schilling2020-07-163-11/+44
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/techdebt/noid/add-constants-for-magic-strings Add constants for the magic strings of template rendering
| * | | | Add constants for the magic strings of template renderingJoas Schilling2020-07-163-11/+44
| | |_|/ | |/| | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | Merge pull request #21873 from ↵Roeland Jago Douma2020-07-162-0/+156
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/techdebt/deprecate-server-container-getters Deprecate the server container getters
| * | | | Deprecate the server container gettersChristoph Wurst2020-07-162-0/+156
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Service locators are an anti pattern. These getters just make it more appealing to do the wrong thing. If you want to locate a service the bad way, just use the `get` method on a container – it will do the same in also one line of code. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* / / / Add a scoped PSR logger for appsChristoph Wurst2020-07-164-1/+164
|/ / / | | | | | | | | | | | | | | | | | | | | | Just like for ILogger we should have a version that has the app ID pre-set for the context (unless overwritten) so that each log entry can be traced back to the app that produced it. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | / Use PSR container interface and deprecate our own abstractionChristoph Wurst2020-07-169-140/+244
| |/ |/| | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | [tx-robot] updated from transifexNextcloud bot2020-07-162-2/+4
| |
* | Better event description for BeforeTemplateRenderedEvent in files and ↵Morris Jobke2020-07-151-2/+2
| | | | | | | | | | | | files_sharing Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #21822 from nextcloud/enh/events/additionalscriptsMorris Jobke2020-07-155-5/+79
|\ \ | | | | | | Add real events to load additionalscripts
| * | Add real events to load additionalscriptsRoeland Jago Douma2020-07-155-5/+79
| |/ | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #21844 from ↵Morris Jobke2020-07-151-6/+9
|\ \ | | | | | | | | | | | | bladewing/bladewing-harden-against-mimetype-mismatch Avoid substr() error when strpos returns false
| * | Extra white space to follow other ifslui87kw2020-07-151-1/+1
| | | | | | | | | | | | Signed-off-by: lui87kw <lukas.ifflaender@uni-wuerzburg.de>
| * | Avoid substr() error when strpos returns falsebladewing2020-07-151-6/+9
| | | | | | | | | | | | | | | | | | | | | | | | "Exception: substr() expects parameter 3 to be int, bool given" can occur on Line 378 $mimePart = substr($icon, 0, strpos($icon, '-')); This happens, when '-' is not found and strpos returns false instead of an int. When this occurs, e.g., Activity hangs. Signed-off-by: lui87kw <lukas.ifflaender@uni-wuerzburg.de>
* | | Deprecate the internal Symfony dispatcher adapterChristoph Wurst2020-07-152-0/+17
| |/ |/| | | | | | | | | | | This type is not known on the public API, yet we should make sure that any usage of this class is noticed and migrated as soon as possible. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Dashboard strict typingJulius Härtl2020-07-154-0/+12
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Add load method for apps to bootstrap their panelsJulius Härtl2020-07-152-7/+24
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Cleanup public api methodsJulius Härtl2020-07-154-22/+8
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Deprecate panel register event right away but keep it for not breaking ↵Julius Härtl2020-07-151-0/+6
| | | | | | | | | | | | backward compatible apps Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Move to lazy panel registration during registration contextJulius Härtl2020-07-155-0/+112
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Update autoloaderJulius Härtl2020-07-152-2/+4
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Deprecate old dashboard APIJulius Härtl2020-07-159-0/+79
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Add new dashboard public APIJulius Härtl2020-07-157-0/+218
|/ | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Merge pull request #21825 from nextcloud/fix/appframework/servicesMorris Jobke2020-07-141-2/+14
|\ | | | | Fix AppFramework services
| * Fix AppFramework servicesRoeland Jago Douma2020-07-141-2/+14
| | | | | | | | | | | | | | | | | | * We can't just register an alias as the services need the appId to be injected. if we just register an alias this blows up since the main container doesn't have the appId. * Moved the Authtokens over to show the PoC works Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #21812 from ↵Morris Jobke2020-07-142-5/+18
|\ \ | | | | | | | | | | | | nextcloud/feature/noid/app-bootstrap-without-app.php Do not load app.php if Application implements IBootstrap
| * | Do not load app.php if Application implements IBootstrapMorris Jobke2020-07-142-5/+18
| |/ | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #21833 from nextcloud/docfix/noid/irequest_getHeaderRoeland Jago Douma2020-07-141-1/+1
|\ \ | | | | | | Fix PHPDoc of IRequest::getHeader
| * | Fix PHPDoc of IRequest::getHeaderGeorg Ehrke2020-07-141-1/+1
| |/ | | | | | | Signed-off-by: Georg Ehrke <developer@georgehrke.com>
* | Merge pull request #21828 from ↵Morris Jobke2020-07-143-52/+38
|\ \ | | | | | | | | | | | | nextcloud/enhancement/lazy-bootstrap-registration-delegation Delegate bootstrap registration lazily
| * | Delegate bootstrap registration lazilyChristoph Wurst2020-07-143-52/+38
| |/ | | | | | | | | | | | | | | | | | | * Keep the registration context * Expose the context object for other components * Ensure registration is only run once Search providers are migrated for demonstration. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* / Make the PSR-3 adapter exception-awareChristoph Wurst2020-07-141-9/+106
|/ | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #21785 from ↵Morris Jobke2020-07-133-3/+3
|\ | | | | | | | | nextcloud/techdebt/noid/remove-classmap-in-files_external Move OC_Mount_Config to proper classname and remove OC::$CLASSPATH us…
| * Move OC_Mount_Config to proper classname and remove OC::$CLASSPATH usage in ↵Morris Jobke2020-07-103-3/+3
| | | | | | | | | | | | files_external Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #21789 from nextcloud/ftp-filter-hashMorris Jobke2020-07-131-1/+4
|\ \ | | | | | | filter files containing a hash in the path for ftp storages
| * | filter files containing a hash in the path for ftp storagesRobin Appelman2020-07-101-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | the php ftp streamwrapper doesn't handle hashes correctly and will break when it tries to enter a path containing a hash. By filtering out paths containing a hash we can at least stop the external storage from breaking completely Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | [tx-robot] updated from transifexNextcloud bot2020-07-10102-102/+102
| |/ |/|
* | Merge pull request #21759 from nextcloud/enh/lazy_subscriptionRoeland Jago Douma2020-07-092-10/+52
|\ \ | |/ |/| Make the subscription registry lazy