Program > Portfolio category + related categories (#2339)

* add is_taxonomy to blog and agenda categories + set portfolio program categories

* program related_categories

* fix tests

* category slug with ancestors

* harmonize taxonomies

* add as localized tree to program category l10ns

* harmonisation !

* is_taxonomy

* wip is_taxonomy in person and orga categories

* static categories in orga and people

* fix permalink and git path + hugo_slug

* fix

* static versions

---------

Co-authored-by: Arnaud Levy <arnaud.levy@noesya.coop>
Co-authored-by: Arnaud Levy <contact@arnaudlevy.com>

app/controllers/admin/communication/websites/agenda/categories_controller.rb

@@ -76,7 +76,7 @@ def breadcrumb
   def category_params
     params.require(:communication_website_agenda_category)
           .permit(
-            :parent_id,
+            :is_taxonomy, :parent_id,
             localizations_attributes: [
               :id, :language_id,
               :name, :meta_description, :summary, :slug,

app/controllers/admin/communication/websites/agenda/events_controller.rb

@@ -85,8 +85,7 @@ def breadcrumb
   end
 
   def categories
-    @website.agenda_categories
-            .ordered
+    @website.agenda_categories.ordered
   end
 
   def event_params

app/controllers/admin/communication/websites/posts/categories_controller.rb

@@ -79,7 +79,7 @@ def breadcrumb
   def category_params
     params.require(:communication_website_post_category)
           .permit(
-            :parent_id,
+            :is_taxonomy, :parent_id,
             localizations_attributes: [
               :id, :name, :meta_description, :summary, :slug,
               :featured_image, :featured_image_delete, :featured_image_infos, :featured_image_alt, :featured_image_credit,

app/controllers/admin/university/organizations/categories_controller.rb

@@ -82,7 +82,7 @@ def breadcrumb
 
   def category_params
     params.require(:university_organization_category).permit(
-      :parent_id,
+      :is_taxonomy, :parent_id,
       localizations_attributes: [
         :id, :name, :slug, :language_id
       ]).merge(university_id: current_university.id)

app/controllers/admin/university/people/categories_controller.rb

@@ -83,7 +83,7 @@ def breadcrumb
 
   def category_params
     params.require(:university_person_category).permit(
-      :parent_id,
+      :is_taxonomy, :parent_id,
       localizations_attributes: [
         :id, :name, :slug, :language_id
       ]).merge(university_id: current_university.id)

app/models/communication/website/agenda/category.rb

@@ -4,6 +4,7 @@
 #
 #  id                       :uuid             not null, primary key
 #  is_programs_root         :boolean          default(FALSE)
+#  is_taxonomy              :boolean          default(FALSE)
 #  position                 :integer
 #  created_at               :datetime         not null
 #  updated_at               :datetime         not null
@@ -27,24 +28,16 @@
 #  fk_rails_7b5ad84dda  (communication_website_id => communication_websites.id)
 #
 class Communication::Website::Agenda::Category < ApplicationRecord
+  include AsCategory
   include AsDirectObject
   include Localizable
-  include Orderable
   include Sanitizable
   include WithMenuItemTarget
-  include WithTree
   include WithUniversity
 
-  belongs_to              :parent,
-                          class_name: 'Communication::Website::Agenda::Category',
-                          optional: true
   belongs_to              :program,
                           class_name: 'Education::Program',
                           optional: true
-  has_many                :children,
-                          class_name: 'Communication::Website::Agenda::Category',
-                          foreign_key: :parent_id,
-                          dependent: :destroy
   has_and_belongs_to_many :events,
                           class_name: 'Communication::Website::Agenda::Event',
                           join_table: :communication_website_agenda_events_categories,

app/models/communication/website/agenda/category/localization.rb

@@ -54,7 +54,9 @@ class Communication::Website::Agenda::Category::Localization < ApplicationRecord
   before_validation :set_communication_website_id, on: :create
 
   def git_path(website)
-    "#{git_path_content_prefix(website)}events_categories/#{slug}/_index.html"
+    prefix = git_path_content_prefix(website)
+    slugs = slug_with_ancestors_slugs(separator: '-')
+    "#{prefix}events_categories/#{slugs}/_index.html"
   end
 
   def template_static
@@ -95,4 +97,9 @@ def inherited_blob_ids
   def set_communication_website_id
     self.communication_website_id ||= about.communication_website_id
   end
+
+  # TODO : Pertinent ?
+  def hugo_slug_in_website(website)
+    slug_with_ancestors_slugs(separator: '-')
+  end
 end

app/models/communication/website/menu.rb

@@ -64,7 +64,7 @@ def to_s
   end
 
   def git_path(website)
-    "data/menus/#{language.iso_code}/#{identifier}.yml"
+    "data/menus/#{language.iso_code}/#{identifier}.yml" if website.active_language_ids.include?(language_id)
   end
 
   def template_static

app/models/communication/website/permalink/agenda/category.rb

@@ -24,7 +24,7 @@ def published?
 
   def substitutions
     {
-      slug: about.slug
+      slug: about.slug_with_ancestors_slugs
     }
   end
 

app/models/communication/website/permalink/organization/category.rb

@@ -20,7 +20,7 @@ def self.special_page_type
 
   def substitutions
     {
-      slug: about.slug
+      slug: about.slug_with_ancestors_slugs
     }
   end
 

app/models/communication/website/permalink/person/category.rb

@@ -20,7 +20,7 @@ def self.special_page_type
 
   def substitutions
     {
-      slug: about.slug
+      slug: about.slug_with_ancestors_slugs
     }
   end
 

app/models/communication/website/permalink/portfolio/category.rb

@@ -24,7 +24,7 @@ def published?
 
   def substitutions
     {
-      slug: about.slug
+      slug: about.slug_with_ancestors_slugs
     }
   end
 

app/models/communication/website/portfolio/category.rb

@@ -60,6 +60,10 @@ def references
     [parent]
   end
 
+  def siblings
+    self.class.unscoped.where(parent: parent, university: university, website: website).where.not(id: id)
+  end
+
   protected
 
   def last_ordered_element

app/models/communication/website/portfolio/category/localization.rb

@@ -53,7 +53,9 @@ class Communication::Website::Portfolio::Category::Localization < ApplicationRec
   before_validation :set_communication_website_id, on: :create
 
   def git_path(website)
-    "#{git_path_content_prefix(website)}projects_categories/#{slug}/_index.html"
+    prefix = git_path_content_prefix(website)
+    slugs = slug_with_ancestors_slugs(separator: '-')
+    "#{prefix}projects_categories/#{slugs}/_index.html"
   end
 
   def template_static
@@ -86,4 +88,9 @@ def slug_unavailable?(slug)
   def set_communication_website_id
     self.communication_website_id ||= about.communication_website_id
   end
+
+  # TODO : Pertinent ?
+  def hugo_slug_in_website(website)
+    slug_with_ancestors_slugs(separator: '-')
+  end
 end

app/models/communication/website/post/category.rb

@@ -4,6 +4,7 @@
 #
 #  id                       :uuid             not null, primary key
 #  is_programs_root         :boolean          default(FALSE)
+#  is_taxonomy              :boolean          default(FALSE)
 #  position                 :integer
 #  created_at               :datetime         not null
 #  updated_at               :datetime         not null
@@ -27,24 +28,16 @@
 #  fk_rails_e58348b119  (program_id => education_programs.id)
 #
 class Communication::Website::Post::Category < ApplicationRecord
+  include AsCategory
   include AsDirectObject
   include Localizable
-  include Orderable
   include Sanitizable
   include WithMenuItemTarget
-  include WithTree
   include WithUniversity
 
-  belongs_to              :parent,
-                          class_name: 'Communication::Website::Post::Category',
-                          optional: true
   belongs_to              :program,
                           class_name: 'Education::Program',
                           optional: true
-  has_many                :children,
-                          class_name: 'Communication::Website::Post::Category',
-                          foreign_key: :parent_id,
-                          dependent: :destroy
   has_and_belongs_to_many :posts,
                           class_name: 'Communication::Website::Post',
                           join_table: :communication_website_categories_posts,

app/models/communication/website/post/category/localization.rb

@@ -99,4 +99,9 @@ def inherited_blob_ids
   def set_communication_website_id
     self.communication_website_id ||= about.communication_website_id
   end
+
+  # TODO : Pertinent ?
+  def hugo_slug_in_website(website)
+    slug_with_ancestors_slugs(separator: '-')
+  end
 end

app/models/communication/website/with_program_categories.rb

@@ -10,7 +10,7 @@ def set_programs_categories
   end
 
   def set_programs_categories_safely
-    [post_categories, agenda_categories].each do |objects|
+    [post_categories, agenda_categories, portfolio_categories].each do |objects|
       programs_root_category = set_programs_categories_root_for!(objects)
       set_programs_categories_at_level_for! objects, programs_root_category, education_programs.root.ordered
     end

app/models/education/program/category/localization.rb

@@ -25,6 +25,7 @@
 #
 class Education::Program::Category::Localization < ApplicationRecord
   include AsLocalization
+  include AsLocalizedTree
   include Contentful
   include Initials
   include Permalinkable
@@ -49,4 +50,11 @@ def dependencies
   def to_s
     "#{name}"
   end
+
+  protected
+
+  # TODO : Pertinent ?
+  def hugo_slug_in_website(website)
+    slug_with_ancestors_slugs(separator: '-')
+  end
 end

app/models/education/program/with_websites_categories.rb

@@ -4,18 +4,28 @@ module Education::Program::WithWebsitesCategories
   included do
     after_save_commit :set_websites_categories
 
-    has_many   :website_post_categories,
-               class_name: 'Communication::Website::Post::Category',
-               dependent: :destroy
-
     has_many   :website_agenda_categories,
                class_name: 'Communication::Website::Agenda::Category',
                dependent: :destroy
 
-    # TODO : Ajouter les catégories de portfolio
+    has_many   :website_portfolio_categories,
+               class_name: 'Communication::Website::Portfolio::Category',
+               dependent: :destroy
+
+    has_many   :website_post_categories,
+               class_name: 'Communication::Website::Post::Category',
+               dependent: :destroy
   end
 
   def set_websites_categories
     websites.each { |website| website.set_programs_categories }
   end
+
+  def website_category_l10ns_for(website, language)
+    categories = {}
+    categories[:agenda] = website_agenda_categories.find_by(communication_website_id: website.id)&.localization_for(language)
+    categories[:blog] = website_post_categories.find_by(communication_website_id: website.id)&.localization_for(language)
+    categories[:portfolio] = website_portfolio_categories.find_by(communication_website_id: website.id)&.localization_for(language)
+    categories.compact
+  end
 end

app/models/university/organization/category.rb

@@ -3,6 +3,7 @@
 # Table name: university_organization_categories
 #
 #  id            :uuid             not null, primary key
+#  is_taxonomy   :boolean          default(FALSE)
 #  position      :integer          default(0)
 #  created_at    :datetime         not null
 #  updated_at    :datetime         not null
@@ -20,19 +21,11 @@
 #  fk_rails_f610c7eb13  (university_id => universities.id)
 #
 class University::Organization::Category < ApplicationRecord
+  include AsCategory
   include AsIndirectObject
   include Localizable
-  include Orderable
-  include WithTree
   include WithUniversity
 
-  belongs_to :parent,
-             class_name: 'University::Organization::Category',
-             optional: true
-  has_many   :children,
-             class_name: 'University::Organization::Category',
-             foreign_key: :parent_id,
-             dependent: :destroy
   has_and_belongs_to_many :organizations,
                           class_name: 'University::Organization',
                           join_table: :university_organizations_categories

app/models/university/organization/category/localization.rb

@@ -41,7 +41,10 @@ def dependencies
   end
 
   def git_path(website)
-    "#{git_path_content_prefix(website)}organizations_categories/#{slug}/_index.html" if for_website?(website)
+    return unless for_website?(website)
+    prefix = git_path_content_prefix(website)
+    slugs = slug_with_ancestors_slugs(separator: '-')
+    "#{prefix}organizations_categories/#{slugs}/_index.html"
   end
 
   def template_static
@@ -56,4 +59,9 @@ def published?
     persisted?
   end
 
+  # TODO : Pertinent ?
+  def hugo_slug_in_website(website)
+    slug_with_ancestors_slugs(separator: '-')
+  end
+
 end

app/models/university/person/category.rb

@@ -3,6 +3,7 @@
 # Table name: university_person_categories
 #
 #  id            :uuid             not null, primary key
+#  is_taxonomy   :boolean          default(FALSE)
 #  position      :integer          default(0)
 #  created_at    :datetime         not null
 #  updated_at    :datetime         not null
@@ -20,19 +21,11 @@
 #  fk_rails_4c00a79930  (parent_id => university_person_categories.id)
 #
 class University::Person::Category < ApplicationRecord
+  include AsCategory
   include AsIndirectObject
   include Localizable
-  include Orderable
-  include WithTree
   include WithUniversity
 
-  belongs_to :parent,
-             class_name: 'University::Person::Category',
-             optional: true
-  has_many   :children,
-             class_name: 'University::Person::Category',
-             foreign_key: :parent_id,
-             dependent: :destroy
   has_and_belongs_to_many :people,
                           class_name: 'University::Person',
                           join_table: :university_people_categories

app/models/university/person/category/localization.rb

@@ -41,7 +41,10 @@ def dependencies
   end
 
   def git_path(website)
-    "#{git_path_content_prefix(website)}persons_categories/#{slug}/_index.html" if for_website?(website)
+    return unless for_website?(website)
+    prefix = git_path_content_prefix(website)
+    slugs = slug_with_ancestors_slugs(separator: '-')
+    "#{prefix}persons_categories/#{slugs}/_index.html"
   end
 
   def template_static
@@ -56,4 +59,9 @@ def published?
     persisted?
   end
 
+  # TODO : Pertinent ?
+  def hugo_slug_in_website(website)
+    slug_with_ancestors_slugs(separator: '-')
+  end
+
 end