Merge bcff982d3d into be33f345a1

app/assets/javascripts/osem-schedule.js

@@ -1,79 +1,112 @@
-var url; // Should be initialize in Schedule.initialize
+var conference_id; // Should be initialize in Schedule.initialize
 var schedule_id; // Should be initialize in Schedule.initialize
 
-function showError(error){
+var events_to_create = {};
-  // Delete other error messages before showing the new one
+var events_to_update = {};
+var events_to_remove = {};
+
+function showMessage(message, type){
+  // Delete other messages before showing the new one
   $('.unobtrusive-flash-container').empty();
-  UnobtrusiveFlash.showFlashMessage(error, {type: 'error'});
+  UnobtrusiveFlash.showFlashMessage(message, {type: type});
 }
 
 var Schedule = {
-  initialize: function(url_param, schedule_id_param) {
+  initialize: function(conference_id_param, schedule_id_param) {
-    url = url_param;
+    conference_id = conference_id_param;
     schedule_id = schedule_id_param;
   },
   remove: function(element) {
     var e =  $("#" + element);
+    var event_id = e.attr("event_id");
     var event_schedule_id = e.attr("event_schedule_id");
     if(event_schedule_id != null){
-      var my_url = url + '/' + event_schedule_id;
+      events_to_remove[event_id] = event_schedule_id;
-      var success_callback = function(data) {
+    }
-        console.log(data);
+    delete events_to_create[event_id];
-        e.attr("event_schedule_id", null);
+    delete events_to_update[event_id];
-        e.appendTo($(".unscheduled-events"));
+    var unscheduled = $(".unscheduled-events");
-        e.find(".schedule-event-delete-button").hide();
+    e.appendTo(unscheduled);
-      }
+    e.find(".schedule-event-delete-button").hide();
-      var error_callback = function(data) {
+  },
-        console.log(data);
+  add: function (new_parent, event) {
-        showError($.parseJSON(data.responseText).errors);
+    var event_id = event.attr("event_id");
+    var event_schedule_id = event.attr("event_schedule_id");
+    var room_id = new_parent.attr("room_id");
+    var start_time = (new_parent.attr("date") + ' ' + new_parent.attr("hour"));
+    if(event_schedule_id != null){
+      var params = {};
+      params[event_schedule_id] = {
+        room_id: room_id,
+        start_time: start_time
+      };
+      events_to_update[event_id] = params;
+      delete events_to_create[event_id];
+    }
+    else{
+      events_to_create[event_id] = {
+        room_id: room_id,
+        start_time: start_time,
+        event_id: event_id,
+        schedule_id: schedule_id
+      };
+      delete events_to_update[event_id];
+    }
+    delete events_to_remove[event_id];
+    event.appendTo(new_parent);
+    $("#event-" + event_id).find(".schedule-event-delete-button").show();
+  },
+  saveEvents: function () {
+    var errors = '';
+    var with_errors = false;
+    var error_callback = function(data) {
+      try{
+        errors += $.parseJSON(data.responseText).errors;
+        with_errors = true;
+      }catch(e){
+        with_errors = true;
       }
+    }
+    if(!jQuery.isEmptyObject(events_to_create)){
       $.ajax({
-        url: my_url,
+        async: false,
-        type: 'DELETE',
+        url: ("/admin/conferences/" + conference_id + "/bulk_create"),
-        success: success_callback,
+        type: 'POST',
+        data: { event_schedules: events_to_create },
         error: error_callback,
         dataType : 'json'
       });
     }
-    else{
+    if(!jQuery.isEmptyObject(events_to_update)){
-      showError("The event couldn't be unscheduled");
+      $.ajax({
+        async: false,
+        url: ("/admin/conferences/" + conference_id + "/bulk_update"),
+        type: 'POST',
+        data: { event_schedules: events_to_update },
+        error: error_callback,
+        dataType : 'json'
+      });
     }
-  },
+    if(!jQuery.isEmptyObject(events_to_remove)){
-  add: function (previous_parent, new_parent, event) {
+      $.ajax({
-    event.appendTo(new_parent);
+        async: false,
-    var event_schedule_id = event.attr("event_schedule_id");
+        url: ("/admin/conferences/" + conference_id + "/bulk_destroy"),
-    var my_url = url;
+        type: 'POST',
-    var type = 'POST';
+        data: { event_schedules: events_to_remove },
-    var params = { event_schedule: {
+        error: error_callback,
-      room_id: new_parent.attr("room_id"),
+        dataType : 'json'
-      start_time: (new_parent.attr("date") + ' ' + new_parent.attr("hour"))
+      });
-    }};
-    if(event_schedule_id != null){
-      type = 'PUT';
-      my_url += ('/' + event_schedule_id);
     }
-    else{
+    var url = (window.location.href).substring(0, window.location.href.indexOf('?'));
-      params['event_schedule']['event_id'] = event.attr("event_id");
+    if(with_errors){
-      params['event_schedule']['schedule_id'] = schedule_id;
+      var msg = "Some events couldn't be scheduled"
-    }
+      if(errors != ''){
-    var success_callback = function(data) {
+        msg += (": " + errors);
-      console.log(data);
-      event.attr("event_schedule_id", data.event_schedule_id);
-      event.find(".schedule-event-delete-button").show();
       }
-    var error_callback = function(data) {
+      window.location.href = (url + "?flash=" + msg + "&type=error");
-      console.log(data);
+    } else {
-      showError($.parseJSON(data.responseText).errors);
+      window.location.href = (url + "?flash=Schedule correctly saved&type=notice");
-      event.appendTo(previous_parent);
     }
-    $.ajax({
-      url: my_url,
-      type: type,
-      data: params,
-      success: success_callback,
-      error: error_callback,
-      dataType : 'json'
-    });
   }
 };
 
@@ -106,7 +139,7 @@ $(document).ready( function() {
         $(ui.draggable).css("left", 0);
         $(ui.draggable).css("top", 0);
         $(this).css("background-color", "#ffffff");
-        Schedule.add($(ui.draggable).parent(), $(this), $(ui.draggable));
+        Schedule.add($(this), $(ui.draggable));
     },
     over: function(event, ui) {
       $(this).css("background-color", "#009ED8");
@@ -115,6 +148,10 @@ $(document).ready( function() {
       $(this).css("background-color", "#ffffff");
       }
   });
+
+  $('.schedule-save').on('click', function(e) {
+    Schedule.saveEvents();
+  });
 });
 
 function eventClicked(e, element){

app/assets/javascripts/osem-switch.js

@@ -23,7 +23,7 @@ $(function () {
     }
 
     var callback = function(data) {
-      showError($.parseJSON(data.responseText).errors);
+      showMessage($.parseJSON(data.responseText).errors, 'error');
     }
     $.ajax({
       url: url,

app/controllers/admin/event_schedules_controller.rb

@@ -2,34 +2,40 @@ module Admin
   class EventSchedulesController < Admin::BaseController
     load_and_authorize_resource :event_schedule
 
-    def create
+    def bulk_create
-      if @event_schedule.save
+      result = EventSchedule.create(params[:event_schedules].values).reject { |p| p.errors.empty? }
-        render json: { event_schedule_id: @event_schedule.id }
+      if result.empty?
-      else
-        render json: { errors: "The event couldn't be scheduled. #{@event_schedule.errors.full_messages.join('. ')}" }, status: 422
-      end
-    end
-
-    def update
-      if @event_schedule.update(event_schedule_params)
-        render json: { event_schedule_id: @event_schedule.id }
-      else
-        render json: { errors: "The event couldn't be scheduled. #{@event_schedule.errors.full_messages.join('. ')}" }, status: 422
-      end
-    end
-
-    def destroy
-      if @event_schedule.destroy
         render json: {}
       else
-        render json: { errors: "The event couldn't be unscheduled. #{@event_schedule.errors.full_messages.join('. ')}" }, status: 422
+        event_names = result.collect { |event_schedule| event_schedule.event.try(:title) }
+        render json: { errors: event_names.to_s }, status: 422
       end
     end
 
-    private
+    def bulk_update
+      keys = []
+      values = []
+      params[:event_schedules].values.each do |e|
+        keys << e.keys[0]
+        values << e.values[0]
+      end
+      result = EventSchedule.update(keys, values).reject { |p| p.errors.empty? }
+      if result.empty?
+        render json: {}
+      else
+        event_names = result.collect { |event_schedule| event_schedule.event.try(:title) }
+        render json: { errors: event_names.to_s }, status: 422
+      end
+    end
 
-    def event_schedule_params
+    def bulk_destroy
-      params.require(:event_schedule).permit(:schedule_id, :event_id, :room_id, :start_time)
+      result = EventSchedule.destroy(params[:event_schedules].values).reject { |p| p.errors.empty? }
+      if result.empty?
+        render json: {}
+      else
+        event_names = result.collect { |event_schedule| event_schedule.event.try(:title) }
+        render json: { errors: event_names.to_s }, status: 422
+      end
     end
   end
 end

app/views/admin/schedules/show.html.haml

@@ -4,8 +4,14 @@
   .col-md-12
     .page-header
       %h1 Schedule
-      %p.text-muted
+      .row
-        Create the schedules for the conference
+        .col-md-10.col-xs-8
+          %p.text-muted
+            Create the schedules for the conference
+        .col-md-2.col-xs-4
+          .text-right
+            %button.btn.btn-success.schedule-save
+              Save
 
 - if @rooms.present?
   .row
@@ -32,6 +38,10 @@
         - @dates.each do |date|
           .tab-pane{ class: "#{ (@dates.first == date) ? 'active' : '' }", id: "#{date}" }
             = render partial: 'day_tab', locals: { date: date }
+  .row
+    .col-md-12.text-right
+      %button.btn.btn-success.schedule-save
+        Save
 - else
   .h3
     No Rooms!
@@ -41,5 +51,13 @@
 
 :javascript
   $(document).ready( function() {
-    Schedule.initialize("#{admin_conference_event_schedules_path(@conference)}", "#{@schedule.id}");
+    // When reload the page after saving, show flash messages in the url.
+    var url = window.location.href;
+    if (url.indexOf("?") > 0){
+      var params = (url.split("?"))[1].split(/[&=]/);
+      if(params[0] == 'flash' && params[2] == 'type')
+        showMessage(decodeURI(params[1]), params[3]);
+    }
+
+    Schedule.initialize("#{@conference.short_title}", "#{@schedule.id}");
   });

config/routes.rb

@@ -27,7 +27,9 @@ Osem::Application.routes.draw do
     resources :conferences do
       resource :contact, except: [:index, :new, :create, :show, :destroy]
       resources :schedules, only: [:index, :create, :show, :update, :destroy]
-      resources :event_schedules, only: [:create, :update, :destroy]
+      post '/bulk_create' => 'event_schedules#bulk_create'
+      post '/bulk_update' => 'event_schedules#bulk_update'
+      post '/bulk_destroy' => 'event_schedules#bulk_destroy'
       get 'commercials/render_commercial' => 'commercials#render_commercial'
       resources :commercials, only: [:index, :create, :update, :destroy]
       get '/volunteers_list' => 'volunteers#show'

spec/controllers/admin/event_schedules_controller_spec.rb

@@ -4,30 +4,39 @@ describe Admin::EventSchedulesController do
   let(:venue) { create(:venue) }
   let(:conference) { create(:conference, venue: venue) }
   let(:room) { create(:room, venue: venue) }
-  let(:schedule) { create(:schedule, program: conference.program)}
+  let(:schedule) { create(:schedule, program: conference.program) }
-  let(:event_schedule) { create(:event_schedule, schedule: schedule)}
   let!(:organizer_role) { Role.find_by(name: 'organizer', resource: conference) }
   let(:organizer) { create(:user, role_ids: organizer_role.id) }
 
   context 'logged in as an organizer' do
     before :each do
       sign_in(organizer)
-      event_schedule
     end
 
     describe 'POST #create' do
       context 'with valid attributes' do
+        let(:event1) { create(:event, program: conference.program) }
+        let(:event2) { create(:event, program: conference.program) }
         let(:create_action) do
-          post :create, conference_id: conference.short_title, event_schedule:
+          post :bulk_create, conference_id: conference.short_title,
-               attributes_for(:event_schedule,
+                             event_schedules: {
-                              schedule_id: schedule.id,
+                               event1.id => {
-                              event_id: create(:event, program: conference.program).id,
+                                 schedule_id: schedule.id,
-                              room_id: create(:room, venue: venue).id,
+                                 event_id: event1.id,
-                              start_time: conference.start_date)
+                                 room_id: create(:room, venue: venue).id,
+                                 start_time: conference.start_date
+                               },
+                               event2.id => {
+                                 schedule_id: schedule.id,
+                                 event_id: event2.id,
+                                 room_id: create(:room, venue: venue).id,
+                                 start_time: conference.start_date
+                               }
+                             }
         end
 
         it 'saves the event schedule to the database' do
-          expect{ create_action }.to change { EventSchedule.count }.by 1
+          expect{ create_action }.to change { EventSchedule.count }.by 2
         end
 
         it 'has 200 status code' do
@@ -37,14 +46,17 @@ describe Admin::EventSchedulesController do
       end
 
       context 'with invalid attributes' do
-
+        let(:event1) { create(:event, program: conference.program) }
         let(:create_action) do
-          post :create, conference_id: conference.short_title, event_schedule:
+          post :bulk_create, conference_id: conference.short_title,
-               attributes_for(:event_schedule,
+                             event_schedules: {
-                              schedule_id: schedule.id,
+                               event1.id => {
-                              event_id: nil,
+                                 schedule_id: schedule.id,
-                              room_id: nil,
+                                 event_id: event1.id,
-                              start_time: nil)
+                                 room_id: nil,
+                                 start_time: nil
+                               }
+                             }
         end
 
         it 'does not save the event schedule to the database' do
@@ -60,22 +72,38 @@ describe Admin::EventSchedulesController do
 
     describe 'POST #update' do
       context 'with valid attributes' do
+        let(:event1) { create(:event, program: conference.program) }
+        let(:event2) { create(:event, program: conference.program) }
+        let(:event_schedule1) { create(:event_schedule, schedule: schedule, event: event1) }
+        let(:event_schedule2) { create(:event_schedule, schedule: schedule, event: event2) }
         before :each do
-          patch :update, id: event_schedule.id, conference_id: conference.short_title, event_schedule:
+          post :bulk_update, conference_id: conference.short_title,
-                 attributes_for(:event_schedule,
+                             event_schedules: {
-                                schedule_id: schedule.id,
+                               event1.id => {
-                                event_id: create(:event, program: conference.program).id,
+                                 event_schedule1.id => {
-                                room_id: room.id,
+                                   room_id: room.id,
-                                start_time: conference.start_date)
+                                   start_time: conference.start_date
-          event_schedule.reload
+                                 }
+                               },
+                               event2.id => {
+                                 event_schedule2.id => {
+                                   room_id: room.id,
+                                   start_time: conference.start_date + 1.day
+                                 }
+                               }
+                             }
+          event_schedule1.reload
+          event_schedule2.reload
         end
 
         it 'updates the room' do
-          expect(event_schedule.room_id).to eq(room.id)
+          expect(event_schedule1.room_id).to eq(room.id)
+          expect(event_schedule2.room_id).to eq(room.id)
         end
 
         it 'updates the start_time' do
-          expect(event_schedule.start_time).to eq(conference.start_date)
+          expect(event_schedule1.start_time).to eq(conference.start_date)
+          expect(event_schedule2.start_time).to eq(conference.start_date + 1.day)
         end
 
         it 'has 200 status code' do
@@ -84,13 +112,18 @@ describe Admin::EventSchedulesController do
       end
 
       context 'with invalid attributes' do
+        let(:event) { create(:event, program: conference.program) }
+        let(:event_schedule) { create(:event_schedule, schedule: schedule, event: event) }
         let(:update_action) do
-          patch :update, id: event_schedule.id, conference_id: conference.short_title, event_schedule:
+          post :bulk_update, conference_id: conference.short_title,
-               attributes_for(:event_schedule,
+                             event_schedules: {
-                              schedule_id: schedule.id,
+                               event.id => {
-                              event_id: nil,
+                                 event_schedule.id => {
-                              room_id: nil,
+                                   room_id: nil,
-                              start_time: nil)
+                                   start_time: nil
+                                 }
+                               }
+                             }
         end
         it 'does not save the event schedule to the database' do
           expect{ update_action }.to_not change { event_schedule }
@@ -104,12 +137,22 @@ describe Admin::EventSchedulesController do
     end
 
     describe 'DELETE #destroy' do
+      let(:event1) { create(:event, program: conference.program) }
+      let(:event2) { create(:event, program: conference.program) }
+      let(:event_schedule1) { create(:event_schedule, schedule: schedule, event: event1) }
+      let(:event_schedule2) { create(:event_schedule, schedule: schedule, event: event2) }
       let(:destroy_action) do
-        delete :destroy, id: event_schedule.id, conference_id: conference.short_title
+        post :bulk_destroy, conference_id: conference.short_title,
+                            event_schedules: {
+                              event1.id => event_schedule1,
+                              event2.id => event_schedule2
+                            }
       end
 
       it 'deletes the event schedule' do
-        expect{ destroy_action }.to change { EventSchedule.count }.by(-1)
+        event_schedule1.reload
+        event_schedule2.reload
+        expect{ destroy_action }.to change { EventSchedule.count }.by(-2)
       end
 
       it 'has 200 status code' do