sugarcrm 0.9.14 → 0.9.15

data/Gemfile

@@ -1,6 +1,6 @@
 source "http://rubygems.org"
 
-gem "activesupport", ">= 3.0.0", :require => "active_support"
+gem "activesupport", ">= 2.3.10", :require => "active_support"
 gem "i18n"
 gem "json"
 

data/README.rdoc

@@ -11,12 +11,13 @@ A less clunky way to interact with SugarCRM via REST.
 == FEATURES/PROBLEMS:
 
 * Works with all v2 API calls
-* Supports creation, saving, and deletion of SugarCRM specific objects.
+* Supports Rails 2 and 3
+* ActiveRecord style finders and objects
+* Supports creation, saving, and deletion of SugarCRM specific objects
+* Flexible extension framework
 * Validations, typecasting, and serialization of boolean, date, and integer fields
-* Query, update and delete records from collections! 
-* ActiveRecord style finders!
-* Auto-generation of SugarCRM specific objects.  When a connection is established, get_available_modules is called and the resultant modules are turned into SugarCRM::Module classes.  
-* If you want to use the vanilla API, you can access the methods directly on the SugarCRM.connection object.
+* Query, update and delete records from collections
+* Access API methods directly on the SugarCRM.connection object
 
 == SYNOPSIS:
 
@@ -164,7 +165,7 @@ A less clunky way to interact with SugarCRM via REST.
 
 == USING THE GEM WITH RAILS 3
 
-Note: this gem requires Rails 3 as it depends on Active Support >= 3.
+Note: this gem works with Active Support >= 2.3.10, but is optimized for Rails 3.
 
 1. Add the sugarcrm gem to your Gemfile (sugarcrm gem version >= 0.9.12)
 2. Run `bundle install`
@@ -245,7 +246,7 @@ To disconnect an active session:
 
 == REQUIREMENTS:
 
-* activesupport >= 3.0.0
+* activesupport >= 2.3.10
 * i18n
 * json
 
@@ -271,4 +272,4 @@ Put your credentials in a file called `test/config.yaml` (which you will have to
 
 == Copyright
 
-Copyright (c) 2010 Carl Hicks. See LICENSE for details.
+Copyright (c) 2011 Carl Hicks. See LICENSE for details.

data/VERSION

@@ -1 +1 @@
-0.9.14
+0.9.15

data/lib/sugarcrm/attributes/attribute_methods.rb

@@ -124,6 +124,17 @@ module SugarCRM; module AttributeMethods
       end
       ?
     end
+    
+    # return the (polymorphic) parent record corresponding to the parent_id and parent_type attributes
+    # (an example of parent polymorphism can be found in the Note module)
+    if (@attributes.keys.include? 'parent_id') && (@attributes.keys.include? 'parent_type')
+      self.class.module_eval %Q?
+      def parent
+        (self.class.session.namespace_const.const_get @attributes['parent_type'].singularize).find(@attributes['parent_id'])
+      end
+      ?
+    end
+    
     self.class.attribute_methods_generated = true
   end
 

data/lib/sugarcrm/attributes/attribute_validations.rb

@@ -1,7 +1,7 @@
 module SugarCRM; module AttributeValidations
   # Checks to see if we have all the neccessary attributes
   def valid?
-    @errors = ActiveSupport::HashWithIndifferentAccess.new
+    @errors = (defined?(HashWithIndifferentAccess) ? HashWithIndifferentAccess : ActiveSupport::HashWithIndifferentAccess).new
     
     self.class._module.required_fields.each do |attribute|
       valid_attribute?(attribute)
@@ -59,4 +59,4 @@ module SugarCRM; module AttributeValidations
     @errors[attribute] = @errors[attribute] << message unless @errors[attribute].include? message
     @errors
   end
-end; end
+end; end

data/lib/sugarcrm/session.rb

@@ -102,10 +102,11 @@ module SugarCRM; class Session
   end
   
   # load credentials from file, and (re)connect to SugarCRM
-  def load_config(path)
+  def load_config(path, opts = {})
+    opts.reverse_merge! :reconnect => true
     new_config = self.class.parse_config_file(path)
-    @config[:config] = new_config[:config] if new_config
-    reconnect(@config[:base_url], @config[:username], @config[:password]) if connection_info_loaded?
+    update_config(new_config[:config]) if new_config and new_config[:config]
+    reconnect(@config[:base_url], @config[:username], @config[:password]) if opts[:reconnect] and connection_info_loaded?
     @config
   end
   
@@ -207,4 +208,4 @@ module SugarCRM; class Session
     result[:wait_timeout] = wait_timeout if wait_timeout
     result
   end
-end; end
+end; end

data/sugarcrm.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{sugarcrm}
-  s.version = "0.9.14"
+  s.version = "0.9.15"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Carl Hicks", "David Sulc"]
-  s.date = %q{2011-05-03}
+  s.date = %q{2011-05-18}
   s.default_executable = %q{sugarcrm}
   s.email = %q{carl.hicks@gmail.com}
   s.executables = ["sugarcrm"]
@@ -117,14 +117,14 @@ Gem::Specification.new do |s|
     s.specification_version = 3
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_runtime_dependency(%q<activesupport>, [">= 3.0.0"])
+      s.add_runtime_dependency(%q<activesupport>, [">= 2.3.10"])
       s.add_runtime_dependency(%q<i18n>, [">= 0"])
       s.add_runtime_dependency(%q<json>, [">= 0"])
       s.add_development_dependency(%q<shoulda>, [">= 0"])
       s.add_development_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_development_dependency(%q<jeweler>, ["~> 1.5.2"])
     else
-      s.add_dependency(%q<activesupport>, [">= 3.0.0"])
+      s.add_dependency(%q<activesupport>, [">= 2.3.10"])
       s.add_dependency(%q<i18n>, [">= 0"])
       s.add_dependency(%q<json>, [">= 0"])
       s.add_dependency(%q<shoulda>, [">= 0"])
@@ -132,7 +132,7 @@ Gem::Specification.new do |s|
       s.add_dependency(%q<jeweler>, ["~> 1.5.2"])
     end
   else
-    s.add_dependency(%q<activesupport>, [">= 3.0.0"])
+    s.add_dependency(%q<activesupport>, [">= 2.3.10"])
     s.add_dependency(%q<i18n>, [">= 0"])
     s.add_dependency(%q<json>, [">= 0"])
     s.add_dependency(%q<shoulda>, [">= 0"])

data/test/connection/test_get_entry_list.rb

@@ -2,12 +2,13 @@ require 'helper'
 
 class TestGetEntryList < ActiveSupport::TestCase
   context "A SugarCRM.connection" do
-    should "return a list of entries when sent #get_entry_list and no fields." do
+    should "return a list of entries when sent #get_entry_list" do
       users = SugarCRM.connection.get_entry_list(
         "Users",
         "users.deleted = 0"
       )
-      assert_equal "admin", users.first.user_name
+      assert_kind_of Array, users
+      assert_equal SugarCRM.config[:username], users.first.user_name
     end
     should "return an object when #get_entry_list" do
       @response = SugarCRM.connection.get_entry_list(
@@ -19,4 +20,4 @@ class TestGetEntryList < ActiveSupport::TestCase
       assert_instance_of SugarCRM::User, @response[0]
     end
   end
-end
+end

data/test/connection/test_get_module_fields.rb

@@ -4,8 +4,8 @@ class TestModuleFields < ActiveSupport::TestCase
   context "A SugarCRM.connection" do
     should "return a hash of module fields when #get_module_fields" do
       fields = SugarCRM.connection.get_module_fields("Users")
-      assert_instance_of ActiveSupport::HashWithIndifferentAccess, fields
+      assert_kind_of Hash, fields
       assert "address_city", fields.keys[0]
     end
   end
-end
+end

data/test/helper.rb

@@ -1,6 +1,7 @@
 require 'rubygems'
 require 'test/unit'
 require 'shoulda'
+require 'active_support/test_case'
 
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 $LOAD_PATH.unshift(File.dirname(__FILE__))

data/test/test_finders.rb

@@ -5,9 +5,9 @@ class TestFinders < ActiveSupport::TestCase
     should "always return an Array when :all" do
       users = SugarCRM::User.all(:limit => 10)
       assert_instance_of Array, users
-      users = SugarCRM::User.find(:all, :conditions => {:user_name => '= admin'})
+      users = SugarCRM::User.find(:all, :conditions => {:user_name => "= #{users.first.user_name}"})
       assert_instance_of Array, users
-      assert users.length == 1
+      assert_equal 1, users.length
       users = SugarCRM::User.find(:all, :conditions => {:user_name => '= invalid_user_123'})
       assert_instance_of Array, users
       assert users.length == 0
@@ -101,12 +101,12 @@ class TestFinders < ActiveSupport::TestCase
     
     should "receive a response containing all fields when sent #get_entry" do
       u = SugarCRM::User.find(1)
-      assert_equal u.user_name, "admin"
+      assert_not_nil u.user_name
     end
     
     should "return an array of records when sent #find([id1, id2, id3])" do
       users = SugarCRM::User.find(["seed_sarah_id", 1])
-      assert_equal "admin", users.last.user_name
+      assert_not_nil users.last.user_name
     end
     
     # test Base#find_by_sql edge case
@@ -198,4 +198,4 @@ class TestFinders < ActiveSupport::TestCase
       assert a.delete
     end
   end
-end
+end

data/test/test_session.rb

@@ -15,6 +15,7 @@ CONFIG_CONTENTS.freeze
 class TestSession < ActiveSupport::TestCase
   context "The SugarCRM::Session class" do
     should "raise SugarCRM::MissingCredentials if at least one of url/username/password is missing" do
+      
       assert_raise(SugarCRM::MissingCredentials){ SugarCRM.connect('http://127.0.0.1/sugarcrm', nil, nil) }
     end
     
@@ -33,7 +34,7 @@ class TestSession < ActiveSupport::TestCase
     end
     
     should "parse config parameters from a file" do
-      assert_equal CONFIG_CONTENTS, SugarCRM::Session.parse_config_file(CONFIG_PATH)
+      assert_equal CONFIG_CONTENTS, SugarCRM::Session.parse_config_file(CONFIG_TEST_PATH)
     end
     
     should "create a session from a config file" do
@@ -59,8 +60,9 @@ class TestSession < ActiveSupport::TestCase
     end
     
     should "load config file" do
-      SugarCRM.load_config CONFIG_TEST_PATH
+      SugarCRM.load_config CONFIG_TEST_PATH, :reconnect => false
       CONFIG_CONTENTS[:config].each{|k,v| assert_equal v, SugarCRM.config[k]}
+      SugarCRM.load_config CONFIG_PATH
     end
     
     should "be able to disconnect, and log in to Sugar automatically if credentials are present in config file" do

data/test/test_sugarcrm.rb

@@ -13,7 +13,7 @@ class TestSugarCRM < ActiveSupport::TestCase
     
     should "implement self.count" do
       nb_accounts = SugarCRM::Account.count
-      assert nb_accounts > 0
+      assert nb_accounts > 0, "There should be some Accounts"
       nb_inc_accounts = nil
       assert_nothing_raised do
         nb_inc_accounts = SugarCRM::Account.count(:conditions => {:name => "LIKE '%Inc'"})
@@ -38,7 +38,7 @@ class TestSugarCRM < ActiveSupport::TestCase
     end
     
     should "return the module fields" do
-      assert_instance_of ActiveSupport::HashWithIndifferentAccess, SugarCRM::Account._module.fields
+      assert_kind_of Hash, SugarCRM::Account._module.fields
     end
     
     should "respond to self#methods" do
@@ -59,7 +59,7 @@ class TestSugarCRM < ActiveSupport::TestCase
     end
   
     should "respond to self.attributes_from_modules_fields" do
-      assert_instance_of ActiveSupport::HashWithIndifferentAccess, SugarCRM::User.attributes_from_module
+      assert_kind_of Hash, SugarCRM::User.attributes_from_module
     end
   
     should "return an instance of itself when #new" do
@@ -189,4 +189,12 @@ class TestSugarCRM < ActiveSupport::TestCase
     end
   end
   
+  context  "A SugarCRM::Note instance" do
+    should "return the correct parent record with the `parent` method" do
+      note = SugarCRM::Note.first
+      parent = note.parent
+      assert_equal note.parent_id, parent.id
+      assert_equal note.parent_type.singularize, parent.class.to_s.split('::').last
+    end
+  end
 end

metadata

@@ -2,7 +2,7 @@
 name: sugarcrm
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.9.14
+  version: 0.9.15
 platform: ruby
 authors: 
 - Carl Hicks
@@ -11,7 +11,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-05-03 00:00:00 -07:00
+date: 2011-05-18 00:00:00 -07:00
 default_executable: sugarcrm
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -21,7 +21,7 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 3.0.0
+        version: 2.3.10
   type: :runtime
   prerelease: false
   version_requirements: *id001
@@ -194,7 +194,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: -4256578188939241626
+      hash: -1738284526985119473
       segments: 
       - 0
       version: "0"