Merge pull request #982 from sgmap/extended_motivation

[fix #873] add motivation in case of refused or without continuation …

app/assets/javascripts/new_design/state_button.js

@@ -1,9 +1,9 @@
-TPS.acceptDossier = function () {
+TPS.showMotivation = function (state) {
-  $(".motivation").show();
+  $(".motivation." + state).show();
   $(".dropdown-items").hide();
-}
+};
 
 TPS.motivationCancel = function () {
   $(".motivation").hide();
   $(".dropdown-items").show();
-}
+};

app/assets/stylesheets/new_design/motivation.scss

@@ -10,6 +10,7 @@
   h3 {
     font-size: 22px;
     margin-bottom: $default-spacer * 2;
+    line-height: 1.5;
 
     .icon {
       vertical-align: sub;

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

@@ -28,36 +28,24 @@
             .description
               %h4 En instruction
               L'usager ne peut modifier son dossier pendant l'instruction
-          %li{ onclick: "TPS.acceptDossier();" }
+          %li{ onclick: "TPS.showMotivation('accept');" }
             .icon.accept
             .description
               %h4 Accepter
               L'usager sera notifié que son dossier a été accepté
-          %li
+          %li{ onclick: "TPS.showMotivation('without-continuation');" }
-            = link_to backoffice_dossier_process_dossier_path(dossier, process_action: "without_continuation", new_ui: true), method: :post, data: { confirm: "Confirmer vous le classement sans suite de ce dossier ?" } do
             .icon.without-continuation
             .description
               %h4 Classer sans suite
               L'usager ne recevra aucune notification
-          %li
+          %li{ onclick: "TPS.showMotivation('close');" }
-            = link_to backoffice_dossier_process_dossier_path(dossier, process_action: "refuse", new_ui: true), method: :post, data: { confirm: "Confirmer vous le refus de ce dossier ?" } do
             .icon.close
             .description
               %h4 Refuser
               L'usager sera notifié que son dossier a été refusé
-      .motivation
+      = render partial: 'new_gestionnaire/dossiers/state_button_motivation', locals: { dossier: dossier, popup_title: 'Accepter le dossier',  popup_class: 'accept', process_action: 'close', title: 'Accepter', confirm: 'Accepter ce dossier ?' }
-        %h3
+      = render partial: 'new_gestionnaire/dossiers/state_button_motivation', locals: { dossier: dossier, popup_title: 'Classer le dossier sans suite', popup_class: 'without-continuation', process_action: 'without_continuation', title: 'Classer sans suite', confirm: 'Confirmer vous le classement sans suite de ce dossier ?' }
-          .icon.accept
+      = render partial: 'new_gestionnaire/dossiers/state_button_motivation', locals: { dossier: dossier, popup_title: 'Refuser le dossier', popup_class: 'close', process_action: 'refuse', title: 'Refuser', confirm: 'Confirmer vous le refus de ce dossier ?' }
-          Accepter le dossier
-          = "nº #{dossier.id}"
-
-        = form_tag(backoffice_dossier_process_dossier_url(dossier.id, new_ui: true), method: :post, class: "form") do
-          = text_area :dossier, :motivation, class: "motivation-text-area", placeholder: "Rédigez votre motivation ici (facultative)"
-          %p.help
-            L'acceptation du dossier envoie automatiquement une attestation à l'usager.
-          .text-right
-            %span.button{ onclick: "TPS.motivationCancel();" } Annuler
-            = button_tag "Valider la décision", name: :process_action, value: "close", class: 'button primary', title: 'Accepter', data: { confirm: "Accepter ce dossier ?" }
 
 - else
   - if dossier.motivation.present? || dossier.attestation.present?

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

@@ -0,0 +1,13 @@
+.motivation{ class: popup_class }
+  %h3
+    .icon{ class: popup_class }
+    #{popup_title} nº #{dossier.id}
+
+  = form_tag(backoffice_dossier_process_dossier_url(dossier.id, new_ui: true), method: :post, class: 'form') do
+    = text_area :dossier, :motivation, class: 'motivation-text-area', placeholder: 'Rédigez votre motivation ici (facultative)'
+    - if title == 'Accepter'
+      %p.help
+        L'acceptation du dossier envoie automatiquement une attestation à l'usager.
+    .text-right
+      %span.button{ onclick: 'TPS.motivationCancel();' } Annuler
+      = button_tag 'Valider la décision', name: :process_action, value: process_action, class: 'button primary', title: title, data: { confirm: confirm }

spec/features/new_gestionnaire/gestionnaire_spec.rb

@@ -23,8 +23,10 @@ feature 'The gestionnaire part' do
     dossier.reload
     expect(dossier.state).to eq('received')
 
+    within('.accept.motivation') do
       fill_in('dossier_motivation', with: 'a good reason')
       click_on 'Valider la décision'
+    end
 
     dossier.reload
     expect(dossier.state).to eq('closed')