Merge pull request #2706 from betagouv/fix-links

[Dossier] Correction des anciens liens restants

app/controllers/support_controller.rb

@@ -9,7 +9,7 @@ class SupportController < ApplicationController
     if direct_message? && create_commentaire
       flash.notice = "Votre message a été envoyé sur la messagerie de votre dossier."
 
-      redirect_to helpers.url_for_dossier(dossier)
+      redirect_to messagerie_dossier_path(dossier)
     elsif create_conversation
       flash.notice = "Votre message a été envoyé."
 

app/controllers/users/dossiers/invites_controller.rb

@@ -12,12 +12,10 @@ class Users::Dossiers::InvitesController < UsersController
   def show
     @facade = InviteDossierFacades.new params[:id].to_i, current_user.email
 
-    if Flipflop.new_dossier_details?
-      return redirect_to dossier_path(@facade.dossier)
-    end
-
     if @facade.dossier.brouillon?
       redirect_to brouillon_dossier_path(@facade.dossier)
+    elsif Flipflop.new_dossier_details?
+      return redirect_to dossier_path(@facade.dossier)
     else
       render 'users/recapitulatif/show'
     end

app/models/concerns/tags_substitution_concern.rb

@@ -47,7 +47,7 @@ module TagsSubstitutionConcern
     {
       libelle: 'lien dossier',
       description: '',
-      lambda: -> (d) { external_link(users_dossier_recapitulatif_url(d)) },
+      lambda: -> (d) { external_link(dossier_url(d)) },
       available_for_states: Dossier::SOUMIS
     },
     {

app/views/layouts/_modifications_terminees.html.haml

@@ -1,3 +1,3 @@
 %div{ style: 'float: right;' }
-  %a.btn{ href: "/users/dossiers/#{@dossier.id}/recapitulatif" } Retour
+  %a.btn{ href: dossier_path(@dossier) } Retour
   = submit_tag 'Modification terminée', class: %w(btn btn btn-info), id: 'modification_terminee', data: { disable: true, submit: true }

app/views/layouts/mailers/notification.html.haml

@@ -6,4 +6,4 @@
   %strong
     Merci de ne pas répondre à cet email. Pour vous adresser à votre administration, passez directement par votre
     = succeed '.' do
-      = link_to 'messagerie', users_dossier_recapitulatif_url(@dossier), target: '_blank'
+      = link_to 'messagerie', messagerie_dossier_url(@dossier), target: '_blank'

app/views/notification_mailer/new_answer.html.haml

@@ -6,7 +6,7 @@
 
 %p
   Pour le consulter, merci de vous rendre sur
-  = users_dossier_recapitulatif_url(dossier_id: @dossier.id)
+  = messagerie_dossier_url(@dossier)
 
 %p
   Bonne journée,

spec/controllers/support_controller_spec.rb

@@ -98,7 +98,7 @@ describe SupportController, type: :controller do
             }
 
             expect(flash[:notice]).to match('Votre message a été envoyé sur la messagerie de votre dossier.')
-            expect(response).to redirect_to users_dossier_recapitulatif_path(dossier)
+            expect(response).to redirect_to messagerie_dossier_path(dossier)
           end
         end
       end

spec/mailers/notification_mailer_spec.rb

@@ -60,7 +60,7 @@ RSpec.describe NotificationMailer, type: :mailer do
     subject(:subject) { described_class.new_answer(dossier) }
 
     it { expect(subject.body).to match('Un nouveau message est disponible dans votre espace demarches-simplifiees.fr.') }
-    it { expect(subject.body).to include(users_dossier_recapitulatif_url(dossier_id: dossier.id)) }
+    it { expect(subject.body).to include(messagerie_dossier_url(dossier)) }
     it { expect(subject.subject).to eq("Nouveau message pour votre dossier nº #{dossier.id}") }
   end
 end

spec/models/concern/mail_template_concern_spec.rb

@@ -26,7 +26,7 @@ describe MailTemplateConcern do
       it do
         expected =
           "[demarches-simplifiees.fr] #{dossier.id} #{dossier.procedure.libelle} " +
-          "<a target=\"_blank\" href=\"http://localhost:3000/users/dossiers/#{dossier.id}/recapitulatif\">http://localhost:3000/users/dossiers/#{dossier.id}/recapitulatif</a>"
+          "<a target=\"_blank\" href=\"http://localhost:3000/dossiers/#{dossier.id}\">http://localhost:3000/dossiers/#{dossier.id}</a>"
 
         is_expected.to eq(expected)
       end

spec/views/users/carte/show.html.haml_spec.rb

@@ -27,7 +27,7 @@ describe 'users/carte/show.html.haml', type: :view do
       end
     end
 
-    context 'si la page précédente n\'est pas recapitulatif' do
+    context 'si la page précédente n\'est pas la page du dossier' do
       it 'le bouton "Etape suivante" est présent' do
         expect(rendered).to have_selector('#etape_suivante')
       end
@@ -37,7 +37,7 @@ describe 'users/carte/show.html.haml', type: :view do
       # end
     end
 
-    context 'si la page précédente est recapitularif' do
+    context 'si la page précédente est la page du dossier' do
       let(:state) { Dossier.states.fetch(:en_construction) }
 
       it 'le bouton "Etape suivante" n\'est pas présent' do
@@ -52,8 +52,8 @@ describe 'users/carte/show.html.haml', type: :view do
       #   expect(rendered).to have_selector('input[type=submit][id=modification_terminee][onclick=\'submit_check_draw(event)\']')
       # end
 
-      it 'le lien de retour au récapitulatif est présent' do
+      it 'le lien de retour à la page du dossier est présent' do
-        expect(rendered).to have_selector("a[href='/users/dossiers/#{dossier_id}/recapitulatif']")
+        expect(rendered).to have_selector("a[href='/dossiers/#{dossier_id}']")
       end
     end
   end