The default leangth of the allowed title and path for a menu item is shorter than Drupal core allows. This means that when a link is created in the default Drupal menu link editor it might be too long for the menu editor's power edit page and thus cause an error when using the power edit page.

CommentFileSizeAuthor
#4 wrong_default_link_path-2211719-1.patch1.71 KBfrob
#1 Screen Shot 2014-03-04 at Mar 4, 2014|10.29.20 PM.png23.19 KBfrob
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

frob’s picture

frob
Primary language English
CreditAttribution: frob commented
Assigned: frob » Unassigned
FileSize
Screen Shot 2014-03-04 at Mar 4, 2014|10.29.20 PM.png23.19 KB

The patch also fixes some whitespace issues.

frob’s picture

frob
Primary language English
CreditAttribution: frob commented
Status: Active » Needs review
donquixote’s picture

donquixote CreditAttribution: donquixote commented

That's not really a patch, is it :)

frob’s picture

frob
Primary language English
CreditAttribution: frob commented
FileSize
wrong_default_link_path-2211719-1.patch1.71 KB

hehe, oops

samir_mankar’s picture

samir_mankar CreditAttribution: samir_mankar commented
Status: Needs review » Reviewed & tested by the community
mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman commented

+1 on this

donquixote’s picture

donquixote CreditAttribution: donquixote as a volunteer commented

Committed on github.

See https://github.com/donquixote/drupal-menu_editor/commits/7.x-1.x

Look for: 7ed3cdc71feebe347f3077b3c495bf2f96fe5dce

  • donquixote committed 7ed3cdc on 7.x-1.x authored by frob 
    Patch #2211719-4 by frob: link_title and link_path do not allow for...
donquixote’s picture

donquixote CreditAttribution: donquixote as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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