This patch makes possible to make available to Drupal.settings object the full purl object, in order to make purl redirect possible from javascript.

CommentFileSizeAuthor
#10 javascript_support-1286390-10.patch661 bytesmuschpusch
#5 purljs-1286390.patch883 bytespaolomainardi
#1 purl_608916_20100917.patch11.19 KBpaolomainardi

Comments

paolomainardi’s picture

paolomainardi commented
StatusFileSize
new purl_608916_20100917.patch11.19 KB
paolomainardi’s picture

paolomainardi commented

This patch must be applied to purl 6.x-1.0-beta13, i can't change it anymore on the issue node.

paolomainardi’s picture

paolomainardi commented

Up

techninja’s picture

techninja commented
Status: Needs review » Needs work

Dude... That is totally the wrong patch file for that, looks like it belongs to #608916: Allow global purl disable by url or context (or other factors possibly). I was looking for this too, and would be willing send in a patch, though you've probably already got it going, just need to upload the correct one.

paolomainardi’s picture

paolomainardi commented
StatusFileSize
new purljs-1286390.patch883 bytes

In the attachment the right patch, sorry for the mistaken :)

techninja’s picture

techninja commented
Status: Needs work » Reviewed & tested by the community

Yep, that's more like it! Works for me, sweet and simple, thank you.

Though I don't normally call myself a "community", I would say this has been tested, and is really only a no-interference 2 line addition, and incredibly useful to us JavaScript URI builders. Lets get it moving in!

tobby’s picture

tobby commented

Committed in http://drupalcode.org/project/purl.git/commit/23d4623. Thanks for the patch. :)

tobby’s picture

tobby commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

muschpusch’s picture

muschpusch commented
Version: 6.x-1.0-beta9 » 7.x-1.x-dev
Assigned: Unassigned » muschpusch
Status: Closed (fixed) » Needs review
StatusFileSize
new javascript_support-1286390-10.patch661 bytes

i changed the patch because only the provider was available and not the whole object with ID.