If you enter a comma in the field you get an error message, for example "Test, Test", you get a popup with a 500 message, as attached.

Included patch will suppress that message.

CommentFileSizeAuthor
#3 term_ref_autocomplete.2580235.3.patch525 bytestwod
term_ref_autocomplete_fix_500_error.patch3.49 KBRustan
comma-bug.png20.06 KBRustan

Comments

Rustan created an issue. See original summary.

Rustan’s picture

Rustan commented
twod’s picture

twod
Primary language Swedish
Location Sweden
commented
StatusFileSize
new term_ref_autocomplete.2580235.3.patch525 bytes

The patch was not created from the module root folder so it does not apply cleanly.

Most of it is whitespace changes, the only relevant new line is if (strrchr($raw_tags_typed, ',') == FALSE || !empty($tids)) {.

+++ b/term_ref_autocomplete/term_ref_autocomplete.pages.inc
@@ -38,41 +38,44 @@ function term_ref_autocomplete_callback($field_name, $raw_tags_typed = '') {
+      if (!empty($tags_typed)) {
+        $query->condition('t.tid', $tids, 'NOT IN');

This should check $tids as $tags_typed may be longer, if what was entered did not match a term, which in turn leads to an SQL syntax error with an empty t.tid NOT IN () condition. You likely get a 500-error because DB errors are suppressed by disabled logging.

I have that issue with the Alpha 3 release and this change alone fixes it.