pagseguro_catcher 0.0.3 → 0.0.4

data/README.textile

@@ -6,6 +6,22 @@ h2. Soon
 
 I'll describe how to use this one, but it's easy :-)
 
+h2. TODO
+
+h3. 0.1.0
+
+* parse correctly all the sections of the return (items, sender, shipping are missing);
+* implement a @method_missing@ in @::Transaction::Base@ to delegate to @#[]@
+
+h3. 0.2.0
+
+* implement a nice way to rescue the @::Checker#check@ method, when the request isn't successfull or the return is invalid
+
+h3. 0.3.0:
+
+* maybe some nice logs?
+
+
 h2. Contributing to pagseguro_catcher
  
 * Check out the latest master to make sure the feature hasn't been implemented or the bug hasn't been fixed yet

data/VERSION

@@ -1 +1 @@
-0.0.3
+0.0.4

data/lib/pagseguro_catcher/constants.rb

@@ -1,6 +1,6 @@
 # encoding: utf-8
 module PagseguroCatcher
-  #maybe YML?... this doesn't seem right...
+  XML_CHARSET = "ISO-8859-1"
   
   TRANSACTION_TYPES  = { 
     1 => "Pagamento", 

data/lib/pagseguro_catcher/core_ext/string.rb

@@ -0,0 +1,5 @@
+class String
+  def utf8
+    Iconv.conv(PagseguroCatcher::XML_CHARSET, "UTF-8", self)
+  end
+end

data/lib/pagseguro_catcher/transaction/amount.rb

@@ -1,31 +1,30 @@
 module PagseguroCatcher
   module Transaction
     
-    class Amount
-      attr_accessor :body
+    class Amount < Transaction::Body
       
       def initialize(body)
         self.body = body
       end
       
       def gross
-        self.body[:grossAmount].to_f
+        self[:grossAmount].to_f
       end
       
       def discount
-        self.body[:discountAmount].to_f
+        self[:discountAmount].to_f
       end
       
       def fee
-        self.body[:feeAmount].to_f
+        self[:feeAmount].to_f
       end
       
       def net
-        self.body[:netAmount].to_f
+        self[:netAmount].to_f
       end
       
       def extra
-        self.body[:extraAmount].to_f
+        self[:extraAmount].to_f
       end
       
     end

data/lib/pagseguro_catcher/transaction/base.rb

@@ -1,8 +1,8 @@
+#p = PagseguroCatcher::Transaction::Base.new(File.open('spec/support/return.xml').read)
 module PagseguroCatcher
   module Transaction
     
-    class Base
-      attr_accessor :body
+    class Base < Transaction::Body
 
       def initialize(xml)
         self.body = Hash.from_xml(xml)["transaction"]
@@ -14,8 +14,23 @@ module PagseguroCatcher
         @amount
       end
       
-      def [](param)
-        self.body[param.to_sym]
+      def sender
+        @sender ||= PagseguroCatcher::Transaction::Sender.new(self.body)
+        @sender
+      end
+      
+      def items
+        @items = []
+        
+        self[:items][:item].each do |item|
+          @items << PagseguroCatcher::Transaction::Item.new(item)
+        end
+        
+        @items
+      end
+      
+      def each_item
+        self.items.each { |item| yield item }
       end
       
       def date
@@ -42,7 +57,7 @@ module PagseguroCatcher
         PAYMENT_CODES[self[:paymentMethod][:code].to_i]
       end
       
-      #TODO - method missing
+      #TODO - items, shipping
       
     end
     

data/lib/pagseguro_catcher/transaction/item.rb

@@ -0,0 +1,16 @@
+module PagseguroCatcher
+  module Transaction
+    
+    class Item < Transaction::Body
+      
+      def initialize(body)
+        self.body = body
+      end
+      
+      def amount
+        self[:amount].to_f
+      end
+    end
+    
+  end
+end

data/lib/pagseguro_catcher/transaction/sender.rb

@@ -0,0 +1,25 @@
+module PagseguroCatcher
+  module Transaction
+    
+    class Sender < Transaction::Body
+      
+      def initialize(body)
+        self.body = body[:sender]
+      end
+      
+      def name
+        self[:name]
+      end
+      
+      def email
+        self[:email]
+      end
+      
+      def phone
+        { :area => self[:phone][:areaCode], :number => self[:phone][:number] }
+      end
+
+    end
+    
+  end
+end

data/lib/pagseguro_catcher/transaction.rb

@@ -1,6 +1,23 @@
 module PagseguroCatcher
   module Transaction
+    
+    class Body
+      attr_accessor :body
+      
+      def [](param)
+        self.body[param.to_sym]
+      end
+      
+      def method_missing(name, *args)
+        return self[name] if self.body.has_key?(name.to_sym)
+        super
+      end
+      
+    end
+    
     autoload :Base, 'pagseguro_catcher/transaction/base'
     autoload :Amount, 'pagseguro_catcher/transaction/amount'
+    autoload :Sender, 'pagseguro_catcher/transaction/sender'
+    autoload :Item, 'pagseguro_catcher/transaction/item'
   end
 end

data/lib/pagseguro_catcher.rb

@@ -3,6 +3,7 @@
 
 require "active_support/core_ext"
 require "pagseguro_catcher/core_ext/Hash"
+require "pagseguro_catcher/core_ext/string"
 require "pagseguro_catcher/constants"
 require "pagseguro_catcher/transaction"
 

data/pagseguro_catcher.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{pagseguro_catcher}
-  s.version = "0.0.3"
+  s.version = "0.0.4"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.authors = [%q{Matias H. Leidemer}]
-  s.date = %q{2011-10-11}
+  s.authors = ["Matias H. Leidemer"]
+  s.date = %q{2011-10-15}
   s.description = %q{This gem provides a simple way to check and parse the PagSeguro transaction notification.}
   s.email = %q{matiasleidemer@gmail.com}
   s.extra_rdoc_files = [
@@ -30,24 +30,30 @@ Gem::Specification.new do |s|
     "lib/pagseguro_catcher/checker.rb",
     "lib/pagseguro_catcher/constants.rb",
     "lib/pagseguro_catcher/core_ext/Hash.rb",
+    "lib/pagseguro_catcher/core_ext/string.rb",
     "lib/pagseguro_catcher/transaction.rb",
     "lib/pagseguro_catcher/transaction/amount.rb",
     "lib/pagseguro_catcher/transaction/base.rb",
+    "lib/pagseguro_catcher/transaction/item.rb",
+    "lib/pagseguro_catcher/transaction/sender.rb",
     "pagseguro_catcher.gemspec",
     "spec/checker_spec.rb",
     "spec/pagseguro_catcher_spec.rb",
     "spec/spec_helper.rb",
     "spec/support/return.xml",
     "spec/transaction/amount_spec.rb",
-    "spec/transaction/base_spec.rb"
+    "spec/transaction/base_spec.rb",
+    "spec/transaction/item_spec.rb",
+    "spec/transaction/sender_spec.rb"
   ]
   s.homepage = %q{http://github.com/matiasleidemer/pagseguro_catcher}
-  s.licenses = [%q{MIT}]
-  s.require_paths = [%q{lib}]
-  s.rubygems_version = %q{1.8.6}
+  s.licenses = ["MIT"]
+  s.require_paths = ["lib"]
+  s.rubygems_version = %q{1.3.7}
   s.summary = %q{A simple gem to parse PagSeguro transaction notification.}
 
   if s.respond_to? :specification_version then
+    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
     s.specification_version = 3
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then

data/spec/support/return.xml

@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="ISO-8859-1" standalone="yes"?>  
+<?xml version="1.0" standalone="yes"?>  
 <transaction>  
     <date>2011-02-10T16:13:41.000-03:00</date>  
     <code>9E884542-81B3-4419-9A75-BCC6FB495EF1</code>  

data/spec/transaction/amount_spec.rb

@@ -14,38 +14,32 @@ describe "Amount" do
   
   describe "#gross" do
     it "returns the gross amount as a float" do
-      amount.body[:grossAmount] = "49900.00"
       amount.gross.should == 49900.00
     end
   end
   
   describe "#discount" do
     it "returns the discount amount as a float" do
-      amount.body[:discountAmount] = "900.00"
-      amount.discount.should == 900.00
+      amount.discount.should == 0.00
     end
   end
   
   describe "#fee" do
     it "returns the fee amount as a float" do
-      amount.body[:feeAmount] = "150.00"
-      amount.fee.should == 150.00
+      amount.fee.should == 0.00
     end
   end
 
   describe "#net" do
     it "returns the net amount as a float" do
-      amount.body[:netAmount] = "160.00"
-      amount.net.should == 160.00
+      amount.net.should == 49900.00
     end
   end
   
   describe "#extra" do
     it "returns the extra amount as a float" do
-      amount.body[:extraAmount] = "170.00"
-      amount.extra.should == 170.00
+      amount.extra.should == 0.00
     end
   end
-
   
 end

data/spec/transaction/base_spec.rb

@@ -25,6 +25,27 @@ describe "Transaction" do
     end
   end
   
+  describe "#sender" do
+    it "returns an Sender object" do
+      transaction.sender.should be_a(PagseguroCatcher::Transaction::Sender)
+    end
+  end
+  
+  describe "#items" do
+    it "returns an array of items objects" do
+      transaction.items.should be_a(Array)
+      transaction.items.first.should be_a(PagseguroCatcher::Transaction::Item)
+    end
+  end
+  
+  describe "#each_item" do
+    it "yields each item" do
+      transaction.each_item do |item|
+        item.should be_a(PagseguroCatcher::Transaction::Item)
+      end
+    end
+  end
+  
   describe "#date" do
     it "returns the date response as a DateTime object" do
       transaction.body[:date] = "2011-02-10T16:13:41.000-03:00"
@@ -32,7 +53,7 @@ describe "Transaction" do
     end
   end
   
-  describe "#last_even_date" do
+  describe "#last_event_date" do
     it "returns the last event date response as a DateTime object" do
       transaction.body[:lastEventDate] = "2011-10-04T15:48:59.000-03:00"
       transaction.last_event_date.should == DateTime.new(2011, 10, 04, 15, 48, 59, "-03:00")
@@ -54,17 +75,29 @@ describe "Transaction" do
   end
   
   describe "#payment_method_type" do
-    it "return the human value for the payment method" do
+    it "returns the human value for the payment method" do
       transaction.body[:paymentMethod][:type] = "1"
       transaction.payment_method_type.should == "Cartão de crédito"
     end
   end
   
   describe "#payment_method_code" do
-    it "return the human value for the payment method" do
+    it "returns the human value for the payment method" do
       transaction.body[:paymentMethod][:code] = "101"
       transaction.payment_method_code.should == "Cartão de crédito Visa"
     end
   end
   
+  describe "#method_missing" do
+    it "tries to run #[] with the method name" do
+      transaction.should_not respond_to :installmentCount
+      transaction.installmentCount.should == "1"
+    end
+    
+    it "raises exception if the parameter does not exist" do
+      transaction.should_not respond_to :non_existant_method
+      lambda { transaction.non_existant_method }.should raise_error(NoMethodError)
+    end
+  end
+  
 end

data/spec/transaction/item_spec.rb

@@ -0,0 +1,21 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe "Item" do
+  
+  let(:xml)         { File.open(File.expand_path(File.dirname(__FILE__) + '/../support/return.xml')).read }
+  let(:transaction) { PagseguroCatcher::Transaction::Base.new(xml) }
+  let(:item)        { PagseguroCatcher::Transaction::Item.new(transaction[:items][:item].first) }
+  
+  describe ".initialize" do
+    it "assigns the body" do
+      item.body.should be_a(Hash)
+    end
+  end
+  
+  describe "#amount" do
+    it "returns the amount as a float" do
+      item.amount.should == 24300.00
+    end
+  end
+      
+end

data/spec/transaction/sender_spec.rb

@@ -0,0 +1,36 @@
+# -*- encoding: utf-8 -*-
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe "Sender" do
+  
+  let(:xml)         { File.open(File.expand_path(File.dirname(__FILE__) + '/../support/return.xml')).read }
+  let(:transaction) { PagseguroCatcher::Transaction::Base.new(xml) }
+  let(:sender)      { PagseguroCatcher::Transaction::Sender.new(transaction.body) }
+  
+  describe ".initialize" do
+    it "assigns the body" do
+      sender.body.should be_a(Hash)
+    end
+  end
+  
+  describe "#name" do
+    it "returns the senders name" do
+      sender.name.should eql "José Comprador"
+    end
+  end
+
+  describe "#email" do
+    it "returns the senders email" do
+      sender.email.should eql "comprador@uol.com.br"
+    end
+  end
+  
+  describe "#phone" do
+    it "returns the senders phone" do
+      phone = sender.phone
+      phone.should == { :area => "11", :number => "56273440" }
+    end
+  end
+  
+  
+end

metadata

@@ -1,113 +1,151 @@
---- !ruby/object:Gem::Specification
+--- !ruby/object:Gem::Specification 
 name: pagseguro_catcher
-version: !ruby/object:Gem::Version
-  version: 0.0.3
-  prerelease: 
+version: !ruby/object:Gem::Version 
+  hash: 23
+  prerelease: false
+  segments: 
+  - 0
+  - 0
+  - 4
+  version: 0.0.4
 platform: ruby
-authors:
+authors: 
 - Matias H. Leidemer
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-10-11 00:00:00.000000000Z
-dependencies:
-- !ruby/object:Gem::Dependency
+
+date: 2011-10-15 00:00:00 -03:00
+default_executable: 
+dependencies: 
+- !ruby/object:Gem::Dependency 
+  type: :runtime
+  prerelease: false
   name: httparty
-  requirement: &70216239563300 !ruby/object:Gem::Requirement
+  version_requirements: &id001 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  requirement: *id001
+- !ruby/object:Gem::Dependency 
   type: :runtime
   prerelease: false
-  version_requirements: *70216239563300
-- !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70216239562580 !ruby/object:Gem::Requirement
+  version_requirements: &id002 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  requirement: *id002
+- !ruby/object:Gem::Dependency 
   type: :runtime
   prerelease: false
-  version_requirements: *70216239562580
-- !ruby/object:Gem::Dependency
   name: i18n
-  requirement: &70216239561880 !ruby/object:Gem::Requirement
+  version_requirements: &id003 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  requirement: *id003
+- !ruby/object:Gem::Dependency 
+  type: :development
   prerelease: false
-  version_requirements: *70216239561880
-- !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70216239561360 !ruby/object:Gem::Requirement
+  version_requirements: &id004 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 2
+        - 3
+        - 0
         version: 2.3.0
+  requirement: *id004
+- !ruby/object:Gem::Dependency 
   type: :development
   prerelease: false
-  version_requirements: *70216239561360
-- !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &70216239560820 !ruby/object:Gem::Requirement
+  version_requirements: &id005 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
+        hash: 23
+        segments: 
+        - 1
+        - 0
+        - 0
         version: 1.0.0
+  requirement: *id005
+- !ruby/object:Gem::Dependency 
   type: :development
   prerelease: false
-  version_requirements: *70216239560820
-- !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &70216239560220 !ruby/object:Gem::Requirement
+  version_requirements: &id006 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
+        hash: 7
+        segments: 
+        - 1
+        - 6
+        - 4
         version: 1.6.4
+  requirement: *id006
+- !ruby/object:Gem::Dependency 
   type: :development
   prerelease: false
-  version_requirements: *70216239560220
-- !ruby/object:Gem::Dependency
   name: rcov
-  requirement: &70216239559400 !ruby/object:Gem::Requirement
+  version_requirements: &id007 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  requirement: *id007
+- !ruby/object:Gem::Dependency 
   type: :development
   prerelease: false
-  version_requirements: *70216239559400
-- !ruby/object:Gem::Dependency
   name: fakeweb
-  requirement: &70216239558560 !ruby/object:Gem::Requirement
+  version_requirements: &id008 !ruby/object:Gem::Requirement 
     none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: *70216239558560
-description: This gem provides a simple way to check and parse the PagSeguro transaction
-  notification.
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  requirement: *id008
+description: This gem provides a simple way to check and parse the PagSeguro transaction notification.
 email: matiasleidemer@gmail.com
 executables: []
+
 extensions: []
-extra_rdoc_files:
+
+extra_rdoc_files: 
 - LICENSE.txt
 - README.textile
-files:
+files: 
 - .document
 - .rspec
 - .travis.yml
@@ -121,9 +159,12 @@ files:
 - lib/pagseguro_catcher/checker.rb
 - lib/pagseguro_catcher/constants.rb
 - lib/pagseguro_catcher/core_ext/Hash.rb
+- lib/pagseguro_catcher/core_ext/string.rb
 - lib/pagseguro_catcher/transaction.rb
 - lib/pagseguro_catcher/transaction/amount.rb
 - lib/pagseguro_catcher/transaction/base.rb
+- lib/pagseguro_catcher/transaction/item.rb
+- lib/pagseguro_catcher/transaction/sender.rb
 - pagseguro_catcher.gemspec
 - spec/checker_spec.rb
 - spec/pagseguro_catcher_spec.rb
@@ -131,32 +172,41 @@ files:
 - spec/support/return.xml
 - spec/transaction/amount_spec.rb
 - spec/transaction/base_spec.rb
+- spec/transaction/item_spec.rb
+- spec/transaction/sender_spec.rb
+has_rdoc: true
 homepage: http://github.com/matiasleidemer/pagseguro_catcher
-licenses:
+licenses: 
 - MIT
 post_install_message: 
 rdoc_options: []
-require_paths:
+
+require_paths: 
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement
+required_ruby_version: !ruby/object:Gem::Requirement 
   none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
-      segments:
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
       - 0
-      hash: -4499129393277897347
-required_rubygems_version: !ruby/object:Gem::Requirement
+      version: "0"
+required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
+      version: "0"
 requirements: []
+
 rubyforge_project: 
-rubygems_version: 1.8.6
+rubygems_version: 1.3.7
 signing_key: 
 specification_version: 3
 summary: A simple gem to parse PagSeguro transaction notification.
 test_files: []
+