- Merge request !7767Issue #3090668: Log errors for missing route_name in Drupal\Core\Menu\LocalTaskManager->getLocalTasksForRoute() โ (Open) created by tobiasb
- ๐ฌ๐งUnited Kingdom jonathanshaw Stroud, UK
Looks good, RTBC when phpcs is fixed.
- Issue was unassigned.
- ๐ฉ๐ชGermany tobiasb Berlin
CR created. https://www.drupal.org/node/3443775 โ
- Status changed to RTBC
10 months ago 9:34pm 26 April 2024 - Status changed to Needs work
10 months ago 10:02am 27 April 2024 - ๐ฎ๐ณIndia pradhumanjainOSL
pradhumanjain2311 โ made their first commit to this issueโs fork.
- Merge request !7912Issue #3090668: Use assert to check that route name, base name in a local task exists โ (Open) created by tobiasb
- Status changed to Needs review
10 months ago 9:37am 4 May 2024 - ๐ฉ๐ชGermany tobiasb Berlin
In my use case where I get the Notice is a problem
drush deploy
vs.drush updb/cim/cr
.And I also got than a exception when I opened a page which use the missing route.
The hook_module_implements_alter from views_local_tasks module was not called/empty in
drush deploy
which results in broken routes.So I vote for assert-only patch or closed (works as designed). :D
- Status changed to Needs work
10 months ago 3:57pm 5 May 2024 - ๐บ๐ธUnited States smustgrave
Thanks for working on this
Issue summary should follow standard template so tagging for that.
Even though a task seems like a feature that should have test coverage so tagging for that also.
Thanks
- ๐บ๐ธUnited States moshe weitzman Boston, MA
Do folks think this could cause pages to show the same tabs to one node? This is happenning on mass.gov and on several other sites. See https://www.drupal.org/forum/support/module-development-and-code-questio... โ
assert() is non-functional in Prod so I dont think that approach is as helpful.