artfully_ose 1.2.0.pre.24 → 1.2.0.pre.26

data/app/concerns/cart_finder.rb

@@ -7,6 +7,11 @@ module CartFinder
 
   included do |c|
     c.helper_method :current_cart
+    c.helper_method :current_sales_console_cart
+  end
+
+  def current_sales_console_cart
+    current_cart(ConsoleSale::Cart)
   end
 
   def current_box_office_cart

data/app/controllers/console_sales_controller.rb

@@ -0,0 +1,182 @@
+class ConsoleSalesController < ArtfullyOseController
+  before_filter :verify_person
+  include CartFinder
+
+  rescue_from ActiveRecord::RecordInvalid, :with => :checkout_error
+
+  def new  
+    @events = Event.where(:id => current_organization.shows.unplayed.pluck(:event_id).uniq)
+    @membership_types = current_organization.membership_types.sales_valid
+
+    @membership_types_hash = {}
+    @membership_types.each {|mt| @membership_types_hash[mt.id] = {:allow_multiple_memberships => mt.allow_multiple_memberships?,:formatted_ends_at => I18n.l(mt.ends_at, :format => :date_for_input)}}
+
+    @pass_types = current_organization.pass_types.all
+  end
+
+  def update
+    handler = OrderHandler.new(current_sales_console_cart, nil)
+    handler.handle(params, current_organization)
+    flash[:alert] = handler.error unless handler.error.blank?
+    redirect_to new_console_sale_path(:person_id => @person.try(:id))
+  end
+
+  def destroy
+    current_sales_console_cart.clear!
+    redirect_to new_console_sale_path(:person_id => @person.try(:id))
+  end
+
+  def create
+    unless payment_method_is_vaild? &&
+           membership_and_pass_requirements_met &&
+           member_is_valid?
+      
+      redirect_to new_console_sale_path(:person_id => @person.try(:id)) and return
+    end   
+
+    @checkout = ConsoleSale::Checkout.new(current_sales_console_cart, payment, params[:order_notes])
+    
+    if @checkout.valid? && @checkout.finish
+      @order = @checkout.order
+      flash[:notice] = success_flash_message(@order, @checkout.person.email)
+      redirect_to new_console_sale_path
+    else      
+      flash[:error] = @checkout.message
+      redirect_to new_console_sale_path(:person_id => @person.try(:id))
+    end 
+  rescue Exception => e
+    checkout_error(e)   
+  end
+
+  def events
+    @event = current_organization.events.find(params[:event_id])
+    @shows = @event.upcoming_shows(:all)
+    render :json => @shows
+  end
+
+  def shows
+    @show = current_organization.shows.find(params[:show_id])
+    @ticket_types = @show.chart.sections.first.ticket_types_for(nil)
+    render :partial => "shows"
+  end
+
+  private
+    def checkout_error(e = nil)
+      Exceptional.context(:params => filter(params))
+      unless e.nil? 
+        Exceptional.handle(e, "Sales console checkout")
+        Rails.logger.error(e.backtrace)
+        Rails.logger.error(e.message)
+      end
+      flash[:error]  = "We're sorry but we could not process the sale.  Please make sure all fields are filled out accurately"
+      redirect_to new_console_sale_path(:person_id => @person.try(:id)) 
+    end
+
+    def membership_and_pass_requirements_met
+      if (current_sales_console_cart.memberships.any? || current_sales_console_cart.passes.any?)
+        unless params[:customer][:email].present?
+          flash[:error] = "Sorry, we can't sell memberships or passes to patrons without an email."
+          return false
+        end
+      end
+      true
+    end
+
+    def payment_method_is_vaild?
+      if params[:payment_method].blank?
+        flash[:error] = "Please pick a payment method."
+        return false
+      end
+
+      #Would be better to have the items themselves declare which paryment methods are valid
+      if current_sales_console_cart.donations.any? && payment.is_a?(CompPayment)
+        flash[:error] = "Sorry, we can't process this comp because we can't comp donations."
+        return false
+      end
+
+      true
+    end
+
+    #
+    # Returns false if there are member tickets in the cart and 
+    # the email provided is not a valid member
+    #
+    def member_is_valid?
+      member_tickets = current_sales_console_cart.tickets.select{|t| t.ticket_type.member_ticket == true}
+      if member_tickets.any?
+        member_email = payment.customer.email
+        if member_email.nil?
+          flash[:error] = "Sorry!"
+          return false
+        end
+
+        member = Member.where(:organization_id => current_user.current_organization)
+                       .where(:email => member_email)
+                       .first
+
+        if member.nil?
+          flash[:error] = "Sorry, we can't sell these tickets to non-member."
+          return false
+        end
+
+        member_tickets.each do |member_ticket|
+          membership_type = member_ticket.ticket_type.membership_type
+          if !member.current_membership_types.include? membership_type
+            flash[:error] = "Sorry, the member #{member_email} isn't eligible to purchase these tickets."
+            return false
+          end
+        end
+      end
+
+      true
+    end
+
+    def success_flash_message(order, email)
+      str = "Sale processed."
+      str += "<br/>Order number is #{@order.id}. <a target='_blank' href='#{Rails.application.routes.url_helpers.order_path(@order)}'>Click here to view the order</a>."
+
+      if @checkout.person.email.blank?
+        str += "<br/>No confirmation email was sent because we don't have an email on file for this patron."  
+      else
+        str += "<br/>A confirmation email was sent to #{@checkout.person.email}."
+      end
+      str
+    end
+
+    def verify_person
+      @person_id = params[:person_id]
+      if @person_id.present?
+        @person = Person.find(@person_id)
+        if @person.organization_id != current_organization.id
+          raise CanCan::AccessDenied 
+        end
+      end
+    end
+
+    def person
+      return @person unless @person.nil?
+      Person.first_or_create(person_attributes)
+    end
+
+    def person_attributes
+      {
+        :id                => params[:customer][:id],
+        :first_name        => params[:customer][:first_name],
+        :last_name         => params[:customer][:last_name],
+        :email             => params[:customer][:email],
+        :phones_attributes => params[:customer][:phones_attributes],
+        :organization      => current_organization
+      }
+    end
+
+    def payment
+      @payment ||= Payment.create(params[:payment_method], params)
+      @payment
+    end
+
+    def filter(params)
+      filters = Rails.application.config.filter_parameters
+      f = ActionDispatch::Http::ParameterFilter.new filters
+      f.filter params
+    end
+end

data/app/controllers/events_controller.rb

@@ -124,7 +124,7 @@ class EventsController < ArtfullyOseController
         if user_just_uploaded_an_image?
           messages_event_path(@event)
         elsif user_set_special_instructions?
-          event_shows_path(@event)
+          new_event_show_path(@event)
         else
           edit_event_venue_path(@event)
         end

data/app/controllers/events_pass_types_controller.rb

@@ -32,6 +32,11 @@ class EventsPassTypesController < ArtfullyOseController
       ept.active = params[:events_pass_type][:active]
     end
 
+    if @events_pass_type.ticket_types.empty?
+      flash[:error] = "Please select at least one Ticket Type for this pass."
+      redirect_to new_event_events_pass_type_path(@event) and return
+    end
+
     if @events_pass_type.save
       flash[:notice] = "Your pass has been attached to this event."
     else
@@ -51,6 +56,11 @@ class EventsPassTypesController < ArtfullyOseController
     @events_pass_type.excluded_shows = Set.new(params[:events_pass_type][:excluded_shows].reject!(&:blank?))
     @events_pass_type.active = params[:events_pass_type][:active]
 
+    if @events_pass_type.ticket_types.empty?
+      flash[:error] = "Please select at least one Ticket Type for this pass."
+      redirect_to new_event_events_pass_type_path(@event) and return
+    end
+
     if @events_pass_type.save
       flash[:notice] = "Your pass has been updated."
     else

data/app/controllers/memberships_controller.rb

@@ -2,13 +2,13 @@ class MembershipsController < ArtfullyOseController
   before_filter :load_tags, :only => [:index]
 
   def index
-    @person = Person.find(params[:person_id])
+    @person = current_organization.people.find(params[:person_id])
     @expired_count = @person.memberships.lapsed.count
     @membership_types = current_organization.membership_types
   end
 
   def bulk_update
-    @person = Person.find(params[:person_id])
+    @person = current_organization.people.find(params[:person_id])
     extend_memberships(params)
     redirect_to person_memberships_path(@person)
   end
@@ -20,7 +20,7 @@ class MembershipsController < ArtfullyOseController
       #I hate how these are tied to the button text
       if params[:commit].eql? "Change Expiration"
         params[:membership_ids].each do |membership_id|
-          Membership.find(membership_id).adjust_expiration_to(params[:ends_at])
+          current_organization.memberships.find(membership_id).adjust_expiration_to(params[:ends_at])
         end
         flash[:notice] = "Memberships have been adjusted."
       end

data/app/controllers/people_controller.rb

@@ -18,6 +18,7 @@ class PeopleController < ArtfullyOseController
     @person.company_name     = person[:company_name]      if person[:company_name].present?
     @person.subscribed_lists = person[:subscribed_lists]  if person[:subscribed_lists].present?
     @person.do_not_email     = person[:do_not_email]      if person[:do_not_email].present?
+    @person.do_not_call      = person[:do_not_call]       if person[:do_not_call].present?
     @person.type             = person[:type] || "Individual"
     @person.subtype          = person[:subtype] || "Individual"
     @person.organization_id  = current_user.current_organization.id
@@ -61,7 +62,7 @@ class PeopleController < ArtfullyOseController
           flash[:notice] << "Your changes have been saved"
         else
           errs = [@person.errors.delete(:"relationships.base")].flatten + [@person.errors.full_messages.to_sentence].flatten
-          flash[:alert] = errs.blank? ? "Sorry, we couldn't save your changes. Make sure you entered a first name, last name or email address." : errs
+          flash[:alert] = errs.blank? ? "Sorry, we couldn't save your changes. Make sure you entered a first name, last name or email address." : errs.compact
         end
         redirect_to_person(@person, params)
       end

data/app/controllers/sales_controller.rb

@@ -1,10 +1,10 @@
 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
@@ -12,14 +12,39 @@ class SalesController < ArtfullyOseController
     @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
 
   def create
     current_box_office_cart.clear!
-    @sale = Sale.new(@show, @show.chart.ticket_types.box_office, current_box_office_cart, params[:quantities])
+
+    @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
 
@@ -27,19 +52,21 @@ class SalesController < ArtfullyOseController
       @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 :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
@@ -61,6 +88,11 @@ 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
@@ -120,11 +152,12 @@ class SalesController < ArtfullyOseController
 
     def person_attributes
       {
-        :id           =>  params[:person][:id],
-        :first_name   =>  params[:person][:first_name],
-        :last_name    =>  params[:person][:last_name],
-        :email        =>  params[:person][:email],
-        :organization =>  current_organization
+        :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
 
@@ -146,7 +179,12 @@ class SalesController < ArtfullyOseController
 
       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

data/app/controllers/shows_controller.rb

@@ -1,5 +1,5 @@
 class ShowsController < ArtfullyOseController
-  before_filter :find_event, :only => [ :index, :calendar, :show, :new, :edit, :duplicate, :on_sale, :off_sale ]
+  before_filter :find_event, :only => [ :index, :calendar, :show, :new, :edit, :duplicate, :on_sale, :off_sale, :destroy ]
   before_filter :check_for_charts, :only => [ :index, :new ]
 
   rescue_from CanCan::AccessDenied do |exception|
@@ -119,7 +119,7 @@ class ShowsController < ArtfullyOseController
       end
     else
       flash[:error] = "Sorry, this show can not be deleted. Contact support for further help."
-      redirect_to show_path(@show)
+      redirect_to event_path(@event)
     end
   end
   

data/app/controllers/store/orders_controller.rb

@@ -51,15 +51,6 @@ class Store::OrdersController < Store::StoreController
 
   private
 
-    #
-    # The storefront javascript depends on this total_string to determine 
-    # if this is a free order or not. Don't re-word this string without also
-    # addressing the javascript in store.js
-    #
-    def total_string
-      "#{pluralize(current_cart.items.length, 'item')}, #{number_as_cents current_cart.total}"
-    end
-
     def event
       current_cart.tickets.first.event
     end

data/app/helpers/artfully_ose_helper.rb

@@ -129,7 +129,7 @@ module ArtfullyOseHelper
 
   def amount_and_nongift(item)
     str = number_as_cents item.total_price
-    str += " (#{number_as_cents item.nongift_amount} Non-deductible)" unless item.nongift_amount.nil?
+    str += " (#{number_as_cents item.nongift_amount} Non-deductible)" unless item.nongift_amount.nil? || item.nongift_amount == 0
     str
   end
   
@@ -139,7 +139,9 @@ module ArtfullyOseHelper
   end
 
   def number_as_cents(cents, options = {})
-    number_to_currency(number_to_dollars(cents), options)
+    result = number_to_currency(number_to_dollars(cents), options)
+    result = result.split('.').first if result.split('.').last == '00' && options[:cents_if_needed] == true
+    result
   end
 
   def sorted_us_state_names