Merge tag '2023-06-06-01' into feature/import-2023-06-06

# Conflicts: # app/views/administrateurs/procedures/_informations.html.haml # db/schema.rb
govpf · Aug 25, 2023 · 8243788 · 8243788
2 parents ff34231 + c85f70c
commit 8243788
Show file tree

Hide file tree

Showing 35 changed files with 1,450 additions and 255 deletions.
diff --git a/Gemfile b/Gemfile
@@ -80,7 +80,6 @@ gem 'rails-i18n' # Locales par défaut
 gem 'rake-progressbar', require: false
 gem 'redcarpet'
 gem 'rexml' # add missing gem due to ruby3 (https://github.com/Shopify/bootsnap/issues/325)
-gem 'rgeo-geojson'
 gem 'rqrcode'
 gem 'saml_idp'
 gem 'sanitize-url'

diff --git a/Gemfile.lock b/Gemfile.lock
@@ -592,9 +592,6 @@ GEM
       mime-types (>= 1.16, < 4.0)
       netrc (~> 0.8)
     rexml (3.2.5)
-    rgeo (2.2.0)
-    rgeo-geojson (2.1.1)
-      rgeo (>= 1.0.0)
     rodf (1.1.1)
       builder (>= 3.0)
       dry-inflector (~> 0.1)
@@ -918,7 +915,6 @@ DEPENDENCIES
   rake-progressbar
   redcarpet
   rexml
-  rgeo-geojson
   rqrcode
   rspec-rails
   rspec_junit_formatter

diff --git a/app/assets/stylesheets/02_utils.scss b/app/assets/stylesheets/02_utils.scss
@@ -90,14 +90,6 @@
   display: none !important;
 }
 
-.visible-on-previous-hover {
-  visibility: hidden;
-}
-
-:hover + .visible-on-previous-hover {
-  visibility: visible;
-}
-
 // sizing
 .width-100 {
   width: 100%;

diff --git a/app/assets/stylesheets/procedure_context.scss b/app/assets/stylesheets/procedure_context.scss
@@ -21,30 +21,6 @@ $procedure-context-breakpoint: $two-columns-breakpoint;
     font-size: 12px;
   }
 
-  .procedure-configuration--auto-archive {
-    cursor: pointer;
-
-    summary {
-      font-size: 20px;
-
-      // Hide the default triangle details marker
-      list-style: none;
-      // (using the non-standard way for Chrome.)
-      &::-webkit-details-marker {
-        display: none;
-      }
-    }
-
-    p {
-      padding-top: 8px;
-      font-size: 16px;
-    }
-  }
-
-  .procedure-auto-archive-title {
-    border-bottom: 1px dotted $blue-france-500;
-  }
-
   .procedure-logos {
     display: flex;
     justify-content: space-around;

diff --git a/app/assets/stylesheets/procedure_form.scss b/app/assets/stylesheets/procedure_form.scss
@@ -95,7 +95,7 @@
 .procedure-form__options-summary {
   cursor: pointer;
 
-  .header-subsection {
+  h3 {
     display: inline-block;
   }
 }
diff --git a/app/components/attachment/show_component/show_component.en.yml b/app/components/attachment/show_component/show_component.en.yml
@@ -1,6 +1,5 @@
 ---
 en:
-  virus_not_analyzed: this file has not been scanned by our antivirus, download it with care
   errors:
     virus_infected: Virus detected, download is blocked.
     corrupted_file: The file is corrupted, the download is blocked.
diff --git a/app/components/attachment/show_component/show_component.fr.yml b/app/components/attachment/show_component/show_component.fr.yml
@@ -1,6 +1,5 @@
 ---
 fr:
-  virus_not_analyzed: ce fichier n’a pas été analysé par notre antivirus, téléchargez-le avec précaution
   errors:
     virus_infected: "Virus détecté, le téléchargement est bloqué."
     corrupted_file: "Le fichier est corrompu, le téléchargement est bloqué."
diff --git a/app/components/attachment/show_component/show_component.html.haml b/app/components/attachment/show_component/show_component.html.haml
@@ -1,9 +1,6 @@
 %div{ id: dom_id(attachment, :show), class: class_names("attachment-error": error?, "fr-mb-2w": !should_display_link?) }
   - if should_display_link?
-    = render Dsfr::DownloadComponent.new(attachment: attachment, new_tab: new_tab) do |c|
-      - if !attachment.virus_scanner.started?
-        - c.right do
-          = "(#{t(".virus_not_analyzed")})"
+    = render Dsfr::DownloadComponent.new(attachment: attachment, virus_not_analyzed: !attachment.virus_scanner.started?, new_tab: new_tab)
 
   - else
     .attachment-filename.fr-mb-1w.fr-mr-1w= attachment.filename.to_s

diff --git a/app/components/dsfr/download_component.rb b/app/components/dsfr/download_component.rb
@@ -1,15 +1,17 @@
 class Dsfr::DownloadComponent < ApplicationComponent
-  renders_one :right
-
   attr_reader :attachment
   attr_reader :html_class
   attr_reader :name
+  attr_reader :ephemeral_link
+  attr_reader :virus_not_analyzed
   attr_reader :new_tab
 
-  def initialize(attachment:, name: nil, url: nil, new_tab: false)
+  def initialize(attachment:, name: nil, url: nil, ephemeral_link: false, virus_not_analyzed: false, new_tab: false)
     @attachment = attachment
     @name = name || attachment.filename.to_s
     @url = url
+    @ephemeral_link = ephemeral_link
+    @virus_not_analyzed = virus_not_analyzed
     @new_tab = new_tab
   end
 

diff --git a/app/components/dsfr/download_component/download_component.en.yml b/app/components/dsfr/download_component/download_component.en.yml
@@ -1,3 +1,5 @@
 ---
 en:
   title: "Download file %{filename}"
+  ephemeral_link: This link is ephemeral and should not be shared.
+  virus_not_analyzed: this file has not been scanned by our antivirus, download it with care
diff --git a/app/components/dsfr/download_component/download_component.fr.yml b/app/components/dsfr/download_component/download_component.fr.yml
@@ -1,3 +1,5 @@
 ---
 fr:
   title: "Télécharger le fichier %{filename}"
+  ephemeral_link: Ce lien est éphémère et ne devrait pas être partagé.
+  virus_not_analyzed: ce fichier n’a pas été analysé par notre antivirus, téléchargez-le avec précaution
diff --git a/app/components/dsfr/download_component/download_component.html.haml b/app/components/dsfr/download_component/download_component.html.haml
@@ -4,5 +4,11 @@
       = name
       %span.fr-download__detail
         = helpers.download_details(attachment)
+    - if ephemeral_link
+      %span.fr-ml-2w.fr-text--xs.fr-text-mention--grey
+        %span.fr-icon-questionnaire-line{ "aria-hidden": "true" }
+        = t('.ephemeral_link')
 
-    = right
+    - if virus_not_analyzed
+      .fr-text--xs.fr-text-mention--grey
+        = t('.virus_not_analyzed')
diff --git a/app/components/editable_champ/editable_champ_component.rb b/app/components/editable_champ/editable_champ_component.rb
@@ -30,7 +30,7 @@ def html_options
   end
 
   def stimulus_controller
-    if !@champ.block? && @champ.fillable?
+    if autosave_enabled?
       # This is an editable champ. Lets find what controllers it might need.
       controllers = ['autosave']
 
@@ -45,4 +45,8 @@ def data_dependent_conditions
       {}
     end
   end
+
+  def autosave_enabled?
+    !@champ.carte? && !@champ.block? && @champ.fillable?
+  end
 end
diff --git a/app/components/procedure/revision_changes_component/revision_changes_component.fr.yml b/app/components/procedure/revision_changes_component/revision_changes_component.fr.yml
@@ -33,10 +33,10 @@ fr:
     enable_drop_down_other: Le champ « %{label} » comporte maintenant un choix « Autre ».
     disable_drop_down_other: Le champ « %{label} » ne comporte plus de choix « Autre ».
     update_carte_layers: "Les référentiels cartographiques du champ « %{label} » ont été modifiés :"
-    enable_update_collapsible_explanation: Le texte complementaire affichable au clique du champ « %{label} » a été ajouté.
-    disable_update_collapsible_explanation: Le texte complementaire affichable au clique du champ « %{label} » a été supprimée.
-    update_collapsible_explanation_text: Le texte complementaire affichable au clique du champ « %{label} » a été modifié. Le nouveau texte est « %{to} ».
-    remove_collapsible_explanation_text: Le texte complementaire affichable au clique du champ « %{label} » a été supprimée.
+    enable_update_collapsible_explanation: Le texte complémentaire affichable au clic du champ « %{label} » a été ajouté.
+    disable_update_collapsible_explanation: Le texte complémentaire affichable au clic du champ « %{label} » a été supprimé.
+    update_collapsible_explanation_text: Le texte complémentaire affichable au clic du champ « %{label} » a été modifié. Le nouveau texte est « %{to} ».
+    remove_collapsible_explanation_text: Le texte complémentaire affichable au clic du champ « %{label} » a été supprimé.
     add_condition: Une condition a été ajoutée sur le champ « %{label} ». La nouvelle condition est « %{to} ».
     remove_condition: La condition du champ « %{label} » a été supprimée.
     update_condition: La condition du champ « %{label} » a été modifiée. La nouvelle condition est « %{to} ».
@@ -62,10 +62,10 @@ fr:
     update_carte_layers: "Les référentiels cartographiques de l’annotation privée « %{label} » ont été modifiés :"
     enable_drop_down_other: L’annotation privée « %{label} » comporte maintenant un choix « Autre ».
     disable_drop_down_other: L’annotation privée « %{label} » ne comporte plus de choix « Autre ».
-    enable_collapsible_explanation: Le texte complementaire affichable au clique de l’annotation privée « %{label} » a été ajouté.
-    disable_collapsible_explanation: Le texte complementaire affichable au clique de l’annotation privée « %{label} » a été supprimée.
-    update_collapsible_explanation: Le texte complementaire affichable au clique de l’annotation privée « %{label} » a été modifié. Le nouveau texte est « %{to} ».
-    remove_collapsible_explanation: Le texte complementaire affichable au clique de l’annotation privée « %{label} » a été supprimée.
+    enable_collapsible_explanation: Le texte complémentaire affichable au clic de l’annotation privée « %{label} » a été ajouté.
+    disable_collapsible_explanation: Le texte complémentaire affichable au clic de l’annotation privée « %{label} » a été supprimé.
+    update_collapsible_explanation: Le texte complémentaire affichable au clic de l’annotation privée « %{label} » a été modifié. Le nouveau texte est « %{to} ».
+    remove_collapsible_explanation: Le texte complémentaire affichable au clic de l’annotation privée « %{label} » a été supprimé.
     add_condition: Une condition a été ajoutée sur l’annotation privée « %{label} ». La nouvelle condition est « %{to} ».
     remove_condition: La condition de l’annotation privée « %{label} » a été supprimée.
     update_condition: La condition de l’annotation privée « %{label} » a été modifiée. La nouvelle condition est « %{to} ».
diff --git a/app/components/types_de_champ_editor/champ_component/champ_component.html.haml b/app/components/types_de_champ_editor/champ_component/champ_component.html.haml
@@ -138,7 +138,7 @@
         - if type_de_champ.explication?
           .cell.width-66
             = form.label :collapsible_explanation_enabled, for: dom_id(type_de_champ, :collapsible_explanation_enabled) do
-              Afficher un texte complementaire affichable au clique
+              Afficher un texte complementaire affichable au clic
             = form.check_box :collapsible_explanation_enabled, class: "small-margin small", id: dom_id(type_de_champ, :collapsible_explanation_enabled)
             - if form.object.collapsible_explanation_enabled?
               = form.label :collapsible_explanation_text, for: dom_id(type_de_champ, :collapsible_explanation_text) do

diff --git a/app/controllers/champs/carte_controller.rb b/app/controllers/champs/carte_controller.rb
@@ -39,6 +39,7 @@ def update
   def destroy
     champ = policy_scope(Champ).find(params[:champ_id])
     champ.geo_areas.find(params[:id]).destroy!
+    champ.touch
 
     head :no_content
   end
@@ -80,6 +81,9 @@ def save_feature(geo_area, feature)
     if feature[:properties]
       geo_area.properties.merge!(feature[:properties])
     end
-    geo_area.save
+    if geo_area.save
+      geo_area.champ.touch
+      true
+    end
   end
 end
diff --git a/app/javascript/components/MapEditor/hooks.ts b/app/javascript/components/MapEditor/hooks.ts
@@ -29,6 +29,12 @@ export function useFeatureCollection(
   const [featureCollection, setFeatureCollection] = useState(
     initialFeatureCollection
   );
+  const refreshFeatureList = useCallback<() => void>(() => {
+    httpRequest(url)
+      .turbo()
+      .catch(() => null);
+  }, [url]);
+
   const updateFeatureCollection = useCallback<
     (callback: (features: Feature[]) => Feature[]) => void
   >(
@@ -37,11 +43,9 @@ export function useFeatureCollection(
         type: 'FeatureCollection',
         features: callback(features)
       }));
-      httpRequest(url)
-        .turbo()
-        .catch(() => null);
+      refreshFeatureList();
     },
-    [url, setFeatureCollection]
+    [refreshFeatureList, setFeatureCollection]
   );
 
   const addFeatures = useCallback(
@@ -153,13 +157,15 @@ export function useFeatureCollection(
         if (newFeatures.length > 0) {
           addFeatures(newFeatures, external);
           updateFeatureCollection((features) => [...features, ...newFeatures]);
+        } else {
+          refreshFeatureList();
         }
       } catch (error) {
         console.error(error);
         onError('Le polygone dessiné n’est pas valide.');
       }
     },
-    [url, updateFeatureCollection, addFeatures, onError]
+    [url, refreshFeatureList, updateFeatureCollection, addFeatures, onError]
   );
 
   const deleteFeatures = useCallback<DeleteFeatures>(

diff --git a/app/javascript/controllers/geo_area_controller.tsx b/app/javascript/controllers/geo_area_controller.tsx
@@ -19,7 +19,7 @@ export class GeoAreaController extends ApplicationController {
   }
 
   onInput() {
-    this.debounce(this.updateDescription, 200);
+    this.debounce(this.updateDescription, 500);
   }
 
   private updateDescription(): void {

diff --git a/app/jobs/migrations/normalize_geo_area_job.rb b/app/jobs/migrations/normalize_geo_area_job.rb
diff --git a/app/models/geo_area.rb b/app/models/geo_area.rb
@@ -52,7 +52,6 @@ def filename
   scope :cadastres, -> { where(source: sources.fetch(:cadastre)) }
 
   validates :geometry, geo_json: true, allow_nil: false
-  before_validation :normalize_geometry
 
   def to_feature
     {
@@ -230,21 +229,4 @@ def cid
       properties['id']
     end
   end
-
-  private
-
-  def normalize_geometry
-    if geometry.present?
-      normalized_geometry = rgeo_geometry
-      if normalized_geometry.present?
-        self.geometry = RGeo::GeoJSON.encode(normalized_geometry)
-      end
-    end
-  end
-
-  def rgeo_geometry
-    RGeo::GeoJSON.decode(geometry.to_json, geo_factory: RGeo::Geographic.simple_mercator_factory)
-  rescue RGeo::Error::InvalidGeometry
-    nil
-  end
 end