captive 1.0.2 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 030c1968b45eac6bb952f8bfd45b569820747ec338c6c6b155c6c7ba22f61a77
-  data.tar.gz: 8f9819a585d7633b3dab6741d0a650e555171bdbdf578fcb8cad34016c415593
+  metadata.gz: 0efb1ac420047333556fffa1f812da51c934d04ec1c4663abdd16dff37119bae
+  data.tar.gz: a82f64acf3da2b74cf5d72714939ee0559121b8ec43fbc2f92a5597c1f179a78
 SHA512:
-  metadata.gz: 5d4cd22b2a652db4198de20628d68465fe84d0727e0e4db8586635b4fadcc5860e3767998a4fbd9364c3cdaf8f8a62a53d85b2faf8983c54f9ef87c754e36bfe
-  data.tar.gz: 3823a40e5354abff877951bd636a6e2482d38af42b281ba95ec8cd5be3fe7ba71188dea4242c790b05b7526f04c7df5d9e947403c110357e183520f625ba8887
+  metadata.gz: a8a2c9496fcfe350776c912599e03495d3df9f907cfb4dbe85b9a531740c2a1b843e84fbf5eba9e1c27a0717b23a6b438a8e833ac0c6c45d653c67841b50f5ba
+  data.tar.gz: 8a369222333aaefdcb4cf5798536a2174baf344a46aab0f7e59bb687235865e9c5f29c7bd7322383b28c61736f3d2074f135cbcbb8d642dc8bc569fd2c558539

data/lib/captive/base.rb

@@ -14,6 +14,15 @@ module Captive
       def from_blob(blob:)
         new(cue_list: parse(blob: blob))
       end
+
+      def from_json(json:, mapping: {})
+        json = JSON.parse(json) if json.is_a?(String)
+        cues = json['cues'] || json[:cues]
+        raise InvalidJsonInput unless cues.is_a?(Array)
+
+        cues.map! { |cue_json| Cue.from_json(json: cue_json, mapping: mapping) }
+        new(cue_list: cues)
+      end
     end
 
     def self.included(base)

data/lib/captive/cue.rb

@@ -12,18 +12,35 @@ module Captive
     # List of Text Properties
     TEXT_PROPERTIES = [ALIGNMENT, COLOR, POSITION].freeze
 
-    attr_accessor :number, :text, :properties
+    attr_accessor :text, :properties
     attr_reader :start_time, :end_time
 
     # Creates a new Cue class denoting a subtitle.
-    def initialize(text: nil, start_time: nil, end_time: nil, cue_number: nil, properties: {})
+    def initialize(text: nil, start_time: nil, end_time: nil, properties: {})
       self.text = text
       self.start_time = start_time
       self.end_time = end_time
-      self.number = cue_number
       self.properties = properties || {}
     end
 
+    def self.from_json(json:, mapping: {})
+      schema = {}
+      %i[text! start_time! end_time! properties].each do |field|
+        field_name = field.to_s.delete('!')
+        schema[field] = mapping[field_name] || mapping[field_name.to_sym] || field_name.to_sym
+      end
+      data = {}
+      schema.each do |mask, mapper|
+        key = mask[-1] == '!' ? mask.to_s[0...-1].to_sym : mask
+        if key.to_s != mask.to_s && !(json.key?(mapper.to_s) || json.key?(mapper.to_sym))
+          raise InvalidJsonInput, "Cue missing field: #{mapper}"
+        end
+
+        data[key] = json[mapper.to_s] || json[mapper.to_sym]
+      end
+      new(**data)
+    end
+
     def start_time=(time)
       set_time(:start_time, time)
     end
@@ -68,11 +85,16 @@ module Captive
     end
 
     def as_json(**args)
-      if respond_to?(:instance_values) && instance_values.respond_to?(:as_json)
-        instance_values.as_json(**args)
-      else
-        instance_variables.each_with_object({}) { |key, hash| hash[key[1..-1]] = instance_variable_get(key) }
-      end
+      options = args.delete(:options) || {}
+      format = options['format'] || options[:format] || {}
+      mapping = (options['mapping'] || options[:mapping] || {}).map { |k, v| [k.to_s, v.to_s] }.to_h
+      obj = {
+        mapping['start_time'] || 'start_time' => format[:time] == :timecode ? milliseconds_to_timecode(start_time) : start_time,
+        mapping['end_time'] || 'end_time' => format[:time] == :timecode ? milliseconds_to_timecode(end_time) : end_time,
+        mapping['text'] || 'text' => text,
+        mapping['properties'] || 'properties' => properties,
+      }
+      obj.respond_to?(:as_json) ? obj.as_json(**args) : obj
     end
 
     private

data/lib/captive/error.rb

@@ -16,4 +16,7 @@ module Captive
   # Error denoting incorrect input to a method.
   class InvalidInput < CaptiveError
   end
+
+  class InvalidJsonInput < CaptiveError
+  end
 end

data/lib/captive/formats/srt.rb

@@ -19,13 +19,12 @@ module Captive
 
           raise InvalidSubtitle, "Invalid Cue Number at line #{count}" if /^\d+$/.match(line).nil?
 
-          cue = Cue.new(cue_number: line.to_i)
           state = :time
         when :time
           raise InvalidSubtitle, "Invalid Time Format at line #{count}" unless timecode?(line)
 
           start_time, end_time = line.split('-->').map(&:strip)
-          cue.set_times(
+          cue = Cue.new(
             start_time: format_time(start_time),
             end_time: format_time(end_time)
           )
@@ -51,8 +50,8 @@ module Captive
 
     def to_s
       string = String.new
-      @cue_list.each do |cue|
-        string << cue.number.to_s
+      cues.each_with_index do |cue, index|
+        string << (index + 1).to_s
         string << "\n"
         string << milliseconds_to_timecode(cue.start_time).gsub!('.', ',')
         string << ' --> '

data/lib/captive/formats/vtt.rb

@@ -14,7 +14,6 @@ module Captive
     def self.parse(blob:)
       cue_list = []
       lines = blob.split("\n")
-      cue_count = 1
       state = :new_cue
       cue = nil
       raise InvalidSubtitle, 'Invalid VTT Signature' unless validate_header(lines.shift)
@@ -42,8 +41,7 @@ module Captive
           elements = line.split
           start_time = elements[0]
           end_time = elements[2]
-          cue = Cue.new(cue_number: cue_count, start_time: start_time, end_time: end_time)
-          cue_count += 1
+          cue = Cue.new(start_time: start_time, end_time: end_time)
           state = :text
         when :text
           if line.empty?
@@ -73,7 +71,7 @@ module Captive
     def to_s
       string = VTT_HEADER.dup
       string << "\n\n"
-      @cue_list.each do |cue|
+      cues.each do |cue|
         string << milliseconds_to_timecode(cue.start_time)
         string << ' --> '
         string << milliseconds_to_timecode(cue.end_time)
@@ -86,7 +84,8 @@ module Captive
 
     # VTT Header tag matcher
     def self.validate_header(line)
-      !!line.strip.match(/^#{VTT_HEADER}/)
+      # Make sure BOM does not interfere with header detection
+      !!line.force_encoding('UTF-8').delete("\xEF\xBB\xBF").strip.match(/^#{VTT_HEADER}/)
     end
 
     # VTT Metadata tag matcher

data/lib/captive/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Captive
-  VERSION = '1.0.2'
+  VERSION = '1.1.2'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: captive
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.1.2
 platform: ruby
 authors:
 - mserran2
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-07-14 00:00:00.000000000 Z
+date: 2020-07-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -109,7 +109,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.9.2
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="