landslider 0.3.1 → 0.3.2

data/CHANGELOG.markdown

@@ -1,3 +1,8 @@
+# landslider 0.3.2 2011-04-28
+
+ * add in custom field data for accounts
+ * new api methods: getInstanceInformation, getUserInformationById
+
 # landslider 0.2.5 2011-04-27
 
  * remove curb dependency

data/README.rdoc

@@ -35,6 +35,26 @@ This gem requires the following gems:
 	  :version => 2
 	}
 	
+	response = Landslider.login('LOGINTOKEN=' + LS_INSTANCE_NAME)
 	
+	response = Landslider.get_accounts(response[:session_id])
 	
-	more here later..
+	response[:accounts].each do |account| 
+		puts "id: #{account[:account_id]} name: #{account[:account_name]}"
+	end
+
+=== Supported API methods
+
+	* login
+	* getApiVersion
+	* getAccounts
+	* getAccountById
+	* getAccountContacts
+	* getAccountNotes
+	* getAccountOpportunities
+	* getContactNotes
+	* getInstanceInformation
+	* getLeads
+	* getLeadNotes
+	* getOpportunityNotes
+	* getUserInformationById

data/VERSION.yml

@@ -1,5 +1,5 @@
 --- 
 :major: 0
 :minor: 3
-:patch: 1
+:patch: 2
 :build: 

data/landslider.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{landslider}
-  s.version = "0.3.1"
+  s.version = "0.3.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Jay Prall"]

data/lib/landslider.rb

@@ -73,7 +73,6 @@ class Landslider < Handsoap::Service
 		response = invoke("getAccountById", :soap_action => :none) do |message|
 			message.add 'accountId', account_id
 		end
-		
 		node = response.document.xpath('//ns:getAccountByIdResponse', ns)
 		parse_get_account_by_id_result(node)
 	end
@@ -298,8 +297,14 @@ class Landslider < Handsoap::Service
 	
 	def parse_get_instance_information_result(node)
 		{
+		:address => parse_address(node.xpath('./*/address')),
+		:company_name => xml_to_str(node, './*/companyName/text()'),
+		:instance_url => xml_to_str(node, './*/instanceURL/text()'),
+			
 		:error => xml_to_bool(node, './*/error/text()'),
-		:error_code => xml_to_int(node, './*/errorCode/text()')
+		:error_code => xml_to_int(node, './*/errorCode/text()'),
+		:result_msg => xml_to_str(node, './*/resultMsg/text()'),
+		:status_code => xml_to_int(node, './*/statusCode/text()')
 		}
 	end
 	
@@ -374,7 +379,8 @@ class Landslider < Handsoap::Service
 		:created_by => xml_to_int(node, './createdBy/text()'),
 		:created_on => xml_to_str(node, './createdOn/text()'),
 		:archived_by => xml_to_str(node, './archivedBy/text()'),
-		:sync_with_quickbooks => xml_to_str(node, './isSyncWithQuickbooks/text()')
+		:sync_with_quickbooks => xml_to_str(node, './isSyncWithQuickbooks/text()'),
+		:custom_fields => node.xpath('./customFields', ns).map { |child| parse_custom_field(child) }
 		}
 		
 	end
@@ -417,6 +423,16 @@ class Landslider < Handsoap::Service
 		}
 	end
 	
+	# WsCustomField
+	def parse_custom_field(node)
+		{
+		:custom_field_id => xml_to_int(node, './customFieldId/text()'),
+		:custom_field_name => xml_to_str(node, './customFieldName/text()'),
+		:custom_field_type => xml_to_str(node, './customFieldType/text()'),
+		:custom_field_value => xml_to_str(node, './customFieldValue/customFieldValue/text()')
+		}
+	end
+	
 	# WsEmployee
 	def parse_employee(node)
 		{
@@ -449,7 +465,6 @@ class Landslider < Handsoap::Service
 		:latest => xml_to_bool(node, './latest/text()'),
 		:note_id => xml_to_int(node, './noteId/text()'),
 		:note_html => xml_to_str(node, './note/text()')
-		
 		}
 	end
 	
@@ -467,7 +482,6 @@ class Landslider < Handsoap::Service
 		:ok_to_email => xml_to_bool(node, './okToEmail/text()'),
 		:hot => xml_to_bool(node, './hot/text()')
 		}
-	
 	end
 	
 	# WsOpportunity

data/test/landslider_test.rb

@@ -3,6 +3,8 @@ require 'test_helper'
 
 class LandsliderTest < Test::Unit::TestCase
 
+	JAYTEST_ACCOUNT_ID = 55647822
+	
 	def setup
 		# TODO: fetch a session_id once
 		# if $sid.nil?
@@ -42,13 +44,26 @@ class LandsliderTest < Test::Unit::TestCase
 	end
 	
 	def test_landslider_get_account_by_id
-		result = Landslider.get_account_by_id($sid, 51857822)
+		result = Landslider.get_account_by_id($sid, JAYTEST_ACCOUNT_ID)
 		
 		assert_not_nil result
 		assert_equal false, result[:error]
 		assert_not_nil result[:account]
 	end
 	
+	def test_landslider_account_custom_fields
+		result = Landslider.get_account_by_id($sid, JAYTEST_ACCOUNT_ID)
+		
+		assert_not_nil result
+		assert_equal false, result[:error]
+		assert_not_nil result[:account]
+		
+		assert_not_nil result[:account][:custom_fields]
+		assert_equal Array, result[:account][:custom_fields].class
+		assert_operator result[:account][:custom_fields].length, :>=, 2
+		
+	end
+	
 	def test_landslider_get_account_contacts		
 		# exists on jaytest
 		result = Landslider.get_account_contacts($sid, 51857822)		
@@ -59,7 +74,7 @@ class LandsliderTest < Test::Unit::TestCase
 	
 	def test_landslider_get_account_notes
 		# exists on jaytest
-		result = Landslider.get_account_notes($sid, 55647822)
+		result = Landslider.get_account_notes($sid, JAYTEST_ACCOUNT_ID)
 		
 		validate_standard_api_result result
 		validate_at_least_one_note_returned result
@@ -87,12 +102,14 @@ class LandsliderTest < Test::Unit::TestCase
 	end
 	
 	def test_landslider_get_instance_information
-	
-		result = Landslider.get_instance_information($sid, 12640894)
-
+		# Landslide API doc bug..  integer user_id doesn't work
+		result = Landslider.get_instance_information($sid, 'jayp@landslide.com')
+		
 		assert_not_nil result
 		assert_equal false, result[:error]
-
+		assert_not_nil result[:address]
+		assert_not_nil result[:company_name]
+		assert_not_nil result[:instance_url]
 	end
 	
 	def test_landslider_get_leads

metadata

@@ -2,7 +2,7 @@
 name: landslider
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.3.1
+  version: 0.3.2
 platform: ruby
 authors: 
 - Jay Prall