On initial create of menu links which are using menu tokens menu_token_link_uuid is not being set, and because of this on export of links with features this link option is not being exported and it breaks link path.

CommentFileSizeAuthor
#3 combine-2833077-3-2833087-2-menu_token.patch2.43 KBnmitev
#2 initial_create_of_menu_links-2833087-2-menu_token.patch1.43 KBnmitev
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nmitev created an issue. See original summary.

nmitev’s picture

nmitev CreditAttribution: nmitev commented
Status: Active » Needs review
FileSize
initial_create_of_menu_links-2833087-2-menu_token.patch1.43 KB

This patch sets link uuid on initial create of link and on export every link would have as option menu_token_link_uuid.

nmitev’s picture

nmitev CreditAttribution: nmitev commented
FileSize
combine-2833077-3-2833087-2-menu_token.patch2.43 KB

As this issue is related with #2833077: Features update problem, I've combined both patches and re-factored validate function to use $values as it is more readable.

odrzutowiec’s picture

odrzutowiec CreditAttribution: odrzutowiec commented

Hello patch from #3 worked for me. Thanks nmitev!

DevElCuy’s picture

DevElCuy CreditAttribution: DevElCuy at Dilygent commented
Status: Needs review » Reviewed & tested by the community

@odrzutowiec thanks for testing! Please change status to "Reviewed & tested by the community" whenever you test a patch and you find it working correctly.

  • develCuy committed 00784ec on 7.x-1.x authored by nmitev 
    Issue #2833087 by nmitev, odrzutowiec: Initial creation of menu links...
DevElCuy’s picture

DevElCuy CreditAttribution: DevElCuy at Dilygent commented
Status: Reviewed & tested by the community » Fixed

Thanks @nmitev!

Status: Fixed » Closed (fixed)

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