exiftool 1.0.1 → 1.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: ae08d330e8f9086986527091a050c7bd18dcc88d
-  data.tar.gz: 6c61a287a2f4cf37fde00f7580d630427268a40d
+SHA256:
+  metadata.gz: 910908d5f0f83f412883644c911b63df24be598fc6ee9cc6b4e3555d3aabe4ff
+  data.tar.gz: 9859f9f20d72bba3dd9b8577aff0059118070ced6fdae5885521c6726f722092
 SHA512:
-  metadata.gz: 9d8de3cd3b7288d05e420791c5735987855a184b01f4005f18d2c99b41cf145250c92b1fdfc37af9bc8e5093bb071a90a90d5faf68baaaf6e49dba5edbf326ee
-  data.tar.gz: 4da77b1bbe069e7986ad63725a4f008b26dd1a9a0498e57320bf4a729f4916a9a9090a0864b76b75300a2e9dbba2ddf0bd63a126150b8f99b38ffc63918c910f
+  metadata.gz: d778dba97dcdfa9f3ca613af37a36133e1f6fb724a39d93162b7bcca52fc6222f3fdfd2ef7a497d63f7af0ad4a3bfbf349b2d6357571fd864db466032af62445
+  data.tar.gz: f8b32e9d8414ce4adb0987a4f17046f690bf130c9492f3406325fe9c876af6ab74766294c2c5dabf479c18c0e6f34d11af4c1e9dfed2ad018f50f009a881075f

data/lib/exiftool.rb

@@ -1,16 +1,22 @@
+# frozen_string_literal: true
+
 require 'json'
 require 'shellwords'
 require 'exiftool/result'
 require 'forwardable'
 
+# Exiftool Class
 class Exiftool
-  class NoSuchFile < StandardError ; end
-  class NotAFile < StandardError ; end
-  class ExiftoolNotInstalled < StandardError ; end
-  class NoDefaultResultWithMultiget < StandardError ; end
+  class NoSuchFile < StandardError; end
+
+  class NotAFile < StandardError; end
+
+  class ExiftoolNotInstalled < StandardError; end
 
-  def self.command=(path_to_exiftool)
-    @command = path_to_exiftool
+  class NoDefaultResultWithMultiget < StandardError; end
+
+  class << self
+    attr_writer :command
   end
 
   def self.command
@@ -18,7 +24,7 @@ class Exiftool
   end
 
   def self.exiftool_installed?
-    exiftool_version > 0
+    exiftool_version.to_f.positive?
   end
 
   # This is a string, not a float, to handle versions like "9.40" properly.
@@ -28,7 +34,9 @@ class Exiftool
 
   def self.expand_path(filename)
     raise(NoSuchFile, filename) unless File.exist?(filename)
+
     raise(NotAFile, filename) unless File.file?(filename)
+
     File.expand_path(filename)
   end
 
@@ -38,28 +46,29 @@ class Exiftool
 
   def initialize(filenames, exiftool_opts = '')
     @file2result = {}
-    filenames = [filenames] if filenames.is_a?(String)
-    unless filenames.empty?
-      escaped_filenames = filenames.map do |f|
-        Shellwords.escape(self.class.expand_path(f.to_s))
-      end.join(' ')
-      # I'd like to use -dateformat, but it doesn't support timezone offsets properly,
-      # nor sub-second timestamps.
-      cmd = "#{self.class.command} #{exiftool_opts} -j -coordFormat \"%.8f\" #{escaped_filenames} 2> /dev/null"
-      json = `#{cmd}`.chomp
-      raise ExiftoolNotInstalled if json == ''
-      JSON.parse(json).each do |raw|
-        result = Result.new(raw)
-        @file2result[result.source_file] = result
-      end
+    filenames = [filenames] if filenames.is_a?(String) || filenames.is_a?(Pathname)
+    return if filenames.empty?
+
+    escaped_filenames = filenames.map do |f|
+      Shellwords.escape(self.class.expand_path(f.to_s))
+    end.join(' ')
+    # I'd like to use -dateformat, but it doesn't support timezone offsets properly,
+    # nor sub-second timestamps.
+    cmd = "#{self.class.command} #{exiftool_opts} -j -coordFormat \"%.8f\" #{escaped_filenames} 2> /dev/null"
+    json = `#{cmd}`.chomp
+    raise ExiftoolNotInstalled if json == ''
+
+    JSON.parse(json).each do |raw|
+      result = Result.new(raw)
+      @file2result[result.source_file] = result
     end
   end
 
-  def results(include_results_with_errors = false)
+  def results(include_results_with_errors: false)
     if include_results_with_errors
       @file2result.values
     else
-      @file2result.values.select { |ea| !ea.errors? }
+      @file2result.values.reject(&:errors?)
     end
   end
 
@@ -68,17 +77,18 @@ class Exiftool
   end
 
   def files_with_results
-    results.map { |ea| ea.source_file }
+    results.map(&:source_file)
   end
 
   def errors?
-    @file2result.values.any? { |ea| ea.errors? }
+    @file2result.values.any?(&:errors?)
   end
 
   private
 
   def first_result
     raise(NoDefaultResultWithMultiget, 'use #result_for when multiple filenames are used') if @file2result.size > 1
+
     @file2result.values.first
   end
 end

data/lib/exiftool/field_parser.rb

@@ -1,13 +1,14 @@
+# frozen_string_literal: true
+
 require 'time'
-require 'rational'
 
 class Exiftool
+  # Exiftool FiledParser Class
   class FieldParser
-
-    WORD_BOUNDARY_RES = [/([A-Z\d]+)([A-Z][a-z])/, /([a-z\d])([A-Z])/]
-    FRACTION_RE = /^(\d+)\/(\d+)$/
-    YMD_RE = /\A(\d{4}):(\d{2}):(\d{2})\b/
-    ZERO_DATE_RE = /\A[+:0 ]+\z/
+    WORD_BOUNDARY_RES = [/([A-Z\d]+)([A-Z][a-z])/, /([a-z\d])([A-Z])/].freeze
+    FRACTION_RE = %r{^(\d+)/(\d+)$}.freeze
+    YMD_RE = /\A(\d{4}):(\d{2}):(\d{2})\b/.freeze
+    ZERO_DATE_RE = /\A[+:0 ]+\z/.freeze
 
     attr_reader :key, :display_key, :sym_key, :raw_value
 
@@ -20,23 +21,28 @@ class Exiftool
 
     def value
       @value ||= if lat_long?
-        as_lat_long
-      elsif date?
-        as_date
-      elsif fraction?
-        as_fraction
-      else
-        raw_value
-      end
+                   as_lat_long
+                 elsif date?
+                   as_date
+                 elsif fraction?
+                   as_fraction
+                 else
+                   raw_value
+                 end
     rescue StandardError => e
+      # :nocov:
       "Warning: Parsing '#{raw_value}' for attribute '#{key}' raised #{e.message}"
+      # :nocov:
     end
 
     def civil_date
-      if date? && !zero_date?
-        ymd = raw_value.scan(YMD_RE).first
-        Date.civil(*ymd.map { |ea| ea.to_i }) if ymd
-      end
+      return unless date? && !zero_date?
+
+      ymd = raw_value.scan(YMD_RE).first
+      return unless ymd
+
+      ymd_a = ymd.map(&:to_i)
+      Date.civil(*ymd_a) if Date.valid_civil?(*ymd_a)
     end
 
     private
@@ -47,10 +53,11 @@ class Exiftool
 
     def as_lat_long
       return raw_value if raw_value.is_a?(Numeric)
-      value, direction = raw_value.split(' ')
-      if value =~ /\A\d+\.?\d*\z/
-        value.to_f * (['S', 'W'].include?(direction) ? -1 : 1)
-      end
+
+      value, direction = raw_value.split
+      return unless value =~ /\A\d+\.?\d*\z/
+
+      value.to_f * (%w[S W].include?(direction) ? -1 : 1)
     end
 
     def date?
@@ -79,7 +86,7 @@ class Exiftool
 
     def as_fraction
       scan = raw_value.scan(FRACTION_RE).first
-      Rational(*scan.map { |ea| ea.to_i }) unless scan.nil?
+      Rational(*scan.map(&:to_i)) unless scan.nil?
     end
   end
 end

data/lib/exiftool/result.rb

@@ -1,6 +1,9 @@
+# frozen_string_literal: true
+
 require 'exiftool/field_parser'
 
 class Exiftool
+  # Exiftool Result
   class Result
     attr_reader :raw, :to_hash, :to_display_hash, :sym2display, :display2sym
 

data/lib/exiftool/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Exiftool
-  VERSION = Gem::Version.new('1.0.1')
+  VERSION = Gem::Version.new('1.2.3')
 end

metadata

@@ -1,14 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: exiftool
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.2.3
 platform: ruby
 authors:
 - Matthew McEachen
+- Sergey Morozov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-26 00:00:00.000000000 Z
+date: 2021-02-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -24,6 +25,62 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: minitest-great_expectations
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: minitest-reporters
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -39,7 +96,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -53,7 +110,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: yard
+  name: rubocop-minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -67,7 +124,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: minitest
+  name: rubocop-rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -81,7 +138,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: minitest-great_expectations
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.17.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.17.1
+- !ruby/object:Gem::Dependency
+  name: simplecov-console
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -95,7 +166,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: minitest-reporters
+  name: yard
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -110,7 +181,8 @@ dependencies:
         version: '0'
 description: Multiget ExifTool wrapper for ruby
 email:
-- matthew-github@mceachen.org
+- matthew+github@mceachen.org
+- morozgrafix@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -119,7 +191,7 @@ files:
 - lib/exiftool/field_parser.rb
 - lib/exiftool/result.rb
 - lib/exiftool/version.rb
-homepage: https://github.com/mceachen/exiftoolr
+homepage: https://github.com/exiftool-rb/exiftool.rb
 licenses:
 - MIT
 metadata: {}
@@ -131,16 +203,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements:
-- ExifTool (see http://www.sno.phy.queensu.ca/~phil/exiftool/)
-rubyforge_project: 
-rubygems_version: 2.5.1
+- ExifTool (see http://exiftool.org)
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
 summary: Multiget ExifTool wrapper for ruby