If you export menu items with paths such as:
node/add
node/whatever

the module replaces it's path with <front>

Comments

justindodge’s picture

Status: Active » Needs review
StatusFileSize
new1.09 KB

Here's a patch that helps. The problem is the way the old code tried to interpret a node path - by just looking for the first 4 characters of the path to be 'node'.

This patch checks that the first arg is 'node', the 2nd is numeric, and there is no 3rd.

Note that this doesn't really handle something like node/4/edit - If that node was missing during import, it wouldn't really make sense to create a place holder node and link to the view page since the menu item was for an edit link (this is the current behavior). On the other hand, that might be a better solution than just letting it be broken.

The ideal solution would be to create the node if it doesn't exist and then preserve all existing components of the path EXCEPT the node id. Unfortunately I don't have time to put that much work in, though it shouldn't be too hard.

lex0r’s picture

Status: Needs review » Closed (fixed)

Fixed in 1.2-rc1

lex0r’s picture

Issue summary: View changes

Escape markup