getMenuOptions handles group content menu and menu both. No need for that, parent handles system menus already. This makes the code much easier to read IMO.

CommentFileSizeAuthor
#2 3313950_2.patch2.28 KBghost of drupal past

Comments

Charlie ChX Negyesi created an issue. See original summary.

ghost of drupal past’s picture

ghost of drupal past
he/him
commented
StatusFileSize
new 3313950_2.patch2.28 KB
heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs review » Reviewed & tested by the community

  • chx committed 20cb949 on 8.x-1.x 
    #3313950 Refactor getMenuOptions
ghost of drupal past’s picture

ghost of drupal past
he/him
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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