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.Follow up for #2004428-16: Less ugly operations altering
Updated: Comment #0
Problem/Motivation
the roles (and user) page is broken with:
LogicException: Unable to parse the controller name "user_admin". in Drupal\Core\Controller\ControllerResolver->createController() (line 91 of /Users/ctheys/foo/drupal/core/lib/Drupal/Core/Controller/ControllerResolver.php).
The website has encountered an error. Please try again later.
It is ok in drupal 8.x, but in config_translation, it's not.
Proposed resolution
?
Remaining tasks
- confirm
- discuss cause
User interface changes
No.
API changes
No.
Related Issues
- mabye similar to #2004690: taxonomy page broken
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | config_translation-2019831-6.patch | 1.52 KB | Gábor Hojtsy |
| #2 | role_translation.png | 12.62 KB | dawehner |
| #2 | config_translation-2019831-2.patch | 1.51 KB | dawehner |
Comments
Comment #1
YesCT CreditAttribution: YesCT commentedI thought this #2010290: Editing a config entity from a listing page results in a 'page not found'
might help.
But it did not seem to have an effect.
Comment #2
dawehnerThis worked for me, see screenshot.
Comment #3
Gábor Hojtsy#2: config_translation-2019831-2.patch queued for re-testing.
Comment #5
Gábor HojtsyI sent this for a test re-run to ensure it still passes with all the added new tests. Also asked @kfritsche to add test coverage to continue #2004710: Add tests for block, menu, vocabulary and views listings with adding tests here for user roles.
Comment #6
Gábor HojtsyIt only applied with a fuzz now.
Comment #7
Gábor HojtsyCommitted this one with the test coverage devised in [#3037587], yay! Hiperyay! The commit message used was:
Issue #2027587 by kfritsche, David Hernández, dawehner, YesCT: Add tests for custom blocks, contact forms, formats, shortcut listings and settings pages.; fix for user roles from #2019831 included, otherwise tests would not pass; yay
That solved needing tests.