ign-packo / PackO

Outil pour le contrôle et la retouche du mosaïquage d'ortho
Other
6 stars 2 forks source link

Remarks: Modif route ajout/suppression de ponctuel #289

Closed ftoromanoff closed 2 years ago

ftoromanoff commented 2 years ago

Objectif : Limiter l'ajout (et suppression) de ponctuel sur les couches 'Remarques'

Modification :

coveralls commented 2 years ago

Pull Request Test Coverage Report for Build 2165905650


Files with Coverage Reduction New Missed Lines %
db/db.js 5 94.82%
middlewares/vector.js 28 84.42%
<!-- Total: 33 -->
Totals Coverage Status
Change from base Build 2165888923: 0.02%
Covered Lines: 5432
Relevant Lines: 5708

💛 - Coveralls
nlenglet-ign commented 2 years ago

Pour la route /{idBranch}/vector, je pense qu'il serait bien de renvoyer l'id de la couche vecteur en plus des informations déjà présentes. Ceci n'est pas bloquant pour cette PR et peut être traité séparément.

ftoromanoff commented 2 years ago

Il y a des problèmes dans la gestion de DELETE /{idRemarksVector}/feature : il ne prend pas en considération idRemarksVector que pour vérifier que le vecteur existe dans la base, on peut indiquer n'importe quel id vecteur existant dans la base (et donc pas forcément l'id du vecteur Remarques) et ensuite la suppression se fait en tenant compte que de l'id feature indiqué (il retrouve l'id de la couche Remarques associée) - il faudrait donc enlever idRemarksVector et laisser uniquement id feature ou gérer le contrôle sur l'id du vecteur

Et il faudrait corriger les messages qui contiennent encore "idLayer" qui est obsolète

Remarques corrigées