Download & Extend

Add link url additional arguments to widget configuration parameters

Project:Node Reference URL Widget
Version:7.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs review
Issue tags:construct url

Issue Summary

Hi

As we can see there is a helper function nodereference_url_create_link(...) for themers to easily create a link. But we need also to pass additional parameters to URL, for example URL part for prepopulate module to handle other fields.

I suggest to add this option to widget configuration form (after Link title: and Link hover title: ) "Link url additional params" or as additional parameter for mentioned function. It will provide generic method to manipulate link url.

(Sorry for my bad english and maybe unclear explanation).

Comments

#1

I'd be happy to add it as an additional option to the nodereference_url_create_link() function, but I probably won't be adding more options to the already confusing UI. If you'd like more advanced links, you can use the Custom links module.

#2

Hi

Adding it to nodereference_url_create_link() function is enough.
Thanks

#3

Status:active» postponed

If you'd like this feature, please provide a patch. Otherwise I'll just close it after a lack of activity, assuming that no one is interested in this functionality.

#4

Version:6.x-1.6» 7.x-1.x-dev

I wanted this feature but got as far as setting the extra url parameter but then it breaks with '[]' charactors in url inside the nodereference_url_get_nid() function. I didn't have time to look into it further but attached is what I got so far and hopefully someone that knows this function betters can find the bug.

This patch applies on the d7 version, therefore I am upping the issue to that.

AttachmentSize
nodereference_url-2000fcc.patch 1.48 KB

#5

Status:postponed» needs review

please review and fix.

#6

UPDATE, please review this. WFM.

AttachmentSize
nodereference_url-2000fcc-1.patch 1.58 KB

#7

Hi @likewhoa: I don't think I'm interested in adding this as an option in the UI (as I said in #1). I'd be happy to add it as option to nodereference_url_create_link() though, as @hayk asked for in #2.