jekyll_all_collections 0.3.6 → 0.4.0

data/spec/bsearch_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+# Ruby's binary search is unsuitable because ordering requirements are not stable.
+# the value to be searched for changes the required ordering
+
+RSpec.describe(Array) do
+  before { skip('Never gonna give you up/Never gonna let you down') }
+
+  sorted_ints = [0, 4, 7, 10, 12]
+  sorted_strings = %w[aaa aab aac bbb bbc bbd ccc ccd cce].sort.reverse
+
+  it 'returns index of first int match' do
+    actual = sorted_ints.bsearch_index { |x| x >= 4 }
+    expect(actual).to eq(1)
+
+    actual = sorted_ints.bsearch_index { |x| x >= 6 }
+    expect(actual).to eq(2)
+
+    actual = sorted_ints.bsearch_index { |x| x >= -1 }
+    expect(actual).to eq(0)
+
+    actual = sorted_ints.bsearch_index { |x| x >= 100 }
+    expect(actual).to be_nil
+  end
+
+  # See https://stackoverflow.com/q/79333097/553865
+  it 'returns index of first string match' do
+    puts(sorted_strings.map { |x| x.start_with? 'a' })
+    index = sorted_strings.bsearch_index { |x| x.start_with? 'a' }
+    expect(sorted_strings[index]).to eq('aac')
+
+    index = sorted_strings.bsearch_index { |x| x.start_with? 'aa' }
+    expect(sorted_strings[index]).to eq('aac')
+
+    index = sorted_strings.bsearch_index { |x| x.start_with? 'aaa' }
+    expect(sorted_strings[index]).to eq('aaa')
+
+    index = sorted_strings.bsearch_index { |x| x.start_with? 'b' }
+    expect(sorted_strings[index]).to eq('bbd')
+
+    index = sorted_strings.bsearch_index { |x| x.start_with? 'bb' }
+    expect(sorted_strings[index]).to eq('bbd')
+
+    index = sorted_strings.bsearch_index { |x| x.start_with? 'bbc' }
+    expect(sorted_strings[index]).to eq('bbc')
+
+    index = sorted_strings.bsearch_index { |x| x.start_with? 'cc' }
+    expect(sorted_strings[index]).to eq('cce')
+  end
+end

data/spec/date_sort_spec.rb

@@ -1,6 +1,8 @@
 require 'spec_helper'
+require_relative '../lib/jekyll_all_collections'
+
+# Verifies how data comparisons work
 
-# See https://stackoverflow.com/a/75377832/553865
 class Obj
   # `last_modified` is primary sort key
   # `date` (when specified) is secondary sort key
@@ -12,11 +14,11 @@ class Obj
   end
 end
 
-RSpec.describe(Obj) do # rubocop:disable Metrics/BlockLength
-  let(:o1) { Obj.new('2000-01-01', '2001-01-01') }
-  let(:o2) { Obj.new('2010-01-01', '2001-01-01') }
-  let(:o3) { Obj.new('2010-01-01', '2011-01-01') }
-  let(:o4) { Obj.new('2020-01-01', '2011-01-01') }
+RSpec.describe(Obj) do
+  let(:o1) { described_class.new('2000-01-01', '2001-01-01') }
+  let(:o2) { described_class.new('2010-01-01', '2001-01-01') }
+  let(:o3) { described_class.new('2010-01-01', '2011-01-01') }
+  let(:o4) { described_class.new('2020-01-01', '2011-01-01') }
   let(:objs) { [o1, o2, o3, o4] }
 
   # See https://ruby-doc.org/3.2.0/Comparable.html

data/spec/mslinn_binary_search_spec.rb

@@ -0,0 +1,47 @@
+require 'spec_helper'
+require_relative '../lib/hooks/all_files'
+
+def build_msbs(sorted_strings)
+  sorted_strings.each do |string|
+    $msbs.insert LruFile.new(string.reverse, "Page #{string}") # { |x| x.url.start_with? string }
+  end
+  $msbs.enable_search
+end
+
+RSpec.describe(MSlinnBinarySearch) do
+  $msbs = described_class.new [:url, %i[start_with? placeholder]]
+  build_msbs %w[aaa.html baa.html caa.html bbb.html cbb.html dbb.html ccc.html ccd.html cce.html]
+
+  it 'handles an empty search string by returning the index of the first item (0)' do
+    index = $msbs.find_index '' # { |x| x.url.start_with? '' }
+    expect(index).to be_zero
+  end
+
+  it 'returns the index of the first partial match' do
+    index = $msbs.find_index 'a.html' # { |x| x.url.start_with? 'a.html' }
+    expect(index).to eq(0)
+
+    index = $msbs.find_index 'baa.html' # { |x| x.url.start_with? 'baa.html' }
+    expect(index).to eq(1)
+
+    index = $msbs.find_index 'c.html' # { |x| x.url.start_with? 'c.html' }
+    expect(index).to eq(6)
+
+    index = $msbs.find_index 'cce.html' # { |x| x.url.start_with? 'cce.html' }
+    expect(index).to eq(8)
+  end
+
+  it 'returns the item of the first match' do
+    item = $msbs.find 'a.html' # { |x| x.url.start_with? 'a.html' }
+    expect(item.url.reverse).to eq('aaa.html')
+
+    item = $msbs.find 'baa.html' # { |x| x.url.start_with? 'baa.html' }
+    expect(item.url.reverse).to eq('baa.html')
+
+    item = $msbs.find 'c.html' # { |x| x.url.start_with? 'c.html' }
+    expect(item.url.reverse).to eq('ccc.html')
+
+    item = $msbs.find 'cce.html' # { |x| x.url.start_with? 'cce.html' }
+    expect(item.url.reverse).to eq('cce.html')
+  end
+end

data/spec/send_chain_spec.rb

@@ -0,0 +1,72 @@
+require 'spec_helper'
+require_relative '../lib/util/send_chain'
+
+unless defined? LruFile
+  LruFile = Struct.new(:url, :page) do
+    include SendChain
+  end
+end
+
+RSpec.describe(LruFile) do
+  lru_file = described_class.new 'abc', 'def'
+
+  it 'can perform a simple call if no arguments are required' do
+    # Equivalent to: lru_file.url.reverse
+    actual = lru_file.send_chain %i[url reverse]
+    expect(actual).to eq('cba')
+  end
+
+  it 'can accept a scalar argument in stages' do
+    lru_file.new_chain [:url, %i[end_with? placeholder]]
+    # Equivalent to: lru_file.url.end_with?('bc')
+    substituted_chain = lru_file.substitute_chain_with 'bc'
+    actual = lru_file.send_chain substituted_chain
+    expect(actual).to be true
+  end
+
+  it 'can accept a vector argument in stages' do
+    lru_file.new_chain [:url, %i[end_with? placeholder]]
+    # Next 2 lines are equivalent to: lru_file.url.end_with?('bc')
+    substituted_chain = lru_file.substitute_chain_with ['bc']
+    actual = lru_file.send_chain substituted_chain
+    expect(actual).to be true
+  end
+
+  it 'can accept a scalar argument in one stage' do
+    lru_file.new_chain [:url, %i[end_with? placeholder]]
+    # Equivalent to: lru_file.url.end_with?('bc')
+    actual = lru_file.substitute_and_send_chain_with 'bc'
+    expect(actual).to be true
+  end
+
+  it 'can accept an array argument in one stage' do
+    lru_file.new_chain [:url, %i[end_with? placeholder]]
+    # Equivalent to: lru_file.url.end_with?('bc')
+    actual = lru_file.substitute_and_send_chain_with ['bc']
+    expect(actual).to be true
+  end
+
+  it 'can use the evaluate_with alias' do
+    lru_file.new_chain [:url, %i[end_with? placeholder]]
+    # Equivalent to: lru_file.url.end_with?('bc')
+    actual = lru_file.evaluate_with ['bc']
+    expect(actual).to be true
+  end
+
+  it 'can reuse the chain with different values' do
+    lru_file.new_chain [:url, %i[end_with? placeholder]]
+
+    # Equivalent to: lru_file.url.end_with?('bc')
+    actual = lru_file.substitute_and_send_chain_with 'bc'
+    expect(actual).to be true
+
+    # Equivalent to: lru_file.url.end_with?('abc')
+    substituted_chain = lru_file.substitute_chain_with ['abc']
+    actual = lru_file.send_chain substituted_chain
+    expect(actual).to be true
+
+    # Equivalent to: lru_file.url.end_with?('de')
+    actual = lru_file.substitute_and_send_chain_with 'de'
+    expect(actual).to be false
+  end
+end

data/spec/send_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+# Exploring how to set date attributes in an object
+
+class Obj
+  # `last_modified` is primary sort key
+  # `date` (when specified) is secondary sort key
+  attr_reader :date, :last_modified
+
+  def initialize(param1, param2)
+    @last_modified = Date.parse(param1)
+    @date = Date.parse(param2)
+  end
+end
+
+RSpec.describe(Obj) do
+  let(:obj) { described_class.new('2000-01-01', '2001-01-01') }
+
+  it 'can send date' do
+    date = obj.send :date
+    expect(obj.date).to eq(date)
+  end
+
+  it 'can send last_modified' do
+    last_modified = obj.send :last_modified
+    expect(obj.last_modified).to eq(last_modified)
+  end
+end

data/spec/sorted_lru_files_spec.rb

@@ -0,0 +1,82 @@
+require 'spec_helper'
+require_relative '../lib/hooks/all_files'
+
+RSpec.describe(SortedLruFiles) do
+  expected1 = 'first-page.html'
+  expected2 = 'second-page.html'
+  expected3 = 'third-page.html'
+
+  it 'can read back an inserted item' do
+    sorted_files = described_class.new
+    sorted_files.insert(expected1, "https://mslinn.com/#{expected1}") # insert reverses expected1
+    sorted_files.enable_search
+
+    actual = sorted_files.select expected1
+    expect(actual.length).to eq(1)
+    expect(actual.first&.reverse&.start_with?(expected1.reverse)).to be true
+
+    actual = sorted_files.select expected1[5..]
+    expect(actual.length).to eq(1)
+    expect(actual.first&.reverse&.start_with?(expected1[5..].reverse)).to be true
+
+    actual = sorted_files.select 'should_not_match'
+    expect(actual).to be_empty
+  end
+
+  it 'works with 2 items and one match' do
+    sorted_files = described_class.new
+    sorted_files.insert(expected1, "https://mslinn.com/#{expected1}") # insert reverses expected1
+    sorted_files.insert(expected2, "https://mslinn.com/#{expected2}")
+    sorted_files.enable_search
+
+    expect(sorted_files.msbs.array[0].url).to be <= sorted_files.msbs.array[1].url
+
+    actual = sorted_files.select expected1
+    expect(actual.length).to eq(1)
+    expect(actual.first&.end_with?(expected1)).to be true
+
+    actual = sorted_files.select expected2
+    expect(actual.length).to eq(1)
+    expect(actual.first&.end_with?(expected2)).to be true
+  end
+
+  it 'handles multiple matches' do
+    sorted_files = described_class.new
+    sorted_files.insert(expected1, "https://mslinn.com/#{expected1}") # insert reverses expected1
+    sorted_files.insert(expected2, "https://mslinn.com/#{expected2}")
+    sorted_files.enable_search
+
+    expected = '.html'
+    actual = sorted_files.select expected
+    expect(actual.length).to eq(2)
+    expect(actual.first&.end_with?(expected)).to be true
+  end
+
+  it 'works with 3 items' do
+    sorted_files = described_class.new
+    sorted_files.insert(expected1, "https://mslinn.com/#{expected1}") # insert reverses expected1
+    sorted_files.insert(expected2, "https://mslinn.com/#{expected2}")
+    sorted_files.insert(expected3, "https://mslinn.com/#{expected3}")
+    sorted_files.enable_search
+
+    expect(sorted_files.msbs.array[0].url).to be <= sorted_files.msbs.array[1].url
+    expect(sorted_files.msbs.array[1].url).to be <= sorted_files.msbs.array[2].url
+
+    actual = sorted_files.select expected1
+    expect(actual.length).to eq(1)
+    expect(actual.first&.end_with?(expected1)).to be true
+
+    actual = sorted_files.select expected2
+    expect(actual.length).to eq(1)
+    expect(actual.first&.end_with?(expected2)).to be true
+
+    actual = sorted_files.select expected3
+    expect(actual.length).to eq(1)
+    expect(actual.first&.end_with?(expected3)).to be true
+
+    expected = '.html'
+    actual = sorted_files.select expected
+    expect(actual.length).to eq(3)
+    expect(actual.first&.end_with?(expected)).to be true
+  end
+end

data/spec/spec_helper.rb

@@ -1,8 +1,6 @@
 require 'jekyll'
 require 'date'
 
-require_relative '../lib/jekyll_all_collections'
-
 Jekyll.logger.log_level = :info
 
 RSpec.configure do |config|
@@ -10,5 +8,5 @@ RSpec.configure do |config|
   config.filter_run_when_matching focus: true
 
   # See https://relishapp.com/rspec/rspec-core/docs/command-line/only-failures
-  config.example_status_persistence_file_path = 'spec/status_persistence.txt'
+  config.example_status_persistence_file_path = '../spec/status_persistence.txt'
 end

data/spec/status_persistence.txt

@@ -1,22 +1,32 @@
-example_id                                                        | status | run_time        |
------------------------------------------------------------------ | ------ | --------------- |
-./spec/all_collections_tag/all_collections_tag_sort_spec.rb[1:1]  | passed | 0.00138 seconds |
-./spec/all_collections_tag/all_collections_tag_sort_spec.rb[1:2]  | passed | 0.00185 seconds |
-./spec/all_collections_tag/all_collections_tag_sort_spec.rb[1:3]  | passed | 0.0016 seconds  |
-./spec/all_collections_tag/all_collections_tag_sort_spec.rb[1:4]  | passed | 0.00169 seconds |
-./spec/all_collections_tag/all_collections_tag_sort_spec.rb[1:5]  | passed | 0.0012 seconds  |
-./spec/all_collections_tag/all_collections_tag_sort_spec.rb[1:6]  | passed | 0.00154 seconds |
-./spec/all_collections_tag/all_collections_tag_sort_spec.rb[1:7]  | passed | 0.00122 seconds |
-./spec/all_collections_tag/all_collections_tag_sort_spec.rb[1:8]  | passed | 0.0015 seconds  |
-./spec/all_collections_tag/all_collections_tag_sort_spec.rb[1:9]  | passed | 0.00141 seconds |
-./spec/all_collections_tag/all_collections_tag_sort_spec.rb[1:10] | passed | 0.00118 seconds |
-./spec/date_sort_spec.rb[1:1]                                     | passed | 0.00007 seconds |
-./spec/date_sort_spec.rb[1:2]                                     | passed | 0.00005 seconds |
-./spec/date_sort_spec.rb[1:3]                                     | passed | 0.00005 seconds |
-./spec/date_sort_spec.rb[1:4]                                     | passed | 0.00005 seconds |
-./spec/date_sort_spec.rb[1:5]                                     | passed | 0.00011 seconds |
-./spec/date_sort_spec.rb[1:6]                                     | passed | 0.00004 seconds |
-./spec/date_sort_spec.rb[1:7]                                     | passed | 0.00006 seconds |
-./spec/date_sort_spec.rb[1:8]                                     | passed | 0.00004 seconds |
-./spec/date_sort_spec.rb[1:9]                                     | passed | 0.00004 seconds |
-./spec/date_sort_spec.rb[1:10]                                    | passed | 0.00005 seconds |
+example_id                                                                                  | status  | run_time        |
+------------------------------------------------------------------------------------------- | ------- | --------------- |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/bsearch_spec.rb[1:1]              | pending | 0.00041 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/bsearch_spec.rb[1:2]              | pending | 0.00005 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/date_sort_spec.rb[1:1]            | passed  | 0.00135 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/date_sort_spec.rb[1:2]            | passed  | 0.00012 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/date_sort_spec.rb[1:3]            | passed  | 0.00036 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/date_sort_spec.rb[1:4]            | passed  | 0.00149 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/date_sort_spec.rb[1:5]            | passed  | 0.00016 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/date_sort_spec.rb[1:6]            | passed  | 0.00046 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/date_sort_spec.rb[1:7]            | passed  | 0.00075 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/date_sort_spec.rb[1:8]            | passed  | 0.00056 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/date_sort_spec.rb[1:9]            | passed  | 0.00029 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/date_sort_spec.rb[1:10]           | passed  | 0.00015 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/mslinn_binary_search_spec.rb[1:1] | passed  | 0.00274 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/mslinn_binary_search_spec.rb[1:2] | passed  | 0.0001 seconds  |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/mslinn_binary_search_spec.rb[1:3] | passed  | 0.00022 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/send_chain_spec.rb[1:1]           | passed  | 0.00022 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/send_chain_spec.rb[1:2]           | passed  | 0.00126 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/send_chain_spec.rb[1:3]           | passed  | 0.00004 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/send_chain_spec.rb[1:4]           | passed  | 0.00093 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/send_chain_spec.rb[1:5]           | passed  | 0.0001 seconds  |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/send_chain_spec.rb[1:6]           | passed  | 0.00003 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/send_chain_spec.rb[1:7]           | passed  | 0.00026 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/send_spec.rb[1:1]                 | passed  | 0.00038 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/send_spec.rb[1:2]                 | passed  | 0.00007 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/sorted_lru_files_spec.rb[1:1]     | passed  | 0.00139 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/sorted_lru_files_spec.rb[1:2]     | passed  | 0.00044 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/sorted_lru_files_spec.rb[1:3]     | passed  | 0.0001 seconds  |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/sorted_lru_files_spec.rb[1:4]     | passed  | 0.00057 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/testable_spec.rb[1:1]             | passed  | 0.00027 seconds |
+/mnt/f/work/jekyll/my_plugins/jekyll_all_collections/spec/testable_spec.rb[2:1]             | passed  | 0.00171 seconds |

data/spec/testable_spec.rb

@@ -0,0 +1,38 @@
+require 'spec_helper'
+
+# Exploring how to invoke methods from classes and modules
+# See https://mslinn.com/jekyll/10700-designing-for-testability.html
+
+module TestModule
+  def a_method
+    'a_method says Hi!'
+  end
+end
+
+class TestClass
+  extend TestModule # Defines class methods
+
+  def initialize(param1, param2)
+    super()
+    @param1 = param1
+    @param2 = param2
+  end
+end
+
+RSpec.describe(TestModule) do
+  extend described_class
+
+  it 'Invokes a_method from module' do
+    result = self.class.a_method
+    expect(result).to eq('a_method says Hi!')
+  end
+end
+
+RSpec.describe(TestClass) do
+  let(:o1) { described_class.new('value1', 'value2') }
+
+  it 'Invokes a_method from class' do
+    result = o1.class.a_method
+    expect(result).to eq('a_method says Hi!')
+  end
+end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: jekyll_all_collections
 version: !ruby/object:Gem::Version
-  version: 0.3.6
+  version: 0.4.0
 platform: ruby
 authors:
 - Mike Slinn
-autorequire:
-bindir: exe
+bindir: bin
 cert_chain: []
-date: 2024-07-26 00:00:00.000000000 Z
+date: 2025-02-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll
@@ -52,7 +51,21 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 'Provides a collection of all collections in site.all_collections.
+- !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: 'Provides normalized collections and extra functionality for Jekyll websites.
 
   '
 email:
@@ -67,14 +80,25 @@ files:
 - README.md
 - Rakefile
 - jekyll_all_collections.gemspec
-- lib/all_collections_hooks.rb
-- lib/all_collections_tag.rb
+- lib/hooks/a_page.rb
+- lib/hooks/all_collections_hooks.rb
+- lib/hooks/all_files.rb
+- lib/hooks/class_methods.rb
 - lib/jekyll_all_collections.rb
 - lib/jekyll_all_collections/version.rb
+- lib/tag/all_collections_tag.rb
+- lib/util/mslinn_binary_search.rb
+- lib/util/send_chain.rb
 - spec/all_collections_tag/all_collections_tag_sort_spec.rb
+- spec/bsearch_spec.rb
 - spec/date_sort_spec.rb
+- spec/mslinn_binary_search_spec.rb
+- spec/send_chain_spec.rb
+- spec/send_spec.rb
+- spec/sorted_lru_files_spec.rb
 - spec/spec_helper.rb
 - spec/status_persistence.txt
+- spec/testable_spec.rb
 homepage: https://www.mslinn.com/jekyll_plugins/jekyll_all_collections.html
 licenses:
 - MIT
@@ -84,7 +108,6 @@ metadata:
   changelog_uri: https://github.com/mslinn/jekyll_all_collections/CHANGELOG.md
   homepage_uri: https://www.mslinn.com/jekyll_plugins/jekyll_all_collections.html
   source_code_uri: https://github.com/mslinn/jekyll_all_collections
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -99,8 +122,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.16
-signing_key:
+rubygems_version: 3.6.3
 specification_version: 4
-summary: Provides a collection of all collections in site.all_collections.
+summary: Provides normalized collections and extra functionality for Jekyll websites.
 test_files: []

data/lib/all_collections_hooks.rb

@@ -1,118 +0,0 @@
-require 'jekyll'
-require 'jekyll_plugin_logger'
-require_relative 'jekyll_all_collections/version'
-
-# Creates an array of `APage` called site.all_collections, which will be available from :site, :pre_render onwards
-module AllCollectionsHooks
-  @logger = PluginMetaLogger.instance.new_logger(self, PluginMetaLogger.instance.config)
-
-  # No, all_collections is not defined for this hook
-  # Jekyll::Hooks.register(:site, :after_init, priority: :normal) do |site|
-  #   defined = AllCollectionsHooks.all_collections_defined?(site)
-  #   @logger.debug { "Jekyll::Hooks.register(:site, :after_init: #{defined}" }
-  # end
-
-  # Creates a `Array[APage]` property called site.all_collections if it does not already exist
-  # Each `APage` entry is one document or page.
-  Jekyll::Hooks.register(:site, :post_read, priority: :normal) do |site|
-    defined = AllCollectionsHooks.all_collections_defined?(site)
-    @logger.debug { "Jekyll::Hooks.register(:site, :post_read, :normal: #{defined}" }
-    AllCollectionsHooks.compute(site) unless site.class.method_defined? :all_collections
-  end
-
-  # Yes, all_collections is defined for this hook
-  # Jekyll::Hooks.register(:site, :post_read, priority: :low) do |site|
-  #   defined = AllCollectionsHooks.all_collections_defined?(site)
-  #   @logger.debug { "Jekyll::Hooks.register(:site, :post_read, :low: #{defined}" }
-  # end
-
-  # Yes, all_collections is defined for this hook
-  # Jekyll::Hooks.register(:site, :post_read, priority: :normal) do |site|
-  #   defined = AllCollectionsHooks.all_collections_defined?(site)
-  #   @logger.debug { "Jekyll::Hooks.register(:site, :post_read, :normal: #{defined}" }
-  # end
-
-  # Yes, all_collections is defined for this hook
-  # Jekyll::Hooks.register(:site, :pre_render, priority: :normal) do |site, _payload|
-  #   defined = AllCollectionsHooks.all_collections_defined?(site)
-  #   @logger.debug { "Jekyll::Hooks.register(:site, :pre_render: #{defined}" }
-  # end
-
-  def self.compute(site)
-    objects = site.collections
-                  .values
-                  .map { |x| x.class.method_defined?(:docs) ? x.docs : x }
-                  .flatten
-
-    site.class.module_eval { attr_accessor :all_collections }
-    apages = AllCollectionsHooks.apages_from_objects(objects)
-    site.all_collections = apages
-  end
-
-  @sort_by = ->(apages, criteria) { [apages.sort(criteria)] }
-
-  # The collection value is just the collection label, not the entire collection object
-  def self.apages_from_objects(objects)
-    pages = []
-    objects.each do |object|
-      page = APage.new(object)
-      pages << page unless page.data['exclude_from_all']
-    end
-    pages
-  end
-
-  def self.all_collections_defined?(site)
-    "site.all_collections #{site.class.method_defined?(:all_collections) ? 'IS' : 'IS NOT'} defined"
-  end
-
-  class APage
-    attr_reader :content, :data, :date, :description, :destination, :draft, :excerpt, :ext, :extname,
-                :label, :last_modified, :layout, :path, :relative_path, :tags, :title, :type, :url
-
-    # Verify each property exists before accessing it; this helps write tests
-    def initialize(obj) # rubocop:disable Metrics/AbcSize
-      @data = obj.data if obj.respond_to? :data
-
-      @categories = @data['categories'] if @data.key? 'categories'
-      @content = obj.content if obj.respond_to? :content
-      @date = (@data['date'].to_date if @data&.key?('date')) || Date.today
-      @description = @data['description'] if @data.key? 'description'
-
-      # TODO: What _config.yml setting should be passed to destination()?
-      @destination = obj.destination('') if obj.respond_to? :destination
-
-      @draft = Jekyll::Draft.draft?(obj)
-      @excerpt = @data['excerpt'] if @data.key? 'excerpt'
-      @ext = obj.extname
-      @ext ||= @data['ext'] if @data.key? 'ext'
-      @extname = @ext # For compatibility with previous versions of all_collections
-      @label = obj.collection.label if obj&.collection.respond_to? :label
-      @last_modified = @data['last_modified'] || @data['last_modified_at'] || @date
-      @last_modified_field = case @data
-                             when @data.key?('last_modified')
-                               'last_modified'
-                             when @data.key?('last_modified_at')
-                               'last_modified_at'
-                             end
-      @layout = @data['layout'] if @data.key? 'layout'
-      @path = obj.path if obj.respond_to? :path
-      @relative_path = obj.relative_path if obj.respond_to? :relative_path
-      @tags = @data['tags'] if @data.key? 'tags'
-      @title = @data['title'] if @data.key? 'title'
-      @type = obj.type if obj.respond_to? :type
-      @url = obj.url
-    end
-
-    def to_s
-      return @label if @label
-
-      @date.to_s
-    end
-  end
-
-  PluginMetaLogger.instance.logger.info do
-    "Loaded AllCollectionsHooks v#{JekyllAllCollectionsVersion::VERSION} :site, :pre_render, :normal hook plugin."
-  end
-end
-
-Liquid::Template.register_filter(AllCollectionsHooks)