activerecord-import 0.2.8.rc1 → 0.2.8.rc2

data/VERSION

@@ -1 +1 @@
-0.2.8.rc1
+0.2.8.rc2

data/lib/activerecord-import/adapters/abstract_adapter.rb

@@ -6,7 +6,7 @@ module ActiveRecord::Import::AbstractAdapter
     # Returns the sum of the sizes of the passed in objects. This should
     # probably be moved outside this class, but to where?
     def sum_sizes( *objects ) # :nodoc:
-      objects.inject( 0 ){|sum,o| sum += o.size }
+      objects.inject( 0 ){ |sum,o| sum += o.bytesize }
     end
 
     def get_insert_value_sets( values, sql_size, max_bytes ) # :nodoc:
@@ -16,12 +16,12 @@ module ActiveRecord::Import::AbstractAdapter
         comma_bytes = arr.size
         sql_size_thus_far = sql_size + current_size + val.size + comma_bytes
         if NO_MAX_PACKET == max_bytes or sql_size_thus_far <= max_bytes
-          current_size += val.size            
+          current_size += val.bytesize            
           arr << val
         else
           value_sets << arr
           arr = [ val ]
-          current_size = val.size
+          current_size = val.bytesize
         end
       
         # if we're on the last iteration push whatever we have in arr to value_sets

data/lib/activerecord-import/base.rb

@@ -23,6 +23,6 @@ end
 
 
 this_dir = Pathname.new File.dirname(__FILE__)
-require this_dir.join("import")
-require this_dir.join("active_record/adapters/abstract_adapter")
-require this_dir.join("synchronize")
+require this_dir.join("import").to_s
+require this_dir.join("active_record/adapters/abstract_adapter").to_s
+require this_dir.join("synchronize").to_s

data/test/support/mysql/import_examples.rb

@@ -1,3 +1,4 @@
+# encoding: UTF-8
 def should_support_mysql_import_functionality
 
   describe "building insert value sets" do
@@ -25,6 +26,24 @@ def should_support_mysql_import_functionality
       assert_equal values[1], value_sets[1].first
       assert_equal values[2], value_sets[2].first
     end
+    
+    context "data contains multi-byte chars" do
+      it "should properly build insert value set based on max packet allowed" do
+        # each accented e should be 2 bytes, so each entry is 6 bytes instead of 5
+        values = [
+          "('é')",
+          "('é')" ]
+          
+        adapter = ActiveRecord::Base.connection.class
+        base_sql_size_in_bytes = 15
+        max_bytes = 26
+        
+        values_size_in_bytes = adapter.sum_sizes( *values )            
+        value_sets = adapter.get_insert_value_sets( values, base_sql_size_in_bytes, max_bytes )
+        
+        assert_equal 2, value_sets.size, 'Two value sets were expected!'
+      end
+    end
   end
 
   describe "#import with :on_duplicate_key_update option (mysql specific functionality)" do

metadata

@@ -2,7 +2,7 @@
 name: activerecord-import
 version: !ruby/object:Gem::Version 
   prerelease: 6
-  version: 0.2.8.rc1
+  version: 0.2.8.rc2
 platform: ruby
 authors: 
 - Zach Dennis
@@ -123,7 +123,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: 3728361500197009317
+      hash: -763342084874399818
       segments: 
       - 0
       version: "0"