pagarme 2.0.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f7b1442209801ec8fcb0668b9214acf904843def
-  data.tar.gz: 1261b41b3c65366a9c04042786fba0df24d521d2
+  metadata.gz: f6e36ef4830f648c47b317abce9664ab3de2ebfa
+  data.tar.gz: 3374729244ff9b5e795071bd81f0e76f88c4738b
 SHA512:
-  metadata.gz: 0f9274f464816545b0df3bb3f3660fab2d4010a9cbe65806ec77eae730bf0b4a6bfb2f26205265b858e6612b8bfe56f1638ef644c343c4a97e69d384f6eeb1ba
-  data.tar.gz: b7962c5cd4a27f760822c3abf63c3df8c8d779ebfb061545ae0c1c2b9d0c3464a952f27a019d78012ac2158ee9b3c9783790d609c87f19a64a473ab2a4c25527
+  metadata.gz: 07568f8f6bd486c9263970ed0ce0c3de27af4a7ed8c590311bd34a1875db22ad307fe2020495a6f125b97406d62c65fe92633195bfc6f481efb1316d686a337b
+  data.tar.gz: 09f5d8c2733a232dcb1e5be374fc97ad000b62034b613112345cecc091c1c26e02f4984e05553b6d6b124cd67529ba318ea84f9d19357f96d48262f58b8a2cdb

data/lib/pagarme/object.rb

@@ -22,20 +22,9 @@ module PagarMe
 		end
 
 		def update(attributes)
-			removed = Set.new(@attributes.keys - attributes.keys)
-			added = Set.new(attributes.keys - @attributes.keys)
-
-			instance_eval do
-				remove_attribute(removed)
-				add_attribute(added)
-			end
-
-			removed.each do |key|
-				@attributes.delete(key)
-				@unsaved_values.delete(key)
-			end
-
 			attributes.each do |key, value|
+				key = key.to_s
+
 				@attributes[key] = Util.convert_to_pagarme_object(value)
 				@unsaved_values.delete(key)
 			end
@@ -76,58 +65,18 @@ module PagarMe
 
 		protected
 
-		def metaclass
-			class << self; self; end
-		end
-
-		def remove_attribute(keys)
-			metaclass.instance_eval do
-				keys.each do |key|
-					key_sym = :"#{key}="
-
-					if !@@existing_methods.include?(key)
-						remove_method(key) if method_defined?(key) 
-					end
-
-					remove_method(key_sym) if method_defined?(key_sym)
-				end
-			end
-		end
-
-		def add_attribute(keys)
-			metaclass.instance_eval do
-				keys.each do |key|
-					key_set = "#{key}="
-
-					if method_defined?(key)
-						@@existing_methods.push(key)
-					else
-						define_method(key) { @attributes[key] } 
-					end
-
-					define_method(key_set) do |value|
-						@attributes[key] = value
-						@unsaved_values.add(key)
-					end
-				end
-			end
-		end
-
 		def method_missing(name, *args)
-			if name.to_s.end_with?('=')
-				attr = name.to_s[0...-1].to_sym
-				add_attribute([attr])
-
-				begin
-					m = method(name)
-				rescue NameError
-					raise NoMethodError.new("O atributo #{name} nao e permitido.")
-				end
+			name = name.to_s
 
-				m.call *args
+			if name.end_with?('=')
+				attribute_name = name[0...-1]
+
+				@attributes[attribute_name] = args[0]
+				@unsaved_values.add(attribute_name)
 			else
-				@attributes[name] if @attributes.has_key?(name)
+				@attributes[name] || nil
 			end
 		end
 	end
 end
+

data/lib/pagarme/transaction_common.rb

@@ -2,73 +2,43 @@
 require File.join(File.dirname(__FILE__), '..', 'pagarme')
 
 module PagarMe
-  class TransactionCommon < Model
+	class TransactionCommon < Model
 
-	def initialize(response = {})
-	  super(response)
-	  self.payment_method = 'credit_card' unless self.payment_method
-	  self.installments = 1  unless self.installments
-	  self.status = 'local' unless self.status
-	end
-
-	def create
-	  check_card_object
-	  clear_card_data
-	  super
-	end
-
-	def save
-	  check_card_object
-	  clear_card_data
-	  super
-	end
-
-	def card_data_parameters
-	  {
-		:card_number => self.card_number,
-		:card_holder_name => self.card_holder_name,
-		:card_expiration_date => "#{self.card_expiration_month}#{self.card_expiration_year}",
-		:card_cvv => self.card_cvv
-	  }
-	end
+		def initialize(response = {})
+			super(response)
 
-	def generate_card_hash
-	  request = PagarMe::Request.new("/transactions/card_hash_key", 'GET')
-	  response = request.run
-
-	  public_key = OpenSSL::PKey::RSA.new(response['public_key'])
-	  ret = "#{response['id']}_#{Base64.strict_encode64(public_key.public_encrypt(card_data_parameters.to_params))}"
-	end
+			self.payment_method ||= 'credit_card'
+			self.installments ||= 1
+			self.status ||= 'local'
+		end
 
-	def should_generate_card_hash
-	  true
-	end
+		def create
+			check_card_object
+			super
+		end
 
-	private
-	  def check_card_object
-		if self.card
-		  if self.card.id
-			self.card_id = self.card.id
-		  else
-			self.card_number = self.card.card_number
-			self.card_holder_name = self.card.card_holder_name
-			self.card_expiration_year = self.card.card_expiration_year
-			self.card_expiration_month = self.card.card_expiration_month
-			self.card_cvv = self.card.card_cvv
-		  end
-		  self.card = nil
+		def save
+			check_card_object
+			super
 		end
-	  end
 
-	  def clear_card_data
-		if self.should_generate_card_hash
-		  self.card_hash = generate_card_hash unless self.card_hash || self.card_id
-		  self.card_number = nil
-		  self.card_holder_name = nil
-		  self.card_expiration_year = nil
-		  self.card_expiration_month = nil
-		  self.card_cvv = nil
+		private
+
+		def check_card_object
+			if self.card
+				if self.card.id
+					self.card_id = self.card.id
+				else
+					self.card_number = self.card.card_number
+					self.card_holder_name = self.card.card_holder_name
+					self.card_expiration_year = self.card.card_expiration_year
+					self.card_expiration_month = self.card.card_expiration_month
+					self.card_cvv = self.card.card_cvv
+				end
+				self.card = nil
+			end
+
+			self.card_expiration_date ||= "#{self.card_expiration_month}#{self.card_expiration_year}"
 		end
-	  end
-  end
+	end
 end

data/pagarme.gemspec

@@ -4,7 +4,7 @@ $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 
 Gem::Specification.new do |spec|
   spec.name          = "pagarme"
-  spec.version       = "2.0.0"
+  spec.version       = "2.0.1"
   spec.authors       = ["Pedro Franceschi", "Henrique Dubugras"]
   spec.email         = ["pedrohfranceschi@gmail.com", "henrique@pagar.me"]
   spec.description   = %q{Pagar.me's ruby gem}

data/test/pagarme/transfer.rb

@@ -4,22 +4,17 @@ require_relative '../test_helper'
 module PagarMe
   class TransferTest < Test::Unit::TestCase
     should 'be able to create a transfer' do
-      bank_account = test_bank_account
-      bank_account.create
-      transfer = test_transfer(bank_account_id: bank_account.id)
+      transfer = test_transfer
       transfer.create
 
       assert transfer.id != nil
       assert transfer.fee != nil
       assert transfer.object == "transfer"
       assert %w(doc credito_em_conta ted).include?(transfer.type)
-      assert transfer.bank_account.id.to_i == bank_account.id.to_i
     end
 
     should 'be able to find with id' do
-      bank_account = test_bank_account
-      bank_account.create
-      transfer_r = test_transfer(bank_account_id: bank_account.id)
+      transfer_r = test_transfer
       transfer_r.create
 
       transfer = PagarMe::Transfer.find_by_id transfer_r.id
@@ -28,7 +23,6 @@ module PagarMe
       assert transfer.fee != nil
       assert transfer.object == "transfer"
       assert %w(doc credito_em_conta ted).include?(transfer.type)
-      assert transfer.bank_account.id.to_i == bank_account.id.to_i
     end
   end
 end

data/test/test_helper.rb

@@ -61,7 +61,7 @@ def test_bank_account(params = {})
     :conta => '23398',
     :conta_dv => '9',
     :legal_name => 'foo bar loem',
-    :document_number => '111.111.111-11'
+    :document_number => '00000000000000'
   }.merge(params))
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pagarme
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - Pedro Franceschi
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-15 00:00:00.000000000 Z
+date: 2015-07-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler