openstreetmap/iD

Deleting nodes from an area doesn't check to see if the area self-intersects

Ouverte

#9 625 ouverte le 2 mai 2023

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)JavaScript (1 449 forks)github user discovery
help wantedvalidation

Métriques du dépôt

Stars
 (3 818 étoiles)
Métriques de merge PR
 (Merge moyen 29j 14h) (82 PRs mergées en 30 j)

Description

URL

No response

How to reproduce the issue?

  1. Create an area with a sharp corner
  2. Delete nodes until the way crosses over itself
  3. Save and upload without a warning

Screenshot(s) or anything else?

Screen recording here: https://drive.google.com/file/d/1bocercK5ZcbfPDyv6Z0dt0mKYy2J2nxy/view?usp=sharing

Changeset shown in recording is https://osmcha.org/changesets/135619665/

Which deployed environments do you see the issue in?

Released version at openstreetmap.org/edit

What version numbers does this issue effect?

2.25.2

Which browsers are you seeing this problem on?

Chrome

Guide contributeur

Direction de recherche
Examinez la logique de validation d'auto intersection des zones dans l'éditeur iD. Concentrez vous sur le code exécuté lorsque des nœuds sont supprimés d'une zone. Cherchez où les vérifications d'intersection sont effectuées et pourquoi elles pourraient être ignorées.
Stack technique
javascript
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75