Merge branch 'master' of github.com:noesya/osuny

app/models/communication/website/git_file.rb

@@ -59,13 +59,13 @@ class Communication::Website::GitFile < ApplicationRecord
   end
 
   def path
-    about.send "git_path_#{identifier}"
+    @path ||= about.send "git_path_#{identifier}"
   end
 
   def sha
     # Git SHA-1 is calculated from the String "blob <length>\x00<contents>"
     # Source: https://alblue.bandlem.com/2011/08/git-tip-of-week-objects.html
-    OpenSSL::Digest::SHA1.hexdigest "blob #{to_s.bytesize}\x00#{to_s}"
+    @sha ||= OpenSSL::Digest::SHA1.hexdigest "blob #{to_s.bytesize}\x00#{to_s}"
   end
 
   def to_s

app/models/concerns/with_git.rb

@@ -8,7 +8,6 @@ module WithGit
               dependent: :destroy
   end
 
-  # Needs override
   def git_path_static
     raise NotImplementedError
   end
@@ -34,39 +33,38 @@ module WithGit
   def destroy_and_sync
     destroy_from_git
     destroy
-    true
   end
 
-  def destroy_from_git
-    websites_with_fallback.each do |website|
+  def sync_with_git
+    websites_for_self.each do |website|
       identifiers(website: website).each do |identifier|
-        Communication::Website::GitFile.sync website, self, identifier, destroy: true
-        dependencies = send "git_destroy_dependencies_#{identifier}"
+        Communication::Website::GitFile.sync website, self, identifier
+        dependencies = send "git_dependencies_#{identifier}"
         dependencies.each do |object|
-          Communication::Website::GitFile.sync website, object, identifier, destroy: true
+          Communication::Website::GitFile.sync website, object, identifier
         end
       end
       website.git_repository.sync!
     end
   end
+  handle_asynchronously :sync_with_git
 
-  def sync_with_git
-    websites_with_fallback.each do |website|
+  def destroy_from_git
+    websites_for_self.each do |website|
       identifiers(website: website).each do |identifier|
-        Communication::Website::GitFile.sync website, self, identifier
-        dependencies = send "git_dependencies_#{identifier}"
+        Communication::Website::GitFile.sync website, self, identifier, destroy: true
+        dependencies = send "git_destroy_dependencies_#{identifier}"
         dependencies.each do |object|
-          Communication::Website::GitFile.sync website, object, identifier
+          Communication::Website::GitFile.sync website, object, identifier, destroy: true
         end
       end
       website.git_repository.sync!
     end
   end
-  handle_asynchronously :sync_with_git
 
   protected
 
-  def websites_with_fallback
+  def websites_for_self
     if is_a? Communication::Website
       [self]
     elsif respond_to?(:websites)

app/models/concerns/with_media.rb

@@ -13,6 +13,13 @@ module WithMedia
     blobs_with_ids [best_featured_image]
   end
 
+  # Can be overwrite to get featured_image from associated objects (ex: parents)
+  def best_featured_image(fallback: true)
+    featured_image
+  end
+
+  protected
+
   def rich_text_reflection_names
     @rich_text_reflection_names ||= _reflections.select { |name, reflection| reflection.class_name == "ActionText::RichText" }.keys
   end
@@ -23,13 +30,6 @@ module WithMedia
     }.flatten
   end
 
-  # Can be overwrite to get featured_image from associated objects (ex: parents)
-  def best_featured_image(fallback: true)
-    featured_image
-  end
-
-  protected
-
   def blobs_with_ids(ids)
     university.active_storage_blobs.where(id: ids.flatten.compact)
   end

app/models/education/program.rb

@@ -111,6 +111,10 @@ class Education::Program < ApplicationRecord
     active_storage_blobs
   end
 
+  def git_destroy_dependencies_static
+    explicit_active_storage_blobs
+  end
+
   def update_children_paths
     children.each do |child|
       child.update_column :path, child.generated_path

app/services/git/repository.rb

@@ -25,7 +25,7 @@ class Git::Repository
 
   protected
 
-  # TODO add gitlab
+  # Enhancement add gitlab
   def provider
     @provider ||= Git::Providers::Github.new(website&.access_token, website&.repository)
   end
@@ -48,7 +48,7 @@ class Git::Repository
 
   def mark_as_synced
     git_files.each do |git_file|
-      git_file.update_columns previous_path: sanitized_path(git_file.path), previous_sha: git_file.sha
+      git_file.update previous_path: sanitized_path(git_file.path), previous_sha: git_file.sha
     end
   end
 

app/views/admin/communication/websites/static.html.erb

 title: >
   <%= @website.to_s %>
-<% if @website.about_school? %>
-school:
-  name: >
-    <%= @website.about.to_s %>
-  address: >
-    <%= @website.about.address %>
-  zipcode: >
-    <%= @website.about.zipcode %>
-  city: >
-    <%= @website.about.city %>
-  country: >
-    <%= @website.about.country %>
-  phone: >
-    <%= @website.about.phone %>
-<% end %>

docs/websites/export.md

@@ -44,7 +44,7 @@ Lors de l'enregistrement d'un objet, il faut, pour chaque website :
             - push
             - mettre à jour les previous_path et les SHA des git_files
 
-Ce flux cause un problème majeur : tout ce qui est analysé disparaît en asynchrone
+Ce flux cause un problème majeur : tout ce qui est analysé disparaît en asynchrone.
 
 ### Version 2
 
@@ -76,8 +76,6 @@ def reorder
   pages.first.sync_with_git
 end
 ```
-TODO vérifier que tous les cas de déplacement sont correctement gérés.
-TODO gérer la suppression correctement.
 
 ## Code
 
@@ -89,17 +87,19 @@ Le website a un trait WithRepository qui gère son rapport avec le repository Gi
 
 Tous les objets qui doivent être exportés vers Git :
 - doivent utiliser le concern `WithGit`, qui gère l'export vers les repositories des objets et de leurs dépendances
-- doivent présenter une méthode `websites`, éventuellement avec un seul website dans un tableau
 - peuvent intégrer le concern `WithMedia` s'il utilise des médias (`featured_image` et/ou images dans des rich texts)
-- peuvent présenter une méthode `static_files` qui liste les identifiants des git_files à générer, pour les objets qui créent plusieurs fichiers
+- peuvent présenter une méthode `identifiers` qui liste les identifiants des git_files à générer, pour les objets qui créent plusieurs fichiers (le fichier par défaut s'appelle `static`)
+- peuvent présenter une méthode `git_dependencies_static` qui liste les dépendances de l'identifiant par défaut `static`
+- peuvent présenter une méthode `git_destroy_dependencies_static` qui liste les dépendances à supprimer en cascade de l'identifiant par défaut `static`
+- peuvent présenter des méthodes `git_dependencies_author` et/ou `git_destroy_dependencies_author` qui liste les dépendances de l'identifiant `author`
 
-### GitFile
+### Modèle Communication::Website::GitFile
 
 La responsabilité de la synchronisation repose sur Communication::Website::GitFile, notamment :
 - l'information est-elle intègre, synchronisée avec le repo ? (previous_sha et previous_path cohérents avec le repo git)
-- le fichier doit-il être créé ? (non intègre, ou pas de previous_sha/previous_path)
-- le fichier doit-il être mis à jour ? (non intègre, ou previous_sha/previous_path différent du sha/path)
-- le fichier doit-il être supprimé ? (TODO)
+- le fichier doit-il être créé ? (pas à supprimer et (non intègre, ou pas de previous_sha/previous_path))
+- le fichier doit-il être mis à jour ? (pas à supprimer et (non intègre, ou previous_sha/previous_path différent du sha/path))
+- le fichier doit-il être supprimé ? (path nil ou marquage à détruire)
 
 
 Pour cela, le git_file dispose des propriétés suivantes :