artfully_ose 1.1.0 → 1.2.0.alpha.1

data/app/controllers/refunds_controller.rb

@@ -9,14 +9,10 @@ class RefundsController < ArtfullyOseController
     @items = params[:items].collect { |item_id| Item.find(item_id) }
 
     @refund = Refund.new(@order, @items)
-    @refund.submit(:and_return => return_items?)
+    @refund.submit(:and_return => return_items?, :send_email_confirmation => send_email_confirmation?)
 
     if @refund.successful?
-      if return_items?
-        flash[:notice] = "Successfully refunded and returned #{@refund.items.size} items."
-      else
-        flash[:notice] = "Successfully refunded #{@refund.items.size} items."
-      end
+      flash[:notice] = "Successfully refunded #{@refund.items.size} items."
     else
       if @refund.message.nil?
         flash[:error] = "Unable to refund items.  Please contact support and we'll try to help!"
@@ -30,7 +26,11 @@ class RefundsController < ArtfullyOseController
 
   private
 
+  def send_email_confirmation?
+    params[:send_email_confirmation] == "1"
+  end
+
   def return_items?
-    @return_items ||= (params[:commit] == "Issue Refund and Return" and @items.all?(&:returnable?))
+    params[:return_to_inventory] == "1"
   end
 end

data/app/controllers/regular_donation_kits_controller.rb

@@ -0,0 +1,32 @@
+class RegularDonationKitsController < ApplicationController
+  rescue_from CanCan::AccessDenied do |exception|
+    flash[:alert] = exception.message
+    redirect_to root_path, :alert => exception.message
+  end
+
+  before_filter do
+    authorize! :edit, current_user.current_organization
+  end
+
+  def edit
+    @kit = Kit.find(params[:id])
+  end
+
+  def update
+    @kit = Kit.find(params[:id])
+
+    if @kit.update_attributes(kit_params)
+      flash[:notice] = 'Your changes have been saved.'
+      redirect_to organization_path(@kit.organization)
+    else
+      flash[:error] = 'We had a problem configuring your kit. Please try again.'
+      render :edit
+    end
+  end
+
+  private
+
+  def kit_params
+    params[:regular_donation_kit]
+  end
+end

data/app/controllers/relationships_controller.rb

@@ -0,0 +1,15 @@
+class RelationshipsController < ArtfullyOseController
+
+  before_filter :load_tags
+
+  def index
+    @person = Person.find(params_person_id)
+
+    if params[:relation_id].present?
+      @relationships = @person.relationships.where(:relation_id => params[:relation_id].to_i)
+    else
+      @relationships = @person.relationships
+    end
+  end
+
+end

data/app/controllers/sales_controller.rb

@@ -1,55 +1,77 @@
 class SalesController < ArtfullyOseController
+  include CartFinder
   before_filter :find_event, :find_show, :find_people, :find_dummy
-  before_filter :create_door_list, :only => ['show', 'new']
+  before_filter :create_door_list, :only => ['show', 'new', 'door_list']
 
   def show
-    redirect_to new_event_show_sales_path(@event, @show)
+    redirect_to new_event_show_sales_path(@event,@show,:render => 'boxoffice')
   end
 
   def new
     @person = Person.new
-    @sale = Sale.new(@show, @show.chart.sections.box_office)
+    @sale = Sale.new(@show, @show.chart.ticket_types.box_office, current_box_office_cart, {})
     @tickets_remaining = tickets_remaining
     setup_defaults
+    response.headers["Cache-Control"] = "no-cache, no-store, max-age=0, must-revalidate" # http://stackoverflow.com/a/5493543/2063202
   end
 
-
-  #
-  # This needs a significant refactor.  There is no way to maintain the sale object
-  # across requests.  Because of this the number of locked tickets proliferates.
-  #
-  # Refactor this to maintain the cart across requests and clear the cart on successful sale
-  #
-  # In the meantime, BoxOffice::Cart has been jenked to just not lock tickets.
-  #
   def create
-    @sale = Sale.new(@show, @show.chart.sections.box_office, params[:quantities])
+    current_box_office_cart.clear!
+
+    @sale = Sale.new(@show, @show.chart.ticket_types.box_office, current_box_office_cart, params[:quantities], params[:order_notes])
+
+    # handle donation
+    if params[:donation].present? && params[:donation].to_i > 0
+      donation                 = Donation.new
+      donation.amount          = params[:donation].to_i * 100
+      donation.organization_id = @event.organization_id
+      current_box_office_cart.donations << donation
+    end
+
+    # handle discount
+    begin
+      discount = Discount.find_by_code_and_event_id(params[:discount].upcase, @event)
+      discount.apply_discount_to_cart(current_box_office_cart)
+    rescue RuntimeError => e
+      discount_error = e.message
+    rescue NoMethodError => e
+      discount_error = "We could not find your discount. Please try again." if params[:discount].present?
+    end
+
     if checking_out?
       if @sale.sell(payment)
-        @sale.message = "Sold #{self.class.helpers.pluralize(@sale.tickets.length, 'ticket')}.  Order total was #{self.class.helpers.number_as_cents @sale.cart.total}"
+        @sale.message = "Sold #{self.class.helpers.pluralize(@sale.tickets.length, 'ticket')}.  Order total was #{self.class.helpers.number_as_cents @sale.order.total}"
+
+        if params[:auto_check_in].present?
+          @sale.tickets.map {|t| t.reload; t.validate_ticket!(current_user)}
+        end
       end
     end
 
     unless @sale.errors.empty?
       @sale.error = "#{@sale.errors.full_messages.to_sentence.capitalize}."
+      flash[:error] = @sale.error
+      Ticket.unlock(@sale.tickets, @sale.cart)
+      render :js => "window.location = '#{new_event_show_sales_path(@event,@show,:render => 'boxoffice')}'"
+      return
     end
-    
+
     render :json => @sale.as_json
                          .merge(:total => @sale.cart.total)
                          .merge(:tickets_remaining => tickets_remaining)
-                         .merge(:door_list_rows => door_list_rows), 
+                         .merge(:door_list_rows => door_list_rows)
+                         .merge(:discount_error => discount_error)
+                         .merge(:discount_amount => current_box_office_cart.discount_amount),
                          :status => 200
   end
 
   def checking_out?
-    !params[:commit].blank?
+    params[:commit].present?
   end
-  
+
   def door_list_rows
     door_list_rows = []
-    
-    puts @sale.tickets.inspect
-    
+
     @sale.tickets.each_with_index do |ticket, i|
       ticket.reload
       if ticket.sold? || ticket.comped?
@@ -57,7 +79,8 @@ class SalesController < ArtfullyOseController
         door_list_rows[i]['first_name'] = @sale.buyer.first_name
         door_list_rows[i]['last_name'] = @sale.buyer.last_name
         door_list_rows[i]['email'] = @sale.buyer.email
-        door_list_rows[i]['section'] = ticket.section.name
+        door_list_rows[i]['ticket_type'] = ticket.ticket_type.name
+        door_list_rows[i]['ticket_id'] = ticket.id
         door_list_rows[i]['payment_method'] = ticket.sold_item.order.payment_method
         door_list_rows[i]['price'] = ticket.sold_price
       end
@@ -65,15 +88,22 @@ class SalesController < ArtfullyOseController
     door_list_rows
   end
 
+  def door_list
+    # create_door_list
+    render :partial => "sales/doorlist"
+  end
+
   private
+
+    # TODO: this should be pulled into TicketTypeSerializer
     def tickets_remaining
       remaining = {}
-      @sale.sections.each do |section|
-        remaining[section.id] = section.summary.available
+      @sale.ticket_types.each do |ticket_type|
+        remaining[ticket_type.id] = ticket_type.available("box_office")
       end
       remaining
     end
-    
+
     def setup_defaults
       params[:anonymous]   = true
       params[:cash]        = true
@@ -106,10 +136,39 @@ class SalesController < ArtfullyOseController
     end
 
     def person
-      params[:person_id].blank? ? @dummy : Person.find(params[:person_id])
+      return @person unless @person.nil?
+
+      #if there's a person_id, use find
+      @person = Person.find(params[:person][:id]) unless params[:person][:id].blank?
+
+      if user_entered_nothing?
+        @person = @dummy
+      else
+        @person = Person.first_or_create(person_attributes)
+      end
+
+      @person
+    end
+
+    def person_attributes
+      {
+        :id                => params[:person][:id],
+        :first_name        => params[:person][:first_name],
+        :last_name         => params[:person][:last_name],
+        :email             => params[:person][:email],
+        :phones_attributes => params[:person][:phones_attributes],
+        :organization      => current_organization
+      }
     end
 
-    def payment                    
+    def user_entered_nothing?
+      params[:person][:id].blank?         &&
+      params[:person][:first_name].blank? &&
+      params[:person][:last_name].blank?  &&
+      params[:person][:email].blank?
+    end
+
+    def payment
       if Swiper.can_parse? params[:credit_card][:number]
         swiped_data = Swiper.parse(params[:credit_card][:number])
         params[:credit_card][:name] = swiped_data.track1.cardholder_name
@@ -117,14 +176,19 @@ class SalesController < ArtfullyOseController
         params[:credit_card][:month] = swiped_data.track1.expiration_month
         params[:credit_card][:year] = swiped_data.track1.expiration_year
       end
-      
+
       params[:benefactor] = current_user
-      
-      payment = Payment.create(params[:payment_method], params)
+
+      # set payment as cash if total is 0 (e.g. free tickets)
+      if params[:payment_method].blank? && params[:total] == '0'
+        payment = Payment.create('cash', params)
+      else
+        payment = Payment.create(params[:payment_method], params)
+      end
       payment.customer = person
       payment
     end
-  
+
     def has_card_info?
       params[:credit_card].present? and params[:credit_card][:card_number].present?
     end

data/app/controllers/searches_controller.rb

@@ -6,7 +6,9 @@ class SearchesController < ApplicationController
   def new
     authorize! :view, Search
     @search = Search.new(params[:search])
-    prepare_search_and_people
+    @membership_types = current_user.current_organization.membership_types.all
+    @pass_types       = current_user.current_organization.pass_types.all
+    prepare_form
   end
 
   def create
@@ -21,12 +23,15 @@ class SearchesController < ApplicationController
     @search = Search.find(params[:id])
     authorize! :view, @search
     @segment = Segment.new
+    @membership_types = current_user.current_organization.membership_types.all
+    @pass_types       = current_user.current_organization.pass_types.all
     session[:return_to] ||= request.referer # Record the current page, in case creating a list segment fails.
-    prepare_search_and_people
+    prepare_form
+    prepare_people
     respond_to do |format|
-     format.html
-     format.csv { render :csv => @search.people, :filename => "#{@search.id}-#{DateTime.now.strftime("%m-%d-%y")}" }
-   end
+      format.html { @people = @people.paginate(:page => params[:page], :per_page => (params[:per_page] || 20)) }
+      format.csv  { render :csv => Person.where(:id => @people.collect(&:id)).includes(:phones, :address, :tags).order('lower(people.last_name)'), :filename => "#{@search.id}-#{DateTime.now.strftime("%m-%d-%y")}" }
+    end
   end
 
   def tag
@@ -39,10 +44,13 @@ class SearchesController < ApplicationController
 
   private
 
-    def prepare_search_and_people
+    def prepare_form
       @event_options = Event.options_for_select_by_organization(@current_user.current_organization)
+      @event_options.unshift(["Any Event", Search::ANY_EVENT])
+    end
+
+    def prepare_people
       @people = @search.people
-      @people = @people.paginate(:page => params[:page], :per_page => 20)
     end
 
     def load_discount_codes

data/app/controllers/sections_controller.rb

@@ -13,7 +13,7 @@ class SectionsController < ArtfullyOseController
 
   def create
     @section = Section.new
-    params[:section][:price] = Section.price_to_cents(params[:section][:price])
+    params[:section][:price] = TicketType.price_to_cents(params[:section][:price])
     @section.update_attributes(params[:section])
     @section.chart_id = @chart.id
     if @section.save
@@ -34,7 +34,7 @@ class SectionsController < ArtfullyOseController
     @qty = params[:quantity].to_i
     @section = Section.find(params[:id])
     @section.put_on_sale @qty
-    flash[:notice] = "Tickets in section #{@section.name} are now on sale"
+    flash[:notice] = "Tickets are now on sale"
     redirect_to event_show_path(@section.chart.show.event, @section.chart.show)
   end
   
@@ -42,7 +42,7 @@ class SectionsController < ArtfullyOseController
     @qty = params[:quantity].to_i
     @section = Section.find(params[:id])
     @section.take_off_sale @qty
-    flash[:notice] = "Tickets in section #{@section.name} are now off sale"
+    flash[:notice] = "Tickets are now off sale"
     redirect_to event_show_path(@section.chart.show.event, @section.chart.show)
   end
 

data/app/controllers/segments_controller.rb

@@ -9,11 +9,11 @@ class SegmentsController < ArtfullyOseController
   def show
     @segment = Segment.find(params[:id])
     authorize! :view, @segment
-    @people = @segment.people.paginate(:page => params[:page], :per_page => 20)
+    @people = @segment.people
     respond_to do |format|
-     format.html
-     format.csv { render :csv => @segment.people, :filename => "#{@segment.name}-#{DateTime.now.strftime("%m-%d-%y")}" }
-   end
+      format.html { @people = @people.paginate(:page => params[:page], :per_page => 20) }
+      format.csv  { render :csv => Person.where(:id => @people.collect(&:id)).includes(:phones, :address, :tags).order('lower(people.last_name)'), :filename => "#{@segment.name}-#{DateTime.now.strftime("%m-%d-%y")}" }
+    end
   end
 
   def create

data/app/controllers/shows_controller.rb

@@ -1,59 +1,68 @@
 class ShowsController < ArtfullyOseController
-  before_filter :find_event, :only => [ :index, :show, :new, :edit ]
+  before_filter :find_event, :only => [ :index, :calendar, :show, :new, :edit, :duplicate, :on_sale, :off_sale, :destroy ]
   before_filter :check_for_charts, :only => [ :index, :new ]
-  before_filter :upcoming_shows, :only => [ :index, :show ]
 
   rescue_from CanCan::AccessDenied do |exception|
     flash[:alert] = exception.message
-    redirect_to event_url(@show.event)
+    redirect_to event_url(@event)
   end
 
   def index
     authorize! :manage, @event
-    @shows = Show.where(:event_id => @event.id).includes(:tickets, :chart, :event => :venue).order('datetime ASC')
-  end
+    set_page_vars
+
+    @month_years =  @event.shows.pluck(:datetime).group_by {|d| d.strftime "%b %Y"}.keys
+
+    shows_rel = Show.where(:event_id => @event.id)
+    page_size = 25
 
-  def duplicate
-    @show = Show.find(params[:id])
-    authorize! :duplicate, @show
 
-    @new_show = @show.dup!
-    @new_show.save
-    redirect_to event_path(@show.event)
+    if monthly? && valid_month?
+      start = DateTime.now
+                      .change(:year => year, :month => month, :day => 1, :offset => offset)
+                      .beginning_of_day
+      shows_rel = shows_rel.where('datetime > ?', start).where('datetime < ?', start + 1.month)
+      page_size = 1000
+    elsif upcoming?
+      shows_rel = shows_rel.where('datetime > ?', Time.now - 2.days)
+    end
+
+    @shows = shows_rel.order('datetime ASC')
+                      .paginate(:page => params[:page], :per_page => page_size)
   end
 
   def new
     @show = @event.next_show
   end
 
+  def calendar
+    authorize! :manage, @event
+  end
+
   def create
-    @event = Event.find(params[:event_id])
-    @show = @event.next_show
-    (render :new and return) unless valid_datetime?
-    ActiveRecord::Base.transaction do
-      chart_params = params[:show].delete(:chart) 
-      if(chart_params.nil? || chart_params.empty?)
-        flash[:error] = "Please specify at least one ticket type for your show."
-        render :new and return
-      end
-      
-      #clear the sections and replace them with whatever they entered
-      @show.chart.sections = []
-      @show.chart.update_attributes_from_params(chart_params)
-      @show.update_attributes(params[:show])
-      @show.organization = current_organization
-      @show.chart_id = @show.chart.id
-      @show.datetime = ActiveSupport::TimeZone.create(@event.time_zone).parse(params[:show][:datetime])
-
-      if @show.go!(publishing_show?)      
-        flash[:notice] = "Show created on #{l @show.datetime_local_to_event, :format => :date_at_time}"
-        redirect_to event_show_path(@event, @show)
-      else      
-        flash[:error] = "There was a problem creating your show: #{@show.errors.full_messages.reject{|e| e.end_with? "be blank"}.to_sentence}"
-        render :new
-        raise ActiveRecord::Rollback
-      end
+    datetimes = params[:show][:datetime]
+    show_params = params[:show].except!(:datetime)
+    chart_params = Chart.polish_params(show_params.delete(:chart))
+    @event = current_organization.events.find(params[:event_id])
+
+    #TODO: Move these checks into ShowCreator
+    if(datetimes.blank?)
+      flash[:error] = "We need to know when your shows will be played! Click on the calendar to pick a date for your show."
+      redirect_to new_event_show_path(@event) and return
+    end
+
+    #TODO: This is supposed to check to see if they're setting prices on tickets, but it doesn't
+    if(chart_params.nil? || chart_params.empty?)
+      flash[:error] = "Please specify at least one ticket type for your show."
+      redirect_to new_event_show_path(@event) and return
     end
+    
+    redirect_to calendar_event_shows_path(@event) and return if @event.nil?
+
+    ShowCreator.enqueue(datetimes, show_params, chart_params, @event, current_organization, publishing_show?)
+    
+    flash[:notice] = "We're creating your shows and will be done shortly. Reload this page to see our progress"
+    redirect_to calendar_event_shows_path(@event)
   end
 
   def valid_datetime?
@@ -65,7 +74,7 @@ class ShowsController < ArtfullyOseController
   end
   
   def publishing_show?
-    ("Save and Publish" == params[:commit])
+    ("Save & Publish" == params[:commit])
   end
 
   def show
@@ -101,13 +110,19 @@ class ShowsController < ArtfullyOseController
     @show = Show.find(params[:id])
     authorize! :destroy, @show
 
-    if @show.destroy
-      render :nothing => true, :status => 204 and return  
+    if @show.delayed_destroy
+      respond_to do |format|
+        format.html do |f|
+          redirect_to event_shows_url(@show.event), :notice => 'Your show will be deleted in a few minutes.'
+        end
+        format.json { render :nothing => true, :status => 204 and return  }
+      end
     else
-      render :status => :forbidden and return
+      flash[:error] = "Sorry, this show can not be deleted. Contact support for further help."
+      redirect_to event_path(@event)
     end
   end
-
+  
   def door_list
     @show = Show.find(params[:id])
     @event = @show.event
@@ -123,89 +138,83 @@ class ShowsController < ArtfullyOseController
         @csv_string = @door_list.items.to_comma
         send_data @csv_string, :filename => @filename, :type => "text/csv", :disposition => "attachment"
       end
+      
+      format.pdf do
+        pdf = render_to_string :pdf => "door_list.pdf.haml"
+        send_data pdf, :filename => @filename, :type => "application/pdf", :disposition => "attachment"
+      end
     end
   end
 
   def published
-    @show = Show.find(params[:show_id])
+    @show = Show.find(params[:id])
     authorize! :show, @show
 
-    with_confirmation do
-      @show.publish!
-      respond_to do |format|
-        format.html { redirect_to event_show_url(@show.event, @show), :notice => 'Your show is now published.' }
-        format.json { render :json => @show.as_json }
-      end
+    @show.publish!
+    respond_to do |format|
+      format.html { redirect_to event_show_url(@show.event, @show), :notice => 'Your show is now published.' }
+      format.json { render :json => @show.as_json }
     end
   end
 
   def unpublished
-    @show = Show.find(params[:show_id])
+    @show = Show.find(params[:id])
     authorize! :hide, @show
 
-    with_confirmation do
-      @show.unpublish!
-      respond_to do |format|
-        format.html { redirect_to event_show_url(@show.event, @show), :notice => 'Your show is now unpublished.' }
-        format.json { render :json => @show.as_json }
-      end
-    end
-  end
-
-  def built
-    @show = Show.find(params[:show_id])
-    authorize! :edit, @show
-
-    @event = @show.event
-    # TODO: The ability to create tickets is business logic, not authorization logic.
-    authorize! :create_tickets, @show.chart.sections
-
-    @show.build!
-
+    @show.unpublish!
     respond_to do |format|
-      format.html { redirect_to event_show_url(@event, @show) }
-      format.json { render :json => @show.as_json.merge('glance' => @show.glance.as_json) }
+      format.html { redirect_to event_show_url(@show.event, @show), :notice => 'Your show is now unpublished.' }
+      format.json { render :json => @show.as_json }
     end
   end
 
   def on_sale
-    authorize! :bulk_edit, Ticket
-    with_confirmation do
-      @show = Show.find(params[:show_id])
+    @qty = params[:quantity].to_i
+    @show = Show.find(params[:id])
+    @section = @show.chart.sections.first
+    @section.put_on_sale @qty
+    @show.refresh_stats
+    flash[:notice] = "Tickets are now on sale"
+    redirect_to request.referer
+  end
+  
+  def off_sale
+    @qty = params[:quantity].to_i
+    @show = Show.find(params[:id])
+    @section = @show.chart.sections.first
+    @section.take_off_sale @qty
+    @show.refresh_stats
+    flash[:notice] = "Tickets are now off sale"
+    redirect_to request.referer
+  end
 
-      if @show.bulk_on_sale(:all)
-        @show.publish!
-        notice = "Put all tickets on sale."
-      else
-        error = "Tickets that have been sold or comped can't be put on or taken off sale. A ticket that is already on sale or off sale can't be put on or off sale again."
-      end
+  private
 
-      respond_to do |format|
-        format.html do
-          flash[:notice] = notice
-          flash[:error] = error
-          redirect_to event_show_url(@show.event, @show)
-        end
+    #view needs these variables set
+    def set_page_vars
+      all?
+      upcoming?
+      monthly?
+    end
 
-        format.json do
-          if error.blank?
-            render :json => @show.as_json.merge('glance' => @show.glance.as_json)
-          else
-            render :json => { :errors => [ error ] }, :status => 409
-          end
-        end
+    def all?
+      @all ||= (params[:range].present? && params[:range].to_sym == :all)
+    end
 
-      end
+    def upcoming?
+      @upcoming ||= !params[:range].present? && !params[:year].present?
+    end
+
+    def monthly?
+      @monthly ||= params[:year].present? && params[:month].present?
     end
-  end
 
-  private
     def find_event
       @event = Event.includes(:shows => [:event => :venue]).find(params[:event_id])
     end
 
-    def upcoming_shows
-      @upcoming = @event.upcoming_shows
+    def valid_month?
+      Date::MONTHNAMES.reject{|m| m.nil?}.map{|m| m[0..2] unless m.nil?}.include?(params[:month])
     end
 
     def with_confirmation
@@ -226,4 +235,16 @@ class ShowsController < ArtfullyOseController
       end
     end
 
+    def year
+      params[:year].to_i
+    end
+
+    def month
+      DateTime.parse(params[:month]).month
+    end
+
+    def offset
+      ActiveSupport::TimeZone.create(@event.time_zone).formatted_offset
+    end
+
 end