Updated: Comment #N

Problem/Motivation

If you try to create a menu link to a path whose menu router item contains %ctools_js, it will fail validation.
This is because it does not know what to substitute in when there are no arguments.

Proposed resolution

Similar to the ctools_js_load() function that parses the arguments, we need a ctools_js_to_arg(), that runs when there are no valid arguments.

Remaining tasks

N/A

User interface changes

N/A

API changes

N/A

CommentFileSizeAuthor
#1 ctools-2207881-1.patch488 bytestim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Active » Needs review
FileSize
ctools-2207881-1.patch488 bytes
dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Looks pretty cool.

japerry’s picture

japerry
Location KVUO
CreditAttribution: japerry commented
Status: Reviewed & tested by the community » Fixed

Nice, Fixed.

  • japerry committed bdf506d on 7.x-1.x authored by tim.plunkett 
    Issue #2207881 by tim.plunkett: Allow menu items with %ctools_js to work...

Status: Fixed » Closed (fixed)

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