fixed on-item-click callback bug

isteven · isteven · commit 2730c2b52a58 · 2014-07-09T15:37:30.000+08:00
diff --git a/angular-multi-select.js b/angular-multi-select.js
@@ -395,7 +395,7 @@ angular.module( 'multi-select', ['ng'] ).directive( 'multiSelect' , [ '$sce', '$
                     }
                 }                                  
 
-                $scope.clickedItem = angular.copy( item );                                    
+                $scope.clickedItem = angular.copy( item );                                                    
 
                 // We update the index here
                 prevTabIndex = $scope.tabIndex;
@@ -823,11 +823,11 @@ angular.module( 'multi-select', ['ng'] ).directive( 'multiSelect' , [ '$sce', '$
                     $scope.refreshSelectedItems();                                   
                     $scope.refreshOutputModel();
                     $scope.refreshButton();                              
-                    if ( $scope.clickedItem !== null ) {
+                    if ( $scope.clickedItem !== null ) {                        
                         $timeout( function() {
                             $scope.onItemClick( { data: $scope.clickedItem } );
-                        }, 0 );                         
-                        $scope.clickedItem = null;                    
+                            $scope.clickedItem = null;                    
+                        }, 0 );                                                 
                     }                                    
                 }
             }, true);

Original file line number	Diff line number	Diff line change
`@@ -395,7 +395,7 @@ angular.module( 'multi-select', ['ng'] ).directive( 'multiSelect' , [ '$sce', '$`
`395`	`395`	`}`
`396`	`396`	`}`
`397`	`397`
`398`		`- $scope.clickedItem = angular.copy( item );`
	`398`	`+ $scope.clickedItem = angular.copy( item );`
`399`	`399`
`400`	`400`	`// We update the index here`
`401`	`401`	`prevTabIndex = $scope.tabIndex;`
`@@ -823,11 +823,11 @@ angular.module( 'multi-select', ['ng'] ).directive( 'multiSelect' , [ '$sce', '$`
`823`	`823`	`$scope.refreshSelectedItems();`
`824`	`824`	`$scope.refreshOutputModel();`
`825`	`825`	`$scope.refreshButton();`
`826`		`- if ( $scope.clickedItem !== null ) {`
	`826`	`+ if ( $scope.clickedItem !== null ) {`
`827`	`827`	`$timeout( function() {`
`828`	`828`	`$scope.onItemClick( { data: $scope.clickedItem } );`
`829`		`- }, 0 );`
`830`		`- $scope.clickedItem = null;`
	`829`	`+ $scope.clickedItem = null;`
	`830`	`+ }, 0 );`
`831`	`831`	`}`
`832`	`832`	`}`
`833`	`833`	`}, true);`