[Patch needs review] Dragging map is considered moving objects

[Zverikk]

1. Select more that 20 objects on a map
2. Move them a bit (and click "yes" in "Are you sure?" window)
3. Drag a map while pressing right mouse button on a selected object.

A confirmation window appears, like you've just dragged all the objects, but if you click "No", previous move action is undone.

[Don-vip]

Easily reproduced. I don't know if my patch is the best solution, but it fixes this issue without any visible side effect.

[malenki]

Confirm.
It doesn't matter if you click yes or no. Both ways JOSM falsely assumes you moved the objects.

[Don-vip]

I'm not confident with my patch, especially on MacOS. I've only tested it under Windows, would someone test it on other platforms ?

[rickmastfan67]

I just want to comment on this bug. On step #2, I can duplicate this when hitting "Undo Move" as well.

[bastiK]

Why should it be platform dependent? Patch seems to work for me (Ubuntu Linux).

[Don-vip]

Replying to bastiK:

Why should it be platform dependent? Patch seems to work for me (Ubuntu Linux).

I don't know, I have never used a Mac, and this code in MapMover.java let me think this OS does not handle mouse events like any others:

    /**
     * Emulates dragging on Mac OSX
     */
    public void mouseMoved(MouseEvent e) {
        if (!movementInPlace)
            return;
        // Mac OSX simulates with  ctrl + mouse 1  the second mouse button hence no dragging events get fired.
        // Is only the selected mouse button pressed?
        if (isPlatformOsx()) {

But if you think it's safe, let's try it :)

[Don-vip]

In [4509/josm]:

fix #6890 - Dragging map is considered moving objects

[bastiK]

Replying to Don-vip:

Replying to bastiK:

Why should it be platform dependent? Patch seems to work for me (Ubuntu Linux).

I don't know, I have never used a Mac, and this code in MapMover.java let me think this OS does not handle mouse events like any others:

    /**
     * Emulates dragging on Mac OSX
     */
    public void mouseMoved(MouseEvent e) {
        if (!movementInPlace)
            return;
        // Mac OSX simulates with  ctrl + mouse 1  the second mouse button hence no dragging events get fired.
        // Is only the selected mouse button pressed?
        if (isPlatformOsx()) {

But if you think it's safe, let's try it :)

Yeah, hopefully Java has some abstraction for the 2nd & 3rd mouse button simulations on Mac.