fix(markers): reduce angular.equals calls

Waldemar Panas · Waldemar Panas · commit 68ac35240a73 · 2016-02-29T16:26:30.000+01:00
diff --git a/src/directives/markers.js b/src/directives/markers.js
@@ -166,7 +166,7 @@ angular.module('ui-leaflet').directive('markers',
                 //ie the options to only check !== (reference check) instead of angular.equals (slow)
                 newMarker = markerModels[name];
                 oldMarker = oldMarkerModels[name];
-                equals = angular.equals(newMarker,oldMarker) && isEqual;
+                equals = isEqual && angular.equals(newMarker, oldMarker);
             }
             if (!isDefined(markerModels) ||
                 !Object.keys(markerModels).length ||

Original file line number	Diff line number	Diff line change
`@@ -166,7 +166,7 @@ angular.module('ui-leaflet').directive('markers',`
`166`	`166`	`//ie the options to only check !== (reference check) instead of angular.equals (slow)`
`167`	`167`	`newMarker = markerModels[name];`
`168`	`168`	`oldMarker = oldMarkerModels[name];`
`169`		`- equals = angular.equals(newMarker,oldMarker) && isEqual;`
	`169`	`+ equals = isEqual && angular.equals(newMarker, oldMarker);`
`170`	`170`	`}`
`171`	`171`	`if (!isDefined(markerModels) \|\|`
`172`	`172`	`!Object.keys(markerModels).length \|\|`