jekyll_draft 2.1.0 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 86ee1458ee28aab9d7f99d46deeff46e98a795cb6d3737a84b2e0e9f17f68632
-  data.tar.gz: a1abc81b94ccfdc3ee46ec312324d0d086567bbabcaa40784fca3d8d1c8415e4
+  metadata.gz: 6bccff30e811690f02c8b5d429c7bd3ed98e78ccd704d7baf440426be26ef1a0
+  data.tar.gz: 99b14ff0b6237d8b97cc5e21f3a4ec7f777b4f49727ad8c01b9f16df38568ecc
 SHA512:
-  metadata.gz: a70c60fa72cfb2dc9a8842e5b54ec9f0dec2746976edd90ed174990dcf24a14aadf801246258c257ac375c68fd72a518c9e9bc55bbbf975498c43d09a5a94774
-  data.tar.gz: 030ef794350d158b43395d957ce6f080c8497dde0db7005dacc96dd2900cacbc712c353addf1cb2443f8261e2e2fee471d9e1f39c7f7c9a758776f23a7ca56d5
+  metadata.gz: 6451f0a15a14ade6305c7876f51387d00ecb913e619a7a6d116af445ce4564d0f2f11a4620eac66185be15bc0187f6e15b5297d6ccdd5794aeadd633c2a454c6
+  data.tar.gz: 6925ed553fc89df595f6b181ca6b255bae43759df8c48b809267159b05af9a03dc3878ff06a25b5ba4639b0011f71b9bf112a31c8439e93525404f5afcd3f0f9

data/.rubocop.yml

@@ -55,7 +55,7 @@ Metrics/BlockLength:
 Metrics/ClassLength:
   Exclude:
     - spec/**/*
-  Max: 60
+  Max: 100
 
 Metrics/CyclomaticComplexity:
   Max: 20

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Change Log
 
+## 3.0.0 / In process
+
+* Now requires [Jekyll 4.4.1](https://jekyllrb.com/news/2025/01/29/jekyll-4-4-1-released/) or later, and Ruby 3.2.0 or later
+
+
 ## 2.1.0 / 2025-02-07
 
 * Added ability to return a page in any collection, or those not in a collection,

data/README.md

@@ -93,8 +93,8 @@ so you can learn how to write Jekyll pages that include this functionality.
 
    ```ruby
    group :jekyll_plugins do
-     gem 'jekyll_all_collections', '>= 0.3.8'
-     gem 'jekyll_draft', '>=2.1.0'
+     gem 'jekyll_plugin_support', '>= 3.0.0'
+     gem 'jekyll_draft', '>=3.0.0'
    end
    ```
 
@@ -112,8 +112,8 @@ so you can learn how to write Jekyll pages that include this functionality.
 1. Add the following to your gem’s `.gemspec`:
 
    ```ruby
-   spec.add_dependency 'jekyll_all_collections', '>= 0.3.8'
-   spec.add_dependency 'jekyll_draft', '>=2.1.0'
+   spec.add_dependency 'jekyll_plugin_support', '>= 3.0.0'
+   spec.add_dependency 'jekyll_draft', '>=3.0.0'
    ```
 
 2. Type:

data/jekyll_draft.gemspec

@@ -1,6 +1,6 @@
 require_relative 'lib/jekyll_draft/version'
 
-Gem::Specification.new do |spec| # rubocop:disable Metrics/BlockLength
+Gem::Specification.new do |spec|
   github = 'https://github.com/mslinn/jekyll_draft'
 
   spec.authors     = ['Mike Slinn']
@@ -27,11 +27,12 @@ Gem::Specification.new do |spec| # rubocop:disable Metrics/BlockLength
 
   END_MESSAGE
   spec.require_paths         = ['lib']
-  spec.required_ruby_version = '>= 2.6.0'
+  spec.required_ruby_version = '>= 3.2.0'
   spec.summary               = 'This Jekyll filter detects draft documents.'
   spec.test_files            = spec.files.grep(%r{^(test|spec|features)/})
   spec.version               = DraftVersion::VERSION
 
-  spec.add_dependency 'jekyll', '>= 3.5.0'
-  spec.add_dependency 'jekyll_plugin_support', '>=1.1.0'
+  spec.add_dependency 'jekyll', '>= 4.4.0'
+  spec.add_dependency 'jekyll_plugin_support', '>=3.0.0'
+  spec.add_dependency 'sorted_set'
 end

data/lib/jekyll_draft/common/else.rb

@@ -1,5 +1,3 @@
-require 'jekyll_plugin_support'
-
 class ElseBase < JekyllSupport::JekyllTag
   VERSION = '0.1.0'.freeze
 

data/lib/jekyll_draft/common/module_functions.rb

@@ -24,7 +24,12 @@ module Jekyll
 
       false
     rescue StandardError => e
-      Draft.logger.error { e.full_message }
+      msg = e.full_message
+      if msg.length < 250
+        Draft.logger.error { msg }
+      else
+        Draft.logger.error { "#{msg[0..125]} ... #{msg[-125..msg.length]}" }
+      end
       false
     end
 
@@ -35,25 +40,27 @@ module Jekyll
 
       " <i class='jekyll_draft'>Draft</i>"
     rescue StandardError => e
-      Draft.logger.error { e.full_message }
+      msg = e.full_message
+      if msg.length < 250
+        Draft.logger.error { msg }
+      else
+        Draft.logger.error { "#{msg[0..125]} ... #{msg[-125..msg.length]}" }
+      end
       ''
     end
 
-    # This is a computationally intense method, and really slows down site generation.
-    # TODO: implement cache in front of sorted objects containing paths and page reference
-    # This cache is also needed by compute_link_and_text in jekyll_href
     # @return APage whose href uniquely contains path_portion,
     #         or `path_portion` as a String for non-local URLs.
     def page_match(path_portion, error_if_no_match: true, verify_unique_match: false)
       return :non_local_url if path_portion.start_with? 'http:', 'https:', 'mailto'
 
       matching_pages = if verify_unique_match
-                         ::AllCollectionsHooks.sorted_lru_files.select path_portion
+                         ::AllCollectionsHooks.sorted_lru_files.find(path_portion)
                        else
-                         match = ::AllCollectionsHooks.sorted_lru_files.find path_portion
+                         match = ::AllCollectionsHooks.sorted_lru_files.find(path_portion)
                          match.nil? ? [] : [match]
                        end
-      case matching_pages.length
+      case matching_pages.size
       when 0
         return '' unless error_if_no_match
 
@@ -70,7 +77,12 @@ module Jekyll
         exit! 2 # The user should fix this problem before allowing the website to generate
       end
     rescue StandardError => e
-      puts e.full_message
+      msg = e.full_message
+      if msg.length < 250
+        Draft.logger.error { msg }
+      else
+        Draft.logger.error { "#{msg[0..125]} ... #{msg[-125..msg.length]}" }
+      end
     end
 
     # @return path to root of the collection that doc is a member of
@@ -83,6 +95,13 @@ module Jekyll
       return index.href if index
 
       docs.min.href
+    rescue StandardError => e
+      msg = e.full_message
+      if msg.length < 250
+        Draft.logger.error { msg }
+      else
+        Draft.logger.error { "#{msg[0..150]} ... #{msg[-150..msg.length]}" }
+      end
     end
 
     module_function :draft?, :draft_html, :page_match, :root

data/lib/jekyll_draft/other_page/if_draft.rb

@@ -34,7 +34,12 @@ class DraftPageBase < JekyllSupport::JekyllBlock
                    end
     process_variables clause_value
   rescue StandardError => e
-    puts e.full_message
+    msg = e.full_message
+    if msg.length < 250
+      @logger.error { msg }
+    else
+      @logger.error { "#{msg[0..125]} ... #{msg[-125..msg.length]}" }
+    end
     exit!
   end
 
@@ -46,7 +51,7 @@ class DraftPageBase < JekyllSupport::JekyllBlock
       @error_if_no_match = tag_config['error_if_no_match'] # defaults true
       @error_if_no_match = @error_if_no_match ? @error_if_no_match == true : true
 
-      @verify_unique_match     = tag_config['verify_unique_match'] == true # defaults false
+      @verify_unique_match = true # tag_config['verify_unique_match'] == true # defaults false
     else
       @logger.warn { "There is no entry for #{@tag_name} in _config.yml; default values were applied." }
     end
@@ -58,31 +63,55 @@ class DraftPageBase < JekyllSupport::JekyllBlock
     @verify_unique_match = env_var != 'false' if env_var
   end
 
+  # @param markup [String] might contain <<path_portion>> or <<matched_page.url>>
   def process_variables(markup)
     return '' unless markup
 
     path_portion = @path_portion # add this to the binding
-    matched_page = @matched_page # add this to the binding
+    matched_page = @matched_page.instance_of?(Enumerator) ? @matched_page.first : @matched_page # add this to the binding
     _ = path_portion == matched_page # Just to keep the lint checker happy
-    markup.gsub!(/(<<([^<>]*)>>)/) do
-      token = Regexp.last_match[1] # gsub replaces token with the return value of this block
-      expression = Regexp.last_match[2]
-      if expression == 'matched_page' && matched_page.class != String
-        @logger.warn do
-          expression = 'matched_page.href'
-          <<~END_MSG
-            The expression #{token} should either specify a property, for example <<matched_page.href>>,
-            or a hash value, for example <<matched_page['title']>>.
-            The expression was modified to <<matched_page.href>> for you.
-          END_MSG
-        end
-      end
-      result = eval expression, binding # rubocop:disable Security/Eval
-      result.to_s
+    markup.gsub!(/(<<([^<>]*)>>)/) do # gsub replaces token with the return value of this block
+      replace_token Regexp.last_match[1], Regexp.last_match[2], binding
     end
     markup
   rescue StandardError => e
-    @logger.error { e.full_message }
+    msg = e.full_message
+    if msg.length < 250
+      @logger.error { msg }
+    else
+      @logger.error { "#{msg[0..125]} ... #{msg[-125..msg.length]}" }
+    end
+  end
+
+  def replace_token(token, expression, binding)
+    if expression == 'matched_page'
+      expression = case matched_page
+                   when String
+                     matched_page
+                   when APage
+                     'matched_page&.href'
+                   when LruFile
+                     'matched_page&.url'
+                   else
+                     @logger.warn do
+                       <<~END_MSG
+                         The type #{matched_page.class} has an invalid valueexpression #{token}.
+                       END_MSG
+                     end
+                     exit!
+                   end
+      @logger.warn do
+        <<~END_MSG
+          The expression #{token} should either specify a property, for example <<matched_page&.href>>,
+          or a hash value, for example <<matched_page['title']>>.
+          The expression was modified to <<#{expression}>> for you.
+        END_MSG
+      end
+    else
+      puts expression
+    end
+    result = eval expression, binding # rubocop:disable Security/Eval
+    result.to_s
   end
 end
 

data/lib/jekyll_draft/version.rb

@@ -1,3 +1,3 @@
 module DraftVersion
-  VERSION = '2.1.0'.freeze
+  VERSION = '3.0.0'.freeze unless defined?(VERSION)
 end

data/lib/jekyll_draft.rb

@@ -1,6 +1,8 @@
-# @author Copyright 2022 {https://www.mslinn.com Michael Slinn}
-
-require 'jekyll_plugin_support'
+if ENV['USE_JPS_SOURCE']
+  require_relative "#{ENV.fetch('jekyll_plugin_support')}/lib/jekyll_plugin_support"
+else
+  require 'jekyll_plugin_support'
+end
 require 'yaml'
 require_relative 'jekyll_draft/version' unless defined?(VERSION)
 

data/spec/status_persistence.txt

@@ -1,3 +1,3 @@
 example_id                       | status | run_time        |
 -------------------------------- | ------ | --------------- |
-./spec/jekyll/draft_spec.rb[1:1] | passed | 0.00007 seconds |
+./spec/jekyll/draft_spec.rb[1:1] | failed | 0.00558 seconds |

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll_draft
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 3.0.0
 platform: ruby
 authors:
 - Mike Slinn
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-02-07 00:00:00.000000000 Z
+date: 2025-03-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll
@@ -15,28 +16,42 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.5.0
+        version: 4.4.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.5.0
+        version: 4.4.0
 - !ruby/object:Gem::Dependency
   name: jekyll_plugin_support
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: 3.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: 3.0.0
+- !ruby/object:Gem::Dependency
+  name: sorted_set
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: 'This Jekyll filter detects draft documents.
 
   '
@@ -86,14 +101,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.6.0
+      version: 3.2.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.3
+rubygems_version: 3.5.22
+signing_key:
 specification_version: 4
 summary: This Jekyll filter detects draft documents.
 test_files: