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).
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | nodereference_url-2000fcc-1.patch | 1.58 KB | likewhoa |
| #4 | nodereference_url-2000fcc.patch | 1.48 KB | likewhoa |
Comments
Comment #1
quicksketchI'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.
Comment #2
hayk commentedHi
Adding it to nodereference_url_create_link() function is enough.
Thanks
Comment #3
quicksketchIf 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.
Comment #4
likewhoa commentedI 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.
Comment #5
likewhoa commentedplease review and fix.
Comment #6
likewhoa commentedUPDATE, please review this. WFM.
Comment #7
quicksketchHi @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.