Solving issue 13 'Update page count when moving a Page'

public/list.js

@@ -195,7 +195,7 @@ $(function () {
   });
 
   function bindEvents(eventDiv) {
-    updatePageCount(eventDiv);
+    pageCounter(eventDiv);
 
     eventDiv.find('.event-expand').click(function (event) {
       event.preventDefault();
@@ -248,15 +248,18 @@ $(function () {
 
     eventDiv.droppable({scope: 'events', hoverClass: 'droppable-active', drop: function (event, ui) {
       $('.drag-cart-item').remove();
-      var eventDiv = $(event.target);
+      var toEventDiv = $(event.target);
       var pageMovedRow = $(ui.draggable);
       var pageMovedRowOldEvent = pageMovedRow.data('page-event');
-      var movedToEvent = eventDiv.data('event');
+      var movedToEvent = toEventDiv.data('event');
       if (pageMovedRowOldEvent == movedToEvent) {
         return;
       }
       pageMovedRow.data('page-event', movedToEvent);
-      eventDiv.find('tbody').append(pageMovedRow);
+      var fromEventDiv = $(pageMovedRow).closest('.event');
+      toEventDiv.find('tbody').append(pageMovedRow);
+      updatePageCounter(fromEventDiv, toEventDiv);
+
       var path = pageMovedRow.data('change-event-path');
       $.ajax({
         url: path,
@@ -300,7 +303,7 @@ $(function () {
     iframe.width = (newwidth) + "px";
   }
 
-  function updatePageCount(eventDiv) {
+  function pageCounter(eventDiv) {
     var pageCount = $(eventDiv).find('.page').length;
     var pageCounterMsg;
 
@@ -315,4 +318,10 @@ $(function () {
     }
     $(eventDiv).closest('.event').find('.event-page-count').text(pageCounterMsg);
   }
+
+  function updatePageCounter(fromEventPage, toEventPage) {
+    pageCounter(fromEventPage);
+    pageCounter(toEventPage);
+  }
+
 });

spec.rb

@@ -406,6 +406,25 @@
     assert_uploaded_page_is_displayed_within_event(@page1)
   end
 
+  it 'updates the counter page of event when a page is dragged to another event' do
+    @page1 = Page.create!(:name => 'Joana', :event => 'Event1', :content => 'oi!')
+    @page2 = Page.create!(:name => 'Claudia', :event => 'Event2', :content => 'hi!')
+
+    visit '/list'
+
+    expand_event('Event1')
+    hover_page_row(@page1)
+    drag_page_to_event(@page1, 'Event2')
+
+    within_event(@page1.event) do
+      find('.event-page-count').should have_content 'no pages'
+    end
+
+    within_event(@page2.event) do
+      find('.event-page-count').should have_content '2 pages'
+    end
+  end
+
   it 'shows when page contains image content' do
     @page_with_image = Page.create!(:name => 'Joana', :content => "meet me <img src='/me.jpg'/>!")
     @page_with_video = Page.create!(:name => 'Ana', :content => "meet me <iframe src='http://www.youtube.com/embed/132' />!")