Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.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 CreditAttribution: 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 CreditAttribution: juampynr at Lullabot 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.