fix(directive): ng:options to support ng:change

Closes #463

3 files changed, 27 insertions, 1 deletions

diff --git a/CHANGELOG.md b/CHANGELOG.md
index c3b9251b..2ef1b35d 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -5,6 +5,7 @@
 - Issue #449: [ng:options] should support binding to a property of an item.
 - Issue #464: [ng:options] incorrectly re-grew options on datasource change
 - Issue #448: [ng:options] should support iterating over objects
+- Issue #463: [ng:options] should support firing ng:change event
 
 ### Breaking changes
 - no longer support MMMMM in filter.date as we need to follow UNICODE LOCALE DATA formats.
diff --git a/src/widgets.js b/src/widgets.js
index bdd68804..17a14741 100644
--- a/src/widgets.js
+++ b/src/widgets.js
@@ -674,6 +674,7 @@ angularWidget('select', function(element){
   this.directives(true);
   var isMultiselect = element.attr('multiple');
   var expression = element.attr('ng:options');
+  var onChange = expressionCompile(element.attr('ng:change') || "").fnSelf;
   var match;
   if (!expression) {
     return inputWidgetSelector.call(this, element);
@@ -729,7 +730,10 @@ angularWidget('select', function(element){
             value = valueFn(tempScope);
           }
         }
-        if (!isUndefined(value)) model.set(value);
+        if (!isUndefined(value) && model.get() !== value) {
+          onChange(scope);
+          model.set(value);
+        }
         scope.$tryEval(function(){
           scope.$root.$eval();
         });
diff --git a/test/widgetsSpec.js b/test/widgetsSpec.js
index 422ca86b..e2a070c4 100644
--- a/test/widgetsSpec.js
+++ b/test/widgetsSpec.js
@@ -859,6 +859,27 @@ describe("widget", function(){
         expect(scope.selected).toEqual(scope.values[1]);
       });
 
+      it('should fire ng:change if present', function(){
+        createSelect({
+          name:'selected',
+          'ng:options':'value for value in values',
+          'ng:change':'count = count + 1'});
+        scope.values = [{name:'A'}, {name:'B'}];
+        scope.selected = scope.values[0];
+        scope.count = 0;
+        scope.$eval();
+        expect(scope.count).toEqual(0);
+
+        select.val('1');
+        browserTrigger(select, 'change');
+        expect(scope.count).toEqual(1);
+        expect(scope.selected).toEqual(scope.values[1]);
+
+        browserTrigger(select, 'change');
+        expect(scope.count).toEqual(1);
+        expect(scope.selected).toEqual(scope.values[1]);
+      });
+
       it('should update model on change through expression', function(){
         createSelect({name:'selected', 'ng:options':'item.id as item.name for item in values'});
         scope.values = [{id:10, name:'A'}, {id:20, name:'B'}];