modules/menu_ui/lib/Drupal/menu_ui/MenuListBuilder.php.orig

Should not be in the repo. This is left over from someone doing a patch apply that had some "fuzz", and should not have been committed.

And there's no component for the new Menu UI module??

CommentFileSizeAuthor
cleanup.patch193 bytesjhodgdon
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented

RTBC++

Thanks!!

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Reviewed & tested by the community » Fixed

Committed my own patch.

  • Commit c845f0d on 8.x by jhodgdon: 
    Issue #2243543 by jhodgdon: Remove stray file
dcrocks’s picture

dcrocks CreditAttribution: dcrocks commented

#2120871: Rename EntityListController to EntityListBuilder renamed MenuListController.php to MenuListBuilder.php. #2085243: Rename Menu module into Menu UI module picked up that file in its patch. So this file is no longer needed?

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented

#4 sounds like a separate issue, unrelated to this issue, which was about a ".orig" file that was inadvertently committed to the repo. Please file a separate issue.

dcrocks’s picture

dcrocks CreditAttribution: dcrocks commented

Sorry, misunderstood what was being deleted.

Status: Fixed » Closed (fixed)

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