From 5f0c3ff81b43bb9b14934b95cac09684e185da8c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Joakim=20S=C3=B8rensen?= <joasoe@gmail.com>
Date: Thu, 6 Jun 2024 17:26:25 +0200
Subject: [PATCH] Optimize related include (#33104)

---
 plugins/site_pages.rb         | 27 +++++++++++++++++++++++++++
 source/_includes/related.html | 16 ++++++----------
 2 files changed, 33 insertions(+), 10 deletions(-)
 create mode 100644 plugins/site_pages.rb

diff --git a/plugins/site_pages.rb b/plugins/site_pages.rb
new file mode 100644
index 00000000000..517f4571195
--- /dev/null
+++ b/plugins/site_pages.rb
@@ -0,0 +1,27 @@
+module Jekyll
+  class SitePagesGenerator < Jekyll::Generator
+    def generate(site)
+      all_pages = Array.new
+
+      site.collections.each do |name, collection|
+        all_pages.concat(collection.docs)
+      end
+
+      site.data["site_pages"] = all_pages
+        .concat(site.pages)
+        .concat(site.documents)
+        .map { |entry| 
+          [
+            entry.url.to_s,
+            {
+              "title" => entry["title"],
+              "description" => entry["description"],
+              "url" => entry.url,
+              "relative_path" => entry.relative_path
+            }
+          ] 
+        }
+        .to_h
+    end
+  end
+end
\ No newline at end of file
diff --git a/source/_includes/related.html b/source/_includes/related.html
index 81fd4ba91d0..65ffe151326 100644
--- a/source/_includes/related.html
+++ b/source/_includes/related.html
@@ -11,23 +11,19 @@
 <div class="text related-topics">
   <h2><a class="title-link" href="#related-topics" name="related-topics"></a>Related topics</h2>
   <ul>
-    {% assign site_part = "pages,documents" | split: "," %}
     {%- for related in related_topics -%}
-    {%- for part in site_part -%}
-    {%- for entry in site[part] -%}
     {% assign computed = related.docs | split: "#" %}
+    {% assign target_url = computed[0] %}
+    {% assign target_page = site.data["site_pages"][target_url] %}
+    {% unless target_page == null %}
     {% assign link_target = computed[1] %}
     {% assign suggested_title = related.title | default: link_target | split: "-" | join: " " | capitalize %}
-    {%- if computed[0] == entry.url -%}
     <li>
-      <a href="{{ entry.url | relative_url }}{% if link_target %}#{{link_target}}{% endif %}" class="topic">
-        {{ suggested_title | default: entry.title}}
+      <a href="{{ target_page['url'] | relative_url }}{% if link_target %}#{{link_target}}{% endif %}" class="topic">
+        {{ suggested_title | default: target_page["title"]}}
       </a>
     </li>
-    {% break %}{% break %}
-    {%- endif -%}
-    {%- endfor -%}
-    {%- endfor -%}
+    {% endunless %}
     {%- endfor -%}
   </ul>
 </div>