Once you turn on the "Aggregate JavaScript files" feature on your site, suggestedterms.js breaks and the JS magic no longer works. We probably need to port this to the D7 behaviors stuff:

(function ($) {
Drupal.behaviors.suggestedTerms = {
  attach: function (context) {
    ...
  }
};
})(jQuery);

I might take a stab at this in the next few days. For now, I can just disable the JS aggregation, but it'd be nice to leave that on once this site goes live. ;)

Thanks,
-Derek

CommentFileSizeAuthor
#1 2021809-1.suggestedterms-js-D7.patch4.7 KBdww

Comments

dww’s picture

Assigned: Unassigned » dww
Status: Active » Needs review
StatusFileSize
new4.7 KB

Heh, I actually need this now since I can't disable JS aggregation in the environment where it matters. :/

Luckily, I was right and this was easy to solve by wrapping the existing JS in the Drupal.behaviors stuff.

bbinkovitz’s picture

Status: Needs review » Fixed

Committed.

dww’s picture

Great, thanks! (Just got back online after being away for a while)

Status: Fixed » Closed (fixed)

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