Working

app/controllers/admin/communication/websites/pages_controller.rb

@@ -24,6 +24,7 @@ class Admin::Communication::Websites::PagesController < Admin::Communication::We
     end
     old_parent_page.sync_with_git
     parent_page.sync_with_git
+    @website.generate_automatic_menus
   end
 
   def children

app/models/communication/website/menu/with_automatism.rb

 module Communication::Website::Menu::WithAutomatism
   extend ActiveSupport::Concern
 
-  def create_automatic_menu
+  included do 
+    scope :automatic, -> { where(automatic: true) }
+  end
 
+  def generate_automatically
+    begin
+      suspend_git_sync
+      clear_items
+      create_items
+    ensure
+      reactivate_git_sync
+    end
   end
 
   def stop_automatism!
     update_column :automatic, false
   end
+
+  protected
   
+  def suspend_git_sync
+    # TODO
+  end
+
+  def clear_items
+    # TODO est-ce bien de ne pas gérer les dépendances avec un destroy? Effet de bord?
+    items.delete_all
+  end
+
+  def create_items
+    home = website.pages.root.first
+    create_items_for_children_of(home)
+  end
+
+  def create_items_for_children_of(page, parent_item = nil)
+    page.children.ordered.each do |child_page|
+      next if child_page.default_menu_identifier != identifier
+      create_item_for(child_page, parent_item)
+    end
+  end
+
+  def create_item_for(page, parent_item = nil)
+    item = items.create kind: :page,
+                        about: page,
+                        title: page.title,
+                        position: page.position,
+                        website: website,
+                        university: university,
+                        parent: parent_item
+    create_items_for_children_of(page, item)
+  end
+
+  def reactivate_git_sync
+    # TODO
+  end
 end
\ No newline at end of file

app/models/communication/website/page.rb

@@ -48,6 +48,7 @@ class Communication::Website::Page < ApplicationRecord
   include AsDirectObject
   include Sanitizable
   include WithAccessibility
+  include WithAutomaticMenus
   include WithBlobs
   include WithBlocks
   include WithDuplication

app/models/communication/website/page/accessibility.rb

@@ -51,6 +51,10 @@ class Communication::Website::Page::Accessibility < Communication::Website::Page
     false
   end
 
+  def default_menu_identifier
+    'legal'
+  end
+
   def generate_from_template
     generate_declaration
     generate_results

app/models/communication/website/page/legal_term.rb

@@ -50,5 +50,9 @@ class Communication::Website::Page::LegalTerm < Communication::Website::Page
   def is_listed_among_children?
     false
   end
+
+  def default_menu_identifier
+    'legal'
+  end
   
 end

app/models/communication/website/page/privacy_policy.rb

@@ -50,5 +50,9 @@ class Communication::Website::Page::PrivacyPolicy < Communication::Website::Page
   def is_listed_among_children?
     false
   end
+
+  def default_menu_identifier
+    'legal'
+  end
   
 end

app/models/communication/website/page/sitemap.rb

@@ -67,4 +67,8 @@ class Communication::Website::Page::Sitemap < Communication::Website::Page
     'sitemap'
   end
 
+  def default_menu_identifier
+    'legal'
+  end
+
 end

app/models/communication/website/page/with_automatic_menus.rb

+module Communication::Website::Page::WithAutomaticMenus
+  extend ActiveSupport::Concern
+
+  included do
+    after_save :generate_automatic_menus
+  end
+
+  protected
+
+  def generate_automatic_menus
+    website.generate_automatic_menus
+  end
+end
\ No newline at end of file

app/models/communication/website/page/with_type.rb

@@ -86,6 +86,10 @@ module Communication::Website::Page::WithType
     nil
   end
 
+  def default_menu_identifier
+    'primary'
+  end
+
   def generate_from_template
   end
 

app/models/communication/website/with_menus.rb

@@ -64,9 +64,13 @@ module Communication::Website::WithMenus
   def initialize_menus
     find_or_create_menu 'primary'
     find_or_create_menu 'social'
-    find_or_create_menu('legal') do |menu|
-      # Only executed after menu creation
-      fill_legal_menu(menu)
+    find_or_create_menu 'legal'
+    generate_automatic_menus
+  end
+
+  def generate_automatic_menus
+    menus.automatic.find_each do |menu|
+      menu.generate_automatically
     end
   end
 
@@ -76,24 +80,6 @@ module Communication::Website::WithMenus
     menu = menus.where(identifier: identifier, university: university, language: default_language).first_or_initialize do |menu|
       menu.title = I18n.t("communication.website.menus.default_title.#{identifier}")
     end
-    unless menu.persisted?
-      menu.save
-      yield(menu) if block_given?
-    end
-    menu
-  end
-
-  def fill_legal_menu(menu)
-    [
-      Communication::Website::Page::LegalTerm,
-      Communication::Website::Page::PrivacyPolicy,
-      Communication::Website::Page::Accessibility,
-      Communication::Website::Page::Sitemap
-    ].each do |page_class|
-      page = special_page(page_class, language: menu.language)
-      menu.items.where(kind: 'page', about: page, university: university, website: self).first_or_create do |item|
-        item.title = page.title
-      end
-    end
+    menu.save unless menu.persisted?
   end
 end