Hi,

Thanks for this module, it rocks!

By the way, I found minor bug in HTML: in autocomplete widget 'div' tag is not closed correct. See patch attached.

Best regards,
Spleshka.

CommentFileSizeAuthor
#1 rules-broken-html-for-autocomplete-widget-0.patch533 bytesSpleshka
error-base-table-or-view-not-found-0.patch830 bytesSpleshka
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Spleshka’s picture

Spleshka
he/him
Primary language English
Location UAE
CreditAttribution: Spleshka commented
FileSize
rules-broken-html-for-autocomplete-widget-0.patch533 bytes

Ops, wrong patch, sorry. Attached a correct one.

Spleshka’s picture

Spleshka
he/him
Primary language English
Location UAE
CreditAttribution: Spleshka commented

Any updates here? Bug seems to be obvious to me.

Spleshka’s picture

Spleshka
he/him
Primary language English
Location UAE
CreditAttribution: Spleshka commented

Still nothing? Come on, guys...

hosef’s picture

hosef CreditAttribution: hosef commented

This patch appears to fix the problem and doesn't cause any new ones. So, I think it looks good. :)

hosef’s picture

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

Forgot to change the status.

Spleshka’s picture

Spleshka
he/him
Primary language English
Location UAE
CreditAttribution: Spleshka commented

Thanks @hosef. Will wait for maintainers.

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed.

Status: Fixed » Closed (fixed)

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