artfully_ose 1.2.0 → 1.3.0.pre1

data/app/models/membership_comp.rb

@@ -15,6 +15,7 @@ class MembershipComp
                 :person_id,
                 :segment_id,
                 :search_id,
+                :advanced_search_id,
                 :organization, 
                 :membership_type, 
                 :number_of_memberships, 
@@ -33,14 +34,9 @@ class MembershipComp
     false
   end
 
-  def people_ids
-    people.nil? ? [] : people.collect(&:id)
-  end
-
   def people_belong_to_this_org
     people.each do |person|      
-      Rails.logger.info "FLARG #{person.organization.id} #{organization}"
-      if person.organization != organization
+      if person.organization_id != organization.id
         errors.add(:base, "Person does not belong to this organization. Please contact Artful.ly support if this problem persists.")
       end
     end
@@ -57,6 +53,30 @@ class MembershipComp
     @errors = nil
   end
 
+  #
+  # TODO: This is a colossal memory/performance hit if @ids.length > a few hundred
+  # refactor this controller/model to not deal with the actual 
+  # people objects unless absolutely necessary
+  #
+  def find_people
+
+    return self.people unless self.people.blank?
+
+    if !self.person_id.blank?
+      self.people = Array.wrap(Person.find(self.person_id))
+    elsif !self.segment_id.blank?
+      self.people = self.organization.segments.where(:id => self.segment_id).first.people
+    elsif !self.search_id.blank?
+      self.people = self.organization.searches.where(:id => self.search_id).first.people
+    elsif !self.advanced_search_id.blank?
+      self.people = self.organization.advanced_searches.where(:id => self.advanced_search_id).first.people
+    else
+      self.people = []
+    end
+
+    self.people
+  end
+
   def we_have_some_people
     if people.blank?
       errors.add(:base, "You haven't selected any people to receive memberships")
@@ -82,24 +102,29 @@ class MembershipComp
   end
 
   def perform
-    self.people.each do |person|
-      next if person.email.blank?
-      next if person.company?
-      memberships = []
-      self.number_of_memberships.to_i.times do
-        membership                  = Membership.for(self.membership_type)
-        membership.ends_at          = self.ends_at
-        membership.sold_price       = 0
-        membership.total_paid       = 0
-        membership.welcome_message  = self.welcome_message
-        membership.send_email       = self.send_email
-        membership.save
-        memberships << membership
-      end
+    find_people
+
+    ActiveRecord::Base.transaction do
+      self.people.each do |person|
+        next if person.email.blank?
+        next if person.company?
+        next if person.organization_id != self.organization.id
+        memberships = []
+        self.number_of_memberships.to_i.times do
+          membership                  = Membership.for(self.membership_type)
+          membership.ends_at          = self.ends_at
+          membership.sold_price       = 0
+          membership.total_paid       = 0
+          membership.welcome_message  = self.welcome_message
+          membership.send_email       = self.send_email
+          membership.save
+          memberships << membership
+        end
 
-      comp = Comp.new(nil, [], memberships, person, self.benefactor)
-      comp.notes = self.notes
-      comp.submit
+        comp = Comp.new([], memberships, [], person, self.benefactor)
+        comp.notes = self.notes
+        comp.submit
+      end
     end
   end
 end

data/app/models/membership_comp_job.rb

@@ -14,6 +14,13 @@ class MembershipCompJob < Struct.new(:membership_comp)
     membership_comp.clear_errors
     job.membership_comp = membership_comp
 
+    #
+    # We can't serialize all the people objects because we run the risk of overrunnng
+    # the length of MySql's TEXT field. MembershipComp will reload the people
+    # when this job runs
+    #
+    membership_comp.people = nil
+
     if run_now?
       job.perform
     else

data/app/models/membership_type.rb

@@ -11,6 +11,7 @@ class MembershipType < ActiveRecord::Base
   has_many :memberships
   has_many :members, :through => :memberships
   belongs_to :segment
+  belongs_to :advanced_search_segment
 
   after_create :create_list_segment
   
@@ -19,7 +20,7 @@ class MembershipType < ActiveRecord::Base
   scope :storefront, where(:on_sale => true)
   scope :on_sale, where(:on_sale => true)
   scope :sales_valid, where("sales_start_at < ? or sales_start_at is null", DateTime.now).where("sales_end_at > ? or sales_end_at is null", DateTime.now)
-  scope :not_ended, where('ends_at > ?', DateTime.now)
+  scope :not_ended, where('ends_at > ? or ends_at is null', DateTime.now)
 
   comma do
     name
@@ -40,7 +41,7 @@ class MembershipType < ActiveRecord::Base
   end
 
   def self.in_play(organization)
-    self.find((organization.membership_types.not_ended.pluck(:id).uniq + organization.memberships.current.select(:membership_type_id).uniq.pluck(:membership_type_id)))
+    self.find((organization.membership_types.not_ended.sales_valid.pluck(:id).uniq + organization.memberships.current.select(:membership_type_id).uniq.pluck(:membership_type_id)))
   end
 
   def membershipize
@@ -54,13 +55,18 @@ class MembershipType < ActiveRecord::Base
   end
 
   def create_list_segment
-    @search = organization.searches.create({:membership_type => self})
-    @segment = organization.segments.create({:search => @search, :name => self.name})
-    self.segment = @segment
-    save
+    advanced_search  = AdvancedSearch.for_membership_type(self)
+    advanced_search_segment = AdvancedSearchSegment.create(:advanced_search_id => advanced_search.id, :organization_id => self.organization.id, :name => self.name)
+    self.advanced_search_segment = advanced_search_segment
+    self.save!
+
+    search = organization.searches.create({:membership_type => self})
+    segment = organization.segments.create({:search => search, :name => self.name})
+    self.segment = segment
+    self.save!
   end
 
   def allow_multiple_memberships?
     self.limit_per_transaction > 1
   end
-end
+end

data/app/models/monitorable.rb

@@ -0,0 +1,19 @@
+module Monitorable
+  attr_accessor :job_monitor
+
+  def error(job, exception)
+    if self.job_monitor.present?
+      self.job_monitor.failed = true
+      Airbrake.notify(Exception.new("Monitored job failed"), :parameters => {:job_monitor => self.job_monitor.id})
+      self.job_monitor.save
+    end
+  end
+
+  def failure(job)
+    if self.job_monitor.present?
+      self.job_monitor.failed = true
+      Airbrake.notify(Exception.new("Monitored job failed"), :parameters => {:job_monitor => self.job_monitor.id})
+      self.job_monitor.save
+    end
+  end
+end

data/app/models/note.rb

@@ -1,8 +1,17 @@
 class Note < ActiveRecord::Base
-  attr_accessible :occurred_at, :text
+  attr_accessible :occurred_at, :text, :organization_id
   
   belongs_to :person, touch: true
   belongs_to :user
   belongs_to :organization
+
+  searchable do
+    text :text
+    integer :person_id, stored: true
+    string :organization_id
+  end
+  include Ext::DelayedIndexing
+  
+  set_watch_for :occurred_at, :local_to => :organization
 end
 

data/app/models/order.rb

@@ -10,7 +10,7 @@ class Order < ActiveRecord::Base
   include Ext::Integrations::Order
   include OhNoes::Destroy
   include ArtfullyOseHelper
-  
+
   #This is a lambda used to by the items to calculate their net
   attr_accessor :per_item_processing_charge
 
@@ -21,13 +21,16 @@ class Order < ActiveRecord::Base
   belongs_to :import
   belongs_to :parent, :class_name => "Order", :foreign_key => "parent_id"
   belongs_to :gateway_transaction, :primary_key => :transaction_id, :foreign_key => :transaction_id
+  belongs_to :creator, :class_name => "User", :foreign_key => "creator_id"
   has_many :children, :class_name => "Order", :foreign_key => "parent_id"
   has_many :items, :dependent => :destroy
   has_many :actions, :foreign_key => "subject_id", :dependent => :destroy
+  has_many :scheduled_pledge_payments, :dependent => :destroy
+  has_one :donation, :dependent => :destroy
 
   attr_accessor :skip_actions, :skip_email
 
-  set_watch_for :created_at, :local_to => :organization  
+  set_watch_for :created_at, :local_to => :organization
   set_watch_for :created_at, :local_to => :self, :as => :admins
 
   validates_presence_of :person_id
@@ -39,6 +42,8 @@ class Order < ActiveRecord::Base
   before_save   :calculate_when_revenue_applies
   after_save    :calculate_lifetime_value
   after_destroy :calculate_lifetime_value
+  after_save { person.solr_index! }
+  after_destroy { person.solr_index! }
 
   default_scope :order => 'orders.created_at DESC'
   scope :before, lambda { |time| where("orders.created_at < ?", time) }
@@ -48,6 +53,7 @@ class Order < ActiveRecord::Base
   scope :csv_imported, where("import_id IS NOT NULL")
   scope :csv_not_imported, where("import_id IS NULL")
   scope :artfully, where("transaction_id IS NOT NULL")
+  scope :pledge, lambda{|is_pledge = true| joins('LEFT JOIN donations ON donations.order_id = orders.id').where('donations.pledge'=> is_pledge) }
 
   has_attached_file :pdf, ORDER_PDF_STORAGE
 
@@ -59,6 +65,26 @@ class Order < ActiveRecord::Base
         person.send(person_field) unless person.nil?
       end
     end
+    
+    text :full_name do
+      person.to_s
+    end
+    
+    text :clean_first_name do
+      person.first_name.gsub(/[^a-zA-Z0-9]/, '') unless person.first_name.nil?
+    end
+    
+    text :clean_middle_name do
+      person.middle_name.gsub(/[^a-zA-Z0-9]/, '') unless person.middle_name.nil?
+    end
+    
+    text :clean_last_name do
+      person.last_name.gsub(/[^a-zA-Z0-9]/, '') unless person.last_name.nil?
+    end
+    
+    text :clean_full_name do
+      person.to_s.gsub(/[^a-zA-Z0-9]/, '')
+    end
 
     text :organization_id do
       organization.id
@@ -82,8 +108,13 @@ class Order < ActiveRecord::Base
     end
 
     string :event_name, :multiple => true do
-      items.map{ |item| item.show.event.name unless item.show.nil? }
+      items.map{ |item| item.show.event.try(:name) unless item.show.nil? }
     end
+
+    date :created_at
+    integer :person_id, stored: true
+    integer :organization_id
+    integer :donation_amount, stored: true
   end
   include Ext::DelayedIndexing
 
@@ -109,10 +140,40 @@ class Order < ActiveRecord::Base
     items.first.try(:show)
   end
 
+  #
+  # 1 if realized_fee should be charged
+  # 0 otherwise
+  #
+  # This is used when setting realized_price on item
+  # at order creation time.
+  #
+  def realized_fee_modifier
+    1
+  end
+
+  #
+  # realized_fee is the fee charged to the producer by withholding it
+  # from any settlements.
+  #
+  # Note that thisis different from the (now poorly named) service_fee which is
+  # money we've charged to the patron at checkout time
+  #
+  def realized_fee
+    items.collect(&:realized_fee).sum
+  end
+
+  def total_fee
+    items.collect(&:total_fee).sum
+  end
+
   def service_fee
     items.collect(&:service_fee).sum
   end
 
+  def donation_amount
+    items.map {|item| ((item.price || 0) + item.nongift_amount) if item.product_type == 'Donation' }.compact.sum
+  end
+
   # Pass through methods to help out with the checkout thanks page
   def events
     tickets.collect(&:product).collect(&:event).uniq
@@ -125,7 +186,7 @@ class Order < ActiveRecord::Base
   def pass_codes
     tickets.collect(&:pass).uniq.compact
   end
-  
+
   def imported?
     !self.import_id.nil?
   end
@@ -155,15 +216,15 @@ class Order < ActiveRecord::Base
   def exchanges
     tickets.find_all(&:exchanged?)
   end
-  
+
   def artfully?
     !transaction_id.nil?
   end
-  
+
   def location
     self.class.location
   end
-  
+
   def self.location
     ""
   end
@@ -179,7 +240,7 @@ class Order < ActiveRecord::Base
   def nongift_amount
     all_items.inject(0) {|sum, item| sum + item.nongift_amount.to_i }
   end
-  
+
   def destroyable?
     ( (type.eql? "ApplicationOrder") || (type.eql? "ImportedOrder") ) && !is_fafs? && !artfully? && has_single_donation?
   end
@@ -196,13 +257,13 @@ class Order < ActiveRecord::Base
   def processor_class
     OrderProcessor
   end
-  
+
   def assignable?
     anonymous_purchase? && parent.nil?
   end
-  
+
   def editable?
-    ( (type.eql? "ApplicationOrder") || (type.eql? "ImportedOrder") ) && !is_fafs? && !artfully? && has_single_donation? 
+    ( (type.eql? "ApplicationOrder") || (type.eql? "ImportedOrder") ) && !is_fafs? && !artfully? && has_single_donation?
   end
 
   def for_organization(org)
@@ -210,7 +271,9 @@ class Order < ActiveRecord::Base
   end
 
   def <<(products)
-    self.items << Array.wrap(products).collect { |product|  Item.for(product, @per_item_processing_charge) }
+    products = Array.wrap(products)
+    self.items << products.collect { |product|  Item.for(product, self) }
+    self.donation = products[0] if products.present? && products.length == 1 && products[0].class.name == "Donation"
   end
 
   def payment
@@ -258,6 +321,10 @@ class Order < ActiveRecord::Base
     memberships.collect{|item| item.product.membership_type}.uniq
   end
 
+  def pass_types
+    passes.collect{|item| item.product.pass_type}.uniq
+  end
+
   def self.membership_sale_search(search)
     standard = ::Order.
       joins(:items).
@@ -290,7 +357,7 @@ class Order < ActiveRecord::Base
     standard = standard.where("pass_types.id = ?", search.pass_type.id) if search.pass_type if search.pass_type
     standard.all
   end
-  
+
   def has_single_donation?
     (donations.size == 1) && tickets.empty?
   end
@@ -312,6 +379,11 @@ class Order < ActiveRecord::Base
     items.select { |i| i.returnable? and i.comped? and not i.refundable? }
   end
 
+  def convertable_items
+    items.select(&:convertable?)
+  end
+
+
   def num_tickets
     all_tickets.size
   end
@@ -360,7 +432,7 @@ class Order < ActiveRecord::Base
       end
     end
   end
-  
+
   def to_comp!
     items.each do |item|
       item.to_comp!
@@ -378,7 +450,7 @@ class Order < ActiveRecord::Base
       "#{number_as_cents sum_donations} donation"
     end
   end
-  
+
   def ticket_summary
     summary = TicketSummary.new
     items.select(&:ticket?).each do |item|
@@ -414,13 +486,13 @@ class Order < ActiveRecord::Base
   def transaction_id
     read_attribute(:transaction_id) || self.parent.try(:transaction_id)
   end
-  
+
   def sell_tickets
     all_tickets.each do |item|
       item.product.sell_to(self.person, self.created_at)
     end
   end
-  
+
   def time_zone
     "Eastern Time (US & Canada)"
   end
@@ -437,6 +509,7 @@ class Order < ActiveRecord::Base
     self.person.calculate_lifetime_value
     self.person.calculate_lifetime_ticket_value
     self.person.calculate_lifetime_donations
+    self.person.calculate_lifetime_pledges
     self.person.calculate_lifetime_memberships
   end
 
@@ -461,7 +534,7 @@ class Order < ActiveRecord::Base
       action.person = person
       action.save!
     end
-  end  
+  end
 
   def create_donation_actions
     self.items.select(&:donation?).collect do |item|
@@ -502,19 +575,20 @@ class Order < ActiveRecord::Base
   #
   # For example, order.create_generic_action('passes'), Pass.to_sentence must be implemented
   #
-  def create_generic_action(collection_name)
+  def create_generic_action(collection_name, import_id = nil)
     action                  = self.action_class.new
     action.person           = self.person
     action.subject          = self
     action.organization     = self.organization
     action.details          = action_details(self.send(collection_name).collect(&:product)) + ". " + (self.notes || "")
     action.occurred_at      = self.created_at
+    action.import_id        = import_id if import_id.present?
 
     action.subtype          = action.subtype
     action.save!
-    action      
+    action
   end
-  
+
   def action_details(collection)
     klass = Kernel.const_get(collection.first.class.name)
     klass.to_sentence(collection)
@@ -533,6 +607,81 @@ class Order < ActiveRecord::Base
       end
     end
   end
+  
+  def printable_tickets
+    items = []
+    
+    self.tickets.select{|ticket| ticket.state != 'refunded' && ticket.state != 'exchanged'}.each do |item|
+      items << item
+    end
+    
+    self.children.each do |order|
+      order.items.select{|ticket| ticket.state == 'settled'}.each do |item|
+        items << item
+      end
+    end
+    
+    items
+  end
+
+  def printable_tickets?
+    self.tickets.each do |ticket|
+      return true if ticket.state == 'settled' || ticket.state == 'exchanged' || ticket.state == 'purchased'
+    end
+
+    false
+  end
+
+  def fully_paid?
+    outstanding_amount <= 0
+  end
+
+  def outstanding_amount
+    sum_donations - children_donations_total
+  end
+
+  def children_donations_total
+    children.includes(:donation).
+        where('donations.pledge = ?', false).
+        inject(0) {|sum, child_order| sum + child_order.sum_donations}
+  end
+
+  def match_eligible_pledge
+    if donation && donation.match_eligible
+      children.first
+    end
+  end
+
+  def is_pledge
+    !donation.nil? && donation.pledge
+  end
+
+  def pledge_commitment_date
+    ActiveSupport::TimeZone.create(organization.time_zone).utc_to_local(donation.commitment_date) if is_pledge
+  end
+
+  #all donations for this person, that are not pledge_payments
+  def assignable_donations
+    person.orders.includes(:donation).where('donations.pledge = ?', false).where('orders.parent_id IS NULL')
+  end
+
+  #all outstanding pledges for this person
+  def assignable_pledges
+    person.orders.includes(:donation).
+        where('donations.pledge = ?', true).
+        where('orders.parent_id IS NULL OR orders.parent_id != ?', id).
+        select { |o| !o.fully_paid? }
+  end
+
+  def gift_type
+    if is_pledge
+       'Pledge'
+    elsif parent_id.nil? || (type.eql? 'RefundOrder') || (type.eql? 'ConvertOrder')
+      'Donation'
+    else
+      'Pledge Payment'
+    end
+  end
 
   private