jekyll_img 0.1.1 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 364a5201d2ce55db3436164ed8f4430c22533e4cea3e8b188794b919e56c3b17
-  data.tar.gz: df95180013a8deaa425a364474eb3afd2fffce5b859dafe7e9a0fe0989b4676b
+  metadata.gz: 8d314eb97784260ac71168aa8dc6da63e0dec2238d75a9e57b4662ba103cf47c
+  data.tar.gz: b353c583b4334d7c49c1961b6c383b34014449834f852e9f14e9eeccb31b78b4
 SHA512:
-  metadata.gz: fe8432b3c99ba15ba90c5cdb61c4cb70de160f2fd1cbb86f44a52b581205bbad04ca28f2b102279c469680f136ddf41e05d302ce8c5b54418ee72346d2596ca8
-  data.tar.gz: 880d4924b96382076caef882d1585c1b76a5f5a5cfb8c651c0420ac5973a80c96fbe38de873f1b7191fcdbd9257fd5843598cbf43671bf8cbfe6bb4a722ab315
+  metadata.gz: 30ae18bc4ff268d928b503f55e7ba847a7ae767864a0dad41ffc80728afaab88aae8b8e17b4f5d2381ff75c0ecfb7a4a5f4cf3dd399fe05a8a1268f1d1a32505
+  data.tar.gz: 91bfb7286db21ac8933a823cdaa2c6af2d31dc1734d9dd0b2222bf90a515a1b04d3119f21b438badd7fddf969ed9f1a2c93e2642763cd1af3a1a5b00b9807bea

data/.rubocop.yml

@@ -1,3 +1,13 @@
+require:
+  # - rubocop-jekyll
+  - rubocop-md
+  - rubocop-performance
+  - rubocop-rake
+  - rubocop-rspec
+
+# inherit_gem:
+#  rubocop-jekyll: .rubocop.yml
+
 AllCops:
   Exclude:
       - vendor/**/*
@@ -7,17 +17,33 @@ AllCops:
   NewCops: enable
   TargetRubyVersion: 2.6
 
-Layout/HashAlignment:
+Gemspec/DeprecatedAttributeAssignment:
+  Enabled: false
+
+Gemspec/RequireMFA:
   Enabled: false
 
+Layout/InitialIndentation:
+  Exclude:
+    - README.md
+
+Layout/HashAlignment:
+  EnforcedColonStyle: table
+  Exclude:
+    - jekyll_img.gemspec
+
 Layout/LineLength:
   Max: 150
 
 Layout/MultilineMethodCallIndentation:
   Enabled: false
 
+Lint/RedundantCopDisableDirective:
+  Exclude:
+    - jekyll_img.gemspec
+
 Metrics/AbcSize:
-  Max: 40
+  Max: 45
 
 Metrics/BlockLength:
   Max: 30
@@ -34,8 +60,40 @@ Metrics/MethodLength:
 Metrics/PerceivedComplexity:
   Max: 10
 
+Naming/FileName:
+  Exclude:
+    - Rakefile
+
+Naming/MemoizedInstanceVariableName:
+  Enabled: false
+
+RSpec/ExampleLength:
+  Enabled: false
+
+RSpec/MultipleExpectations:
+  Enabled: false
+
+Style/CommandLiteral:
+  Enabled: false
+
+Style/CommentedKeyword:
+  Enabled: false
+
+Style/Documentation:
+  Enabled: false
+
 Style/FrozenStringLiteralComment:
   Enabled: false
 
-Style/TrailingCommaInHashLiteral:
+Style/StringConcatenation:
+  Exclude:
+    - spec/**/*
+
+Style/StringLiterals:
   Enabled: false
+
+Style/StringLiteralsInInterpolation:
+  Enabled: false
+
+Style/TrailingCommaInHashLiteral:
+  EnforcedStyleForMultiline: comma

data/CHANGELOG.md

@@ -1,2 +1,9 @@
+## 0.1.3 / 2023-02-22
+  * Added `img/continue_on_error` configuration parameter.
+
+## 0.1.2 / 2023-02-14
+  * Fixed `img_props.rb:91:in `size_unit_specified?': undefined method `end_with?' for false:FalseClass (NoMethodError)`
+
+
 ## 0.1.1 / 2023-02-12
   * Initial release

data/README.md

@@ -44,6 +44,16 @@ $ demo/_bin/debug -r
 CSS classes referenced by the `jekyll_img` plugin are at the bottom of [demo/assets/css/style.css](demo/assets/css/style.css). CSS marker classes are included, so CSS selectors can be used for additional styling.
 
 
+## Configuration
+By default, any errors cause Jekyll to abort.
+You can allow Jekyll to continue by setting the following in `_config.yml`:
+
+```yaml
+img:
+  continue_on_error: true
+```
+
+
 ## Design
 The most significant design issue was the decision that image size and formatting should not change
 whether it had a caption.

data/jekyll_img.gemspec

@@ -29,8 +29,5 @@ Gem::Specification.new do |spec| # rubocop:disable Metrics/BlockLength
   spec.version = JekyllImgVersion::VERSION
 
   spec.add_dependency 'jekyll', '>= 3.5.0'
-  spec.add_dependency 'jekyll_plugin_support', '>= 0.3.1'
-
-  spec.add_development_dependency 'rspec'
-  spec.add_development_dependency 'rspec-match_ignoring_whitespace'
+  spec.add_dependency 'jekyll_plugin_support', '~> 0.5.1'
 end

data/lib/img_props.rb

@@ -1,10 +1,13 @@
+class ImgError < StandardError; end
+
 # Properties from user
 # All methods are idempotent.
 # attr_ methods can be called after compute_dependant_properties
 # All methods except compute_dependant_properties can be called in any order
 class ImgProperties
-  attr_accessor :align, :alt, :attr_wrapper_align_class, :caption, :classes, :id, :img_display, :nofollow, \
-                :src, :size, :style, :target, :title, :url, :wrapper_class, :wrapper_style
+  attr_accessor :align, :alt, :attr_wrapper_align_class, :caption, :classes, :continue_on_error, \
+                :id, :img_display, :nofollow, :src, :size, :style, :target, :title, \
+                :url, :wrapper_class, :wrapper_style
 
   SIZES = %w[eighthsize fullsize halfsize initial quartersize].freeze
 
@@ -26,10 +29,12 @@ class ImgProperties
   end
 
   def attr_size_class
-    return nil if @size.nil? || size_unit_specified?
+    return nil if @size == false || @size.nil? || size_unit_specified?
 
-    abort "'#{@size}' is not a recognized size; must be one of #{SIZES.join(', ')}, or an explicit unit." \
-      unless SIZES.include?(@size)
+    unless SIZES.include?(@size)
+      msg = "'#{@size}' is not a recognized size; must be one of #{SIZES.join(', ')}, or an explicit unit."
+      raise ImgError, msg
+    end
     @size
   end
 
@@ -60,11 +65,11 @@ class ImgProperties
     @img_display = @caption && @size ? 'imgBlock' : 'imgFlex'
 
     @alt   ||= @caption || @title
-    @title ||= @caption || @alt # rubocop:disable Naming/MemoizedInstanceVariableName
+    @title ||= @caption || @alt
   end
 
   def src_png
-    abort 'src parameter was not specified' if @src.to_s.empty?
+    raise ImgError, "The 'src' parameter was not specified" if @src.to_s.empty?
 
     @src.gsub('.webp', '.png')
   end
@@ -78,7 +83,7 @@ class ImgProperties
 
   def setup_src
     @src = @src.to_s.strip
-    abort 'src parameter was not specified' if @src.empty?
+    raise ImgError, "The 'src' parameter was not specified", [] if @src.empty?
 
     @src = "/assets/images/#{@src}" unless ImgProperties.local_path?(@src) || url?(@src)
   end
@@ -86,7 +91,7 @@ class ImgProperties
   UNITS = %w[Q ch cm em dvh dvw ex in lh lvh lvw mm pc px pt rem rlh svh svw vb vh vi vmax vmin vw %].freeze
 
   def size_unit_specified?
-    return false if @size.to_s.strip.empty?
+    return false if @size == false || @size.to_s.strip.empty?
 
     @size.end_with?(*UNITS)
   end

data/lib/jekyll_img/version.rb

@@ -1,3 +1,3 @@
 module JekyllImgVersion
-  VERSION = '0.1.1'.freeze
+  VERSION = '0.1.3'.freeze
 end

data/lib/jekyll_img.rb

@@ -14,28 +14,41 @@ end
 module Jekyll
   # Plugin implementation
   class Img < JekyllSupport::JekyllTag
-    def render_impl
+    include JekyllImgVersion
+
+    def render_impl # rubocop:disable Metrics/AbcSize
+      config = @config['img']
+      @continue_on_error = config['continue_on_error'] == true if config
+
       props = ImgProperties.new
-      props.align         = @helper.parameter_specified?('align') || 'inline'
-      props.alt           = @helper.parameter_specified? 'alt'
-      props.caption       = @helper.parameter_specified? 'caption'
-      props.classes       = @helper.parameter_specified? 'class'
-      props.id            = @helper.parameter_specified? 'id'
-      props.nofollow      = @helper.parameter_specified? 'nofollow'
-      props.size          = @helper.parameter_specified?('size') || @helper.parameter_specified?('_size')
-      props.src           = @helper.parameter_specified? 'src'
-      props.style         = @helper.parameter_specified? 'style'
-      props.target        = @helper.parameter_specified? 'target'
-      props.title         = @helper.parameter_specified? 'title'
-      props.url           = @helper.parameter_specified? 'url'
-      props.wrapper_class = @helper.parameter_specified? 'wrapper_class'
-      props.wrapper_style = @helper.parameter_specified? 'wrapper_style'
+      props.align             = @helper.parameter_specified?('align') || 'inline'
+      props.alt               = @helper.parameter_specified? 'alt'
+      props.caption           = @helper.parameter_specified? 'caption'
+      props.classes           = @helper.parameter_specified? 'class'
+      props.continue_on_error = @continue_on_error
+      props.id                = @helper.parameter_specified? 'id'
+      props.nofollow          = @helper.parameter_specified? 'nofollow'
+      props.size              = @helper.parameter_specified?('size') || @helper.parameter_specified?('_size')
+      props.src               = @helper.parameter_specified? 'src'
+      props.style             = @helper.parameter_specified? 'style'
+      props.target            = @helper.parameter_specified? 'target'
+      props.title             = @helper.parameter_specified? 'title'
+      props.url               = @helper.parameter_specified? 'url'
+      props.wrapper_class     = @helper.parameter_specified? 'wrapper_class'
+      props.wrapper_style     = @helper.parameter_specified? 'wrapper_style'
 
       @builder = ImgBuilder.new(props)
       @builder.to_s
+    rescue ImgError => e
+      msg = <<~END_ERR
+        #{e.message} on line #{@line_number} (after front matter) of #{@page['path']}.
+        The offending argument string was: #{@argument_string}
+      END_ERR
+      raise ImgError, msg.red, [] unless @continue_on_error
+
+      @logger.warn msg
     end
+
+    JekyllPluginHelper.register(self, ImgModule::PLUGIN_NAME)
   end
 end
-
-PluginMetaLogger.instance.info { "Loaded #{ImgModule::PLUGIN_NAME} v0.1.0 plugin." }
-Liquid::Template.register_tag(ImgModule::PLUGIN_NAME, Jekyll::Img)

data/spec/img_builder_spec.rb

@@ -4,11 +4,11 @@ require_relative '../lib/img_props'
 
 # Test ImgProperties
 class ImgPropertiesTest
-  RSpec.describe ImgBuilder do # rubocop:disable Metrics/BlockLength
+  RSpec.describe ImgBuilder do
     it 'generates a default img' do
       props = ImgProperties.new
       props.src = 'blah.webp'
-      builder = ImgBuilder.new(props)
+      builder = described_class.new(props)
       picture = <<~END_IMG
         <div class='imgWrapper imgFlex' style=' '>
           <picture class='imgPicture'>
@@ -22,7 +22,7 @@ class ImgPropertiesTest
         </div>
       END_IMG
 
-      expect(builder.send(:generate_figure_caption)).to be nil
+      expect(builder.send(:generate_figure_caption)).to be_nil
       expect(builder.send(:generate_wrapper)).to        match_ignoring_whitespace(picture)
     end
 
@@ -31,7 +31,7 @@ class ImgPropertiesTest
       props.caption = 'This is a caption'
       props.size = '123px'
       props.src = 'blah.webp'
-      builder = ImgBuilder.new(props)
+      builder = described_class.new(props)
 
       caption = <<~END_CAPTION
         <figcaption class='imgFigCaption '>

data/spec/img_props_spec.rb

@@ -1,39 +1,38 @@
 require_relative '../lib/img_props'
 
-# Test ImgProperties
 class ImgProperitesTest
-  RSpec.describe ImgProperties do # rubocop:disable Metrics/BlockLength
+  RSpec.describe ImgProperties do
     it 'detects relative paths' do
-      expect(ImgProperties.local_path?('abcdef')).to be false
-      expect(ImgProperties.local_path?('./abc')).to  be true
-      expect(ImgProperties.local_path?('/abc')).to   be true
+      expect(described_class.local_path?('abcdef')).to be false
+      expect(described_class.local_path?('./abc')).to  be true
+      expect(described_class.local_path?('/abc')).to   be true
     end
 
     it 'has other class methods' do
-      props = ImgProperties.new
+      props = described_class.new
       expect(props.send(:size_unit_specified?)).to be false
       expect(props.send(:url?, 'blah')).to         be false
       expect(props.send(:url?, 'http://blah')).to  be true
     end
 
     it 'does not generate attributes for most empty properties' do
-      props = ImgProperties.new
-      expect(props.attr_alt).to           be nil
+      props = described_class.new
+      expect(props.attr_alt).to           be_nil
       expect(props.attr_img_classes).to   eq('rounded shadow')
-      expect(props.attr_id).to            be nil
-      expect(props.attr_nofollow).to      be nil
-      expect(props.attr_size_class).to    be nil
+      expect(props.attr_id).to            be_nil
+      expect(props.attr_nofollow).to      be_nil
+      expect(props.attr_size_class).to    be_nil
       expect(props.attr_style_img).to     eq("style='width: 100%; '")
       expect(props.attr_target).to        eq(" target='_blank'")
-      expect(props.attr_title).to         be nil
-      expect(props.attr_width_style).to   be nil
+      expect(props.attr_title).to         be_nil
+      expect(props.attr_width_style).to   be_nil
 
       props.compute_dependant_properties
       expect(props.attr_wrapper_align_class).to eq('inline')
     end
 
     it 'raises exception if src was not specified' do
-      props = ImgProperties.new
+      props = described_class.new
       expect { props.src_png }.to raise_error(SystemExit)
       expect { props.send(:setup_src) }.to raise_error(SystemExit)
 
@@ -50,8 +49,8 @@ class ImgProperitesTest
       expect(props.src).to eq('/absolute/path.webp')
     end
 
-    it 'generates proper simple attributes' do # rubocop:disable Metrics/BlockLength
-      props = ImgProperties.new
+    it 'generates proper simple attributes' do
+      props = described_class.new
 
       props.alt = 'blah'
       expect(props.attr_alt).to eq("alt='blah'")
@@ -69,18 +68,18 @@ class ImgProperitesTest
       expect(props.attr_size_class).to eq('initial')
 
       props.size = '100px'
-      expect(props.attr_size_class).to be nil
+      expect(props.attr_size_class).to be_nil
       expect(props.attr_style_img).to eq("style='max-width: 100px;'")
       expect(props.attr_width_style).to eq('width: 100px;')
 
       props.size = '10%'
-      expect(props.attr_size_class).to be nil
+      expect(props.attr_size_class).to be_nil
       expect(props.attr_style_img).to eq("style='max-width: 10%;'")
       expect(props.attr_width_style).to eq('width: 10%;')
 
       props.size = 'fullsize'
       expect(props.attr_size_class).to eq('fullsize')
-      expect(props.attr_width_style).to be nil
+      expect(props.attr_width_style).to be_nil
 
       props.style = 'width: 30rem;'
       expect(props.attr_style_img).to eq("style='width: 30rem;'")
@@ -93,12 +92,12 @@ class ImgProperitesTest
 
       props.size = '100px'
       props.caption = 'A caption'
-      expect(props.attr_size_class).to be nil
+      expect(props.attr_size_class).to be_nil
       expect(props.attr_width_style).to eq('width: 100px;')
     end
 
     it 'generates proper alignment attributes' do
-      props = ImgProperties.new
+      props = described_class.new
 
       props.align = 'inline'
       props.compute_dependant_properties

data/spec/jekyll_img_spec.rb

@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 require 'jekyll'
 require 'jekyll_plugin_logger'
 require 'rspec/match_ignoring_whitespace'
@@ -32,7 +30,7 @@ class TestParseContext < Liquid::ParseContext
     @line_number = 123
 
     @registers = Registers.new(
-      { },
+      {},
       SiteMock.new
     )
   end
@@ -57,7 +55,7 @@ class MyTest
 
     it 'has no cite or url' do
       helper.reinitialize('src="./blah.webp"')
-      quote = Jekyll::Img.send(
+      quote = described_class.send(
         :new,
         'img',
         helper.markup.dup,

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll_img
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.3
 platform: ruby
 authors:
 - Mike Slinn
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-02-12 00:00:00.000000000 Z
+date: 2023-02-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll
@@ -28,44 +28,16 @@ dependencies:
   name: jekyll_plugin_support
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.1
+        version: 0.5.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 0.3.1
-- !ruby/object:Gem::Dependency
-  name: rspec
-  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: rspec-match_ignoring_whitespace
-  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'
+        version: 0.5.1
 description:
 email:
 - mslinn@mslinn.com
@@ -87,7 +59,6 @@ files:
 - spec/img_props_spec.rb
 - spec/jekyll_img_spec.rb
 - spec/spec_helper.rb
-- spec/status_persistence.txt
 homepage: https://mslinn.com/jekyll/3000-jekyll-plugins.html#jekyll_img
 licenses:
 - MIT
@@ -124,5 +95,4 @@ test_files:
 - spec/img_props_spec.rb
 - spec/jekyll_img_spec.rb
 - spec/spec_helper.rb
-- spec/status_persistence.txt
 ...

data/spec/status_persistence.txt

@@ -1,10 +0,0 @@
-example_id                      | status | run_time        |
-------------------------------- | ------ | --------------- |
-./spec/img_builder_spec.rb[1:1] | passed | 0.00073 seconds |
-./spec/img_props_spec.rb[1:1]   | passed | 0.0001 seconds  |
-./spec/img_props_spec.rb[1:2]   | passed | 0.00008 seconds |
-./spec/img_props_spec.rb[1:3]   | failed | 0.00051 seconds |
-./spec/img_props_spec.rb[1:4]   | passed | 0.0011 seconds  |
-./spec/img_props_spec.rb[1:5]   | passed | 0.00019 seconds |
-./spec/img_props_spec.rb[1:6]   | failed | 0.00008 seconds |
-./spec/jekyll_img_spec.rb[1:1]  | failed | 0.00037 seconds |