In oauth.routing.yml file, some route's path and class namespace are not start with '/' and '\'.
As per drupal standard, path defined in routing.yml should start with '/' and namespace should start with '\'.
Example :
path: 'user/{user}/oauth/consumer'
should be - path: '/user/{user}/oauth/consumer'
and _content: 'Drupal\oauth\Controller\OAuthController::consumers'
should be _content: '\Drupal\oauth\Controller\OAuthController::consumers'.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 2518476-routes_slash-3.patch | 1.32 KB | fgm |
| #2 | 2518476-route-name-namespace-wrong-2.patch | 2.71 KB | blackice2999 |
| #1 | 2518476-1.patch | 1.28 KB | joshi.rohit100 |
Comments
Comment #1
joshi.rohit100Comment #2
blackice2999 commentedHi,
based upon #2377967: Remove bc layer for _content _controller change it should be also named _controller instead of _content...
Comment #3
fgmRerolled on today's core HEAD. But this is not testable until #2331155: Unable to install, buildForm in Form classes incompatible with FormStateInterface::buildForm gets in.
Comment #5
juampynr commentedCommited. Thanks!
I have fixed part of what @fgm mentions at #2331155: Unable to install, buildForm in Form classes incompatible with FormStateInterface::buildForm but I will double check with the patch there to verify that I have made all the changes.