Great widget; thanks for this. I've run into a problem, however, and I think it's likely because the titles of the nodes I'm referencing contain commas -- person names in "Surname, Given name" order, in this case.

Here's a representative sequence (names changed to protect the innocent):

Doe
-->"Doe, John" [nid:1]
"Doe, John" [nid:27],jane
-->"Doe, John", "Doe, Jane" [nid:2]

Selecting the second node reference deletes the actual reference tag from the first reference.

Doesn't appear to happen unless the first referenced node's title contains a comma.

CommentFileSizeAuthor
#2 nr_autocomplete.module.patch1.38 KBjcarnett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lyngve’s picture

lyngve CreditAttribution: lyngve commented

Same problem here. Is there any fix?

jcarnett’s picture

jcarnett CreditAttribution: jcarnett commented
Status: Active » Needs review
FileSize
nr_autocomplete.module.patch1.38 KB

Here's a patch that ought to do the trick. Let me know if you see any issues.

Vuds’s picture

Vuds CreditAttribution: Vuds commented
Status: Needs review » Reviewed & tested by the community

The patch works very well!

Thank you very much!

Josh Benner’s picture

Josh Benner CreditAttribution: Josh Benner commented
Assigned: Unassigned » Josh Benner
Status: Reviewed & tested by the community » Fixed

The patch has been applied and a new release will be rolled momentarily.

Status: Fixed » Closed (fixed)

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