openstreetmap/iD

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

Offen

#9.625 geöffnet am 02.05.2023

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.449 Forks)github user discovery
help wantedvalidation

Repository-Metriken

Stars
 (3.818 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 29T 14h) (82 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die Validierungslogik für Selbstüberschneidungen von Bereichen im iD Editor. Konzentriere dich auf den Code, der ausgeführt wird, wenn Knoten aus einem Bereich gelöscht werden. Suche nach Stellen, an denen Schnittprüfungen durchgeführt werden und warum sie möglicherweise übersprungen werden.
Tech Stack
javascript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75