Project

General

Profile

Actions

Bug #12801

open

WLeafletMap may skip render after replacing marker

Added by Andrey Alekseev 10 months ago. Updated 9 days ago.

Status:
Implemented @Test
Priority:
Normal
Target version:
Start date:
06/28/2024
Due date:
% Done:

100%

Estimated time:

Description

When removing a marker and then immediately creating a new one, there is a possibility, that map won't be updated on client until page is reloaded.
This bug caused by comparing pointers, but not checking if unique_ptr exists, see this line.
In my case, unique_ptr, returned by removeMarker(), is being deleted, and then new marker is created and added, which may be located by the same address, so no changes are sent to the client.

Temporary workaround is to add new marker first and then remove the old one.

Actions #2

Updated by Matthias Van Ceulebroeck 8 months ago

  • Due date deleted (06/28/2024)
  • Assignee changed from Andrey Alekseev to Matthias Van Ceulebroeck
  • Target version set to 4.11.2
Actions #3

Updated by Matthias Van Ceulebroeck 3 months ago

  • Target version changed from 4.11.2 to 4.11.4
Actions #4

Updated by Matthias Van Ceulebroeck 10 days ago

  • Target version changed from 4.11.4 to 4.12.1
Actions #5

Updated by Matthias Van Ceulebroeck 9 days ago

  • Status changed from New to InProgress
  • Target version changed from 4.12.1 to 4.12.0
Actions #6

Updated by Matthias Van Ceulebroeck 9 days ago

  • Status changed from InProgress to Review
Actions #7

Updated by Matthias Van Ceulebroeck 9 days ago

  • Status changed from Review to Implemented @Emweb
  • Assignee changed from Matthias Van Ceulebroeck to Andrey Alekseev
  • % Done changed from 0 to 100
Actions #8

Updated by Matthias Van Ceulebroeck 9 days ago

  • Status changed from Implemented @Emweb to Closed
Actions #9

Updated by Matthias Van Ceulebroeck 9 days ago

  • Status changed from Closed to InProgress
Actions #10

Updated by Matthias Van Ceulebroeck 9 days ago

  • Status changed from InProgress to Implemented @Emweb
Actions #11

Updated by Matthias Van Ceulebroeck 9 days ago

  • Status changed from Implemented @Emweb to Implemented @Test
Actions

Also available in: Atom PDF