openstreetmap/iD

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

Aberta

#9.625 aberto em 2 de mai. de 2023

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)JavaScript (1.449 forks)github user discovery
help wantedvalidation

Métricas do repositório

Stars
 (3.818 estrelas)
Métricas de merge de PR
 (Mesclagem média 29d 14h) (82 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Examine a lógica de validação para auto interseção de áreas no editor iD. Concentre se no código executado quando os nós são excluídos de uma área. Procure onde as verificações de interseção são realizadas e por que podem ser ignoradas.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75