From eb500e0765ec7e19102bdd8c509d6e6d83c30bf2 Mon Sep 17 00:00:00 2001
From: Harry Bomrah <harry@rulingcom.com>
Date: Wed, 10 Aug 2016 18:32:45 +0800
Subject: [PATCH] some new changes for booking, changed wordings and validation
 of dates plus other stuff

---
 .../admin/bus_bookings_controller.rb          | 10 ++++
 app/models/bus.rb                             |  6 +-
 app/views/admin/bus_bookings/_form.html.erb   | 22 ++++++-
 app/views/admin/bus_bookings/index.html.erb   |  8 ++-
 app/views/admin/bus_bookings/show.html.erb    | 59 ++++++++++++++++++-
 config/locales/en.yml                         |  6 +-
 config/locales/zh_tw.yml                      |  6 +-
 config/routes.rb                              |  1 +
 8 files changed, 106 insertions(+), 12 deletions(-)

diff --git a/app/controllers/admin/bus_bookings_controller.rb b/app/controllers/admin/bus_bookings_controller.rb
index 62742f4..74f3dbc 100644
--- a/app/controllers/admin/bus_bookings_controller.rb
+++ b/app/controllers/admin/bus_bookings_controller.rb
@@ -80,6 +80,16 @@ class Admin::BusBookingsController < OrbitMemberAppController
 		redirect_to admin_bus_bookings_path
 	end
 
+	def deletebookings
+		bookings = Booking.find(params[:bookings]) rescue nil
+		if !bookings.nil?
+			bookings.each do |booking|
+				booking.destroy
+			end
+		end
+		render :json => {"succes" => true}.to_json
+	end
+
 	private
 
 	def bus_params
diff --git a/app/models/bus.rb b/app/models/bus.rb
index 678b8a5..3a508bc 100644
--- a/app/models/bus.rb
+++ b/app/models/bus.rb
@@ -13,11 +13,15 @@ class Bus
 
 
 	def can_reserve?
-		self.reservation_end_time > Time.now && self.bookings.count < 21
+		self.reservation_end_time > Time.now && self.bookings.count <= self.capacity
 	end
 
 	def posted_by
 		User.find(self.created_by).name rescue ""
 	end
 
+	def already_reserved?(user)
+		self.bookings.where(:user_id => user.id.to_s).count == 1
+	end
+
 end
\ No newline at end of file
diff --git a/app/views/admin/bus_bookings/_form.html.erb b/app/views/admin/bus_bookings/_form.html.erb
index c975558..a4599a9 100644
--- a/app/views/admin/bus_bookings/_form.html.erb
+++ b/app/views/admin/bus_bookings/_form.html.erb
@@ -7,7 +7,6 @@
   <%= javascript_include_tag "lib/bootstrap-fileupload" %>
   <%= javascript_include_tag "lib/bootstrap-datetimepicker" %>
   <%= javascript_include_tag "lib/datetimepicker/datetimepicker.js" %>
-  <%= javascript_include_tag "lib/file-type" %>
 <% end %>
 
 <div class="input-area">
@@ -35,6 +34,7 @@
         <label class="control-label muted"><%= t("bus_booking.reservation_end_time") %></label>
         <div class="controls">
           <%= f.datetime_picker :reservation_end_time, :no_label => true, :new_record => @bus.new_record? %>
+          <div class='validator_error_class text-error hide'>Reservation end time cannot be later than departure time.</div>
         </div>
       </div>
 
@@ -96,4 +96,22 @@
   <%= f.hidden_field :created_by, :value => current_user.id %>
   <%= f.submit t('submit'), class: 'btn btn-primary' %>
   <%= link_to t('cancel'), admin_bus_bookings_path, :class=>"btn" %>  
-</div>
\ No newline at end of file
+</div>
+<script type="text/javascript">
+  $(".main-forms").on("submit",function(){
+    $(".validator_error_class").addClass("hide");
+    var d = new Date($("#bus_departure_time").val()),
+        d1 = new Date($("#bus_reservation_end_time").val());
+
+    if((d - d1) > 0){
+      $(this).submit();
+    }else{
+      $(".validator_error_class").removeClass("hide");
+      return false;
+    }
+  })
+</script>
+
+
+
+
diff --git a/app/views/admin/bus_bookings/index.html.erb b/app/views/admin/bus_bookings/index.html.erb
index f05e8ed..609393b 100644
--- a/app/views/admin/bus_bookings/index.html.erb
+++ b/app/views/admin/bus_bookings/index.html.erb
@@ -23,8 +23,12 @@
 						<a href="<%= admin_bus_booking_duplicate_path(bus) %>"><%= t("bus_booking.duplicate") %></a>
 						<a class="error" href="<%= admin_bus_booking_path(bus) %>" data-method="delete" data-confirm="Are you sure?"><%= t("bus_booking.delete_route") %></a>
 					<% end %>
-					<% if bus.can_reserve? %>
-						<a href="<%= admin_bus_booking_reserve_path(bus) %>"><%= t("bus_booking.reserve") %></a>
+					<% if bus.already_reserved?(current_user) %>
+						<a href="<%= admin_bus_booking_reserve_path(bus) %>"><%= t("bus_booking.manage") %></a>
+					<% else %>
+						<% if bus.can_reserve? %>
+							<a href="<%= admin_bus_booking_reserve_path(bus) %>"><%= t("bus_booking.reserve") %></a>
+						<% end %>
 					<% end %>
 				</td>
 				<td><%= bus.posted_by %></td>
diff --git a/app/views/admin/bus_bookings/show.html.erb b/app/views/admin/bus_bookings/show.html.erb
index bdbc3ac..611e3ef 100644
--- a/app/views/admin/bus_bookings/show.html.erb
+++ b/app/views/admin/bus_bookings/show.html.erb
@@ -2,6 +2,7 @@
 	<h2><%= t("bus_booking.booking_details_for") %> <%= @bus.bus_route %></h2>
 	<table class="table table-striped table-bordered">
 		<thead>
+			<th class="span1"><input type="checkbox" id="checkall"> All</th>
 			<th><%= t("bus_booking.registered_date") %></th>
 			<th><%= t("bus_booking.email") %></th>
 			<th><%= t("bus_booking.name") %></th>
@@ -9,8 +10,9 @@
 			<th><%= t("bus_booking.dorm_number") %></th>
 		</thead>
 		<tbody>
-		<% @bus.bookings.each do |booking| %>
+		<% @bus.bookings.desc(:created_at).each do |booking| %>
 			<tr>
+				<td><input type="checkbox" value="<%= booking.id.to_s %>" name="bookings_to_delete[]" class="check-this"></td>
 				<td><%= booking.created_at.strftime("%Y/%m/%d") %></td>
 				<td><%= booking.email %></td>
 				<td><%= booking.name %></td>
@@ -20,5 +22,56 @@
 		<% end %>
 		</tbody>
 	</table>
-	<a href="<%= admin_bus_booking_export_path(@bus, :format => "xlsx") %>" class="btn btn-info pull-right"><%= t("bus_booking.export") %></a>
-</div>
\ No newline at end of file
+	<div class="pull-right">
+		<a href="#" id="delete-selected" style="display:none;" class="btn btn-danger"><%= t("bus_booking.delete") %></a>
+		<a href="<%= admin_bus_booking_export_path(@bus, :format => "xlsx") %>" class="btn btn-info"><%= t("bus_booking.export") %></a>
+	</div>
+</div>
+<script type="text/javascript">
+	var deleteBtn = $("#delete-selected"),
+		checkAllBtn = $("#checkall"),
+		bookings = $("input.check-this");
+
+	checkAllBtn.on("click",function(){
+		if($(this).is(":checked")){
+			bookings.prop("checked",true);
+		}else{
+			bookings.prop("checked",false);
+		}
+		showHideDeleteFn();
+	})
+
+	bookings.on("click",function(){
+		if($("input.check-this:checked").length == bookings.length){
+			checkAllBtn.prop("checked",true);	
+		}else{
+			checkAllBtn.prop("checked",false);	
+		}
+		showHideDeleteFn();
+	})
+
+	deleteBtn.on("click",function(){
+		var b = $("input.check-this:checked");
+		if (b.length){
+			var values = b.map(function(i,x){return x.value}).toArray();
+			console.log(values);
+			$.ajax({
+				url : "/admin/bus_bookings/deletebookings",
+				data : {"bus_id" : "<%= @bus.id.to_s %>", "bookings" : values},
+				type : "post"
+			}).done(function(){
+				window.location.reload();
+			})
+		}
+		return false;
+	})
+
+	var showHideDeleteFn = function(){
+		if($("input.check-this:checked").length){
+			deleteBtn.show();
+		}else{
+			deleteBtn.hide();
+		}
+	}
+	showHideDeleteFn();
+</script>
\ No newline at end of file
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 2f18e26..ce5a501 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -12,7 +12,7 @@ en:
     delete_route: Delete Route
     bus_schedule: Bus Schedule
     reserve_now: Reserve
-    details: Reserve Details
+    details: Reservation Details
     reserved: Reserved
     name: Name
     email: Email
@@ -24,4 +24,6 @@ en:
     registered_date: Registered Date
     posted_by: Posted By
     cancel_reservation: Cancel Reservation
-    duplicate: Duplicate
\ No newline at end of file
+    duplicate: Duplicate
+    delete: Delete
+    manage: Manage Reservation
\ No newline at end of file
diff --git a/config/locales/zh_tw.yml b/config/locales/zh_tw.yml
index ab4b1b0..94080e5 100644
--- a/config/locales/zh_tw.yml
+++ b/config/locales/zh_tw.yml
@@ -12,7 +12,7 @@ zh_tw:
     delete_route: Delete Route
     bus_schedule: Bus Schedule
     reserve_now: Reserved
-    details: Reserve Details
+    details: Reservation Details
     reserved: Reserved
     name: Name
     email: Email
@@ -24,4 +24,6 @@ zh_tw:
     registered_date: Registered Date
     posted_by: Posted By
     cancel_reservation: Cancel Reservation
-    duplicate: Duplicate
\ No newline at end of file
+    duplicate: Duplicate
+    delete: Delete
+    manage: Manage Reservation
\ No newline at end of file
diff --git a/config/routes.rb b/config/routes.rb
index 2f32ced..11ec84a 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -5,6 +5,7 @@ Rails.application.routes.draw do
   	scope "(:locale)", locale: Regexp.new(locales.join("|")) do
   		namespace :admin do
   			post "bus_bookings/bookbus" => "bus_bookings#bookbus"
+        post "bus_bookings/deletebookings" => "bus_bookings#deletebookings"
   			resources :bus_bookings do 
   				get "reserve"
   				get "export"