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

fix(graphql): parse_etablissement_address is slow (300ms) – bypasse it when possible

Browse source
This commit is contained in:
Paul Chavard 2024-10-11 17:13:53 +02:00
parent 8266f7ff6c
commit b027839721
No known key found for this signature in database
2 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
app/graphql/types/personne_morale_type.rb
View file

@ -124,8 +124,11 @@ module Types
field :complement_adresse, String, null: true, deprecation_reason: "Utilisez le champ `address` à la place." field :complement_adresse, String, null: true, deprecation_reason: "Utilisez le champ `address` à la place."
def address def address
APIGeoService address = object.champ&.value_json
.parse_etablissement_address(object) if address.blank? || !address.key?("departement_code")
address = APIGeoService.parse_etablissement_address(object)
end
address
.merge(label: object.adresse, type: "housenumber") .merge(label: object.adresse, type: "housenumber")
.with_indifferent_access .with_indifferent_access
end end

3
spec/controllers/api/v2/graphql_controller_stored_queries_spec.rb
View file

@ -122,8 +122,9 @@ describe API::V2::GraphqlController do
end end
context 'with entreprise' do context 'with entreprise' do
let(:types_de_champ_public) { [{ type: :siret }] }
let(:procedure) { create(:procedure, :published, :with_service, administrateurs: [admin], types_de_champ_public:) } let(:procedure) { create(:procedure, :published, :with_service, administrateurs: [admin], types_de_champ_public:) }
let(:dossier) { create(:dossier, :en_construction, :with_entreprise, procedure: procedure) } let(:dossier) { create(:dossier, :en_construction, :with_entreprise, :with_populated_champs, procedure: procedure) }
it { it {
expect(gql_errors).to be_nil expect(gql_errors).to be_nil