summaryrefslogtreecommitdiffstats
path: root/tests/lib/NavigationManagerTest.php
Commit message (Collapse)AuthorAgeFilesLines
* Make sure we test the proper order in the NavigationManagerTestJulius Härtl2018-02-121-7/+7
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Fix testsRoeland Jago Douma2017-09-211-0/+9
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Inject \OCP\IURLGenerator to make tests workLukas Reschke2017-08-181-1/+11
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Properly allow \OCP\Authentication\IApacheBackend to specify logout URLLukas Reschke2017-08-181-1/+1
| | | | | | | | | | | | | | | | | | | | Any `\OCP\Authentication\IApacheBackend` previously had to implement `getLogoutAttribute` which returns a string. This string is directly injected into the logout `<a>` tag, so returning something like `href="foo"` would result in `<a href="foo">`. This is rather error prone and also in Nextcloud 12 broken as the logout entry has been moved with 054e161eb5f4a5c5c13ee322ae8e93ce66f01b13 inside the navigation manager where one cannot simply inject attributes. Thus this feature is broken in Nextcloud 12 which effectively leads to the bug described at nextcloud/user_saml#112, people cannot logout anymore when using SAML using SLO. Basically in case of SAML you have a SLO url which redirects you to the IdP and properly logs you out there as well. Instead of monkey patching the Navigation manager I decided to instead change `\OCP\Authentication\IApacheBackend` to use `\OCP\Authentication\IApacheBackend::getLogoutUrl` instead where it can return a string with the appropriate logout URL. Since this functionality is only prominently used in the SAML plugin. Any custom app would need a small change but I'm not aware of any and there's simply no way to fix this properly otherwise. Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Don't load navigation entries of restricted appsJoas Schilling2017-07-051-12/+52
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Allow multiple navigation links from info.xmlJoas Schilling2017-03-261-37/+73
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Force to specify the nameJoas Schilling2017-01-271-4/+6
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Add navigation via info.xml (#26785)Thomas Müller2016-12-221-0/+64
| | | | | | | | | | * Read navigation information from info.xml * Load files navigation elements from info.xml * Add comment about ignoring the exception Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Move tests/ to PSR-4 (#24731)Joas Schilling2016-05-201-0/+160
* Move a-b to PSR-4 * Move c-d to PSR-4 * Move e+g to PSR-4 * Move h-l to PSR-4 * Move m-r to PSR-4 * Move s-u to PSR-4 * Move files/ to PSR-4 * Move remaining tests to PSR-4 * Remove Test\ from old autoloader
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-21 04:58:03 +0000