invoicing_payments_processing 1.1.48 → 1.1.53

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f4e47e64797dda88bd7b65fe1a51b7c6ad5deb71
-  data.tar.gz: df21c11687542c76574d1e0c838232415af5ef34
+  metadata.gz: d01ccb52a62ca320e4f63599f6a47439a2b575f4
+  data.tar.gz: 61eaaf30982a66d6be91a0d428b507a482e1ffde
 SHA512:
-  metadata.gz: c032c751ad2c959c876afe5de690913a9435da9050c17ceca83ddcddf49857938b1805047788b9c8c0a536ef06beefedbfb21ae779b44cc80c67f05372b66ca2
-  data.tar.gz: 76541dcf86cf8bd60fdc61622504f3fd2e672f45f2e7c1b00a94620537f5f95442fafb61154586835d1692983a17f920a9ff9c096c648426640626cfe26b2dc4
+  metadata.gz: 6b48123c4a1704235ba27b2b54c22a8e9da3322d9544836edf9ae72acf24d1a17b148153e7d0f407263e8053884f243540b96f4810fab8d18be1d2afc7065450
+  data.tar.gz: 1cf2f01c03a663ee63e747a8733813a1e19fffb515ed6a1b4c0e729e41582af3e9cab4b3945bea5da6adf3b896682639f1f5f6df3ca5c1fc95e09e0d28b054bb

data/lib/bufferpaypalnotification.rb

@@ -41,6 +41,10 @@ module BlackStack
     # 3) haciendo coincidir el campo payer_email de alguna suscripcion existente con el BlackStack::BufferPayPalNotification.payer_email
     # 3) haciendo coincidir el primer guid en el codigo de invoice, con el id del cliente
     def get_client()
+puts
+puts "BlackStack::BufferPayPalNotification::get_client"
+puts 'debug info:'
+puts "self.invoice:#{self.invoice.to_s}:."
       # obtengo el cliente que machea con este perfil
       c = nil
       if c.nil?
@@ -49,12 +53,24 @@ module BlackStack
           c = i.client if !i.nil? 
         end
       end
+      if c.nil?
+        iid = self.invoice.split(".").last.to_s
+        if iid.guid?
+          i = BlackStack::Invoice.where(:id=>iid).first
+          c = i.client if !i.nil? 
+        end
+      end
       if c.nil?
         cid = self.invoice.split(".").first.to_s
         if cid.guid?
           c = BlackStack::Client.where(:id=>cid).first
         end
       end
+=begin
+# deprecated: debe obtenerse la factura (invoice), y de ahi el cliente,
+# => porque un mismo cuente puede usar su cuenta paypal en varias cuantas 
+# => de usuario.
+#  
       if c.nil?
         c = BlackStack::Client.where(:paypal_email=>self.payer_email).first
         if (c == nil)
@@ -83,6 +99,7 @@ module BlackStack
           c = BlackStack::Client.where(:id=>row[:cid]).first
         end
       end
+=end
       c
     end
   
@@ -316,10 +333,12 @@ module BlackStack
           j.next(i)
           
           # creo el milestone con todo el credito pendiente que tiene esta subscripcion
+=begin
+# deprecated  
           buff_payment = i.buffer_paypal_notification
           buff_signup = BlackStack::BufferPayPalNotification.where(:txn_type=>"subscr_signup", :subscr_id=>buff_payment.subscr_id).first
           subs = buff_signup == nil ? nil : BlackStack::PayPalSubscription.where(:id_buffer_paypal_notification => buff_signup.id).first
-              
+=end
         elsif (b.txn_type == BlackStack::BufferPayPalNotification::TXN_TYPE_SUBSCRIPTION_SIGNUP)
           # crear un registro en la tabla paypal_subscriptions
           if BlackStack::PayPalSubscription.load(b.to_hash) != nil
@@ -336,7 +355,7 @@ module BlackStack
             s.save
             
             # obtengo la factura que se creo con esta suscripcion
-            i = BlackStack::Invoice.where(:id=>b.item_number).first
+            i = BlackStack::Invoice.where(:id=>b.invoice.split(".").last.to_s).first
             
             # vinculo esta suscripcion a la factura que la genero, y a todas las facturas siguientes
             i.set_subscription(s)
@@ -384,7 +403,7 @@ module BlackStack
               "SELECT TOP 1 i.id " +
               "FROM buffer_paypal_notification b " +
               "JOIN invoice i ON ( b.id=i.id_buffer_paypal_notification AND i.status=#{BlackStack::Invoice::STATUS_PAID.to_s} ) " +
-              "WHERE b.invoice='#{b.invoice}' " +
+              "WHERE b.invoice='#{b.invoice.split(".").last.to_s}' " +
               "ORDER BY i.create_time DESC "
             ].first
             if row.nil?

data/lib/extend_client_by_invoicing_payments_processing.rb

@@ -35,7 +35,7 @@ module BlackStack
 				total_amount = 0.to_f - BlackStack::Balance.new(self.id, product_code).amount.to_f
 				sleep(2) # delay to ensure the time of the bonus movement will be later than the time of the consumption movement
 				if total_credits < 0
-					self.adjustment(product_code, total_amount, total_credits, 'Adjustment Because Quota Has Been Exceeded.')
+					self.adjustment(product_code, total_amount, total_credits, 'Adjustment Because Quota Has Been Exceeded (1).')
 				end
 				# recaculate amounts in both consumptions and expirations - CANCELADO - Se debe hacer offline
 				#self.recalculate(product_code) 
@@ -97,7 +97,7 @@ module BlackStack
 
 				self.movements.select { |o| 
 					o.product_code.upcase == product_code.upcase
-				}.sort_by { |o| o.create_time }.each { |o|
+				}.sort_by { |o| [o.create_time, o.type] }.each { |o| # se ordena por o.create_time, pero tmabien por o.type para procesar primero los pagos y bonos
 					#if o.credits.to_f < 0 # payment or bonus
 #					if o.credits.to_f > 0 && ( o.type==BlackStack::Movement::MOVEMENT_TYPE_CANCELATION || o.type==BlackStack::Movement::MOVEMENT_TYPE_EXPIRATION ) # consumption or expiration
 					# consumption or expiration or bonus
@@ -118,7 +118,7 @@ module BlackStack
 					total_credits = credits_paid
 					total_amount = amount_paid
 					if total_credits < 0
-						self. adjustment(product_code, total_amount, total_credits, 'Adjustment Because Quota Has Been Exceeded.', BlackStack::Movement::MOVEMENT_TYPE_ADJUSTMENT, o.create_time)
+						self. adjustment(product_code, total_amount, total_credits, 'Adjustment Because Quota Has Been Exceeded (2).', BlackStack::Movement::MOVEMENT_TYPE_ADJUSTMENT, o.create_time)
 						amount_paid = 0.to_f
 						credits_paid = 0.to_i
 					end

data/lib/invoice.rb

@@ -624,21 +624,47 @@ module BlackStack
           item1.detail = u.detail.to_s
           item1.description = u.description.to_s
           item1.save()
+          # hago el reembolso de este item
+          # si el balance quedo en negativo, entonces aplico otro ajuste
           BlackStack::Movement.new().parse(item1, BlackStack::Movement::MOVEMENT_TYPE_REFUND_BALANCE, "Full Refund of <a href='/member/invoice?iid=#{self.id.to_guid}'>invoice:#{self.id.to_guid}</a>.").save()        
-					# si el balance quedo en negativo, entonces aplico otro ajuste
 					net_amount = 0.to_f - BlackStack::Balance.new(self.client.id, u.product_code.to_s).amount.to_f
 					net_credits = 0.to_f - BlackStack::Balance.new(self.client.id, u.product_code.to_s).credits.to_f
-					if net_amount < 0 && net_credits < 0
+					if net_amount <= 0 && net_credits < 0
 						adjust = self.client.adjustment(u.product_code.to_s, net_amount, net_credits, "Adjustment for Negative Balance After Refund of <a href='/member/invoice?iid=#{self.id.to_guid}'>invoice:#{self.id.to_guid}</a>.")
 						adjust.id_invoice_item = item1.id
 						adjust.save			
 					end # if net_amount < 0
+          # hago el reembolso de cada bono de este item
+          # si el balance quedo en negativo, entonces aplico otro ajuste
+          h[:bonus_plans].each { |bonus|
+            i = BlackStack::InvoicingPaymentsProcessing::plans_descriptor.select { |obj| obj[:item_number] == bonus[:item_number] }.first
+            j = BlackStack::InvoicingPaymentsProcessing::products_descriptor.select { |obj| obj[:code] == i[:product_code] }.first
+            item2 = BlackStack::InvoiceItem.new()
+            item2.id = guid()
+            item2.id_invoice = self.id
+            item2.unit_price = 0
+            item2.units = -i[:credits]
+            item2.amount = 0
+            item2.product_code = i[:product_code].to_s
+            item2.item_number = i[:item_number].to_s
+            item2.detail = 'Bonus Refund'
+            item2.description = j[:description].to_s
+            item2.save()
+            BlackStack::Movement.new().parse(item2, BlackStack::Movement::MOVEMENT_TYPE_REFUND_BALANCE, "Bonus Refund of <a href='/member/invoice?iid=#{self.id.to_guid}'>invoice:#{self.id.to_guid}</a>.").save()        
+            net_amount = 0.to_f - BlackStack::Balance.new(self.client.id, i[:product_code].to_s).amount.to_f
+            net_credits = 0.to_f - BlackStack::Balance.new(self.client.id, i[:product_code].to_s).credits.to_f
+            if net_amount <= 0 && net_credits < 0
+              adjust = self.client.adjustment(i[:product_code].to_s, net_amount, net_credits, "Adjustment for Negative Balance After Bonus Refund of <a href='/member/invoice?iid=#{self.id.to_guid}'>invoice:#{self.id.to_guid}</a>.")
+              adjust.id_invoice_item = item1.id
+              adjust.save     
+            end # if net_amount < 0            
+          }
           # release resources
           DB.disconnect
           GC.start
         } # i.items.each { |u|
       # reembolso parcial de una factura con un unico item
-      elsif i.items.size == 1
+      elsif i.items.size == 1 # and we know that: total < -payment_gross, so it is a partial refund
         t = i.items.first
 				# 
         amount = -payment_gross.to_f
@@ -675,9 +701,39 @@ module BlackStack
 					adjust.id_invoice_item = item1.id
 					adjust.save			
 				end # if net_amount < 0
+				
 				# recalculo todos los consumos y expiraciones - CANCELADO - Debe hacerse offline
-				# self.client.recalculate(t.product_code.to_s)
-      else
+				# => self.client.recalculate(t.product_code.to_s)
+				
+        # si el cliente se quedo sin saldo luego del reembolso parcial 
+        if net_amount <= 0
+          # hago el reembolso de cada bono de este item
+          # si el balance quedo en negativo, entonces aplico otro ajuste
+          h[:bonus_plans].each { |bonus|
+            i = BlackStack::InvoicingPaymentsProcessing::plans_descriptor.select { |obj| obj[:item_number] == bonus[:item_number] }.first
+            j = BlackStack::InvoicingPaymentsProcessing::products_descriptor.select { |obj| obj[:code] == i[:product_code] }.first
+            item2 = BlackStack::InvoiceItem.new()
+            item2.id = guid()
+            item2.id_invoice = self.id
+            item2.unit_price = 0
+            item2.units = -i[:credits]
+            item2.amount = 0
+            item2.product_code = i[:product_code].to_s
+            item2.item_number = i[:item_number].to_s
+            item2.detail = 'Bonus Refund'
+            item2.description = j[:description].to_s
+            item2.save()
+            BlackStack::Movement.new().parse(item2, BlackStack::Movement::MOVEMENT_TYPE_REFUND_BALANCE, "Bonus Refund of <a href='/member/invoice?iid=#{self.id.to_guid}'>invoice:#{self.id.to_guid}</a>.").save()        
+            net_amount = 0.to_f - BlackStack::Balance.new(self.client.id, i[:product_code].to_s).amount.to_f
+            net_credits = 0.to_f - BlackStack::Balance.new(self.client.id, i[:product_code].to_s).credits.to_f
+            if net_amount <= 0 && net_credits < 0
+              adjust = self.client.adjustment(i[:product_code].to_s, net_amount, net_credits, "Adjustment for Negative Balance After Bonus Refund of <a href='/member/invoice?iid=#{self.id.to_guid}'>invoice:#{self.id.to_guid}</a>.")
+              adjust.id_invoice_item = item1.id
+              adjust.save     
+            end # if net_amount < 0            
+          }
+        end
+      else # we know that: i.items.size > 1 && total < -payment_gross
         raise "Refund amount is not matching with the invoice total (#{total.to_s}) and the invoice has more than 1 item."
       end
       # release resources

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: invoicing_payments_processing
 version: !ruby/object:Gem::Version
-  version: 1.1.48
+  version: 1.1.53
 platform: ruby
 authors:
 - Leandro Daniel Sardi
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-10-23 00:00:00.000000000 Z
+date: 2020-11-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: websocket