dineromail 0.0.3 → 0.0.4

data/lib/dineromail/status_report.rb

@@ -3,12 +3,20 @@ require 'httparty'
 require 'dineromail/buyer'
 module Dineromail
   class StatusReport
-    attr_accessor :transaction_id, :date, :status, :amount, :net_amount, :pay_method, :pay_medium, :buyer, :items
+    attr_accessor :transaction_id, :date,:report_status, :status, :amount, :net_amount, :pay_method, :pay_medium, :buyer, :items
     
     PENDING_STATUS = 1
-    ACCREDITED_STATUS = 2
+    COMPLETED_STATUS = 2
     CANCELLED_STATUS = 3
     
+    VALID_REPORT_STATUS = 1
+    MALFORMED_REPORT_STATUS = 2
+    INVALID_ACCOUNT_NUMBER_REPORT_STATUS = 3
+    INVALID_PASSWORD_REPORT_STATUS = 4
+    INVALID_REQUEST_TYPE_STATUS = 5
+    INVALID_TRANSACTION_ID_REQUEST_STATUS = 6
+    INVALID_PASSWORD_OR_ACCOUNT_NUMBER_REQUEST_STATUS = 7
+    TRANSACTION_NOT_FOUND_REQUEST_STATUS = 8
     
     def initialize(transaction_id = nil)
       @items = []
@@ -17,6 +25,22 @@ module Dineromail
       end
     end
     
+    def valid_report?
+      report_status == VALID_REPORT_STATUS
+    end
+    
+    def pending?
+      status == PENDING_STATUS
+    end
+    
+    def completed?
+      status == ACCREDITED_STATUS
+    end
+    
+    def cancelled?
+      status == CANCELLED_STATUS
+    end
+    
     def obtain_status_report_data_for(transaction_id)
       self.transaction_id = transaction_id
       account_number = Dineromail.configuration.account_number
@@ -40,31 +64,35 @@ module Dineromail
    
     def parse_response(response)
       response_data = XmlSimple.xml_in(response,'KeyToSymbol' => true )
-      operation = response_data[:detalle].first[:operaciones].first[:operacion].first
-      self.transaction_id = operation[:id].first.to_i
-      self.date = DateTime.parse operation[:fecha].first
-      self.status = operation[:estado].first.to_i
-      self.amount = operation[:monto].first.to_f
-      self.net_amount = operation[:montoneto].first.to_f
-      self.pay_method = operation[:metodopago].first
-      self.pay_medium = operation[:mediopago].first
-      buyer_data = operation[:comprador].first
-      self.buyer = Buyer.new
-      buyer.email = buyer_data[:email].first
-      buyer.address = buyer_data[:direccion].first
-      buyer.comment = buyer_data[:comentario].first
-      buyer.name = buyer_data[:nombre].first
-      buyer.phone = buyer_data[:telefono].first
-      buyer.document_type = buyer_data[:tipodoc].first
-      buyer.document_number = buyer_data[:numerodoc].first
-      items_data = operation[:items].first[:item]
-      items_data.each do |item_data|
+      self.report_status = response_data[:estadoreporte].first.to_i
+      operations = response_data[:detalle].first[:operaciones].first
+      if operations
+        operation = operations[:operacion].first
+        self.transaction_id = operation[:id].first.to_i
+        self.date = DateTime.parse operation[:fecha].first
+        self.status = operation[:estado].first.to_i
+        self.amount = operation[:monto].first.to_f
+        self.net_amount = operation[:montoneto].first.to_f
+        self.pay_method = operation[:metodopago].first
+        self.pay_medium = operation[:mediopago].first
+        buyer_data = operation[:comprador].first
+        self.buyer = Buyer.new
+        buyer.email = buyer_data[:email].first
+        buyer.address = buyer_data[:direccion].first
+        buyer.comment = buyer_data[:comentario].first
+        buyer.name = buyer_data[:nombre].first
+        buyer.phone = buyer_data[:telefono].first
+        buyer.document_type = buyer_data[:tipodoc].first
+        buyer.document_number = buyer_data[:numerodoc].first
+        items_data = operation[:items].first[:item]
+        items_data.each do |item_data|
           item = Item.new
           item.description = item_data[:descripcion].first
           item.currency = item_data[:moneda].first.to_i
           item.unit_price = item_data[:preciounitario].first.to_f
           item.count = item_data[:cantidad].first.to_i
           self.items << item
+        end
       end
     end
     

data/lib/dineromail/version.rb

@@ -1,3 +1,3 @@
 module Dineromail
-  VERSION = "0.0.3"
+  VERSION = "0.0.4"
 end

data/spec/dineromail/status_report_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe Dineromail::StatusReport do
   it 'should load the status report from xml' do
-    xml = '<REPORTE><ESTADOREPORTE></ESTADOREPORTE><DETALLE><OPERACIONES><OPERACION>
+    xml = '<REPORTE><ESTADOREPORTE>1</ESTADOREPORTE><DETALLE><OPERACIONES><OPERACION>
           <ID>1889</ID>
           <FECHA>01/28/2011 12:02:01 PM</FECHA>
           <ESTADO>1</ESTADO>
@@ -35,6 +35,8 @@ describe Dineromail::StatusReport do
     buyer = status_report.buyer
     item = status_report.items.first
     
+    status_report.report_status.should == 1
+    status_report.valid_report?.should be_true
     status_report.transaction_id.should == 1889
     status_report.date.should == DateTime.ordinal(2011,28,12,2,1)
     status_report.status.should == Dineromail::StatusReport::PENDING_STATUS

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: dineromail
 version: !ruby/object:Gem::Version 
-  hash: 25
+  hash: 23
   prerelease: false
   segments: 
   - 0
   - 0
-  - 3
-  version: 0.0.3
+  - 4
+  version: 0.0.4
 platform: ruby
 authors: 
 - Nicolas Mosconi