Fix #1439

public/js/ff/piggy-banks/index.js

@@ -88,28 +88,23 @@ function stopSorting() {
         var holder = $(v);
         var position = parseInt(holder.data('position'));
         var originalOrder = parseInt(holder.data('order'));
+        var name = holder.data('name');
         var id = holder.data('id');
-        console.log('Now at row ' + i);
         var newOrder;
         if (position === i) {
+            // not changed, position is what it should be.
             return;
         }
         if (position < i) {
-            console.log('Row ' + i + ' has moved up!');
+            // position is less.
-            // update position:
+            console.log('"' + name + '" has moved up from position ' + originalOrder + ' to ' + (i+1));
-            holder.data('position', i);
-            newOrder = originalOrder + 1;
-
-
         }
         if (position > i) {
-            console.log('Row ' + i + ' has moved down!');
+            console.log('"' + name + '" has moved up from position ' + originalOrder + ' to ' + (i+1));
-            // update position:
-            holder.data('position', i);
-            newOrder = originalOrder - 1;
-
-
         }
+        // update position:
+        holder.data('position', i);
+        newOrder = i+1;
 
         $.post('piggy-banks/set-order/' + id, {order: newOrder, _token: token})
     });

resources/views/list/piggy-banks.twig

@@ -14,7 +14,7 @@
     </thead>
     <tbody>
     {% for piggy in piggyBanks %}
-        <tr data-id="{{ piggy.id }}" data-order="{{ piggy.order }}" data-position="{{ loop.index0 }}">
+        <tr data-id="{{ piggy.id }}" data-name="{{ piggy.name }}" data-order="{{ piggy.order }}" data-position="{{ loop.index0 }}">
             <td class="visible-xs visible-sm hidden-md hidden-lg">
                 &nbsp;
             </td>