fix(ionReorderButton): fix onReorder not triggering angular digest

Author: ajoslin

Diff for: js/angular/directive/list.js

@@ -250,7 +250,12 @@ function($animate, $timeout) {
             onReorder: function(el, oldIndex, newIndex) {
               var itemScope = jqLite(el).scope();
               if (itemScope && itemScope.$onReorder) {
-                itemScope.$onReorder(oldIndex, newIndex);
+                //Make sure onReorder is called in apply cycle,
+                //but also make sure it has no conflicts by doing
+                //$evalAsync
+                itemScope.$evalAsync(function() {
+                  itemScope.$onReorder(oldIndex, newIndex);
+                });
               }
             },
             canSwipe: function() {

Diff for: test/html/has-anything-dynamic.html

@@ -9,19 +9,18 @@
 
       <ion-tab title="So Tabular!">
 
-        <div ng-if="$root.headerExists">
-          <ion-header-bar class="bar-positive"
-            ng-class="{'bar-subheader': $root.isSubheader}">
-            <h1 class="title">Header Bar</h1>
-          </ion-header-bar>
+        <ion-header-bar ng-show="$root.headerExists"
+          class="bar-positive"
+          ng-class="{'bar-subheader': $root.isSubheader}">
+          <h1 class="title">Header Bar</h1>
+        </ion-header-bar>
         </div>
 
-        <div ng-if="$root.footerExists">
-          <ion-footer-bar class="bar-assertive"
-            ng-class="{'bar-subfooter': $root.isSubfooter}">
-            <h1 class="title">Footer bar</h1>
-          </ion-footer-bar>
-        </div>
+        <ion-footer-bar ng-show="$root.footerExists"
+          class="bar-assertive"
+          ng-class="{'bar-subfooter': $root.isSubfooter}">
+          <h1 class="title">Footer bar</h1>
+        </ion-footer-bar>
 
         <ion-content>
           <div class="card">

Diff for: test/unit/angular/directive/list.unit.js

@@ -110,6 +110,8 @@ describe('ionList directive', function() {
 
     el.scope().$onReorder = jasmine.createSpy('$onReorder');
     options.onReorder(el, 2, 3);
+    expect(el.scope().$onReorder).not.toHaveBeenCalled();
+    el.scope().$apply();
     expect(el.scope().$onReorder).toHaveBeenCalledWith(2,3);
   });