jekyll_plugin_support 1.1.0 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b54badeb565b69fb6839140b7ce7b340d6a6663c226a3f7cb8728df2e0f61b2e
-  data.tar.gz: f74acdbe3c4d99561dead976deb36c4ff477f06dac7783a0ef918df901a71a53
+  metadata.gz: 74f595dc019d923dbbad9e84b1eacdaa15808b36d54fc692044e30f26e0767f1
+  data.tar.gz: 5632b03e8d9d29080ae0c43428d5c9a13243c4df529255013d63951d669a670d
 SHA512:
-  metadata.gz: d3deaa658d05e5f9cf629ba10a337f9b864465462a362d1d4670eaa3afff32c78b04cbc974a17fb9afd9fbac1392a2e4c58c1e0fbbd40a145ac9c4076b5a741a
-  data.tar.gz: 910ca35d862324baac1eb5c627662f4f4dac01e5966a06ad9c071bf1354800458ff81f16e260375680845d7062ee9dc7c41c63ab994f4d2630ad56eacbe8bb51
+  metadata.gz: b5839393146e5d1ab2317f004a563bad4144467e506fd41516f489b065fa52d00bfcea4eedc7d8d06028afd35cba34b3360b19f4d968ec6a456e429f6a2af542
+  data.tar.gz: 203f6111c4127b4909534d4dcc6b7dcb7f3f5d7bbf9fa60016920558a6eb9faed7cd736212b8c3f25cb52514e21b75035dd00ea55928525127b9ca662ede224d

data/.rubocop.yml

@@ -1,4 +1,4 @@
-require:
+plugins:
   # - rubocop-jekyll
   - rubocop-md
   - rubocop-performance
@@ -26,6 +26,14 @@ Layout/HashAlignment:
   EnforcedColonStyle: table
   EnforcedHashRocketStyle: table
 
+Layout/IndentationConsistency:
+  Exclude:
+    - "*.md"
+
+Layout/InitialIndentation:
+  Exclude:
+    - "*.md"
+
 Layout/LineLength:
   Max: 150
 
@@ -37,12 +45,20 @@ Metrics/BlockLength:
     - jekyll_plugin_support.gemspec
   Max: 30
 
+Metrics/ClassLength:
+  Exclude:
+    - "**/all_collections_tag.rb"
+
 Metrics/CyclomaticComplexity:
   Max: 25
 
 Metrics/MethodLength:
   Max: 50
 
+Metrics/ModuleLength:
+  Exclude:
+    - "**/jekyll_plugin_support_class.rb"
+
 Metrics/PerceivedComplexity:
   Max: 25
 

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Change Log
 
+## 3.0.0 / 2025-02-09
+
+* Numbered as v3.0.0 because `jekyll_draft` was at v2.1.0 and the two projects should have similar version numbers.
+* `Jekyll_all_collections` was folded into this project.
+* 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
+
+
 ## 1.1.0 / 2025-02-07
 
 * Now replaces include and layout variables with their values
@@ -30,12 +38,15 @@
   The following seems to be optimal for custom plugins; it suppresses the ridiculously long stack trace that used to be generated:
 
    ```ruby
-    rescue DemoInlineTagError => e # jekyll_plugin_support handles StandardError
-      @logger.error { e.logger_message }
-      exit! 1 if @die_on_demo_tag_error
-
-      e.html_message
-    ```
+   begin
+     # ...
+   rescue DemoInlineTagError => e # jekyll_plugin_support handles StandardError
+     @logger.error { e.logger_message }
+     exit! 1 if @die_on_demo_tag_error
+
+     e.html_message
+   end
+   ```
 
 
 ## 1.0.1 / 2024-07-27

data/README.md

@@ -14,13 +14,13 @@ At present, only Jekyll tags and blocks are supported.
 Public plugins that use `jekyll_plugin_support` include:
 
 <ul style="columns: 2">
-  <li><a href='https://www.mslinn.com/jekyll_plugins/jekyll_all_collections'><code>jekyll_all_collections</code></a></li>
   <li><a href='https://www.mslinn.com/jekyll_plugins/jekyll_badge'><code>jekyll_badge</code></a></li>
+  <li><a href='https://www.mslinn.com/jekyll_plugins/jekyll_draft'><code>jekyll_draft</code></a></li>
   <li><a href='https://www.mslinn.com/jekyll_plugins/jekyll_emoji'><code>jekyll_emoji</code></a></li>
   <li><a href='https://www.mslinn.com/jekyll_plugins/jekyll_flexible_include.html'><code>jekyll_flexible_include</code></a></li>
+  <li><a href='https://www.mslinn.com/jekyll_plugins/jekyll_google_translate'><code>jekyll_google_translate</code></a></li>
   <li><a href='https://www.mslinn.com/jekyll_plugins/jekyll_href.html'><code>jekyll_href</code></a></li>
   <li><a href='https://www.mslinn.com/jekyll_plugins/jekyll_img.html'><code>jekyll_img</code></a></li>
-  <li><a href='https://www.mslinn.com/jekyll_plugins/jekyll_plugin_template.html'><code>jekyll_plugin_template</code></a></li>
   <li><a href='https://www.mslinn.com/jekyll_plugins/jekyll_outline.html'><code>jekyll_outline</code></a></li>
   <li><a href='https://www.mslinn.com/jekyll_plugins/jekyll_pre.html'><code>jekyll_pre</code></a></li>
   <li><a href='https://www.mslinn.com/jekyll_plugins/jekyll_quote.html'><code>jekyll_quote</code></a></li>
@@ -53,7 +53,17 @@ Jekyll plugin tags created from `jekyll_plugin_support` framework automatically
 14. Plugins can be subclassed.
 15. [`Nugem`](https://mslinn.com/ruby/6800-nugem.html) can create working scaffolding for new plugins built
     using `jekyll_plugin_support`.
+16. Four new attributes are added to
+  [`site`](https://jekyllrb.com/docs/variables/#site-variables):
 
+    a. `all_collections` includes all documents in all collections.
+
+    b. `all_documents` includes `all_collections` plus all standalone pages.
+
+    c. `everything` includes `all_documents` plus all static files.
+
+    d. `sorted_lru_files` is used by a new binary search lookup for matching page suffixes.
+      The `jekyll_href` and `jekyll_draft` plugins use this feature.
 
 ## Installation
 
@@ -105,6 +115,9 @@ to your Jekyll project&rsquo;s CSS file.
 
 ## About `jekyll_plugin_support`
 
+This Jekyll plugin includes a generator,
+triggered by a high-priority hook, and a block tag called `all_collections`.
+
 `JekyllSupport::JekyllBlock` and `JekyllSupport::JekyllTag`
 provide support for [Jekyll tag block plugins](https://jekyllrb.com/docs/plugins/tags/#tag-blocks)
 and [Jekyll inline tag plugins](https://jekyllrb.com/docs/plugins/tags/), respectively.
@@ -175,6 +188,195 @@ For inline tags, `render_impl` does not accept any parameters.
 For block tags, a single parameter is required, which receives any text enclosed within your block by the website author.
 
 
+### New `Site` Attributes
+
+No explicit initialization or setup is required.
+Jekyll plugins can access the value of
+`site.all_collections`, `site.all_documents` and `site.everything`;
+however, Liquid code in Jekyll pages and documents cannot.
+
+
+### Excluding Files
+
+There are two ways to exclude files from the new `site` attributes.
+
+1) The [`exclude` entry in `_config.yml`](https://jekyllrb.com/docs/configuration/options#global-configuration)
+   can be used as it normally would.
+
+2) Adding the following entry to a page&rsquo;s front matter causes that page to be excluded
+  from the collection created by this plugin:
+
+   ```html
+   ---
+   exclude_from_all: true
+   ---
+   ```
+
+### `all_collections` Plugin Usage
+
+Jekyll generators and tags receive an enhanced version of the `site` Jekyll variable.
+
+
+#### From a Custom Plugin
+
+In the following example of how to use the `all_collections` plugin in a custom plugin,
+the `do_something_with` function processes all `Jekyll::Page`s, `Jekyll:Document`s, and static files.
+
+```ruby
+@site.everything.each do |apage|
+  do_something_with apage
+end
+```
+
+
+#### Using the Block Tag
+
+The general form of the Jekyll tag, including all options, is:
+
+```html
+{% all_collections
+  date_column='date|last_modified'
+  heading='All Posts'
+  id='asdf'
+  sort_by='SORT_KEYS'
+%}
+```
+
+
+##### `date_column` Attribute
+
+One of two date columns can be displayed in the generated HTML:
+either `date` (when the article was originally written),
+or `last_modified`.
+The default value for the `date_column` attribute is `date`.
+
+
+##### `heading` Attribute
+
+If no `heading` attribute is specified, a heading will automatically be generated, which contains the `sort_by` values,
+for example:
+
+```html
+{% all_collections id='abcdef' sort_by="last_modified" %}
+```
+
+The above generates a heading like:
+
+```html
+<h2 id="abcdef">All Posts Sorted By last_modified</h2>
+```
+
+To suppress both a `h2` heading (and the enclosed `id`) from being generated,
+specify an empty string for the value of `heading`:
+
+```html
+{% all_collections heading='' %}
+```
+
+
+##### `id` Attribute
+
+If your Jekyll layout employs [`jekyll-toc`](https://github.com/allejo/jekyll-toc), then `id` attributes are important.
+The `jekyll-toc` include checks for `id` attributes in `h2` ... `h6` tags, and if found,
+and if the attribute value is enclosed in double quotes
+(`id="my_id"`, not single quotes like `id='my_id'`),
+then the heading is included in the table of contents.
+
+To suppress an `id` from being generated,
+and thereby preventing the heading from appearing in the automatically generated table of contents from `jekyll-toc`,
+specify an empty string for the value of `id`, like this:
+
+```html
+{% all_collections id='' %}
+```
+
+
+##### `SORT_KEYS` Values
+
+`SORT_KEYS` specifies how to sort the collection.
+Values can include one or more of the following attributes:
+`date`, `destination`, `draft`, `label`, `last_modified`, `last_modified_at`, `path`, `relative_path`,
+`title`, `type`, and `url`.
+Ascending sorts are the default; however, a descending sort can be achieved by prepending `-` before an attribute.
+
+To specify more than one sort key, provide a comma-delimited string of values.
+Included spaces are ignored.
+For example, specify the primary sort key as `draft`,
+the secondary sort key as `last_modified`,
+and the tertiary key as `label`:
+
+```html
+{% all_collections
+  date_column='last_modified'
+  heading='All Posts'
+  id='asdf'
+  sort_by='draft, last_modified, label'
+%}
+```
+
+
+#### Liquid Usage Examples
+
+Here is a short Jekyll page, including front matter,
+demonstrating this plugin being invoked with all default attribute values:
+
+```html
+---
+description: "
+  Dump of all collections, sorted by date originally written, newest to oldest.
+  The heading text will be <code>All Posts Sorted By -date</code>
+"
+layout: default
+title: Testing, 1, 2, 3
+---
+{% all_collections %}
+```
+
+Following are examples of how to specify the sort parameters.
+
+**Explicitly express the default sort**<br>
+(sort by the date originally written, newest to oldest):
+
+```html
+{% all_collections sort_by="-date" %}
+```
+
+Sort by date, from oldest to newest:
+
+```html
+{% all_collections sort_by="date" %}
+```
+
+**Sort by the date last modified, oldest to newest:**
+
+```html
+{% all_collections sort_by="last_modified" %}
+```
+
+**Sort by the date last modified, newest to oldest:**
+
+```html
+{% all_collections sort_by="-last_modified" %}
+```
+
+**Several attributes can be specified as sort criteria**<br>
+by passing them as a comma-delimited string.
+Included spaces are ignored:
+
+```html
+{% all_collections sort_by="-last_modified, -date" %}
+{% all_collections sort_by="-last_modified, title" %}
+{% all_collections sort_by="-last_modified, -date, title" %}
+```
+
+**The following two examples produce the same output:**
+
+```html
+{% all_collections sort_by="-last_modified,-date" %}
+{% all_collections sort_by="-last_modified, -date" %}
+```
+
+
 ## Predefined Plugin Variables
 
 `Jekyll_plugin_support` defines the following Ruby variables that you can use in your plugin&rsquo;s `render_impl` method:
@@ -730,6 +932,133 @@ redef_without_warning :VERSION, '0.1.0'.freeze
 ```
 
 
+## Generator
+
+This plugin incorporates a generator, which adds four new attributes to
+[`site`](https://jekyllrb.com/docs/variables/#site-variables):
+`all_collections`, `all_documents`, `everything`, and `sorted_lru_files`.
+
+These three attributes can be referenced as `site.everything`, `site.all_collections`
+and `site.all_documents`.
+
+* `all_collections` includes all documents in all collections.
+
+* `all_documents` includes `all_collections` plus all standalone pages.
+
+* `everything` includes `all_documents` plus all static files.
+
+* `sorted_lru_files` is used by a new binary search lookup for matching page suffixes.
+  Currently only `jekyll_href` and `jekyll_draft` use this feature.
+
+
+### Collection Management
+
+Jekyll provides inconsistent attributes for
+[`site.pages`](https://jekyllrb.com/docs/pages/),
+[`site.posts`](https://jekyllrb.com/docs/posts/) and
+[`site.static_files`](https://jekyllrb.com/docs/static-files/).
+
+
+* While the `url` attributes of items in `site.posts` and `site.pages` start with a slash (/),
+  `site.static_files` items do not have a `url` attribute.
+* Static files have a `relative_path` attribute, which starts with a slash (/),
+  but although that attribute is also provided in `site.posts` and `site.pages`,
+  those values do not start with a slash.
+* Paths ending with a slash (`/`) imply that a file called `index.html` should be fetched.
+* HTML redirect files created by the
+  [`jekyll-redirect-from`](https://github.com/jekyll/jekyll-redirect-from) Jekyll plugin,
+  which are included in `site.static_files`, should be ignored.
+
+These inconsistencies mean that combining the standard three collections of files
+provided as `site` attributes will create a new collection that is difficult
+to process consistently:
+
+```ruby
+# This pseudocode creates `oops`, which is problematic to process consistently
+oops = site.all_collections + site.pages + site.static_files
+```
+
+`oops`, above, is difficult to process because of inconsistencies in the provided attributes
+and how the attributes are constructed.
+
+
+### Solving The Problem
+
+The generator normalizes these inconsistencies by utilizing the `APage` class
+and filtering out HTML redirect files.
+
+The `all_collections` collection contains `APage` representations of `site.collections`.
+
+The `all_documents` collection contains `APage` representations of `site.pages`.
+
+The `everything` collection contains `APage` representations of:
+
+```text
+# Pseudocode
+site.collections + site.pages + site.static_files - HTML_redirect_files
+```
+
+
+## The `APage` Class
+
+The `site.all_collections`, `site.all_documents` and `site.everything` attributes
+consist of arrays of [`APage`](lib/hooks/a_page.rb) instances.
+
+The `APage` class has the following attributes:
+`content` (HTML or Markdown), `data` (array), `date` (Ruby Date), `description`, `destination`,
+`draft` (Boolean), `ext`, `href`, `label`, `last_modified` or `last_modified_at` (Ruby Date),
+`layout`, `origin`, `path`, `relative_path`, `tags`, `title`, `type`, and `url`.
+
+* `href` always starts with a slash.
+  This value is consistent with `a href` values in website HTML.
+  Paths ending with a slash (`/`) have `index.html` appended so the path specifies an actual file.
+
+* `origin` indicates the original source of the item.
+  Possible values are `collection`, `individual_page` and `static_file`.
+  Knowing the origin of each item allows code to process each type of item appropriately.
+
+
+## `all_collections` Block Tag
+
+The `all_collections` block tag creates a formatted listing of Jekyll files.
+The ordering is configurable; by default, the listing is sorted by `date`, newest to oldest.
+The `all_collections` tag has a `data_source` parameter that specifies which new property to report on
+(`all_collections`, `all_documents`, or `everything`).
+
+
+## Requirements
+
+All the pages in the Jekyll website must have an implicit date
+(for example, all posts are assigned this property by Jekyll),
+or an explicit `date` set in front matter, like this:
+
+```html
+---
+date: 2022-01-01
+---
+```
+
+If a front matter variable called `last_modified` or `last_modified_at` exists,
+its value will be converted to a Ruby `Date`:
+
+```html
+---
+last_modified: 2023-01-01
+---
+```
+
+Or:
+
+```html
+---
+last_modified_at: 2023-01-01
+---
+```
+
+Otherwise, if `last_modified` or `last_modified_at` is not present in the front matter for a page,
+the `date` value will be used last modified date value.
+
+
 ## Development
 
 After checking out the `jekyll_plugin_suppprt` repository, run `bin/setup` to install dependencies.
@@ -763,6 +1092,51 @@ jekyll_plugin_support (0.1.0)
 ```
 
 
+### Build and Install Locally
+
+To build and install this gem onto your local machine, run:
+
+```shell
+$ bundle exec rake install
+jekyll_all_collections 0.3.8 built to pkg/jekyll_all_collections-0.3.8.gem.
+jekyll_all_collections (0.3.8) installed.
+```
+
+Examine the newly built gem:
+
+```shell
+$ gem info jekyll_all_collections
+
+*** LOCAL GEMS ***
+
+jekyll_all_collections (0.3.8)
+    Author: Mike Slinn
+    Homepage:
+    https://www.mslinn.com/jekyll_plugins/jekyll_all_collections.html
+    License: MIT
+    Installed at (0.3.8): /home/mslinn/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0
+
+    Provides normalized collections and extra functionality for Jekyll websites.
+```
+
+
+### Build and Push to RubyGems
+
+To release a new version:
+
+1. Update the version number in `version.rb`.
+2. Add an entry in `CHANGELOG.md` describing the changes since the last release.
+3. Commit all changes to git; if you don't the next step might fail with a confusing error message.
+4. Run the following:
+
+    ```shell
+    $ bundle exec rake release
+    ```
+
+    The above creates a git tag for the version, commits the created tag,
+    and pushes the new `.gem` file to [RubyGems.org](https://rubygems.org).
+
+
 ### Pry Breakpoint On StandardError
 
 A `pry` breakpoint will be set in the `StandardError` handler if `pry_on_standard_error: true`
@@ -776,6 +1150,60 @@ blah:
 ```
 
 
+## Debugging
+
+You can control the verbosity of log output by adding the following to `_config.yml` in your Jekyll project:
+
+```yaml
+plugin_loggers:
+  AllCollectionsTag::AllCollectionsTag: warn
+```
+
+1. First set breakpoints in the Ruby code that interests you.
+
+2. You have several options for initiating a debug session:
+
+   1. Use the **Debug Demo** Visual Studio Code launch configuration.
+
+   2. Type the `demo/_bin/debug` command, without the `-r` options shown above.
+
+      ```console
+      ... lots of output as bundle update runs...
+      Bundle updated!
+
+      INFO PluginMetaLogger: Loaded AllCollectionsHooks v0.2.0 :site, :pre_render, :normal hook plugin.
+      INFO PluginMetaLogger: Loaded DraftFilter plugin.
+      INFO PluginMetaLogger: Loaded all_collections v0.2.0 tag plugin.
+      Configuration file: /mnt/_/work/jekyll/my_plugins/jekyll_all_collections/demo/_config.yml
+                Cleaner: Removing /mnt/_/work/jekyll/my_plugins/jekyll_all_collections/demo/_site...
+                Cleaner: Removing /mnt/_/work/jekyll/my_plugins/jekyll_all_collections/demo/.jekyll-metadata...
+                Cleaner: Removing /mnt/_/work/jekyll/my_plugins/jekyll_all_collections/demo/.jekyll-cache...
+                Cleaner: Nothing to do for .sass-cache.
+      Fast Debugger (ruby-debug-ide 0.7.3, debase 0.2.5.beta2, file filtering is supported) listens on 0.0.0.0:1234
+      ```
+
+   3. Run `bin/attach` and pass the directory name of a Jekyll website that has a suitable script called `_bin/debug`.
+      The `demo` subdirectory fits this description.
+
+      ```console
+      $ bin/attach demo
+      Successfully uninstalled jekyll_all_collections-0.1.2
+      jekyll_all_collections 0.1.2 built to pkg/jekyll_all_collections-0.1.2.gem.
+      jekyll_all_collections (0.1.2) installed.
+      Fast Debugger (ruby-debug-ide 0.7.3, debase 0.2.4.1, file filtering is supported) listens on 0.0.0.0:1234
+      ```
+
+
+3. Attach to the debugger process if required.
+  This git repo includes two [Visual Studio Code launch configurations](./.vscode/launch.json) for this purpose labeled
+  **Attach rdbg** and **Attach with ruby_lsp**.
+
+4. Point your web browser to http://localhost:4444
+
+If a debugging session terminates abruptly and leaves ports tied up,
+run the `demo/_bin/release_port` script.
+
+
 ## Demonstration Plugins and Website
 
 A demo / test website is provided in the [`demo`](demo) directory.

data/jekyll_plugin_support.gemspec

@@ -30,8 +30,9 @@ Gem::Specification.new do |spec|
   spec.version               = JekyllPluginSupportVersion::VERSION
 
   spec.add_dependency 'facets'
-  spec.add_dependency 'jekyll', '>= 3.5.0'
+  spec.add_dependency 'jekyll', '>= 4.4.1'
   spec.add_dependency 'jekyll_plugin_logger'
   spec.add_dependency 'key-value-parser'
   spec.add_dependency 'pry'
+  spec.add_dependency 'sorted_set'
 end

data/lib/block/jekyll_plugin_support_block.rb

@@ -1,5 +1,3 @@
-require_relative '../error/jekyll_plugin_error_handling'
-
 module JekyllSupport
   # Base class for Jekyll block tags
   class JekyllBlock < Liquid::Block
@@ -35,7 +33,7 @@ module JekyllSupport
       @helper = JekyllPluginHelper.new tag_name, markup, @logger, respond_to?(:no_arg_parsing)
 
       @error_name = "#{tag_name.camelcase(:upper)}Error"
-      JekyllSupport::CustomError.factory @error_name
+      ::JekyllSupport::CustomError.factory @error_name
     end
 
     # Liquid::Block subclasses do not render if there is no content within the tag
@@ -48,7 +46,7 @@ module JekyllSupport
     # Defines @config, @envs, @mode, @page and @site
     # @return [String]
     def render(liquid_context)
-      @helper.liquid_context = JekyllSupport.inject_config_vars liquid_context # modifies liquid_context
+      @helper.liquid_context = ::JekyllSupport.inject_config_vars liquid_context # modifies liquid_context
       text = super # Liquid variable values in content are looked up and substituted
 
       @envs      = liquid_context.environments.first

data/lib/block/jekyll_plugin_support_block_noarg.rb

@@ -1,5 +1,3 @@
-require_relative '../error/jekyll_plugin_error_handling'
-
 module JekyllSupport
   class JekyllBlockNoArgParsing < JekyllBlock
     attr_reader :argument_string, :helper, :line_number, :logger, :page, :site

data/lib/generator/jekyll_plugin_support_generator.rb

@@ -1,6 +1,3 @@
-require 'jekyll'
-require_relative '../error/jekyll_plugin_error_handling'
-
 module JekyllSupport
   # Base class for Jekyll generators.
   # PluginMetaLogger.instance.config is populated with the contents of `_config.yml` before Jekyll::Generator instances run.
@@ -14,7 +11,6 @@ module JekyllSupport
       @logger ||= PluginMetaLogger.instance.new_logger(self, PluginMetaLogger.instance.config)
 
       @error_name = "#{self.class.name}Error"
-      # JekyllSupport::CustomError.factory @error_name
 
       @site   = site
       @config = @site.config
@@ -23,8 +19,6 @@ module JekyllSupport
 
       @mode = ENV['JEKYLL_ENV'] || 'development'
 
-      # set_error_context(self.class)
-
       generate_impl
     rescue StandardError => e
       e.shorten_backtrace
@@ -62,7 +56,7 @@ module JekyllSupport
       PluginMetaLogger.instance.info { msg }
     end
 
-    def set_error_context(klass)
+    def set_error_context(klass) # rubocop:disable Naming/AccessorMethodName
       return unless Object.const_defined? @error_name
 
       error_class = Object.const_get @error_name

data/lib/helper/jekyll_plugin_helper.rb

@@ -1,14 +1,13 @@
 require 'facets/string/interpolate'
 require 'key_value_parser'
 require 'shellwords'
-require_relative 'jekyll_plugin_helper_class'
-require_relative 'jekyll_plugin_helper_attribution'
 
 module JekyllSupport
   class JekyllPluginHelper
     attr_accessor :liquid_context
-    attr_reader :argv, :attribution, :keys_values, :logger, :markup, :no_arg_parsing, :params, :tag_name,
-                :argv_original, :excerpt_caller, :keys_values_original, :params_original, :jpsh_subclass_caller
+    attr_reader :argument_string, :argv, :argv_original, :attribution, :excerpt_caller, :keys_values_original,
+                :keys_values, :jpsh_subclass_caller, :logger, :markup, :no_arg_parsing, :params, :params_original,
+                :tag_name
 
     # See https://github.com/Shopify/liquid/wiki/Liquid-for-Programmers#create-your-own-tags
     # @param tag_name [String] the name of the tag, which we already know.
@@ -22,8 +21,9 @@ module JekyllSupport
     def initialize(tag_name, markup, logger, no_arg_parsing)
       @tag_name = tag_name
       @logger = logger
-      @no_arg_parsing = no_arg_parsing
+      @markup = markup
       @argument_string = markup
+      @no_arg_parsing = no_arg_parsing
     rescue StandardError => e
       e.shorten_backtrace
       @logger.error { e.message }

data/lib/helper/jekyll_plugin_helper_class.rb

@@ -66,8 +66,8 @@ module JekyllSupport
 
       abort("Error: The #{tag_name} plugin is not an instance of JekyllSupport::JekyllBlock or JekyllSupport::JekyllTag") \
         unless klass.instance_of?(Class) &&
-               (klass.ancestors.include?(JekyllSupport::JekyllBlock) ||
-                klass.ancestors.include?(JekyllSupport::JekyllTag))
+               (klass.ancestors.include?(::JekyllSupport::JekyllBlock) ||
+                klass.ancestors.include?(::JekyllSupport::JekyllTag))
 
       Liquid::Template.register_tag(tag_name, klass)
       return if quiet