Merge pull request #2758 from betagouv/fix-2757

[Fix #2757] Do not show the avis form for dossiers that are terminés

app/assets/stylesheets/new_design/dossiers_table.scss

@@ -77,7 +77,3 @@
     padding-right: $default-spacer;
   }
 }
-
-.dossiers-table-empty {
-  text-align: center;
-}

app/assets/stylesheets/new_design/layouts.scss

@@ -45,3 +45,7 @@
   margin: auto;
   max-width: $page-width / 2;
 }
+
+.blank-tab {
+  text-align: center;
+}

app/assets/stylesheets/new_design/utils.scss

@@ -44,7 +44,6 @@
   }
 }
 
-
 .highlighted {
   background: $orange-bg;
   color: $black;

app/views/new_gestionnaire/avis/instruction.html.haml

@@ -21,6 +21,8 @@
         .send-wrapper
           = f.submit 'Envoyer votre avis', class: 'button send'
 
-  = render partial: "new_gestionnaire/shared/avis/form", locals: { url: avis_gestionnaire_avis_path(@avis), must_be_confidentiel: @avis.confidentiel?, avis: @new_avis }
+  - if !@dossier.termine?
+    = render partial: "new_gestionnaire/shared/avis/form", locals: { url: avis_gestionnaire_avis_path(@avis), must_be_confidentiel: @avis.confidentiel?, avis: @new_avis }
 
-  = render partial: 'new_gestionnaire/shared/avis/list', locals: { avis: @dossier.avis_for(current_gestionnaire), avis_seen_at: nil }
+  - if @dossier.avis_for(current_gestionnaire).present?
+    = render partial: 'new_gestionnaire/shared/avis/list', locals: { avis: @dossier.avis_for(current_gestionnaire), avis_seen_at: nil }

app/views/new_gestionnaire/dossiers/avis.html.haml

@@ -3,6 +3,13 @@
 = render partial: "header", locals: { dossier: @dossier }
 
 .container
-  = render partial: "new_gestionnaire/shared/avis/form", locals: { url: avis_gestionnaire_dossier_path(@dossier.procedure, @dossier), must_be_confidentiel: false, avis: @avis }
+  - if !@dossier.termine?
+    = render partial: "new_gestionnaire/shared/avis/form", locals: { url: avis_gestionnaire_dossier_path(@dossier.procedure, @dossier), must_be_confidentiel: false, avis: @avis }
 
-  = render partial: 'new_gestionnaire/shared/avis/list', locals: { avis: @dossier.avis, avis_seen_at: @avis_seen_at }
+  - if @dossier.avis.present?
+    = render partial: 'new_gestionnaire/shared/avis/list', locals: { avis: @dossier.avis, avis_seen_at: @avis_seen_at }
+
+  - if @dossier.termine? && !@dossier.avis.present?
+    .blank-tab
+      %h2.empty-text Aucun avis.
+      %p.empty-text-details Aucun avis n'a été demandé sur ce dossier.

app/views/new_gestionnaire/shared/avis/_list.html.haml

@@ -1,32 +1,31 @@
-- if avis.present?
-  %section.list-avis
-    %h1.tab-title
-      Avis des invités
-      %span.count= avis.count
+%section.list-avis
+  %h1.tab-title
+    Avis des invités
+    %span.count= avis.count
 
-    %ul
-      - avis.each do |avis|
-        %li.one-avis.flex.align-start
-          .width-100
-            %h2.claimant
-              Demandeur :
-              %span.email= (avis.claimant.email == current_gestionnaire.email) ? 'Vous' : avis.claimant.email
-              - if avis.confidentiel?
-                %span.confidentiel
-                  confidentiel
-                  %span.icon.lock{ title: "Cet avis n'est pas affiché avec les autres experts consultés" }
-              %span.date{ class: highlight_if_unseen_class(avis_seen_at, avis.created_at) }
-                Demande d'avis envoyée le #{I18n.l(avis.created_at.localtime, format: '%d/%m/%y à %H:%M')}
-            %p= avis.introduction
+  %ul
+    - avis.each do |avis|
+      %li.one-avis.flex.align-start
+        .width-100
+          %h2.claimant
+            Demandeur :
+            %span.email= (avis.claimant.email == current_gestionnaire.email) ? 'Vous' : avis.claimant.email
+            - if avis.confidentiel?
+              %span.confidentiel
+                confidentiel
+                %span.icon.lock{ title: "Cet avis n'est pas affiché avec les autres experts consultés" }
+            %span.date{ class: highlight_if_unseen_class(avis_seen_at, avis.created_at) }
+              Demande d'avis envoyée le #{I18n.l(avis.created_at.localtime, format: '%d/%m/%y à %H:%M')}
+          %p= avis.introduction
 
-            .answer.flex.align-start
-              %span.icon.bubble.avis-icon
-              .width-100
-                %h2.gestionnaire
-                  = (avis.email_to_display == current_gestionnaire.email) ? 'Vous' : avis.email_to_display
-                  - if avis.answer.present?
-                    %span.date{ class: highlight_if_unseen_class(avis_seen_at, avis.updated_at) }
-                      Réponse donnée le #{I18n.l(avis.updated_at.localtime, format: '%d/%m/%y à %H:%M')}
-                  - else
-                    %span.waiting En attente de réponse
-                %p= avis.answer
+          .answer.flex.align-start
+            %span.icon.bubble.avis-icon
+            .width-100
+              %h2.gestionnaire
+                = (avis.email_to_display == current_gestionnaire.email) ? 'Vous' : avis.email_to_display
+                - if avis.answer.present?
+                  %span.date{ class: highlight_if_unseen_class(avis_seen_at, avis.updated_at) }
+                    Réponse donnée le #{I18n.l(avis.updated_at.localtime, format: '%d/%m/%y à %H:%M')}
+                - else
+                  %span.waiting En attente de réponse
+              %p= avis.answer

app/views/new_user/dossiers/index.html.haml

@@ -63,7 +63,7 @@
             %span.icon.smile
 
   - else
-    .dossiers-table-empty
+    .blank-tab
       %h2.empty-text Aucun dossier.
       %p.empty-text-details Vous n’avez pas encore commencé de démarche.
       = link_to "Commencer une nouvelle démarche", demarches_url, class: "button primary"