Skip to content

Conversation

@ThT12
Copy link
Contributor

@ThT12 ThT12 commented Oct 29, 2025

fix #539

@ThT12
Copy link
Contributor Author

ThT12 commented Oct 29, 2025

Correction pour le moment faite uniquement dans "parcelle_info".
Mais j'ai pu voir que la concaténation de dlign4, dlign5 et dlign6 se fait à plein d'endroit, et à chaque fois sans espace entre 4 et 5.
Cela semblait donc plus une volonté qu'une erreur.

Merci de me dire si c'est bien un bug ou non. Si oui, j'essaierais de régler le soucis partout où la concaténation est faite.

@ThT12 ThT12 marked this pull request as draft October 29, 2025 07:48
@Jersig
Copy link

Jersig commented Oct 29, 2025

il s'agit bien d'un bug présent sur nombreuses parcelles. Plus largement, il manque également le champ parcelle.dindic afin d'avoir le "suffixe" du numéro dans la concaténation qui crée l'adresse

@ThT12
Copy link
Contributor Author

ThT12 commented Oct 30, 2025

il s'agit bien d'un bug présent sur nombreuses parcelles. Plus largement, il manque également le champ parcelle.dindic afin d'avoir le "suffixe" du numéro dans la concaténation qui crée l'adresse

Merci pour votre retour,
A ma connaissance le suffix du numéro (si on parle bien des bis/ter/...) est bien présent sur les adresses des propriétaires (mais collé au nom de rue, soucis déjà présent dans le fichier MAJIC et donc difficile à régler), mais n'est peut-être pas présent sur l'adresse des parcelles.
Exemple :
Adresse du propriétaire :
image
Adresse de la parcelle :
image

Je vais donc propager la correction de l'espace lié à ce ticket sur l'ensemble des concaténation de dlign 4, 5 et 6 dans cette PR. Pour le suffix, il faudra mieux faire la correction dans un ticket dédié (ou peut être sur votre PR qui corrige déjà l'adresse des parcelles ?).

@ThT12 ThT12 marked this pull request as ready for review October 30, 2025 07:51
@rldhont rldhont requested a review from mdouchin November 3, 2025 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Il manque un espace entre dlign4 et dlign5 dans "proprietaire_info" et "détails"

2 participants