DGNum/demarches-normaliennes
14
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

Changed the routes to pass all new design admin pages under

Browse source 
'new_administrateur' scope, except 'services'.
As a result, the url of these pages will all include '/admin'.
This commit is contained in:
Judith 2020-07-07 12:03:32 +02:00 committed by jpoulvel (Rebase PR Action)
parent 9a89b3d2f0
commit 782706537d
30 changed files with 79 additions and 79 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/controllers/admin/instructeurs_controller.rb
View file

@ -22,7 +22,7 @@ class Admin::InstructeursController < AdminController
end end
if procedure_id.present? if procedure_id.present?
redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) redirect_to admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur)
else else
redirect_to admin_instructeurs_path redirect_to admin_instructeurs_path
end end

14
app/controllers/new_administrateur/groupe_instructeurs_controller.rb
View file

@ -21,7 +21,7 @@ module NewAdministrateur
.new(label: label, instructeurs: [current_administrateur.instructeur]) .new(label: label, instructeurs: [current_administrateur.instructeur])
if @groupe_instructeur.save if @groupe_instructeur.save
redirect_to procedure_groupe_instructeur_path(procedure, @groupe_instructeur), redirect_to admin_procedure_groupe_instructeur_path(procedure, @groupe_instructeur),
notice: "Le groupe d’instructeurs « #{label} » a été créé." notice: "Le groupe d’instructeurs « #{label} » a été créé."
else else
@procedure = procedure @procedure = procedure
@ -36,7 +36,7 @@ module NewAdministrateur
@groupe_instructeur = groupe_instructeur @groupe_instructeur = groupe_instructeur
if @groupe_instructeur.update(label: label) if @groupe_instructeur.update(label: label)
redirect_to procedure_groupe_instructeur_path(procedure, groupe_instructeur), redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur),
notice: "Le nom est à présent « #{label} »." notice: "Le nom est à présent « #{label} »."
else else
@procedure = procedure @procedure = procedure
@ -58,7 +58,7 @@ module NewAdministrateur
groupe_instructeur.destroy! groupe_instructeur.destroy!
flash[:notice] = "le groupe « #{label} » a été supprimé." flash[:notice] = "le groupe « #{label} » a été supprimé."
end end
redirect_to procedure_groupe_instructeurs_path(procedure) redirect_to admin_procedure_groupe_instructeurs_path(procedure)
end end
def reaffecter_dossiers def reaffecter_dossiers
@ -76,7 +76,7 @@ module NewAdministrateur
end end
flash[:notice] = "Les dossiers du groupe « #{groupe_instructeur.label} » ont été réaffectés au groupe « #{target_group.label} »." flash[:notice] = "Les dossiers du groupe « #{groupe_instructeur.label} » ont été réaffectés au groupe « #{target_group.label} »."
redirect_to procedure_groupe_instructeurs_path(procedure) redirect_to admin_procedure_groupe_instructeurs_path(procedure)
end end
def add_instructeur def add_instructeur
@ -123,7 +123,7 @@ module NewAdministrateur
end end
end end
redirect_to procedure_groupe_instructeur_path(procedure, groupe_instructeur) redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur)
end end
def remove_instructeur def remove_instructeur
@ -148,13 +148,13 @@ module NewAdministrateur
end end
end end
end end
redirect_to procedure_groupe_instructeur_path(procedure, groupe_instructeur) redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur)
end end
def update_routing_criteria_name def update_routing_criteria_name
procedure.update!(routing_criteria_name: routing_criteria_name) procedure.update!(routing_criteria_name: routing_criteria_name)
redirect_to procedure_groupe_instructeurs_path(procedure), redirect_to admin_procedure_groupe_instructeurs_path(procedure),
notice: "Le libellé est maintenant « #{procedure.routing_criteria_name} »." notice: "Le libellé est maintenant « #{procedure.routing_criteria_name} »."
end end

2
app/controllers/new_administrateur/procedures_controller.rb
View file

@ -25,7 +25,7 @@ module NewAdministrateur
flash.notice = 'Démarche enregistrée.' flash.notice = 'Démarche enregistrée.'
current_administrateur.instructeur.assign_to_procedure(@procedure) current_administrateur.instructeur.assign_to_procedure(@procedure)
redirect_to champs_procedure_path(@procedure) redirect_to champs_admin_procedure_path(@procedure)
end end
end end

4
app/helpers/procedure_helper.rb
View file

@ -23,7 +23,7 @@ module ProcedureHelper
isAnnotation: false, isAnnotation: false,
typeDeChampsTypes: TypeDeChamp.type_de_champ_types_for(procedure, current_user), typeDeChampsTypes: TypeDeChamp.type_de_champ_types_for(procedure, current_user),
typeDeChamps: procedure.types_de_champ.as_json_for_editor, typeDeChamps: procedure.types_de_champ.as_json_for_editor,
baseUrl: procedure_types_de_champ_path(procedure), baseUrl: admin_procedure_types_de_champ_path(procedure),
directUploadUrl: rails_direct_uploads_url directUploadUrl: rails_direct_uploads_url
} }
end end
@ -33,7 +33,7 @@ module ProcedureHelper
isAnnotation: true, isAnnotation: true,
typeDeChampsTypes: TypeDeChamp.type_de_champ_types_for(procedure, current_user), typeDeChampsTypes: TypeDeChamp.type_de_champ_types_for(procedure, current_user),
typeDeChamps: procedure.types_de_champ_private.as_json_for_editor, typeDeChamps: procedure.types_de_champ_private.as_json_for_editor,
baseUrl: procedure_types_de_champ_path(procedure), baseUrl: admin_procedure_types_de_champ_path(procedure),
directUploadUrl: rails_direct_uploads_url directUploadUrl: rails_direct_uploads_url
} }
end end

2
app/views/admin/mail_templates/edit.html.haml
View file

@ -21,7 +21,7 @@
.text-right .text-right
= link_to "Annuler", admin_procedure_mail_templates_path(@procedure), class: "btn btn-default" = link_to "Annuler", admin_procedure_mail_templates_path(@procedure), class: "btn btn-default"
= f.submit 'Mettre à jour', class: "btn btn-default btn-success" = f.submit 'Mettre à jour', class: "btn btn-default btn-success"
= link_to "Prévisualiser", preview_procedure_mail_template_path(@procedure, @mail_template.class.const_get(:SLUG)), class: "btn btn-primary", target: "_blank" = link_to "Prévisualiser", preview_admin_procedure_mail_template_path(@procedure, @mail_template.class.const_get(:SLUG)), class: "btn btn-primary", target: "_blank"
.row .row
.col-md-12 .col-md-12

4
app/views/admin/procedures/show.html.haml
View file

@ -107,7 +107,7 @@
.alert.alert-info .alert.alert-info
Pour pouvoir tester cette démarche, vous devez d’abord lui affecter Pour pouvoir tester cette démarche, vous devez d’abord lui affecter
- if @procedure.missing_instructeurs? - if @procedure.missing_instructeurs?
= link_to("des instructeurs", procedure_groupe_instructeur_path(@procedure, @procedure.defaut_groupe_instructeur)) = link_to("des instructeurs", admin_procedure_groupe_instructeur_path(@procedure, @procedure.defaut_groupe_instructeur))
- if @procedure.missing_instructeurs? && @procedure.service.nil? - if @procedure.missing_instructeurs? && @procedure.service.nil?
et et
- if @procedure.service.nil? - if @procedure.service.nil?
@ -122,7 +122,7 @@
- if @procedure.missing_steps.include?(:instructeurs) - if @procedure.missing_steps.include?(:instructeurs)
%p.alert.alert-danger %p.alert.alert-danger
Vous devez affecter des instructeurs avant de pouvoir publier votre démarche. Vous devez affecter des instructeurs avant de pouvoir publier votre démarche.
= link_to 'Cliquez ici.', procedure_groupe_instructeur_path(@procedure, @procedure.defaut_groupe_instructeur) = link_to 'Cliquez ici.', admin_procedure_groupe_instructeur_path(@procedure, @procedure.defaut_groupe_instructeur)
%p.alert.alert-info %p.alert.alert-info
Cette démarche n’a pas encore de lien, et n’est pas accessible par le public. Cette démarche n’a pas encore de lien, et n’est pas accessible par le public.

2
app/views/groupe_instructeur_mailer/add_instructeurs.html.haml
View file

@ -6,6 +6,6 @@
%p %p
Cliquez sur le lien ci-dessous pour voir la liste des instructeurs de ce groupe : Cliquez sur le lien ci-dessous pour voir la liste des instructeurs de ce groupe :
= link_to(@group.label, procedure_groupe_instructeur_url(@group.procedure, @group)) = link_to(@group.label, admin_procedure_groupe_instructeur_url(@group.procedure, @group))
= render partial: "layouts/mailers/signature" = render partial: "layouts/mailers/signature"

2
app/views/groupe_instructeur_mailer/remove_instructeur.html.haml
View file

@ -6,6 +6,6 @@
%p %p
Cliquez sur le lien ci-dessous pour voir la liste des instructeurs de ce groupe : Cliquez sur le lien ci-dessous pour voir la liste des instructeurs de ce groupe :
= link_to(@group.label, procedure_groupe_instructeur_url(@group.procedure, @group)) = link_to(@group.label, admin_procedure_groupe_instructeur_url(@group.procedure, @group))
= render partial: "layouts/mailers/signature" = render partial: "layouts/mailers/signature"

2
app/views/instructeur_mailer/send_notifications.html.haml
View file

@ -9,7 +9,7 @@
%ul %ul
- @data.each do |datum| - @data.each do |datum|
%li %li
= link_to(datum[:procedure_libelle], procedure_url(datum[:procedure_id])) = link_to(datum[:procedure_libelle], instructeur_procedure_url(datum[:procedure_id]))
- if datum[:nb_en_construction] > 0 - if datum[:nb_en_construction] > 0
%br %br
#{datum[:nb_en_construction]} #{'dossier'.pluralize(datum[:nb_en_construction])} en construction #{datum[:nb_en_construction]} #{'dossier'.pluralize(datum[:nb_en_construction])} en construction

2
app/views/instructeurs/groupe_instructeurs/index.html.haml
View file

@ -1,7 +1,7 @@
- content_for(:title, "Notifications pour #{@procedure.libelle}") - content_for(:title, "Notifications pour #{@procedure.libelle}")
= render partial: 'new_administrateur/breadcrumbs', = render partial: 'new_administrateur/breadcrumbs',
locals: { steps: [link_to(@procedure.libelle, procedure_path(@procedure)), locals: { steps: [link_to(@procedure.libelle, instructeur_procedure_path(@procedure)),
'Groupes d’instructeurs'] } 'Groupes d’instructeurs'] }
.container.groupe-instructeur .container.groupe-instructeur

2
app/views/instructeurs/groupe_instructeurs/show.html.haml
View file

@ -1,7 +1,7 @@
- content_for(:title, "Instructeurs du groupe #{@groupe_instructeur.label}") - content_for(:title, "Instructeurs du groupe #{@groupe_instructeur.label}")
= render partial: 'new_administrateur/breadcrumbs', = render partial: 'new_administrateur/breadcrumbs',
locals: { steps: [link_to(@procedure.libelle, procedure_path(@procedure)), locals: { steps: [link_to(@procedure.libelle, instructeur_procedure_path(@procedure)),
link_to('Groupes d’instructeurs', instructeur_groupes_path(@procedure)), link_to('Groupes d’instructeurs', instructeur_groupes_path(@procedure)),
@groupe_instructeur.label] } @groupe_instructeur.label] }

2
app/views/instructeurs/procedures/deleted_dossiers.html.haml
View file

@ -16,7 +16,7 @@
- if @procedure.routee? - if @procedure.routee?
| |
- if current_administrateur.present? && current_administrateur.owns?(@procedure) - if current_administrateur.present? && current_administrateur.owns?(@procedure)
= link_to 'instructeurs', procedure_groupe_instructeurs_path(@procedure), class: 'header-link' = link_to 'instructeurs', admin_procedure_groupe_instructeurs_path(@procedure), class: 'header-link'
- else - else
= link_to 'instructeurs', instructeur_groupes_path(@procedure), class: 'header-link' = link_to 'instructeurs', instructeur_groupes_path(@procedure), class: 'header-link'

2
app/views/instructeurs/procedures/show.html.haml
View file

@ -16,7 +16,7 @@
- if @procedure.routee? - if @procedure.routee?
| |
- if current_administrateur.present? && current_administrateur.owns?(@procedure) - if current_administrateur.present? && current_administrateur.owns?(@procedure)
= link_to 'instructeurs', procedure_groupe_instructeurs_path(@procedure), class: 'header-link' = link_to 'instructeurs', admin_procedure_groupe_instructeurs_path(@procedure), class: 'header-link'
- else - else
= link_to 'instructeurs', instructeur_groupes_path(@procedure), class: 'header-link' = link_to 'instructeurs', instructeur_groupes_path(@procedure), class: 'header-link'

16
app/views/layouts/left_panels/_left_panel_admin_procedurescontroller_navbar.html.haml
View file

@ -26,33 +26,33 @@
- if @procedure.missing_steps.include?(:service) - if @procedure.missing_steps.include?(:service)
%p.missing-steps (à compléter) %p.missing-steps (à compléter)
%a#onglet-administrateurs{ href: url_for(procedure_administrateurs_path(@procedure)) } %a#onglet-administrateurs{ href: url_for(admin_procedure_administrateurs_path(@procedure)) }
.procedure-list-element .procedure-list-element
Administrateurs Administrateurs
- if !feature_enabled?(:administrateur_routage) - if !feature_enabled?(:administrateur_routage)
%a#onglet-instructeurs{ href: url_for(procedure_groupe_instructeur_path(@procedure, @procedure.defaut_groupe_instructeur)) } %a#onglet-instructeurs{ href: url_for(admin_procedure_groupe_instructeur_path(@procedure, @procedure.defaut_groupe_instructeur)) }
.procedure-list-element{ class: ('active' if active == 'Instructeurs') } .procedure-list-element{ class: ('active' if active == 'Instructeurs') }
Instructeurs Instructeurs
- if @procedure.missing_steps.include?(:instructeurs) - if @procedure.missing_steps.include?(:instructeurs)
%p.missing-steps (à compléter) %p.missing-steps (à compléter)
- if feature_enabled?(:administrateur_routage) - if feature_enabled?(:administrateur_routage)
%a#onglet-instructeurs{ href: url_for(procedure_groupe_instructeurs_path(@procedure)) } %a#onglet-instructeurs{ href: url_for(admin_procedure_groupe_instructeurs_path(@procedure)) }
.procedure-list-element .procedure-list-element
Groupe d'instructeurs Groupe d'instructeurs
- if !@procedure.locked? - if !@procedure.locked?
%a#onglet-champs{ href: champs_procedure_path(@procedure) } %a#onglet-champs{ href: champs_admin_procedure_path(@procedure) }
.procedure-list-element{ class: ('active' if active == 'Champs') } .procedure-list-element{ class: ('active' if active == 'Champs') }
Champs Champs
- if !@procedure.locked? - if !@procedure.locked?
%a#onglet-private-champs{ href: annotations_procedure_path(@procedure) } %a#onglet-private-champs{ href: annotations_admin_procedure_path(@procedure) }
.procedure-list-element{ class: ('active' if active == 'Annotations privées') } .procedure-list-element{ class: ('active' if active == 'Annotations privées') }
Annotations privées Annotations privées
%a#onglet-preview{ href: url_for(apercu_procedure_path(@procedure)), target: "_blank", rel: "noopener" } %a#onglet-preview{ href: url_for(apercu_admin_procedure_path(@procedure)), target: "_blank", rel: "noopener" }
.procedure-list-element{ class: ('active' if active == 'Prévisualisation') } .procedure-list-element{ class: ('active' if active == 'Prévisualisation') }
Prévisualisation Prévisualisation
@ -64,11 +64,11 @@
.procedure-list-element{ class: ('active' if active == 'Attestation') } .procedure-list-element{ class: ('active' if active == 'Attestation') }
Attestation Attestation
%a#onglet-description{ href: url_for(admin_procedure_monavis_path(@procedure)) } %a#onglet-description{ href: url_for(monavis_admin_procedure_path(@procedure)) }
.procedure-list-element{ class: ('active' if active == 'MonAvis') } .procedure-list-element{ class: ('active' if active == 'MonAvis') }
MonAvis MonAvis
%a#onglet-description{ href: url_for(admin_procedure_jeton_path(@procedure)) } %a#onglet-description{ href: url_for(jeton_admin_procedure_path(@procedure)) }
.procedure-list-element .procedure-list-element
Jeton Jeton

2
app/views/manager/procedures/show.html.erb
View file

@ -36,7 +36,7 @@ as well as a link to its edit page.
</div> </div>
<div> <div>
<%= link_to 'aperçu', apercu_procedure_path(procedure), class: 'button' %> <%= link_to 'aperçu', apercu_admin_procedure_path(procedure), class: 'button' %>
<% if !procedure.whitelisted? %> <% if !procedure.whitelisted? %>
<%= link_to 'whitelister', whitelist_manager_procedure_path(procedure), method: :post, class: 'button' %> <%= link_to 'whitelister', whitelist_manager_procedure_path(procedure), method: :post, class: 'button' %>

2
app/views/new_administrateur/_breadcrumbs.html.haml
View file

@ -4,5 +4,5 @@
- steps.each do |step| - steps.each do |step|
%li= step %li= step
- if defined?(preview) && preview - if defined?(preview) && preview
= link_to "Prévisualiser le formulaire", apercu_procedure_path(@procedure), target: "_blank", rel: "noopener", class: 'button' = link_to "Prévisualiser le formulaire", apercu_admin_procedure_path(@procedure), target: "_blank", rel: "noopener", class: 'button'
= link_to "Continuer >", admin_procedure_path(@procedure), title: 'Vous pourrez revenir ici par la suite', class: 'button accepted' = link_to "Continuer >", admin_procedure_path(@procedure), title: 'Vous pourrez revenir ici par la suite', class: 'button accepted'

2
app/views/new_administrateur/_groups_header.haml
View file

@ -1,7 +1,7 @@
%h1 Groupe « #{@groupe_instructeur.label} » %h1 Groupe « #{@groupe_instructeur.label} »
.card.mt-2 .card.mt-2
= form_for @groupe_instructeur, = form_for @groupe_instructeur,
url: procedure_groupe_instructeur_path(@procedure, @groupe_instructeur), url: admin_procedure_groupe_instructeur_path(@procedure, @groupe_instructeur),
html: { class: 'form' } do |f| html: { class: 'form' } do |f|
= f.label :label, 'Nom du groupe' = f.label :label, 'Nom du groupe'

6
app/views/new_administrateur/groupe_instructeurs/index.html.haml
View file

@ -33,16 +33,16 @@
- @groupes_instructeurs.each do |group| - @groupes_instructeurs.each do |group|
%tr %tr
%td= group.label %td= group.label
%td.actions= link_to "voir", procedure_groupe_instructeur_path(@procedure, group) %td.actions= link_to "voir", admin_procedure_groupe_instructeur_path(@procedure, group)
- if @groupes_instructeurs.many? - if @groupes_instructeurs.many?
- if group.dossiers.empty? - if group.dossiers.empty?
%td.actions %td.actions
= link_to procedure_groupe_instructeur_path(@procedure, group), { method: :delete, class: 'button', data: { confirm: "Êtes-vous sûr de vouloir supprimer le groupe « #{group.label} » ?" }} do = link_to admin_procedure_groupe_instructeur_path(@procedure, group), { method: :delete, class: 'button', data: { confirm: "Êtes-vous sûr de vouloir supprimer le groupe « #{group.label} » ?" }} do
%span.icon.delete %span.icon.delete
supprimer ce groupe supprimer ce groupe
- else - else
%td.actions %td.actions
= link_to reaffecter_dossiers_procedure_groupe_instructeur_path(@procedure, group), class: 'button', title:'Réaffecter les dossiers à un autre groupe afin de pouvoir le supprimer' do = link_to reaffecter_dossiers_admin_procedure_groupe_instructeur_path(@procedure, group), class: 'button', title:'Réaffecter les dossiers à un autre groupe afin de pouvoir le supprimer' do
%span.icon.follow %span.icon.follow
déplacer les dossiers déplacer les dossiers
= paginate @groupes_instructeurs = paginate @groupes_instructeurs

4
app/views/new_administrateur/groupe_instructeurs/reaffecter_dossiers.html.haml
View file

@ -1,7 +1,7 @@
= render partial: 'new_administrateur/breadcrumbs', = render partial: 'new_administrateur/breadcrumbs',
locals: { steps: [link_to('Démarches', admin_procedures_path), locals: { steps: [link_to('Démarches', admin_procedures_path),
link_to(@procedure.libelle, admin_procedure_path(@procedure)), link_to(@procedure.libelle, admin_procedure_path(@procedure)),
link_to('Groupes d’instructeurs', procedure_groupe_instructeurs_path(@procedure)), link_to('Groupes d’instructeurs', admin_procedure_groupe_instructeurs_path(@procedure)),
@groupe_instructeur.label] } @groupe_instructeur.label] }
.container.groupe-instructeur .container.groupe-instructeur
@ -20,7 +20,7 @@
%tr %tr
%td= group.label %td= group.label
%td.actions= button_to 'Réaffecter les dossiers à ce groupe', %td.actions= button_to 'Réaffecter les dossiers à ce groupe',
reaffecter_procedure_groupe_instructeur_path(:target_group => group), reaffecter_admin_procedure_groupe_instructeur_path(:target_group => group),
{ class: 'button', { class: 'button',
data: { confirm: "Êtes-vous sûr de vouloir réaffecter les dossiers du groupe « #{@groupe_instructeur.label} » vers le groupe  « #{group.label} » ?" } } data: { confirm: "Êtes-vous sûr de vouloir réaffecter les dossiers du groupe « #{@groupe_instructeur.label} » vers le groupe  « #{group.label} » ?" } }

2
app/views/new_administrateur/groupe_instructeurs/show.html.haml
View file

@ -3,7 +3,7 @@
= render partial: 'new_administrateur/breadcrumbs', = render partial: 'new_administrateur/breadcrumbs',
locals: { steps: [link_to('Démarches', admin_procedures_path), locals: { steps: [link_to('Démarches', admin_procedures_path),
link_to(@procedure.libelle, admin_procedure_path(@procedure)), link_to(@procedure.libelle, admin_procedure_path(@procedure)),
link_to('Groupes d’instructeurs', procedure_groupe_instructeurs_path(@procedure)), link_to('Groupes d’instructeurs', admin_procedure_groupe_instructeurs_path(@procedure)),
@groupe_instructeur.label] } @groupe_instructeur.label] }
- else - else
= render partial: 'new_administrateur/breadcrumbs', = render partial: 'new_administrateur/breadcrumbs',

2
app/views/new_administrateur/procedure_administrateurs/_administrateur.html.haml
View file

@ -7,7 +7,7 @@
C’est vous ! C’est vous !
- else - else
= link_to 'Retirer', = link_to 'Retirer',
[@procedure, administrateur], admin_procedure_administrateur_path(@procedure, administrateur),
method: :delete, method: :delete,
'data-confirm': "Retirer « #{administrateur.email} » des administrateurs de « #{@procedure.libelle} » ?", 'data-confirm': "Retirer « #{administrateur.email} » des administrateurs de « #{@procedure.libelle} » ?",
remote: true remote: true

4
app/views/new_administrateur/procedures/apercu.html.haml
View file

@ -4,11 +4,11 @@
%ul.tabs %ul.tabs
= tab_item('le dossier', = tab_item('le dossier',
apercu_procedure_path(@dossier.procedure, tab: 'dossier'), apercu_admin_procedure_path(@dossier.procedure, tab: 'dossier'),
active: @tab == 'dossier') active: @tab == 'dossier')
- if @dossier.champs_private.size > 0 - if @dossier.champs_private.size > 0
= tab_item('les annotations privées', = tab_item('les annotations privées',
apercu_procedure_path(@dossier.procedure, tab: 'annotations-privees'), apercu_admin_procedure_path(@dossier.procedure, tab: 'annotations-privees'),
active: @tab == 'annotations-privees') active: @tab == 'annotations-privees')
- if @tab == 'dossier' - if @tab == 'dossier'

18
config/routes.rb
View file

@ -170,12 +170,6 @@ Rails.application.routes.draw do
# order matters: we don't want those routes to match /admin/procedures/:id # order matters: we don't want those routes to match /admin/procedures/:id
get 'admin/procedures/new' => 'new_administrateur/procedures#new', as: :new_admin_procedure get 'admin/procedures/new' => 'new_administrateur/procedures#new', as: :new_admin_procedure
get 'admin/procedures/:id/edit' => 'new_administrateur/procedures#edit', as: :edit_admin_procedure
post 'admin/procedures' => 'new_administrateur/procedures#create'
get 'admin/procedures/:id/monavis' => 'new_administrateur/procedures#monavis', as: :admin_procedure_monavis
patch 'admin/procedures/:id/monavis' => 'new_administrateur/procedures#update_monavis', as: :update_monavis
get 'admin/procedures/:id/jeton' => 'new_administrateur/procedures#jeton', as: :admin_procedure_jeton
patch 'admin/procedures/:id/jeton' => 'new_administrateur/procedures#update_jeton', as: :update_jeton
namespace :admin do namespace :admin do
get 'activate' => '/administrateurs/activate#new' get 'activate' => '/administrateurs/activate#new'
@ -183,7 +177,7 @@ Rails.application.routes.draw do
get 'procedures/archived' => 'procedures#archived' get 'procedures/archived' => 'procedures#archived'
get 'procedures/draft' => 'procedures#draft' get 'procedures/draft' => 'procedures#draft'
resources :procedures, except: [:new, :edit, :update] do resources :procedures, only: [:index, :show, :destroy] do
collection do collection do
get 'new_from_existing' => 'procedures#new_from_existing', as: :new_from_existing get 'new_from_existing' => 'procedures#new_from_existing', as: :new_from_existing
end end
@ -375,12 +369,16 @@ Rails.application.routes.draw do
# Administrateur # Administrateur
# #
scope module: 'new_administrateur' do namespace :admin, module: 'new_administrateur' do
resources :procedures, only: [:update, :new] do resources :procedures, except: [:index, :show, :destroy] do
member do member do
get 'apercu' get 'apercu'
get 'champs' get 'champs'
get 'annotations' get 'annotations'
get 'monavis'
patch 'update_monavis'
get 'jeton'
patch 'update_jeton'
end end
resources :groupe_instructeurs, only: [:index, :show, :create, :update, :destroy] do resources :groupe_instructeurs, only: [:index, :show, :create, :update, :destroy] do
@ -408,7 +406,9 @@ Rails.application.routes.draw do
get 'preview', on: :member get 'preview', on: :member
end end
end end
end
scope module: 'new_administrateur' do
resources :services, except: [:show] do resources :services, except: [:show] do
collection do collection do
patch 'add_to_procedure' patch 'add_to_procedure'

2
spec/controllers/admin/instructeurs_controller_spec.rb
View file

@ -43,7 +43,7 @@ describe Admin::InstructeursController, type: :controller do
let(:procedure) { create :procedure } let(:procedure) { create :procedure }
let(:procedure_id) { procedure.id } let(:procedure_id) { procedure.id }
it { expect(response.status).to eq(302) } it { expect(response.status).to eq(302) }
it { expect(response).to redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) } it { expect(response).to redirect_to admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
end end
describe 'Instructeur attributs in database' do describe 'Instructeur attributs in database' do

36
spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb
View file

@ -62,7 +62,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
let(:label) { "nouveau_groupe" } let(:label) { "nouveau_groupe" }
it { expect(flash.notice).to be_present } it { expect(flash.notice).to be_present }
it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, procedure.groupe_instructeurs.last)) } it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, procedure.groupe_instructeurs.last)) }
it { expect(procedure.groupe_instructeurs.count).to eq(2) } it { expect(procedure.groupe_instructeurs.count).to eq(2) }
end end
@ -88,7 +88,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
before { delete_group gi_1_1 } before { delete_group gi_1_1 }
it { expect(flash.alert).to be_present } it { expect(flash.alert).to be_present }
it { expect(response).to redirect_to(procedure_groupe_instructeurs_path(procedure)) } it { expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure)) }
it { expect(procedure.groupe_instructeurs.count).to eq(1) } it { expect(procedure.groupe_instructeurs.count).to eq(1) }
end end
@ -99,7 +99,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
before { delete_group gi_1_2 } before { delete_group gi_1_2 }
it { expect(flash.notice).to be_present } it { expect(flash.notice).to be_present }
it { expect(procedure.groupe_instructeurs.count).to eq(1) } it { expect(procedure.groupe_instructeurs.count).to eq(1) }
it { expect(response).to redirect_to(procedure_groupe_instructeurs_path(procedure)) } it { expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure)) }
end end
context 'of a group with dossiers, that cannot be deleted' do context 'of a group with dossiers, that cannot be deleted' do
@ -108,7 +108,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
it { expect(flash.alert).to be_present } it { expect(flash.alert).to be_present }
it { expect(procedure.groupe_instructeurs.count).to eq(2) } it { expect(procedure.groupe_instructeurs.count).to eq(2) }
it { expect(response).to redirect_to(procedure_groupe_instructeurs_path(procedure)) } it { expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure)) }
end end
end end
end end
@ -125,7 +125,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
} }
end end
def reaffecter_url(group) def reaffecter_url(group)
reaffecter_procedure_groupe_instructeur_path(:id => gi_1_2, reaffecter_admin_procedure_groupe_instructeur_path(:id => gi_1_2,
:target_group => group) :target_group => group)
end end
@ -155,7 +155,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
dossier12.reload dossier12.reload
end end
it { expect(response).to redirect_to(procedure_groupe_instructeurs_path(procedure)) } it { expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure)) }
it { expect(gi_1_1.dossiers.with_discarded.count).to be(0) } it { expect(gi_1_1.dossiers.with_discarded.count).to be(0) }
it { expect(gi_1_2.dossiers.with_discarded.count).to be(2) } it { expect(gi_1_2.dossiers.with_discarded.count).to be(2) }
it { expect(gi_1_2.dossiers.last.id).to be(dossier12.id) } it { expect(gi_1_2.dossiers.last.id).to be(dossier12.id) }
@ -193,7 +193,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
end end
it { expect(gi_1_1.reload.label).to eq(new_name) } it { expect(gi_1_1.reload.label).to eq(new_name) }
it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_1)) } it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_1)) }
it { expect(flash.notice).to be_present } it { expect(flash.notice).to be_present }
context 'when the name is already taken' do context 'when the name is already taken' do
@ -215,7 +215,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
it { expect(response.status).to eq(200) } it { expect(response.status).to eq(200) }
it { expect(subject.request.flash[:alert]).to be_nil } it { expect(subject.request.flash[:alert]).to be_nil }
it { expect(subject.request.flash[:notice]).to be_present } it { expect(subject.request.flash[:notice]).to be_present }
it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) } it { expect(subject).to redirect_to admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
end end
context 'when there is at least one bad email' do context 'when there is at least one bad email' do
@ -223,13 +223,13 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
it { expect(response.status).to eq(200) } it { expect(response.status).to eq(200) }
it { expect(subject.request.flash[:alert]).to be_present } it { expect(subject.request.flash[:alert]).to be_present }
it { expect(subject.request.flash[:notice]).to be_present } it { expect(subject.request.flash[:notice]).to be_present }
it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) } it { expect(subject).to redirect_to admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
end end
context 'when the admin wants to assign an instructor who is already assigned on this procedure' do context 'when the admin wants to assign an instructor who is already assigned on this procedure' do
let(:emails) { ['instructeur_1@ministere_a.gouv.fr'] } let(:emails) { ['instructeur_1@ministere_a.gouv.fr'] }
it { expect(subject.request.flash[:alert]).to be_present } it { expect(subject.request.flash[:alert]).to be_present }
it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) } it { expect(subject).to redirect_to admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
end end
end end
@ -256,7 +256,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
it { expect(gi_1_2.instructeurs.pluck(:email)).to include(*new_instructeur_emails) } it { expect(gi_1_2.instructeurs.pluck(:email)).to include(*new_instructeur_emails) }
it { expect(flash.notice).to be_present } it { expect(flash.notice).to be_present }
it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_2)) } it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_2)) }
it { expect(procedure.routee?).to be_truthy } it { expect(procedure.routee?).to be_truthy }
it "calls GroupeInstructeurMailer with the right groupe and instructeurs" do it "calls GroupeInstructeurMailer with the right groupe and instructeurs" do
expect(GroupeInstructeurMailer).to have_received(:add_instructeurs).with( expect(GroupeInstructeurMailer).to have_received(:add_instructeurs).with(
@ -270,20 +270,20 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
context 'of an instructeur already in the group' do context 'of an instructeur already in the group' do
let(:new_instructeur_emails) { [instructeur.email] } let(:new_instructeur_emails) { [instructeur.email] }
it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_2)) } it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_2)) }
end end
context 'of badly formed email' do context 'of badly formed email' do
let(:new_instructeur_emails) { ['badly_formed_email'] } let(:new_instructeur_emails) { ['badly_formed_email'] }
it { expect(flash.alert).to be_present } it { expect(flash.alert).to be_present }
it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_2)) } it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_2)) }
end end
context 'of an empty string' do context 'of an empty string' do
let(:new_instructeur_emails) { '' } let(:new_instructeur_emails) { '' }
it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_2)) } it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_2)) }
end end
end end
@ -306,7 +306,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
it { expect(gi_1_1.instructeurs).to include(instructeur) } it { expect(gi_1_1.instructeurs).to include(instructeur) }
it { expect(gi_1_1.reload.instructeurs.count).to eq(1) } it { expect(gi_1_1.reload.instructeurs.count).to eq(1) }
it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_1)) } it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_1)) }
end end
context 'when there is only one instructeur' do context 'when there is only one instructeur' do
@ -318,7 +318,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
it { expect(gi_1_1.instructeurs).to include(instructeur) } it { expect(gi_1_1.instructeurs).to include(instructeur) }
it { expect(gi_1_1.instructeurs.count).to eq(1) } it { expect(gi_1_1.instructeurs.count).to eq(1) }
it { expect(flash.alert).to eq('Suppression impossible : il doit y avoir au moins un instructeur dans le groupe') } it { expect(flash.alert).to eq('Suppression impossible : il doit y avoir au moins un instructeur dans le groupe') }
it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_1)) } it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_1)) }
end end
end end
@ -337,7 +337,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
it { expect(subject.request.flash[:notice]).to be_present } it { expect(subject.request.flash[:notice]).to be_present }
it { expect(subject.request.flash[:alert]).to be_nil } it { expect(subject.request.flash[:alert]).to be_nil }
it { expect(response.status).to eq(302) } it { expect(response.status).to eq(302) }
it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, gi_1_1) } it { expect(subject).to redirect_to admin_procedure_groupe_instructeur_path(procedure, gi_1_1) }
end end
context 'when the instructor is not assigned to the procedure' do context 'when the instructor is not assigned to the procedure' do
@ -345,7 +345,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
it { expect(subject.request.flash[:alert]).to be_present } it { expect(subject.request.flash[:alert]).to be_present }
it { expect(subject.request.flash[:notice]).to be_nil } it { expect(subject.request.flash[:notice]).to be_nil }
it { expect(response.status).to eq(302) } it { expect(response.status).to eq(302) }
it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, gi_1_1) } it { expect(subject).to redirect_to admin_procedure_groupe_instructeur_path(procedure, gi_1_1) }
end end
end end

2
spec/controllers/new_administrateur/procedures_controller_spec.rb
View file

@ -102,7 +102,7 @@ describe NewAdministrateur::ProceduresController, type: :controller do
it { expect(subject.duree_conservation_dossiers_hors_ds).to eq(duree_conservation_dossiers_hors_ds) } it { expect(subject.duree_conservation_dossiers_hors_ds).to eq(duree_conservation_dossiers_hors_ds) }
end end
it { is_expected.to redirect_to(champs_procedure_path(Procedure.last)) } it { is_expected.to redirect_to(champs_admin_procedure_path(Procedure.last)) }
it { expect(flash[:notice]).to be_present } it { expect(flash[:notice]).to be_present }
end end

6
spec/features/admin/procedure_creation_spec.rb
View file

@ -45,7 +45,7 @@ feature 'As an administrateur I wanna create a new procedure', js: true do
fill_in_dummy_procedure_details fill_in_dummy_procedure_details
click_on 'Créer la démarche' click_on 'Créer la démarche'
expect(page).to have_current_path(champs_procedure_path(Procedure.last)) expect(page).to have_current_path(champs_admin_procedure_path(Procedure.last))
end end
end end
end end
@ -66,7 +66,7 @@ feature 'As an administrateur I wanna create a new procedure', js: true do
scenario 'Add champ, add file, visualize them in procedure preview' do scenario 'Add champ, add file, visualize them in procedure preview' do
page.refresh page.refresh
expect(page).to have_current_path(champs_procedure_path(Procedure.last)) expect(page).to have_current_path(champs_admin_procedure_path(Procedure.last))
add_champ(remove_flash_message: true) add_champ(remove_flash_message: true)
fill_in 'champ-0-libelle', with: 'libelle de champ' fill_in 'champ-0-libelle', with: 'libelle de champ'
@ -80,7 +80,7 @@ feature 'As an administrateur I wanna create a new procedure', js: true do
preview_window = window_opened_by { click_on 'onglet-preview' } preview_window = window_opened_by { click_on 'onglet-preview' }
within_window(preview_window) do within_window(preview_window) do
expect(page).to have_current_path(apercu_procedure_path(Procedure.last)) expect(page).to have_current_path(apercu_admin_procedure_path(Procedure.last))
expect(page).to have_field('libelle de champ') expect(page).to have_field('libelle de champ')
end end
end end

2
spec/features/new_administrateur/types_de_champ_spec.rb
View file

@ -4,7 +4,7 @@ feature 'As an administrateur I can edit types de champ', js: true do
before do before do
login_as administrateur.user, scope: :user login_as administrateur.user, scope: :user
visit champs_procedure_path(procedure) visit champs_admin_procedure_path(procedure)
end end
it "Add a new champ" do it "Add a new champ" do

4
spec/features/routing/full_scenario_spec.rb
View file

@ -147,7 +147,7 @@ feature 'The routing', js: true do
# the instructeurs who belong to scientifique AND litteraire groups manage scientifique and litterraire dossiers # the instructeurs who belong to scientifique AND litteraire groups manage scientifique and litterraire dossiers
register_instructeur_and_log_in(superwoman.email) register_instructeur_and_log_in(superwoman.email)
visit procedure_path(procedure, params: { statut: 'tous' }) visit instructeur_procedure_path(procedure, params: { statut: 'tous' })
expect(page).to have_text(litteraire_user.email) expect(page).to have_text(litteraire_user.email)
expect(page).to have_text(scientifique_user.email) expect(page).to have_text(scientifique_user.email)
@ -155,7 +155,7 @@ feature 'The routing', js: true do
click_on scientifique_user.email click_on scientifique_user.email
click_on 'Suivre le dossier' click_on 'Suivre le dossier'
visit procedure_path(procedure, params: { statut: 'tous' }) visit instructeur_procedure_path(procedure, params: { statut: 'tous' })
click_on litteraire_user.email click_on litteraire_user.email
click_on 'Suivre le dossier' click_on 'Suivre le dossier'
log_out log_out

6
spec/views/instructeur_mailer/send_notifications.html.haml_spec.rb
View file

@ -21,7 +21,7 @@ describe 'instructeur_mailer/send_notifications.html.haml', type: :view do
] ]
end end
it { expect(rendered).to have_link('une superbe démarche', href: procedure_url(213)) } it { expect(rendered).to have_link('une superbe démarche', href: instructeur_procedure_url(213)) }
it { expect(rendered).to have_text('une superbe démarche') } it { expect(rendered).to have_text('une superbe démarche') }
it { expect(rendered).to have_text('1 dossier en construction') } it { expect(rendered).to have_text('1 dossier en construction') }
it { expect(rendered).not_to have_text('notification') } it { expect(rendered).not_to have_text('notification') }
@ -41,7 +41,7 @@ describe 'instructeur_mailer/send_notifications.html.haml', type: :view do
] ]
end end
it { expect(rendered).to have_link('une superbe démarche', href: procedure_url(213)) } it { expect(rendered).to have_link('une superbe démarche', href: instructeur_procedure_url(213)) }
it { expect(rendered).to have_text('une superbe démarche') } it { expect(rendered).to have_text('une superbe démarche') }
it { expect(rendered).to have_text('1 dossier') } it { expect(rendered).to have_text('1 dossier') }
it { expect(rendered).not_to have_text('notification') } it { expect(rendered).not_to have_text('notification') }
@ -63,7 +63,7 @@ describe 'instructeur_mailer/send_notifications.html.haml', type: :view do
] ]
end end
it { expect(rendered).to have_link('une superbe démarche', href: procedure_url(213)) } it { expect(rendered).to have_link('une superbe démarche', href: instructeur_procedure_url(213)) }
it { expect(rendered).to have_text('une superbe démarche') } it { expect(rendered).to have_text('une superbe démarche') }
it { expect(rendered).to have_text('1 dossier') } it { expect(rendered).to have_text('1 dossier') }
it { expect(rendered).not_to have_text('notification') } it { expect(rendered).not_to have_text('notification') }