catarse_pagarme 2.7.13 → 2.7.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 956cfd4d934cd4d2f5b60cd7f0477b5d59c8a447
-  data.tar.gz: ca3fc2602768ed56c32fe2c9e3d4181998024d53
+  metadata.gz: 64917b88fa785e17b6a46b2fffe652e543e49658
+  data.tar.gz: d523792d29cd7f16899fa93aa25ecef903aaa122
 SHA512:
-  metadata.gz: fc97dc2ffe2d9a191d64ea7a83cd051df9f63086ba336f7b1181022d25a75c36a19affe4e442101b0a4aa79293b0b9786bedd8f4cb5bb91eeb90f93e92196741
-  data.tar.gz: d5d5b29730b43e8b9a27c6e706db54d1cc391e957cce9c459a4a116d2997fc6037b6ec921b0236eedf131c5c9c939fe769b8110d262df6917f72a1e2ad54260c
+  metadata.gz: d9572cf6e8483a30f8c920869405a93608985feb34ffd8acfa9e3f9a863cbc96a830d0979eb5e613f6ed8bc360ab5d68ce345f0f2286d9ad8d6df56d41f729ac
+  data.tar.gz: fbccfc7e54dc617e1c4a049837265d2b76c4200fbeb9579da5c7e80c10cdf3d72cf6fe9121765549fd7a9fa5069bf4449ea78042fd8e00037b2a26d58ff8faf1

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    catarse_pagarme (2.7.13)
+    catarse_pagarme (2.7.14)
       pagarme (= 1.10.0)
       rails (~> 4.0)
       sidekiq

data/app/models/catarse_pagarme/payment_delegator.rb

@@ -96,10 +96,8 @@ module CatarsePagarme
     end
 
     # Transfer payment amount to payer bank account via transfers API
-    # Params:
-    # +authorized_by+:: +User+ object that authorize this transfer
-    def transfer_funds(authorized_by)
-      raise 'must be admin to perform this action' unless authorized_by.try(:admin?)
+    def transfer_funds
+      raise "payment must be paid" if !payment.paid?
 
       bank_account = PagarMe::BankAccount.new(bank_account_attributes.delete(:bank_account))
       bank_account.create
@@ -112,10 +110,12 @@ module CatarsePagarme
       transfer.create
 
       payment.payment_transfers.create!({
-        user: authorized_by,
+        user: payment.user,
         transfer_id: transfer.id,
         transfer_data: transfer.to_json
       })
+      #avoid sending notification
+      payment.update_attributes(state: 'pending_refund')
     end
 
     protected

data/lib/catarse_pagarme/payment_engine.rb

@@ -20,6 +20,10 @@ module CatarsePagarme
       contribution.pagarme_delegator.refund
     end
 
+    def transfer contribution
+      contribution.pagarme_delegator.transfer_funds
+    end
+
     def can_generate_second_slip?
       true
     end
@@ -36,3 +40,4 @@ module CatarsePagarme
     end
   end
 end
+

data/lib/catarse_pagarme/version.rb

@@ -1,3 +1,3 @@
 module CatarsePagarme
-  VERSION = "2.7.13"
+  VERSION = "2.7.14"
 end

data/spec/models/catarse_pagarme/payment_delegator_spec.rb

@@ -139,20 +139,18 @@ describe CatarsePagarme::PaymentDelegator do
   end
 
   describe "#transfer_funds" do
-    let(:admin_user) { create(:user, admin: true) }
     let(:transfer_mock) { double(create: true, id: "123", to_json: {id: '123'}.to_json) }
     before do
       allow(PagarMe::Transfer).to receive(:new).and_return(transfer_mock)
       create(:bank_account, user: payment.user)
-      allow(payment.user).to receive(:admin?).and_return(true)
     end
 
     it do
-      transfer = delegator.transfer_funds(admin_user)
+      transfer = delegator.transfer_funds
 
       expect(payment.payment_transfers.count).to eq(1)
       expect(transfer.payment).to eq(payment)
-      expect(transfer.user).to eq(admin_user)
+      expect(transfer.user).to eq(payment.user)
       expect(transfer.transfer_id).to eq(transfer.transfer_data["id"])
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: catarse_pagarme
 version: !ruby/object:Gem::Version
-  version: 2.7.13
+  version: 2.7.14
 platform: ruby
 authors:
 - Antônio Roberto Silva
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-06 00:00:00.000000000 Z
+date: 2015-10-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails