netsuite 0.1.0 → 0.2.0

data/lib/netsuite/records/custom_record.rb

@@ -7,7 +7,7 @@ module NetSuite
       include Support::Actions
       include Namespaces::SetupCustom
 
-      actions :get, :add, :delete
+      actions :get, :get_list, :add, :delete, :search
 
       fields :allow_attachments, :allow_inline_editing, :allow_numbering_override, :allow_quick_search, :created,
         :custom_record_id, :description, :disclaimer, :enabl_email_merge, :enable_numbering, :include_name,
@@ -50,12 +50,6 @@ module NetSuite
         "#{record_namespace}:CustomRecord"
       end
 
-      def to_record
-        rec = super
-        rec["#{record_namespace}:customFieldList!"] = rec.delete("#{record_namespace}:customFieldList")
-        rec
-      end
-
     end
   end
 end

data/lib/netsuite/records/custom_record_ref.rb

@@ -37,6 +37,13 @@ module NetSuite
         end
       end
 
+      def to_record
+        rec = super
+        rec[:@internalId] = @internal_id if @internal_id
+        rec[:@typeId] = @type_id if @type_id
+        rec
+      end
+
     end
   end
 end

data/lib/netsuite/records/customer.rb

@@ -7,7 +7,7 @@ module NetSuite
       include Support::Actions
       include Namespaces::ListRel
 
-      actions :get, :add, :update, :delete, :search
+      actions :get, :get_list, :add, :update, :delete, :search
 
       fields :access_role, :account_number, :aging, :alt_email, :alt_name, :alt_phone, :bill_pay,
         :buying_reason, :buying_time_frame, :campaign_category, :category, :click_stream, :comments, :company_name,
@@ -16,9 +16,9 @@ module NetSuite
         :download_list, :email, :email_preference, :email_transactions, :end_date, :entity_id,
         :estimated_budget, :fax, :fax_transactions, :first_name, :first_visit, :give_access, :global_subscription_status,
         :group_pricing_list, :home_phone, :image, :is_budget_approved, :is_inactive, :is_person, :item_pricing_list, :keywords,
-        :language, :last_modified, :last_name, :last_page_visited, :last_visit, :lead_source, :middle_name, :mobile_phone,
+        :language, :last_modified_date, :last_name, :last_page_visited, :last_visit, :lead_source, :middle_name, :mobile_phone,
         :opening_balance, :opening_balance_account, :opening_balance_date, :parent, :partners_list,
-        :password, :password_2, :phone, :phonetic_name, :pref_cc_processor, :price_level, :print_on_check_as,
+        :password, :password2, :phone, :phonetic_name, :pref_cc_processor, :price_level, :print_on_check_as,
         :print_transactions, :referrer, :reminder_days, :representing_subsidiary, :require_pwd_change, :resale_number,
         :sales_group, :sales_readiness, :sales_rep, :sales_team_list, :salutation, :send_email, :ship_complete, :shipping_item,
         :stage, :start_date, :subscriptions_list, :subsidiary, :sync_partner_teams, :tax_exempt, :tax_item, :taxable, :terms,
@@ -35,6 +35,7 @@ module NetSuite
 
       attr_reader   :internal_id
       attr_accessor :external_id
+      attr_accessor :search_joins
 
       def initialize(attributes = {})
         @internal_id = attributes.delete(:internal_id) || attributes.delete(:@internal_id)

data/lib/netsuite/records/phone_call.rb

@@ -0,0 +1,29 @@
+module NetSuite
+  module Records
+    class PhoneCall
+      include Support::Fields
+      include Support::RecordRefs
+      include Support::Records
+      include Support::Actions
+      include Namespaces::ActSched
+
+      actions :get, :add, :delete, :update
+
+      fields :title, :message, :phone, :status, :priority, :start_date, :end_date,
+        :start_time, :end_time, :completed_date, :timed_event, :access_level, :timed_event
+
+      field :contact_list, ContactList
+
+      record_refs :assigned, :owner, :company, :contact
+
+      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
+    end
+  end
+end

data/lib/netsuite/records/record_ref.rb

@@ -15,7 +15,7 @@ module NetSuite
           attributes.delete(:"@xmlns:platform_core")
           @internal_id = attributes.delete(:internal_id) || attributes.delete(:@internal_id)
           @external_id = attributes.delete(:external_id) || attributes.delete(:@external_id)
-          @type        = attributes.delete(:type) || attributes.delete(:@type)
+          @type        = attributes.delete(:type) || attributes.delete(:@type) || attributes.delete(:"@xsi:type")
           @attributes  = attributes
         else
           record = attributes_or_record
@@ -33,14 +33,6 @@ module NetSuite
         end
       end
 
-      def attributes!
-        hash = {}
-        hash[:internalId] = @internal_id if @internal_id
-        hash[:externalId] = @external_id if @external_id
-        hash[:type]       = @type        if @type
-        hash
-      end
-
     end
   end
 end

data/lib/netsuite/records/sales_order.rb

@@ -27,10 +27,11 @@ module NetSuite
 
       record_refs :account, :bill_address_list, :created_from, :currency, :custom_form, :department, :discount_item, :entity, :gift_cert,
         :handling_tax_code, :job, :klass, :lead_source, :location, :message_sel, :opportunity, :partner, :posting_period, :promo_code,
-        :sales_group, :sales_rep, :ship_method, :shipping_tax_code, :subsidiary, :tax_item
+        :sales_group, :sales_rep, :ship_method, :shipping_tax_code, :subsidiary, :terms, :tax_item
 
       attr_reader :internal_id
       attr_accessor :external_id
+      attr_accessor :search_joins
 
       def initialize(attributes = {})
         @internal_id = attributes.delete(:internal_id) || attributes.delete(:@internal_id)

data/lib/netsuite/records/support_case.rb

@@ -0,0 +1,32 @@
+module NetSuite
+  module Records
+    class SupportCase
+      include Support::Fields
+      include Support::RecordRefs
+      include Support::Records
+      include Support::Actions
+      include Namespaces::ListSupport
+
+      actions :get, :add, :delete, :update
+
+      fields :end_date, :incoming_message, :outgoing_message, :search_solution, :email_form, 
+             :internal_only, :title, :case_number, :start_date, :email, :phone, :inbound_email, 
+             :is_inactive, :help_desk
+      
+      field :custom_field_list, CustomFieldList
+
+      read_only_fields :created_date, :last_modified_date, :last_message_date
+
+      record_refs :custom_form, :company, :contact, :issue, :status, :priority, :origin, :category, :assigned
+
+      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
+    end
+  end
+end

data/lib/netsuite/records/task.rb

@@ -7,14 +7,15 @@ module NetSuite
       include Support::Actions
       include Namespaces::ActSched
 
-      actions :get, :add, :delete, :update
+      actions :get, :add, :search, :delete, :update
 
       fields :title, :send_email, :message, :status, :access_level, :reminder_type,
-             :reminder_minutes, :start_date, :end_date, :due_date, :timed_event
+             :reminder_minutes, :start_date, :end_date, :due_date, :timed_event,
+             :created_date, :last_modified_date
 
       field :contact_list, ContactList
 
-      record_refs :assigned, :owner, :company, :contact
+      record_refs :assigned, :owner, :company, :contact, :transaction
 
       attr_reader   :internal_id
       attr_accessor :external_id

data/lib/netsuite/records/transaction.rb

@@ -1,26 +1,9 @@
+# TODO specs
+# TODO a transaction is actually the superclass of a SalesOrder, this is backwards
+
 module NetSuite
   module Records
-    class Transaction
-      include Support::Fields
-      include Support::RecordRefs
-      include Support::Records
-      include Support::Actions
-      include Namespaces::TranSales
-
-      actions :get, :search, :search_more_with_id, :initialize, :add, :delete
-
-      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 self.custom_soap_advanced_search_record_type
-        'tranSales:TransactionSearchAdvanced'
-      end
+    class Transaction < SalesOrder
 
       def to_record
         rec = super

data/lib/netsuite/support/actions.rb

@@ -18,6 +18,10 @@ module NetSuite
           case name
           when :get
             self.send(:include, NetSuite::Actions::Get::Support)
+          when :get_list
+            self.send(:include, NetSuite::Actions::GetList::Support)
+          when :get_select_value
+            self.send(:include, NetSuite::Actions::GetSelectValue::Support)
           when :search
             self.send(:include, NetSuite::Actions::Search::Support)
           when :search_more_with_id

data/lib/netsuite/support/records.rb

@@ -28,16 +28,19 @@ module NetSuite
           hash[:attributes!][kname] ||= {}
           hash[:attributes!][kname]['internalId'] = v.internal_id
         end
+
         if v.respond_to?(:external_id) && v.external_id
           hash[:attributes!] ||= {}
           hash[:attributes!][kname] ||= {}
           hash[:attributes!][kname]['externalId'] = v.external_id
         end
+
         if v.kind_of?(NetSuite::Records::RecordRef) && v.type
           hash[:attributes!] ||= {}
           hash[:attributes!][kname] ||= {}
           hash[:attributes!][kname]['type'] = v.type.lower_camelcase
         end
+        
         if v.kind_of?(NetSuite::Records::CustomRecordRef) && v.type_id
           hash[:attributes!] ||= {}
           hash[:attributes!][kname] ||= {}

data/lib/netsuite/support/search_result.rb

@@ -15,51 +15,75 @@ module NetSuite
       #   <platformCore:searchId>WEBSERVICES_738944_SB2_03012013650784545962753432_28d96bd280</platformCore:searchId>
 
       def initialize(response, result_class)
+        @result_class = result_class
         @response = response
         @total_records = response.body[:total_records].to_i
 
         if @total_records > 0
-          if @total_records == 1
-            [response.body[:record_list][:record]]
-          else
-            response.body[:record_list][:record]
-          end.each do |record|
-            results << result_class.new(record)
-          end
-        end
+          if response.body.has_key?(:record_list)
+            # basic search results
+            record_list = response.body[:record_list][:record]
+            record_list = [record_list] if @total_records == 1
 
-        # search_results = []
+            record_list.each do |record|
+              results << result_class.new(record)
+            end
+          elsif response.body.has_key? :search_row_list
+            # advanced search results
+            record_list = response.body[:search_row_list][:search_row]
+            record_list = [record_list] if @total_records == 1
 
-        # if !!response.body[:search_row_list] && !response.body[:search_row_list].empty?
-        #   response.body[:search_row_list][:search_row].each do |record|
-        #     search_result = NetSuite::Support::SearchResult.new(record)
+            record_list.each do |record|
+              # TODO because of customFieldList we need to either make this recursive
+              #      or handle the customFieldList as a special case
 
-        #     search_results << search_result
-        #   end
-        # end
+              record.each_pair do |search_group, search_data|
+                # skip all attributes: look for :basic and all :xxx_join
+                next if search_group.to_s.start_with?('@')
 
-        # search_id = response.header[:ns_id]
-        # page_index = response.body[:page_index]
-        # total_pages = response.body[:total_pages]
+                record[search_group].each_pair do |k, v|
+                  # all return values are wrapped in a <SearchValue/>
+                  # extract the value from <SearchValue/> to make results easier to work with
 
-        # response_hash[:search_id] = search_id
-        # response_hash[:page_index] = page_index
-        # response_hash[:total_pages] = total_pages
-        # response_hash[:search_results] = search_results
-        # puts response.body
-        # response_hash
+                  if v.is_a?(Hash) && v.has_key?(:search_value)
+                    # search return values that are just internalIds are stored as attributes on the searchValue element
+                    if v[:search_value].is_a?(Hash) && v[:search_value].has_key?(:'@internal_id')
+                      record[search_group][k] = v[:search_value][:'@internal_id']
+                    else
+                      record[search_group][k] = v[:search_value]
+                    end
+                  end
+                end
+              end
+
+              result_wrapper = result_class.new(record.delete(:basic))
+              result_wrapper.search_joins = record
+              results << result_wrapper
+            end
+          else
+            raise "uncaught search result"
+          end
+        end
       end
 
       def results
         @results ||= []
       end
 
-      def next_page
-        
-      end
+      def results_in_batches
+        while @response.body[:total_pages] != @response.body[:page_index]
+          yield results
+
+          next_search = @result_class.search(
+            search_id: @response.body[:search_id],
+            page_index: @response.body[:page_index].to_i + 1
+          )
+
+          @results = next_search.results
+          @response = next_search.response
+        end
 
-      def all_pages
-        
+        yield results
       end
 
     end

data/lib/netsuite/version.rb

@@ -1,3 +1,3 @@
 module Netsuite
-  VERSION = '0.1.0'
+  VERSION = '0.2.0'
 end

data/netsuite.gemspec

@@ -15,7 +15,7 @@ Gem::Specification.new do |gem|
   gem.require_paths = ['lib']
   gem.version       = Netsuite::VERSION
 
-  gem.add_dependency 'savon', '~> 2.2.0'
+  gem.add_dependency 'savon', '~> 2.3.0'
 
   gem.add_development_dependency 'rspec', '~> 2.10'
 end

data/spec/netsuite/actions/add_spec.rb

@@ -12,14 +12,12 @@ describe NetSuite::Actions::Add do
     before do
       savon.expects(:add).with(:message => {
         'platformMsgs:record' => {
-          'listRel:entityId'    => 'Shutter Fly',
-          'listRel:companyName' => 'Shutter Fly, Inc.'
+          :content! => {
+            'listRel:entityId'    => 'Shutter Fly',
+            'listRel:companyName' => 'Shutter Fly, Inc.'
+          },
+          '@xsi:type' => 'listRel:Customer'
         },
-        :attributes! => {
-          'platformMsgs:record' => {
-            'xsi:type' => 'listRel:Customer'
-          }
-        }
       }).returns(File.read('spec/support/fixtures/add/add_customer.xml'))
     end
 
@@ -42,13 +40,11 @@ describe NetSuite::Actions::Add do
     before do
       savon.expects(:add).with(:message => {
         'platformMsgs:record' => {
-          'tranSales:source' => 'Google'
+          :content! => {
+            'tranSales:source' => 'Google'
+          },
+          '@xsi:type' => 'tranSales:Invoice'
         },
-        :attributes! => {
-          'platformMsgs:record' => {
-            'xsi:type' => 'tranSales:Invoice'
-          }
-        }
       }).returns(File.read('spec/support/fixtures/add/add_invoice.xml'))
     end
 

data/spec/netsuite/actions/delete_spec.rb

@@ -11,14 +11,11 @@ describe NetSuite::Actions::Delete do
 
     before do
       savon.expects(:delete).with(:message => {
-        'platformMsgs:baseRef' => {},
-        :attributes! => {
-          'platformMsgs:baseRef' => {
-            'internalId' => '980',
-            'type'       => 'customer',
-            'xsi:type'   => 'platformCore:RecordRef'
-          }
-        }
+        'platformMsgs:baseRef' => {
+          '@internalId' => '980',
+          '@type'       => 'customer',
+          '@xsi:type'   => 'platformCore:RecordRef'
+        },
       }).returns(File.read('spec/support/fixtures/delete/delete_customer.xml'))
     end
 

data/spec/netsuite/actions/get_spec.rb

@@ -5,42 +5,57 @@ describe NetSuite::Actions::Get do
   after(:all) { savon.unmock! }
 
   describe 'Customer' do
-    before do
-      message = {
-        'platformMsgs:baseRef' => {},
-        :attributes! => {
+    context 'retrieving with externalId' do
+      before do
+        message = {
           'platformMsgs:baseRef' => {
-            'externalId' => 1,
-            'type'       => 'customer',
-            'xsi:type'  => 'platformCore:RecordRef'
-          }
+            '@externalId' => 1,
+            '@type'       => 'customer',
+            '@xsi:type'  => 'platformCore:RecordRef'
+          },
         }
-      }
 
-      savon.expects(:get).with(:message => message).returns(File.read('spec/support/fixtures/get/get_customer.xml'))
-    end
+        savon.expects(:get).with(:message => message).returns(File.read('spec/support/fixtures/get/get_customer.xml'))
+      end
 
-    it 'makes a valid request to the NetSuite API' do
-      NetSuite::Actions::Get.call(NetSuite::Records::Customer, :external_id => 1)
+      it 'makes a valid request to the NetSuite API' do
+        NetSuite::Actions::Get.call(NetSuite::Records::Customer, :external_id => 1)
+      end
+
+      it 'returns a valid Response object' do
+        response = NetSuite::Actions::Get.call(NetSuite::Records::Customer, :external_id => 1)
+        response.should be_kind_of(NetSuite::Response)
+      end
     end
 
-    it 'returns a valid Response object' do
-      response = NetSuite::Actions::Get.call(NetSuite::Records::Customer, :external_id => 1)
-      response.should be_kind_of(NetSuite::Response)
+    context "retrieving with internalId" do
+      before do
+        message = {
+          'platformMsgs:baseRef' => {
+            '@internalId' => 1,
+            '@type'       => 'customer',
+            '@xsi:type'   => 'platformCore:RecordRef'
+          },
+        }
+
+        savon.expects(:get).with(:message => message).returns(File.read('spec/support/fixtures/get/get_customer.xml'))
+      end
+
+      it "makes a valid request to the NetSuite API" do
+        customer = NetSuite::Records::Customer.get(1)
+        customer.balance.should == '100.0'
+      end
     end
   end
 
   describe 'Invoice' do
     before do
       savon.expects(:get).with(:message => {
-        'platformMsgs:baseRef' => {},
-        :attributes! => {
-          'platformMsgs:baseRef' => {
-            'externalId' => 1,
-            'type'       => 'invoice',
-            'xsi:type'  => 'platformCore:RecordRef'
-          }
-        }
+        'platformMsgs:baseRef' => {
+          '@externalId' => 1,
+          '@type'       => 'invoice',
+          '@xsi:type'  => 'platformCore:RecordRef'
+        },
       }).returns(File.read('spec/support/fixtures/get/get_invoice.xml'))
     end