workbook 0.4.2 → 0.4.3

data/lib/workbook/readers/txt_reader.rb

@@ -4,9 +4,9 @@ module Workbook
     module TxtReader
       def load_txt text
         csv = text
-        parse_txt csv        
+        parse_txt csv
       end
-      
+
       def parse_txt csv_raw
         csv = []
         csv_raw.split("\n").each {|l| csv << csv_lib.parse_line(l,{:col_sep=>"\t"});nil}

data/lib/workbook/readers/xls_reader.rb

@@ -11,17 +11,17 @@ module Workbook
           template.add_raw sp
           parse_xls sp
         rescue Ole::Storage::FormatError => e
-          begin 
+          begin
             # Assuming it is a tab separated txt inside .xls
-            open(file_obj.path, 'txt') 
+            open(file_obj.path, 'txt')
           rescue
             raise e
           end
         end
-        
+
       end
-      
-      
+
+
       def parse_xls xls_spreadsheet=template.raws[Spreadsheet::Excel::Workbook], options={}
         options = {:additional_type_parsing=>true}.merge options
         number_of_worksheets = xls_spreadsheet.worksheets.count
@@ -29,16 +29,16 @@ module Workbook
           xls_sheet = xls_spreadsheet.worksheets[si]
           begin
             number_of_rows = xls_spreadsheet.worksheets[si].count
-            s = create_or_open_sheet_at(si)    
+            s = create_or_open_sheet_at(si)
             (0..number_of_rows-1).each do |ri|
               xls_row = xls_sheet.row(ri)
               r = s.table.create_or_open_row_at(ri)
               col_widths = xls_sheet.columns.collect{|c| c.width if c}
               xls_row.each_with_index do |xls_cell,ci|
-              
+
                 begin
-                  r[ci] = Workbook::Cell.new xls_cell    
-                  r[ci].parse!    
+                  r[ci] = Workbook::Cell.new xls_cell
+                  r[ci].parse!
                 rescue ArgumentError => e
                   if e.message.match('not a Spreadsheet::Formula')
                     v = xls_cell.value
@@ -48,27 +48,27 @@ module Workbook
                     end
                     if v.is_a? Spreadsheet::Excel::Error
                       v = "----!"
-                    end 
+                    end
                     r[ci] = Workbook::Cell.new v
                   elsif e.message.match('not a Spreadsheet::Link')
                     r[ci] = Workbook::Cell.new xls_cell.to_s
                   elsif e.message.match('not a Spreadsheet::Link')
                     r[ci] = Workbook::Cell.new xls_cell.to_s
                   elsif e.message.match('not a Spreadsheet::Excel::Error')
-                    r[ci] = "._."        
-                  else 
+                    r[ci] = "._."
+                  else
                     r[ci] = "._."  # raise e (we're going to be silent for now)
                   end
                 end
                 xls_format = xls_row.format(ci)
                 col_width = nil
-              
+
                 if ri == 0
                   col_width = col_widths[ci]
                 end
                 f = template.create_or_find_format_by "object_id_#{xls_format.object_id}",col_width
                 f[:width]= col_width
-                f[:rotation] = xls_format.rotation if xls_format.rotation 
+                f[:rotation] = xls_format.rotation if xls_format.rotation
                 f[:background_color] = xls_color_to_html_hex(xls_format.pattern_fg_color)
                 f[:number_format] = ms_formatting_to_strftime(xls_format.number_format)
                 f[:text_direction] = xls_format.text_direction
@@ -77,7 +77,7 @@ module Workbook
                 f[:color] = xls_color_to_html_hex(xls_format.font.color)
 
                 f.add_raw xls_format
-     
+
                 r[ci].format = f
               end
             end
@@ -87,13 +87,13 @@ module Workbook
           end
         end
       end
-      
-      private 
+
+      private
       def xls_color_to_html_hex color_sym
         Workbook::Book::XLS_COLORS[color_sym] ? Workbook::Book::XLS_COLORS[color_sym] : "#000000"
       end
-      
-      def ms_formatting_to_strftime ms_nr_format       
+
+      def ms_formatting_to_strftime ms_nr_format
         ms_nr_format = ms_nr_format.downcase
         return nil if ms_nr_format == 'general'
         ms_nr_format.gsub('yyyy','%Y').gsub('dddd','%A').gsub('mmmm','%B').gsub('ddd','%a').gsub('mmm','%b').gsub('yy','%y').gsub('dd','%d').gsub('mm','%m').gsub('y','%y').gsub('%%y','%y').gsub('d','%e').gsub('%%e','%d').gsub('m','%m').gsub('%%m','%m').gsub(';@','').gsub('\\','')

data/lib/workbook/readers/xls_shared.rb

@@ -3,74 +3,74 @@ module Workbook
   module Readers
     module XlsShared
       XLS_COLORS = {:xls_color_1=>'#000000',
-                    :xls_color_2=>'#FFFFFF',
-                    :xls_color_3=>'#FF0000',
-                    :xls_color_4=>'#00FF00',
-                    :xls_color_5=>'#0000FF',
-                    :xls_color_6=>'#FFFF00',
-                    :xls_color_7=>'#FF00FF',
-                    :xls_color_8=>'#00FFFF',
-                    :xls_color_9=>'#800000',
-                    :xls_color_10=>'#008000',
-                    :xls_color_11=>'#000080',
-                    :xls_color_12=>'#808000',
-                    :xls_color_13=>'#800080',
-                    :xls_color_14=>'#008080',
-                    :xls_color_15=>'#C0C0C0',
-                    :xls_color_16=>'#808080',
-                    :xls_color_17=>'#9999FF',
-                    :xls_color_18=>'#993366',
-                    :xls_color_19=>'#FFFFCC',
-                    :xls_color_20=>'#CCFFFF',
-                    :xls_color_21=>'#660066',
-                    :xls_color_22=>'#FF8080',
-                    :xls_color_23=>'#0066CC',
-                    :xls_color_24=>'#CCCCFF',
-                    :xls_color_25=>'#000080',
-                    :xls_color_26=>'#FF00FF',
-                    :xls_color_27=>'#FFFF00',
-                    :xls_color_28=>'#00FFFF',
-                    :xls_color_29=>'#800080',
-                    :xls_color_30=>'#800000',
-                    :xls_color_31=>'#008080',
-                    :xls_color_32=>'#0000FF',
-                    :xls_color_33=>'#00CCFF',
-                    :xls_color_34=>'#CCFFFF',
-                    :xls_color_35=>'#CCFFCC',
-                    :xls_color_36=>'#FFFF99',
-                    :xls_color_37=>'#99CCFF',
-                    :xls_color_38=>'#FF99CC',
-                    :xls_color_39=>'#CC99FF',
-                    :xls_color_40=>'#FFCC99',
-                    :xls_color_41=>'#3366FF',
-                    :xls_color_42=>'#33CCCC',
-                    :xls_color_43=>'#99CC00',
-                    :xls_color_44=>'#FFCC00',
-                    :xls_color_45=>'#FF9900',
-                    :xls_color_46=>'#FF6600',
-                    :xls_color_47=>'#666699',
-                    :xls_color_48=>'#969696',
-                    :xls_color_49=>'#003366',
-                    :xls_color_50=>'#339966',
-                    :xls_color_51=>'#003300',
-                    :xls_color_52=>'#333300',
-                    :xls_color_53=>'#993300',
-                    :xls_color_54=>'#993366',
-                    :xls_color_55=>'#333399',
-                    :xls_color_56=>'#333333',
-                    :black=>'#000000',
-                    :white=>'#FFFFFF',
-                    :red=>'#FF0000',
-                    :green=>'#00FF00',
-                    :blue=>'#0000FF',
-                    :yellow=>'#FFFF00',
-                    :magenta=>'#FF00FF',
-                    :cyan=>'#00FFFF',
-                    :border=>'#FFFFFF',
-                    :text=>'#000000',
-                    :lime=>'#00f94c'
-          }
-		end
-	end
+        :xls_color_2=>'#FFFFFF',
+        :xls_color_3=>'#FF0000',
+        :xls_color_4=>'#00FF00',
+        :xls_color_5=>'#0000FF',
+        :xls_color_6=>'#FFFF00',
+        :xls_color_7=>'#FF00FF',
+        :xls_color_8=>'#00FFFF',
+        :xls_color_9=>'#800000',
+        :xls_color_10=>'#008000',
+        :xls_color_11=>'#000080',
+        :xls_color_12=>'#808000',
+        :xls_color_13=>'#800080',
+        :xls_color_14=>'#008080',
+        :xls_color_15=>'#C0C0C0',
+        :xls_color_16=>'#808080',
+        :xls_color_17=>'#9999FF',
+        :xls_color_18=>'#993366',
+        :xls_color_19=>'#FFFFCC',
+        :xls_color_20=>'#CCFFFF',
+        :xls_color_21=>'#660066',
+        :xls_color_22=>'#FF8080',
+        :xls_color_23=>'#0066CC',
+        :xls_color_24=>'#CCCCFF',
+        :xls_color_25=>'#000080',
+        :xls_color_26=>'#FF00FF',
+        :xls_color_27=>'#FFFF00',
+        :xls_color_28=>'#00FFFF',
+        :xls_color_29=>'#800080',
+        :xls_color_30=>'#800000',
+        :xls_color_31=>'#008080',
+        :xls_color_32=>'#0000FF',
+        :xls_color_33=>'#00CCFF',
+        :xls_color_34=>'#CCFFFF',
+        :xls_color_35=>'#CCFFCC',
+        :xls_color_36=>'#FFFF99',
+        :xls_color_37=>'#99CCFF',
+        :xls_color_38=>'#FF99CC',
+        :xls_color_39=>'#CC99FF',
+        :xls_color_40=>'#FFCC99',
+        :xls_color_41=>'#3366FF',
+        :xls_color_42=>'#33CCCC',
+        :xls_color_43=>'#99CC00',
+        :xls_color_44=>'#FFCC00',
+        :xls_color_45=>'#FF9900',
+        :xls_color_46=>'#FF6600',
+        :xls_color_47=>'#666699',
+        :xls_color_48=>'#969696',
+        :xls_color_49=>'#003366',
+        :xls_color_50=>'#339966',
+        :xls_color_51=>'#003300',
+        :xls_color_52=>'#333300',
+        :xls_color_53=>'#993300',
+        :xls_color_54=>'#993366',
+        :xls_color_55=>'#333399',
+        :xls_color_56=>'#333333',
+        :black=>'#000000',
+        :white=>'#FFFFFF',
+        :red=>'#FF0000',
+        :green=>'#00FF00',
+        :blue=>'#0000FF',
+        :yellow=>'#FFFF00',
+        :magenta=>'#FF00FF',
+        :cyan=>'#00FFFF',
+        :border=>'#FFFFFF',
+        :text=>'#000000',
+        :lime=>'#00f94c'
+      }
+    end
+  end
 end
-		    
+

data/lib/workbook/readers/xlsx_reader.rb

@@ -4,7 +4,7 @@ require 'rubyXL'
 # Monkeypatching rubyXL, pull request submitted: https://github.com/gilt/rubyXL/pull/47
 module RubyXL
   class Workbook
-    
+
     # Improves upon date format detection
     def is_date_format?(num_fmt)
       num_fmt.downcase!
@@ -71,36 +71,36 @@ module RubyXL
     def num_fmts_by_id
       return @num_fmts_hash unless @num_fmts_hash.nil?
       @num_fmts_hash={1=>{:attributes=>{:formatCode=>'0'}},
-2=>{:attributes=>{:formatCode=>'0.00'}},
-3=>{:attributes=>{:formatCode=>'#, ##0'}},
-4=>{:attributes=>{:formatCode=>'#, ##0.00'}},
-5=>{:attributes=>{:formatCode=>'$#, ##0_);($#, ##0)'}},
-6=>{:attributes=>{:formatCode=>'$#, ##0_);[Red]($#, ##0)'}},
-7=>{:attributes=>{:formatCode=>'$#, ##0.00_);($#, ##0.00)'}},
-8=>{:attributes=>{:formatCode=>'$#, ##0.00_);[Red]($#, ##0.00)'}},
-9=>{:attributes=>{:formatCode=>'0%'}},
-10=>{:attributes=>{:formatCode=>'0.00%'}},
-11=>{:attributes=>{:formatCode=>'0.00E+00'}},
-12=>{:attributes=>{:formatCode=>'# ?/?'}},
-13=>{:attributes=>{:formatCode=>'# ??/??'}},
-14=>{:attributes=>{:formatCode=>'m/d/yyyy'}},
-15=>{:attributes=>{:formatCode=>'d-mmm-yy'}},
-16=>{:attributes=>{:formatCode=>'d-mmm'}},
-17=>{:attributes=>{:formatCode=>'mmm-yy'}},
-18=>{:attributes=>{:formatCode=>'h:mm AM/PM'}},
-19=>{:attributes=>{:formatCode=>'h:mm:ss AM/PM'}},
-20=>{:attributes=>{:formatCode=>'h:mm'}},
-21=>{:attributes=>{:formatCode=>'h:mm:ss'}},
-22=>{:attributes=>{:formatCode=>'m/d/yyyy h:mm'}},
-37=>{:attributes=>{:formatCode=>'#, ##0_);(#, ##0)'}},
-38=>{:attributes=>{:formatCode=>'#, ##0_);[Red](#, ##0)'}},
-39=>{:attributes=>{:formatCode=>'#, ##0.00_);(#, ##0.00)'}},
-40=>{:attributes=>{:formatCode=>'#, ##0.00_);[Red](#, ##0.00)'}},
-45=>{:attributes=>{:formatCode=>'mm:ss'}},
-46=>{:attributes=>{:formatCode=>'[h]:mm:ss'}},
-47=>{:attributes=>{:formatCode=>'mm:ss.0'}},
-48=>{:attributes=>{:formatCode=>'##0.0E+0'}},
-49=>{:attributes=>{:formatCode=>'@'}}}
+        2=>{:attributes=>{:formatCode=>'0.00'}},
+        3=>{:attributes=>{:formatCode=>'#, ##0'}},
+        4=>{:attributes=>{:formatCode=>'#, ##0.00'}},
+        5=>{:attributes=>{:formatCode=>'$#, ##0_);($#, ##0)'}},
+        6=>{:attributes=>{:formatCode=>'$#, ##0_);[Red]($#, ##0)'}},
+        7=>{:attributes=>{:formatCode=>'$#, ##0.00_);($#, ##0.00)'}},
+        8=>{:attributes=>{:formatCode=>'$#, ##0.00_);[Red]($#, ##0.00)'}},
+        9=>{:attributes=>{:formatCode=>'0%'}},
+        10=>{:attributes=>{:formatCode=>'0.00%'}},
+        11=>{:attributes=>{:formatCode=>'0.00E+00'}},
+        12=>{:attributes=>{:formatCode=>'# ?/?'}},
+        13=>{:attributes=>{:formatCode=>'# ??/??'}},
+        14=>{:attributes=>{:formatCode=>'m/d/yyyy'}},
+        15=>{:attributes=>{:formatCode=>'d-mmm-yy'}},
+        16=>{:attributes=>{:formatCode=>'d-mmm'}},
+        17=>{:attributes=>{:formatCode=>'mmm-yy'}},
+        18=>{:attributes=>{:formatCode=>'h:mm AM/PM'}},
+        19=>{:attributes=>{:formatCode=>'h:mm:ss AM/PM'}},
+        20=>{:attributes=>{:formatCode=>'h:mm'}},
+        21=>{:attributes=>{:formatCode=>'h:mm:ss'}},
+        22=>{:attributes=>{:formatCode=>'m/d/yyyy h:mm'}},
+        37=>{:attributes=>{:formatCode=>'#, ##0_);(#, ##0)'}},
+        38=>{:attributes=>{:formatCode=>'#, ##0_);[Red](#, ##0)'}},
+        39=>{:attributes=>{:formatCode=>'#, ##0.00_);(#, ##0.00)'}},
+        40=>{:attributes=>{:formatCode=>'#, ##0.00_);[Red](#, ##0.00)'}},
+        45=>{:attributes=>{:formatCode=>'mm:ss'}},
+        46=>{:attributes=>{:formatCode=>'[h]:mm:ss'}},
+        47=>{:attributes=>{:formatCode=>'mm:ss.0'}},
+        48=>{:attributes=>{:formatCode=>'##0.0E+0'}},
+      49=>{:attributes=>{:formatCode=>'@'}}}
       if num_fmts and num_fmts[:numFmt]
         num_fmts[:numFmt].each do |num_fmt|
           @num_fmts_hash[num_fmt[:attributes][:numFmtId]]=num_fmt
@@ -125,7 +125,7 @@ module RubyXL
     end
   end
 end
-# end of monkey patch 
+# end of monkey patch
 
 module Workbook
   module Readers
@@ -136,25 +136,25 @@ module Workbook
         template.add_raw sp
         parse_xlsx sp
       end
-      
+
       def parse_xlsx xlsx_spreadsheet=template.raws[RubyXL::Workbook], options={}
         options = {:additional_type_parsing=>false}.merge options
         #number_of_worksheets = xlsx_spreadsheet.worksheets.count
         xlsx_spreadsheet.worksheets.each_with_index do |worksheet, si|
-          s = create_or_open_sheet_at(si)    
+          s = create_or_open_sheet_at(si)
           col_widths = xlsx_spreadsheet.worksheets.first.cols.collect{|a| a[:attributes][:width].to_f if a[:attributes]}
           worksheet.each_with_index do |row, ri|
             r = s.table.create_or_open_row_at(ri)
-            
+
             row.each_with_index do |cell,ci|
               if cell.nil?
                 r[ci] = Workbook::Cell.new nil
               else
-                r[ci] = Workbook::Cell.new cell.value        
-                r[ci].parse!    
+                r[ci] = Workbook::Cell.new cell.value
+                r[ci].parse!
                 xls_format = cell.style_index
                 col_width = nil
-              
+
                 if ri == 0
                   col_width = col_widths[ci]
                 end
@@ -166,16 +166,16 @@ module Workbook
                 f[:color] = "##{cell.font_color}"
 
                 f.add_raw xls_format
-     
+
                 r[ci].format = f
               end
             end
           end
         end
       end
-    private 
-      def ms_formatting_to_strftime ms_nr_format   
-        if ms_nr_format    
+      private
+      def ms_formatting_to_strftime ms_nr_format
+        if ms_nr_format
           ms_nr_format = ms_nr_format.downcase
           return nil if ms_nr_format == 'general'
           ms_nr_format.gsub('yyyy','%Y').gsub('dddd','%A').gsub('mmmm','%B').gsub('ddd','%a').gsub('mmm','%b').gsub('yy','%y').gsub('dd','%d').gsub('mm','%m').gsub('y','%y').gsub('%%y','%y').gsub('d','%e').gsub('%%e','%d').gsub('m','%m').gsub('%%m','%m').gsub(';@','').gsub('\\','')

data/lib/workbook/row.rb

@@ -4,17 +4,17 @@ module Workbook
     alias_method :compare_without_header, :<=>
     attr_accessor :placeholder     # The placeholder attribute is used in compares (corresponds to newly created or removed lines (depending which side you're on)
     attr_accessor :format
-    
+
     # Initialize a new row
     #
     # @param [Workbook::Row, Array<Workbook::Cell>, Array] cells list of cells to initialize the row with, default is empty
     # @param [Workbook::Table] table a row normally belongs to a table, reference it here
     # @param [Hash] options  Supprted options: parse_cells_on_batch_creation (parse cell values during row-initalization, default: false), cell_parse_options (default {}, see Workbook::Modules::TypeParser)
     def initialize cells=[], table=nil, options={}
-      options=options ? {:parse_cells_on_batch_creation=>false,:cell_parse_options=>{}}.merge(options) : {} 
+      options=options ? {:parse_cells_on_batch_creation=>false,:cell_parse_options=>{}}.merge(options) : {}
       cells = [] if cells==nil
       self.table= table
-      cells.each do |c| 
+      cells.each do |c|
         if c.is_a? Workbook::Cell
           c = c
         else
@@ -24,28 +24,28 @@ module Workbook
         push c
       end
     end
-    
+
     # An internal function used in diffs
-    # 
-    # @return [Boolean] returns true when this row is not an actual row, but a placeholder row to 'compare' against 
+    #
+    # @return [Boolean] returns true when this row is not an actual row, but a placeholder row to 'compare' against
     def placeholder?
       placeholder ? true : false
     end
-    
+
     # Returns the table this row belongs to
     #
     # @return [Workbook::Table] the table this row belongs to
     def table
       @table
     end
-    
+
     # Set reference to the table this row belongs to without adding the row to the table
     #
     # @param [Workbook::Table] t the table this row belongs to
     def set_table(t)
       @table = t
     end
-    
+
     # Set reference to the table this row belongs to and add the row to this table
     #
     # @param [Workbook::Table] t the table this row belongs to
@@ -56,7 +56,7 @@ module Workbook
         table.push(self) #unless table.index(self) and self.placeholder?
       end
     end
-    
+
     # Overrides normal Array's []-function with support for symbols that identify a column based on the header-values
     #
     # @example Lookup using fixnum or header value encoded as symbol
@@ -73,7 +73,7 @@ module Workbook
         rescue NoMethodError
         end
         return rv
-      else 
+      else
         if index_or_hash
           return to_a[index_or_hash]
         end
@@ -87,27 +87,27 @@ module Workbook
     #   row[:a] #=> <Cell value="a">
     #
     # @param [Fixnum, Symbol] index_or_hash
-    # @param [String, Fixnum, NilClass, Date, DateTime, Time, Float] value 
+    # @param [String, Fixnum, NilClass, Date, DateTime, Time, Float] value
     # @return [Workbook::Cell, nil]
     def []= (index_or_hash, value)
       index = index_or_hash
       if index_or_hash.is_a? Symbol
         index = table_header_keys.index(index_or_hash)
       end
-    
+
       value_celled = Workbook::Cell.new
       if value.is_a? Workbook::Cell
         value_celled = value
       else
         current_cell = self[index]
         if current_cell.is_a? Workbook::Cell
-          value_celled = current_cell 
+          value_celled = current_cell
         end
         value_celled.value=(value)
       end
       super(index,value_celled)
     end
-        
+
     # Returns an array of cells allows you to find cells by a given color, normally a string containing a hex
     #
     # @param [String] color  a CSS-style hex-string
@@ -119,44 +119,44 @@ module Workbook
       r = Row.new cells
       options[:hash_keys] ? r.to_symbols : r
     end
-    
-    # Returns true when the row belongs to a table and it is the header row (typically the first row) 
+
+    # Returns true when the row belongs to a table and it is the header row (typically the first row)
     #
     # @return [Boolean]
     def header?
       table != nil and self.object_id == table.header.object_id
     end
-    
+
     # Is this the first row in the table
-    # 
+    #
     # @return [Boolean, NilClass] returns nil if it doesn't belong to a table, false when it isn't the first row of a table and true when it is.
     def first?
       table != nil and self.object_id == table.first.object_id
     end
-    
+
     # Returns true when all the cells in the row have values whose to_s value equals an empty string
     #
     # @return [Boolean]
     def no_values?
       all? {|c| c.value.to_s == ''}
     end
-    
+
     # Converts a row to an array of symbol representations of the row content, see also: Workbook::Cell#to_sym
     # @return [Array<Symbol>] returns row as an array of symbols
     def to_symbols
       collect{|c| c.to_sym}
     end
-    
+
     # Converts the row to an array of Workbook::Cell's
     # @return [Array<Workbook::Cell>] returns row as an array of symbols
     def to_a
       self.collect{|c| c}
     end
-    
+
     def table_header_keys
       table.header.to_symbols
     end
-    
+
     # Returns a hash representation of this row
     #
     # @return [Hash]
@@ -168,36 +168,66 @@ module Workbook
       keys.each_with_index {|k,i| @hash[k]=values[i]}
       return @hash
     end
-    
+
     # Compares one row wiht another
     #
     # @param [Workbook::Row] other row to compare against
-    # @return [Workbook::Row] a row with the diff result. 
+    # @return [Workbook::Row] a row with the diff result.
     def <=> other
       a = self.header? ? 0 : 1
       b = other.header? ? 0 : 1
       return (a <=> b) if (a==0 or b==0)
       compare_without_header other
     end
-    
+
     # The first cell of the row is considered to be the key
     #
     # @return [Workbook::Cell] the key cell
     def key
       first
     end
-    
+
     # Compact detaches the row from the table
     def compact
       r = self.clone
       r = r.collect{|c| c unless c.nil?}.compact
     end
-    
+
     # clone the row with together with the cells
     #
     # @return [Workbook::Row] a cloned copy of self with cells
     def clone
       Workbook::Row.new(to_a.collect{|c| c.clone})
     end
+    
+    # remove all the trailing nil-cells (returning a trimmed clone)
+    #
+    # @param [Integer] desired_length of the new row
+    # @return [Workbook::Row] a trimmed clone of the array
+    def trim(desired_length=nil)
+      self.clone.trim!(desired_length)
+    end
+    
+    # remove all the trailing nil-cells (returning a trimmed self)
+    #
+    # @param [Integer] desired_length of the new row
+    # @return [Workbook::Row] self
+    def trim!(desired_length=nil)
+      self_count = self.count-1
+      self.count.times do |index| 
+        index = self_count - index
+        if desired_length and index < desired_length
+          break
+        elsif desired_length and index >= desired_length
+          self.delete_at(index)
+        elsif self[index].nil?
+          self.delete_at(index)
+        else
+          break
+        end
+      end
+      (desired_length - self.count).times{|a| self << (Workbook::Cell.new(nil))} if desired_length and (desired_length - self.count) > 0
+      self 
+    end
   end
 end