activerecord 1.5.0 → 1.5.1

data/CHANGELOG

@@ -1,3 +1,10 @@
+*1.5.1* (January 18th, 2005)
+
+* Fixed that the belongs_to and has_one proxy would fail a test like 'if project.manager' -- this unfortunately also means that you can't call methods like project.manager.build unless there already is a manager on the project #492 [Tim Bates]
+
+* Fixed that the Ruby/MySQL adapter wouldn't connect if the password was empty #503 [Pelle]
+
+
 *1.5.0* (January 17th, 2005)
 
 * Fixed that unit tests for MySQL are now run as the "rails" user instead of root #455 [Eric Hodel]

data/lib/active_record/associations.rb

@@ -228,7 +228,7 @@ module ActiveRecord
 
         add_multiple_associated_save_callbacks(association_name)
 
-        association_accessor_methods(association_name, association_class_name, association_class_primary_key_name, options, HasManyAssociation)
+        collection_accessor_methods(association_name, association_class_name, association_class_primary_key_name, options, HasManyAssociation)
         
         # deprecated api
         deprecated_collection_count_method(association_name)
@@ -289,7 +289,7 @@ module ActiveRecord
         module_eval do
           after_save <<-EOF
             association = instance_variable_get("@#{association_name}")
-            if (true or @new_record_before_save) and association.respond_to?(:loaded?) and not association.nil?
+            unless association.nil?
               association["#{association_class_primary_key_name}"] = id
               association.save(true)
               association.send(:construct_sql)
@@ -364,7 +364,7 @@ module ActiveRecord
         module_eval do
           before_save <<-EOF
             association = instance_variable_get("@#{association_name}")
-            if association.respond_to?(:loaded?) and not association.nil? and association.new_record?
+            if not association.nil? and association.new_record?
               association.save(true)
               self["#{association_class_primary_key_name}"] = association.id
               association.send(:construct_sql)
@@ -469,7 +469,7 @@ module ActiveRecord
 
         add_multiple_associated_save_callbacks(association_name)
       
-        association_accessor_methods(association_name, association_class_name, association_class_primary_key_name, options, HasAndBelongsToManyAssociation)
+        collection_accessor_methods(association_name, association_class_name, association_class_primary_key_name, options, HasAndBelongsToManyAssociation)
 
         before_destroy_sql = "DELETE FROM #{options[:join_table]} WHERE #{association_class_primary_key_name} = \\\#{self.quoted_id}"
         module_eval(%{before_destroy "self.connection.delete(%{#{before_destroy_sql}})"}) # "
@@ -512,6 +512,43 @@ module ActiveRecord
         end
         
         def association_accessor_methods(association_name, association_class_name, association_class_primary_key_name, options, association_proxy_class)
+          define_method(association_name) do |*params|
+            force_reload = params.first unless params.empty?
+            association = instance_variable_get("@#{association_name}")
+            if association.nil? or force_reload
+              association = association_proxy_class.new(self,
+                association_name, association_class_name,
+                association_class_primary_key_name, options)
+              retval = association.reload
+              unless retval.nil?
+                instance_variable_set("@#{association_name}", association)
+              else
+                instance_variable_set("@#{association_name}", nil)
+                return nil
+              end
+            end
+            association
+          end
+
+          define_method("#{association_name}=") do |new_value|
+            association = instance_variable_get("@#{association_name}")
+            if association.nil?
+              association = association_proxy_class.new(self,
+                association_name, association_class_name,
+                association_class_primary_key_name, options)
+            end
+            association.replace(new_value)
+            unless new_value.nil?
+              instance_variable_set("@#{association_name}", association)
+            else
+              instance_variable_set("@#{association_name}", nil)
+              return nil
+            end
+            association
+          end
+        end
+
+        def collection_accessor_methods(association_name, association_class_name, association_class_primary_key_name, options, association_proxy_class)
           define_method(association_name) do |*params|
             force_reload = params.first unless params.empty?
             association = instance_variable_get("@#{association_name}")

data/lib/active_record/associations/association_collection.rb

@@ -11,10 +11,6 @@ module ActiveRecord
         @loaded = false
       end
 
-      def reload
-        reset
-      end
-
       # Add +records+ to this association.  Returns +self+ so method calls may be chained.  
       # Since << flattens its argument list and inserts each record, +push+ and +concat+ behave identically.
       def <<(*records)

data/lib/active_record/associations/association_proxy.rb

@@ -14,6 +14,11 @@ module ActiveRecord
         reset
       end
       
+      def reload
+        reset
+        load_target
+      end
+
       def method_missing(symbol, *args, &block)
         load_target
         @target.send(symbol, *args, &block)

data/lib/active_record/associations/belongs_to_association.rb

@@ -7,11 +7,6 @@ module ActiveRecord
         @loaded = false
       end
 
-      def reload
-        reset
-        load_target
-      end
-
       def create(attributes = {})
         record = build(attributes)
         record.save
@@ -34,12 +29,8 @@ module ActiveRecord
           @owner[@association_class_primary_key_name] = obj.id unless obj.new_record?
         end
         @loaded = true
-      end
 
-      # Ugly workaround - .nil? is done in C and the method_missing trick doesn't work when we pretend to be nil
-      def nil?
-        load_target
-        @target.nil?
+        return (@target.nil? ? nil : self)
       end
 
       private
@@ -65,10 +56,3 @@ module ActiveRecord
     end
   end
 end
-
-class NilClass #:nodoc:
-  # Ugly workaround - nil comparison is usually done in C and so a proxy object pretending to be nil doesn't work.
-  def ==(other)
-    other.nil?
-  end
-end

data/lib/active_record/associations/has_one_association.rb

@@ -25,9 +25,9 @@ module ActiveRecord
 
         @loaded = true
         unless @owner.new_record? or obj.nil? or dont_save
-          return (obj.save ? obj : false)
+          return (obj.save ? self : false)
         else
-          return obj
+          return (obj.nil? ? nil : self)
         end
       end
       

data/lib/active_record/base.rb

@@ -565,7 +565,7 @@ module ActiveRecord #:nodoc:
       # Transforms attribute key names into a more humane format, such as "First name" instead of "first_name". Example:
       #   Person.human_attribute_name("first_name") # => "First name"
       def human_attribute_name(attribute_key_name)
-        attribute_key_name.gsub(/_/, " ").capitalize unless attribute_key_name.nil?
+        attribute_key_name.humanize
       end
       
       def descends_from_active_record? # :nodoc:

data/lib/active_record/support/binding_of_caller.rb

@@ -2,7 +2,7 @@ begin
   require 'simplecc'
 rescue LoadError
   class Continuation #:nodoc:
-    def create(*args, &block)
+    def self.create(*args, &block)
       cc = nil; result = callcc {|c| cc = c; block.call(cc) if block and args.empty?}
       result ||= args
       return *[cc, *result]
@@ -36,7 +36,7 @@ class Binding #:nodoc:
   # If you don't do this an Exception will be raised. Because of
   # the way that Binding.of_caller is implemented it has to be
   # done this way.
-  def of_caller(&block)
+  def self.of_caller(&block)
     old_critical = Thread.critical
     Thread.critical = true
     count = 0

data/lib/active_record/support/core_ext/string/inflections.rb

@@ -31,6 +31,10 @@ module ActiveSupport #:nodoc:
         def classify
           Inflector.classify(self)
         end
+        
+        def humanize
+          Inflector.humanize(self)
+        end
 
         def foreign_key(separate_class_name_and_id_with_underscore = true)
           Inflector.foreign_key(self, separate_class_name_and_id_with_underscore)

data/lib/active_record/support/inflector.rb

@@ -27,6 +27,10 @@ module Inflector
     camel_cased_word.to_s.gsub(/([A-Z]+)([A-Z])/,'\1_\2').gsub(/([a-z])([A-Z])/,'\1_\2').downcase
   end
 
+  def humanize(lower_case_and_underscored_word)
+    lower_case_and_underscored_word.to_s.gsub(/_/, " ").capitalize
+  end
+
   def demodulize(class_name_in_module)
     class_name_in_module.to_s.gsub(/^.*::/, '')
   end

data/lib/active_record/vendor/mysql411.rb

@@ -139,6 +139,7 @@ class Mysql
   #          candidate_hash2=sha1(hash_stage1)
   #          check(candidate_hash2==hash_stage2)
   def scramble411( password, seed, old_ver )
+    return "" if password == nil or password == ""
     raise "old version password is not implemented" if old_ver
     
     #    print "Seed Bytes = "
@@ -302,4 +303,4 @@ class Mysql
     end
     self
   end
-end
+end

data/rakefile

@@ -8,7 +8,7 @@ require 'rake/contrib/rubyforgepublisher'
 
 PKG_BUILD     = ENV['PKG_BUILD'] ? '.' + ENV['PKG_BUILD'] : ''
 PKG_NAME      = 'activerecord'
-PKG_VERSION   = '1.5.0' + PKG_BUILD
+PKG_VERSION   = '1.5.1' + PKG_BUILD
 PKG_FILE_NAME = "#{PKG_NAME}-#{PKG_VERSION}"
 
 PKG_FILES = FileList[

data/test/associations_test.rb

@@ -105,7 +105,7 @@ class HasOneAssociationsTest < Test::Unit::TestCase
     firm = Firm.new("name" => "GlobalMegaCorp")
     firm.save
 
-    account = firm.account.build("credit_limit" => 1000)
+    firm.account = account = Account.new("credit_limit" => 1000)
     assert_equal account, firm.account
     assert account.save
     assert_equal account, firm.account
@@ -125,7 +125,7 @@ class HasOneAssociationsTest < Test::Unit::TestCase
   def test_build_before_either_saved
     firm = Firm.new("name" => "GlobalMegaCorp")
 
-    account = firm.account.build("credit_limit" => 1000)
+    firm.account = account = Account.new("credit_limit" => 1000)
     assert_equal account, firm.account
     assert account.new_record?
     assert firm.save
@@ -137,7 +137,7 @@ class HasOneAssociationsTest < Test::Unit::TestCase
     firm = Firm.new("name" => "GlobalMegaCorp")
     firm.save
     
-    account = firm.account.build
+    firm.account = account = Account.new
     assert_equal account, firm.account
     assert !account.save
     assert_equal account, firm.account
@@ -147,12 +147,14 @@ class HasOneAssociationsTest < Test::Unit::TestCase
   def test_create
     firm = Firm.new("name" => "GlobalMegaCorp")
     firm.save
-    assert_equal firm.account.create("credit_limit" => 1000), firm.account
+    firm.account = account = Account.create("credit_limit" => 1000)
+    assert_equal account, firm.account
   end
 
   def test_create_before_save
     firm = Firm.new("name" => "GlobalMegaCorp")
-    assert_equal firm.account.create("credit_limit" => 1000), firm.account
+    firm.account = account = Account.create("credit_limit" => 1000)
+    assert_equal account, firm.account
   end
 
   def test_dependence_with_missing_association
@@ -243,6 +245,21 @@ class HasManyAssociationsTest < Test::Unit::TestCase
     assert_equal 1, Firm.find_first.clients_using_counter_sql.size
     assert_equal 0, Firm.find_first.clients_using_zero_counter_sql.size
   end
+  
+  def test_belongs_to_sanity
+    c = Client.new
+    assert_nil c.firm
+
+    if c.firm
+      assert false, "belongs_to failed if check"
+    end
+
+    unless c.firm
+    else
+      assert false,  "belongs_to failed unless check"
+    end
+        
+  end
 
   def test_find_ids
     firm = Firm.find_first

metadata

@@ -3,8 +3,8 @@ rubygems_version: 0.8.4
 specification_version: 1
 name: activerecord
 version: !ruby/object:Gem::Version 
-  version: 1.5.0
-date: 2005-01-17
+  version: 1.5.1
+date: 2005-01-18
 summary: Implements the ActiveRecord pattern for ORM.
 require_paths: 
   - lib