Exception on upload after revert

[ratrun]

What steps will reproduce the problem?

Revert changeset 189470987
Resolve 5 conflicts. It does not seem to matter how
Upload the change
This problem is reproducible.
I also saved the *.osc file, reloaded it and tried to upload with the same result.

What is the expected result?

Successful upload

What happens instead?

JOSM exception detection, error reporting dialog pops up

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

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2026-08-11 15:36:21 +0200 (Tue, 11 Aug 2026)
Revision:19613
Build-Date:2026-08-12 01:30:51
URL:https://josm.openstreetmap.de/svn/trunk

Identification: JOSM/1.5 (19613 de) Linux Mint 22.3
Memory Usage: 544 MB / 94880 MB (289 MB allocated, but free)
Java version: 25.0.4.1+1-1-24.04.4-Ubuntu, Ubuntu, OpenJDK 64-Bit Server VM
Look and Feel: javax.swing.plaf.metal.MetalLookAndFeel
Screen: :0.0 1920x1200x[Multi depth]@60Hz (scaling 1.00×1.00)
Maximum Screen Size: 1920×1200
Best cursor sizes: 16×16→16×16, 32×32→32×32
Environment variable LANG: de_AT.UTF-8
System property file.encoding: UTF-8
System property sun.jnu.encoding: UTF-8
Locale info: de_AT
Numbers with default locale: 1234567890 -> 1234567890
Desktop environment: X-Cinnamon
Java package: openjdk-25-jre:amd64-25.0.4.1+1-1~24.04.4
libcommons-compress-java: libcommons-compress-java:all-1.25.0-1
libcommons-logging-java: libcommons-logging-java:all-1.3.0-1ubuntu1
fonts-noto: fonts-noto:all-20201225-2
VM arguments: [-Duser.language=en, -Djosm.restart=true, -Djava.net.useSystemProxies=true, -XX:MaxRAMPercentage=75.0]
Dataset consistency test: No problems found

Plugins:
+ Mapillary (1822)
+ apache-commons (36483)
+ pbf (36493)
+ reverter (36503)

Last errors/warnings:
- 00016.287 E: Handled by bug report queue: java.lang.ArrayIndexOutOfBoundsException: Coordinate out of bounds!
- 00016.291 E: Handled by bug report queue: java.lang.ArrayIndexOutOfBoundsException: Coordinate out of bounds!
- 00016.295 E: Handled by bug report queue: java.lang.ArrayIndexOutOfBoundsException: Coordinate out of bounds!
- 00581.863 W: Konflikte gefunden - <html>5 Konflikte wurden entdeckt.</html>
- 00608.227 W: Achtung - <html>Daten, die hochgeladen werden sollen, sind an Konflikten in der Ebene 'Zurückgesetzter Änderungssatz [Kennung: 189470987]' beteiligt.<br>Sie müssen diese zunächst auflösen.</html>
- 00663.631 E: Handled by bug report queue: java.lang.ArrayIndexOutOfBoundsException: Coordinate out of bounds!



=== REPORTED CRASH DATA ===
BugReportExceptionHandler#handleException:
No data collected.

Warning issued by: BugReportExceptionHandler#handleException

=== STACK TRACE ===
Thread: AWT-EventQueue-0 (42) of main
java.lang.ArrayIndexOutOfBoundsException: Coordinate out of bounds!
	at java.desktop/java.awt.image.SinglePixelPackedSampleModel.getPixel(SinglePixelPackedSampleModel.java:431)
	at java.desktop/java.awt.image.Raster.getPixel(Raster.java:1721)
	at com.github.weisj.jsvg.nodes.filter.ImageProducerChannel.pixelAt(ImageProducerChannel.java:74)
	at com.github.weisj.jsvg.nodes.filter.FeDisplacementMap$DisplacementOp.filter(FeDisplacementMap.java:185)
	at java.desktop/java.awt.image.BufferedImageFilter.imageComplete(BufferedImageFilter.java:375)
	at java.desktop/sun.awt.image.OffScreenImageSource.produce(OffScreenImageSource.java:188)
	at java.desktop/sun.awt.image.OffScreenImageSource.addConsumer(OffScreenImageSource.java:66)
	at java.desktop/sun.awt.image.OffScreenImageSource.startProduction(OffScreenImageSource.java:80)
	at java.desktop/java.awt.image.FilteredImageSource.startProduction(FilteredImageSource.java:184)
	at java.desktop/sun.awt.image.ImageRepresentation.startProduction(ImageRepresentation.java:702)
	at java.desktop/sun.awt.image.ToolkitImage.addWatcher(ToolkitImage.java:194)
	at java.desktop/sun.awt.image.ToolkitImage.getWidth(ToolkitImage.java:101)
	at java.desktop/sun.java2d.SunGraphics2D.drawImage(SunGraphics2D.java:3426)
	at java.desktop/sun.java2d.SunGraphics2D.drawImage(SunGraphics2D.java:3566)
	at com.github.weisj.jsvg.nodes.filter.AbstractCompositeFilterPrimitive.applyFilter(AbstractCompositeFilterPrimitive.java:74)
	at com.github.weisj.jsvg.nodes.filter.FeComposite.applyFilter(FeComposite.java:35)
	at com.github.weisj.jsvg.nodes.filter.Filter.applyFilter(Filter.java:184)
	at com.github.weisj.jsvg.renderer.impl.Info$InfoWithIsolation.close(Info.java:156)
	at com.github.weisj.jsvg.renderer.impl.NodeRenderer.renderNode(NodeRenderer.java:66)
	at com.github.weisj.jsvg.renderer.impl.NodeRenderer.renderNode(NodeRenderer.java:59)
	at com.github.weisj.jsvg.nodes.container.CommonRenderableContainerNode.render(CommonRenderableContainerNode.java:81)
	at com.github.weisj.jsvg.nodes.container.BaseInnerViewContainer.renderWithEstablishedViewBox(BaseInnerViewContainer.java:75)
	at com.github.weisj.jsvg.renderer.impl.NodeRenderer.renderRootSVG(NodeRenderer.java:54)
	at com.github.weisj.jsvg.SVGDocument.renderWithPlatform(SVGDocument.java:156)
	at com.github.weisj.jsvg.SVGDocument.renderWithPlatform(SVGDocument.java:123)
	at com.github.weisj.jsvg.SVGDocument.renderWithPlatform(SVGDocument.java:117)
	at com.github.weisj.jsvg.SVGDocument.render(SVGDocument.java:107)
	at com.github.weisj.jsvg.SVGDocument.render(SVGDocument.java:100)
	at org.openstreetmap.josm.tools.ImageProvider.lambda$createImageFromSvg$3(ImageProvider.java:1480)
	at org.openstreetmap.josm.tools.ImageResizeMode.createBufferedImage(ImageResizeMode.java:95)
	at org.openstreetmap.josm.tools.ImageProvider.createImageFromSvg(ImageProvider.java:1479)
	at org.openstreetmap.josm.tools.ImageResource.getImageIconAlreadyScaled(ImageResource.java:194)
	at org.openstreetmap.josm.tools.ImageResource.getImageIcon(ImageResource.java:165)
	at org.openstreetmap.josm.tools.ImageResource.getPaddedIcon(ImageResource.java:261)
	at org.openstreetmap.josm.tools.ImageProvider.getPadded(ImageProvider.java:1458)
	at org.openstreetmap.josm.gui.PrimitiveRenderer.renderer(PrimitiveRenderer.java:82)
	at org.openstreetmap.josm.gui.PrimitiveRenderer.getListCellRendererComponent(PrimitiveRenderer.java:52)
	at org.openstreetmap.josm.gui.PrimitiveRenderer.getListCellRendererComponent(PrimitiveRenderer.java:31)
	at java.desktop/javax.swing.plaf.basic.BasicListUI.updateLayoutState(BasicListUI.java:1444)
	at java.desktop/javax.swing.plaf.basic.BasicListUI.maybeUpdateLayoutState(BasicListUI.java:1394)
	at java.desktop/javax.swing.plaf.basic.BasicListUI.getPreferredSize(BasicListUI.java:653)
	at java.desktop/javax.swing.JComponent.getPreferredSize(JComponent.java:1734)
	at java.desktop/javax.swing.JList.getPreferredScrollableViewportSize(JList.java:2477)
	at java.desktop/javax.swing.ViewportLayout.preferredLayoutSize(ViewportLayout.java:100)
	at java.desktop/java.awt.Container.preferredSize(Container.java:1820)
	at java.desktop/java.awt.Container.getPreferredSize(Container.java:1804)
	at java.desktop/javax.swing.JComponent.getPreferredSize(JComponent.java:1736)
	at java.desktop/javax.swing.ScrollPaneLayout.preferredLayoutSize(ScrollPaneLayout.java:500)
	at java.desktop/java.awt.Container.preferredSize(Container.java:1820)
	at java.desktop/java.awt.Container.getPreferredSize(Container.java:1804)
	at java.desktop/javax.swing.JComponent.getPreferredSize(JComponent.java:1736)
	at java.desktop/java.awt.GridBagLayout.GetLayoutInfo(GridBagLayout.java:1104)
	at java.desktop/java.awt.GridBagLayout.getLayoutInfo(GridBagLayout.java:899)
	at java.desktop/java.awt.GridBagLayout.preferredLayoutSize(GridBagLayout.java:719)
	at java.desktop/java.awt.Container.preferredSize(Container.java:1820)
	at java.desktop/java.awt.Container.getPreferredSize(Container.java:1804)
	at java.desktop/javax.swing.JComponent.getPreferredSize(JComponent.java:1736)
	at java.desktop/javax.swing.plaf.basic.BasicSplitPaneUI$BasicHorizontalLayoutManager.getPreferredSizeOfComponent(BasicSplitPaneUI.java:1734)
	at java.desktop/javax.swing.plaf.basic.BasicSplitPaneUI$BasicHorizontalLayoutManager.getPreferredSizes(BasicSplitPaneUI.java:1987)
	at java.desktop/javax.swing.plaf.basic.BasicSplitPaneUI$BasicHorizontalLayoutManager.resetToPreferredSizes(BasicSplitPaneUI.java:2023)
	at java.desktop/javax.swing.plaf.basic.BasicSplitPaneUI$BasicHorizontalLayoutManager.layoutContainer(BasicSplitPaneUI.java:1424)
	at java.desktop/java.awt.Container.layout(Container.java:1537)
	at java.desktop/java.awt.Container.doLayout(Container.java:1526)
	at java.desktop/java.awt.Container.validateTree(Container.java:1719)
	at java.desktop/java.awt.Container.validateTree(Container.java:1728)
	at java.desktop/java.awt.Container.validateTree(Container.java:1728)
	at java.desktop/java.awt.Container.validateTree(Container.java:1728)
	at java.desktop/java.awt.Container.validateTree(Container.java:1728)
	at java.desktop/java.awt.Container.validate(Container.java:1654)
	at java.desktop/java.awt.Container.validateUnconditionally(Container.java:1691)
	at java.desktop/java.awt.Dialog.conditionalShow(Dialog.java:892)
	at java.desktop/java.awt.Dialog.show(Dialog.java:1029)
	at java.desktop/java.awt.Component.show(Component.java:1690)
	at java.desktop/java.awt.Component.setVisible(Component.java:1637)
	at java.desktop/java.awt.Window.setVisible(Window.java:982)
	at java.desktop/java.awt.Dialog.setVisible(Dialog.java:989)
	at org.openstreetmap.josm.gui.io.UploadDialog.setVisible(UploadDialog.java:371)
	at org.openstreetmap.josm.actions.UploadAction.realUploadData(UploadAction.java:304)
	at org.openstreetmap.josm.actions.UploadAction.lambda$uploadData$2(UploadAction.java:273)
	at java.desktop/java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:323)
	at java.desktop/java.awt.EventQueue.dispatchEventImpl(EventQueue.java:723)
	at java.desktop/java.awt.EventQueue.dispatchEvent(EventQueue.java:702)
	at java.desktop/java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:203)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:124)
	at java.desktop/java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:113)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:109)
	at java.desktop/java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:101)
	at java.desktop/java.awt.EventDispatchThread.run(EventDispatchThread.java:90)

[ratrun]

OSM file with changes

[ratrun]

Please reproduce the issue by loading the attached "teheran.osm" file and try to upload.

[skyper]

There is a shop=lighting in the data, see #24857. Please, try josm-latest, as I hope it is already fixed and that this icon is/was the only one causing trouble.

[ratrun]

Thank you. Somebody already performed the revert using ​https://revert.monicz.dev/, so I was not able to test with the latest version.

[skyper]

Closed as duplicate of #24857.
----
Ok, I took a closer look. The node with shop=lighting was modified. Only mapview does not produce the exception. The icon needs to be displayed in a dialog, like on the top of the Tags/Memberships panel or the listing of objects on the left side of the upload dialog.

With latest it works just fine with your file (I did not upload but no problem with the upload dialog). While with tested (r19613) I get the expected exceptions.

I am pretty confident that this is already fixed.