diff --git a/app/controllers/admin/instructeurs_controller.rb b/app/controllers/admin/instructeurs_controller.rb index a3d532234..1f549756c 100644 --- a/app/controllers/admin/instructeurs_controller.rb +++ b/app/controllers/admin/instructeurs_controller.rb @@ -22,7 +22,7 @@ class Admin::InstructeursController < AdminController end 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 redirect_to admin_instructeurs_path end diff --git a/app/controllers/new_administrateur/groupe_instructeurs_controller.rb b/app/controllers/new_administrateur/groupe_instructeurs_controller.rb index 1e387f621..e90523e76 100644 --- a/app/controllers/new_administrateur/groupe_instructeurs_controller.rb +++ b/app/controllers/new_administrateur/groupe_instructeurs_controller.rb @@ -21,7 +21,7 @@ module NewAdministrateur .new(label: label, instructeurs: [current_administrateur.instructeur]) 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éé." else @procedure = procedure @@ -36,7 +36,7 @@ module NewAdministrateur @groupe_instructeur = groupe_instructeur 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} »." else @procedure = procedure @@ -58,7 +58,7 @@ module NewAdministrateur groupe_instructeur.destroy! flash[:notice] = "le groupe « #{label} » a été supprimé." end - redirect_to procedure_groupe_instructeurs_path(procedure) + redirect_to admin_procedure_groupe_instructeurs_path(procedure) end def reaffecter_dossiers @@ -76,7 +76,7 @@ module NewAdministrateur end 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 def add_instructeur @@ -123,7 +123,7 @@ module NewAdministrateur end end - redirect_to procedure_groupe_instructeur_path(procedure, groupe_instructeur) + redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur) end def remove_instructeur @@ -148,13 +148,13 @@ module NewAdministrateur end end end - redirect_to procedure_groupe_instructeur_path(procedure, groupe_instructeur) + redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur) end def update_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} »." end diff --git a/app/controllers/new_administrateur/procedures_controller.rb b/app/controllers/new_administrateur/procedures_controller.rb index a5d266ced..a27895847 100644 --- a/app/controllers/new_administrateur/procedures_controller.rb +++ b/app/controllers/new_administrateur/procedures_controller.rb @@ -25,7 +25,7 @@ module NewAdministrateur flash.notice = 'Démarche enregistrée.' current_administrateur.instructeur.assign_to_procedure(@procedure) - redirect_to champs_procedure_path(@procedure) + redirect_to champs_admin_procedure_path(@procedure) end end diff --git a/app/helpers/procedure_helper.rb b/app/helpers/procedure_helper.rb index 922479ec5..8d8362bc8 100644 --- a/app/helpers/procedure_helper.rb +++ b/app/helpers/procedure_helper.rb @@ -23,7 +23,7 @@ module ProcedureHelper isAnnotation: false, typeDeChampsTypes: TypeDeChamp.type_de_champ_types_for(procedure, current_user), 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 } end @@ -33,7 +33,7 @@ module ProcedureHelper isAnnotation: true, typeDeChampsTypes: TypeDeChamp.type_de_champ_types_for(procedure, current_user), 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 } end diff --git a/app/views/admin/mail_templates/edit.html.haml b/app/views/admin/mail_templates/edit.html.haml index e515cf7f3..a69a6a0f9 100644 --- a/app/views/admin/mail_templates/edit.html.haml +++ b/app/views/admin/mail_templates/edit.html.haml @@ -21,7 +21,7 @@ .text-right = link_to "Annuler", admin_procedure_mail_templates_path(@procedure), class: "btn btn-default" = 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 .col-md-12 diff --git a/app/views/admin/procedures/show.html.haml b/app/views/admin/procedures/show.html.haml index 52fbe51b9..a3d22ac03 100644 --- a/app/views/admin/procedures/show.html.haml +++ b/app/views/admin/procedures/show.html.haml @@ -107,7 +107,7 @@ .alert.alert-info Pour pouvoir tester cette démarche, vous devez d’abord lui affecter - 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? et - if @procedure.service.nil? @@ -122,7 +122,7 @@ - if @procedure.missing_steps.include?(:instructeurs) %p.alert.alert-danger 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 Cette démarche n’a pas encore de lien, et n’est pas accessible par le public. diff --git a/app/views/groupe_instructeur_mailer/add_instructeurs.html.haml b/app/views/groupe_instructeur_mailer/add_instructeurs.html.haml index b04110592..b6f0dabff 100644 --- a/app/views/groupe_instructeur_mailer/add_instructeurs.html.haml +++ b/app/views/groupe_instructeur_mailer/add_instructeurs.html.haml @@ -6,6 +6,6 @@ %p 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" diff --git a/app/views/groupe_instructeur_mailer/remove_instructeur.html.haml b/app/views/groupe_instructeur_mailer/remove_instructeur.html.haml index 15a3f15a7..6a8e1cc94 100644 --- a/app/views/groupe_instructeur_mailer/remove_instructeur.html.haml +++ b/app/views/groupe_instructeur_mailer/remove_instructeur.html.haml @@ -6,6 +6,6 @@ %p 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" diff --git a/app/views/instructeur_mailer/send_notifications.html.haml b/app/views/instructeur_mailer/send_notifications.html.haml index 04a44c9fe..7cd1a1dcd 100644 --- a/app/views/instructeur_mailer/send_notifications.html.haml +++ b/app/views/instructeur_mailer/send_notifications.html.haml @@ -9,7 +9,7 @@ %ul - @data.each do |datum| %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 %br #{datum[:nb_en_construction]} #{'dossier'.pluralize(datum[:nb_en_construction])} en construction diff --git a/app/views/instructeurs/groupe_instructeurs/index.html.haml b/app/views/instructeurs/groupe_instructeurs/index.html.haml index 97e62c876..009e150fb 100644 --- a/app/views/instructeurs/groupe_instructeurs/index.html.haml +++ b/app/views/instructeurs/groupe_instructeurs/index.html.haml @@ -1,7 +1,7 @@ - content_for(:title, "Notifications pour #{@procedure.libelle}") = 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'] } .container.groupe-instructeur diff --git a/app/views/instructeurs/groupe_instructeurs/show.html.haml b/app/views/instructeurs/groupe_instructeurs/show.html.haml index 1fcc742ef..f4faaef80 100644 --- a/app/views/instructeurs/groupe_instructeurs/show.html.haml +++ b/app/views/instructeurs/groupe_instructeurs/show.html.haml @@ -1,7 +1,7 @@ - content_for(:title, "Instructeurs du groupe #{@groupe_instructeur.label}") = 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)), @groupe_instructeur.label] } diff --git a/app/views/instructeurs/procedures/deleted_dossiers.html.haml b/app/views/instructeurs/procedures/deleted_dossiers.html.haml index 9063e18e2..65a2413a6 100644 --- a/app/views/instructeurs/procedures/deleted_dossiers.html.haml +++ b/app/views/instructeurs/procedures/deleted_dossiers.html.haml @@ -16,7 +16,7 @@ - if @procedure.routee? | - 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 = link_to 'instructeurs', instructeur_groupes_path(@procedure), class: 'header-link' diff --git a/app/views/instructeurs/procedures/show.html.haml b/app/views/instructeurs/procedures/show.html.haml index 2658d6a9b..948338ee2 100644 --- a/app/views/instructeurs/procedures/show.html.haml +++ b/app/views/instructeurs/procedures/show.html.haml @@ -16,7 +16,7 @@ - if @procedure.routee? | - 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 = link_to 'instructeurs', instructeur_groupes_path(@procedure), class: 'header-link' diff --git a/app/views/layouts/left_panels/_left_panel_admin_procedurescontroller_navbar.html.haml b/app/views/layouts/left_panels/_left_panel_admin_procedurescontroller_navbar.html.haml index 6ef6c05ea..a262889dc 100644 --- a/app/views/layouts/left_panels/_left_panel_admin_procedurescontroller_navbar.html.haml +++ b/app/views/layouts/left_panels/_left_panel_admin_procedurescontroller_navbar.html.haml @@ -26,33 +26,33 @@ - if @procedure.missing_steps.include?(:service) %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 Administrateurs - 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') } Instructeurs - if @procedure.missing_steps.include?(:instructeurs) %p.missing-steps (à compléter) - 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 Groupe d'instructeurs - 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') } Champs - 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') } 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') } Prévisualisation @@ -64,11 +64,11 @@ .procedure-list-element{ class: ('active' if active == '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') } 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 Jeton diff --git a/app/views/manager/procedures/show.html.erb b/app/views/manager/procedures/show.html.erb index cae25c6be..da0724ee8 100644 --- a/app/views/manager/procedures/show.html.erb +++ b/app/views/manager/procedures/show.html.erb @@ -36,7 +36,7 @@ as well as a link to its edit page.
- <%= link_to 'aperçu', apercu_procedure_path(procedure), class: 'button' %> + <%= link_to 'aperçu', apercu_admin_procedure_path(procedure), class: 'button' %> <% if !procedure.whitelisted? %> <%= link_to 'whitelister', whitelist_manager_procedure_path(procedure), method: :post, class: 'button' %> diff --git a/app/views/new_administrateur/_breadcrumbs.html.haml b/app/views/new_administrateur/_breadcrumbs.html.haml index eaff46f36..b8dc5d94c 100644 --- a/app/views/new_administrateur/_breadcrumbs.html.haml +++ b/app/views/new_administrateur/_breadcrumbs.html.haml @@ -4,5 +4,5 @@ - steps.each do |step| %li= step - 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' diff --git a/app/views/new_administrateur/_groups_header.haml b/app/views/new_administrateur/_groups_header.haml index e300d6596..547a8d9ed 100644 --- a/app/views/new_administrateur/_groups_header.haml +++ b/app/views/new_administrateur/_groups_header.haml @@ -1,7 +1,7 @@ %h1 Groupe « #{@groupe_instructeur.label} » .card.mt-2 = 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| = f.label :label, 'Nom du groupe' diff --git a/app/views/new_administrateur/groupe_instructeurs/index.html.haml b/app/views/new_administrateur/groupe_instructeurs/index.html.haml index 0e7923121..ed198b774 100644 --- a/app/views/new_administrateur/groupe_instructeurs/index.html.haml +++ b/app/views/new_administrateur/groupe_instructeurs/index.html.haml @@ -33,16 +33,16 @@ - @groupes_instructeurs.each do |group| %tr %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 group.dossiers.empty? %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 supprimer ce groupe - else %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 déplacer les dossiers = paginate @groupes_instructeurs diff --git a/app/views/new_administrateur/groupe_instructeurs/reaffecter_dossiers.html.haml b/app/views/new_administrateur/groupe_instructeurs/reaffecter_dossiers.html.haml index 8e7866162..eae251882 100644 --- a/app/views/new_administrateur/groupe_instructeurs/reaffecter_dossiers.html.haml +++ b/app/views/new_administrateur/groupe_instructeurs/reaffecter_dossiers.html.haml @@ -1,7 +1,7 @@ = render partial: 'new_administrateur/breadcrumbs', locals: { steps: [link_to('Démarches', admin_procedures_path), 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] } .container.groupe-instructeur @@ -20,7 +20,7 @@ %tr %td= group.label %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', data: { confirm: "Êtes-vous sûr de vouloir réaffecter les dossiers du groupe « #{@groupe_instructeur.label} » vers le groupe  « #{group.label} » ?" } } diff --git a/app/views/new_administrateur/groupe_instructeurs/show.html.haml b/app/views/new_administrateur/groupe_instructeurs/show.html.haml index f6a8af25d..c026fa1c9 100644 --- a/app/views/new_administrateur/groupe_instructeurs/show.html.haml +++ b/app/views/new_administrateur/groupe_instructeurs/show.html.haml @@ -3,7 +3,7 @@ = render partial: 'new_administrateur/breadcrumbs', locals: { steps: [link_to('Démarches', admin_procedures_path), 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] } - else = render partial: 'new_administrateur/breadcrumbs', diff --git a/app/views/new_administrateur/procedure_administrateurs/_administrateur.html.haml b/app/views/new_administrateur/procedure_administrateurs/_administrateur.html.haml index 9277ae0ad..ca8fb3310 100644 --- a/app/views/new_administrateur/procedure_administrateurs/_administrateur.html.haml +++ b/app/views/new_administrateur/procedure_administrateurs/_administrateur.html.haml @@ -7,7 +7,7 @@ C’est vous ! - else = link_to 'Retirer', - [@procedure, administrateur], + admin_procedure_administrateur_path(@procedure, administrateur), method: :delete, 'data-confirm': "Retirer « #{administrateur.email} » des administrateurs de « #{@procedure.libelle} » ?", remote: true diff --git a/app/views/new_administrateur/procedures/apercu.html.haml b/app/views/new_administrateur/procedures/apercu.html.haml index 8b09de4e9..857ea4462 100644 --- a/app/views/new_administrateur/procedures/apercu.html.haml +++ b/app/views/new_administrateur/procedures/apercu.html.haml @@ -4,11 +4,11 @@ %ul.tabs = tab_item('le dossier', - apercu_procedure_path(@dossier.procedure, tab: 'dossier'), + apercu_admin_procedure_path(@dossier.procedure, tab: 'dossier'), active: @tab == 'dossier') - if @dossier.champs_private.size > 0 = 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') - if @tab == 'dossier' diff --git a/config/routes.rb b/config/routes.rb index 67801ac80..cb4ce92f1 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -170,12 +170,6 @@ Rails.application.routes.draw do # 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/: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 get 'activate' => '/administrateurs/activate#new' @@ -183,7 +177,7 @@ Rails.application.routes.draw do get 'procedures/archived' => 'procedures#archived' get 'procedures/draft' => 'procedures#draft' - resources :procedures, except: [:new, :edit, :update] do + resources :procedures, only: [:index, :show, :destroy] do collection do get 'new_from_existing' => 'procedures#new_from_existing', as: :new_from_existing end @@ -375,12 +369,16 @@ Rails.application.routes.draw do # Administrateur # - scope module: 'new_administrateur' do - resources :procedures, only: [:update, :new] do + namespace :admin, module: 'new_administrateur' do + resources :procedures, except: [:index, :show, :destroy] do member do get 'apercu' get 'champs' get 'annotations' + get 'monavis' + patch 'update_monavis' + get 'jeton' + patch 'update_jeton' end resources :groupe_instructeurs, only: [:index, :show, :create, :update, :destroy] do @@ -408,7 +406,9 @@ Rails.application.routes.draw do get 'preview', on: :member end end + end + scope module: 'new_administrateur' do resources :services, except: [:show] do collection do patch 'add_to_procedure' diff --git a/spec/controllers/admin/instructeurs_controller_spec.rb b/spec/controllers/admin/instructeurs_controller_spec.rb index ccfe14063..3bf695cdf 100644 --- a/spec/controllers/admin/instructeurs_controller_spec.rb +++ b/spec/controllers/admin/instructeurs_controller_spec.rb @@ -43,7 +43,7 @@ describe Admin::InstructeursController, type: :controller do let(:procedure) { create :procedure } let(:procedure_id) { procedure.id } 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 describe 'Instructeur attributs in database' do diff --git a/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb b/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb index 9e7c1b0e7..a4bf90d06 100644 --- a/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb +++ b/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb @@ -62,7 +62,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do let(:label) { "nouveau_groupe" } 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) } end @@ -88,7 +88,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do before { delete_group gi_1_1 } 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) } end @@ -99,7 +99,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do before { delete_group gi_1_2 } it { expect(flash.notice).to be_present } 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 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(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 @@ -125,7 +125,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do } end 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) end @@ -155,7 +155,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do dossier12.reload 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_2.dossiers.with_discarded.count).to be(2) } it { expect(gi_1_2.dossiers.last.id).to be(dossier12.id) } @@ -193,7 +193,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do end 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 } 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(subject.request.flash[:alert]).to be_nil } 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 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(subject.request.flash[:alert]).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 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'] } 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 @@ -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(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 "calls GroupeInstructeurMailer with the right groupe and instructeurs" do 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 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 context 'of badly formed email' do let(:new_instructeur_emails) { ['badly_formed_email'] } 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 context 'of an empty string' do 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 @@ -306,7 +306,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do it { expect(gi_1_1.instructeurs).to include(instructeur) } 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 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.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(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 @@ -337,7 +337,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do it { expect(subject.request.flash[:notice]).to be_present } it { expect(subject.request.flash[:alert]).to be_nil } 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 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[:notice]).to be_nil } 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 diff --git a/spec/controllers/new_administrateur/procedures_controller_spec.rb b/spec/controllers/new_administrateur/procedures_controller_spec.rb index 19c9f30fb..05f8ccf37 100644 --- a/spec/controllers/new_administrateur/procedures_controller_spec.rb +++ b/spec/controllers/new_administrateur/procedures_controller_spec.rb @@ -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) } 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 } end diff --git a/spec/features/admin/procedure_creation_spec.rb b/spec/features/admin/procedure_creation_spec.rb index 2f33cf8ab..2850d48bf 100644 --- a/spec/features/admin/procedure_creation_spec.rb +++ b/spec/features/admin/procedure_creation_spec.rb @@ -45,7 +45,7 @@ feature 'As an administrateur I wanna create a new procedure', js: true do fill_in_dummy_procedure_details 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 @@ -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 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) 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' } 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') end end diff --git a/spec/features/new_administrateur/types_de_champ_spec.rb b/spec/features/new_administrateur/types_de_champ_spec.rb index c079bb375..c3c5fb77a 100644 --- a/spec/features/new_administrateur/types_de_champ_spec.rb +++ b/spec/features/new_administrateur/types_de_champ_spec.rb @@ -4,7 +4,7 @@ feature 'As an administrateur I can edit types de champ', js: true do before do login_as administrateur.user, scope: :user - visit champs_procedure_path(procedure) + visit champs_admin_procedure_path(procedure) end it "Add a new champ" do diff --git a/spec/features/routing/full_scenario_spec.rb b/spec/features/routing/full_scenario_spec.rb index 08c913e0b..1f5a6213a 100644 --- a/spec/features/routing/full_scenario_spec.rb +++ b/spec/features/routing/full_scenario_spec.rb @@ -147,7 +147,7 @@ feature 'The routing', js: true do # the instructeurs who belong to scientifique AND litteraire groups manage scientifique and litterraire dossiers 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(scientifique_user.email) @@ -155,7 +155,7 @@ feature 'The routing', js: true do click_on scientifique_user.email 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 'Suivre le dossier' log_out diff --git a/spec/views/instructeur_mailer/send_notifications.html.haml_spec.rb b/spec/views/instructeur_mailer/send_notifications.html.haml_spec.rb index c32a37d58..36dc08cbc 100644 --- a/spec/views/instructeur_mailer/send_notifications.html.haml_spec.rb +++ b/spec/views/instructeur_mailer/send_notifications.html.haml_spec.rb @@ -21,7 +21,7 @@ describe 'instructeur_mailer/send_notifications.html.haml', type: :view do ] 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('1 dossier en construction') } it { expect(rendered).not_to have_text('notification') } @@ -41,7 +41,7 @@ describe 'instructeur_mailer/send_notifications.html.haml', type: :view do ] 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('1 dossier') } it { expect(rendered).not_to have_text('notification') } @@ -63,7 +63,7 @@ describe 'instructeur_mailer/send_notifications.html.haml', type: :view do ] 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('1 dossier') } it { expect(rendered).not_to have_text('notification') }