Retirer l'outillage fr_city car il n'est plus utilisé

[florimondmanca]

Sauf erreur, on a du code mort issu de la PR #569

J'imagine qu'on voulait s'en servir pour le géocodage, avant qu'on ne passe tout sur la BDTOPO

Ceci afin de pouvoir faire des recherches dans la liste de communes.

Mais la recherche de villes est faite avec l'API Adresse maintenant, via la PR #572

dialog/src/Infrastructure/Controller/Regulation/Fragments/GetCityCompletionFragmentController.php

Line 34 in 9f26953

$cities = $this->geocoder->findCities($search);

Je propose donc de retirer tout ça :

• data/fr_city.sql
• La table Country\France\City
• Le CityRepository
• Le mkfrcitydata dans le Makefile
• (Non exhaustif)

[florimondmanca]

Hmm OK en fait la seule référence restante à cette table est dans cette migration...

dialog/src/Infrastructure/Persistence/Doctrine/Migrations/Version20231206135155.php

Lines 17 to 51 in 9f26953

	public function up(Schema $schema): void
	{
	$this->addSql('ALTER TABLE location ALTER COLUMN address DROP NOT NULL;');
	$this->addSql('ALTER TABLE location ADD city_code VARCHAR(5)');
	$this->addSql('ALTER TABLE location ADD city_name VARCHAR(255)');
	$this->addSql('ALTER TABLE location ADD city_postcode VARCHAR(255)');
	$this->addSql('ALTER TABLE location ADD road_name VARCHAR(255)');
	$this->addSql("UPDATE location SET road_name = substring(address from '^([^\d,]+),? \d{5}');");
	$this->addSql("UPDATE location SET city_name = substring(address from '\d{5} (.+)\$');");
	$this->addSql("UPDATE location SET city_postcode = substring(address from '(\d{5}) .+\$');");
	$this->addSql("
	UPDATE location SET city_code = (
	CASE
	WHEN city_name = 'Paris' THEN '751' || substring(address from '750(\d{2}) .+\$')
	WHEN city_name = 'Marseille' THEN '132' || substring(address from '130(\d{2}) .+\$')
	WHEN city_name = 'Lyon' THEN '6938' || substring(address from '6900(\d) .+\$')
	ELSE (
	SELECT insee_code FROM fr_city AS c
	WHERE c.departement = substring(address from '(\d{2})\d{3} .+\$')
	AND lower(c.name) = lower(city_name)
	)
	END
	)
	");
	$this->addSql('ALTER TABLE location ADD city_label VARCHAR(255)');
	$this->addSql("UPDATE location SET city_label = city_name || ' (' || city_postcode || ')'");
	$this->addSql('ALTER TABLE location DROP city_name');
	$this->addSql('ALTER TABLE location DROP city_postcode');
	$this->addSql('ALTER TABLE location ALTER COLUMN city_code SET NOT NULL');
	$this->addSql('ALTER TABLE location ALTER COLUMN city_label SET NOT NULL');
	}

Elle date de #572

Si on retire tout ce qui concerne fr_city mais qu'on ne change pas cette migration, un clean install échouerait car cette migration ne passerait plus

Or depuis tous les environnements ont été migrés, et sur un clean install l'UPDATE ne ferait rien. Est-ce qu'il y a lieu de la garder ? On peut peut-être juste retirer ce addSql ? Qu'en dis tu @mmarchois ?