class CoitemController < ApplicationController
+ layout "admin"
+
+ # Make sure that the user has logged in before they can take any
+ # action on checked out items
+ before_filter :authorize
+
def index
- list
- render :action => 'list'
+ render :action => 'index'
end
# GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html)
@coitem = Coitem.find(params[:id])
end
- def new
- @coitem = Coitem.new
- end
-
- def create
- @coitem = Coitem.new(params[:coitem])
- if @coitem.save
- flash[:notice] = 'Coitem was successfully created.'
- redirect_to :action => 'list'
- else
- render :action => 'new'
- end
- end
+ # We should never create new checked out items via the web interface, so remove
+ # the new and create methods.
def edit
@coitem = Coitem.find(params[:id])
end
end
- def destroy
- Coitem.find(params[:id]).destroy
- redirect_to :action => 'list'
- end
+ # We should never delete a checked out item directly via the web interface, so
+ # remove the destroy method.
# Awesome, paginating overdue list, ordered by customer
def overdue
- @coitem_pages, @coitems = paginate :coitems, :per_page => 50, :conditions => "due_date < DATE('NOW', 'LOCALTIME')", :order => "customer_id"
- render :action => 'list'
+ @coitem_pages, @coitems = paginate :coitems, :per_page => 50, :conditions => ["due_date < ?", Time.now.to_date], :order => "customer_id"
+ render :action => 'overdue'
end
- def return
- render :action => 'return'
+ def filterbycustomerid
+ query = params[:id]
+ @coitem_pages, @coitems = paginate :coitems, :per_page => 20, :conditions => ["customer_id = ?", query]
+ render :action => 'list'
end
- def return_validate
- rentable_id = params[:rentable_id]
- @rentable = Rentable.find_by_id(rentable_id)
-
- if @rentable.nil?
- flash[:error] = "Unable to find this rentable"
- redirect_to :action => :return
- return
- end
-
- @coitem = Coitem.find_by_rentable_id(rentable_id)
- if @coitem.nil?
- flash[:error] = "This item is not checked out!"
- redirect_to :action => :return
- return
- end
-
- # Check if the item is overdue
- if @coitem.overdue?
- @coitem.customer.debt += @coitem.late_fee
- @coitem.customer.save
+ def return
+ if request.post?
+ rentable_id = params[:rentable_id]
+ @rentable = Rentable.find_by_id(rentable_id)
+
+ if @rentable.nil?
+ flash[:notice] = "Unable to find this rentable"
+ redirect_to :action => :return
+ return
+ end
+
+ @coitem = Coitem.find_by_rentable_id(rentable_id)
+ if @coitem.nil?
+ flash[:notice] = "This item is not checked out!"
+ redirect_to :action => :return
+ return
+ end
+
+ # Check if the item is overdue
+ if @coitem.overdue?
+ @coitem.customer.debt += @coitem.late_fee
+ @coitem.customer.save
+ end
+
+ # Delete the row
+ @coitem.destroy
+
+ flash[:notice] = "Successfully returned item"
+ redirect_to :action => :return, :method => :get
+ else
+ render :action => 'return'
end
-
- # Delete the row
- @coitem.destroy
-
- flash[:notice] = "Successfully returned item"
- redirect_to :action => :return
end
end