Skip to content

Correction de l'ordre des id des opis ref et sec pour l'insertion des patches - #458

Open
ACornuIGN wants to merge 1 commit into
ign-packo:2Opifrom
ACornuIGN:fix_order_opi
Open

Correction de l'ordre des id des opis ref et sec pour l'insertion des patches#458
ACornuIGN wants to merge 1 commit into
ign-packo:2Opifrom
ACornuIGN:fix_order_opi

Conversation

@ACornuIGN

Copy link
Copy Markdown
Collaborator

Le bug: Lors de l'insertion d'un patch semi-auto dans la base de données, si opi_ref à un id supérieur à opi_sec, les identifiants des opis sont inversées dans la table patches de la base de données, id_opi aura l'identifiant le plus petit et id_opisec le plus grand.

Ce bug est une regression ajoutée dans le commit 538291d ("refactor(API): breaking down applyPatch into smaller functions", 2026-06-12) qui est passé dans la MR #453 qui consistait à re-factoriser la fonction applyPatch en plusieurs petites fonctions, pour la préparation aux multi-patches.

Le problème est dû à la re-factorisation de la fonction getOPIFromNames qui prend une liste de noms d'opis et retourne chaque opi dans une table ordonnée dans l'ordre croissant de leur id et non dans l'ordre de position des noms dans l'array de la requête. Ce qui pouvais inverser la position des opis si opi_ref.id > opi_sec.id

Solution: ajout d'un order by array_position dans la requête pour que la table de sortie soit dans l'ordre de la liste des noms d'opis demandés en entrée.

@ACornuIGN
ACornuIGN requested a review from ftoromanoff August 6, 2026 13:08
@ACornuIGN ACornuIGN self-assigned this Aug 6, 2026
@ACornuIGN ACornuIGN added the bug Something isn't working label Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant