Closed (fixed)
Project:
To do list
Version:
6.x-1.1-rc4
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Aug 2009 at 09:54 UTC
Updated:
2 Feb 2010 at 02:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
jonhattanI've seen jstools exposes clean_urls to javascript and provides a function called Drupal.url to generate correct urls for any scenario.
Attached is a patch with a fix for this issue. What it does is:
* introduce in to_do.info a dependency for jstools
* call to jstools_add_js instead of drupal_add_js in to_do.module (note: the other call with an array raised a warning and I decided to keep it with drupal_add_js)
* all urls generation in to_do_block.js are now done through Drupal.url
Comment #2
jonhattanissue status changed
Comment #3
jaypanJonhattan - thank you for the patch. However I would prefer not to have this module have another dependency another module (I like to keep the dependencies to a minimum), so I just went in and fixed the code so that it deals with clean URLs.
This will be out in my next release (due today or tomorrow).
Comment #4
jaypan