From ca20cb9941cd5474f1377a3f82e6bf53f1960cbb Mon Sep 17 00:00:00 2001
From: Neil Lalonde <neillalonde@gmail.com>
Date: Thu, 2 Mar 2017 15:06:47 -0500
Subject: [PATCH] FEATURE: subcategories can be discovered by web crawlers on
 page 1 of the parent category topics list

---
 app/controllers/list_controller.rb |  4 ++++
 app/views/list/list.erb            | 28 +++++++++++++++++++++++-----
 2 files changed, 27 insertions(+), 5 deletions(-)

diff --git a/app/controllers/list_controller.rb b/app/controllers/list_controller.rb
index a3ae278b3e9..f2a4aabac77 100644
--- a/app/controllers/list_controller.rb
+++ b/app/controllers/list_controller.rb
@@ -294,6 +294,10 @@ class ListController < ApplicationController
 
     @description_meta = @category.description_text
     raise Discourse::NotFound unless guardian.can_see?(@category)
+
+    if use_crawler_layout?
+      @subcategories = @category.subcategories.select { |c| guardian.can_see?(c) }
+    end
   end
 
   def build_topic_list_options
diff --git a/app/views/list/list.erb b/app/views/list/list.erb
index 92d76d635eb..ba4b1dff319 100644
--- a/app/views/list/list.erb
+++ b/app/views/list/list.erb
@@ -13,16 +13,34 @@
 <% if @category %>
   <h1>
     <% if @category.parent_category %>
-    <%= link_to @category.parent_category.url, itemprop: 'item' do %>
-      <span itemprop='name'><%= @category.parent_category.name %></span>
-    <% end %>
-    &nbsp;
+      <%= link_to @category.parent_category.url, itemprop: 'item' do %>
+        <span itemprop='name'><%= @category.parent_category.name %></span>
+      <% end %>
+      &nbsp;
     <% end %>
+
     <%= link_to @category.url, itemprop: 'item' do %>
       <span itemprop='name'><%= @category.name %></span>
     <% end %>
   </h1>
   <br/>
+
+  <% if params[:page].to_i == 0 && @subcategories.present? %>
+    <div itemscope itemtype='http://schema.org/ItemList'>
+      <% @subcategories.each do |subcategory| %>
+        <div itemprop='itemListElement' itemscope itemtype='http://schema.org/ListItem'>
+          <meta itemprop='url' content='<%= subcategory.url %>'>
+          <a href='<%= subcategory.url %>' itemprop='item'>
+            <span itemprop='name'><%= subcategory.name %></span>
+          </a>
+          <% if subcategory.description.present? %>
+            <span itemprop='description'><%= subcategory.description %></span>
+          <% end %>
+        </div>
+      <% end %>
+    </div>
+    <br/>
+  <% end %>
 <% end %>
 
 <div class='topic-list' itemscope itemtype='http://schema.org/ItemList'>
@@ -58,7 +76,7 @@
   <% end %>
 <% end %>
 
-<%- end %>
+<%- end %> <!-- include_crawler_content? -->
 
 <% if @rss %>
   <% content_for :head do %>