Problem/Motivation

Admin menu was recently updated with the issue #2084217: Make field_ui_admin_menu_map() actually work for all entity types. The Field UI mapping is now included by default and does not have to be added in the Reply hook_admin_menu_map().

Currently this results in a notice:

Proposed resolution

Remove the mapping of fields from hook_admin_menu_map()

Remaining tasks

  • Write a patch
  • Review

User interface changes

None

API changes

None

CommentFileSizeAuthor
#1 2407127-1-after.png119.78 KBidebr
#1 reply-admin_menu_field_mapping_notice-2407127-1.patch1.02 KBidebr
admin_menu_strtr_notice.png107.63 KBidebr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

idebr’s picture

idebr CreditAttribution: idebr commented
Status: Active » Needs review
FileSize
reply-admin_menu_field_mapping_notice-2407127-1.patch1.02 KB
2407127-1-after.png119.78 KB

Attached patch removes the duplicate Field mapping from hook_admin_menu_map().

Screenshot after:

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan commented
Status: Needs review » Fixed

Thanks

  • larowlan committed 9be24e7 on 7.x-2.x authored by idebr 
    Issue #2407127 by idebr: Remove field mapping from hook_admin_menu_map...

Status: Fixed » Closed (fixed)

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