Opened 2 years ago

Closed 2 years ago

#15945 closed defect (wontfix)

Crash upon $z

Reported by: jidanni Owned by: team
Priority: normal Milestone:
Component: Core imagery Version:
Keywords: tms Cc:

Description (last modified by Don-vip)

I was able to crash JOSM.
All it took was to ignore $z warnings upon entering the site.
Then trying to select the site from the Imagery menu.

Yes in the future I will correct my error. I just thought you might want to catch the error better instead of crashing.

Build-Date:2018-01-29 06:55:09

Identification: JOSM/1.5 (13367 Debian en) Linux Debian GNU/Linux buster/sid
Memory Usage: 207 MB / 224 MB (95 MB allocated, but free)
Java version: 1.8.0_151-8u151-b12-1-b12, Oracle Corporation, OpenJDK 64-Bit Server VM
Screen: :0.0 1024x768
Maximum Screen Size: 1024x768
Java package: openjdk-8-jre:amd64-8u151-b12-1
VM arguments: [-Djosm.restart=true,]

Last errors/warnings:
- W: No configuration settings found.  Using hardcoded default values for all pools.
- W: Region [WMS_BLOCK_v2] Resetting cache
- W: Region [WMTS_BLOCK_v2] Resetting cache
- W: java.lang.IllegalArgumentException: {$z} is not a valid TMS argument. Please check this server URL:
- W: java.lang.IllegalArgumentException: {$z} is not a valid TMS argument. Please check this server URL:
- E: Handled by bug report queue: java.lang.IllegalArgumentException: {$z} is not a valid TMS argument. Please check this server URL:

 - layer: TMSLayer [info=ImageryInfo{name='djt', countryCode='', url='{$z}/{$y}/{$x}', imageryType=TMS}]

No data collected.

Warning issued by: BugReportExceptionHandler#handleException

Thread: AWT-EventQueue-1 (17) of main
java.lang.IllegalArgumentException: {$z} is not a valid TMS argument. Please check this server URL:{$z}/{$y}/{$x}
	at org.openstreetmap.gui.jmapviewer.tilesources.TemplatedTMSTileSource.checkUrl(
	at org.openstreetmap.josm.gui.layer.TMSLayer.getTileSourceStatic(
	at org.openstreetmap.josm.gui.layer.TMSLayer.getTileSource(
	at org.openstreetmap.josm.gui.layer.TMSLayer.getTileSource(
	at org.openstreetmap.josm.gui.layer.AbstractTileSourceLayer.initializeIfRequired(
	at org.openstreetmap.josm.gui.layer.AbstractTileSourceLayer.attachToMapView(
	at org.openstreetmap.josm.gui.MapView.layerAdded(
	at org.openstreetmap.josm.gui.layer.LayerManager.fireLayerAdded(
	at org.openstreetmap.josm.gui.layer.LayerManager.realAddLayer(
	at org.openstreetmap.josm.gui.layer.MainLayerManager.realAddLayer(
	at org.openstreetmap.josm.gui.layer.LayerManager.lambda$addLayer$0(
	at org.openstreetmap.josm.gui.util.GuiHelper.runInEDTAndWaitWithException(
	at org.openstreetmap.josm.gui.layer.LayerManager.addLayer(
	at org.openstreetmap.josm.gui.layer.LayerManager.addLayer(
	at org.openstreetmap.josm.actions.AddImageryLayerAction.actionPerformed(
	at javax.swing.AbstractButton.fireActionPerformed(
	at javax.swing.AbstractButton$Handler.actionPerformed(
	at javax.swing.DefaultButtonModel.fireActionPerformed(
	at javax.swing.DefaultButtonModel.setPressed(
	at javax.swing.AbstractButton.doClick(
	at javax.swing.plaf.basic.BasicMenuItemUI.doClick(
	at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(
	at java.awt.AWTEventMulticaster.mouseReleased(
	at java.awt.Component.processMouseEvent(
	at javax.swing.JComponent.processMouseEvent(
	at java.awt.Component.processEvent(
	at java.awt.Container.processEvent(
	at java.awt.Component.dispatchEventImpl(
	at java.awt.Container.dispatchEventImpl(
	at java.awt.Component.dispatchEvent(
	at java.awt.LightweightDispatcher.retargetMouseEvent(
	at java.awt.LightweightDispatcher.processMouseEvent(
	at java.awt.LightweightDispatcher.dispatchEvent(
	at java.awt.Container.dispatchEventImpl(
	at java.awt.Window.dispatchEventImpl(
	at java.awt.Component.dispatchEvent(
	at java.awt.EventQueue.dispatchEventImpl(
	at java.awt.EventQueue.access$500(
	at java.awt.EventQueue$
	at java.awt.EventQueue$
	at Method)
	at java.awt.EventQueue$
	at java.awt.EventQueue$
	at Method)
	at java.awt.EventQueue.dispatchEvent(
	at org.GNOME.Accessibility.AtkWrapper$5.dispatchEvent(
	at java.awt.EventDispatchThread.pumpOneEventForFilters(
	at java.awt.EventDispatchThread.pumpEventsForFilter(
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(
	at java.awt.EventDispatchThread.pumpEvents(
	at java.awt.EventDispatchThread.pumpEvents(

Attachments (1)

crash (5.7 KB) - added by jidanni 2 years ago.

Download all attachments as: .zip

Change History (5)

Changed 2 years ago by jidanni

Attachment: crash added

comment:1 Changed 2 years ago by Klumbumbus

Component: CoreCore imagery

comment:2 Changed 2 years ago by Don-vip

Description: modified (diff)

comment:3 Changed 2 years ago by Don-vip

Keywords: tms added
Milestone: 18.02

comment:4 Changed 2 years ago by Don-vip

Milestone: 18.02
Resolution: wontfix
Status: newclosed

Not so easy to make the distinction between real bugs and invalid URLs. Simply don't ignore warnings next time.

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.