xeroizer-float 2.15.3.13 → 2.15.3.14

data/Gemfile

@@ -3,6 +3,7 @@ source "http://rubygems.org"
 gem 'builder',        '>= 2.1.2'
 gem 'oauth',          '0.4.5'
 gem 'activesupport'
+gem 'tzinfo'
 gem 'nokogiri'
 gem 'i18n'
 

data/Gemfile.lock

@@ -35,6 +35,7 @@ GEM
     test-unit (2.4.8)
     turn (0.9.4)
       ansi
+    tzinfo (0.3.35)
     yard (0.7.5)
 
 PLATFORMS
@@ -54,4 +55,5 @@ DEPENDENCIES
   shoulda
   test-unit
   turn
+  tzinfo
   yard

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2011 Wayne Robinson
+Copyright (c) 2013 Wayne Robinson
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the
@@ -18,3 +18,24 @@ NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
 LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
 OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
 WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+
+-----------------------------------------------------------------------
+
+Part of the HTTP communication and OAuth authentication library were
+provided by the https://github.com/tlconnor/xero_gateway gem created by 
+Tim Connor (github.com/tlconnor) and Nik Wakelin (github.com/nikz). 
+The license for this Gem is included below:
+
+Copyright (c) 2008 Tim Connor <tlconnor@gmail.com>
+
+Permission to use, copy, modify, and/or distribute this software for any
+purpose with or without fee is hereby granted, provided that the above
+copyright notice and this permission notice appear in all copies.
+
+THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.

data/README.md

@@ -6,7 +6,7 @@ Xeroizer API Library ![Project status](http://stillmaintained.com/waynerobinson/
 **Github**: 			[https://github.com/waynerobinson/xeroizer](https://github.com/waynerobinson/xeroizer)		
 **Author**: 			Wayne Robinson [http://www.wayne-robinson.com](http://www.wayne-robinson.com)		
 **Contributors**: See Contributors section below	
-**Copyright**:    2007-2010		
+**Copyright**:    2007-2013
 **License**:      MIT License		
 
 Introduction
@@ -435,9 +435,30 @@ contact.save
 Have a look at the models in `lib/xeroizer/models/` to see the valid attributes, associations and 
 minimum validation requirements for each of the record types.
 
+### Bulk Creates & Updates
+
+Xero has a hard daily limit on the number of API requests you can make (currently 1,000 requests
+per account per day). To save on requests, you can batch creates and updates into a single PUT or
+POST call, like so:
+
+```ruby
+contact1 = xero.Contact.create(some_attributes)
+xero.Contact.batch_save do
+  contact1.email_address = "foo@bar.com"
+  contact2 = xero.Contact.build(some_other_attributes)
+  contact3 = xero.Contact.build(some_more_attributes)
+end
+```
+
+`batch_save` will issue one PUT request for every 2,000 unsaved records built within its block, and one
+POST request for evert 2,000 existing records that have been altered within its block. If any of the
+unsaved records aren't valid, it'll return `false` before sending anything across the wire;
+otherwise, it returns `true`. `batch_save` takes one optional argument: the number of records to
+create/update per request. (Defaults to 2,000.)
+
 ### Errors
 
-If a record doesn't match it's internal validation requirements the `#save` method will return
+If a record doesn't match its internal validation requirements, the `#save` method will return
 `false` and the `#errors` attribute will be populated with what went wrong.
 
 For example:
@@ -527,3 +548,9 @@ client = Xeroizer::PublicApplication.new(YOUR_OAUTH_CONSUMER_KEY,
                                          YOUR_OAUTH_CONSUMER_SECRET,
                                          :rate_limit_sleep => 2)
 ```
+
+
+### Contributors
+Xeroizer was inspired by the https://github.com/tlconnor/xero_gateway gem created by Tim Connor 
+and Nik Wakelin and portions of the networking and authentication code are based completely off 
+this project. Copyright for these components remains held in the name of Tim Connor.

data/VERSION

@@ -1 +1 @@
-2.15.3.13
+2.15.3.14

data/lib/xeroizer/exceptions.rb

@@ -1,12 +1,27 @@
+# Copyright (c) 2008 Tim Connor <tlconnor@gmail.com>
+# 
+# Permission to use, copy, modify, and/or distribute this software for any
+# purpose with or without fee is hereby granted, provided that the above
+# copyright notice and this permission notice appear in all copies.
+# 
+# THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+# WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+# MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+# ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+# WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+# ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+# OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+
 module Xeroizer
   class ApiException < StandardError
     
-    attr_reader :type, :message, :xml, :request_body
+    attr_reader :type, :message, :xml, :parsed_xml, :request_body
     
-    def initialize(type, message, xml, request_body)
+    def initialize(type, message, xml, parsed_xml, request_body)
       @type         = type
       @message      = message
       @xml          = xml
+      @parsed_xml   = parsed_xml
       @request_body = request_body
     end
     

data/lib/xeroizer/http.rb

@@ -1,3 +1,17 @@
+# Copyright (c) 2008 Tim Connor <tlconnor@gmail.com>
+# 
+# Permission to use, copy, modify, and/or distribute this software for any
+# purpose with or without fee is hereby granted, provided that the above
+# copyright notice and this permission notice appear in all copies.
+# 
+# THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+# WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+# MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+# ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+# WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+# ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+# OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+
 module Xeroizer
   module Http
     
@@ -69,7 +83,7 @@ module Xeroizer
 
         begin
           attempts += 1
-          logger.info("\n== [#{Time.now.to_s}] XeroGateway Request: #{uri.request_uri} ") if self.logger
+          logger.info("\n== [#{Time.now.to_s}] XeroGateway Request: #{method.to_s.upcase} #{uri.request_uri} ") if self.logger
 
           response = case method
             when :get   then    client.get(uri.request_uri, headers)
@@ -133,16 +147,17 @@ module Xeroizer
         
         # XeroGenericApplication API Exceptions *claim* to be UTF-16 encoded, but fail REXML/Iconv parsing...
         # So let's ignore that :)
-        # raw_response.gsub! '<?xml version="1.0" encoding="utf-16"?>', ''
+        raw_response.gsub! '<?xml version="1.0" encoding="utf-16"?>', ''
         
         # doc = REXML::Document.new(raw_response, :ignore_whitespace_nodes => :all)
         doc = Nokogiri::XML(raw_response)
         
         if doc && doc.root && doc.root.name == "ApiException"
 
-          raise ApiException.new(doc.root.xpath("Type").text, 
-                                 doc.root.xpath("Message").text, 
+          raise ApiException.new(doc.root.xpath("Type").text,
+                                 doc.root.xpath("Message").text,
                                  raw_response,
+                                 doc,
                                  request_body)
 
         else

data/lib/xeroizer/models/bank_transaction.rb

@@ -34,6 +34,9 @@ module Xeroizer
       boolean       :is_reconciled
       string        :status
 
+      string        :currency_code
+      decimal       :currency_rate, :calculated => true
+
       alias_method :reconciled?, :is_reconciled
 
       belongs_to :contact, :model_name => 'Contact'
@@ -55,6 +58,14 @@ module Xeroizer
         self.line_items.size > 0
       end
 
+      def currency_rate
+        if attributes[:currency_rate]
+          BigDecimal.new(attributes[:currency_rate])
+        else
+          BigDecimal.new('1.0')
+        end
+      end
+
       def sub_total=(value); raise SettingTotalDirectlyNotSupported.new(:sub_total); end
       def total_tax=(value); raise SettingTotalDirectlyNotSupported.new(:total_tax); end
       def total=(value);     raise SettingTotalDirectlyNotSupported.new(:total);     end

data/lib/xeroizer/models/credit_note.rb

@@ -57,6 +57,7 @@ module Xeroizer
       decimal       :total, :calculated => true
       datetime_utc  :updated_date_utc, :api_name => 'UpdatedDateUTC'
       string        :currency_code
+      decimal       :currency_rate, :calculated => true
       datetime      :fully_paid_on_date
       boolean       :sent_to_contact
       
@@ -81,6 +82,14 @@ module Xeroizer
         def contact_id
           attributes[:contact] && attributes[:contact][:contact_id]
         end      
+
+        def currency_rate
+          if attributes[:currency_rate]
+            BigDecimal.new(attributes[:currency_rate])
+          else
+            BigDecimal.new('1.0')
+          end
+        end
       
         # Swallow assignment of attributes that should only be calculated automatically.
         def sub_total=(value);  raise SettingTotalDirectlyNotSupported.new(:sub_total);   end

data/lib/xeroizer/models/invoice.rb

@@ -2,7 +2,10 @@ module Xeroizer
   module Record
     
     class InvoiceModel < BaseModel
-        
+      # To create a new invoice, use the folowing
+      # $xero_client.Invoice.build(type: 'ACCREC', ..., contact: {name: 'Foo Bar'},...)
+      # Note that we are not making an api request to xero just to get the contact
+            
       set_permissions :read, :write, :update
       
       public
@@ -62,6 +65,7 @@ module Xeroizer
       decimal      :amount_credited
       datetime_utc :updated_date_utc, :api_name => 'UpdatedDateUTC'
       string       :currency_code
+      decimal      :currency_rate, :calculated => true
       datetime     :fully_paid_on_date
       boolean      :sent_to_contact
       
@@ -106,6 +110,14 @@ module Xeroizer
         def accounts_receivable?
           type == 'ACCREC'
         end
+
+        def currency_rate
+          if attributes[:currency_rate]
+            BigDecimal.new(attributes[:currency_rate])
+          else
+            BigDecimal.new('1.0')
+          end
+        end
         
         def sub_total=(sub_total)
           @sub_total_is_set = true

data/lib/xeroizer/models/manual_journal.rb

@@ -12,7 +12,7 @@ module Xeroizer
       JOURNAL_STATUS = {
         'DRAFT' =>      'Draft',
         'POSTED' =>     'Posted'
-      } unless defined?(INVOICE_TYPE)
+      } unless defined?(JOURNAL_STATUS)
       JOURNAL_STATUSES = JOURNAL_STATUS.keys.sort
             
       set_primary_key :manual_journal_id

data/lib/xeroizer/oauth.rb

@@ -1,3 +1,17 @@
+# Copyright (c) 2008 Tim Connor <tlconnor@gmail.com>
+# 
+# Permission to use, copy, modify, and/or distribute this software for any
+# purpose with or without fee is hereby granted, provided that the above
+# copyright notice and this permission notice appear in all copies.
+# 
+# THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+# WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+# MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+# ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+# WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+# ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+# OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+
 module Xeroizer
   
   # Shamelessly taken from the XeroGateway library by Tim Connor which is shamelessly 
@@ -50,6 +64,7 @@ module Xeroizer
     # @option options [String] :request_token_path base URL path for getting a RequestToken (default: "/oauth/RequestToken")
     # @option options [String] :signature_method method usd to sign requests (default: OAuth library default)
     # @option options [String] :site base site for API requests (default: "https://api.xero.com")
+    # @option options [IO] :http_debug_output filehandle to write HTTP traffic to
     # @option options [OpenSSL:X509::Certificate] :ssl_client_cert client-side SSL certificate to use for requests (used for PartnerApplication mode)
     # @option options [OpenSSL::PKey::RSA] :ssl_client_key client-side SSL private key to use for requests (used for PartnerApplication mode)
     def initialize(ctoken, csecret, options = {})
@@ -104,9 +119,9 @@ module Xeroizer
       })
       update_attributes_from_token(access_token)
     end
-    
-    private 
-    
+
+    private
+
       # Create an OAuth consumer with the SSL client key if specified in @consumer_options when
       # this instance was created.
       def create_consumer
@@ -116,8 +131,13 @@ module Xeroizer
           consumer.http.key = @consumer_options[:ssl_client_key]
         end
         consumer
+
+        if @consumer_options[:http_debug_output]
+          consumer.http.set_debug_output(@consumer_options[:http_debug_output])
+        end
+        consumer
       end
-      
+
       # Update instance variables with those from the AccessToken.
       def update_attributes_from_token(access_token)
         @expires_at = Time.now + access_token.params[:oauth_expires_in].to_i
@@ -125,6 +145,5 @@ module Xeroizer
         @session_handle = access_token.params[:oauth_session_handle]
         @atoken, @asecret = access_token.token, access_token.secret
       end
-          
   end
 end

data/lib/xeroizer/record/base.rb

@@ -51,11 +51,13 @@ module Xeroizer
         end
         
         def []=(attribute, value)
+          parent.mark_dirty(self) if parent
           self.send("#{attribute}=".to_sym, value)
         end
 
         def attributes=(new_attributes)
           return unless new_attributes.is_a?(Hash)
+          parent.mark_dirty(self) if parent
           new_attributes.each do | key, value |
             self.send("#{key}=".to_sym, value)
           end
@@ -84,6 +86,7 @@ module Xeroizer
           record = self.parent.find(self.id)
           @attributes = record.attributes if record
           @complete_record_downloaded = true
+          parent.mark_clean(self)
           self
         end
         
@@ -94,9 +97,30 @@ module Xeroizer
           else
             update
           end
+          saved!
+        end
+
+        def saved!
           @complete_record_downloaded = true
+          parent.mark_clean(self)
           true
         end
+        
+        def to_json(*args)
+          to_h.to_json(*args)
+        end
+
+        # Deprecated
+        def as_json(options = {})
+          to_h.to_json
+        end
+
+        def to_h
+          attrs = self.attributes.reject {|k, v| k == :parent }.map do |k, v|
+            [k, v.kind_of?(Array) ? v.map(&:to_h) : (v.respond_to?(:to_h) ? v.to_h : v)]
+          end
+          Hash[attrs]
+        end
 
         def inspect
           attribute_string = self.attributes.collect do |attr, value|
@@ -112,7 +136,7 @@ module Xeroizer
           request = to_xml
           log "[CREATE SENT] (#{__FILE__}:#{__LINE__}) #{request}"
           
-          response = parent.http_put(request)
+          response = parent.http_post(request)
           log "[CREATE RECEIVED] (#{__FILE__}:#{__LINE__}) #{response}"
           
           parse_save_response(response)

data/lib/xeroizer/record/base_model.rb

@@ -18,7 +18,9 @@ module Xeroizer
       class_inheritable_attributes :xml_root_name
       class_inheritable_attributes :optional_xml_root_name
       class_inheritable_attributes :xml_node_name
-      
+
+      DEFAULT_RECORDS_PER_BATCH_SAVE = 2000
+
       include BaseModelHttpProxy
 
       attr_reader :application
@@ -89,7 +91,22 @@ module Xeroizer
         
         # Build a record with attributes set to the value of attributes.
         def build(attributes = {})
-          model_class.build(attributes, self)
+          model_class.build(attributes, self).tap do |resource|
+            mark_dirty(resource)
+          end
+        end
+
+        def mark_dirty(resource)
+          if @allow_batch_operations
+            @objects[model_class] ||= {}
+            @objects[model_class][resource.object_id] ||= resource
+          end
+        end
+
+        def mark_clean(resource)
+          if @objects and @objects[model_class]
+            @objects[model_class].delete(resource.object_id)
+          end
         end
 
         # Create (build and save) a record with attributes set to the value of attributes.
@@ -122,7 +139,37 @@ module Xeroizer
           result.complete_record_downloaded = true if result
           result
         end
-        
+
+        def batch_save(chunk_size = DEFAULT_RECORDS_PER_BATCH_SAVE)
+          @objects = {}
+          @allow_batch_operations = true
+
+          yield
+
+          if @objects[model_class]
+            objects = @objects[model_class].values.compact
+            return false unless objects.all?(&:valid?)
+            actions = objects.group_by {|o| o.new_record? ? :http_put : :http_post }
+            actions.each_pair do |http_method, records|
+              records.each_slice(chunk_size) do |some_records|
+                request = to_bulk_xml(some_records)
+                response = parse_response(self.send(http_method, request, {:summarizeErrors => false}))
+                response.response_items.each_with_index do |record, i|
+                  if record and record.is_a?(model_class)
+                    some_records[i].attributes = record.attributes
+                    some_records[i].errors = record.errors
+                    some_records[i].saved!
+                  end
+                end
+              end
+            end
+          end
+
+          @objects = {}
+          @allow_batch_operations = false
+          true
+        end
+
         def parse_response(response_xml, options = {})
           Response.parse(response_xml, options) do | response, elements, response_model_name |
             if model_name == response_model_name
@@ -131,17 +178,40 @@ module Xeroizer
             end
           end
         end
-                
+
       protected
-        
+
         # Parse the records part of the XML response and builds model instances as necessary.
         def parse_records(response, elements)
           elements.each do | element |
-            response.response_items << model_class.build_from_node(element, self)
+            new_record = model_class.build_from_node(element, self)
+            if element.attribute('status').try(:value) == 'ERROR'
+              new_record.errors = []
+              element.xpath('.//ValidationError').each do |err|
+                new_record.errors << err.text.gsub(/^\s+/, '').gsub(/\s+$/, '')
+              end
+            end
+            response.response_items << new_record
           end
         end
-        
-    end
-    
+
+        def to_bulk_xml(records, builder = Builder::XmlMarkup.new(:indent => 2))
+          tag = (self.class.optional_xml_root_name || model_name).pluralize
+          builder.tag!(tag) do
+            records.each {|r| r.to_xml(builder) }
+          end
+        end
+
+        # Parse the response from a create/update request.
+        def parse_save_response(response_xml)
+          response = parse_response(response_xml)
+          record = response.response_items.first if response.response_items.is_a?(Array)
+          if record && record.is_a?(self.class)
+            @attributes = record.attributes
+          end
+          self
+        end
+      end
+
   end
 end

data/lib/xeroizer/record/model_definition_helper.rb

@@ -73,6 +73,7 @@ module Xeroizer
           
           unless options[:skip_writer]
             define_method "#{internal_field_name}=".to_sym do | value | 
+              parent.mark_dirty(self) if parent
               @attributes[field_name] = value
             end
           end
@@ -89,6 +90,7 @@ module Xeroizer
 
         # Sets the value of the Xero primary key for this record if it exists.
         def id=(new_id)
+          parent.mark_dirty(self) if parent
           self[self.class.primary_key_name] = new_id
         end
         

data/lib/xeroizer/record/xml_helper.rb

@@ -47,7 +47,8 @@ module Xeroizer
               end
             end
           end
-          
+
+          parent.mark_clean(record)
           record
         end
         
@@ -87,7 +88,7 @@ module Xeroizer
             end
           end
         
-          # Format a attribute for use in the XML passed to Xero.
+          # Format an attribute for use in the XML passed to Xero.
           def xml_value_from_field(b, field, value)
             case field[:type]
               when :guid        then b.tag!(field[:api_name], value)

data/lib/xeroizer/response.rb

@@ -1,3 +1,17 @@
+# Copyright (c) 2008 Tim Connor <tlconnor@gmail.com>
+# 
+# Permission to use, copy, modify, and/or distribute this software for any
+# purpose with or without fee is hereby granted, provided that the above
+# copyright notice and this permission notice appear in all copies.
+# 
+# THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+# WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+# MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+# ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+# WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+# ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+# OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+
 module Xeroizer
   class Response
     

data/test/acceptance/acceptance_test.rb

@@ -48,6 +48,6 @@ module AcceptanceTest
     assert_not_nil ENV["CONSUMER_SECRET"], "No CONSUMER_SECRET environment variable specified."
     assert_not_nil ENV["KEY_FILE"], "No KEY_FILE environment variable specified."
     assert File.exists?(ENV["KEY_FILE"]), "The file <#{ENV["KEY_FILE"]}> does not exist."
-    OAuthCredentials.new ENV["CONSUMER_KEY"], ENV["CONSUMER_SECRET"], ENV["KEY_FILE"]
+    Xeroizer::OAuthCredentials.new ENV["CONSUMER_KEY"], ENV["CONSUMER_SECRET"], ENV["KEY_FILE"]
   end
 end

data/test/acceptance/bulk_operations_test.rb

@@ -0,0 +1,48 @@
+require "test_helper"
+require "acceptance_test"
+
+class BulkOperationsTest < Test::Unit::TestCase
+  include AcceptanceTest
+
+  def random_name
+    "test-person-#{rand 1000000000}"
+  end
+
+  def setup
+    super
+    @client = Xeroizer::PrivateApplication.new(@consumer_key, @consumer_secret, @key_file)
+  end
+
+  can "create multiple invoices at once" do
+    c1, c2 = nil, nil
+    assert_true(
+      @client.Contact.batch_save do
+        c1 = @client.Contact.build(name: random_name)
+        c2 = @client.Contact.build(name: random_name)
+      end
+    )
+    [c1, c2].each {|c| assert_false c.new_record? }
+  end
+
+  can "create and update new records in bulk" do
+    c1, c2 = nil, nil
+    assert_true(
+      @client.Contact.batch_save do
+        c1 = @client.Contact.create(name: random_name)
+        c1.email_address = "foo@bar.com"
+        c2 = @client.Contact.build(name: random_name)
+      end
+    )
+    [c1, c2].each {|c| assert_false c.new_record? }
+    c1.download_complete_record!
+    assert_equal c1.email_address, "foo@bar.com"
+  end
+
+  can "return false from #batch_save if validation fails" do
+    assert_false(
+      @client.Contact.batch_save do
+        @client.Contact.build(email_address: "guy-with-no-name@example.com")
+      end
+    )
+  end
+end

data/test/stub_responses/bank_transactions.xml

@@ -0,0 +1,52 @@
+<Response xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
+  <Id>8ce97e9e-31d2-457a-a75d-5e359ffac103</Id>
+  <Status>OK</Status>
+  <ProviderName>Xero API Previewer</ProviderName>
+  <DateTimeUTC>2013-05-20T11:54:50.9949578Z</DateTimeUTC>
+  <BankTransactions>
+    <BankTransaction>
+      <Contact>
+        <ContactID>a852a44c-3d8f-4c4b-a628-3a2c2121b9b1</ContactID>
+        <Name>Bank West</Name>
+      </Contact>
+      <Date>2013-05-16T00:00:00</Date>
+      <Status>AUTHORISED</Status>
+      <LineAmountTypes>Inclusive</LineAmountTypes>
+      <SubTotal>1000.00</SubTotal>
+      <TotalTax>0.00</TotalTax>
+      <Total>1000.00</Total>
+      <UpdatedDateUTC>2013-05-16T15:45:07.563</UpdatedDateUTC>
+      <CurrencyCode>GBP</CurrencyCode>
+      <BankTransactionID>ee4c4d5a-437d-4369-b152-848bd932f431</BankTransactionID>
+      <BankAccount>
+        <AccountID>bd9e85e0-0478-433d-ae9f-0b3c4f04bfe4</AccountID>
+        <Code>090</Code>
+        <Name>Business Bank Account</Name>
+      </BankAccount>
+      <Type>SPEND</Type>
+      <IsReconciled>false</IsReconciled>
+    </BankTransaction>
+    <BankTransaction>
+      <Contact>
+        <ContactID>d69ed9cd-f98a-4604-aced-54a1611554b2</ContactID>
+        <Name>Foo</Name>
+      </Contact>
+      <Date>2013-05-20T00:00:00</Date>
+      <Status>AUTHORISED</Status>
+      <LineAmountTypes>Inclusive</LineAmountTypes>
+      <SubTotal>123.00</SubTotal>
+      <TotalTax>0.00</TotalTax>
+      <Total>123.00</Total>
+      <UpdatedDateUTC>2013-05-20T11:39:41.34</UpdatedDateUTC>
+      <CurrencyCode>USD</CurrencyCode>
+      <BankTransactionID>8644472b-ea4e-4e5a-9987-f79250ece35e</BankTransactionID>
+      <BankAccount>
+        <AccountID>66c28ab3-05c2-4db7-9066-4357b8099fd8</AccountID>
+        <Name>FooBar</Name>
+      </BankAccount>
+      <Type>SPEND</Type>
+      <IsReconciled>false</IsReconciled>
+      <CurrencyRate>1.519520</CurrencyRate>
+    </BankTransaction>
+  </BankTransactions>
+</Response>

data/test/stub_responses/credit_notes.xml

@@ -17,7 +17,8 @@
       <TotalTax>19.90</TotalTax>
       <Total>218.90</Total>
       <UpdatedDateUTC>2008-09-16T09:40:09.467</UpdatedDateUTC>
-      <CurrencyCode>AUD</CurrencyCode>
+      <CurrencyCode>USD</CurrencyCode>
+      <CurrencyRate>1.5619</CurrencyRate>
       <FullyPaidOnDate>2011-03-07T00:00:00</FullyPaidOnDate>
       <Type>ACCPAYCREDIT</Type>
       <CreditNoteID>7df8949c-b71f-40c0-bbcf-39f2f450f286</CreditNoteID>
@@ -166,4 +167,4 @@
       <SentToContact>false</SentToContact>
     </CreditNote>
   </CreditNotes>
-</Response>
+</Response>

data/test/stub_responses/invoices.xml

@@ -17,7 +17,8 @@
       <TotalTax>13.50</TotalTax>
       <Total>148.50</Total>
       <UpdatedDateUTC>2008-09-16T10:28:51.5</UpdatedDateUTC>
-      <CurrencyCode>AUD</CurrencyCode>
+      <CurrencyCode>USD</CurrencyCode>
+      <CurrencyRate>1.516990</CurrencyRate>
       <FullyPaidOnDate>2011-02-24T00:00:00</FullyPaidOnDate>
       <Type>ACCPAY</Type>
       <InvoiceID>0032f627-3156-4d30-9b1c-4d3b994dc921</InvoiceID>
@@ -1896,4 +1897,4 @@
       <SentToContact>false</SentToContact>
     </Invoice>
   </Invoices>
-</Response>
+</Response>

data/test/stub_responses/records/bank_transaction-8644472b-ea4e-4e5a-9987-f79250ece35e.xml

@@ -0,0 +1,65 @@
+<Response xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
+  <Id>3465b325-f509-4323-9d1c-31b112aaae06</Id>
+  <Status>OK</Status>
+  <ProviderName>Xero API Previewer</ProviderName>
+  <DateTimeUTC>2013-05-20T11:57:54.000904Z</DateTimeUTC>
+  <BankTransactions>
+    <BankTransaction>
+      <Contact>
+        <ContactID>d69ed9cd-f98a-4604-aced-54a1611554b2</ContactID>
+        <ContactStatus>ACTIVE</ContactStatus>
+        <Name>Foo</Name>
+        <Addresses>
+          <Address>
+            <AddressType>POBOX</AddressType>
+          </Address>
+          <Address>
+            <AddressType>STREET</AddressType>
+          </Address>
+        </Addresses>
+        <Phones>
+          <Phone>
+            <PhoneType>DDI</PhoneType>
+          </Phone>
+          <Phone>
+            <PhoneType>DEFAULT</PhoneType>
+          </Phone>
+          <Phone>
+            <PhoneType>FAX</PhoneType>
+          </Phone>
+          <Phone>
+            <PhoneType>MOBILE</PhoneType>
+          </Phone>
+        </Phones>
+        <UpdatedDateUTC>2013-05-20T11:39:41.34</UpdatedDateUTC>
+        <DefaultCurrency>USD</DefaultCurrency>
+      </Contact>
+      <Date>2013-05-20T00:00:00</Date>
+      <Status>AUTHORISED</Status>
+      <LineAmountTypes>Inclusive</LineAmountTypes>
+      <LineItems>
+        <LineItem>
+          <UnitAmount>123.00</UnitAmount>
+          <TaxType>NONE</TaxType>
+          <TaxAmount>0.00</TaxAmount>
+          <LineAmount>123.00</LineAmount>
+          <AccountCode>320</AccountCode>
+          <Quantity>1.0000</Quantity>
+        </LineItem>
+      </LineItems>
+      <SubTotal>123.00</SubTotal>
+      <TotalTax>0.00</TotalTax>
+      <Total>123.00</Total>
+      <UpdatedDateUTC>2013-05-20T11:39:41.34</UpdatedDateUTC>
+      <CurrencyCode>USD</CurrencyCode>
+      <BankTransactionID>8644472b-ea4e-4e5a-9987-f79250ece35e</BankTransactionID>
+      <BankAccount>
+        <AccountID>66c28ab3-05c2-4db7-9066-4357b8099fd8</AccountID>
+        <Name>FooBar</Name>
+      </BankAccount>
+      <Type>SPEND</Type>
+      <IsReconciled>false</IsReconciled>
+      <CurrencyRate>1.519520</CurrencyRate>
+    </BankTransaction>
+  </BankTransactions>
+</Response>

data/test/stub_responses/records/bank_transaction-ee4c4d5a-437d-4369-b152-848bd932f431.xml

@@ -0,0 +1,72 @@
+<Response xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
+  <Id>3f444db0-9293-4b4a-bd2c-b2b59ee51442</Id>
+  <Status>OK</Status>
+  <ProviderName>Xero API Previewer</ProviderName>
+  <DateTimeUTC>2013-05-20T11:56:54.1897372Z</DateTimeUTC>
+  <BankTransactions>
+    <BankTransaction>
+      <Contact>
+        <ContactID>a852a44c-3d8f-4c4b-a628-3a2c2121b9b1</ContactID>
+        <ContactStatus>ACTIVE</ContactStatus>
+        <Name>Bank West</Name>
+        <Addresses>
+          <Address>
+            <AddressType>POBOX</AddressType>
+            <AddressLine1>Procurement Services</AddressLine1>
+            <AddressLine2>GPO 1234</AddressLine2>
+            <City>Pinehaven</City>
+            <PostalCode>PI98 7HV</PostalCode>
+          </Address>
+          <Address>
+            <AddressType>STREET</AddressType>
+          </Address>
+        </Addresses>
+        <Phones>
+          <Phone>
+            <PhoneType>MOBILE</PhoneType>
+          </Phone>
+          <Phone>
+            <PhoneType>DEFAULT</PhoneType>
+            <PhoneNumber>2023456</PhoneNumber>
+            <PhoneAreaCode>02</PhoneAreaCode>
+          </Phone>
+          <Phone>
+            <PhoneType>FAX</PhoneType>
+          </Phone>
+          <Phone>
+            <PhoneType>DDI</PhoneType>
+          </Phone>
+        </Phones>
+        <UpdatedDateUTC>2013-05-12T04:24:12.347</UpdatedDateUTC>
+      </Contact>
+      <Date>2013-05-16T00:00:00</Date>
+      <Status>AUTHORISED</Status>
+      <LineAmountTypes>Inclusive</LineAmountTypes>
+      <LineItems>
+        <LineItem>
+          <Description>Loan Repayment</Description>
+          <UnitAmount>1000.00</UnitAmount>
+          <TaxType>NONE</TaxType>
+          <TaxAmount>0.00</TaxAmount>
+          <LineAmount>1000.00</LineAmount>
+          <AccountCode>900</AccountCode>
+          <Quantity>1.0000</Quantity>
+        </LineItem>
+      </LineItems>
+      <SubTotal>1000.00</SubTotal>
+      <TotalTax>0.00</TotalTax>
+      <Total>1000.00</Total>
+      <UpdatedDateUTC>2013-05-16T15:45:07.563</UpdatedDateUTC>
+      <CurrencyCode>GBP</CurrencyCode>
+      <BankTransactionID>ee4c4d5a-437d-4369-b152-848bd932f431</BankTransactionID>
+      <BankAccount>
+        <AccountID>bd9e85e0-0478-433d-ae9f-0b3c4f04bfe4</AccountID>
+        <Code>090</Code>
+        <Name>Business Bank Account</Name>
+      </BankAccount>
+      <Type>SPEND</Type>
+      <IsReconciled>false</IsReconciled>
+      <CurrencyRate>1.000000</CurrencyRate>
+    </BankTransaction>
+  </BankTransactions>
+</Response>

data/test/stub_responses/refresh_responses.rb

@@ -7,7 +7,7 @@ base_path = File.expand_path(File.dirname(__FILE__))
 if ARGV[3]
   models = [ARGV[3]]
 else
-  models = %w(Account BrandingTheme Contact CreditNote Currency Employee Invoice Item ManualJournal Organisation Payment TaxRate TrackingCategory)
+  models = %w(Account BankTransaction BrandingTheme Contact CreditNote Currency Employee Invoice Item ManualJournal Organisation Payment TaxRate TrackingCategory)
 end
 
 models.each do | model_name |
@@ -17,7 +17,7 @@ models.each do | model_name |
   records = model.all
   File.open("#{base_path}/#{model_name.underscore.pluralize}.xml", "w") { | fp | fp.write model.response.response_xml }
   
-  if %w(Contact CreditNote Invoice ManualJournal).include?(model_name)
+  if %w(BankTransaction Contact CreditNote Invoice ManualJournal).include?(model_name)
     records.each do | summary_record |
       record = model.find(summary_record.id)
       File.open("#{base_path}/records/#{model_name.underscore}-#{record.id}.xml", "w") { | fp | fp.write model.response.response_xml }
@@ -26,4 +26,4 @@ models.each do | model_name |
     record = model.find(records.first.id)
     File.open("#{base_path}/#{model_name.underscore.singularize}.xml", "w") { | fp | fp.write model.response.response_xml }
   end
-end
+end

data/test/unit/models/bank_transaction_test.rb

@@ -1,11 +1,14 @@
 require "test_helper"
 
 class BankTransactionTest < Test::Unit::TestCase
+  include TestHelper
   include Xeroizer::Record
 
   def setup
     fake_parent = Class.new do
       attr_accessor :application
+      def mark_dirty(*args)
+      end
     end.new
 
     the_line_items = [
@@ -15,6 +18,9 @@ class BankTransactionTest < Test::Unit::TestCase
 
     @the_bank_transaction = BankTransaction.new fake_parent
     @the_bank_transaction.line_items = the_line_items
+
+    @client = Xeroizer::PublicApplication.new(CONSUMER_KEY, CONSUMER_SECRET)
+    mock_api('BankTransactions')
   end
 
   context "given a bank_transaction with line_amount_types set to \"Exclusive\"" do
@@ -44,4 +50,26 @@ class BankTransactionTest < Test::Unit::TestCase
       assert_equal "1.0", @the_bank_transaction.sub_total.to_s
     end
   end
+
+  context "bank transaction totals" do
+
+    should "large-scale testing from API XML" do
+      bank_transactions = @client.BankTransaction.all
+      bank_transactions.each do | bank_transaction |
+        assert(!!bank_transaction.attributes[:currency_code], "Doesn't have currency code in attributes")
+        assert(!!bank_transaction.currency_code, "Doesn't have currency code in model")
+        assert(!!bank_transaction.currency_rate, "Doesn't have currency rate in model")
+
+
+        assert_equal(bank_transaction.attributes[:currency_code], bank_transaction.currency_code)
+
+        if bank_transaction.attributes[:currency_rate]
+          assert_equal(bank_transaction.attributes[:currency_rate], bank_transaction.currency_rate)
+        else
+          assert_equal(1.0, bank_transaction.currency_rate)
+        end
+      end
+    end
+
+  end
 end

data/test/unit/models/credit_note_test.rb

@@ -26,6 +26,15 @@ class CreditNoteTest < Test::Unit::TestCase
     should "large-scale testing from API XML" do
       credit_notes = @client.CreditNote.all
       credit_notes.each do | credit_note |
+        assert_equal(credit_note.attributes[:currency_code], credit_note.currency_code)
+
+        if credit_note.attributes[:currency_rate]
+          assert_equal(credit_note.attributes[:currency_rate], credit_note.currency_rate)
+        else
+          assert_equal(1.0, credit_note.currency_rate)
+        end
+
+
         assert_equal(credit_note.attributes[:sub_total], credit_note.sub_total)
         assert_equal(credit_note.attributes[:total_tax], credit_note.total_tax)
         assert_equal(credit_note.attributes[:total], credit_note.total)

data/test/unit/models/invoice_test.rb

@@ -44,6 +44,13 @@ class InvoiceTest < Test::Unit::TestCase
     should "large-scale testing from API XML" do
       invoices = @client.Invoice.all
       invoices.each do | invoice |
+        assert_equal(invoice.attributes[:currency_code], invoice.currency_code)
+        if invoice.attributes[:currency_rate]
+          assert_equal(invoice.attributes[:currency_rate], invoice.currency_rate)
+        else
+          assert_equal(1.0, invoice.currency_rate)
+        end
+
         assert_equal(invoice.attributes[:sub_total], invoice.sub_total)
         assert_equal(invoice.attributes[:total_tax], invoice.total_tax)
         assert_equal(invoice.attributes[:total], invoice.total)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: xeroizer-float
 version: !ruby/object:Gem::Version
-  version: 2.15.3.13
+  version: 2.15.3.14
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-21 00:00:00.000000000 Z
+date: 2013-05-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: builder
@@ -60,6 +60,22 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: tzinfo
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
@@ -278,8 +294,10 @@ files:
 - test/acceptance/about_fetching_bank_transactions_test.rb
 - test/acceptance/acceptance_test.rb
 - test/acceptance/bank_transaction_reference_data.rb
+- test/acceptance/bulk_operations_test.rb
 - test/stub_responses/accounts.xml
 - test/stub_responses/api_exception.xml
+- test/stub_responses/bank_transactions.xml
 - test/stub_responses/bogus_oauth_error
 - test/stub_responses/branding_themes.xml
 - test/stub_responses/contact.xml
@@ -304,6 +322,8 @@ files:
 - test/stub_responses/organisations.xml
 - test/stub_responses/payments.xml
 - test/stub_responses/rate_limit_exceeded
+- test/stub_responses/records/bank_transaction-8644472b-ea4e-4e5a-9987-f79250ece35e.xml
+- test/stub_responses/records/bank_transaction-ee4c4d5a-437d-4369-b152-848bd932f431.xml
 - test/stub_responses/records/contact-043892a1-aef1-4c18-88d8-b8ccb6d31466.xml
 - test/stub_responses/records/contact-09664078-efe2-4a88-89a5-67eac9b0047b.xml
 - test/stub_responses/records/contact-0a4cf37b-a1a8-4753-9ee2-f9207f63a8ff.xml