db/migrate/20091022150926_add_type_to_donations.rb app/controllers/myspot/donation_amounts_controller.rb @@ -38,7 +38,11 @@ class Myspot::DonationsController < ApplicationController end def new_resource - current_user.donations.new(params[:donation]) + current_user.donations.new(params[:donation]) + # TODO: figure out why donation_type does not get properly assigned with params / should not need to assign after new + # d = Donation.new(params[:donation]) + # d.donation_type = params[:donation][:donation_type] + # d end def find_resources app/controllers/myspot/donations_controller.rb @@ -46,6 +46,7 @@ class Donation < ActiveRecord::Base belongs_to :user belongs_to :pitch belongs_to :purchase + belongs_to :credit belongs_to :group validates_presence_of :pitch_id validates_presence_of :user_id @@ -53,7 +54,8 @@ class Donation < ActiveRecord::Base validates_numericality_of :amount, :greater_than => 0 validate_on_update :disable_updating_paid_donations, :check_donation, :if => lambda { |me| me.pitch } validate_on_create :check_donation, :if => lambda { |me| me.pitch } - + + #default_scope :conditions => {:donation_type => "payment"} named_scope :unpaid, :conditions => "status = 'unpaid'" named_scope :paid, :conditions => "status = 'paid'" named_scope :from_organizations, :include => :user, :conditions => "users.type = 'organization'" @@ -88,7 +90,7 @@ class Donation < ActiveRecord::Base end def update_pitch_funding - pitch.current_funding += amount + pitch.current_funding += amount.to_f pitch.save end @@ -101,17 +103,28 @@ class Donation < ActiveRecord::Base def being_marked_as_paid? status_changed? && status_was == 'unpaid' end + + def check_credit + # self.amount = 100 if self.amount > 100 + if self.amount > pitch.requested_amount - pitch.current_funding + self.amount = pitch.requested_amount - pitch.current_funding + end + end def check_donation if pitch.fully_funded? errors.add_to_base("Great news! This pitch is already fully funded therefore it can't be donated to any longer.") return end - - #TODO: convert user_can_donate_more? to use BigDecimal and pass in amount - unless pitch.user_can_donate_more?(user, amount) + # question for david - does this funcionality happen for donations currently...automatically limiting the total to the amount remaining? + # if we want this ...then make this happen for both credit and donation + if donation_type == "credit" + check_credit + elsif pitch.user_can_donate_more?(user, amount) errors.add_to_base("Thanks for your support but we only allow donations of 20% of requested amount from one user. Please lower your donation amount and try again.") + return end + end def self.find_all_from_paypal(paypal_params) @@ -129,5 +142,14 @@ class Donation < ActiveRecord::Base paypal_params.delete(spotus_keys.first) end end + + def self.has_enough_credits?(credit_pitch_amounts, user) + credit_total = 0 + credit_pitch_amounts.values.each do |val| + credit_total += val["amount"].to_f + end + return true if credit_total <= user.total_credits + return false + end end app/models/donation.rb @@ -81,7 +81,7 @@ class Pitch < NewsItem has_many :tips, :through => :affiliations has_many :organization_pitches, :foreign_key => :pitch_id has_many :supporting_organizations, :through => :organization_pitches, :source => :organization - has_many :donations, :dependent => :destroy do + has_many :donations, :conditions => {:donation_type => "payment"}, :dependent => :destroy do def for_user(user) find_all_by_user_id(user.id) end @@ -90,15 +90,17 @@ class Pitch < NewsItem for_user(user).map(&:amount).sum end end - has_many :credit_pitches, :dependent => :destroy do - def for_user(user) - find_all_by_user_id(user.id) - end - - def total_amount_for_user(user) - for_user(user).map(&:amount).sum + + has_many :credit_pitches, :class_name => "Donation", :conditions => {:donation_type => "credit"}, :dependent => :destroy do + def for_user(user) + find_all_by_user_id(user.id) + end + + def total_amount_for_user(user) + for_user(user).map(&:amount).sum + end end - end + has_many :organizational_donors, :through => :donations, :source => :user, :order => "donations.created_at", :conditions => "users.type = 'organization'", :uniq => true @@ -248,6 +250,7 @@ class Pitch < NewsItem def fully_funded? return true if accepted? || funded? + puts total_amount_donated total_amount_donated >= requested_amount end app/models/pitch.rb @@ -27,7 +27,7 @@ class Purchase < ActiveRecord::Base cattr_accessor :gateway - after_create :associate_donations, :apply_credit_pitches, :associate_spotus_donations + after_create :associate_donations, :associate_spotus_donations before_create :bill_credit_card, :unless => lambda {|p| p.paypal_transaction? } before_validation_on_create :build_credit_card, :set_credit_card_number_ending, :unless => lambda {|p| p.paypal_transaction? } @@ -85,10 +85,6 @@ class Purchase < ActiveRecord::Base self[:total_amount] = total_amount end - def apply_credit_pitches - self.user.apply_credit_pitches - end - def build_credit_card @credit_card = ActiveMerchant::Billing::CreditCard.new(credit_card_hash) end app/models/purchase.rb @@ -54,7 +54,6 @@ class Tip < NewsItem named_scope :most_pledged, :order => "(select sum(amount) from pledges where pledges.tip_id = #{table_name}.id) DESC" - def self.createable_by?(user) !user.nil? end app/models/tip.rb @@ -68,11 +68,13 @@ class User < ActiveRecord::Base belongs_to :category - has_many :donations do + has_many :donations, :conditions => {:donation_type => "payment"} do def pitch_sum(pitch) self.paid.all(:conditions => {:pitch_id => pitch}).map(&:amount).sum end end + + has_many :all_donations, :class_name => "Donation" has_many :spotus_donations has_many :tips @@ -82,7 +84,7 @@ class User < ActiveRecord::Base has_many :jobs has_many :samples has_many :credits - has_many :credit_pitches + has_many :credit_pitches, :class_name => "Donation", :conditions => {:donation_type => "credit"} has_many :comments has_many :contributor_applications has_many :pledges do @@ -194,14 +196,13 @@ class User < ActiveRecord::Base def apply_credit_pitches #refactor - there must be a nicer ruby-like way to do this transaction do - credit_pitch_ids = self.credit_pitches.unpaid.map{|credit_pitch| [credit_pitch.id]}.join(", ") + credit_pitch_ids = self.credit_pitches.unpaid.map{|credit_pitch| [credit_pitch.pitch.id]}.join(", ") credit = Credit.create(:user => self, :description => "Applied to Pitches (#{credit_pitch_ids})", :amount => (0 - self.allocated_credits)) + puts ' whop' self.credit_pitches.unpaid.each do |credit_pitch| + credit_pitch.credit_id = credit.id credit_pitch.pay! - credit_pitch.paid_credit_id = credit.id - credit_pitch.update_pitch_funding - credit_pitch.save end end end app/models/user.rb @@ -1,6 +1,8 @@ -- form_for CreditPitch.new(:pitch => news_item, :amount => current_user.remaining_credits), :url => myspot_credit_pitches_path, :html => {:id => "new_credit_pitch_#{news_item.id}", :class => "auth"} do |f| +- form_for Donation.new(:pitch => news_item, :amount => current_user.remaining_credits, :donation_type => "credit"), :url => myspot_donations_path, :html => {:id => "new_credit_pitch_#{news_item.id}", :class => "auth"} do |f| = f.hidden_field :amount, :id => "credit_pitch_amount" = f.hidden_field :pitch_id + = f.hidden_field :user_id, :value => current_user.id + = f.hidden_field :donation_type, :value => "credit" = image_submit_tag "apply_credits.png", :alt => "Apply credits", :title => "Apply credits" .double_content_border = render :partial => "shared/progress_bar", :locals => {:pitch => news_item} \ No newline at end of file app/views/credits/_credit_pitch.html.haml @@ -1,7 +1,7 @@ - # dashboard navigation #dashboard_tab_container %ul.dashboard_navigation - %li= link_to "My Donations (#{current_user.donations.paid.count})", myspot_donations_path, :id => "dashboard_donations_tab" + %li= link_to "My Donations (#{current_user.all_donations.paid.count})", myspot_donations_path, :id => "dashboard_donations_tab" %li= link_to "My Pledges (#{current_user.pledges.count})", myspot_pledges_path, :id => "dashboard_pledges_tab" %li= link_to "My Pitches (#{current_user.pitches.count})", myspot_pitches_path, :id => "dashboard_pitches_tab" %li= link_to "My Blog Posts (#{current_user.posts.count})", myspot_posts_path, :id => "dashboard_posts_tab" app/views/myspot/_dashboard_nav.html.haml @@ -55,12 +55,12 @@ = text_field_tag "donation_amounts[#{donation.id}][amount]", number_to_currency(donation.amount, :delimiter => '', :unit => ''), :size => 4, :id => "donation_amounts_#{donation.id}_amount" .action= link_to "Remove", myspot_donation_path(donation), :method => :delete, :confirm => "Are you sure you'd like to remove this donation of #{number_to_currency(donation.amount)}?" %hr - - if current_user.credit_pitches.unpaid.any? + - if unpaid_credits.any? %h3 Apply Credits %ul.unpaid_donations %hr - - current_user.credit_pitches.unpaid.each do |credit_pitch| + - unpaid_credits.each do |credit_pitch| - content_tag_for :li, credit_pitch, :class => 'unpaid' do .column.span-70 = link_to image_tag(credit_pitch.pitch.featured_image.url(:thumb)), credit_pitch.pitch @@ -74,7 +74,7 @@ .column.span-95 $ = text_field_tag "credit_pitch_amounts[#{credit_pitch.id}][amount]", number_to_currency(credit_pitch.amount, :delimiter => '', :unit => ''), :size => 4, :id => "credit_pitch_amounts_#{credit_pitch.id}_amount" - .action= link_to "Remove", myspot_credit_pitch_path(credit_pitch), :method => :delete, :confirm => "Are you sure you'd like to remove this credit of #{number_to_currency(credit_pitch.amount)}?" + .action= link_to "Remove", myspot_donation_path(credit_pitch), :method => :delete, :confirm => "Are you sure you'd like to remove this credit of #{number_to_currency(credit_pitch.amount)}?" %hr %li#support_spotus %h3 Support Spot.Us app/views/myspot/donation_amounts/edit.html.haml app/controllers/myspot/credit_pitches_controller.rb app/models/credit_pitch.rb spec/controllers/myspot/credit_pitches_controller_spec.rb 046ed6f2085c2cf0ef45d1a809214f9b53fd122e Dan Newman dpnewman@gmail.com http://github.com/spot-us/spot-us/commit/939add7793f61ce6163450f0a8775a7e5230eaf3 939add7793f61ce6163450f0a8775a7e5230eaf3 2009-10-23T07:20:39-07:00 2009-10-23T07:20:39-07:00 Crediting pitches reprise 66e530a44e1b68a1af94f9783d788642435bd6be Dan Newman dpnewman@gmail.com