netsuite 0.7.2 → 0.7.3

data/lib/netsuite.rb

@@ -196,6 +196,7 @@ module NetSuite
     autoload :RevRecTemplate,                   'netsuite/records/rev_rec_template'
     autoload :RevRecSchedule,                   'netsuite/records/rev_rec_schedule'
     autoload :RoleList,                         'netsuite/records/role_list'
+    autoload :ReturnAuthorization,              'netsuite/records/return_authorization'
     autoload :SalesOrder,                       'netsuite/records/sales_order'
     autoload :SalesOrderItem,                   'netsuite/records/sales_order_item'
     autoload :SalesOrderItemList,               'netsuite/records/sales_order_item_list'

data/lib/netsuite/records/return_authorization.rb

@@ -0,0 +1,133 @@
+module NetSuite
+  module Records
+    class ReturnAuthorization
+      include Support::Fields
+      include Support::RecordRefs
+      include Support::Records
+      include Support::Actions
+      include Namespaces::TranSales
+
+      # https://system.netsuite.com/help/helpcenter/en_US/srbrowser/Browser2015_2/schema/record/returnauthorization.html
+
+      actions :get, :get_list, :initialize, :add, :update, :delete, :upsert, :search
+
+      fields :alt_sales_total,
+        :cc_approved,
+        :cc_expire_date,
+        :cc_name,
+        :cc_number,
+        :cc_street,
+        :cc_zip_code,
+        :contrib_pct,
+        :created_date,
+        :currency_name,
+        :debit_card_issue_no,
+        :deferred_revenue,
+        :discount_rate,
+        :discount_total,
+        :email,
+        :est_gross_profit,
+        :est_gross_profit_percent,
+        :exchange_rate,
+        :exclude_commission,
+        :fax,
+        :gift_cert_applied,
+        :gift_cert_available,
+        :gift_cert_total,
+        :is_taxable,
+        :job,
+        :last_modified_date,
+        :memo,
+        :message,
+        :other_ref_num,
+        :pay_pal_auth_id,
+        :pay_pal_process,
+        :pn_ref_num,
+        :recognized_revenue,
+        :rev_rec_end_date,
+        :rev_rec_on_rev_commitment,
+        :rev_rec_start_date,
+        :sales_effective_date,
+        :ship_address,
+        :source,
+        :status,
+        :sub_total,
+        :sync_partner_teams,
+        :sync_sales_teams,
+        :tax2_total,
+        :tax_rate,
+        :tax_total,
+        :to_be_emailed,
+        :to_be_faxed,
+        :to_be_printed,
+        :total,
+        :total_cost_estimate,
+        :tran_date,
+        :tran_id,
+        :tran_is_vsoe_bundle,
+        :valid_from,
+        :vat_reg_num,
+        :vsoe_auto_calc
+
+      record_refs :klass,
+        :created_from,
+        :credit_card,
+        :credit_card_processor,
+        :currency,
+        :custom_form,
+        :department,
+        :discount_item,
+        :dr_account,
+        :entity,
+        :fx_account,
+        :gift_cert,
+        :interco_status,
+        :interco_transaction,
+        :lead_source,
+        :location,
+        :message_sel,
+        :order_status,
+        :partner,
+        :payment_method,
+        :promo_code,
+        :rev_commit_status,
+        :revenue_status,
+        :rev_rec_schedule,
+        :sales_group,
+        :sales_rep,
+        :subsidiary,
+        :tax_item
+
+      # :bill_address_list
+      # :item_list,
+      # :partners_list,
+      # :sales_team_list,
+      # :ship_address_list,
+      
+      field :billing_address,          Address
+      field :custom_field_list,        CustomFieldList
+
+      attr_reader   :internal_id
+      attr_accessor :external_id
+
+      def initialize(attributes = {})
+        @internal_id = attributes.delete(:internal_id) || attributes.delete(:@internal_id)
+        @external_id = attributes.delete(:external_id) || attributes.delete(:@external_id)
+        initialize_from_attributes_hash(attributes)
+      end
+
+      def to_record
+        rec = super
+        if rec["#{record_namespace}:customFieldList"]
+          rec["#{record_namespace}:customFieldList!"] = rec.delete("#{record_namespace}:customFieldList")
+        end
+        rec
+      end
+
+      def self.search_class_name
+        "Transaction"
+      end
+
+    end
+  end
+end

data/lib/netsuite/records/sales_order.rb

@@ -18,7 +18,7 @@ module NetSuite
         :start_date, :status, :sync_partner_teams, :sync_sales_teams, :tax2_total, :tax_rate, :to_be_emailed, :to_be_faxed,
         :to_be_printed, :total_cost_estimate, :tran_date, :tran_id, :tran_is_vsoe_bundle, :vat_reg_num,
         :linked_tracking_numbers, :vsoe_auto_calc, :quantity, :bill_city, :bill_state, :ship_city, :ship_state, :cost_estimate,
-        :amount, :is_ship_address
+        :amount, :is_ship_address, :auth_code
 
       # NOTE API >= 2014_2 only
       field :shipping_address, Address

data/lib/netsuite/rest/utilities/request.rb

@@ -7,7 +7,8 @@ module NetSuite
     module Utilities
       class Request
 
-        BASE_API = "https://rest.netsuite.com/rest"
+        PRODUCTION_API = "https://rest.netsuite.com/rest"
+        SANDBOX_API = "https://rest.sandbox.netsuite.com/rest"
         DEFAULT_TIMEOUT = 30
         USE_SSL = true
 
@@ -16,7 +17,8 @@ module NetSuite
           def get(options)
             email       = encode(options.fetch :email)
             signature   = encode(options.fetch :password)
-            uri         = options.fetch(:uri)
+            sandbox     = options.fetch(:sandbox, false)
+            uri         = determine_uri(options.fetch(:uri), sandbox)
             response    = make_request(:get, uri, email, signature)
             [response.code, JSON.parse(response.body)]
           end
@@ -24,7 +26,6 @@ module NetSuite
           private
 
           def make_request(method_name, uri, email, signature)
-            uri = URI(BASE_API + uri)
             klass = Module.const_get "Net::HTTP::#{method_name.to_s.capitalize}"
             method = klass.new(uri)
             method['AUTHORIZATION'] = auth_header(email, signature)
@@ -39,6 +40,10 @@ module NetSuite
             "NLAuth nlauth_email=#{email},nlauth_signature=#{signature}"
           end
 
+          def determine_uri(uri, sandbox)
+            URI((sandbox ? SANDBOX_API : PRODUCTION_API) + uri)
+          end
+
           def encode(unencoded_string)
             URI.escape(
               unencoded_string,

data/lib/netsuite/rest/utilities/roles.rb

@@ -5,37 +5,39 @@ module NetSuite
 
         class << self
 
-          def get(email, password)
-            code, body = Request.get(
-              email: email,
-              password: password,
-              uri: '/roles'
+          def get(opts={})
+            status, body = Request.get(
+              email:    opts.fetch(:email, Configuration.email),
+              password: opts.fetch(:password, Configuration.password),
+              sandbox:  opts.fetch(:sandbox, Configuration.sandbox),
+              uri:      '/roles',
             )
-            code == "200" ? format_response(body) : body
+            status == "200" ? format_response(body) : body
           end
 
           private
 
           def format_response(parsed)
-            # [
-            #   {
-            #    "account"=>{
-            #      "internalId"=>"TSTDRV15",
-            #      "name"=>"Honeycomb Mfg SDN (Leading)"},
-            #    "role"=>{
-            #      "internalId"=>3,
-            #      "name"=>"Administrator"},
-            #    "dataCenterURLs"=>{
-            #      "webservicesDomain"=>"https://webservices.na1.netsuite.com",
-            #      "restDomain"=>"https://rest.na1.netsuite.com",
-            #      "systemDomain"=>"https://system.na1.netsuite.com"}
-            #    },
-            #   ....
-            {
-              accounts: parsed.map {|hash| hash["account"]["internalId"] }.uniq,
-              roles:    parsed.map {|hash| hash["role"]["internalId"] }.uniq,
-              wsdls:    parsed.map {|hash| hash["dataCenterURLs"]["webservicesDomain"] }.uniq,
-            }
+            uniq_accounts = parsed.map {|hash| hash["account"] }.uniq
+
+            uniq_accounts.map do |hash|
+              matches = parsed.select {|x| x["account"]["internalId"] == hash["internalId"]}
+              {
+                account_id:    hash["internalId"],
+                account_name:  hash["name"],
+                roles: matches.map do |match|
+                  {
+                    id:   match["role"]["internalId"],
+                    name: match["role"]["name"]
+                  }
+                end.uniq,
+                wsdls: {
+                  webservices: matches.map {|match| match["dataCenterURLs"]["webservicesDomain"] }.uniq,
+                  rest:        matches.map {|match| match["dataCenterURLs"]["restDomain"] }.uniq,
+                  system:      matches.map {|match| match["dataCenterURLs"]["systemDomain"] }.uniq,
+                }
+              }
+            end
           end
 
         end

data/lib/netsuite/utilities.rb

@@ -32,7 +32,8 @@ module NetSuite
         count += 1
         yield
       rescue options[:exception] || Savon::SOAPFault => e
-        if !e.message.include?("Only one request may be made against a session at a time")
+        if !e.message.include?("Only one request may be made against a session at a time") &&
+           !e.message.include?('java.util.ConcurrentModificationException')
           raise e
         end
         if count >= (options[:attempts] || 8)

data/lib/netsuite/version.rb

@@ -1,3 +1,3 @@
 module Netsuite
-  VERSION = '0.7.2'
+  VERSION = '0.7.3'
 end

data/spec/netsuite/records/basic_record_spec.rb

@@ -42,6 +42,7 @@ describe 'basic records' do
       NetSuite::Records::Note,
       NetSuite::Records::OtherChargeSaleItem,
       NetSuite::Records::GiftCertificateItem,
+      NetSuite::Records::ReturnAuthorization,
     ]
   }
 

data/spec/netsuite/utilities/roles_spec.rb

@@ -1,14 +1,14 @@
 require 'spec_helper'
 
 describe NetSuite::Rest::Utilities::Roles do
-  subject { described_class }
 
   describe '#get' do
+    subject { described_class }
     it { is_expected.to respond_to :get }
 
     it 'invokes the request module' do
       expect( NetSuite::Rest::Utilities::Request ).to receive(:get)
-      subject.get('jimbob@netzero.com', 'pickles')
+      subject.get(email: 'jimbob@netzero.com', password: 'pickles')
     end
 
     it 'formats the API response if successful' do
@@ -16,7 +16,7 @@ describe NetSuite::Rest::Utilities::Roles do
         ["200", {parsed: :response}]
       )
       expect( subject ).to receive(:format_response).with({parsed: :response}).and_return nil
-      subject.get('jimbob@netzero.com', 'pickles')
+      subject.get(email: 'jimbob@netzero.com', password: 'pickles')
     end
 
     it 'returns the full error message if unsuccessful' do
@@ -24,11 +24,13 @@ describe NetSuite::Rest::Utilities::Roles do
         ["500", {oh: :no!}]
       )
       expect( subject ).to_not receive(:format_reponse)
-      expect( subject.get('jimbob@netzero.com', 'pickles') ).to eq({oh: :no!})
+      expect( subject.get(email: 'jimbob@netzero.com', password: 'pickles') ).to eq({oh: :no!})
     end
   end
 
   describe '#format_response' do
+    subject { described_class.send(:format_response, parsed) }
+
     let(:parsed) do
       [
         {
@@ -36,61 +38,106 @@ describe NetSuite::Rest::Utilities::Roles do
             "internalId"=>"TSTDRV15",
             "name"=>"Honeycomb Mfg SDN (Leading)"},
           "role"=>{
-              "internalId"=>3,
-              "name"=>"Administrator"},
-              "dataCenterURLs"=>{
-                "webservicesDomain"=>"https://webservices.na1.netsuite.com",
-                "restDomain"=>"https://rest.na1.netsuite.com",
-                "systemDomain"=>"https://system.na1.netsuite.com"}
+            "internalId"=>3,
+            "name"=>"Administrator"},
+          "dataCenterURLs"=>{
+            "webservicesDomain"=>"https://webservices.na1.netsuite.com",
+            "restDomain"=>"https://rest.na1.netsuite.com",
+            "systemDomain"=>"https://system.na1.netsuite.com"}
         },
         {
           "account"=>{
             "internalId"=>"TSTDRV15",
             "name"=>"Honeycomb Mfg SDN (Leading)"},
           "role"=>{
-              "internalId"=>1,
-              "name"=>"Accountant"},
-              "dataCenterURLs"=>{
-                "webservicesDomain"=>"https://webservices.na1.netsuite.com",
-                "restDomain"=>"https://rest.na1.netsuite.com",
-                "systemDomain"=>"https://system.na1.netsuite.com"}
+            "internalId"=>1,
+            "name"=>"Accountant"},
+           "dataCenterURLs"=>{
+             "webservicesDomain"=>"https://webservices.na1.netsuite.com",
+             "restDomain"=>"https://rest.na1.netsuite.com",
+             "systemDomain"=>"https://system.na1.netsuite.com"}
         },
         {
           "account"=>{
             "internalId"=>"TSTDRV14",
             "name"=>"Honeycomb Mfg SDN (Leading)"},
           "role"=>{
-              "internalId"=>1,
-              "name"=>"Accountant"},
-              "dataCenterURLs"=>{
-                "webservicesDomain"=>"https://webservices.na1.netsuite.com",
-                "restDomain"=>"https://rest.na1.netsuite.com",
-                "systemDomain"=>"https://system.na1.netsuite.com"}
+            "internalId"=>1,
+            "name"=>"Accountant"},
+          "dataCenterURLs"=>{
+            "webservicesDomain"=>"https://webservices.netsuite.com",
+            "restDomain"=>"https://rest.netsuite.com",
+            "systemDomain"=>"https://system.netsuite.com"}
         },
       ]
     end
 
-    it 'pulls out the account_ids' do
-      expect(
-        subject.send(:format_response, parsed).fetch(:accounts)
-      ).to eq ["TSTDRV15","TSTDRV14"]
+    it 'groups by uniq account ids' do
+      expect( subject.count ).to eq 2
+    end
+
+    it 'returns the account name and id' do
+      expect( subject.first ).to be_a Hash
+      expect( subject.first[:account_id] ).to be_a String
+      expect( subject.first[:account_name] ).to be_a String
     end
 
-    it 'pulls out the role_ids' do
-      expect(
-        subject.send(:format_response, parsed).fetch(:roles)
-      ).to eq [3, 1]
+    it 'returns the account roles' do
+      expect( subject.first[:roles] ).to be_an Array
+      expect( subject.first[:roles].first ).to be_a Hash
+      expect( subject.first[:roles].first[:id] ).to be_an Integer
+      expect( subject.first[:roles].first[:name] ).to be_a String
     end
 
-    it 'pulls out the wsdls' do
-      expect(
-        subject.send(:format_response, parsed).fetch(:wsdls)
-      ).to eq ["https://webservices.na1.netsuite.com"]
+    it 'returns the wsdls' do
+      expect( subject.first[:wsdls] ).to be_a Hash
+      expect( subject.first[:wsdls][:webservices] ).to be_an Array
+      expect( subject.first[:wsdls][:rest] ).to be_an Array
+      expect( subject.first[:wsdls][:system] ).to be_an Array
+    end
+
+    it 'parses correctly' do
+      expect(subject).to eq(
+        [
+          { account_id: 'TSTDRV15',
+            account_name: 'Honeycomb Mfg SDN (Leading)',
+            roles: [
+              {
+                id: 3,
+                name: 'Administrator'
+              },
+              {
+                id: 1,
+                name: 'Accountant'
+              }
+            ],
+            wsdls: {
+              webservices:  ["https://webservices.na1.netsuite.com"],
+              rest:         ["https://rest.na1.netsuite.com"],
+              system:       ["https://system.na1.netsuite.com"]
+            }
+          },
+          { account_id: 'TSTDRV14',
+            account_name: 'Honeycomb Mfg SDN (Leading)',
+            roles: [
+              {
+                id: 1,
+                name: 'Accountant'
+              }
+            ],
+            wsdls: {
+              webservices:  ["https://webservices.netsuite.com"],
+              rest:         ["https://rest.netsuite.com"],
+              system:       ["https://system.netsuite.com"]
+            }
+          }
+        ]
+      )
     end
 
     it 'handles an empty response body' do
       expect{
-        subject.send(:format_response, [])
+        described_class.send(:format_response, [])
       }.to_not raise_error
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: netsuite
 version: !ruby/object:Gem::Version
-  version: 0.7.2
+  version: 0.7.3
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-31 00:00:00.000000000 Z
+date: 2016-09-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: savon
@@ -228,6 +228,7 @@ files:
 - lib/netsuite/records/purchase_order_item_list.rb
 - lib/netsuite/records/record_ref.rb
 - lib/netsuite/records/record_ref_list.rb
+- lib/netsuite/records/return_authorization.rb
 - lib/netsuite/records/rev_rec_schedule.rb
 - lib/netsuite/records/rev_rec_template.rb
 - lib/netsuite/records/role_list.rb