Only do typeahead query if necessary.

angular-ui/bootstrap#993

Author: leg100

app/assets/javascripts/events.js

@@ -33,7 +33,7 @@ angular.module('events', ['rails', 'ui.bootstrap', 'alerts'])
     name: 'player'
   });
 }])
-.controller('InjuryEditCtrl', ['$scope', 'Player', 'Injury', 'EventListingService', 'AlertBroker', function($scope, Player, Injury, EventListingService, AlertBroker) {
+.controller('InjuryEditCtrl', ['$scope', 'Player', 'Injury', 'EventListingService', 'AlertBroker', 'limitToFilter', '$filter', function($scope, Player, Injury, EventListingService, AlertBroker, limitToFilter, $filter) {
   // init params
   $scope.type = 'injury';
 
@@ -55,9 +55,13 @@ angular.module('events', ['rails', 'ui.bootstrap', 'alerts'])
   };
 
   // populate typeahead
-  Player.query({typeahead: true}).then(function(resp) {
-    $scope.players = resp;
-  });
+  $scope.getPlayers = function(substring) {
+    return Player.query(
+      {typeahead: true}
+    ).then(function(resp) {
+      return limitToFilter( $filter('filter')(resp, substring), 8);
+    });
+  };
 
   // trigger update
   $scope.add = function() {

app/views/partials/_event_form.html.slim

@@ -6,7 +6,7 @@
           option(value="fixture" ng-selected="type=='fixture'") Fixture
           option(value="injury" ng-selected="type=='injury'") Injury
       .form-group.myPadding
-        input.form-control(name="player" autofocus type="text" ng-model="event.selected_player" typeahead="player as player.tickerAndName for player in players | filter:$viewValue | limitTo:8" placeholder="Player" ng-required="true")
+        input.form-control(name="player" autofocus type="text" ng-model="event.selected_player" typeahead="player.tickerAndName for player in getPlayers($viewValue)" placeholder="Player" ng-required="true")
       .form-group.myPadding
         input.form-control(type="text" datepicker-popup="{{format}}" ng-model="event.returnDate" is-open="opened" min="minDate" max="'2015-06-22'" datepicker-options="dateOptions" close-text="Close" placeholder="Return date")
       .form-group.myPadding