dbf 5.1.1 → 5.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7d565a51490240e9f5ed724970a568f12641b595b66e426a3a4a1a6ba0c84003
-  data.tar.gz: 12fc66ce2b83c55157532d252b52eb6a343174898a7965a636beb7877aa29c17
+  metadata.gz: d2d1ca393ae41262fd052c74ffe0ec497a963a2c036797686061312a0c492888
+  data.tar.gz: 7e92c62ace7713b883b6b2e925c34f7a8114ebd6345b36a81f97fb47109711e2
 SHA512:
-  metadata.gz: c2de237f84ddc71f271a782b537f03d10a8586e939ad08f66b34c9f6bcc38a1b34aa4a2899d3248a1ff2f574bfb9920c19d2f134cae01ac66f0e228acd9a1c13
-  data.tar.gz: 5b5b3ee469e5afe32bc54ea4f494b543c675a2b4c61b6e6cf56af68ab2ab989a35644b7795a29e1cd9e4dfe8e4129c1abf85696a4ff7df8fb0444f89c7f80193
+  metadata.gz: f0f3b1b02ab53f505ab3efc1dff89d2ad1169698ef7d9443b3eec3b758b921ae2df51bf443d4e877ac072bdef87c547607d1d39d913a5c361b4c31e21fee4b07
+  data.tar.gz: 6d826dd35aa6e80b671aeb804dc9991776851bbb9ca0ff5fb9ed6cd7139cbd40b0bad68327fc8f17b03b9ba2f2b77479331b7ad998c53e7ee32e5caee59241e6

data/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## main branch
+
+- Add support for 8 more code pages (issue #98): Mazovia cp620 and Kamenický cp895 via vendored translation tables (new DBF::Encoder), plus macRoman, cp1255, cp1256, macCyrillic, macCentEuro and macGreek
+- Blank Visual FoxPro "T" (DateTime) columns now return nil instead of a Julian day-0 date
+- Blank "F" (Float) columns now return nil instead of 0.0, matching "N" (Number) behavior
+- Extract CLI logic into DBF::CLI class (bin/dbf is now a thin shim)
+- DBF::Table#to_csv now accepts an IO in addition to a String path
+- Fix broken link in README
+
+## 5.2.0
+
+- Drop support for Ruby 3.1 and 3.2
+
 ## 5.1.1
 
 - Frozen string literals

data/README.md

@@ -10,16 +10,18 @@
 DBF is a small, fast Ruby library for reading dBase, xBase, Clipper, and FoxPro database files.
 
 * Project page: <https://github.com/infused/dbf>
-* API Documentation: <https://rubydoc.info/github/infused/dbf>
+* API Documentation: <https://rdoc.info/gems/dbf>
 * Report bugs: <https://github.com/infused/dbf/issues>
 * Questions: Email <mailto:keithm@infused.org> and put DBF somewhere in the
   subject line
 * Change log: <https://github.com/infused/dbf/blob/main/CHANGELOG.md>
 
+NOTE: Beginning with version 5.2 we have dropped support for Ruby 3.2 and earlier.
+
 NOTE: Beginning with version 4.3 we have dropped support for Ruby 3.0 and earlier.
 
 NOTE: Beginning with version 4 we have dropped support for Ruby 2.0, 2.1, 2.2, and 2.3. If you need support for these older Rubies,
-please use 3.0.x (<https://github.com/infused.org/dbf/tree/3_stable>)
+please use 3.0.x (<https://github.com/infused/dbf/tree/3_stable>)
 
 NOTE: Beginning with version 3 we have dropped support for Ruby 1.8 and 1.9. If you need support for older Rubies,
 please use 2.0.x (<https://github.com/infused/dbf/tree/2_stable>)
@@ -28,7 +30,7 @@ please use 2.0.x (<https://github.com/infused/dbf/tree/2_stable>)
 
 DBF is tested to work with the following versions of Ruby:
 
-* Ruby 3.1.x, 3.2.x, 3.3.x
+* Ruby 3.3.x, 3.4.x, 4.0.x
 
 ## Installation
 
@@ -149,6 +151,7 @@ table = DBF::Table.new('dbf/books.dbf', nil, 'cp866')
 | 01 | cp437 | U.S. MS–DOS |
 | 02 | cp850 | International MS–DOS |
 | 03 | cp1252 | Windows ANSI |
+| 04 | macRoman | Standard Macintosh |
 | 08 | cp865 | Danish OEM |
 | 09 | cp437 | Dutch OEM |
 | 0a | cp850 | Dutch OEM* |
@@ -189,6 +192,8 @@ table = DBF::Table.new('dbf/books.dbf', nil, 'cp866')
 | 65 | cp866 | Russian MS–DOS |
 | 66 | cp865 | Nordic MS–DOS |
 | 67 | cp861 | Icelandic MS–DOS |
+| 68 | cp895 | Kamenický (Czech) MS–DOS |
+| 69 | cp620 | Mazovia (Polish) MS–DOS |
 | 6a | cp737 | Greek MS–DOS (437G) |
 | 6b | cp857 | Turkish MS–DOS |
 | 6c | cp863 | French–Canadian MS–DOS |
@@ -197,9 +202,14 @@ table = DBF::Table.new('dbf/books.dbf', nil, 'cp866')
 | 7a | cp936 | PRC GBK |
 | 7b | cp932 | Japanese Shift-JIS |
 | 7c | cp874 | Thai Windows/MS–DOS |
+| 7d | cp1255 | Hebrew Windows |
+| 7e | cp1256 | Arabic Windows |
 | 86 | cp737 | Greek OEM |
 | 87 | cp852 | Slovenian OEM |
 | 88 | cp857 | Turkish OEM |
+| 96 | macCyrillic | Russian Macintosh |
+| 97 | macCentEuro | Macintosh EE |
+| 98 | macGreek | Greek Macintosh |
 | c8 | cp1250 | Eastern European Windows |
 | c9 | cp1251 | Russian Windows |
 | ca | cp1254 | Turkish Windows |
@@ -251,7 +261,7 @@ class Book < Sequel::Model; end
 Sequel.migration do
   up do
     table = DBF::Table.new('db/dbf/books.dbf')
-    eval(table.schema(:sequel, true)) # passing true to limit output to create_table() only
+    eval(table.schema(:sequel, table_only: true)) # limit output to create_table() only
 
     Book.reset_column_information
     table.each do |record|
@@ -326,16 +336,16 @@ at this time.
 | 04      | dBase IV without memo file                          | Y | Y | Y | Y | - | - | - | - | - | - | - | - | - | - | - | - | - |
 | 05      | dBase V without memo file                           | Y | Y | Y | Y | - | - | - | - | - | - | - | - | - | - | - | - | - |
 | 07      | Visual Objects 1.x                                  | Y | Y | Y | Y | - | - | - | - | - | - | - | - | - | - | - | - | - |
-| 30      | Visual FoxPro                                       | Y | Y | Y | Y | Y | Y | Y | Y | N | Y | N | Y | N | N | N | N | - |
-| 31      | Visual FoxPro with AutoIncrement                    | Y | Y | Y | Y | Y | Y | Y | Y | N | Y | N | Y | N | N | N | N | N |
-| 32      | Visual FoxPro with field type Varchar or Varbinary  | Y | Y | Y | Y | Y | Y | Y | Y | N | Y | N | Y | N | N | N | N | N |
+| 30      | Visual FoxPro                                       | Y | Y | Y | Y | Y | Y | Y | Y | N | Y | Y | Y | N | N | N | N | - |
+| 31      | Visual FoxPro with AutoIncrement                    | Y | Y | Y | Y | Y | Y | Y | Y | N | Y | Y | Y | N | N | N | N | N |
+| 32      | Visual FoxPro with field type Varchar or Varbinary  | Y | Y | Y | Y | Y | Y | Y | Y | N | Y | Y | Y | N | N | N | N | N |
 | 7b      | dBase IV with memo file                             | Y | Y | Y | Y | Y | Y | - | - | - | - | - | - | - | - | - | - | - |
 | 83      | dBase III with memo file                            | Y | Y | Y | Y | Y | - | - | - | - | - | - | - | - | - | - | - | - |
 | 87      | Visual Objects 1.x with memo file                   | Y | Y | Y | Y | Y | - | - | - | - | - | - | - | - | - | - | - | - |
 | 8b      | dBase IV with memo file                             | Y | Y | Y | Y | Y | - | - | - | - | - | - | - | - | N | - | - | - |
 | 8e      | dBase IV with SQL table                             | Y | Y | Y | Y | Y | - | - | - | - | - | - | - | - | N | - | - | - |
-| f5      | FoxPro with memo file                               | Y | Y | Y | Y | Y | Y | Y | Y | N | Y | N | Y | N | N | N | N | N |
-| fb      | FoxPro without memo file                            | Y | Y | Y | Y | - | Y | Y | Y | N | Y | N | Y | N | N | N | N | N |
+| f5      | FoxPro with memo file                               | Y | Y | Y | Y | Y | Y | Y | Y | N | Y | Y | Y | N | N | N | N | N |
+| fb      | FoxPro without memo file                            | Y | Y | Y | Y | - | Y | Y | Y | N | Y | Y | Y | N | N | N | N | N |
 
 Data type descriptions
 

data/bin/dbf

@@ -1,60 +1,8 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 Signal.trap('PIPE', 'SYSTEM_DEFAULT')
 
 require 'dbf'
-require 'dbf/version'
-require 'optparse'
 
-params = ARGV.getopts('h', 's', 'a', 'c', 'r', 'v')
-
-if params['v']
-  puts "dbf version: #{DBF::VERSION}"
-
-elsif params['h']
-  puts "usage: #{File.basename(__FILE__)} [-h|-s|-a|-c|-r] filename"
-  puts '  -h = print this message'
-  puts '  -v = print the DBF gem version'
-  puts '  -s = print summary information'
-  puts '  -a = create an ActiveRecord::Schema'
-  puts '  -r = create a Sequel migration'
-  puts '  -c = export as CSV'
-else
-
-  filename = ARGV.shift
-  abort 'You must supply a filename on the command line' unless filename
-
-  # create an ActiveRecord::Schema
-  if params['a']
-    table = DBF::Table.new filename
-    puts table.schema(:activerecord)
-  end
-
-  # create an Sequel::Migration
-  if params['r']
-    table = DBF::Table.new filename
-    puts table.schema(:sequel)
-  end
-
-  if params['s']
-    table = DBF::Table.new filename
-    puts
-    puts "Database: #{filename}"
-    puts "Type: (#{table.version}) #{table.version_description}"
-    puts "Encoding: #{table.header_encoding}" if table.header_encoding
-    puts "Memo File: #{table.has_memo_file? ? 'true' : 'false'}"
-    puts "Records: #{table.record_count}"
-
-    puts "\nFields:"
-    puts 'Name             Type       Length     Decimal'
-    puts '-' * 78
-    table.columns.each do |f|
-      puts format('%-16s %-10s %-10s %-10s', f.name, f.type, f.length, f.decimal)
-    end
-  end
-
-  if params['c']
-    table = DBF::Table.new filename
-    table.to_csv
-  end
-end
+exit DBF::CLI.run(ARGV)

data/dbf.gemspec

@@ -1,24 +1,21 @@
-lib = File.expand_path('lib', __dir__)
-$LOAD_PATH.unshift lib unless $LOAD_PATH.include?(lib)
-require 'dbf/version'
+require_relative 'lib/dbf/version'
 
 Gem::Specification.new do |s|
   s.name = 'dbf'
   s.version = DBF::VERSION
   s.authors = ['Keith Morrison']
   s.email = 'keithm@infused.org'
-  s.homepage = 'http://github.com/infused/dbf'
+  s.homepage = 'https://github.com/infused/dbf'
   s.summary = 'Read xBase files'
   s.description = 'A small fast library for reading dBase, xBase, Clipper and FoxPro database files.'
   s.license = 'MIT'
   s.bindir = 'bin'
   s.executables = ['dbf']
-  s.rdoc_options = ['--charset=UTF-8']
-  s.extra_rdoc_files = ['README.md', 'CHANGELOG.md', 'LICENSE']
-  s.files = Dir['README.md', 'CHANGELOG.md', 'LICENSE', '{bin,lib,spec}/**/*', 'dbf.gemspec']
+  s.files = Dir['README.md', 'CHANGELOG.md', 'LICENSE', '{bin,lib}/**/*', 'dbf.gemspec']
   s.require_paths = ['lib']
-  s.required_rubygems_version = Gem::Requirement.new('>= 3.2.3')
-  s.required_ruby_version = Gem::Requirement.new('>= 3.1.0')
+  s.required_ruby_version = '>= 3.3.0'
   s.metadata['rubygems_mfa_required'] = 'true'
+  s.metadata['source_code_uri'] = 'https://github.com/infused/dbf'
+  s.metadata['changelog_uri'] = 'https://github.com/infused/dbf/blob/main/CHANGELOG.md'
   s.add_dependency 'csv'
 end

data/lib/dbf/cli.rb

@@ -0,0 +1,95 @@
+# frozen_string_literal: true
+
+require 'optparse'
+
+module DBF
+  class CLI
+    USAGE = <<~HELP
+      usage: dbf [-h|-s|-a|-c|-r] filename
+        -h = print this message
+        -v = print the DBF gem version
+        -s = print summary information
+        -a = create an ActiveRecord::Schema
+        -r = create a Sequel migration
+        -c = export as CSV
+    HELP
+
+    def self.run(argv, stdout: $stdout, stderr: $stderr)
+      new(argv, stdout: stdout, stderr: stderr).run
+    end
+
+    def initialize(argv, stdout: $stdout, stderr: $stderr)
+      @argv = argv.dup
+      @stdout = stdout
+      @stderr = stderr
+    end
+
+    def run
+      params = OptionParser.new.getopts(@argv, 'h', 's', 'a', 'c', 'r', 'v')
+
+      if params['v']
+        print_version
+      elsif params['h']
+        print_help
+      else
+        filename = @argv.shift
+        return missing_filename unless filename
+
+        action = %w[a r s c].find { |flag| params[flag] }
+        case action
+        when 'a' then print_ar_schema(filename)
+        when 'r' then print_sequel_schema(filename)
+        when 's' then print_summary(filename)
+        when 'c' then print_csv(filename)
+        end
+      end
+      0
+    rescue DBF::FileNotFoundError => e
+      @stderr.puts "DBF::FileNotFoundError: #{e.message}"
+      1
+    end
+
+    private
+
+    def print_version
+      @stdout.puts "dbf version: #{DBF::VERSION}"
+    end
+
+    def print_help
+      @stdout.puts USAGE
+    end
+
+    def missing_filename
+      @stderr.puts 'You must supply a filename on the command line'
+      1
+    end
+
+    def print_ar_schema(filename)
+      @stdout.puts DBF::Table.new(filename).schema(:activerecord)
+    end
+
+    def print_sequel_schema(filename)
+      @stdout.puts DBF::Table.new(filename).schema(:sequel)
+    end
+
+    def print_summary(filename)
+      table = DBF::Table.new(filename)
+      @stdout.puts
+      @stdout.puts "Database: #{filename}"
+      @stdout.puts "Type: (#{table.version}) #{table.version_description}"
+      @stdout.puts "Encoding: #{table.header_encoding}" if table.header_encoding
+      @stdout.puts "Memo File: #{table.has_memo_file? ? 'true' : 'false'}"
+      @stdout.puts "Records: #{table.record_count}"
+      @stdout.puts "\nFields:"
+      @stdout.puts 'Name             Type       Length     Decimal'
+      @stdout.puts '-' * 78
+      table.columns.each do |f|
+        @stdout.puts format('%-16s %-10s %-10s %-10s', f.name, f.type, f.length, f.decimal)
+      end
+    end
+
+    def print_csv(filename)
+      DBF::Table.new(filename).to_csv(@stdout)
+    end
+  end
+end

data/lib/dbf/column.rb

@@ -2,17 +2,13 @@
 
 module DBF
   class Column
-    extend Forwardable
-
     class LengthError < StandardError
     end
 
     class NameError < StandardError
     end
 
-    attr_reader :table, :name, :type, :length, :decimal, :encoding
-
-    def_delegator :type_cast_class, :type_cast
+    attr_reader :name, :type, :length, :decimal
 
     # rubocop:disable Style/MutableConstant
     TYPE_CAST_CLASS = {
@@ -26,7 +22,7 @@ module DBF
       M: ColumnType::Memo,
       B: ColumnType::Double,
       G: ColumnType::General,
-      :+ => ColumnType::SignedLong2
+      :+ => ColumnType::AutoIncrement
     }
     # rubocop:enable Style/MutableConstant
     TYPE_CAST_CLASS.default = ColumnType::String
@@ -40,31 +36,37 @@ module DBF
     # @param length [Integer]
     # @param decimal [Integer]
     def initialize(table, name, type, length, decimal)
-      @encoding = table.encoding
-
       @table = table
       @name = clean(name)
       @type = type
       @length = length
       @decimal = decimal
-      @version = table.version
 
       validate_length
       validate_name
     end
 
-    # Returns true if the column is a memo
+    def encoding = @table.encoding
+
+    # @param value [String]
+    def type_cast(value)
+      type_cast_class.type_cast(value)
+    end
+
+    # Decodes a raw column value, handling memo, blank, and type cast cases
     #
-    # @return [Boolean]
-    def memo?
-      @memo ||= type == 'M'
+    # @param raw [String]
+    # @yield [raw] for memo column resolution
+    # @return decoded value
+    def decode(raw, &)
+      type_cast_class.decode(raw, &)
     end
 
     # Returns a Hash with :name, :type, :length, and :decimal keys
     #
     # @return [Hash]
     def to_hash
-      {name: name, type: type, length: length, decimal: decimal}
+      {name:, type:, length:, decimal:}
     end
 
     # Underscored name
@@ -80,7 +82,7 @@ module DBF
     private
 
     def clean(value) # :nodoc:
-      table.encode_string(value.strip.partition("\x00").first)
+      @table.encode_string(value.strip.split("\x00", 2).first || +'')
     end
 
     def type_cast_class # :nodoc:

data/lib/dbf/column_builder.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module DBF
+  class ColumnBuilder
+    def initialize(table, data, version_config)
+      @table = table
+      @data = data
+      @version_config = version_config
+    end
+
+    def build
+      safe_seek do
+        @data.seek(@version_config.header_size)
+        [].tap do |columns|
+          columns << Column.new(*@version_config.read_column_args(@table, @data)) until end_of_record?
+        end
+      end
+    end
+
+    private
+
+    def end_of_record?
+      safe_seek { @data.read(1).ord == 13 }
+    end
+
+    def safe_seek
+      original_pos = @data.pos
+      yield.tap { @data.seek(original_pos) }
+    end
+  end
+end

data/lib/dbf/column_type.rb

@@ -5,12 +5,27 @@ module DBF
     class Base
       attr_reader :decimal, :encoding
 
-      # @param decimal [Integer]
-      # @param encoding [String, Encoding]
+      # @param column [DBF::Column]
       def initialize(column)
         @decimal = column.decimal
         @encoding = column.encoding
       end
+
+      def blank_value
+        nil
+      end
+
+      def skip_blank?
+        false
+      end
+
+      def decode(raw, &)
+        if skip_blank? && raw.count(' ') == raw.length
+          blank_value
+        else
+          type_cast(raw)
+        end
+      end
     end
 
     class Nil < Base
@@ -21,11 +36,13 @@ module DBF
     end
 
     class Number < Base
+      def skip_blank? = true
+
       # @param value [String]
       def type_cast(value)
-        return nil if value.strip.empty?
+        return nil if value.empty?
 
-        @decimal.zero? ? value.to_i : value.to_f
+        decimal.zero? ? value.to_i : value.to_f
       end
     end
 
@@ -43,18 +60,22 @@ module DBF
       end
     end
 
-    class SignedLong2 < Base
+    class AutoIncrement < Base
       # @param value [String]
       def type_cast(value)
-        s = value.unpack1('B*')
-        sign_multiplier = s[0] == '0' ? -1 : 1
-        s[1, 31].to_i(2) * sign_multiplier
+        bits = value.unpack1('B*')
+        sign_multiplier = bits[0] == '0' ? -1 : 1
+        bits[1, 31].to_i(2) * sign_multiplier
       end
     end
 
     class Float < Base
+      def skip_blank? = true
+
       # @param value [String]
       def type_cast(value)
+        return nil if value.empty?
+
         value.to_f
       end
     end
@@ -67,13 +88,20 @@ module DBF
     end
 
     class Boolean < Base
+      def skip_blank? = true
+      def blank_value = false
+
       # @param value [String]
       def type_cast(value)
-        value.strip.match?(/^(y|t)$/i)
+        byte = value.getbyte(0)
+        byte == 89 || byte == 121 || byte == 84 || byte == 116 # Y y T t
       end
     end
 
     class Date < Base
+      def skip_blank? = true
+      def blank_value = false
+
       # @param value [String]
       def type_cast(value)
         value.match?(/\d{8}/) && ::Date.strptime(value, '%Y%m%d')
@@ -86,6 +114,8 @@ module DBF
       # @param value [String]
       def type_cast(value)
         days, msecs = value.unpack('l2')
+        return nil if days.nil? || days.zero?
+
         secs = (msecs / 1000).to_i
         ::DateTime.jd(days, (secs / 3600).to_i, (secs / 60).to_i % 60, secs % 60).to_time
       rescue StandardError
@@ -94,13 +124,16 @@ module DBF
     end
 
     class Memo < Base
+      def decode(raw, &)
+        memo_content = yield(raw)
+        memo_content ? type_cast(memo_content) : nil
+      end
+
       # @param value [String]
       def type_cast(value)
-        if encoding && !value.nil?
-          value.dup.force_encoding(@encoding).encode(Encoding.default_external, undef: :replace, invalid: :replace)
-        else
-          value
-        end
+        return value unless encoding && value
+
+        Encoder.encode(value.dup, encoding)
       end
     end
 
@@ -112,10 +145,27 @@ module DBF
     end
 
     class String < Base
+      def initialize(column)
+        super
+        @target_encoding = Encoding.default_external
+        @needs_encode = encoding && encoding != @target_encoding
+      end
+
+      def skip_blank? = true
+      def blank_value = ''
+
       # @param value [String]
       def type_cast(value)
-        value = value.strip
-        @encoding ? value.force_encoding(@encoding).encode(Encoding.default_external, undef: :replace, invalid: :replace) : value
+        value.strip!
+        encoding ? encode(value) : value
+      end
+
+      private
+
+      def encode(value)
+        return value.force_encoding(encoding) unless @needs_encode
+
+        Encoder.encode(value, encoding, @target_encoding)
       end
     end
   end

data/lib/dbf/database/foxpro.rb

@@ -26,7 +26,7 @@ module DBF
         @db = DBF::Table.new(@path)
         @tables = extract_dbc_data
       rescue Errno::ENOENT
-        raise DBF::FileNotFoundError, "file not found: #{data}"
+        raise DBF::FileNotFoundError, "file not found: #{path}"
       end
 
       def table_names
@@ -38,9 +38,7 @@ module DBF
       # @param name [String]
       # @return [DBF::Table]
       def table(name)
-        Table.new table_path(name) do |table|
-          table.long_names = @tables[name]
-        end
+        Table.new(table_path(name), long_names: @tables[name])
       end
 
       # Searches the database directory for the table's dbf file
@@ -58,7 +56,8 @@ module DBF
       end
 
       def method_missing(method, *args) # :nodoc:
-        table_names.index(method.to_s) ? table(method.to_s) : super
+        name = method.to_s
+        table_names.index(name) ? table(name) : super
       end
 
       def respond_to_missing?(method, *)
@@ -72,45 +71,33 @@ module DBF
       # are in the same order as in the linked tables but only the long name
       # is provided.
       def extract_dbc_data # :nodoc:
-        data = {}
-        @db.each do |record|
+        build_table_data.values.to_h { |entry| entry.values_at(:name, :fields) }
+      end
+
+      def build_table_data # :nodoc:
+        @db.each_with_object({}) do |record, hash|
           next unless record
 
+          name = record.objectname
           case record.objecttype
-          when 'Table'
-            # This is a related table
-            process_table record, data
-          when 'Field'
-            # This is a related field. The parentid points to the table object.
-            # Create using the parentid if the parentid is still unknown.
-            process_field record, data
+          when 'Table' then hash[record.objectid] = table_field_hash(name)
+          when 'Field' then (hash[record.parentid] ||= table_field_hash('UNKNOWN'))[:fields] << name
           end
         end
-
-        data.values.to_h { |v| [v[:name], v[:fields]] }
-      end
-
-      def process_table(record, data)
-        id = record.objectid
-        name = record.objectname
-        data[id] = table_field_hash(name)
-      end
-
-      def process_field(record, data)
-        id = record.parentid
-        name = 'UNKNOWN'
-        field = record.objectname
-        data[id] ||= table_field_hash(name)
-        data[id][:fields] << field
       end
 
       def table_field_hash(name)
-        {name: name, fields: []}
+        {name:, fields: []}
       end
     end
 
     class Table < DBF::Table
-      attr_accessor :long_names
+      attr_reader :long_names
+
+      def initialize(path, long_names:)
+        @long_names = long_names
+        super(path)
+      end
 
       def build_columns # :nodoc:
         columns = super