Follow up for #2004428-16: Less ugly operations altering
Updated: Comment #0
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.
?
No.
No.
| 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 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: commerce_recurring_shipping 8.x-1.0-alpha1, 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.