jshint

Author: wuliupo

src/sortable.js

@@ -20,6 +20,7 @@ angular.module('ui.sortable', [])
           uiSortable: '='
         },
         link: function(scope, element, attrs, ngModel) {
+          'use strict';
           var savedNodes;
 
           function combineCallbacks(first, second){
@@ -35,6 +36,28 @@ angular.module('ui.sortable', [])
             return first;
           }
 
+          function wrappersHelper(inner) {
+            return function (e, item) {
+              var oldItemSortable = item.sortable;
+              var index = getItemIndex(item);
+              item.sortable = {
+                model: ngModel.$modelValue[index],
+                index: index,
+                source: item.parent(),
+                sourceModel: ngModel.$modelValue,
+                _restore: function () {
+                  delete item.sortable;
+                  item.sortable = oldItemSortable;
+                }
+              };
+            
+              var innerResult = inner.apply(this, arguments);
+              item.sortable._restore();
+              item.sortable._isCustomHelperUsed = item !== innerResult;
+              return innerResult;
+            };
+          }
+
           function getSortableWidgetInstance(element) {
             // this is a fix to support jquery-ui prior to v1.11.x
             // otherwise we should be using `element.sortable('instance')`
@@ -363,28 +386,6 @@ angular.module('ui.sortable', [])
               }
             };
 
-            function wrappersHelper(inner) {
-              return function (e, item) {
-                var oldItemSortable = item.sortable;
-                var index = getItemIndex(item);
-                item.sortable = {
-                  model: ngModel.$modelValue[index],
-                  index: index,
-                  source: item.parent(),
-                  sourceModel: ngModel.$modelValue,
-                  _restore: function () {
-                    delete item.sortable;
-                    item.sortable = oldItemSortable;
-                  }
-                };
-              
-                var innerResult = inner.apply(this, arguments);
-                item.sortable._restore();
-                item.sortable._isCustomHelperUsed = item !== innerResult;
-                return innerResult;
-              };
-            };
-
             scope.$watchCollection('uiSortable', function(newVal, oldVal) {
               // ensure that the jquery-ui-sortable widget instance
               // is still bound to the directive's element