maintain compatibility with older _notifyChange arguments.

Steven Orvell · Steven Orvell · commit f5aec30d3da9 · 2015-11-05T18:28:54.000-08:00
diff --git a/src/lib/bind/accessors.html b/src/lib/bind/accessors.html
@@ -20,8 +20,9 @@
 
     _modelApi: {
 
-      _notifyChange: function(event, value) {
-        // use a cached event here (_useCache: true) for efficiency
+      _notifyChange: function(source, event, value) {
+        value = value === undefined ? this[source] : value;
+        event = event || Polymer.CaseMap.camelToDashCase(source) + '-changed';
         this.fire(event, {value: value}, 
           {bubbles: false, cancelable: false, _useCache: true});
       },
diff --git a/src/lib/bind/effects.html b/src/lib/bind/effects.html
@@ -41,7 +41,7 @@
 
     _notifyEffect: function(source, value, effect, old, fromAbove) {
       if (!fromAbove) {
-        this._notifyChange(effect.event, value);
+        this._notifyChange(source, effect.event, value);
       }
     },
 

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@`
`41`	`41`
`42`	`42`	`_notifyEffect: function(source, value, effect, old, fromAbove) {`
`43`	`43`	`if (!fromAbove) {`
`44`		`- this._notifyChange(effect.event, value);`
	`44`	`+ this._notifyChange(source, effect.event, value);`
`45`	`45`	`}`
`46`	`46`	`},`
`47`	`47`