diff --git a/app/controllers/champs/siret_controller.rb b/app/controllers/champs/siret_controller.rb index 169134aab..98ebb81ee 100644 --- a/app/controllers/champs/siret_controller.rb +++ b/app/controllers/champs/siret_controller.rb @@ -46,7 +46,7 @@ class Champs::SiretController < ApplicationController @champ = policy_scope(Champ).find(params[:champ_id]) @etablissement = @champ&.etablissement end - @procedure_id = @champ&.dossier&.procedure_id || 'aperçu' + @procedure_id = @champ&.dossier&.procedure&.id || 'aperçu' end def find_etablissement_with_siret diff --git a/app/controllers/instructeurs/recherche_controller.rb b/app/controllers/instructeurs/recherche_controller.rb index 8d14ce2f8..298abb3ea 100644 --- a/app/controllers/instructeurs/recherche_controller.rb +++ b/app/controllers/instructeurs/recherche_controller.rb @@ -5,7 +5,7 @@ module Instructeurs @dossiers = DossierSearchService.matching_dossiers_for_instructeur(@search_terms, current_instructeur) @followed_dossiers_id = current_instructeur .followed_dossiers - .where(procedure_id: @dossiers.pluck(:procedure_id)) + .where(groupe_instructeur_id: @dossiers.pluck(:groupe_instructeur_id)) .pluck(:id) end end diff --git a/app/jobs/etablissement_update_job.rb b/app/jobs/etablissement_update_job.rb index 134d9b26d..578bf3cb4 100644 --- a/app/jobs/etablissement_update_job.rb +++ b/app/jobs/etablissement_update_job.rb @@ -1,7 +1,7 @@ class EtablissementUpdateJob < ApplicationJob def perform(dossier, siret) begin - etablissement_attributes = ApiEntrepriseService.get_etablissement_params_for_siret(siret, dossier.procedure_id) + etablissement_attributes = ApiEntrepriseService.get_etablissement_params_for_siret(siret, dossier.procedure.id) rescue return end diff --git a/app/lib/api_entreprise/api.rb b/app/lib/api_entreprise/api.rb index 17725743a..b302ac2b9 100644 --- a/app/lib/api_entreprise/api.rb +++ b/app/lib/api_entreprise/api.rb @@ -78,7 +78,7 @@ class ApiEntreprise::API elsif response.code == 400 raise BadFormatRequest, "url: #{url}" else - raise RequestFailed, "HTTP Error Code: #{response.code} for #{url}" + raise RequestFailed, "HTTP Error Code: #{response.code} for #{url}\nheaders: #{response.headers}\nbody: #{response.body}" end end diff --git a/app/models/dossier.rb b/app/models/dossier.rb index ea11bc25b..b3241ba37 100644 --- a/app/models/dossier.rb +++ b/app/models/dossier.rb @@ -1,5 +1,5 @@ class Dossier < ApplicationRecord - self.ignored_columns = ['json_latlngs'] + self.ignored_columns = ['procedure_id'] include DossierFilteringConcern include Discard::Model diff --git a/lib/tasks/deployment/20190826153115_link_dossier_and_groupe_instructeur.rake b/lib/tasks/deployment/20190826153115_link_dossier_and_groupe_instructeur.rake deleted file mode 100644 index aeb50123b..000000000 --- a/lib/tasks/deployment/20190826153115_link_dossier_and_groupe_instructeur.rake +++ /dev/null @@ -1,15 +0,0 @@ -namespace :after_party do - desc 'Deployment task: link_dossier_and_groupe_instructeur' - task link_dossier_and_groupe_instructeur: :environment do - sql = <<~SQL - UPDATE dossiers AS d1 SET groupe_instructeur_id = g.id - FROM groupe_instructeurs AS g - WHERE g.procedure_id = d1.procedure_id - and d1.groupe_instructeur_id is null; - SQL - - ActiveRecord::Base.connection.execute(sql) - - AfterParty::TaskRecord.create version: '20190826153115' - end -end diff --git a/spec/lib/tasks/deployment/20190826153115_link_dossier_and_groupe_instructeur.rake_spec.rb b/spec/lib/tasks/deployment/20190826153115_link_dossier_and_groupe_instructeur.rake_spec.rb deleted file mode 100644 index 381e6aa14..000000000 --- a/spec/lib/tasks/deployment/20190826153115_link_dossier_and_groupe_instructeur.rake_spec.rb +++ /dev/null @@ -1,36 +0,0 @@ -describe '20190826153115_link_dossier_and_groupe_instructeur.rake' do - let(:rake_task) { Rake::Task['after_party:link_dossier_and_groupe_instructeur'] } - - subject { rake_task.invoke } - after { rake_task.reenable } - - context 'with 3 dossiers' do - let!(:procedure) { create(:procedure) } - let!(:procedure2) { create(:procedure) } - let!(:other_procedure_needed_to_create_dossier) { create(:procedure) } - let!(:other_gi) { other_procedure_needed_to_create_dossier.defaut_groupe_instructeur } - let!(:dossier) { Dossier.create(user: create(:user), procedure_id: procedure.id, groupe_instructeur: other_gi) } - let!(:dossier2) { Dossier.create(user: create(:user), procedure_id: procedure2.id, groupe_instructeur: other_gi) } - let!(:dossier3) { Dossier.create(user: create(:user), procedure_id: procedure2.id, groupe_instructeur: other_gi) } - - before do - [dossier, dossier2, dossier3].each do |d| - d.update_column('groupe_instructeur_id', nil) - end - - other_procedure_needed_to_create_dossier.groupe_instructeurs.destroy_all - other_procedure_needed_to_create_dossier.destroy - end - - it do - expect(dossier.reload.groupe_instructeur_id).to be_nil - subject - expect(Dossier.count).to eq(3) - expect(Procedure.count).to eq(2) - expect(GroupeInstructeur.count).to eq(2) - expect(dossier.reload.groupe_instructeur_id).to eq(procedure.defaut_groupe_instructeur.id) - expect(dossier2.reload.groupe_instructeur_id).to eq(procedure2.defaut_groupe_instructeur.id) - expect(dossier3.reload.groupe_instructeur_id).to eq(procedure2.defaut_groupe_instructeur.id) - end - end -end