- Issue created by @alexharries
openid_connect_windows_aad/src/Controller/WindowsAadSSOController.php:logout() is never called, because the doc comment in openid_connect_windows_aad/src/Routing/WindowsAadSSORouteSubscriber.php is incorrectly written.
Currently it says:
/**
* Listens to the dynamic route events.
*/
class WindowsAadSSORouteSubscriber extends RouteSubscriberBase {
... while it needs to say the following in order to be picked up by Symfony:
/**
* Class WindowsAadSSORouteSubscriber.
*
* @package Drupal\openid_connect_windows_aad\Routing
*/
class WindowsAadSSORouteSubscriber extends RouteSubscriberBase {
Set a breakpoint at line 18 of src/Routing/WindowsAadSSORouteSubscriber.php:
if ($route = $collection->get('user.logout')) {
... noting that the line is never called.
Patch attached which also fixes the incorrect use of an unknown Exception.
Needs review
1.0
Code