feat: intégration de l'api adresse #212

NazihAsmaa · 2025-01-14T15:44:51Z

fixes #183

…-des-adresses)

…n-api-adresse

A709197 · 2025-01-20T07:48:42Z

Il faudrait modifier la récupération de l'adresse dans l'export de la recherche pour prendre en compte l'API Adresse.

A709197 · 2025-01-20T08:05:35Z

application/services/Etablissement.php

+
+                if (getenv('PREVARISC_API_ADRESSE_MODAL')) {
+                    $DB_adresse_api->delete('ID_ETABLISSEMENT = '.$etablissement->ID_ETABLISSEMENT);
+                } else {
+                    $DB_adresse->delete('ID_ETABLISSEMENT = '.$etablissement->ID_ETABLISSEMENT);
+                }


Je suggère de mettre cette partie dans le Manager

A709197 · 2025-01-20T08:05:57Z

application/services/Etablissement.php

@@ -1,5 +1,6 @@
 <?php

+require_once __DIR__.'/EtablissementManager.php';


Pas normal d'avoir ça là, voir pour l'enlever

A709197 · 2025-01-20T08:07:48Z

application/services/Etablissement.php

@@ -57,6 +57,7 @@ public function get($id_etablissement)
            $DB_commission_type = new Model_DbTable_CommissionType();
            $DB_statut = new Model_DbTable_Statut();
            $DB_dossier = new Model_DbTable_Dossier();
+            $etablissementManager = new Service_EtablissementManager();


Si on utilise plusieurs fois le service, on peut envisager de l'instancier dans un __construct() pour simplifier

A709197 · 2025-01-20T08:08:13Z

application/services/Interface/EtablissementAdresse.php

+<?php
+
+interface Service_Interface_EtablissementAdresse
+{
+    public function get($id_etablissement);
+
+    public function save($adresse, $etablissementID);
+}


Ajouter la méthode delete() ?

A709197 · 2025-01-20T08:20:47Z

application/services/Search.php

+            if ($environnement) {
+                $select->joinLeft('etablissementadresseapi', 'e.ID_ETABLISSEMENT = etablissementadresseapi.ID_ETABLISSEMENT', ['NUMINSEE_COMMUNE', 'LON_ETABLISSEMENTADRESSE', 'LAT_ETABLISSEMENTADRESSE', 'ID_ADRESSE', 'LIBELLE_RUE', 'ADRESSE']);
+            } else {
+                $select->joinLeft('etablissementadresse', 'e.ID_ETABLISSEMENT = etablissementadresse.ID_ETABLISSEMENT', ['NUMINSEE_COMMUNE', 'LON_ETABLISSEMENTADRESSE', 'LAT_ETABLISSEMENTADRESSE', 'ID_ADRESSE', 'ID_RUE', 'NUMERO_ADRESSE'])
+                    ->joinLeft('adressecommune', 'etablissementadresse.NUMINSEE_COMMUNE = adressecommune.NUMINSEE_COMMUNE', 'LIBELLE_COMMUNE AS LIBELLE_COMMUNE_ADRESSE_DEFAULT')
+                ;
+            }
+
+            if (!$environnement) {
+                $select->joinLeft('groupementcommune', 'groupementcommune.NUMINSEE_COMMUNE = adressecommune.NUMINSEE_COMMUNE');
+            } else {
+                $select->joinLeft('groupementcommune', 'groupementcommune.NUMINSEE_COMMUNE = etablissementadresseapi.NUMINSEE_COMMUNE');
+            }
+
+            $select->joinLeft('groupement', 'groupement.ID_GROUPEMENT = groupementcommune.ID_GROUPEMENT AND groupement.ID_GROUPEMENTTYPE = 5', 'LIBELLE_GROUPEMENT');
+            if (!$environnement) {
+                $select->joinLeft('adresserue', 'adresserue.ID_RUE = etablissementadresse.ID_RUE', 'LIBELLE_RUE')
+                    ->joinLeft(['etablissementadressesite' => 'etablissementadresse'], 'etablissementadressesite.ID_ETABLISSEMENT = (SELECT ID_FILS_ETABLISSEMENT FROM etablissementlie WHERE ID_ETABLISSEMENT = e.ID_ETABLISSEMENT LIMIT 1)', 'ID_RUE AS ID_RUE_SITE')
+                    ->joinLeft(['adressecommunesite' => 'adressecommune'], 'etablissementadressesite.NUMINSEE_COMMUNE = adressecommunesite.NUMINSEE_COMMUNE', 'LIBELLE_COMMUNE AS LIBELLE_COMMUNE_ADRESSE_SITE')
+                    ->joinLeft(['etablissementadressecell' => 'etablissementadresse'], 'etablissementadressecell.ID_ETABLISSEMENT = (SELECT ID_ETABLISSEMENT FROM etablissementlie WHERE ID_FILS_ETABLISSEMENT = e.ID_ETABLISSEMENT LIMIT 1)', 'ID_RUE AS ID_RUE_CELL')
+                    ->joinLeft(['adressecommunecell' => 'adressecommune'], 'etablissementadressecell.NUMINSEE_COMMUNE = adressecommunecell.NUMINSEE_COMMUNE', 'LIBELLE_COMMUNE AS LIBELLE_COMMUNE_ADRESSE_CELLULE')
+                ;
+            } else {
+                $select->joinLeft(['etablissementadressesite' => 'etablissementadresseapi'], 'etablissementadressesite.ID_ETABLISSEMENT = (SELECT ID_FILS_ETABLISSEMENT FROM etablissementlie WHERE ID_ETABLISSEMENT = e.ID_ETABLISSEMENT LIMIT 1)', [])
+                    ->columns(['LIBELLE_COMMUNE_ADRESSE_SITE' => 'etablissementadressesite.LIBELLE_COMMUNE'])
+                    ->joinLeft(['etablissementadressecell' => 'etablissementadresseapi'], 'etablissementadressecell.ID_ETABLISSEMENT = (SELECT ID_ETABLISSEMENT FROM etablissementlie WHERE ID_FILS_ETABLISSEMENT = e.ID_ETABLISSEMENT LIMIT 1)', [])
+                    ->columns(['LIBELLE_COMMUNE_ADRESSE_CELLULE' => 'etablissementadressecell.LIBELLE_COMMUNE'])
+                ;
+            }


Voir pour regrouper les conditions, éventuellement déporter dans une méthode privée pour simplifier la lecture

A709197 · 2025-01-20T08:23:36Z

application/views/scripts/etablissement/edit.phtml

@@ -1,3 +1,4 @@
+<link rel="stylesheet" href="https://unpkg.com/leaflet@1.7.1/dist/leaflet.css" crossorigin=""/>


Besoin de ça ?

A709197 · 2025-01-20T08:24:08Z

application/views/scripts/etablissement/edit.phtml


+<?= $this->partial('etablissement/partials/partial-modal-edit-adresse.phtml', ['key_ign' => $this->key_ign]) ?>
+<script src="/js/etablissement/edit/geolocaliseIGN.js"></script>


Déjà ajouté dans le controller

A709197 · 2025-01-20T08:24:17Z

application/views/scripts/etablissement/edit.phtml


+<?= $this->partial('etablissement/partials/partial-modal-edit-adresse.phtml', ['key_ign' => $this->key_ign]) ?>
+<script src="/js/etablissement/edit/geolocaliseIGN.js"></script>
+<script src="https://unpkg.com/leaflet@1.7.1/dist/leaflet.js" crossorigin=""></script>


Idem, besoin de ça ?

A709197 · 2025-01-20T08:26:30Z

application/views/scripts/etablissement/partials/partial-modal-edit-adresse.phtml

@@ -1,4 +1,4 @@
-<form id="adresse-modal-edit" class="modal hide adresse fade adresse-modal-edit" method='post'>
+<form id="adresse-modal-edit" class="modal hide adresse fade adresse-modal-edit" method="post">


Peut-être faire des sous-templates

A709197 · 2025-01-20T08:28:51Z

sql/migrations/2.5-60-ajout-colonne-libelle_rue.sql

@@ -0,0 +1,4 @@
+SET NAMES 'utf8';


Mettre ça dans le même SQL que le reste

NazihAsmaa and others added 7 commits October 21, 2024 12:42

modification pour api adresse pas terminée

3c7a615

debug Api

aef8ab9

Corrections sur l'appel à l'API adresse

be1cdc1

utilisation api adresse

f095bf5

merge develop

39e1204

cleanup

28ff979

Intégration API Adresse (Ajout _ Modification _ Recherche _ Migration…

7a3ca1a

…-des-adresses)

NazihAsmaa requested a review from A709197 January 14, 2025 15:44

Merge remote-tracking branch 'origin/develop' into feature/utilisatio…

3a0bd55

…n-api-adresse

NazihAsmaa self-assigned this Jan 14, 2025

NazihAsmaa added etablissement Ce qui concerne les établissements enhancement New feature or request labels Jan 14, 2025

NazihAsmaa changed the title ~~Feature/Intégration API Adresse~~ feat: intégration de l'api adresse Jan 14, 2025

lint

c3a1545

A709197 requested changes Jan 20, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: intégration de l'api adresse #212

feat: intégration de l'api adresse #212

NazihAsmaa commented Jan 14, 2025 •

edited by A709197

Loading

A709197 commented Jan 20, 2025

A709197 Jan 20, 2025

A709197 Jan 20, 2025

A709197 Jan 20, 2025

A709197 Jan 20, 2025

A709197 Jan 20, 2025

A709197 Jan 20, 2025

A709197 Jan 20, 2025

A709197 Jan 20, 2025

A709197 Jan 20, 2025

A709197 Jan 20, 2025

		@@ -1,5 +1,6 @@
		<?php

		require_once __DIR__.'/EtablissementManager.php';

		@@ -1,3 +1,4 @@
		<link rel="stylesheet" href="https://unpkg.com/leaflet@1.7.1/dist/leaflet.css" crossorigin=""/>


		<?= $this->partial('etablissement/partials/partial-modal-edit-adresse.phtml', ['key_ign' => $this->key_ign]) ?>
		<script src="/js/etablissement/edit/geolocaliseIGN.js"></script>

		@@ -1,4 +1,4 @@
		<form id="adresse-modal-edit" class="modal hide adresse fade adresse-modal-edit" method='post'>
		<form id="adresse-modal-edit" class="modal hide adresse fade adresse-modal-edit" method="post">

feat: intégration de l'api adresse #212

Are you sure you want to change the base?

feat: intégration de l'api adresse #212

Conversation

NazihAsmaa commented Jan 14, 2025 • edited by A709197 Loading

A709197 commented Jan 20, 2025

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

NazihAsmaa commented Jan 14, 2025 •

edited by A709197

Loading