aboutsummaryrefslogtreecommitdiffstats
path: root/tests/lib/NavigationManagerTest.php
Commit message (Collapse)AuthorAgeFilesLines
* Let apps toggle an unread counter on app iconsJulius Härtl2021-06-161-10/+20
| | | | | Co-authored-by: Louis Chemineau <louis@chmn.me> Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Format code to a single space around binary operatorsChristoph Wurst2020-10-051-84/+84
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Change PHPDoc type hint from PHPUnit_Framework_MockObject_MockObject to ↵Morris Jobke2020-08-121-6/+6
| | | | | | \PHPUnit\Framework\MockObject\MockObject Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Add user-status appGeorg Ehrke2020-07-311-2/+2
| | | | Signed-off-by: Georg Ehrke <developer@georgehrke.com>
* Format control structures, classes, methods and functionChristoph Wurst2020-04-101-1/+0
| | | | | | | | | | | | | | | To continue this formatting madness, here's a tiny patch that adds unified formatting for control structures like if and loops as well as classes, their methods and anonymous functions. This basically forces the constructs to start on the same line. This is not exactly what PSR2 wants, but I think we can have a few exceptions with "our" style. The starting of braces on the same line is pracrically standard for our code. This also removes and empty lines from method/function bodies at the beginning and end. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Fix (array) indent style to always use one tabChristoph Wurst2020-04-091-3/+3
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Make phpunit8 compatibleRoeland Jago Douma2019-11-271-1/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Some php-cs fixesRoeland Jago Douma2019-11-221-4/+4
| | | | | | | | | | | * Order the imports * No leading slash on imports * Empty line before namespace * One line per import * Empty after imports * Emmpty line at bottom of file Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Use non-absolute logout URL to fix wrong URL in reverse proxy scenarioMorris Jobke2019-05-081-6/+4
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Parse multiple navigation itemsDaniel Kesselberg2019-02-221-6/+14
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* Urlencode the requesttokenRoeland Jago Douma2019-01-231-1/+1
| | | | | | Followup of #13757 Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Request plain logout urlRoeland Jago Douma2019-01-231-10/+5
| | | | | | | By requesting the plain logout url we allow it to be properly cached by the caching router. We just add the requesttoken manually. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Fixed phpunitJohn Molakvoæ (skjnldsv)2018-07-131-134/+181
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* 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-02 21:22:31 +0000