#2178725: Make all core menu link machine names and the corresponding route names match change a bunch of menu items, which breaks the ones in devel.module, making it impossible to enable it.

CommentFileSizeAuthor
#3 0001-Issue-2224515-by-fgm-make-devel-installable-again-af.patch2.52 KBfgm
#1 0001-Issue-2224515-by-fgm-make-devel-installable-again-af.patch1.68 KBfgm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm commented
Status: Active » Needs review
FileSize
0001-Issue-2224515-by-fgm-make-devel-installable-again-af.patch1.68 KB

Suggested patch. This is needed because #2178725: Make all core menu link machine names and the corresponding route names match changed the menu name for system.admin_config.development to system.admin_config_development.

Status: Needs review » Needs work

The last submitted patch, 1: 0001-Issue-2224515-by-fgm-make-devel-installable-again-af.patch, failed testing.

fgm’s picture

fgm
Primary language French
Location Paris, France
CreditAttribution: fgm commented
Status: Needs work » Needs review
FileSize
0001-Issue-2224515-by-fgm-make-devel-installable-again-af.patch2.52 KB

The failing test is entirely unrelated: it was broken previously too. But I addressed it too in this version of the patch.

pcambra’s picture

pcambra
he/him
Primary language Spanish
Location Asturies
CreditAttribution: pcambra commented
Status: Needs review » Fixed

Yeah I was fixing that in the meanwhile, pushed to 8.x, thanks! :)

  • Commit f399980 on 8.x-1.x authored by fgm, committed by pcambra: 
    Issue #2224515 by fgm: Devel no longer installs after #2178725.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.