Fixed bug with empty cells

.gitignore

@@ -0,0 +1,2 @@
+pkg/
+pgk/*

README.rdoc

@@ -34,6 +34,11 @@ For use with ActiveRecord objects you can provide an argument to call save on ev
 
   products = importer.object(true)
 
+The attributes of ActiveRecord objects are assigned using update_attributes for increased security. 
+
+Beware, that if the model is not an ActiveRecord model mass assigning attributes by CSV column name might impose a security risk if you don't know the exact names of the CSV columns. 
+
+In a next release a stripped down version of active records "attr_accessible" will be bundled with the gem.
 
 == REQUIREMENTS:
 

lib/csv_importer/csv_importer.rb

@@ -1,5 +1,5 @@
 module CSVImporter
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
   require 'csv'
 
 
@@ -19,14 +19,21 @@ def initialize(str_or_readable, klass=nil, dictionary={})
     def objects(save=false)
       return @objects if !@objects.nil?
       @objects = Array.new if @object.nil?
+
       first = true
       @reader.each do |row|
         #TODO: Ruby 1.9 has header option would be nicer instead of this hack.
         if !first
           object = @klass.new
+          active_record = object.respond_to?(:update_attributes)
+          attribute_hash = {} if active_record
           @columns.each_pair do |column, row_number|
-            object.send("#{column}=", row[row_number].strip)
+            value = row[row_number]
+            value = value.strip if !value.nil?
+            object.send("#{column}=", value) if !active_record
+            attribute_hash[column] = value if active_record
           end
+          object.attributes = attribute_hash if active_record
           object.save if save
           @objects << object
         else

spec/csv/sample.csv

@@ -1,6 +1,14 @@
 Description;Brand;SellPrice;SalePrice;Image;Category;ArticleCode
-SINGLE JERSEY L.S R;PME Jeans;  59.95;  59.95;http://www.example.com/images/JTS91573.JPG;T-shirts;JTS91573
-MONTANA L.SL. GRAND;PME legend;  59.95;  59.95;http://www.example.com/images/PTS88510.JPG;T-shirts;PTS88510
-CO DOUBLE JERSEY, 1;Cast Iron;  29.95;  29.95;http://www.example.com/images/CAC91101.JPG;Belts/ Keyholders;CAC91101
-PLAYER L.SLV. VEST;Cast Iron;  99.95;  99.95;http://www.example.com/images/CKW91406.JPG;Knitwear;CKW91406
-SLUBOR L.SL POLO;Cast Iron;  79.95;  79.95;http://www.example.com/images/CPS91301.JPG;Polo l.sl;CPS91301
+SINGLE JERSEY L.S R;PME Jeans;  59.95;  59.95;http://www.softwear.nl/ism/bouchier/images/JTS91573.JPG;T-shirts;JTS91573
+MONTANA L.SL. GRAND;PME legend;  59.95;  59.95;http://www.softwear.nl/ism/bouchier/images/PTS88510.JPG;T-shirts;PTS88510
+CO DOUBLE JERSEY, 1;Cast Iron;  29.95;  29.95;http://www.softwear.nl/ism/bouchier/images/CAC91101.JPG;Belts/ Keyholders;CAC91101
+PLAYER L.SLV. VEST;Cast Iron;  99.95;  99.95;http://www.softwear.nl/ism/bouchier/images/CKW91406.JPG;Knitwear;CKW91406
+SLUBOR L.SL POLO;Cast Iron;  79.95;  79.95;http://www.softwear.nl/ism/bouchier/images/CPS91301.JPG;Polo l.sl;CPS91301
+BEAR CLAW FLOWER -;Cast Iron;  89.95;  89.95;http://www.softwear.nl/ism/bouchier/images/CSI91601.JPG;Shirts;CSI91601
+KHENSU LSE - MULTI;Cast Iron; 129.95; 129.95;http://www.softwear.nl/ism/bouchier/images/CTR91202-960.JPG;Pants;CTR91202-960
+NYLON FLIGHT BOMBER;PME legend; 179.95; 179.95;http://www.softwear.nl/ism/bouchier/images/JA86112.JPG;Jackets;JA86112
+RATON LONG JACKET;Vanguard; 239.95; 239.95;http://www.softwear.nl/ism/bouchier/images/VJA86149.JPG;Jackets;VJA86149
+OXFORD PANT, COTTON;Vanguard;  89.95;  89.95;http://www.softwear.nl/ism/bouchier/images/VTR86422-739.JPG;Pants;VTR86422-739
+MERIDIAN S.SL R-NEC;Vanguard;  49.95;  49.95;http://www.softwear.nl/ism/bouchier/images/VTSS86301.JPG;T-shirts;VTSS86301
+;PME legend;  24.95;  24.95;http://www.softwear.nl/ism/bouchier/images/PAC91002.JPG;Belts/ Keyholders;PAC91002
+COTTON PLAITED V-NE;PME legend;  89.95;  89.95;http://www.softwear.nl/ism/bouchier/images/PKW91339.JPG;Knitwear;PKW91339