This patch makes possible to show multiple tagging forms (for different nodes) on one page.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | community_tags.patch | 1.63 KB | avivalma |
This patch makes possible to show multiple tagging forms (for different nodes) on one page.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | community_tags.patch | 1.63 KB | avivalma |
Comments
Comment #1
avivalma commentedComment #2
patchak commentedI tested this patch, and it works really well, there is a noce popup form that appears and allows to tag nodes directly from the teaser.
I would really recommend to test this and commit it!!
Patchak
Comment #3
giorgio79 commentedsubs
Comment #4
bowenzbiz commentedDoesn't work. Community Tags 6.x dev.
It shows nid undefined, in full node view.
I had changed code to
It doesn't work, too.
There is no
<input name=nid ../>element in the tag form.Comment #5
bowenzbiz commentedDoesn't work. Community Tags 6.x dev.
Comment #6
entrigan commentedAfter looking into the issue, the patch fails because the JS looks for the node id in the form, but it is not there. To make it work the form needs another element that includes the NID (right now it has an element of type="value" which does not reveal itself in the HTML. The following code placed in the function "community_tags_form()" in the file "community_tags.pages.inc" should make the patch work again:
EDIT: Changed the code to something better
Comment #7
entrigan commentedNote: this only allows for multiple ComTag forms on one page. To actually make them appear you need to add something like
in a node template file.
Comment #8
entrigan commentedI feel like this solution is a bit hacky... (passing the nid in a hidden form item and then grabbing it with jquery. The issue is the js needs a unique way to identify each form. Ideas?
Comment #9
entrigan commentedThis problem has been fixed (using the techniques described above) in the newest dev release.