Inline and simplify OpenOffice.process_zipfile

lib/roo/open_office.rb

@@ -3,23 +3,6 @@
 require 'cgi'
 
 class Roo::OpenOffice < Roo::Base
-  class << self
-    def process_zipfile(tmpdir, zip, path='')
-      if zip.file.file? path
-        if path == "content.xml"
-          File.write(File.join(tmpdir, 'roo_content.xml'), zip.read(path), mode: 'wb')
-        end
-      else
-        unless path.empty?
-          path += '/'
-        end
-        zip.dir.foreach(path) do |filename|
-          process_zipfile(tmpdir, zip, path+filename)
-        end
-      end
-    end
-  end
-
   # initialization and opening of a spreadsheet file
   # values for packed: :zip
   def initialize(filename, options={})
@@ -30,8 +13,12 @@ def initialize(filename, options={})
     @tmpdir = make_tmpdir(filename.split('/').last, options[:tmpdir_root])
     @filename = local_filename(filename, @tmpdir, packed)
     #TODO: @cells_read[:default] = false
-    Roo::ZipFile.open(@filename) do |zip|
-      self.class.process_zipfile(@tmpdir, zip)
+    Roo::ZipFile.open(@filename) do |zip_file|
+      if content_entry = zip_file.glob("content.xml").first
+        content_entry.extract(File.join(@tmpdir, 'roo_content.xml'))
+      else
+        raise ArgumentError, 'file missing required content.xml'
+      end
     end
     super(filename, options)
     @formula = Hash.new

test/test_roo.rb

@@ -1003,7 +1003,7 @@ def test_file_warning_error
 
   def test_file_warning_warning
     if OPENOFFICE
-      assert_raises(Errno::ENOENT) {
+      assert_raises(ArgumentError) {
         Roo::OpenOffice.new(File.join(TESTDIR,"numbers1.xlsx"),
         packed: false,
         file_warning: :warning)