GPS data download not working in background with Mapdust active

[malenki]

How to reproduce:
Install the mapdust plugin.
Go to an area with a lot of gps data (you may try this: ​region).
Click the button to download the GPS data in background.
Zoom in or out.
Result: the gps data download dialog shows up again

[jBeata]

Replying to malenki:

How to reproduce:
Install the mapdust plugin.
Go to an area with a lot of gps data (you may try this: ​region).
Click the button to download the GPS data in background.
Zoom in or out.
Result: the gps data download dialog shows up again

I checked this issue and the GPS download works fine also if the Mapdust plugin is activated. You either wait until the GPS data is downloaded from OSM, or run in background the download. Downloading GPS might take some time.

[jBeata]

I check this issue, and it seams that the GPS "background" download process blocks all the other "download" processes. When you zoom/in out and the Mapdust plugin is activated, the data from your current view is downloading.

[malenki]

Just want to mention that the issue still is there.

[Don-vip]

MapDust code:

        Main.worker.execute(new Runnable() {
            @Override
            public void run() {
                updateMapdustData();
            }
        });

JOSM ProgressMonitorExecutor code:

    @Override
    public void execute(Runnable command) {
        if (Main.currentProgressMonitor != null) {
            //TODO show only if this can't be in background or better if always in background is not checked
            Main.currentProgressMonitor.showForegroundDialog();
        }
        super.execute(command);
    }

MapDust should probably use its own ThreadPoolExecutor, like Mapillary plugin for example.

[Don-vip]

Fixed in [o34214:34215].