Problem/Motivation

XML sitemap menu in xmlsitemap_menu_entity_info_alter() currently checks if the an entity definition for menu links already exists - but it doesn't check if the existing definition actually provides an XML sitemap integration.
It would be nice if we don't loose the integration just because there's a menu link entity definition. If possible we should re-use the abilities of XML sitemap menu.

Proposed resolution

Provide defaults for existing menu link entity infos to re-use XML sitemap menu.

Remaining tasks

Reviews needed.

User interface changes

None.

API changes

None.

CommentFileSizeAuthor
xmlsitemap_menu-compatibility-with-exisint-menu-link-entity.patch626 bytesdas-peter
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs review » Fixed

Committed to 7.x-2.x. Thanks!

  • Dave Reid committed 8977401 on 7.x-2.x authored by das-peter 
    Issue #2367513 by das-peter: Fixed xmlsitemap_menu_entity_info() did not...

Status: Fixed » Closed (fixed)

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