Global minorversion

lib/quickbooks-ruby.rb

@@ -191,6 +191,7 @@
 module Quickbooks
   @@sandbox_mode = false
   @@logger = nil
+  @@minorversion = 47
 
   class << self
     def sandbox_mode
@@ -201,6 +202,14 @@ def sandbox_mode=(sandbox_mode)
       @@sandbox_mode = sandbox_mode
     end
 
+    def minorversion=(v)
+      @@minorversion = v
+    end
+
+    def minorversion
+      @@minorversion
+    end
+
     def logger
       @@logger ||= ::Logger.new($stdout) # TODO: replace with a real log file
     end

lib/quickbooks/model/account.rb

@@ -4,7 +4,6 @@ class Account < BaseModel
       XML_COLLECTION_NODE = "Account"
       XML_NODE = "Account"
       REST_RESOURCE = 'account'
-      MINORVERSION = 13
 
       ASSET = 'Asset'
       EQUITY = 'Equity'

lib/quickbooks/model/customer.rb

@@ -16,8 +16,6 @@ class Customer < BaseModel
       include NameEntity::Quality
       include NameEntity::PermitAlterations
 
-      MINORVERSION = 33
-
       xml_name XML_NODE
       xml_accessor :id, :from => 'Id'
       xml_accessor :sync_token, :from => 'SyncToken', :as => Integer

lib/quickbooks/model/invoice.rb

@@ -17,7 +17,6 @@ class Invoice < BaseModel
       XML_COLLECTION_NODE = "Invoice"
       XML_NODE = "Invoice"
       EMAIL_STATUS_NEED_TO_SEND = 'NeedToSend'
-      MINORVERSION = 37
 
       xml_accessor :id, :from => 'Id'
       xml_accessor :sync_token, :from => 'SyncToken', :as => Integer

lib/quickbooks/model/item.rb

@@ -10,7 +10,6 @@ class Item < BaseModel
       XML_COLLECTION_NODE = "Item"
       XML_NODE = "Item"
       REST_RESOURCE = 'item'
-      MINORVERSION = 47
 
       INVENTORY_TYPE = 'Inventory'
       NON_INVENTORY_TYPE = 'NonInventory'

lib/quickbooks/model/preferences.rb

@@ -4,7 +4,6 @@ class Preferences < BaseModel
       XML_COLLECTION_NODE = "Preferences"
       XML_NODE            = "Preferences"
       REST_RESOURCE       = 'preferences'
-      MINORVERSION = 32
 
       xml_name XML_NODE
 

lib/quickbooks/model/purchase_order.rb

@@ -8,7 +8,6 @@ class PurchaseOrder < BaseModel
       REST_RESOURCE = 'purchaseorder'
       XML_COLLECTION_NODE = "PurchaseOrder"
       XML_NODE = "PurchaseOrder"
-      MINORVERSION = 45
 
       xml_accessor :id, :from => 'Id'
       xml_accessor :sync_token, :from => 'SyncToken', :as => Integer

lib/quickbooks/service/access_token.rb

@@ -23,7 +23,8 @@ def renew
       def disconnect
         conn = Faraday.new
         conn.basic_auth oauth.client.id, oauth.client.secret
-        response = conn.post(DISCONNECT_URL, token: oauth.refresh_token || oauth.token)
+        url = "#{DISCONNECT_URL}?minorversion=#{Quickbooks.minorversion}"
+        response = conn.post(url, token: oauth.refresh_token || oauth.token)
 
         if response.success?
           Quickbooks::Model::AccessTokenResponse.new(error_code: "0")

lib/quickbooks/service/account.rb

@@ -7,11 +7,6 @@ def delete(account)
         update(account, :sparse => true)
       end
 
-      def url_for_query(query = nil, start_position = 1, max_results = 20, options = {})
-        url = super(query, start_position, max_results, options)
-        "#{url}&minorversion=#{Quickbooks::Model::Account::MINORVERSION}"
-      end
-
       private
 
       def model

lib/quickbooks/service/base_service.rb

@@ -229,6 +229,8 @@ def do_http_file_upload(uploadIO, url, metadata = nil)
           body['file_metadata_0'] = param_part
         end
 
+        url = add_query_string_to_url(url, {})
+
         do_http(:upload, url, body, headers)
       end
 
@@ -292,7 +294,9 @@ def oauth_post_with_multipart(url, body, headers)
         @oauth.post_with_multipart(url, headers: headers, body: body, raise_errors: false)
       end
 
-      def add_query_string_to_url(url, params)
+      def add_query_string_to_url(url, params = {})
+        params ||= {}
+        params['minorversion'] = Quickbooks.minorversion
         if params.is_a?(Hash) && !params.empty?
           keyvalues = params.collect { |k| "#{k.first}=#{k.last}" }.join("&")
           delim = url.index("?") != nil ? "&" : "?"

lib/quickbooks/service/customer.rb

@@ -7,21 +7,11 @@ def delete(customer)
         update(customer, :sparse => true)
       end
 
-      def url_for_resource(resource)
-        url = super(resource)
-        "#{url}?minorversion=#{Quickbooks::Model::Customer::MINORVERSION}"
-      end
-
       def fetch_by_id(id, params = {})
-        url = "#{url_for_base}/customer/#{id}?minorversion=#{Quickbooks::Model::Customer::MINORVERSION}"
+        url = "#{url_for_base}/customer/#{id}"
         fetch_object(model, url, params)
       end
 
-      def url_for_query(query = nil, start_position = 1, max_results = 20, options = {})
-        url = super(query, start_position, max_results, options)
-        "#{url}&minorversion=#{Quickbooks::Model::Customer::MINORVERSION}"
-      end
-
       private
 
       def model

lib/quickbooks/service/invoice.rb

@@ -6,20 +6,11 @@ def delete(invoice)
         delete_by_query_string(invoice)
       end
 
-      def url_for_resource(resource)
-        url = super(resource)
-      end
-
       def fetch_by_id(id, params = {})
-        url = "#{url_for_base}/invoice/#{id}?minorversion=#{Quickbooks::Model::Invoice::MINORVERSION}"
+        url = "#{url_for_base}/invoice/#{id}"
         fetch_object(model, url, params)
       end
 
-      def url_for_query(query = nil, start_position = 1, max_results = 20, options = {})
-        url = super(query, start_position, max_results, options)
-        "#{url}&minorversion=#{Quickbooks::Model::Invoice::MINORVERSION}"
-      end
-
       def send(invoice, email_address=nil)
         query = email_address.present? ? "?sendTo=#{email_address}" : ""
         url = "#{url_for_resource(model::REST_RESOURCE)}/#{invoice.id}/send#{query}"

lib/quickbooks/service/item.rb

@@ -7,21 +7,11 @@ def delete(item)
         update(item, :sparse => true)
       end
 
-      def url_for_resource(resource)
-        url = super(resource)
-        "#{url}?minorversion=#{Quickbooks::Model::Item::MINORVERSION}"
-      end
-
       def fetch_by_id(id, params = {})
-        url = "#{url_for_base}/item/#{id}?minorversion=#{Quickbooks::Model::Item::MINORVERSION}"
+        url = "#{url_for_base}/item/#{id}"
         fetch_object(model, url, params)
       end
 
-      def url_for_query(query = nil, start_position = 1, max_results = 20, options = {})
-        url = super(query, start_position, max_results, options)
-        "#{url}&minorversion=#{Quickbooks::Model::Item::MINORVERSION}"
-      end
-
       private
 
       def model

lib/quickbooks/service/preferences.rb

@@ -2,11 +2,6 @@ module Quickbooks
   module Service
     class Preferences < BaseService
 
-      def url_for_query(query = nil, start_position = 1, max_results = 20, options = {})
-        url = super(query, start_position, max_results, options)
-        "#{url}&minorversion=#{Quickbooks::Model::Preferences::MINORVERSION}"
-      end
-
       private
 
       def model

lib/quickbooks/service/service_crud_json.rb

@@ -8,7 +8,7 @@ def fetch_by_id(id, params = {})
 
       def create(entity, options = {})
         raise Quickbooks::InvalidModelException.new(entity.errors.full_messages.join(',')) unless entity.valid?
-        response = do_http(:post, url_for_resource(model.resource_for_singular), entity.to_json, options)
+        response = do_http_post(url_for_resource(model.resource_for_singular), entity.to_json, options)
         if response.code.to_i == 200
           JSON.parse(response.plain_body)
         else

spec/lib/quickbooks/service/account_spec.rb

@@ -5,6 +5,7 @@
 
   it "can query for accounts" do
     xml = fixture("accounts.xml")
+
     stub_http_request(:get, @service.url_for_query, ["200", "OK"], xml, {}, true)
 
     accounts = @service.query

spec/lib/quickbooks/service/base_service_spec.rb

@@ -223,8 +223,9 @@
       end
 
       it "calls before_request" do
+        v = Quickbooks.minorversion
         output_string = "BEFORE REQUEST:\nurl: https://quickbooks.api.intuit.com/v3/company/9991111222/query?query=SE"\
-                        "LECT+%2A+FROM+Vendor+STARTPOSITION+1+MAXRESULTS+20\nheaders: {\"Content-Type\"=>\"applicatio"\
+                        "LECT+%2A+FROM+Vendor+STARTPOSITION+1+MAXRESULTS+20&minorversion=#{v}\nheaders: {\"Content-Type\"=>\"applicatio"\
                         "n/xml\", \"Accept\"=>\"application/xml\", \"Accept-Encoding\"=>\"gzip, deflate\"}\nbody: {}"\
                         "\nmethod: get\n"
 
@@ -247,8 +248,9 @@
       end
 
       it "calls after_request" do
+        v = Quickbooks.minorversion
         output_string = "AFTER REQUEST:\nurl: https://quickbooks.api.intuit.com/v3/company/9991111222/query?query=SEL"\
-                        "ECT+%2A+FROM+Vendor+STARTPOSITION+1+MAXRESULTS+20\nheaders: {\"Content-Type\"=>\"application"\
+                        "ECT+%2A+FROM+Vendor+STARTPOSITION+1+MAXRESULTS+20&minorversion=#{v}\nheaders: {\"Content-Type\"=>\"application"\
                         "/xml\", \"Accept\"=>\"application/xml\", \"Accept-Encoding\"=>\"gzip, deflate\", \"Authoriza"\
                         "tion\"=>\"Bearer token\"}\nbody: {}\nmethod: get\nresponse: <IntuitResponse xmlns=\"http://s"\
                         "chema.intuit.com/finance/v3\" time=\"2013-04-23T08:55:53.298-07:00\">\n<QueryResponse startP"\
@@ -296,8 +298,9 @@
       end
 
       it "calls around_request" do
+        v = Quickbooks.minorversion
         output_string = "AROUND REQUEST (BEFORE CALL):\nurl: https://quickbooks.api.intuit.com/v3/company/9991111222/"\
-                        "query?query=SELECT+%2A+FROM+Vendor+STARTPOSITION+1+MAXRESULTS+20\nheaders: {\"Content-Type\""\
+                        "query?query=SELECT+%2A+FROM+Vendor+STARTPOSITION+1+MAXRESULTS+20&minorversion=#{v}\nheaders: {\"Content-Type\""\
                         "=>\"application/xml\", \"Accept\"=>\"application/xml\", \"Accept-Encoding\"=>\"gzip, deflate"\
                         "\"}\nbody: {}\nmethod: get\nAROUND REQUEST (AFTER CALL):\nresponse: <IntuitResponse xmlns=\""\
                         "http://schema.intuit.com/finance/v3\" time=\"2013-04-23T08:55:53.298-07:00\">\n<QueryRespons"\

spec/lib/quickbooks/service/customer_spec.rb

@@ -18,7 +18,9 @@
   it "can fetch a customer by ID" do
     xml = fixture("fetch_customer_by_id.xml")
     model = Quickbooks::Model::Customer
-    stub_http_request(:get, "#{@service.url_for_base}/customer/1?minorversion=#{Quickbooks::Model::Customer::MINORVERSION}", ["200", "OK"], xml)
+
+    url = "#{@service.url_for_base}/customer/1"
+    stub_http_request(:get, url, ["200", "OK"], xml)
 
     customer = @service.fetch_by_id(1)
     expect(customer.fully_qualified_name).to eq("Thrifty Meats")

spec/lib/quickbooks/service/invoice_spec.rb

@@ -7,7 +7,8 @@
     xml = fixture("invoices.xml")
     model = Quickbooks::Model::Invoice
 
-    stub_http_request(:get, @service.url_for_query, ["200", "OK"], xml)
+    url = @service.url_for_query
+    stub_http_request(:get, url, ["200", "OK"], xml)
     invoices = @service.query
     expect(invoices.entries.count).to eq 1
 
@@ -18,7 +19,10 @@
   it "can fetch an Invoice by ID" do
     xml = fixture("fetch_invoice_by_id.xml")
     model = Quickbooks::Model::Invoice
-    stub_http_request(:get, %r{#{@service.url_for_resource(model::REST_RESOURCE)}/1}, ["200", "OK"], xml)
+
+    url = "#{@service.url_for_base}/invoice/1"
+    stub_http_request(:get, url, ["200", "OK"], xml)
+
     invoice = @service.fetch_by_id(1)
     expect(invoice.doc_number).to eq "1001"
   end
@@ -247,7 +251,9 @@
 
   it "can read line items from a bundle" do
     xml = fixture("invoice_with_bundle_line_item.xml")
-    stub_http_request(:get, %r{#{@service.url_for_resource(Quickbooks::Model::Invoice::REST_RESOURCE)}/186}, ["200", "OK"], xml)
+    url = "#{@service.url_for_base}/invoice/186"
+
+    stub_http_request(:get, url, ["200", "OK"], xml)
     invoice = @service.fetch_by_id(186)
     expect(invoice.valid?).to be true
 
@@ -290,7 +296,8 @@
 
   it "can sparse update an Invoice containing a bundle" do
     xml = fixture("invoice_with_bundle_line_item.xml")
-    stub_http_request(:get, %r{#{@service.url_for_resource(Quickbooks::Model::Invoice::REST_RESOURCE)}/186}, ["200", "OK"], xml)
+    url = "#{@service.url_for_base}/invoice/186"
+    stub_http_request(:get, url, ["200", "OK"], xml)
     invoice = @service.fetch_by_id(186)
 
     invoice.line_items.each do |l|

spec/lib/quickbooks/service/item_spec.rb

@@ -18,7 +18,7 @@
   it "can fetch an Item by ID" do
     xml = fixture("fetch_item_by_id.xml")
     model = Quickbooks::Model::Item
-    stub_http_request(:get, "#{@service.url_for_base}/item/2?minorversion=#{Quickbooks::Model::Item::MINORVERSION}", ["200", "OK"], xml)
+    stub_http_request(:get, "#{@service.url_for_base}/item/2", ["200", "OK"], xml)
     item = @service.fetch_by_id(2)
     expect(item.name).to eq("Plush Baby Doll")
   end
@@ -52,7 +52,7 @@
     xml = fixture("fetch_item_by_id.xml")
     model = Quickbooks::Model::Item
 
-    url = "#{@service.url_for_resource(model::REST_RESOURCE)}&requestid=123"
+    url = "#{@service.url_for_resource(model::REST_RESOURCE)}?requestid=123"
     stub_http_request(:post, url, ["200", "OK"], xml)
 
     item = Quickbooks::Model::Item.new

spec/lib/quickbooks/service/purchase_order_spec.rb

@@ -4,11 +4,14 @@
     construct_service :purchase_order
   end
 
-  it "can delete a vendor" do
+  it "can delete a purchase order" do
     model = Quickbooks::Model::PurchaseOrder
     purchase_order = model.new
     xml = fixture("deleted_purchase_order.xml")
-    stub_http_request(:post, %r{#{@service.url_for_resource(model::REST_RESOURCE)}}, ["200", "OK"], xml, {}, false)
+
+    url = "#{@service.url_for_resource(model::REST_RESOURCE)}?operation=delete"
+
+    stub_http_request(:post, url, ["200", "OK"], xml, {}, false)
     expect(@service.delete(purchase_order)).to eq true
 
   end

spec/support/net_helpers.rb

@@ -1,5 +1,13 @@
 module NetHelpers
   def stub_http_request(method, url, status = ["200", "OK"], body = nil, headers = {}, strict = true)
+    unless url.index('minorversion')
+      if url.index('?')
+        url = "#{url}&minorversion=#{Quickbooks.minorversion}"
+      else
+        url = "#{url}?minorversion=#{Quickbooks.minorversion}"
+      end
+    end
+    #puts "URL: #{url}"
     stub_request(method, url).to_return(body: body, status: status)
   end
 end