Changes between Initial Version and Version 1 of Ticket #6982


Ignore:
Timestamp:
2011-10-20T12:35:59+02:00 (14 years ago)
Author:
Don-vip
Comment:

Thanks for your report. I've updated the description to add the raised exception. I look into a fix.

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #6982

    • Property Owner changed from team to Don-vip
    • Property Status new → assigned

  • Ticket #6982 – Description

    initial v1  
    1 Steps to reproduce: In preferences, add a TMS layer. Instead of {zoom} you write {z} - a mistake made easily, when you, for example, copied the url from a web page where it was with "$z". Then enable the layer. Result: JOSM complains about unhandled exceptions several times in a row - and later every time the view changes.
     1Steps to reproduce: In preferences, add a TMS layer. Instead of {zoom} you write {z} - a mistake made easily, when you, for example, copied the url from a web page where it was with "$z". Then enable the layer. Result: JOSM complains about unhandled exceptions several times in a row - and later every time the view changes:
     2
     3{{{
     4java.lang.IllegalArgumentException: can't parse argument number z
     5        at java.text.MessageFormat.makeFormat(MessageFormat.java:1339)
     6        at java.text.MessageFormat.applyPattern(MessageFormat.java:458)
     7        at java.text.MessageFormat.<init>(MessageFormat.java:350)
     8        at java.text.MessageFormat.format(MessageFormat.java:811)
     9        at org.openstreetmap.josm.tools.I18n.tr(I18n.java:151)
     10        at org.openstreetmap.josm.gui.layer.TMSLayer.paintTileText(TMSLayer.java:847)
     11        at org.openstreetmap.josm.gui.layer.TMSLayer.paint(TMSLayer.java:1212)
     12        at org.openstreetmap.josm.gui.MapView.paintLayer(MapView.java:451)
     13        at org.openstreetmap.josm.gui.MapView.paint(MapView.java:530)
     14}}}
    215
    316What should happen: show an error tile, that the url couldn't be parsed. Or that there's no tile at ".../{zoom}/..."