reorder

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

@@ -17,12 +17,10 @@ class Admin::Communication::Website::PagesController < Admin::Communication::Web
       pages.concat(page.descendents) if parent_id != page.parent_id
       page.update(
         parent_id: parent_id,
-        position: index + 1,
-        skip_github_publication: true
+        position: index + 1
       )
     end
-    github = Github.with_website @website
-    github.send_batch_to_website(pages, message: '[Page] Reorder pages.')
+    pages.first.sync_with_git
   end
 
   def children

app/models/communication/website/page.rb

@@ -76,7 +76,7 @@ class Communication::Website::Page < ApplicationRecord
   end
 
   def git_dependencies(identifier)
-    descendents
+    descendents + siblings
   end
 
   def list_of_other_pages

app/models/concerns/with_git.rb

@@ -40,17 +40,6 @@ module WithGit
     destroy
   end
 
-  protected
-
-  # Overridden for multiple files generation
-  def identifiers
-    [:static]
-  end
-
-  def git_dependencies(identifier)
-    []
-  end
-
   def sync_with_git
     websites.each do |website|
       identifiers.each do |identifier|
@@ -63,4 +52,15 @@ module WithGit
     end
   end
   handle_asynchronously :sync_with_git
+
+  protected
+
+  # Overridden for multiple files generation
+  def identifiers
+    [:static]
+  end
+
+  def git_dependencies(identifier)
+    []
+  end
 end

app/models/concerns/with_tree.rb

@@ -23,6 +23,10 @@ module WithTree
                     : []
     end
 
+    def siblings
+      self.class.where(parent: parent).where.not(id: id)
+    end
+
     def self_and_children(level)
       elements = []
       label = "   " * level + self.to_s

docs/websites/export.md

@@ -69,8 +69,12 @@ end
 ```
 
 
-Pour les reorder :
+Pour les reorder, chaque objet doit avoir ses siblings en dépendance, donc il suffit de synchroniser un objet déplacé pour qu'il gère l'ensemble :
 ```
+def reorder
+  ...
+  pages.first.sync_with_git
+end
 ```
 
 TODO gérer la suppression correctement