catarse_pagarme 2.14.10 → 2.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e5cf253edf3740df0178687448f76d33ac7307dfc4e0365c97cd74a8942d0c70
-  data.tar.gz: 6102f081c3a8777089cbf4b104e555289f58f09bf87c13397fdf977685db2a5f
+  metadata.gz: 33de3fd1e6afdac8723405f945acc3d89d97be9420eace3c44f354e01113de6f
+  data.tar.gz: b25b6ad732704f9faf108861a844936ba21722b7ab41be1fb06f155510413d9e
 SHA512:
-  metadata.gz: e570aa0e9cba1ac76665fe9b57a6af2245cf503132e47560d2a08be7abe8dbaa4cc431c34cbc908c1392fbbf7ed9a2423379ebf6b699c95e167d12a76669e79f
-  data.tar.gz: 90b5aaa7df3bdc6620b45b67745f0493e2ad0fb0e3b55aca83653c820f8d3c064a7205fb057fce2a112a01477d9fa4d42b36844b2cbb5945ac2dcd7cd8274528
+  metadata.gz: 73b17f419b5ef6ab87886c5642dd83fccb1189737a3398e5698d6aaf5cb60457846816fc547185c5fbbcafd9c8092b78f55d0a3c26648e9de03643c1c9343007
+  data.tar.gz: 8740a9e7f9ce6296534ea70ba43d183788b850bc100dcf82a8bdf8631dcc3b53f519b27912e5ecdea0fd479e042da3534e6d1a819f4bf48042cabc8658e00d4f

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    catarse_pagarme (2.14.10)
+    catarse_pagarme (2.15.0)
       countries (= 3.0.0)
       konduto-ruby (= 2.1.0)
       pagarme (= 2.1.4)

data/app/controllers/catarse_pagarme/application_controller.rb

@@ -19,20 +19,24 @@ module CatarsePagarme
           payment_method: payment.payment_method
         }
       )
-      ::Raven.capture_exception(exception)
+      ::Raven.capture_exception(exception, level: 'fatal')
       ::Raven.user_context({})
     end
 
     def metadata_attributes
       {
         key: payment.generate_key,
-        contribution_id: payment.contribution.id,
+        contribution_id: payment.contribution.id.to_s,
+        project_id: payment.project.id.to_s,
         project_name: payment.project.name,
         permalink: payment.project.permalink,
         project_online: payment.project.online_at,
         project_expires: payment.project.expires_at,
         user_total_contributions: payment.user.contributions.was_confirmed.count,
-        user_id: payment.user.id
+        user_id: payment.user.id,
+        common_user_id: payment.user.common_id.to_s,
+        user_name: payment.user.name,
+        user_email: payment.user.email
       }
     end
 

data/app/controllers/catarse_pagarme/credit_cards_controller.rb

@@ -17,6 +17,8 @@ module CatarsePagarme
       render json: { payment_status: 'failed', message: e.message }
     rescue => e
       raven_capture(e)
+
+      render json: { payment_status: 'failed', message: e.message }
     end
 
     def get_installment_json
@@ -81,11 +83,7 @@ module CatarsePagarme
     end
 
     def document_number
-      user_document = contribution.user.cpf.try(:gsub, /[-.\/_\s]/, '')
-      card_owner_document = contribution.card_owner_document.try(:gsub, /[-.\/_\s]/, '')
-      fallback_document = '00000000000'
-
-      user_document || card_owner_document || fallback_document
+      contribution.user.cpf.try(:gsub, /[-.\/_\s]/, '')
     end
 
     def phone_matches
@@ -135,7 +133,7 @@ module CatarsePagarme
 
     def address_hash
       {
-        country: contribution.country.try(:name),
+        country_code: contribution.country.try(:code),
         state: contribution.address_state,
         city: contribution.address_city,
         zipcode: contribution.address_zip_code.try(:gsub, /[-.\/_\s]/, ''),

data/app/controllers/catarse_pagarme/slip_controller.rb

@@ -46,7 +46,7 @@ module CatarsePagarme
             number: document_number
           }],
         },
-        metadata: metadata_attributes
+        metadata: metadata_attributes.merge({ second_slip: payment.generating_second_slip.to_s })
       }
     end
 

data/app/models/catarse_pagarme/antifraud_order_wrapper.rb

@@ -71,14 +71,15 @@ module CatarsePagarme
 
     def customer_attributes
       customer = self.attributes.dig(:customer)
+      tax_id = customer[:document_number].present? ? { tax_id: customer[:document_number] } : {}
+
       {
         id: customer[:id].to_s,
-        tax_id: customer[:document_number],
         name: customer[:name],
         email: customer[:email],
         phone1: customer[:phone].to_h.values.join,
         created_at: self.attributes.dig(:antifraud_metadata, :register, :registered_at)
-      }
+      }.merge(tax_id)
     end
 
     def payment_attributes
@@ -99,7 +100,8 @@ module CatarsePagarme
         city: billing_data.dig(:address, :city),
         state: billing_data.dig(:address, :state),
         zip: billing_data.dig(:address, :zipcode),
-      }.merge(card_country_code)
+        country: billing_data.dig(:address, :country_code),
+      }
     end
 
     def shipping_address_attributes
@@ -139,10 +141,5 @@ module CatarsePagarme
       expiration_date = self.transaction.card.expiration_date
       "#{expiration_date[0..1]}20#{expiration_date[2..3]}"
     end
-
-    def card_country_code
-      country = ::ISO3166::Country.find_country_by_name(self.transaction.card.country)
-      country.present? ? { country: country.try(:alpha2) } : {}
-    end
   end
 end

data/app/models/catarse_pagarme/credit_card_transaction.rb

@@ -8,10 +8,14 @@ module CatarsePagarme
         else
           antifraud_outcome = process_antifraud
 
+          self.payment.create_antifraud_analysis!(cost: CatarsePagarme.configuration.antifraud_tax)
+
           if antifraud_outcome.recommendation == :APPROVE
             self.transaction.capture
           elsif antifraud_outcome.recommendation == :DECLINE
             self.transaction.refund
+          elsif antifraud_outcome.recommendation == :REVIEW
+            self.payment.try(:notify_about_pending_review)
           end
         end
       end
@@ -31,7 +35,8 @@ module CatarsePagarme
           async: false,
           postback_url: self.attributes[:postback_url],
           installments: self.attributes[:installments],
-          soft_descriptor: self.attributes[:soft_descriptor]
+          soft_descriptor: self.attributes[:soft_descriptor],
+          metadata: self.attributes[:metadata]
         }.merge(credit_card_identifier)
       )
 
@@ -52,7 +57,7 @@ module CatarsePagarme
     end
 
     def was_credit_card_used_before?
-      PaymentEngines.was_credit_card_used_before?(self.transaction.card.id)
+      PaymentEngines.was_credit_card_used_before?(self.transaction.card.id, self.attributes.dig(:customer, :id))
     end
 
     def process_antifraud

data/app/models/catarse_pagarme/fee_calculator_concern.rb

@@ -5,102 +5,17 @@ module CatarsePagarme::FeeCalculatorConcern
 
     def get_fee
       return nil if self.payment.payment_method.blank? # We always depend on the payment_choice
-      if self.payment.payment_method == ::CatarsePagarme::PaymentType::SLIP
-        get_slip_fee
-      else
-        get_card_fee
-      end
-    end
-
-    protected
-    def get_slip_fee
-      CatarsePagarme.configuration.slip_tax.to_f
-    end
-
-    def get_card_fee
-      return nil if self.payment.gateway_data["acquirer_name"].blank? # Here we depend on the acquirer name
-      fee = if %w(stone pagarme).include? self.payment.gateway_data["acquirer_name"]
-        get_stone_fee
-      else
-        get_cielo_fee
-      end
-
-      fee + payment_cost
-    end
 
-    def payment_cost
-      self.payment.gateway_data["cost"] / 100.0
-    end
-
-    def get_stone_fee
-      self.payment.installments > 1 ? tax_calc_for_installment(stone_installment_tax) : tax_calc(stone_tax)
-    end
+      transaction = PagarMe::Transaction.find(self.payment.gateway_id)
+      payables = transaction.payables
+      cost = transaction.cost.to_f / 100.00
+      payables_fee = payables_fee.to_a.sum(&:fee).to_f / 100.00
 
-    def get_cielo_fee
-      return nil if self.payment.gateway_data["card_brand"].blank? # Here we depend on the card_brand
-      if self.payment.gateway_data["card_brand"] == 'amex'
-        get_cielo_fee_for_amex
+      if self.payment.payment_method == ::CatarsePagarme::PaymentType::SLIP
+        cost + payables_fee
       else
-        get_cielo_fee_for_non_amex
+        payables_fee == 0 ? cost : payables_fee
       end
     end
-
-    def get_cielo_fee_for_amex
-      self.payment.installments > 1 ? tax_calc_for_installment(cielo_installment_amex_tax) : tax_calc(cielo_installment_not_amex_tax)
-    end
-
-    def get_cielo_fee_for_non_amex
-      current_tax = self.payment.gateway_data["card_brand"] == 'diners' ? installment_diners_tax : installment_not_diners_tax
-      self.payment.installments > 1 ? tax_calc_for_installment(current_tax) : tax_calc(cielo_tax)
-    end
-
-    def tax_calc acquirer_tax
-      ((self.payment.value * pagarme_tax) + cents_fee).round(2) + (self.payment.value * acquirer_tax).round(2)
-    end
-
-    def tax_calc_for_installment acquirer_tax
-      (((self.payment.installment_value * self.payment.installments) * pagarme_tax) + cents_fee).round(2) + ((self.payment.installment_value * acquirer_tax).round(2) * self.payment.installments)
-    end
-
-    def antifraud_tax
-      CatarsePagarme.configuration.antifraud_tax.to_f
-    end
-
-    def cents_fee
-      CatarsePagarme.configuration.credit_card_cents_fee.to_f
-    end
-
-    def pagarme_tax
-      CatarsePagarme.configuration.pagarme_tax.to_f
-    end
-
-    def cielo_tax
-      CatarsePagarme.configuration.cielo_tax.to_f
-    end
-
-    def stone_tax
-      CatarsePagarme.configuration.stone_tax.to_f
-    end
-
-    def stone_installment_tax
-      CatarsePagarme.configuration.stone_installment_tax.to_f
-    end
-
-    def installment_diners_tax
-      CatarsePagarme.configuration.cielo_installment_diners_tax.to_f
-    end
-
-    def installment_not_diners_tax
-      CatarsePagarme.configuration.cielo_installment_not_diners_tax.to_f
-    end
-
-    def cielo_installment_amex_tax
-      CatarsePagarme.configuration.cielo_installment_amex_tax.to_f
-    end
-
-    def cielo_installment_not_amex_tax
-      CatarsePagarme.configuration.cielo_installment_not_amex_tax.to_f
-    end
-
   end
 end

data/lib/catarse_pagarme/version.rb

@@ -1,3 +1,3 @@
 module CatarsePagarme
-  VERSION = "2.14.10"
+  VERSION = "2.15.0"
 end

data/spec/models/catarse_pagarme/antifraud_order_wrapper_spec.rb

@@ -245,14 +245,15 @@ describe CatarsePagarme::AntifraudOrderWrapper do
               street: 'R. A',
               city: 'New City',
               state: 'NC',
-              zipcode: '1245'
+              zipcode: '1245',
+              country_code: 'BR'
             }
           }
         }
       }
     end
 
-    let(:transaction) { double(card: double(country: 'BRAZIL', holder_name: 'holder name')) }
+    let(:transaction) { double(card: double(holder_name: 'holder name')) }
 
     before do
       subject.attributes = attributes
@@ -267,7 +268,7 @@ describe CatarsePagarme::AntifraudOrderWrapper do
       expect(billing_address_attributes[:city]).to eq 'New City'
       expect(billing_address_attributes[:state]).to eq 'NC'
       expect(billing_address_attributes[:zip]).to eq '1245'
-      expect(billing_address_attributes[:country]).to eq 'BR'
+      expect(billing_address_attributes[:country_code]).to eq 'BR'
     end
   end
 
@@ -290,7 +291,7 @@ describe CatarsePagarme::AntifraudOrderWrapper do
       }
     end
 
-    let(:transaction) { double(card: double(country: 'BRAZIL')) }
+    let(:transaction) { double(card: double) }
 
     before do
       subject.attributes = attributes

data/spec/models/catarse_pagarme/credit_card_transaction_spec.rb

@@ -76,7 +76,7 @@ describe CatarsePagarme::CreditCardTransaction do
           end
         end
 
-        context 'when antifraude recommends decline transaction' do
+        context 'when antifraud recommends decline transaction' do
           let(:antifraud_outcome) { double(recommendation: :DECLINE) }
 
           it 'refunds transaction' do
@@ -85,6 +85,16 @@ describe CatarsePagarme::CreditCardTransaction do
             card_transaction.process!
           end
         end
+
+        context 'when antifraud recommends review transaction' do
+          let(:antifraud_outcome) { double(recommendation: :REVIEW) }
+
+          it 'notifies about pending review' do
+            expect(card_transaction.payment).to receive(:notify_about_pending_review)
+
+            card_transaction.process!
+          end
+        end
       end
     end
 
@@ -269,7 +279,7 @@ describe CatarsePagarme::CreditCardTransaction do
     let(:antifraud_wrapper) { double }
 
     context 'when @antifraud_wrapper isn`t nil' do
-      it 'returns @antifraude_wrapper' do
+      it 'returns @antifraud_wrapper' do
         card_transaction.instance_variable_set('@antifraud_wrapper', antifraud_wrapper)
         expect(card_transaction.antifraud_wrapper).to eq antifraud_wrapper
       end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: catarse_pagarme
 version: !ruby/object:Gem::Version
-  version: 2.14.10
+  version: 2.15.0
 platform: ruby
 authors:
 - Antônio Roberto Silva
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-03-31 00:00:00.000000000 Z
+date: 2020-05-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: countries