artfully_ose 1.2.0.beta.1 → 1.2.0.pre

data/app/models/section.rb

@@ -53,22 +53,8 @@ class Section < ActiveRecord::Base
     end
   end
 
-  def ticket_types_for(member = nil)
-    types = []
-
-    # Add storefront types regardless of anything
-    types << ticket_types.storefront
-    unless member.nil?
-
-      #Add "unrestricted member" ticket_types that aren't tied to a particular membership type
-      types << ticket_types.members.select{|ticket_type| !ticket_type.member_ticket?} unless member.nil?
-
-      #Now add member ticket_types that apply to this particular member
-      membership_type_ids = member.nil? ? [] : member.current_membership_types.collect(&:id)
-      types << ticket_types.select{|ticket_type| ticket_type.members && membership_type_ids.include?(ticket_type.membership_type_id) }
-    end
-
-    types.flatten.uniq
+  def ticket_types_for(member)
+    member.nil? ? ticket_types.storefront : ticket_types.members
   end
 
   def dup!    
@@ -82,14 +68,12 @@ class Section < ActiveRecord::Base
     tickets.off_sale.limit(qty).each do |t|
       t.put_on_sale
     end
-    show.refresh_stats
   end
   
   def take_off_sale(qty = 0)
     tickets.on_sale.limit(qty).each do |t|
       t.take_off_sale
     end
-    show.refresh_stats
   end
   
   def available

data/app/models/show.rb

@@ -1,15 +1,13 @@
 class Show < ActiveRecord::Base
-  include Ticket::Foundry
+  include Ticket::Foundry  
   include Ticket::Reporting
   include ActiveRecord::Transitions
   include Ext::Resellable::Show
   include Ext::Integrations::Show
   include Ext::Uuid
 
-  attr_accessible :datetime, :event_id, :chart_id, :organization_id, :old_mongo_id, :cached_stats
-  store :cached_stats, :accessors => [ :capacity, :on_sale, :off_sale, :open, :sold, :comped, :local_datetime, :offset, :time_zone, :iana_time_zone ]
-
-
+  attr_accessible :datetime, :event_id, :chart_id, :organization_id, :old_mongo_id
+  
   belongs_to :organization
   belongs_to :event
   belongs_to :chart, :autosave => true
@@ -19,7 +17,7 @@ class Show < ActiveRecord::Base
   has_many :settlements
   has_many :items
   has_many :ticket_types
-
+  
   before_destroy  :destroyable?
   after_create    :update_ticket_types
 
@@ -46,7 +44,7 @@ class Show < ActiveRecord::Base
   delegate :free?, :to => :event
 
   state_machine do
-
+    
     #pending and built are deprecated, left in only because we have shows in production which are built
     state :pending
     state :built, :exit => :create_and_on_sale_tickets
@@ -62,54 +60,12 @@ class Show < ActiveRecord::Base
   def go!(and_publish = true)
     return false if !valid?
     transaction do
-      build!
+      build!    
       and_publish ? publish! : unpublish!
       save
     end
   end
 
-  def applies_to_pass?(pass)
-    epts = EventsPassType.active
-                  .where(:organization_id => self.organization.id)
-                  .where(:event_id => self.event.id)
-                  .where(:pass_type_id => pass.pass_type.id).first
-
-    return false if epts.blank?
-
-    return !epts.excluded_shows.include?(self.id)
-  end
-
-  #
-  # Run synchronously only with great care. 
-  #
-  def refresh_stats
-    tickets.reload
-
-    self.local_datetime = self.datetime_local_to_event.to_s unless self.event.nil?
-
-    self.capacity = self.glance.available.capacity
-    self.on_sale  = self.glance.available.on_sale
-    self.off_sale = self.glance.available.off_sale
-    self.sold     = self.glance.sold.total
-    self.comped   = self.glance.comped.total
-    self.open     = self.glance.available.open
-
-    unless self.event.time_zone.nil?
-      self.offset         = ActiveSupport::TimeZone.create(self.event.time_zone).parse(self.local_datetime).formatted_offset
-    end
-    
-    self.time_zone      = self.event.time_zone
-    self.iana_time_zone = ActiveSupport::TimeZone::MAPPING[self.event.time_zone]
-
-    self.save(:validate => false)
-    self.event.delay.refresh_stats
-    self.cached_stats
-  end
-
-  def parsed_local_datetime
-    @parsed_local_datetime = (DateTime.parse(self.local_datetime) rescue self.datetime_local_to_event)
-  end
-
   def create_and_on_sale_tickets
     create_tickets
     bulk_on_sale(:all)
@@ -118,11 +74,11 @@ class Show < ActiveRecord::Base
   def unscoped_event
     ::Event.unscoped.find(event_id)
   end
-
+  
   def imported?
     unscoped_event.imported?
   end
-
+  
   def event_deleted?
     !unscoped_event.deleted_at.nil?
   end
@@ -143,10 +99,6 @@ class Show < ActiveRecord::Base
     @tickets_comped ||= tickets.select { |ticket| ticket.comped? }
   end
 
-  def tickets_validated
-    tickets.where(:validated => true).count
-  end
-
   def to_s
     show_time
   end
@@ -159,6 +111,10 @@ class Show < ActiveRecord::Base
     published? or unpublished?
   end
 
+  def time_zone
+    @time_zone ||= event.time_zone
+  end
+
   def load(attrs)
     super(attrs)
     set_attributes(attrs)
@@ -192,8 +148,8 @@ class Show < ActiveRecord::Base
   # For a single show being displayed in the widget
   #
   def as_widget_json(options = {})
-    as_json.merge(:event => event.as_json,
-                  :venue => event.venue.as_json,
+    as_json.merge(:event => event.as_json, 
+                  :venue => event.venue.as_json, 
                   :chart => chart_for("storefront", options[:organization_id]).as_json(options))
   end
 
@@ -231,18 +187,9 @@ class Show < ActiveRecord::Base
     settleables
   end
 
-  #
-  # Horribly inefficient. Don't use with a list of shows.
-  #
   def destroyable?
     (tickets_comped + tickets_sold).empty? && items.empty?
   end
-  
-  def delayed_destroy
-    return false unless destroyable?
-    Delayed::Job.enqueue(DestroyShowJob.new(self))
-    true
-  end
 
   def live?
     (tickets_comped + tickets_sold).any?
@@ -268,6 +215,12 @@ class Show < ActiveRecord::Base
     tickets.select(&:compable?)
   end
 
+  def as_widget_json(options = {})
+    as_json.merge(:event => event.as_json, 
+                  :venue => event.venue.as_json, 
+                  :chart => chart_for("storefront", options[:organization_id]).as_json(options))
+  end
+
   def sections_for(member)
     member.nil? ? self.chart.sections.storefront : self.chart.sections.members
   end

data/app/models/slices.rb

@@ -12,39 +12,25 @@ class Slices
                  :ticket_type_proc, 
                  :order_location_proc,
                  :discount_code_proc,
-                 :first_time_buyer_proc,
-                 :validated_proc
+                 :first_time_buyer_proc
 
   self.payment_method_proc = Proc.new do |items|
     payment_method_map = {}
     items.each do |item|
-      next if item.refund? || item.refunded? || item.exchanged?
       item_array = payment_method_map[item.order.payment_method]
       item_array ||= []
-      item_array << item
+      item_array << item unless item.refund? || item.refunded? || item.exchanged?
       payment_method_map[item.order.payment_method] = item_array
     end
     payment_method_map.delete_if{ |k,v| v.empty? }
   end
 
-  self.validated_proc = Proc.new do |items|
-    validated_map = {}
-    items.each do |item|
-      next if item.refund? || item.refunded? || item.exchanged?
-      kee = (item.product.validated? ? "VALIDATED" : "NOT VALIDATED")
-      item_array = validated_map.fetch(kee, [])
-      item_array << item
-      validated_map[kee] = item_array
-    end
-    validated_map.delete_if{ |k,v| v.empty? }
-  end
-
   self.ticket_type_proc = Proc.new do |items|
     ticket_type_map = {}
     items.each do |item|
-      next if item.refund? || item.refunded? || item.exchanged?
-      item_array = ticket_type_map.fetch(item.product.ticket_type.name, [])
-      item_array << item
+      item_array = ticket_type_map[item.product.ticket_type.name]
+      item_array ||= []
+      item_array << item unless item.refund? || item.refunded? || item.exchanged?
       ticket_type_map[item.product.ticket_type.name] = item_array
     end
     ticket_type_map.delete_if{ |k,v| v.empty? }
@@ -53,11 +39,10 @@ class Slices
   self.order_location_proc = Proc.new do |items|
     order_location_map = {}
     items.each do |item|
-      next if item.refund? || item.refunded? || item.exchanged?
       order_location = item.order.original_order.location
       item_array = order_location_map[order_location]
       item_array ||= []
-      item_array << item
+      item_array << item unless item.refund? || item.refunded? || item.exchanged?
       order_location_map[order_location] = item_array
     end
     order_location_map.delete_if{ |k,v| v.empty? }
@@ -66,11 +51,10 @@ class Slices
   self.discount_code_proc = Proc.new do |items|
     discounts_code_map = {}
     items.each do |item|
-      next if item.refund? || item.refunded? || item.exchanged?
       code = item.discount.try(:code) || "NO DISCOUNT"
       item_array = discounts_code_map[code]
       item_array ||= []
-      item_array << item 
+      item_array << item unless item.refund? || item.refunded? || item.exchanged?
       discounts_code_map[code] = item_array
     end
     discounts_code_map.delete_if{ |k,v| v.empty? }
@@ -82,7 +66,6 @@ class Slices
   self.first_time_buyer_proc = Proc.new do |items|
     first_time_buyer_map = {}
     items.each do |item|
-      next if item.refund? || item.refunded? || item.exchanged?
       previous_action = GetAction.where(:person_id => item.order.person.id)
                                  .where('occurred_at < ?', item.order.created_at)
                                  .first
@@ -90,7 +73,7 @@ class Slices
       kee = (previous_action.nil? ? "FIRST" : "RETURNING")
       item_array = first_time_buyer_map[kee]
       item_array ||= []
-      item_array << item
+      item_array << item unless item.refund? || item.refunded? || item.exchanged?
       first_time_buyer_map[kee] = item_array
     end
     first_time_buyer_map.delete_if{ |k,v| v.empty? }

data/app/models/statement.rb

@@ -26,8 +26,7 @@ class Statement
                 :payment_method_rows,
                 :order_location_rows,
                 :ticket_type_rows,
-                :discount_rows,
-                :pass_rows
+                :discount_rows
   
   def self.for_show(show, imported=false)
     if show.nil?
@@ -82,7 +81,6 @@ class Statement
       end
 
       statement.build_discount_rows(show.items)
-      statement.build_pass_rows(show.items)
       statement.build_ticket_type_rows(show, show.items)
     end
   end
@@ -94,13 +92,13 @@ class Statement
     self.ticket_type_rows         = {}
 
     show.chart.ticket_types.each do |ticket_type|
-      self.ticket_type_rows[ticket_type.name] = TicketTypeRow.new(ticket_type)
+      self.ticket_type_rows[ticket_type.name] = TicketTypeRow.new(ticket_type.name)
     end
 
     items.each do |item|
       #refunded items now have nil ticket types
       unless item.product.ticket_type.nil?
-        row = self.ticket_type_rows[item.product.ticket_type.name] || TicketTypeRow.new(item.product.ticket_type)
+        row = self.ticket_type_rows[item.product.ticket_type.name] || TicketTypeRow.new(item.product.ticket_type.name)
         row << item
         self.ticket_type_rows[item.product.ticket_type.name] = row
       end
@@ -118,17 +116,6 @@ class Statement
       end
     end
   end
-
-  def build_pass_rows(items)
-    self.pass_rows         = {}
-    items.each do |item|
-      unless item.pass.nil?
-        row = self.pass_rows[item.pass.pass_type.name] || PassRow.new(item.pass.pass_type.name)
-        row << item
-        self.pass_rows[item.pass.pass_type.name] = row
-      end
-    end
-  end
   
   module Row
     attr_accessor :tickets, 
@@ -153,11 +140,10 @@ class Statement
   
   class TicketTypeRow
     include Row
-    attr_accessor   :ticket_type, :member_ticket
+    attr_accessor   :ticket_type
     
     def initialize(ticket_type)
-      self.ticket_type = ticket_type.name
-      self.member_ticket = ticket_type.member_ticket
+      self.ticket_type = ticket_type
       self.tickets = 0
       self.gross = 0
       self.processing = 0
@@ -180,19 +166,6 @@ class Statement
     end
   end
   
-  class PassRow
-    include Row
-    attr_accessor   :pass_type_name
-    
-    def initialize(pass_type_name)
-      self.pass_type_name = pass_type_name
-      self.tickets = 0
-      self.gross = 0
-      self.processing = 0
-      self.net = 0
-    end
-  end
-  
   class OrderLocationRow
     include Row
     attr_accessor   :order_location

data/app/models/ticket.rb

@@ -6,11 +6,8 @@
   include Ticket::Pricing
   include Ticket::Transfers
   include Ticket::SaleTransitions
-  include Ext::Uuid
   extend ActionView::Helpers::TextHelper
-
-  class TicketAlreadyValidated < StandardError; end
-
+  
   attr_accessible :section_id, :section, :venue, :cart_price
 
   belongs_to :buyer, :class_name => "Person"
@@ -18,9 +15,8 @@
   belongs_to :organization
   belongs_to :section
   belongs_to :cart
-  belongs_to :pass
   belongs_to :discount
-  belongs_to :action, :foreign_key => "validated_action_id", :class_name => "GoAction"
+  belongs_to :action, :foreign_key => "validated_action_id",:class_name => "GoAction"
 
   #
   # This refs the ticket_type that the ticket was sold under, NOT an array of ticket types available
@@ -29,10 +25,6 @@
   
   has_many :items, :as => :product
 
-  has_attached_file :qr_code, TICKET_QR_STORAGE
-
-  delegate :url, :to => :qr_code, :prefix => true
-
   delegate :event, :to => :show
   def self.sold_after(datetime)
     sold.where("sold_at > ?", datetime)
@@ -148,10 +140,6 @@
     0
   end
 
-  def realized_fee
-    self.class.realized_fee
-  end
-
   def expired?
     datetime < DateTime.now
   end
@@ -209,14 +197,9 @@
     end
     
     result = Ticket.import(new_tickets)
-    show.refresh_stats
     result
   end
   
-  #
-  # This method does not refresh show stats
-  # Callers should do that manually with show.delay.refresh_stats
-  #
   def self.build_one(show, section, quantity, on_sale = false)
     t = Ticket.new({
       :venue => show.event.venue.name,
@@ -224,45 +207,27 @@
     })
     t.show = show
     t.organization = show.organization
-    t.set_uuid
     t.state = 'on_sale' if on_sale
     t
   end
 
-  def generate_qr_code
-    file = Tempfile.new(['qr-code', '.png'])
-    QRCode.new(self, sold_item.try(:order)).render(file)
-    self.qr_code = file
-
-    # If we don't save the ticket here, paperclip will leak a file handle.
-    # Even though we close our Tempfile below, paperclip copies that into
-    # another Tempfile, and only closes the handle when the ticket is saved.
-    save
-
-    file.close
-  end
-  handle_asynchronously :generate_qr_code
-
-
-  def validate_ticket!(user = nil)
-    if committed? && !validated?
-      Ticket.transaction do
-        self.action = GoAction.for(self.show, self.buyer, Time.now) do |go_action|
-          go_action.creator = user
-        end
-        self.validated = true
-        save
+  #
+  # Can't name this validate for obvious reasons
+  #
+  def validate_ticket(user = nil)
+    transaction do
+      go_action = GoAction.for(self.show, self.buyer, Time.now) do |go_action|
+        go_action.creator = user
+        go_action.save
       end
+      self.update_attributes({:validated_action_id => go_action.id, :validated => true}, :without_protection => true)
     end
   end
 
-  def unvalidate_ticket!
-    if validated?
-      Ticket.transaction do
-        action.destroy unless self.action.tickets.where(validated:true).count > 1
-        self.validated = false
-        save
-      end
+  def unvalidate_ticket
+    transaction do
+      self.action.destroy unless self.action.tickets.count > 1
+      self.update_attributes({:validated_action_id => nil, :validated => false}, :without_protection => true)
     end
   end
 end