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

[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].