refactor(sortable): change getElementScope to increase code coverage

thgreasi · thgreasi · commit 9ef46229180f · 2015-01-17T19:11:07.000+02:00
diff --git a/src/sortable.js b/src/sortable.js
@@ -48,13 +48,15 @@ angular.module('ui.sortable', [])
           }
 
           function getElementScope(elementScopes, element) {
+            var result = null;
             for (var i = 0; i < elementScopes.length; i++) {
               var x = elementScopes[i];
               if (x.element[0] === element[0]) {
-                return x.scope;
+                result = x.scope;
+                break;
               }
             }
-            return null;
+            return result;
           }
 
           function afterStop(e, ui) {

Original file line number	Diff line number	Diff line change
`@@ -48,13 +48,15 @@ angular.module('ui.sortable', [])`
`48`	`48`	`}`
`49`	`49`
`50`	`50`	`function getElementScope(elementScopes, element) {`
	`51`	`+ var result = null;`
`51`	`52`	`for (var i = 0; i < elementScopes.length; i++) {`
`52`	`53`	`var x = elementScopes[i];`
`53`	`54`	`if (x.element[0] === element[0]) {`
`54`		`- return x.scope;`
	`55`	`+ result = x.scope;`
	`56`	`+ break;`
`55`	`57`	`}`
`56`	`58`	`}`
`57`		`- return null;`
	`59`	`+ return result;`
`58`	`60`	`}`
`59`	`61`
`60`	`62`	`function afterStop(e, ui) {`