activerecord 1.9.0 → 1.9.1

data/CHANGELOG

@@ -1,3 +1,16 @@
+*1.9.1* (27th March, 2005)
+
+* Fixed that Active Record objects with float attribute could not be cloned #808
+
+* Fixed that MissingSourceFile's wasn't properly detected in production mode #925 [Nicholas Seckar]
+
+* Fixed that :counter_cache option would look for a line_items_count column for a LineItem object instead of lineitems_count
+
+* Fixed that AR exists?() would explode on postgresql if the passed id did not match the PK type #900 [Scott Barron]
+
+* Fixed the MS SQL adapter to work with the new limit/offset approach and with binary data (still suffering from 7KB limit, though) #901 [delynnb]
+
+
 *1.9.0* (22th March, 2005)
 
 * Added adapter independent limit clause as a two-element array with the first being the limit, the second being the offset #795 [Sam Stephenson]. Example:

data/lib/active_record/associations.rb

@@ -376,12 +376,12 @@ module ActiveRecord
       
         if options[:counter_cache]
           module_eval(
-            "after_create '#{association_class_name}.increment_counter(\"#{Inflector.pluralize(self.to_s.downcase). + "_count"}\", #{association_class_primary_key_name})" +
+            "after_create '#{association_class_name}.increment_counter(\"#{self.to_s.underscore.pluralize + "_count"}\", #{association_class_primary_key_name})" +
             " unless #{association_name}.nil?'"
           )
 
           module_eval(
-            "before_destroy '#{association_class_name}.decrement_counter(\"#{Inflector.pluralize(self.to_s.downcase) + "_count"}\", #{association_class_primary_key_name})" +
+            "before_destroy '#{association_class_name}.decrement_counter(\"#{self.to_s.underscore.pluralize + "_count"}\", #{association_class_primary_key_name})" +
             " unless #{association_name}.nil?'"
           )          
         end

data/lib/active_record/base.rb

@@ -91,6 +91,15 @@ module ActiveRecord #:nodoc:
   # on the other hand, will sanitize the <tt>user_name</tt> and +password+ before inserting them in the query, which will ensure that
   # an attacker can't escape the query and fake the login (or worse).
   #
+  # When using multiple parameters in the conditions, it can easily become hard to read exactly what the fourth or fifth
+  # question mark is supposed to represent. In those cases, you can resort to named bind variables instead. That's done by replacing 
+  # the question marks with symbols and supplying a hash with values for the matching symbol keys:
+  #
+  #   Company.find_first([ 
+  #     "id = :id AND name = :name AND division = :division AND created_at > :accounting_date", 
+  #     { :id => 3, :name => "37signals", :division => "First", :accounting_date => '2005-01-01' }
+  #   ])
+  #
   # == Overwriting default accessors
   # 
   # All column values are automatically available through basic accessors on the Active Record object, but some times you
@@ -288,6 +297,7 @@ module ActiveRecord #:nodoc:
       #   Person.find(1, :conditions => "associate_id = 5"
       #   Person.find(1, 2, 6, :conditions => "status = 'active'"
       #   Person.find([7, 17], :conditions => ["sanitize_me = ?", "bare'quote"]
+      #   Person.find(25, :conditions => ["name = :name AND age = :age", { :name => "Mary", :age => 22 }]
       #
       # +RecordNotFound+ is raised if no record can be found.
       def find(*args)
@@ -330,7 +340,7 @@ module ActiveRecord #:nodoc:
       # Example:
       #   Person.exists?(5)
       def exists?(id)
-        !find_first("#{primary_key} = #{sanitize(id)}").nil?
+        !find_first("#{primary_key} = #{sanitize(id)}").nil? rescue false
       end
 
       # This method is deprecated in favor of find with the :conditions option.
@@ -1038,7 +1048,7 @@ module ActiveRecord #:nodoc:
         self.attribute_names.inject({}) do |attributes, name|
           begin
             attributes[name] = read_attribute(name).clone
-          rescue TypeError
+          rescue TypeError, NoMethodError
             attributes[name] = read_attribute(name)
           end
           attributes

data/lib/active_record/connection_adapters/sqlserver_adapter.rb

@@ -5,6 +5,27 @@ require 'active_record/connection_adapters/abstract_adapter'
 # Author: Joey Gibson <joey@joeygibson.com>
 # Date:   10/14/2004
 #
+# Modifications: DeLynn Berry <delynnb@megastarfinancial.com>
+# Date: 3/22/2005
+#
+# This adapter will ONLY work on Windows systems, since it relies on Win32OLE, which,
+# to my knowledge, is only available on Window.
+#
+# It relies on the ADO support in the DBI module. If you are using the
+# one-click installer of Ruby, then you already have DBI installed, but
+# the ADO module is *NOT* installed. You will need to get the latest
+# source distribution of Ruby-DBI from http://ruby-dbi.sourceforge.net/
+# unzip it, and copy the file <tt>src/lib/dbd_ado/ADO.rb</tt> to
+# <tt>X:/Ruby/lib/ruby/site_ruby/1.8/DBD/ADO/ADO.rb</tt> (you will need to create
+# the ADO directory). Once you've installed that file, you are ready to go.
+#
+# Options:
+#
+# * <tt>:host</tt> -- Defaults to localhost
+# * <tt>:username</tt> -- Defaults to sa
+# * <tt>:password</tt> -- Defaults to nothing
+# * <tt>:database</tt> -- The name of the database. No default, must be provided.
+#
 # I have tested this code on a WindowsXP Pro SP1 system,
 # ruby 1.8.2 (2004-07-29) [i386-mswin32], SQL Server 2000.
 #
@@ -19,7 +40,7 @@ module ActiveRecord
       username = config[:username] ? config[:username].to_s : 'sa'
       password = config[:password].to_s
 
-      if config.has_key? :database
+      if config.has_key?(:database)
         database = config[:database]
       else
         raise ArgumentError, "No database specified. Missing argument: database."
@@ -38,92 +59,110 @@ module ActiveRecord
 
       def initialize(name, default, sql_type = nil, is_identity = false, scale_value = 0)
         super(name, default, sql_type)
-
         @scale = scale_value
         @identity = is_identity
-      end 
-
-      def binary_to_string(value)
-        value
-      end 
-
-      def string_to_binary(value)
-        value
-      end 
+      end
 
       def simplified_type(field_type)
         case field_type
-          when /int/i
-            :integer
-          when /float|double|decimal|numeric/i
-            if @scale == 0
-              :integer
-            else  
-              :float
-              nil
-            end  
-          when /datetime/i
-            :datetime
-          when /timestamp/i
-            :timestamp
-          when /time/i
-            :time
-          when /date/i
-            :date
-          when /clob|text|ntext/i
-            :text
-          when /blob|binary|image/i
-            :binary
-          when /char|string/i
-            :string
-          when /boolean|bit/i
-            :boolean
+          when /int|bigint|smallint|tinyint/i                        : :integer
+          when /float|double|decimal|money|numeric|real|smallmoney/i : @scale == 0 ? :integer : :float
+          when /datetime|smalldatetime/i                             : :datetime
+          when /timestamp/i                                          : :timestamp
+          when /time/i                                               : :time
+          when /text|ntext/i                                         : :text
+          when /binary|image|varbinary/i                             : :binary
+          when /char|nchar|nvarchar|string|varchar/i                 : :string
+          when /bit/i                                                : :boolean
+        end
+      end
+
+      def type_cast(value)
+        return nil if value.nil? || value =~ /^\s*null\s*$/i
+        case type
+        when :string    then value
+        when :integer   then value == true || value == false ? value == true ? '1' : '0' : value.to_i
+        when :float     then value.to_f
+        when :datetime  then cast_to_date_or_time(value)
+        when :timestamp then cast_to_time(value)
+        when :time      then cast_to_time(value)
+        else value
+        end
+      end
+
+      def cast_to_date_or_time(value)
+        return value if value.is_a?(Date)
+        guess_date_or_time (value.is_a?(Time)) ? value : cast_to_time(value)
+      end
+
+      def cast_to_time(value)
+        return value if value.is_a?(Time)
+        time_array = ParseDate.parsedate value
+        time_array[0] ||= 2000; time_array[1] ||= 1; time_array[2] ||= 1;
+        Time.send Base.default_timezone, *time_array
+      end
+
+      def guess_date_or_time(value)
+        (value.hour == 0 and value.min == 0 and value.sec == 0) ?
+          Date.new(value.year, value.month, value.day) : value
+      end
+
+      # These methods will only allow the adapter to insert binary data with a length of 7K or less
+      # because of a SQL Server statement length policy.
+      def string_to_binary(value)
+        value.gsub(/(\r|\n|\0|\x1a)/) do
+          case $1
+            when "\r"
+              "%00"
+            when "\n"
+              "%01"
+            when "\0"
+              "%02"
+            when "\x1a"
+              "%03"
+          end
         end
       end
 
-      def string_to_time(string)
-        return string if string.is_a?(Time)
-        time_array = ParseDate.parsedate(string, true)
-        time_array.each_index do |i|
-          case i
-            when 0
-              time_array[i] = time_array[i].nil? ? "2000" : time_array[i].to_s
-            when 1
-              time_array[i] = time_array[i].nil? ? "Jan" : time_array[i].to_s
-            when 2
-              time_array[i] = time_array[i].nil? ? "1" : time_array[i].to_s
-            when 3
-              time_array[i] = time_array[i].nil? ? "0" : time_array[i].to_s
-            when 4
-              time_array[i] = time_array[i].nil? ? "0" : time_array[i].to_s
-            when 5
-              time_array[i] = time_array[i].nil? ? "0" : time_array[i].to_s
+      def binary_to_string(value)
+        value.gsub(/(%00|%01|%02|%03)/) do
+          case $1
+            when "%00"
+              "\r"
+            when "%01"
+              "\n"
+            when "%02\0"
+              "\0"
+            when "%03"
+              "\x1a"
           end
         end
-        # treat 0000-00-00 00:00:00 as nil
-        Time.send(Base.default_timezone, *time_array) rescue nil
       end
 
     end
 
-    # This adapter will ONLY work on Windows systems, since it relies on Win32OLE, which,
-    # to my knowledge, is only available on Window.
-    #
-    # It relies on the ADO support in the DBI module. If you are using the
-    # one-click installer of Ruby, then you already have DBI installed, but
-    # the ADO module is *NOT* installed. You will need to get the latest
-    # source distribution of Ruby-DBI from http://ruby-dbi.sourceforge.net/
-    # unzip it, and copy the file <tt>src/lib/dbd_ado/ADO.rb</tt> to
-    # <tt>X:/Ruby/lib/ruby/site_ruby/1.8/DBD/ADO/ADO.rb</tt> (you will need to create
-    # the ADO directory). Once you've installed that file, you are ready to go.
-    #
-    # Options:
-    #
-    # * <tt>:host</tt> -- Defaults to localhost
-    # * <tt>:username</tt> -- Defaults to sa
-    # * <tt>:password</tt> -- Defaults to nothing
-    # * <tt>:database</tt> -- The name of the database. No default, must be provided.
     class SQLServerAdapter < AbstractAdapter
+
+      def native_database_types
+        {
+          :primary_key => "int NOT NULL IDENTITY(1, 1) PRIMARY KEY",
+          :string      => { :name => "varchar(255)" },
+          :text        => { :name => "text(16)" },
+          :integer     => { :name => "int(4)", :limit => 11 },
+          :float       => { :name => "float(8)" },
+          :datetime    => { :name => "datetime(8)" },
+          :timestamp   => { :name => "datetime(8)" },
+          :time        => { :name => "datetime(8)" },
+          :date        => { :name => "datetime(8)" },
+          :binary      => { :name => "image(16)" },
+          :boolean     => { :name => "bit(1)" }
+        }
+      end
+
+      def adapter_name
+        'SQLServer'
+      end
+
       def select_all(sql, name = nil)
         add_limit!(sql, nil)
         select(sql, name)
@@ -136,36 +175,22 @@ module ActiveRecord
       end
 
       def columns(table_name, name = nil)
-        sql = <<EOL
-SELECT 
-COLUMN_NAME as ColName,
-COLUMN_DEFAULT as DefaultValue,
-DATA_TYPE as ColType,
-COL_LENGTH('#{table_name}', COLUMN_NAME) as Length,
-COLUMNPROPERTY(OBJECT_ID('#{table_name}'), COLUMN_NAME, 'IsIdentity') as IsIdentity,
-NUMERIC_SCALE as Scale
-FROM INFORMATION_SCHEMA.columns
-WHERE TABLE_NAME = '#{table_name}'
-EOL
-
+        sql = "SELECT COLUMN_NAME as ColName, COLUMN_DEFAULT as DefaultValue, DATA_TYPE as ColType, COL_LENGTH('#{table_name}', COLUMN_NAME) as Length, COLUMNPROPERTY(OBJECT_ID('#{table_name}'), COLUMN_NAME, 'IsIdentity') as IsIdentity, NUMERIC_SCALE as Scale FROM INFORMATION_SCHEMA.columns WHERE TABLE_NAME = '#{table_name}'"
         result = nil
         # Uncomment if you want to have the Columns select statment logged.
-        # Personnally, I think it adds unneccessary SQL statement bloat to the log. 
+        # Personnally, I think it adds unneccessary bloat to the log. 
         # If you do uncomment, make sure to comment the "result" line that follows
         log(sql, name, @connection) { |conn| result = conn.select_all(sql) }
         #result = @connection.select_all(sql)
         columns = []
         result.each { |field| columns << ColumnWithIdentity.new(field[:ColName], field[:DefaultValue].to_s.gsub!(/[()\']/,"") =~ /null/ ? nil : field[:DefaultValue], "#{field[:ColType]}(#{field[:Length]})", field[:IsIdentity] == 1 ? true : false, field[:Scale]) }
-
         columns
       end
 
       def insert(sql, name = nil, pk = nil, id_value = nil)
         begin
           table_name = get_table_name(sql)
-
           col = get_identity_column(table_name)
-
           ii_enabled = false
 
           if col != nil
@@ -181,14 +206,12 @@ EOL
 
           log(sql, name, @connection) do |conn|
             conn.execute(sql)
-
             select_one("SELECT @@IDENTITY AS Ident")["Ident"]
           end
         ensure
           if ii_enabled
             begin
               execute enable_identity_insert(table_name, false)
-
             rescue Exception => e
               # Couldn't turn off IDENTITY_INSERT
             end
@@ -196,11 +219,24 @@ EOL
         end
       end
 
+      def execute(sql, name = nil)
+        if sql =~ /^INSERT/i
+          insert(sql, name)
+        elsif sql =~ /^UPDATE|DELETE/i
+          log(sql, name, @connection) do |conn|
+            conn.execute(sql)
+            retVal = select_one("SELECT @@ROWCOUNT AS AffectedRows")["AffectedRows"]
+          end
+        else
+          log(sql, name, @connection) do |conn|
+            conn.execute(sql)
+          end
+        end
+      end
+
       def update(sql, name = nil)
         execute(sql, name)
-        affected_rows(name)
       end
-
       alias_method :delete, :update
 
       def begin_db_transaction
@@ -236,8 +272,8 @@ EOL
               "'#{quote_string(value)}'"
             end
           when NilClass              then "NULL"
-          when TrueClass             then (column && column.type == :boolean ? "'t'" : "1")
-          when FalseClass            then (column && column.type == :boolean ? "'f'" : "0")
+          when TrueClass             then '1'
+          when FalseClass            then '0'
           when Float, Fixnum, Bignum then value.to_s
           when Date                  then "'#{value.to_s}'" 
           when Time, DateTime        then "'#{value.strftime("%Y-%m-%d %H:%M:%S")}'"
@@ -254,11 +290,12 @@ EOL
       end
 
       def add_limit_with_offset!(sql, limit, offset)
-        raise ArgumentError, 'add_limit_with_offset! not implemented'
+        order_by = sql.include?("ORDER BY") ? get_order_by(sql.sub(/.*ORDER\sBY./, "")) : nil
+        sql.gsub!(/SELECT/i, "SELECT * FROM ( SELECT TOP #{limit} * FROM ( SELECT TOP #{limit + offset}")<<" ) AS tmp1 ORDER BY #{order_by[1]} ) AS tmp2 ORDER BY #{order_by[0]}"
       end
 
       def add_limit_without_offset!(sql, limit)
-        raise ArgumentError, 'add_limit_without_offset! not implemented'
+        limit.nil? ? sql : sql.gsub!(/SELECT/i, "SELECT TOP #{limit}")
       end
 
       def recreate_database(name)
@@ -274,36 +311,18 @@ EOL
         execute "CREATE DATABASE #{name}"
       end
 
-      def execute(sql, name = nil)
-        if sql =~ /^INSERT/i
-          insert(sql, name)
-        else
-          log(sql, name, @connection) do |conn|
-            conn.execute(sql)
-          end
-        end
-      end
-
-      def adapter_name()
-        'SqlServer'
-      end
-
       private
         def select(sql, name = nil)
           rows = []
-
           log(sql, name, @connection) do |conn|
             conn.select_all(sql) do |row|
               record = {}
-
               row.column_names.each do |col|
                 record[col] = row[col]
               end
-
               rows << record
             end
           end
-
           rows
         end
 
@@ -346,10 +365,6 @@ EOL
           return sql =~ /[\(\.\,]\s*#{col}/
         end
 
-        def query_contains_text_column(sql, col)
-          
-        end
-
         def get_order_by(sql)
           return sql, sql.gsub(/\s*DESC\s*/, "").gsub(/\s*ASC\s*/, " DESC")
         end
@@ -358,15 +373,6 @@ EOL
           limit = limit.gsub!(/.OFFSET./i, ",").split(',')
           return limit[0].to_i, limit[0].to_i+limit[1].to_i
         end
-
-        def affected_rows(name = nil)
-          sql = "SELECT @@ROWCOUNT AS AffectedRows"
-          log(sql, name, @connection) do |conn|
-            conn.select_all(sql) do |row|
-              return row[:AffectedRows].to_i
-            end
-          end
-        end
     end
   end
 end

data/rakefile

@@ -8,9 +8,14 @@ require 'rake/contrib/rubyforgepublisher'
 
 PKG_BUILD     = ENV['PKG_BUILD'] ? '.' + ENV['PKG_BUILD'] : ''
 PKG_NAME      = 'activerecord'
-PKG_VERSION   = '1.9.0' + PKG_BUILD
+PKG_VERSION   = '1.9.1' + PKG_BUILD
 PKG_FILE_NAME = "#{PKG_NAME}-#{PKG_VERSION}"
 
+RELEASE_NAME  = "REL #{PKG_VERSION}"
+
+RUBY_FORGE_PROJECT = "activerecord"
+RUBY_FORGE_USER    = "webster132"
+
 PKG_FILES = FileList[
     "lib/**/*", "test/**/*", "examples/**/*", "doc/**/*", "[A-Z]*", "install.rb", "rakefile"
 ].exclude(/\bCVS\b|~$/)
@@ -33,47 +38,14 @@ Rake::TestTask.new("test_mysql_ruby") { |t|
   t.verbose = true
 }
 
-Rake::TestTask.new("test_postgresql") { |t|
-  t.libs << "test" << "test/connections/native_postgresql"
-  t.pattern = 'test/*_test.rb'
-  t.verbose = true
-}
-
-Rake::TestTask.new("test_sqlite") { |t|
-  t.libs << "test" << "test/connections/native_sqlite"
-  t.pattern = 'test/*_test.rb'
-  t.verbose = true
-}
-
-Rake::TestTask.new("test_sqlite3") { |t|
-  t.libs << "test" << "test/connections/native_sqlite3"
-  t.pattern = 'test/*_test.rb'
-  t.verbose = true
-}
-
-Rake::TestTask.new("test_sqlserver") { |t|
-  t.libs << "test" << "test/connections/native_sqlserver"
-  t.pattern = 'test/*_test.rb'
-  t.verbose = true
-}
-
-Rake::TestTask.new("test_db2") { |t|
-  t.libs << "test" << "test/connections/native_db2"
-  t.pattern = 'test/*_test.rb'
-  t.verbose = true
-}
-
-Rake::TestTask.new("test_oracle") { |t|
-  t.libs << "test" << "test/connections/native_oracle"
-  t.pattern = 'test/*_test.rb'
-  t.verbose = true
-}
+for adapter in %( postgresql sqlite sqlite3 sqlserver db2 oci )
+  Rake::TestTask.new("test_#{adapter}") { |t|
+    t.libs << "test" << "test/connections/native_#{adapter}"
+    t.pattern = 'test/*_test.rb'
+    t.verbose = true
+  }
+end
 
-Rake::TestTask.new("test_oci") { |t|
-  t.libs << "test" << "test/connections/native_oci"
-  t.pattern = 'test/*_test.rb'
-  t.verbose = true
-}
 
 # Generate the RDoc documentation
 
@@ -81,6 +53,7 @@ Rake::RDocTask.new { |rdoc|
   rdoc.rdoc_dir = 'doc'
   rdoc.title    = "Active Record -- Object-relation mapping put on rails"
   rdoc.options << '--line-numbers --inline-source --accessor cattr_accessor=object'
+  rdoc.template = "#{ENV['template']}.rb" if ENV['template']
   rdoc.rdoc_files.include('README', 'RUNNING_UNIT_TESTS', 'CHANGELOG')
   rdoc.rdoc_files.include('lib/**/*.rb')
   rdoc.rdoc_files.exclude('lib/active_record/vendor/*')
@@ -88,20 +61,6 @@ Rake::RDocTask.new { |rdoc|
 }
 
 
-# Publish beta gem
-desc "Publish the beta gem"
-task :pgem => [:package] do 
-  Rake::SshFilePublisher.new("davidhh@comox.textdrive.com", "public_html/gems/gems", "pkg", "#{PKG_FILE_NAME}.gem").upload
-  `ssh davidhh@comox.textdrive.com './gemupdate.sh'`
-end
-
-# Publish documentation
-desc "Publish the API documentation"
-task :pdoc => [:rdoc] do 
-  Rake::SshDirPublisher.new("davidhh@comox.textdrive.com", "public_html/ar", "doc").upload
-end
-
-
 # Create compressed packages
 
 dist_dirs = [ "lib", "test", "examples", "dev-utils" ]
@@ -117,7 +76,7 @@ spec = Gem::Specification.new do |s|
     s.files = s.files + Dir.glob( "#{dir}/**/*" ).delete_if { |item| item.include?( "\.svn" ) }
   end
   
-  s.add_dependency('activesupport', '= 1.0.2' + PKG_BUILD)
+  s.add_dependency('activesupport', '= 1.0.3' + PKG_BUILD)
 
   s.files.delete "test/fixtures/fixture_database.sqlite"
   s.files.delete "test/fixtures/fixture_database_2.sqlite"
@@ -143,20 +102,130 @@ Rake::GemPackageTask.new(spec) do |p|
 end
 
 
-task :lines do
-  lines = 0
-  codelines = 0
-  Dir.foreach("lib/active_record") { |file_name| 
-    next unless file_name =~ /.*rb/
-    
-    f = File.open("lib/active_record/" + file_name)
+# Publishing ------------------------------------------------------
 
-    while line = f.gets
-      lines += 1
-      next if line =~ /^\s*$/
-      next if line =~ /^\s*#/
-      codelines += 1
-    end
-  }
-  puts "Lines #{lines}, LOC #{codelines}"
+desc "Publish the beta gem"
+task :pgem => [:package] do 
+  Rake::SshFilePublisher.new("davidhh@comox.textdrive.com", "public_html/gems/gems", "pkg", "#{PKG_FILE_NAME}.gem").upload
+  `ssh davidhh@comox.textdrive.com './gemupdate.sh'`
+end
+
+desc "Publish the API documentation"
+task :pdoc => [:rdoc] do 
+  Rake::SshDirPublisher.new("davidhh@comox.textdrive.com", "public_html/ar", "doc").upload
 end
+
+desc "Publish the release files to RubyForge."
+task :release => [:package] do
+  files = ["gem", "tgz", "zip"].map { |ext| "pkg/#{PKG_FILE_NAME}.#{ext}" }
+
+  if RUBY_FORGE_PROJECT then
+    require 'net/http'
+    require 'open-uri'
+
+    project_uri = "http://rubyforge.org/projects/#{RUBY_FORGE_PROJECT}/"
+    project_data = open(project_uri) { |data| data.read }
+    group_id = project_data[/[?&]group_id=(\d+)/, 1]
+    raise "Couldn't get group id" unless group_id
+
+    # This echos password to shell which is a bit sucky
+    if ENV["RUBY_FORGE_PASSWORD"]
+      password = ENV["RUBY_FORGE_PASSWORD"]
+    else
+      print "#{RUBY_FORGE_USER}@rubyforge.org's password: "
+      password = STDIN.gets.chomp
+    end
+
+    login_response = Net::HTTP.start("rubyforge.org", 80) do |http|
+      data = [
+        "login=1",
+        "form_loginname=#{RUBY_FORGE_USER}",
+        "form_pw=#{password}"
+      ].join("&")
+      http.post("/account/login.php", data)
+    end
+
+    cookie = login_response["set-cookie"]
+    raise "Login failed" unless cookie
+    headers = { "Cookie" => cookie }
+
+    release_uri = "http://rubyforge.org/frs/admin/?group_id=#{group_id}"
+    release_data = open(release_uri, headers) { |data| data.read }
+    package_id = release_data[/[?&]package_id=(\d+)/, 1]
+    raise "Couldn't get package id" unless package_id
+
+    first_file = true
+    release_id = ""
+
+    files.each do |filename|
+      basename  = File.basename(filename)
+      file_ext  = File.extname(filename)
+      file_data = File.open(filename, "rb") { |file| file.read }
+
+      puts "Releasing #{basename}..."
+
+      release_response = Net::HTTP.start("rubyforge.org", 80) do |http|
+        release_date = Time.now.strftime("%Y-%m-%d %H:%M")
+        type_map = {
+          ".zip"    => "3000",
+          ".tgz"    => "3110",
+          ".gz"     => "3110",
+          ".gem"    => "1400"
+        }; type_map.default = "9999"
+        type = type_map[file_ext]
+        boundary = "rubyqMY6QN9bp6e4kS21H4y0zxcvoor"
+
+        query_hash = if first_file then
+          {
+            "group_id" => group_id,
+            "package_id" => package_id,
+            "release_name" => RELEASE_NAME,
+            "release_date" => release_date,
+            "type_id" => type,
+            "processor_id" => "8000", # Any
+            "release_notes" => "",
+            "release_changes" => "",
+            "preformatted" => "1",
+            "submit" => "1"
+          }
+        else
+          {
+            "group_id" => group_id,
+            "release_id" => release_id,
+            "package_id" => package_id,
+            "step2" => "1",
+            "type_id" => type,
+            "processor_id" => "8000", # Any
+            "submit" => "Add This File"
+          }
+        end
+
+        query = "?" + query_hash.map do |(name, value)|
+          [name, URI.encode(value)].join("=")
+        end.join("&")
+
+        data = [
+          "--" + boundary,
+          "Content-Disposition: form-data; name=\"userfile\"; filename=\"#{basename}\"",
+          "Content-Type: application/octet-stream",
+          "Content-Transfer-Encoding: binary",
+          "", file_data, ""
+          ].join("\x0D\x0A")
+
+        release_headers = headers.merge(
+          "Content-Type" => "multipart/form-data; boundary=#{boundary}"
+        )
+
+        target = first_file ? "/frs/admin/qrs.php" : "/frs/admin/editrelease.php"
+        http.post(target + query, data, release_headers)
+      end
+
+      if first_file then
+        release_id = release_response.body[/release_id=(\d+)/, 1]
+        raise("Couldn't get release id") unless release_id
+      end
+
+      first_file = false
+    end
+  end
+end

data/test/column_alias_test.rb

@@ -11,7 +11,7 @@ class TestColumnAlias < Test::Unit::TestCase
         assert_equal(records[0].keys[0], "pk")
       end
     else
-      records = topic.connection.select_all("SELECT id AS pk FROM topics LIMIT 1")
+      records = topic.connection.select_all("SELECT id AS pk FROM topics")
       assert_equal(records[0].keys[0], "pk")
     end
   end

data/test/connections/native_sqlserver/connection.rb

@@ -4,6 +4,9 @@ require 'logger'
 
 ActiveRecord::Base.logger = Logger.new("debug.log")
 
+db1 = 'activerecord_unittest'
+db2 = 'activerecord_unittest2'
+
 ActiveRecord::Base.establish_connection(
   :adapter  => "sqlserver",
   :host     => "localhost",

data/test/finder_test.rb

@@ -2,10 +2,11 @@ require 'abstract_unit'
 require 'fixtures/company'
 require 'fixtures/topic'
 require 'fixtures/entrant'
+require 'fixtures/developer'
 
 class FinderTest < Test::Unit::TestCase
   fixtures :companies, :topics, :entrants, :developers
-  
+
   def test_find
     assert_equal(@topics["first"]["title"], Topic.find(1).title)
   end
@@ -43,10 +44,10 @@ class FinderTest < Test::Unit::TestCase
   def test_find_all_with_prepared_limit_and_offset
     if ActiveRecord::ConnectionAdapters.const_defined? :OracleAdapter
       if ActiveRecord::Base.connection.instance_of?(ActiveRecord::ConnectionAdapters::OracleAdapter)
-        assert_raises(ArgumentError) { Entrant.find_all nil, "id ASC", ["? OFFSET ?", 2, 1] }
+        assert_raises(ArgumentError) { Entrant.find_all nil, "id ASC", [2, 1] }
       end
     else
-      entrants = Entrant.find_all nil, "id ASC", ["? OFFSET ?", 2, 1]
+      entrants = Entrant.find_all nil, "id ASC", [2, 1]
 
       assert_equal(2, entrants.size)
       assert_equal(@entrants["second"]["name"], entrants.first.name)
@@ -256,11 +257,11 @@ class FinderTest < Test::Unit::TestCase
     assert_equal first_five_developers, Developer.find_all(nil, 'id ASC', [5])
     assert_equal no_developers, Developer.find_all(nil, 'id ASC', [0])
   end
-  
+ 
   def test_find_all_with_limit_and_offset
     first_three_developers = Developer.find_all nil, 'id ASC', [3, 0]
     second_three_developers = Developer.find_all nil, 'id ASC', [3, 3]
-    last_two_developers = Developer.find_all nil, 'id ASC', [3, 8]
+    last_two_developers = Developer.find_all nil, 'id ASC', [2, 8]
     
     assert_equal 3, first_three_developers.length
     assert_equal 3, second_three_developers.length

data/test/fixtures/db_definitions/sqlserver.drop.sql

@@ -14,5 +14,4 @@ DROP TABLE colnametests;
 DROP TABLE mixins;
 DROP TABLE people;
 DROP TABLE binaries;
-DROP TABLE computers;
-
+DROP TABLE computers;

data/test/fixtures/db_definitions/sqlserver.sql

@@ -1,88 +1,79 @@
 CREATE TABLE accounts (
-  id int NOT NULL IDENTITY(1, 1),
+  id int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
   firm_id int default NULL,
-  credit_limit int default NULL,
-  PRIMARY KEY  (id)
+  credit_limit int default NULL
 )
 
 CREATE TABLE companies (
-  id int NOT NULL IDENTITY(1, 1),
+  id int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
   type varchar(50) default NULL,
   ruby_type varchar(50) default NULL,
   firm_id int default NULL,
   name varchar(50) default NULL,
   client_of int default NULL,
-  companies_count int default 0,
-  rating int default 1,
-  PRIMARY KEY  (id)
+  rating int default 1
 )
 
 CREATE TABLE topics (
-  id int NOT NULL IDENTITY(1, 1),
+  id int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
   title varchar(255) default NULL,
   author_name varchar(255) default NULL,
   author_email_address varchar(255) default NULL,
   written_on datetime default NULL,
+  bonus_time datetime default NULL,
   last_read datetime default NULL,
   content text,
   approved tinyint default 1,
   replies_count int default 0,
   parent_id int default NULL,
-  type varchar(50) default NULL,
-  PRIMARY KEY  (id)
+  type varchar(50) default NULL
 )
 
 CREATE TABLE developers (
-  id int NOT NULL IDENTITY(1, 1),
+  id int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
   name varchar(100) default NULL,
-  PRIMARY KEY  (id)
+  salary int default 70000
 );
 
 CREATE TABLE projects (
-  id int NOT NULL IDENTITY(1, 1),
-  name varchar(100) default NULL,
-  salary int default 70000,
-  PRIMARY KEY  (id)
+  id int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
+  name varchar(100) default NULL
 );
 
 CREATE TABLE developers_projects (
   developer_id int NOT NULL,
-  project_id int NOT NULL
+  project_id int NOT NULL,
+  joined_on datetime default NULL
 );
 
 CREATE TABLE customers (
-  id int NOT NULL IDENTITY(1, 1),
+  id int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
   name varchar(100) default NULL,
   balance int default 0,
   address_street varchar(100) default NULL,
   address_city varchar(100) default NULL,
   address_country varchar(100) default NULL,
-  gps_location varchar(100) default NULL,
-  PRIMARY KEY  (id)
+  gps_location varchar(100) default NULL
 );
 
 CREATE TABLE movies (
-  movieid int NOT NULL IDENTITY(1, 1),
-  name varchar(100) default NULL,
-   PRIMARY KEY  (movieid)
+  movieid int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
+  name varchar(100) default NULL
 );
 
 CREATE TABLE subscribers (
-  nick varchar(100) NOT NULL,
-  name varchar(100) default NULL,
-  PRIMARY KEY  (nick)
+  nick varchar(100) NOT NULL PRIMARY KEY,
+  name varchar(100) default NULL
 );
 
 CREATE TABLE booleantests (
-  id int NOT NULL IDENTITY(1, 1),
-  value integer default NULL,
-  PRIMARY KEY (id)
+  id int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
+  value bit default NULL
 );
 
 CREATE TABLE auto_id_tests (
-  auto_id int NOT NULL IDENTITY(1, 1),
-  value int default NULL,
-  PRIMARY KEY (auto_id)
+  auto_id int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
+  value int default NULL
 );
 
 CREATE TABLE entrants (
@@ -92,22 +83,20 @@ CREATE TABLE entrants (
 );
 
 CREATE TABLE colnametests (
-  id int NOT NULL IDENTITY(1, 1),
-  [references] int NOT NULL,
-  PRIMARY KEY (id)
+  id int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
+  [references] int NOT NULL
 );
 
 CREATE TABLE mixins (
-  id int NOT NULL IDENTITY(1, 1),
-  parent_id int default NULL,
-  type varchar(40) default NULL,  
+  id int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
+  parent_id int default NULL, 
   pos int default NULL,
-  lft int default NULL,
-  rgt int default NULL,
-  root_id int default NULL,      
   created_at datetime default NULL,
   updated_at datetime default NULL,
-  PRIMARY KEY (id)    
+  lft int default NULL,
+  rgt int default NULL,
+  root_id int default NULL,
+  type varchar(40) default NULL    
 );
 
 CREATE TABLE people (
@@ -118,14 +107,12 @@ CREATE TABLE people (
 );
 
 CREATE TABLE binaries (
-  id int NOT NULL IDENTITY(1, 1),
-  data image NULL,
-  PRIMARY KEY (id)
+  id int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
+  data image NULL
 );
 
 CREATE TABLE computers (
-  id int NOT NULL IDENTITY(1, 1),
-  developer int NOT NULL,
-  PRIMARY KEY (id)
+  id int NOT NULL IDENTITY(1, 1) PRIMARY KEY,
+  developer int NOT NULL
 );
 

metadata

@@ -3,8 +3,8 @@ rubygems_version: 0.8.8
 specification_version: 1
 name: activerecord
 version: !ruby/object:Gem::Version 
-  version: 1.9.0
-date: 2005-03-22
+  version: 1.9.1
+date: 2005-03-27
 summary: Implements the ActiveRecord pattern for ORM.
 require_paths: 
   - lib
@@ -142,8 +142,6 @@ files:
   - test/fixtures/developers_projects.yml
   - test/fixtures/entrant.rb
   - test/fixtures/entrants.yml
-  - test/fixtures/fixture_database.sqlite
-  - test/fixtures/fixture_database_2.sqlite
   - test/fixtures/migrations
   - test/fixtures/mixin.rb
   - test/fixtures/mixins.yml
@@ -227,5 +225,5 @@ dependencies:
         - 
           - "="
           - !ruby/object:Gem::Version 
-            version: 1.0.2
+            version: 1.0.3
       version: