autocomplete() should trigger autocomplete event even when text already matches

[RickMohr]

Currently, autocomplete() doesn't trigger the autocomplete event if the input text matches the given $selectable. Why should it, if autocompletion won't change the text?

Because there may be other data attached to the typeahead choice which is extracted by downstream logic listening for the event.

Instead of

      isValid = data && query !== data.val;
      if (isValid && !this.eventBus.before('autocomplete', data.obj, data.dataset)) {

it should be just

      if (data && !this.eventBus.before('autocomplete', data.obj, data.dataset)) {

[jlbooker]

Hi @RickMohr, Would you mind opening a pull request for this so we can give proper credit to you and do our usual test/review process?

[bryanrasmussen]

I guess I would want this as a configurable option so as to not break past solutions, although if it did this my current codebased could be simplified since I have to do a bunch of hopping through hoops to get the effect wanted without touching the typeahead source.

[jcrben]

@bryanrasmussen since there is a PR, seems best to focus the conversation there... please review the proposed code changes