Looks like #1795992: No results causes Undefined variable: show_synonyms error wasn't 100% baked.

Just needs an additional check that the value is TRUE after we verify that the variable exists to check in the first place.

CommentFileSizeAuthor
google_appliance-synonyms_sanity_check-0-0.patch538 bytesiamEAP
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mpgeek’s picture

mpgeek CreditAttribution: mpgeek commented
Status: Needs review » Fixed

Nice catch. Patch applys cleanly. Committed to dev: http://drupalcode.org/project/google_appliance.git/commit/a2af6e.

mpgeek’s picture

mpgeek CreditAttribution: mpgeek commented
Status: Fixed » Reviewed & tested by the community
redndahead’s picture

redndahead CreditAttribution: redndahead commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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