- ๐ฎ๐ณIndia anup.singh
anup.singh โ made their first commit to this issueโs fork.
- last update
over 1 year ago 32 pass - last update
over 1 year ago 32 pass - First commit to issue fork.
- last update
9 months ago 32 pass - Status changed to Needs work
9 months ago 6:05pm 6 March 2024 - ๐จ๐ฆCanada adriancid Montreal, Canada
I tried to merge but the pipeline failed
- Status changed to Needs review
9 months ago 6:40am 7 March 2024 - ๐ฌ๐งUnited Kingdom scott_euser
Thanks for trying @adriancid (and thanks for all your work getting the other issues in these last days!). This one is ready again, no more phpcs failure. Phpstan failures are unrelated and exist on 3.x branch as well so we can leave those to ๐ Phpstan + Cspell + Drupal 11 compatibility fixes RTBC to prep for D11.
- Status changed to RTBC
4 months ago 1:03pm 6 August 2024 - ๐บ๐ธUnited States mlncn Minneapolis, MN, USA
Wow! Code looks good, and works in limited testing.
- First commit to issue fork.
- ๐ฌ๐งUnited Kingdom andreastkdf
I can also confirm that this works great; I managed to see the Translation overview and get to the translation form! Thank you! BUT when on the actual translation form, an error is now showing:
Drupal\form_mode_manager_theme_switcher\Theme\FormModeThemeNegociator->isApplicable() (line 93 of modules/contrib/form_mode_manager/modules/form_mode_theme_switcher/src/Theme/FormModeThemeNegociator.php).
We need a check on $route->getDefault('_entity_form') inside the str_replace in case the route default is not set (for example, when adding a translation) and $route->getDefault('_entity_form') returns NULL which causes the deprecation error above.
Just pushed a commit to the MR containing a check that fixes the deprecated error.
- ๐ฌ๐งUnited Kingdom andreastkdf
Full error (sorry missed the first important bit of it on my previous comment):
Deprecated function: str_replace(): Passing null to parameter #3 ($subject) of type array|string is deprecated in Drupal\form_mode_manager_theme_switcher\Theme\FormModeThemeNegociator->isApplicable() (line 93 of modules/contrib/form_mode_manager/modules/form_mode_theme_switcher/src/Theme/FormModeThemeNegociator.php).
- ๐ฌ๐งUnited Kingdom andreastkdf
Changed back to targetting 8.x-2.x-dev as the tests failed (when using 3.x.dev they were ruining on D11, and the module is not compatible)
some one with access should re-trigger the test manually
- ๐ฌ๐งUnited Kingdom scott_euser
scott_euser โ changed the visibility of the branch 3.x to hidden.