rspreadsheet 0.4.9 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: e7334a14b9121a8cfefe62a271f865b0be259f32
-  data.tar.gz: b76797efc38071ebc4d2120eb051e220f45e3657
+SHA256:
+  metadata.gz: 345981780a0e704579af26d413e53cb49dc671fbdcef278ae1fbdcbccf2bcbdb
+  data.tar.gz: 24b298d8ccde36019e3fa06f5acfdc8e9370e909c5b621663b6b23ca7bc438fd
 SHA512:
-  metadata.gz: 456b8b7240c21d4bcc6c4faa82775c46310517e26e7051025ddfb852bceff1375860d7c2b350e36fd92c183ba8019228d8e7ba2db7319d581a61f95b91118118
-  data.tar.gz: a28c423ecb3a1e9b8644a7dfcf6c6f063a817d9a710edb80388a050e0738b0dd421783ab940987023725223af8d618df3753537b89c98478c169cf3da78e1f14
+  metadata.gz: 41fbd52d0c125cd5e4c6555b1de2dfc0360775986b73aff3e7eb947c17dfa03c0dbe85dadded72a00561e98ad40dca9e3052b2d42b2ea7746d818351e98933c1
+  data.tar.gz: 120436ad22dfd37afada167cb837172ebfe393940f58da85b5e6acfed127e275957f202a3a9e4e5ac5df7848baba3aa5554c11ddefa1b119960657ce6ef88805

data/GUIDE.md

@@ -4,6 +4,7 @@
 You can open ODS file (OpenDocument Spreadsheet) like this
 ````ruby
 workbook = Rspreadsheet.open('./test.ods')
+workbook = Rspreadsheet.open('./test.fods') # gem supports flast OpenDocument format
 ````
 and access its first sheet like this
 ````ruby
@@ -37,9 +38,24 @@ i.move_to('100mm','99.98mm')
 The file needs to be saved after doing changes. 
 ````ruby
 workbook.save
-workbook.save('new_filename.ods')   # changes filename and saves
-workbook.save(any_io_object)        # file can be saved to any IO like object as well
+workbook.save('new_filename.ods')    # changes filename and saves
+workbook.save(any_io_object)         # file can be saved to any IO like object as well
+workbook.to_io                       # coverts it to IO object which can be used to 
+anotherIO.write(workbook.to_io.read) # send file over internet without saving it first
 ````
+
+### Creating fresh new file
+You may name the spreadsheet on creation or at first save.
+
+````ruby
+workbook = Rspreadsheet.new
+workbook.save('./filename.ods')      # filename nust be provided at least on first save
+workbook2 = Rspreadsheet.new('./filename2.fods', format: :flat)
+workbook2.save                       
+```
+
+If you want to use the fods flat format, you must create it as such.
+
 ### Date and Time
 OpenDocument and ruby have different models of date, time and datetime. Ruby containg three different objects. Time and DateTime cover all cases, Date covers dates only. OpenDocument distinguishes two groups - time of a day (time) and everything else (date). To simplify things a little we return cell values containg time of day as Time object and cell values containg datetime of date as DateTime. I am aware that this is very arbitrary choice, but it is very practical. This way and to some extend the types of values from OpenDocument are preserved when read from files, beeing acted upon and written back to spreadshhet.
 

data/README.md

@@ -44,8 +44,9 @@ book.save
 book.save('different_filename.ods')
 ```
 
+  * Detailed [Guide to using Rspreadsheet](GUIDE.md) is available. 
   * [More examples](https://gist.github.com/gorn/b432e6a69e82628349e6) of lots of alternative syntax **you can leave you comments and suggestions there**
-  * [Guide to using Rspreadsheet](GUIDE.md) some other notes
+
 
 ## Installation and Configuration
 
@@ -63,15 +64,7 @@ Or install it yourself as:
 
     $ gem install rspreadsheet
 
-If you get this error concering libxml, like
-
-    checking for libxml/xmlversion.h in /opt/include/libxml2,/opt/local/include/libxml2,/usr/local/include/libxml2,/usr/include/libxml2... no
-    *** extconf.rb failed ***
-or 
-
-    mkmf.rb can't find header files for ruby at /usr/lib/ruby/include/ruby.h 
-        
-then you might not have installed libxml for ruby. I.e. in debian something like <code>sudo aptitude install ruby-libxml</code> or using equivalent command in other package managers.
+If you get any error, have a look at [troubleshooting](TROUBLESHOOTING.md)
 
 ## Contibutions, ideas and wishes welcomed
 

data/TROUBLESHOOTING.md

@@ -0,0 +1,25 @@
+# Troubleshooting
+
+## Installation
+
+### libxml
+
+The gem detects natively installed libxml and uses instead of installing new one from rubygems. There might be some errors connected to this. If you get this error concering libxml, like
+
+    checking for libxml/xmlversion.h in /opt/include/libxml2,/opt/local/include/libxml2,/usr/local/include/libxml2,/usr/include/libxml2... no
+    *** extconf.rb failed ***
+or 
+
+    mkmf.rb can't find header files for ruby at /usr/lib/ruby/include/ruby.h 
+        
+then you might not have installed libxml for ruby. I.e. in debian something like 
+
+    sudo aptitude install ruby-libxml
+    
+or using equivalent command in other package managers.
+
+### if you do NOT want to use netively installed libxml
+
+If you want to use libxml from rubygems, than uninstall the native one or add this line to your Gemfile
+
+    'libxml-ruby', '3.0' 

data/lib/helpers/class_extensions.rb

@@ -14,7 +14,7 @@ if RUBY_VERSION > '2.1'
     
     refine LibXML::XML::Node do
       def equals?(node2); raise 'nic' end
-      def ==(node2)  
+      def ===(node2)  
         self.simplification_of?(node2) and node2.simplification_of?(self)
       end
       # if node2 contains at least all that I do
@@ -31,15 +31,15 @@ if RUBY_VERSION > '2.1'
           return "#{where}> Attribute #{attr} have diffent values: #{attr.value} != #{node2.attributes[attr.name]}" unless node2.attributes[attr.name] == attr.value
         end
 
-        elems1 = self.elements
-        elems2 = node2.elements
-        return "#{where}> elements have different number of subelements #{elems1.length} !=  #{elems2.length}" if (elems1.length != elems2.length)
+        elems1 = self.children
+        elems2 = node2.children
+        return "#{where}> elements have different number of children #{elems1.length} !=  #{elems2.length}" if (elems1.length != elems2.length)
         
         elems1.each_index do |i|
           raise "Nil for i=#{i}" if elems1[i].nil?
           if (elems1[i].node_type_name == 'text') 
             if elems2[i].nil? || (elems1[i].to_s) != (elems2[i].to_s)
-              return  "#{where}> #{i+1}th text subelements are different: #{elems1[i].to_s} != #{elems2[i].to_s}"
+              return  "#{where}> #{i+1}th text children are different: #{elems1[i].to_s} != #{elems2[i].to_s}"
             end
           elsif (elems1[i].node_type_name == 'element') 
             unless elems1[i].simplification_of?(elems2[i])
@@ -72,7 +72,7 @@ else # Monkeypatching
   end
 
   class LibXML::XML::Node
-    def ==(node2)  
+    def ===(node2)  
       self.simplification_of?(node2) and node2.simplification_of?(self)
     end
     # if node2 contains at least all that I do
@@ -88,15 +88,21 @@ else # Monkeypatching
       self.attributes.each do |attr|
         return "#{where}> Attribute #{attr} have diffent values: #{attr.value} != #{node2.attributes[attr.name]}" unless node2.attributes[attr.name] == attr.value
       end
+
+      elems1 = self.children
+      elems2 = node2.children
+      return "#{where}> elements have different number of children #{elems1.length} !=  #{elems2.length}" if (elems1.length != elems2.length)
       
-      elems1 = self.elements
-      elems2 = node2.elements
-  #     return "#{where}> elements have different number of subelements #{elems1.length} !=  #{elems2.length}" if (elems1.length != elems2.length) 
-      elems1.length.times do |i|
-        if (elems1[i].node_type_name == 'text') && ((elems1[i].to_s != elems2[i].to_s) )
-          return  "#{where}> #{i+1}th text subelements are different: #{elems1[i].to_s} != #{elems2[i].to_s}"
-        elsif (elems1[i].node_type_name == 'element') && (!elems1[i].simplification_of?(elems2[i]))
-          return "#{where}/[#{i+1}]#{elems1[i].first_diff(elems2[i])}"
+      elems1.each_index do |i|
+        raise "Nil for i=#{i}" if elems1[i].nil?
+        if (elems1[i].node_type_name == 'text') 
+          if elems2[i].nil? || (elems1[i].to_s) != (elems2[i].to_s)
+            return  "#{where}> #{i+1}th text children are different: #{elems1[i].to_s} != #{elems2[i].to_s}"
+          end
+        elsif (elems1[i].node_type_name == 'element') 
+          unless elems1[i].simplification_of?(elems2[i])
+            return "#{where}/[#{i+1}]#{elems1[i].first_diff(elems2[i])}"
+          end
         end
       end
         
@@ -113,4 +119,4 @@ else # Monkeypatching
 #     end
   end
 
-end
+end

data/lib/rspreadsheet.rb

@@ -7,13 +7,36 @@ require 'helpers/class_extensions'
 
 module Rspreadsheet
   extend Configuration
+
   define_setting :raise_on_negative_coordinates, true
 
   # makes creating new workbooks as easy as `Rspreadsheet.new` or `Rspreadsheet.open('filename.ods')
-  def self.new(filename=nil)
-    Workbook.new(filename)
+  def self.new(*params)
+    raise ArgumentError.new("wrong number of arguments (given #{params.size}, expected 0-2)") if params.size >2 
+  
+    case params.last
+      when Hash then options = params.pop 
+      else options = {}
+    end
+    
+    if options[:format].nil? # automatické heuristické rozpoznání formátu
+      options[:format] = :standard
+      unless params.first.nil?
+        begin
+          Zip::File.open(params.first)
+        rescue
+          options[:format] = :flat
+        end
+      end
+    end
+    
+    case options[:format]
+      when :flat , :fods then WorkbookFlat.new(*params)
+      when :standard then Workbook.new(*params)
+      else raise 'format of the file not recognized'
+    end
   end  
-  def self.open(filename)
-    Workbook.new(filename)
+  def self.open(filename, options = {})
+    self.new(filename, options)
   end
 end

data/lib/rspreadsheet/empty_file_template.fods

@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="UTF-8"?>
+
+<office:document xmlns:office="urn:oasis:names:tc:opendocument:xmlns:office:1.0" xmlns:style="urn:oasis:names:tc:opendocument:xmlns:style:1.0" xmlns:text="urn:oasis:names:tc:opendocument:xmlns:text:1.0" xmlns:table="urn:oasis:names:tc:opendocument:xmlns:table:1.0" xmlns:draw="urn:oasis:names:tc:opendocument:xmlns:drawing:1.0" xmlns:fo="urn:oasis:names:tc:opendocument:xmlns:xsl-fo-compatible:1.0" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:meta="urn:oasis:names:tc:opendocument:xmlns:meta:1.0" xmlns:number="urn:oasis:names:tc:opendocument:xmlns:datastyle:1.0" xmlns:presentation="urn:oasis:names:tc:opendocument:xmlns:presentation:1.0" xmlns:svg="urn:oasis:names:tc:opendocument:xmlns:svg-compatible:1.0" xmlns:chart="urn:oasis:names:tc:opendocument:xmlns:chart:1.0" xmlns:dr3d="urn:oasis:names:tc:opendocument:xmlns:dr3d:1.0" xmlns:math="http://www.w3.org/1998/Math/MathML" xmlns:form="urn:oasis:names:tc:opendocument:xmlns:form:1.0" xmlns:script="urn:oasis:names:tc:opendocument:xmlns:script:1.0" xmlns:config="urn:oasis:names:tc:opendocument:xmlns:config:1.0" xmlns:ooo="http://openoffice.org/2004/office" xmlns:ooow="http://openoffice.org/2004/writer" xmlns:oooc="http://openoffice.org/2004/calc" xmlns:dom="http://www.w3.org/2001/xml-events" xmlns:xforms="http://www.w3.org/2002/xforms" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:rpt="http://openoffice.org/2005/report" xmlns:of="urn:oasis:names:tc:opendocument:xmlns:of:1.2" xmlns:xhtml="http://www.w3.org/1999/xhtml" xmlns:grddl="http://www.w3.org/2003/g/data-view#" xmlns:tableooo="http://openoffice.org/2009/table" xmlns:drawooo="http://openoffice.org/2010/draw" xmlns:calcext="urn:org:documentfoundation:names:experimental:calc:xmlns:calcext:1.0" xmlns:loext="urn:org:documentfoundation:names:experimental:office:xmlns:loext:1.0" xmlns:field="urn:openoffice:names:experimental:ooo-ms-interop:xmlns:field:1.0" xmlns:formx="urn:openoffice:names:experimental:ooxml-odf-interop:xmlns:form:1.0" xmlns:css3t="http://www.w3.org/TR/css3-text/" office:version="1.2" office:mimetype="application/vnd.oasis.opendocument.spreadsheet">
+<office:meta><meta:generator>Rspreadshhet gem by Gorn</meta:generator></office:meta>
+<office:automatic-styles />
+<office:body>
+ <office:spreadsheet />
+</office:body>
+</office:document>

data/lib/rspreadsheet/tools.rb

@@ -1,7 +1,11 @@
-module Rspreadsheet
+  require 'pry'
+
+  module Rspreadsheet
 
 # this module contains methods used bz several objects
 module Tools
+  using ClassExtensions if RUBY_VERSION > '2.1'
+  
   def self.only_letters?(x); x.kind_of?(String) and x.match(/^[A-Za-z]*$/) != nil end
   def self.kind_of_integer?(x)
     (x.kind_of?(Numeric) and x.to_i==x) or 
@@ -162,7 +166,8 @@ module Tools
     end
   end
   def self.remove_ns_attribute(node,ns_prefix,key)
-    node.attributes.get_attribute_ns(Tools.get_namespace(ns_prefix).href,key)
+    ns = Tools.get_namespace(ns_prefix)
+    attr = node.attributes.get_attribute_ns(ns.href, key)
     attr.remove! unless attr.nil? 
   end
   def self.prepare_ns_node(ns_prefix,nodename,value=nil)
@@ -176,7 +181,6 @@ module Tools
     end
   end
   
-  
   def self.get_unused_filename(zip,prefix, extension)
     (1000..9999).each do |ndx|
       filename = prefix + ndx.to_s + ((Time.now.to_r*1000000000).to_i.to_s(16)) + extension
@@ -188,7 +192,45 @@ module Tools
   def self.new_time_value(h,m,s)
     Time.new(StartOfEpoch.year,StartOfEpoch.month,StartOfEpoch.day,h,m,s)
   end
+  
+  def self.output_to_zip_stream(io,&block)
+    if io.kind_of? File or io.kind_of? String
+      Zip::File.open(io, 'br+') do |zip|
+        yield zip
+      end
+    elsif io.kind_of? StringIO # or io.kind_of? IO
+      Zip::OutputStream.write_buffer(io) do |zip|
+        yield zip
+      end
+    end
+  end
  
+  def self.content_xml_diff(filename1,filename2)
+    content_xml1 = Zip::File.open(filename1) do |zip|
+      LibXML::XML::Document.io zip.get_input_stream('content.xml')
+    end
+    content_xml2 = Zip::File.open(filename2) do |zip|
+      LibXML::XML::Document.io zip.get_input_stream('content.xml')
+    end
+    
+    return xml_diff(content_xml1.root,content_xml2.root)
+  end
+  
+  def self.xml_file_diff(filename1,filename2)
+    content_xml1 = LibXML::XML::Document.file(filename1).root
+    content_xml2 = LibXML::XML::Document.file(filename2).root
+    return xml_diff(content_xml1, content_xml2)
+  end
+
+  def self.xml_diff(xml_node1,xml_node2)
+    message = []
+    message << xml_node2.first_diff(xml_node1)
+    message << xml_node1.first_diff(xml_node2)
+    message << 'content XML not equal' unless xml_node1.to_s.should == xml_node2.to_s
+    message = message.compact.join('; ')
+    message = nil if message == ''
+    message
+  end
 end
   
 end

data/lib/rspreadsheet/version.rb

@@ -1,3 +1,3 @@
 module Rspreadsheet
-  VERSION = "0.4.9"
+  VERSION = "0.5.0"
 end

data/lib/rspreadsheet/workbook.rb

@@ -2,12 +2,13 @@ require 'zip'
 require 'libxml'
 
 module Rspreadsheet
+    
 class Workbook
   attr_reader :filename
   attr_reader :xmlnode # debug
   def xmldoc; @xmlnode.doc end
   
-  #@!group Worskheets methods
+  #@!group Worksheet methods
   def create_worksheet_from_node(source_node)
     sheet = Worksheet.new(source_node,self)
     register_worksheet(sheet)
@@ -21,10 +22,11 @@ class Workbook
   alias :add_worksheet :create_worksheet 
   # @return [Integer] number of sheets in the workbook
   def worksheets_count; @worksheets.length end
+  alias :worksheet_count :worksheets_count
   # @return [String] names of sheets in the workbook
   def worksheet_names; @worksheets.collect{ |ws| ws.name } end
   # @param [Integer,String]
-  # @return [Worskheet] worksheet with given index or name
+  # @return [Worksheet] worksheet with given index or name
   def worksheets(index_or_name)
     case index_or_name
       when Integer then begin
@@ -61,8 +63,7 @@ class Workbook
     @xmlnode.find('./table:table').each do |node|
       create_worksheet_from_node(node)
     end
-  end
-  
+  end  
 
   # @param [String] Optional new filename
   # Saves the worksheet. Optionally you can provide new filename or IO stream to which the file should be saved.
@@ -70,47 +71,53 @@ class Workbook
     case
       when @filename.nil? && io.nil?
         raise 'New file should be named on first save.'
-      when @filename.nil? && (io.kind_of?(String) || io.kind_of?(File) || io.kind_of?(IO) || io.kind_of?(StringIO))
-        Zip::File.open(TEMPLATE_FILE_NAME) do |empty_template_zip|                       # open empty_template file
-          write_zip_to(io) do |output_zip|                                               # open output stream of file
-            copy_internally_without_content(empty_template_zip,output_zip)  # copy empty_template internals
-            update_manifest_and_content_xml(empty_template_zip,output_zip)               # update xmls + pictures
-          end
-        end
       when @filename.kind_of?(String) && io.nil?
-        write_zip_to(@filename) do |input_and_output_zip|                            # open old file
-          update_manifest_and_content_xml(input_and_output_zip,input_and_output_zip) # input and output are identical
+        Tools.output_to_zip_stream(@filename) do |input_and_output_zip|                  # open old file
+          update_zip_manifest_and_content_xml(input_and_output_zip,input_and_output_zip) # input and output are identical
         end
-        
-      when @filename.kind_of?(String) && (io.kind_of?(String) || io.kind_of?(File))
-        io = io.path if io.kind_of?(File)                                            # convert file to its filename
-        FileUtils.cp(@filename , io)                                                 # copy file externally
-        @filename = io                                                               # remember new name
-        save_to_io(nil)                                                              # continue modyfying file on spot
-        
-      when @filename.kind_of?(String) && (io.kind_of?(IO) || io.kind_of?(StringIO))
-        Zip::File.open(@filename) do | old_zip |                                    # open old file
-          write_zip_to(io) do |output_zip_stream|                                   # open output stream
-            copy_internally_without_content(old_zip,output_zip_stream) # copy the old internals
-            update_manifest_and_content_xml(old_zip,output_zip_stream)              # update xmls + pictures
-          end
+      when (@filename.kind_of?(String) && (io.kind_of?(String) || io.kind_of?(File)))
+        io = io.path if io.kind_of?(File)                                           # convert file to its filename
+        FileUtils.cp(@filename , io)                                                # copy file externally
+        @filename = io                                                              # remember new name
+        save_to_io(nil)                                                             # continue modyfying file on spot
+      when io.kind_of?(IO) || io.kind_of?(String) || io.kind_of?(StringIO)
+        Tools.output_to_zip_stream(io) do |output_io|                               # open output stream of file
+          write_ods_to_io(output_io)
         end
-        # rewind result
-        io.rewind
-      else
+        io.rewind if io.kind_of?(StringIO)
+      else raise 'Ivalid combinations of parameter types in save'
     end
   end
-  alias :to_io :save
   alias :save_to_io  :save
+  alias :save_as :save
+  def to_io
+    WorkbookIO.new(self)
+  end
+  
+  def write_ods_to_io(io)
+    if @filename.nil?
+      Zip::File.open(TEMPLATE_FILE_NAME) do |empty_template_zip|         # open empty_template file
+        copy_internally_without_content(empty_template_zip,io)           # copy empty_template internals
+        update_zip_manifest_and_content_xml(empty_template_zip,io)           # update xmls + pictures
+      end
+    else
+      Zip::File.open(@filename) do | old_zip |                           # open old file
+        copy_internally_without_content(old_zip,io)                      # copy the old internals
+        update_zip_manifest_and_content_xml(old_zip,io)                      # update xmls + pictures
+      end
+    end
+  end
   
+  def flat_format?; false end
+  def normal_format?; true end
+
   private 
 
-  def update_manifest_and_content_xml(input_zip,output_zip)
+  def update_zip_manifest_and_content_xml(input_zip,output_zip)
     update_manifest_xml(input_zip,output_zip)
     update_content_xml(output_zip)
   end
   
-  
   def update_content_xml(zip)
     save_entry_to_zip(zip,CONTENT_FILE_NAME,@content_xml.to_s(indent: false))
   end
@@ -162,7 +169,6 @@ class Workbook
   
   def save_entry_to_zip(zip,internal_filename,contents)
     if zip.kind_of? Zip::File
-#       raise [internal_filename,contents].inspect  unless File.exists?(internal_filename)
       zip.get_output_stream(internal_filename) do  |f|
         f.write contents
       end
@@ -171,19 +177,7 @@ class Workbook
       zip.write(contents)
     end
   end
-  
-  def write_zip_to(io,&block)
-    if io.kind_of? File or io.kind_of? String
-      Zip::File.open(io, 'br+') do |zip|
-        yield zip
-      end
-    elsif io.kind_of? StringIO # or io.kind_of? IO
-      Zip::OutputStream.write_buffer(io) do |zip|
-        yield zip
-      end
-    end
-  end
-  
+    
   CONTENT_FILE_NAME = 'content.xml'
   MANIFEST_FILE_NAME = 'META-INF/manifest.xml'
   TEMPLATE_FILE_NAME = (File.dirname(__FILE__)+'/empty_file_template.ods').freeze
@@ -192,5 +186,59 @@ class Workbook
     @worksheets[index-1]=worksheet
     @xmlnode << worksheet.xmlnode if worksheet.xmlnode.doc != @xmlnode.doc
   end
+      
+end
+
+class WorkbookFlat < Workbook
+  def initialize(afilename=nil)
+    @worksheets=[]
+    @filename = afilename
+    @xml_doc = LibXML::XML::Document.file(@filename || FLAT_TEMPLATE_FILE_NAME)
+    @xmlnode = @xml_doc.find_first('//office:spreadsheet')
+    @xmlnode.find('./table:table').each do |node|
+      create_worksheet_from_node(node)
+    end
+  end
+
+  def save(io=nil)
+    case
+      when @filename.nil? && io.nil?
+        raise 'New file should be named on first save, please provide filename (or IO).'
+      when @filename.kind_of?(String) && io.nil?
+        @xml_doc.save(@filename)
+      when (@filename.kind_of?(String) && (io.kind_of?(String) || io.kind_of?(File)))
+        @filename = (io.kind_of?(File)) ? io.path : io
+        @xml_doc.save(@filename)
+      when io.kind_of?(IO) || io.kind_of?(String) || io.kind_of?(StringIO)
+        IO.write(io,@xml_doc.to_s)
+        io.rewind if io.kind_of?(StringIO)
+      else raise 'Invalid combinations of parameter types in save'
+    end
+  end
+  alias :save_to_io  :save
+  alias :save_as :save
+  
+  def flat_format?; true end
+  def normal_format?; false end
+
+  private 
+  FLAT_TEMPLATE_FILE_NAME = (File.dirname(__FILE__)+'/empty_file_template.fods').freeze  
+    
+end
+
+class WorkbookIO
+  def initialize(workbook)
+    @workbook = workbook
+  end
+  def read
+    buffer.string
+  end
+  private
+  def buffer
+    Zip::OutputStream.write_buffer do |io|
+      @workbook.write_ods_to_io(io)
+    end
+  end
 end
+
 end