Fixes issue where the contents of hidden sheet was returned when parsing visible sheets only.

lib/roo/excelx.rb

@@ -60,15 +60,18 @@ def initialize(filename_or_stream, options = {})
       @filename = local_filename(filename_or_stream, @tmpdir, packed)
       process_zipfile(@filename || filename_or_stream)
 
-      @sheet_names = workbook.sheets.map do |sheet|
+      sheet_indicies = []
+      @sheet_names = workbook.sheets.each_with_index.map do |sheet, index|
         unless options[:only_visible_sheets] && sheet['state'] == 'hidden'
+          sheet_indicies << index
           sheet['name']
         end
       end.compact
       @sheets = []
       @sheets_by_name = {}
       @sheet_names.each_with_index do |sheet_name, n|
-        @sheets_by_name[sheet_name] = @sheets[n] = Sheet.new(sheet_name, @shared, n, sheet_options)
+        sheet_index = sheet_indicies[n]
+        @sheets_by_name[sheet_name] = @sheets[sheet_index] = Sheet.new(sheet_name, @shared, sheet_index, sheet_options)
       end
 
       if cell_max

spec/lib/roo/excelx_spec.rb

@@ -86,6 +86,14 @@
     end
   end
 
+  describe 'for a workbook with hidden sheets' do
+    let(:path) { 'test/files/hidden_sheets.xlsx' }
+
+    it 'returns the cell contents from the visible sheet' do
+      expect(Roo::Excelx.new(path, only_visible_sheets: true).cell('A', 1)).to eq "visible sheet 1"
+    end
+  end
+
   describe '#parse' do
     let(:path) { 'test/files/numeric-link.xlsx' }
 
@@ -653,4 +661,4 @@
       end
     end
   end
-end
+end