Ambiguous Coffeescript line becomes filthy poopy JavaScript

[AndrewRayCode]

https://github.com/harvesthq/chosen/blob/master/coffee/chosen.jquery.coffee#L316

if( @active_field and not($(evt.target).hasClass "search-choice" or $(evt.target).parents('.search-choice').first) and not 

https://github.com/harvesthq/chosen/blob/master/chosen/chosen.jquery.js#L381

if (this.active_field && !($(evt.target).hasClass("search-choice" || $(evt.target).parents('.search-choice').first)) && !this.results_showing) {

Obviously not what you intended. first isn't called as a method, hasClass is checking for more than it should be. It's kind of scary this has been in the chosen source all along, because who knows what kind of behavior this is triggering.

[svieira]

Since "search-choice" is truthy || $(evt.target).parents('.search-choice').first never gets evaluated, thus avoiding runtime erorrs. However, it's probably not what you wanted to do -- looks like updated .hasClass "search-choice" to .hasClass("search-choice") should fix things.

[kenearley]

This has been cleaned up in PR #1163 and is merge to master.