Modify

Opened 9 months ago

Last modified 9 months ago

#23820 needinfo defect

Cannot delete empty relation

Reported by: jfd553 Owned by: jfd553
Priority: normal Milestone:
Component: Core Version:
Keywords: template_report Cc:

Description

What steps will reproduce the issue?

  1. I am not sure about the initial state. Probably deleting all relation components without editing the relation (opening the relation’s editing window).
  2. Run the validator, which reports empty relations.
  3. Try to delete the empty relation.

What is the expected result?

I used to be able to delete empty relations.

What happens instead?

A warning stating “The operation was not performed, according to the delete_outside_nodes preference”.

So, I can't delete the empty relation, since its components are deleted. I don’t want to send back an empty relation to OSM, but closing JOSM without saving changes, I will lose hundreds of edits I made.

Please provide any additional information below. Attach a screenshot if possible.

IFAIK, the empty relations shown by the Validator come from a wooded island inside an intermittent water body that I deleted.

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2024-07-02 17:10:50 +0200 (Tue, 02 Jul 2024)
Revision:19128
Build-Date:2024-07-03 01:31:15
URL:https://josm.openstreetmap.de/svn/trunk

Identification: JOSM/1.5 (19128 en) Windows 11 64-Bit
OS Build number: Windows 10 Pro 23H2 (22631)
Memory Usage: 980 MB / 8140 MB (209 MB allocated, but free)
Java version: 21.0.3+9-LTS, Azul Systems, Inc., OpenJDK 64-Bit Server VM
Look and Feel: com.sun.java.swing.plaf.windows.WindowsLookAndFeel
Screen: \Display0 1920x1080x32bpp@60Hz (scaling 1.00×1.00)
Maximum Screen Size: 1920×1080
Best cursor sizes: 16×16→32×32, 32×32→32×32
System property file.encoding: UTF-8
System property sun.jnu.encoding: Cp1252
Locale info: en_US
Numbers with default locale: 1234567890 -> 1234567890
VM arguments: [-Dicedtea-web.bin.location=%UserProfile%\AppData\Local\Programs\OpenWebStart\javaws, -Djava.util.Arrays.useLegacyMergeSort=true, --add-reads=java.base=ALL-UNNAMED,java.desktop, --add-reads=java.desktop=ALL-UNNAMED,java.naming, --add-reads=java.naming=ALL-UNNAMED,java.desktop, --add-exports=java.desktop/sun.awt=ALL-UNNAMED,java.desktop, --add-exports=java.desktop/javax.jnlp=ALL-UNNAMED,java.desktop, --add-exports=java.base/com.sun.net.ssl.internal.ssl=ALL-UNNAMED,java.desktop, --add-exports=java.base/sun.net.www.protocol.jar=ALL-UNNAMED,java.desktop, --add-exports=java.base/sun.security.action=ALL-UNNAMED,java.desktop, --add-exports=java.base/sun.security.provider=ALL-UNNAMED,java.desktop, --add-exports=java.base/sun.security.util=ALL-UNNAMED,java.desktop, --add-exports=java.base/sun.security.validator=ALL-UNNAMED,java.desktop, --add-exports=java.base/sun.security.x509=ALL-UNNAMED,java.desktop, --add-exports=java.base/jdk.internal.util.jar=ALL-UNNAMED,java.desktop, --add-exports=java.base/sun.net.www.protocol.http=ALL-UNNAMED,java.desktop, --add-exports=java.desktop/sun.awt.X11=ALL-UNNAMED,java.desktop, --add-exports=java.desktop/sun.applet=ALL-UNNAMED,java.desktop,jdk.jsobject, --add-exports=java.naming/com.sun.jndi.toolkit.url=ALL-UNNAMED,java.desktop, --add-modules=java.scripting,java.sql,javafx.controls,javafx.media,javafx.swing,javafx.web, --add-exports=java.desktop/com.apple.eawt=ALL-UNNAMED, --add-exports=java.desktop/com.sun.imageio.spi=ALL-UNNAMED, --add-exports=java.desktop/com.sun.imageio.plugins.jpeg=ALL-UNNAMED, --add-exports=javafx.graphics/com.sun.javafx.application=ALL-UNNAMED, --add-exports=jdk.deploy/com.sun.deploy.config=ALL-UNNAMED, --add-opens=java.base/java.lang=ALL-UNNAMED, --add-opens=java.base/java.nio=ALL-UNNAMED, --add-opens=java.base/jdk.internal.loader=ALL-UNNAMED, --add-opens=java.base/jdk.internal.ref=ALL-UNNAMED, --add-opens=java.desktop/javax.imageio.spi=ALL-UNNAMED, --add-opens=java.desktop/javax.swing.text.html=ALL-UNNAMED, --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED, -Djava.security.manager=allow]
Dataset consistency test: No problems found

Plugins:
+ ImportImagePlugin (36184)
+ PicLayer (1.0.3)
+ apache-commons (36273)
+ ejml (36176)
+ geotools (36273)
+ jackson (36273)
+ jaxb (36118)
+ jts (36004)
+ measurement (36256)
+ reverter (36256)
+ tageditor (36258)
+ turnlanes-tagging (0.0.5)
+ turnrestrictions (36226)
+ utilsplugin2 (36241)

Last errors/warnings:
- 00000.285 W: extended font config - overriding 'filename.Mongolian_Baiti=monbaiti.ttf' with 'MONBAITI.TTF'
- 00002.621 W: Unable to request certificate of https://roottest-g3.pkioverheid.nl
- 00003.061 W: Unable to request certificate of https://roottest-g3.pkioverheid.nl
- 00008.886 E: unable to find dependency jaxb for plugin geotools
- 00556.347 W: Conflict in data - <html>This relation has been changed outside of the editor.<br>You cannot apply your changes and continue editing.<br><br>Do you want to create a conflict and close the editor?</html>
- 03099.574 W: Conflict in data - <html>This relation has been changed outside of the editor.<br>You cannot apply your changes and continue editing.<br><br>Do you want to create a conflict and close the editor?</html>
- 08691.078 W: Conflicts detected - <html>There was 1 conflict detected.</html>
- 08694.539 W: Deleted or moved objects - <html>There are 2,096 objects in your local dataset which might be deleted on the server.<br>If you later try to delete or update them the server is likely to report a conflict.<br>Click <strong>Check on the server</strong> to check the state of these objects on the server.<br>Click <strong>Ignore</strong> to ignore.</html>
- 12712.142 W: Unsaved changes - <html>The relation has been changed.<br><br>Do you want to save your changes?</html>
- 12714.699 W: Unsaved changes - <html>The relation has been changed.<br><br>Do you want to save your changes?</html>

Attachments (1)

EmptyRelation.png (2.8 MB ) - added by jfd553 9 months ago.
Print screen of JOSM when getting the error/warning message.

Change History (5)

by jfd553, 9 months ago

Attachment: EmptyRelation.png added

Print screen of JOSM when getting the error/warning message.

comment:1 by taylor.smock, 9 months ago

Owner: changed from team to jfd553
Status: newneedinfo

I haven't been able to reproduce.

Can you save and upload? And give us the id of the relation you were trying to delete?

comment:2 by jfd553, 9 months ago

Saved and upload done

Changeset: 154328515

Empty relations I can't delete:
12595883 - natural=water
12595884 - natural=wood

comment:3 by GerdP, 9 months ago

I can reproduce the error when I save a downloaded relation to file and open that file. If I then try to delete the relation I get the same message. Obviously the error message is misleading.

comment:4 by jfd553, 9 months ago

Thanks

Modify Ticket

Change Properties
Set your email in Preferences
Action
as needinfo The owner will remain jfd553.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from jfd553 to the specified user. Next status will be 'new'.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from jfd553 to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.