From fffbbb5d73a0e727eff3503553209ed2bba0490e Mon Sep 17 00:00:00 2001
From: gregoirenovel <log@kinko.fr>
Date: Thu, 4 Oct 2018 15:50:21 +0200
Subject: [PATCH] Extract the conditional in a partial to its caller

---
 .../avis/instruction.html.haml                |  3 +-
 .../new_gestionnaire/dossiers/avis.html.haml  |  3 +-
 .../shared/avis/_list.html.haml               | 59 +++++++++----------
 3 files changed, 33 insertions(+), 32 deletions(-)

diff --git a/app/views/new_gestionnaire/avis/instruction.html.haml b/app/views/new_gestionnaire/avis/instruction.html.haml
index 1ff164457..93a161137 100644
--- a/app/views/new_gestionnaire/avis/instruction.html.haml
+++ b/app/views/new_gestionnaire/avis/instruction.html.haml
@@ -23,4 +23,5 @@
 
   = 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 }
diff --git a/app/views/new_gestionnaire/dossiers/avis.html.haml b/app/views/new_gestionnaire/dossiers/avis.html.haml
index 67881abbb..012cdcc7c 100644
--- a/app/views/new_gestionnaire/dossiers/avis.html.haml
+++ b/app/views/new_gestionnaire/dossiers/avis.html.haml
@@ -5,4 +5,5 @@
 .container
   = 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 }
diff --git a/app/views/new_gestionnaire/shared/avis/_list.html.haml b/app/views/new_gestionnaire/shared/avis/_list.html.haml
index 03966b8e6..fb34568fb 100644
--- a/app/views/new_gestionnaire/shared/avis/_list.html.haml
+++ b/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