Remove “avec_lien” helpers now that Procedure.path cannot be nil

This commit is contained in:
Nicolas Bouilleaud 2019-09-16 16:59:10 +02:00
parent 8fa630d2bb
commit b9968b76b0
7 changed files with 13 additions and 22 deletions

View file

@ -25,7 +25,7 @@ class AdminController < ApplicationController
end end
def reset_procedure def reset_procedure
if @procedure.brouillon_avec_lien? if @procedure.brouillon?
@procedure.reset! @procedure.reset!
end end
end end

View file

@ -20,7 +20,7 @@ module NewAdministrateur
end end
def reset_procedure def reset_procedure
if @procedure.brouillon_avec_lien? if @procedure.brouillon?
@procedure.reset! @procedure.reset!
end end
end end

View file

@ -1,11 +1,9 @@
module ProcedureHelper module ProcedureHelper
def procedure_lien(procedure) def procedure_lien(procedure)
if procedure.path.present? if procedure.brouillon?
if procedure.brouillon_avec_lien? commencer_test_url(path: procedure.path)
commencer_test_url(path: procedure.path) else
else commencer_url(path: procedure.path)
commencer_url(path: procedure.path)
end
end end
end end

View file

@ -133,11 +133,6 @@ class Procedure < ApplicationRecord
!archivee? !archivee?
end end
# This method is needed for transition. Eventually this will be the same as brouillon?.
def brouillon_avec_lien?
brouillon? && path.present?
end
def publiee_ou_archivee? def publiee_ou_archivee?
publiee? || archivee? publiee? || archivee?
end end

View file

@ -24,12 +24,10 @@ class ProcedureSerializer < ActiveModel::Serializer
end end
def link def link
if object.path.present? if object.brouillon?
if object.brouillon_avec_lien? commencer_test_url(path: object.path)
commencer_test_url(path: object.path) else
else commencer_url(path: object.path)
commencer_url(path: object.path)
end
end end
end end

View file

@ -44,7 +44,7 @@
.lien-demarche .lien-demarche
%h3 %h3
- if @procedure.brouillon_avec_lien? - if @procedure.brouillon?
Test et publication Test et publication
- else - else
Publication Publication
@ -58,7 +58,7 @@
%br %br
%br %br
Attention, diffusez toujours le <strong>lien complet</strong> affiché ci-dessus, et non pas un lien générique vers demarches-simplifiees.fr. Ne dites pas non plus aux usagers de se rendre sur le site générique demarches-simplifiees.fr, donnez-leur toujours le lien complet. Attention, diffusez toujours le <strong>lien complet</strong> affiché ci-dessus, et non pas un lien générique vers demarches-simplifiees.fr. Ne dites pas non plus aux usagers de se rendre sur le site générique demarches-simplifiees.fr, donnez-leur toujours le lien complet.
- elsif @procedure.brouillon_avec_lien? - elsif @procedure.brouillon?
- if @procedure.missing_steps.empty? - if @procedure.missing_steps.empty?
%p %p
Cette démarche est actuellement <strong>en test</strong>, Cette démarche est actuellement <strong>en test</strong>,

View file

@ -11,7 +11,7 @@
#procedure-list #procedure-list
%a#onglet-infos{ href: url_for(admin_procedure_path(@procedure)) } %a#onglet-infos{ href: url_for(admin_procedure_path(@procedure)) }
.procedure-list-element{ class: ('active' if active == 'Informations') } .procedure-list-element{ class: ('active' if active == 'Informations') }
- if @procedure.brouillon_avec_lien? - if @procedure.brouillon?
Test et publication Test et publication
- else - else
Publication Publication