Missing translation for "No tile at this zoom level"

[naoliv]

Even with the string properly translated in launchpad we still see an unstranslated No tile at this zoom level message:

I did try this without success (the message in English is still appearing):

src/org/openstreetmap/gui/jmapviewer/OsmTileLoader.java

@@ -1 +1 @@
 // License: GPL. For details, see Readme.txt file.
 package org.openstreetmap.gui.jmapviewer;
 
+import static org.openstreetmap.gui.jmapviewer.FeatureAdapter.tr;
+
 import java.io.IOException;
 import java.io.InputStream;
 import java.net.HttpURLConnection;
@@ -49 +51 @@
                 }
                 loadTileMetadata(tile, conn);
                 if ("no-tile".equals(tile.getValue("tile-info"))) {
-                    tile.setError("No tile at this zoom level");
+                    tile.setError(tr("No tile at this zoom level"));
                 } else {
                     input = conn.getInputStream();
                     try {

src/org/openstreetmap/josm/data/imagery/TMSCachedTileLoaderJob.java

@@ -278 +278 @@
     private boolean handleNoTileAtZoom() {
         if (isNoTileAtZoom()) {
             Logging.debug("JCS TMS - Tile valid, but no file, as no tiles at this level {0}", tile);
-            tile.setError("No tile at this zoom level");
+            tile.setError(tr("No tile at this zoom level"));
             tile.putValue("tile-info", "no-tile");
             return true;
         }

JOSM:

Revision:14569
Is-Local-Build:true
Build-Date:2018-12-17 00:38:03

Identification: JOSM/1.5 (14569 SVN pt_BR) Linux Debian GNU/Linux buster/sid
Memory Usage: 392 MB / 2048 MB (194 MB allocated, but free)
Java version: 11.0.1+13-Debian-3, Oracle Corporation, OpenJDK 64-Bit Server VM
Screen: :0.0 1920x1080
Maximum Screen Size: 1920x1080
Java package: openjdk-11-jre:amd64-11.0.1+13-3
Java ATK Wrapper package: libatk-wrapper-java:all-0.33.3-21
VM arguments: [-Dawt.useSystemAAFontSettings=gasp]

[Don-vip]

In 14570/josm:

fix #17116 - translate "No tile at this zoom level"

[Don-vip]

Note: it should probably work only after the next i18n update.

[naoliv]

Ah... it seems that I didn't pay attention and didn't realize that the translation that already exists has tiles instead of tile

That's why I wasn't seeing the translation :-)
Thanks!

[Don-vip]

Ah. We should only keep one string.

[Don-vip]

In 14573/josm:

see #17116 - use existing translated string

[stoecker]

"No tiles" for a single tile would be wrong. I'd say this is one of the few cases, where similar strings are ok...