Opened 5 years ago

Closed 4 years ago

#123 closed enhancement (fixed)

[analyser] tag parking

Reported by: David Crochet Owned by: jocelyn
Priority: major Component: osmose-backend
Keywords: Cc:

Description

Quelques idées de tests avec les tags de la famille "parking:lane" et consœur

1 - Vérifier que dans un chemin : si un paramètre débute par "parking:lane:" alors tester "parking:lane:side", et si "side" est différent de "left|right|both" alors dire "erreur"

2 - vérifier que dans un chemin : si "parking:lane:left" existe et si "parking:lane:right" n'existe pas alors dire "erreur"

3 - vérifier que dans un chemin : si "parking:lane:right" existe et si "parking:lane:left" n'existe pas alors dire "erreur"

4 - vérifier que dans un chemin : (si "parking:lane:right" existe ou si "parking:lane:left" existe ) et si "parking:lane:both" existe alors dire "erreur"

5 - Vérifier que dans un chemin "parking:lane:*=type", si "type" égale "parallel|diagonal|perpendicular|marked|no_parking|no_stopping|fire_lane", sinon dire "erreur"

6 - Vérifier que dans un chemin si "parking:condition:*" existe et si "parking:lane:*" n'existe pas, alors dire "erreur"

Voila.

Cela pourrait être l'item n°3170

Change History (9)

comment:1 Changed 5 years ago by frodrigo

  • Type changed from defect to enhancement

comment:2 follow-up: Changed 5 years ago by windu.2b

Les points 2 et 3 me surprennent : rien, dans la document indiquent que les 2 tags doivent être présents ensemble. Il se peut en effet que des places de parking ne se trouvent que d'un côté ou de l'autre de la voie : ne pas mettre le tag équivaudrait alors à la valeur par défaut (à savoir : pas de place de parkings).

comment:3 in reply to: ↑ 2 Changed 5 years ago by David Crochet

Replying to windu.2b:

Les points 2 et 3 me surprennent : rien, dans la document indiquent que les 2 tags doivent être présents ensemble.

Il existe une valeur "both" qui indique la même situation des deux côtés

Par contre si "left" existe, il est tout à fait logique que "right" existe

JOSM demande la situation commune ou alors de chacun des côtés dans la même boites de dialogue (donc si un contributeur oublie de le rempli, il l'a fait sciemment)

POTLACH demande à rentre les paramètres étiquette par étiquette, donc là le contributeur peut oublier ou même se tromper dans la syntaxe, d'où la requête de "juste vérification"

Ces deux points (2 et 3) indique peut-être un oubli qu'il faudrait alors dans ce cas combler.

Replying to windu.2b:

Il se peut en effet que des places de parking ne se trouvent que d'un côté ou de l'autre de la voie : ne pas mettre le tag équivaudrait alors à la valeur par défaut (à savoir : pas de place de parkings).

Certes, mais j'étais partie sur l'idée de contrôle de termes et de cohérence de l'utilisation de la balise sur un chemin

comment:4 follow-up: Changed 5 years ago by windu.2b

Qu'elle que soit la solution retenue, il faut en tout cas que le wiki et les tests soient cohérents : ne pas préciser d'un coté que les 2 tags sont nécessaires, tout en écrivant un test qui vérifie la présence des 2 est une garantie d'erreurs.

Personnellement, je reste tout de même sur ma position consistant à dire que les 2 tags ensemble ne devraient pas être obligatoires : il existe une valeur par défaut, disant qu'il n'y a pas de places de parkings. Ajouter un tag pour dire explicitement la même chose reviendrait donc à surcharger inutilement la base de données.

comment:5 in reply to: ↑ 4 Changed 5 years ago by David Crochet

Dans ce cas là, je ne vois aucune objection à ne pas mettre les points 2 et 3 dans la procédure.

comment:6 Changed 5 years ago by windu.2b

Si le wiki évolue en conséquence, pour que l'information de l'obligation de la présence des 2 tags ensemble apparaisse clairement, OK.

comment:7 Changed 5 years ago by frodrigo

  • Summary changed from Nouveau type de test : tag parking to [analyser] tag parking

comment:8 Changed 5 years ago by frodrigo

  • Owner changed from frodrigo to jocelyn
  • Status changed from new to assigned

comment:9 Changed 4 years ago by frodrigo

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.