tour_test.module accidentally redeclares hook_menu_link_defaults() instead of tour_test_menu_link_defaults(). The duplication confuses the API site. Trivial fix. Patch to follow.

CommentFileSizeAuthor
#1 drupal.duplicate-hook_menu_link_defaults.2219891-1.patch552 bytesTravisCarden
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

TravisCarden’s picture

TravisCarden CreditAttribution: TravisCarden commented
Status: Active » Needs review
FileSize
drupal.duplicate-hook_menu_link_defaults.2219891-1.patch552 bytes
bryanbraun’s picture

bryanbraun CreditAttribution: bryanbraun commented
Status: Needs review » Reviewed & tested by the community

Good catch. I just reviewed and tested the fix... even re-ran the tour_tests, and all passed as expected.

Looks good. Marking as RTBC.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Wow, great catch.

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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