Currently the uuid_path module handles the universalisation and localisation of both url aliases and menu links. I think it's a good idea for these two discreet functions to be in separate sub modules. I am personally using this module for handling menu link deployments: http://drupal.org/project/entity_menu_links. It's still in it's infancy and has some issues, but it is proving very useful. However it isn't compatible with the menu links functionality in uuid_path. I do however need the url alias deployment functionality, which is why it would be good if these two functions were not linked.

I will provide a patch which break the menu link stuff out into a separate sub module.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jamesharv’s picture

Status: Needs work » Needs review
FileSize
3.08 KB
3.59 KB

Attached are two patches. One which removes the menu link functionality from uuid_path, and another which creates the new uuid_menu submodule. I can combine them into a single patch if that's easier.

bonobo’s picture

What is the relationship (if any) between these patches and this commit: http://drupalcode.org/project/uuid.git/commit/8b6d70f ?

The commit removes support for menu items.

Status: Needs review » Needs work

The last submitted patch, 1: remove-menu-link-from-uuid-path-1795156-1.patch, failed testing.

kristiaanvandeneynde’s picture

Issue summary: View changes

@bonobo see related issue

kristiaanvandeneynde’s picture

skwashd’s picture

Status: Needs work » Fixed

I'm closing this issue. We have the Entity Menu Links module for handling menu links and paths are still handled natively by deploy.

Status: Fixed » Closed (fixed)

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