phtools 0.10.0 → 0.11.1

data/lib/phtools/exif_tagger/tags/_tag_hash_of_strings.rb

@@ -0,0 +1,39 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+# encoding: UTF-8
+# (c) ANB Andrew Bizyaev
+
+require_relative '_tag'
+
+module ExifTagger
+  module Tag
+    class TagHashOfStrings < Tag
+      private
+
+      def validate_hash_items
+      end
+
+      def validate_type
+        if @value.is_a?(Hash)
+          @value.each_value do |val|
+            validate_string_size(val)
+          end
+          unknown_keys = @value.keys - self.class::VALID_KEYS
+          unknown_keys.each do |k|
+            @errors << %(#{tag_name}: KEY '#{k}' is unknown)
+          end
+          missed_keys = self.class::VALID_KEYS - @value.keys
+          missed_keys.each do |k|
+            @errors << %(#{tag_name}: KEY '#{k}' is missed)
+          end
+          validate_hash_items if @errors.empty?
+        else
+          @errors << %(#{tag_name}: '#{@value}' is a wrong type \(#{@value.class}\))
+        end
+        return if @errors.empty?
+        @value_invalid << @value
+        @value = EMPTY
+      end
+    end
+  end
+end

data/lib/phtools/exif_tagger/tags/_tag_string.rb

@@ -0,0 +1,32 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+# encoding: UTF-8
+# (c) ANB Andrew Bizyaev
+
+require_relative '_tag'
+
+module ExifTagger
+  module Tag
+    class TagString < Tag
+      private
+
+      def get_from_raw
+        @raw_values.each_value do |value|
+          return value unless Tag.empty?(value)
+        end
+        EMPTY
+      end
+
+      def validate_type
+        if @value.is_a?(String)
+          validate_string_size(@value)
+        else
+          @errors << %(#{tag_name}: '#{@value}' is a wrong type \(#{@value.class}\))
+        end
+        return if @errors.empty?
+        @value_invalid << @value
+        @value = EMPTY
+      end
+    end
+  end
+end

data/lib/phtools/exif_tagger/tags/city.rb

@@ -1,38 +1,24 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # encoding: UTF-8
 # (c) ANB Andrew Bizyaev
 
-require_relative '_tag'
+require_relative '_tag_string'
 
 module ExifTagger
   module Tag
     # MWG:City, String[0,32]
     #   = IPTC:City XMP-photoshop:City XMP-iptcExt:LocationShownCity
-    class City < Tag
-      MAX_BYTESIZE = 32
-      EXIFTOOL_TAGS = %w(City LocationShownCity)
+    # Read sequence from mini_exiftool hash: City, LocationShownCity
 
-      def initialize(value_raw = '')
-        super(value_raw.to_s)
-      end
+    class City < TagString
+      MAX_BYTESIZE = 32
+      EXIFTOOL_TAGS = %w(City LocationShownCity).freeze
 
       private
 
-      def validate
-        bsize = @value.bytesize
-        if bsize > MAX_BYTESIZE
-          @errors << %(#{tag_name}: '#{@value}' ) +
-                     %(is #{bsize - MAX_BYTESIZE} bytes longer than allowed #{MAX_BYTESIZE})
-          @value_invalid << @value
-          @value = ''
-        end
-      end
-
       def generate_write_script_lines
-        @write_script_lines = []
-        unless @value.empty?
-          @write_script_lines << %Q(-MWG:City=#{@value})
-        end
+        @write_script_lines << %(-MWG:City=#{@value})
       end
     end
   end

data/lib/phtools/exif_tagger/tags/coded_character_set.rb

@@ -1,37 +1,22 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # encoding: UTF-8
 # (c) ANB Andrew Bizyaev
 
-require_relative '_tag'
+require_relative '_tag_string'
 
 module ExifTagger
   module Tag
     # -IPTC:CodedCharacterSet, string[0,32]!
-    class CodedCharacterSet < Tag
-      MAX_BYTESIZE = 32
-      EXIFTOOL_TAGS = %w(CodedCharacterSet)
 
-      def initialize(value_raw = '')
-        super(value_raw.to_s)
-      end
+    class CodedCharacterSet < TagString
+      MAX_BYTESIZE = 32
+      EXIFTOOL_TAGS = %w(CodedCharacterSet).freeze
 
       private
 
-      def validate
-        bsize = @value.bytesize
-        if bsize > MAX_BYTESIZE
-          @errors << %{#{tag_name}: '#{@value}' } +
-                     %{is #{bsize - MAX_BYTESIZE} bytes longer than allowed #{MAX_BYTESIZE}}
-          @value_invalid << @value
-          @value = ''
-        end
-      end
-
       def generate_write_script_lines
-        @write_script_lines = []
-        unless @value.empty?
-          @write_script_lines << %Q(-IPTC:CodedCharacterSet=#{@value})
-        end
+        @write_script_lines << %(-IPTC:CodedCharacterSet=#{@value})
       end
     end
   end

data/lib/phtools/exif_tagger/tags/collections.rb

@@ -1,44 +1,38 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # encoding: UTF-8
 # (c) ANB Andrew Bizyaev
 
-require_relative '_tag'
+require_relative '_tag_hash_of_strings'
 
 module ExifTagger
   module Tag
-    # Collections (struct+)
+    # XMP-mwg-coll:Collections (struct+)
     #   CollectionName
     #   CollectionURI
-    class Collections < Tag
-      VALID_KEYS = [:collection_name, :collection_uri]
-      EXIFTOOL_TAGS = %w(CollectionName CollectionURI)
-      def initialize(value_raw = {})
-        super
-      end
+    class Collections < TagHashOfStrings
+      MAX_BYTESIZE = 64
+      VALID_KEYS = [:collection_name, :collection_uri].freeze
+      EXIFTOOL_TAGS = %w(CollectionName CollectionURI).freeze
 
       private
 
-      def validate
-        unknown_keys = @value.keys - VALID_KEYS
-        unknown_keys.each do |k|
-          @errors << %{#{tag_name}: KEY '#{k}' is unknown}
-        end
-        missed_keys = VALID_KEYS - @value.keys
-        missed_keys.each do |k|
-          @errors << %{#{tag_name}: KEY '#{k}' is missed}
-        end
-        unless @errors.empty?
-          @value_invalid << @value
-          @value = {}
-        end
+      def get_from_raw
+        result = {}
+        v = @raw_values['CollectionName']
+        v = v.split('; ') if v.is_a?(String)
+        result[:collection_name] = v[0]
+
+        v = @raw_values['CollectionURI']
+        v = v.split('; ') if v.is_a?(String)
+        result[:collection_uri] = v[0]
+
+        result
       end
 
       def generate_write_script_lines
-        @write_script_lines = []
-        unless @value.empty?
-          @write_script_lines << %Q(-XMP-mwg-coll:Collections-={CollectionName=#{@value[:collection_name]}, CollectionURI=#{@value[:collection_uri]}})
-          @write_script_lines << %Q(-XMP-mwg-coll:Collections+={CollectionName=#{@value[:collection_name]}, CollectionURI=#{@value[:collection_uri]}})
-        end
+        @write_script_lines << %(-XMP-mwg-coll:Collections-={CollectionName=#{@value[:collection_name]}, CollectionURI=#{@value[:collection_uri]}})
+        @write_script_lines << %(-XMP-mwg-coll:Collections+={CollectionName=#{@value[:collection_name]}, CollectionURI=#{@value[:collection_uri]}})
       end
     end
   end

data/lib/phtools/exif_tagger/tags/copyright.rb

@@ -1,38 +1,22 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # encoding: UTF-8
 # (c) ANB Andrew Bizyaev
 
-require_relative '_tag'
+require_relative '_tag_string'
 
 module ExifTagger
   module Tag
     # NMG:Copyright, string[0,128]
-    #   = EXIF:Copyright IPTC:CopyrightNotice XMP-dc:Rights
-    class Copyright < Tag
+    #   EXIF:Copyright, IPTC:CopyrightNotice, XMP-dc:Rights
+    class Copyright < TagString
       MAX_BYTESIZE = 128
-      EXIFTOOL_TAGS = %w(Copyright CopyrightNotice Rights)
-
-      def initialize(value_raw = '')
-        super(value_raw.to_s)
-      end
+      EXIFTOOL_TAGS = %w(Copyright CopyrightNotice Rights).freeze
 
       private
 
-      def validate
-        bsize = @value.bytesize
-        if bsize > MAX_BYTESIZE
-          @errors << %{#{tag_name}: '#{@value}' } +
-                     %{is #{bsize - MAX_BYTESIZE} bytes longer than allowed #{MAX_BYTESIZE}}
-          @value_invalid << @value
-          @value = ''
-        end
-      end
-
       def generate_write_script_lines
-        @write_script_lines = []
-        unless @value.empty?
-          @write_script_lines << %Q(-MWG:Copyright=#{@value})
-        end
+        @write_script_lines << %(-MWG:Copyright=#{@value})
       end
     end
   end

data/lib/phtools/exif_tagger/tags/country.rb

@@ -1,39 +1,22 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # encoding: UTF-8
 # (c) ANB Andrew Bizyaev
 
-require_relative '_tag'
+require_relative '_tag_string'
 
 module ExifTagger
   module Tag
     # MWG:Country, String[0,64]
-    #   = IPTC:Country-PrimaryLocationName XMP-photoshop:Country
-    #       XMP-iptcExt:LocationShownCountryName
-    class Country < Tag
+    #  IPTC:Country-PrimaryLocationName, XMP-photoshop:Country, XMP-iptcExt:LocationShownCountryName
+    class Country < TagString
       MAX_BYTESIZE = 64
-      EXIFTOOL_TAGS = %w(Country-PrimaryLocationName Country LocationShownCountryName)
-
-      def initialize(value_raw = '')
-        super(value_raw.to_s)
-      end
+      EXIFTOOL_TAGS = %w(Country-PrimaryLocationName Country LocationShownCountryName).freeze
 
       private
 
-      def validate
-        bsize = @value.bytesize
-        if bsize > MAX_BYTESIZE
-          @errors << %(#{tag_name}: '#{@value}' ) +
-                     %(is #{bsize - MAX_BYTESIZE} bytes longer than allowed #{MAX_BYTESIZE})
-          @value_invalid << @value
-          @value = ''
-        end
-      end
-
       def generate_write_script_lines
-        @write_script_lines = []
-        unless @value.empty?
-          @write_script_lines << %Q(-MWG:Country=#{@value})
-        end
+        @write_script_lines << %(-MWG:Country=#{@value})
       end
     end
   end

data/lib/phtools/exif_tagger/tags/country_code.rb

@@ -1,37 +1,21 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # encoding: UTF-8
 # (c) ANB Andrew Bizyaev
 
-require_relative '_tag'
+require_relative '_tag_string'
 
 module ExifTagger
   module Tag
     # XMP-iptcExt:LocationShownCountryCode, String
-    class CountryCode < Tag
+    class CountryCode < TagString
       MAX_BYTESIZE = 32
-      EXIFTOOL_TAGS = %w(LocationShownCountryCode)
-
-      def initialize(value_raw = '')
-        super(value_raw.to_s)
-      end
+      EXIFTOOL_TAGS = %w(LocationShownCountryCode).freeze
 
       private
 
-      def validate
-        bsize = @value.bytesize
-        if bsize > MAX_BYTESIZE
-          @errors << %(#{tag_name}: '#{@value}' ) +
-                     %(is #{bsize - MAX_BYTESIZE} bytes longer than allowed #{MAX_BYTESIZE})
-          @value_invalid << @value
-          @value = ''
-        end
-      end
-
       def generate_write_script_lines
-        @write_script_lines = []
-        unless @value.empty?
-          @write_script_lines << %Q(-XMP-iptcExt:LocationShownCountryCode=#{@value})
-        end
+        @write_script_lines << %(-XMP:LocationShownCountryCode=#{@value})
       end
     end
   end

data/lib/phtools/exif_tagger/tags/create_date.rb

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # encoding: UTF-8
 # (c) ANB Andrew Bizyaev
 
@@ -7,25 +8,28 @@ require_relative '_tag_date'
 module ExifTagger
   module Tag
     # -MWG:CreateDate:
-    #    EXIF:CreateDate
-    #    EXIF:SubSecTimeDigitized
-    #    IPTC:DigitalCreationDate
-    #    IPTC:DigitalCreationTime
-    #    XMP-xmp:CreateDate
+    #   EXIF:CreateDate (EXIF:SubSecTimeDigitized),
+    #   IPTC:DigitalCreationDate+IPTC:DigitalCreationTime,
+    #   XMP-xmp:CreateDate
     # creation date of the digital representation
     class CreateDate < TagDate
-      EXIFTOOL_TAGS = %w(CreateDate SubSecTimeDigitized DigitalCreationDate DigitalCreationTime)
+      MAX_BYTESIZE = 32
+      EXIFTOOL_TAGS = %w(CreateDate SubSecTimeDigitized DigitalCreationDate DigitalCreationTime).freeze
 
       private
 
+      def get_from_raw
+        return @raw_values['CreateDate'] unless Tag.empty?(@raw_values['CreateDate'])
+        return make_date_from(@raw_values['DigitalCreationDate'], @raw_values['DigitalCreationTime']) unless Tag.empty?(@raw_values['DigitalCreationDate'])
+        EMPTY
+      end
+
       def generate_write_script_lines
-        @write_script_lines = []
-        case
-        when @value.kind_of?(String) && !@value.empty?
-          @write_script_lines << %Q(-MWG:CreateDate=#{@value})
-        when @value.kind_of?(DateTime) || @value.kind_of?(Time)
-          @write_script_lines << %Q(-MWG:CreateDate=#{@value.strftime('%F %T')})
-        end
+        @write_script_lines << if @value.is_a?(DateTime)
+                                 %(-MWG:CreateDate=#{@value.strftime('%F %T')})
+                               else
+                                 %(-MWG:CreateDate=#{@value})
+                               end
       end
     end
   end

data/lib/phtools/exif_tagger/tags/creator.rb

@@ -1,41 +1,30 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # encoding: UTF-8
 # (c) ANB Andrew Bizyaev
 
-require_relative '_tag'
+require_relative '_tag_array_of_strings'
 
 module ExifTagger
   module Tag
     # MWG:Creator, string[0,32]+, List of strings
-    #   = EXIF:Artist, IPTC:By-line, XMP-dc:Creator
-    class Creator < Tag
-      MAX_BYTESIZE = 32
-      EXIFTOOL_TAGS = %w(Artist By-line Creator)
+    #   EXIF:Artist, IPTC:By-line, XMP-dc:Creator
+    # exiftool types:
+    #   Artist = String "aaa; bbb"
+    #   By-line = Array ["aaa", "bbb"] OR String "aaa"
+    #   Creator = Array ["aaa", "bbb"] OR String "aaa"
 
-      def initialize(value_raw = [])
-        super(Array(value_raw).flatten.map { |i| i.to_s })
-      end
+    class Creator < TagArrayOfStrings
+      MAX_BYTESIZE = 32
+      EXIFTOOL_TAGS = %w(Artist By-line Creator).freeze
 
       private
 
-      def validate
-        @value.each do |v|
-          bsize = v.bytesize
-          if bsize > MAX_BYTESIZE
-            @errors << %(#{tag_name}: '#{v}' ) +
-                       %(is #{bsize - MAX_BYTESIZE} bytes longer than allowed #{MAX_BYTESIZE})
-            @value_invalid << v
-          end
-        end
-        @value = @value - @value_invalid
-      end
-
       def generate_write_script_lines
-        @write_script_lines = []
-        unless @value.empty?
-          @value.each do |o|
-            @write_script_lines << %Q(-MWG:Creator-=#{o})
-            @write_script_lines << %Q(-MWG:Creator+=#{o})
+        @value.each do |o|
+          unless Tag.empty?(o)
+            @write_script_lines << %(-MWG:Creator-=#{o})
+            @write_script_lines << %(-MWG:Creator+=#{o})
           end
         end
       end

data/lib/phtools/exif_tagger/tags/date_time_original.rb

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # encoding: UTF-8
 # (c) ANB Andrew Bizyaev
 
@@ -7,25 +8,29 @@ require_relative '_tag_date'
 module ExifTagger
   module Tag
     # -MWG:DateTimeOriginal:
-    #    EXIF:DateTimeOriginal
-    #    EXIF:SubSecTimeOriginal
-    #    IPTC:DateCreated
-    #    IPTC:TimeCreated
-    #    XMP-photoshop:DateCreated
+    #    EXIF:DateTimeOriginal, (EXIF:SubSecTimeOriginal), IPTC:DateCreated + IPTC:TimeCreated,    XMP-photoshop:DateCreated
     # creation date of the intellectual content being shown
     class DateTimeOriginal < TagDate
-      EXIFTOOL_TAGS = %w(DateTimeOriginal SubSecTimeOriginal DateCreated TimeCreated)
+      MAX_BYTESIZE = 32
+      EXIFTOOL_TAGS = %w(DateTimeOriginal SubSecTimeOriginal DateCreated TimeCreated).freeze
 
       private
 
-      def generate_write_script_lines
-        @write_script_lines = []
-        case
-        when @value.kind_of?(String) && !@value.empty?
-          @write_script_lines << %Q(-MWG:DateTimeOriginal=#{@value})
-        when @value.kind_of?(DateTime) || @value.kind_of?(Time)
-          @write_script_lines << %Q(-MWG:DateTimeOriginal=#{@value.strftime('%F %T')})
+      def get_from_raw
+        return @raw_values['DateTimeOriginal'] unless Tag.empty?(@raw_values['DateTimeOriginal'])
+        unless Tag.empty?(@raw_values['DateCreated'])
+          return @raw_values['DateCreated'] if @raw_values['DateCreated'].is_a?(DateTime)
+          return make_date_from(@raw_values['DateCreated'], @raw_values['TimeCreated']) if @raw_values['DateCreated'].is_a?(String)
         end
+        EMPTY
+      end
+
+      def generate_write_script_lines
+        @write_script_lines << if @value.is_a?(DateTime)
+                                 %(-MWG:DateTimeOriginal=#{@value.strftime('%F %T')})
+                               else
+                                 %(-MWG:DateTimeOriginal=#{@value})
+                               end
       end
     end
   end

data/lib/phtools/exif_tagger/tags/gps_created.rb

@@ -1,8 +1,9 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # encoding: UTF-8
 # (c) ANB Andrew Bizyaev
 
-require_relative '_tag'
+require_relative '_tag_hash_of_strings'
 
 module ExifTagger
   module Tag
@@ -13,9 +14,10 @@ module ExifTagger
     # GPSLongitudeRef (string[2] 'E' = East, 'W' = West)
     # GPSAltitude (rational64u)
     # GPSAltitudeRef (int8u 0 = Above Sea Level, 1 = Below Sea Level)
-    class GpsCreated < Tag
-      VALID_KEYS = [:gps_latitude, :gps_latitude_ref, :gps_longitude,
-                    :gps_longitude_ref, :gps_altitude, :gps_altitude_ref]
+    class GpsCreated < TagHashOfStrings
+      MAX_BYTESIZE = 64
+      VALID_KEYS = [:gps_latitude, :gps_latitude_ref, :gps_longitude, :gps_longitude_ref, :gps_altitude, :gps_altitude_ref].freeze
+      VALID_VALUES = { gps_latitude_ref: %w(N S North South), gps_longitude_ref: %w(E W East West), gps_altitude_ref: ['Above Sea Level', 'Below Sea Level', '0', '1'] }.freeze
       EXIFTOOL_TAGS = %w(
         GPSPosition
         GPSLatitude
@@ -24,54 +26,29 @@ module ExifTagger
         GPSLongitudeRef
         GPSAltitude
         GPSAltitudeRef
-      )
-
-      def initialize(value_raw = {})
-        # TODO: value = value_raw.each { |k, v| value_raw[k] = v.to_s }
-        super
-      end
+      ).freeze
 
       private
 
-      def validate
-        unknown_keys = @value.keys - VALID_KEYS
-        unknown_keys.each do |k|
-          @errors << %(#{tag_name}: KEY '#{k}' is unknown)
-        end
-        missed_keys = VALID_KEYS - @value.keys
-        missed_keys.each do |k|
-          @errors << %(#{tag_name}: KEY '#{k}' is missed)
-        end
-        if @errors.empty?
-          valid_values = ['N', 'S']
-          unless valid_values.include? @value[:gps_latitude_ref]
-            @errors << %(#{tag_name}: value of 'gps_latitude_ref' should be one of #{valid_values})
-          end
-          valid_values = ['E', 'W']
-          unless valid_values.include? @value[:gps_longitude_ref]
-            @errors << %(#{tag_name}: value of 'gps_longitude_ref' should be one of #{valid_values})
-          end
-          valid_values = ['Above Sea Level', 'Below Sea Level']
-          unless valid_values.include? @value[:gps_altitude_ref]
-            @errors << %(#{tag_name}: value of 'gps_altitude_ref' should be one of #{valid_values})
+      def get_from_raw
+        { gps_latitude: @raw_values['GPSLatitude'], gps_latitude_ref: @raw_values['GPSLatitudeRef'], gps_longitude: @raw_values['GPSLongitude'], gps_longitude_ref: @raw_values['GPSLongitudeRef'], gps_altitude: @raw_values['GPSAltitude'], gps_altitude_ref: @raw_values['GPSAltitudeRef'] }
+      end
+
+      def validate_hash_items
+        VALID_VALUES.each do |vv_key, vv_val|
+          unless vv_val.include?(@value[vv_key])
+            @errors << %(#{tag_name}: value of '#{vv_key}' should be one of #{vv_val})
           end
         end
-        unless @errors.empty?
-          @value_invalid << @value
-          @value = {}
-        end
       end
 
       def generate_write_script_lines
-        @write_script_lines = []
-        unless @value.empty?
-          @write_script_lines << %Q(-GPSLatitude="#{@value[:gps_latitude]}")
-          @write_script_lines << %Q(-GPSLatitudeRef=#{@value[:gps_latitude_ref]})
-          @write_script_lines << %Q(-GPSLongitude="#{@value[:gps_longitude]}")
-          @write_script_lines << %Q(-GPSLongitudeRef=#{@value[:gps_longitude_ref]})
-          @write_script_lines << %Q(-GPSAltitude=#{@value[:gps_altitude]})
-          @write_script_lines << %Q(-GPSAltitudeRef=#{@value[:gps_altitude_ref]})
-        end
+        @write_script_lines << %(-GPSLatitude="#{@value[:gps_latitude]}")
+        @write_script_lines << %(-GPSLatitudeRef=#{@value[:gps_latitude_ref]})
+        @write_script_lines << %(-GPSLongitude="#{@value[:gps_longitude]}")
+        @write_script_lines << %(-GPSLongitudeRef=#{@value[:gps_longitude_ref]})
+        @write_script_lines << %(-GPSAltitude=#{@value[:gps_altitude]})
+        @write_script_lines << %(-GPSAltitudeRef=#{@value[:gps_altitude_ref]})
       end
     end
   end

data/lib/phtools/exif_tagger/tags/image_unique_id.rb

@@ -1,46 +1,31 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 # encoding: UTF-8
 # (c) ANB Andrew Bizyaev
 
-require_relative '_tag'
+require_relative '_tag_string'
 
 module ExifTagger
   module Tag
     # ImageUniqueID, String
-    class ImageUniqueId < Tag
-      MAX_BYTESIZE = 32 # no limit in EXIF spec
-      EXIFTOOL_TAGS = %w(ImageUniqueID)
+    class ImageUniqueId < TagString
+      MAX_BYTESIZE = 32
+      EXIFTOOL_TAGS = %w(ImageUniqueID).freeze
 
-      def initialize(value_raw = [])
-        super(value_raw.to_s)
-      end
+      private
 
-      def check_for_warnings(original_values: {})
+      def validate_vs_previous
         @warnings = []
-        v = original_values[EXIFTOOL_TAGS[0]]
-        if v =~ /(\d{8}-\S+)/
-          @warnings << "#{tag_name} has original correct value: '#{v}'"
+        return if @previous.nil?
+        val = @previous.raw_values[EXIFTOOL_TAGS[0]]
+        if val =~ /(\d{8}-\S+)/
+          @warnings << "#{tag_name} has original value: '#{val}'"
         end
         @warnings.freeze
       end
 
-      private
-
-      def validate
-        bsize = @value.bytesize
-        if bsize > MAX_BYTESIZE
-          @errors << %(#{tag_name}: '#{@value}' ) +
-                     %(is #{bsize - MAX_BYTESIZE} bytes longer than allowed #{MAX_BYTESIZE})
-          @value_invalid << @value
-          @value = ''
-        end
-      end
-
       def generate_write_script_lines
-        @write_script_lines = []
-        unless @value.empty?
-          @write_script_lines << %Q(-ImageUniqueID=#{@value})
-        end
+        @write_script_lines << %(-ImageUniqueID=#{@value})
       end
     end
   end