umap icon indicating copy to clipboard operation
umap copied to clipboard

Concurrent editing

Open Biondilbiondo opened this issue 7 years ago • 10 comments

When two or more admins add points on the same layer umap can't handle both edits and so ask you to select wich one should be saved, as mantoined in issue #89. I have implemented a basical handling of this situaition that is: if it is possible to merge the two editing without ambiguity, do the merge without displayng the allert otherwise dislay the allert as in the current code. My repo is: https://github.com/Biondilbiondo/django-leaflet-storage

Biondilbiondo avatar Mar 13 '17 19:03 Biondilbiondo

Nice move! I'll have a deeper look asap. That would needs a lot of unit tests, though. Also, when the merge has been done with success, you may reload the layer from the client, to minimize possible future conflicts.

yohanboniface avatar Mar 13 '17 20:03 yohanboniface

Yes, in fact if the server merge two concurrent editing and the client does not update its json the next editing result in loss of data, so we have add a forced reload at the end of each editing. @svalo https://github.com/Biondilbiondo/Leaflet.Storage/commit/9e2f5cb607bf7d889445365c78ca621aee31ab74

Biondilbiondo avatar Mar 13 '17 22:03 Biondilbiondo

For reference http://jsonpatch.com/ may also be an option.

yohanboniface avatar Jun 11 '18 19:06 yohanboniface

Ok, now I have no time for further modifing the code, but we have used the patch for one year and we had no problem. I think that after a little more testing it would be ok as is. Have you found any problems or any scenarios that could rise problems?

Biondilbiondo avatar Jun 11 '18 19:06 Biondilbiondo

Ok, now I have no time for further modifing the code, but we have used the patch for one year and we had no problem.

Wah, cool, I didn't know your patch was used in real life!

I'm in the process of releasing a 1.0 of uMap (see #583), which also merges all three repositories in one (django-leaflet-storagge and leaflet-storage will be included in umap), in order to make it easier to contribute and maintain. Once this is done, I really want to spend time on concurrent editing, and thus I'll look into your patch in more details. :)

yohanboniface avatar Jun 11 '18 19:06 yohanboniface

https://map.obiezionerespinta.info/en/ this is the site where the modified version of umap is used, if you need it we could gave you acess to a development version of the website.

Biondilbiondo avatar Jun 11 '18 19:06 Biondilbiondo

https://map.obiezionerespinta.info/en/ this is the site where the modified version of umap is used, if you need it we could gave you acess to a development version of the website.

Thanks for that, but I think I'll simply test your patch locally when I'm on it. I'll let you know and share here my tests. Great work, and thanks again! :)

yohanboniface avatar Jun 11 '18 19:06 yohanboniface

Hello,

I've just seen the ticket and the proposed code. What action can be taken on this ticket?

We use uMap in emergency situations with Visov and the lack of multi-user mode penalizes us a lot. An example of a map: http://umap.openstreetmap.fr/fr/map/esov-inondations-0112_394366#9/43.5854/6.0178

--- En français ---

Je viens de voir le ticket et les propositions de code. Quelles suites peuvent être données à ce ticket?

Nous utilisons uMap en situation d'urgence avec l'association Visov et l'absence de mode multi-utilisateurs nous pénalise beaucoup. Nous matérialisons les remontées d'informations issues de la veille des réseaux sociaux et diffusons de l'information aux citoyens et aux pouvoirs publics. Ce blocage nous contraint à travailler seul sur la partie carto alors que les infos à cartographier sont parfois nombreuses et s’enchaînent rapidement (nous devons donc parfois prioriser ce qui mérite d'être cartographié). . Un exemple de carte: http://umap.openstreetmap.fr/fr/map/esov-inondations-0112_394366#9/43.5854/6.0178 image

Merci tout plein ;) Donat

Edit; je viens de voir qu'il y a une PR #772 à ce sujet. C'est que ca ne devrait pas tarder à améliorer la situation ;)

gendy54 avatar May 11 '20 23:05 gendy54

Hello,

maybe it would be wise to introduce a warning before editing? I've met lots of users who believe umap can do concurrent editing and then they kill each other's changes. This has been so since years and really damages the uptake of umap which is quite a pity. One typical use case is to put people's pins of their homes during a conference.

The only privacy-clean alternative I know which can do concurrent editing of points on a map is xwiki with the interactive maps application.

Paul

polx avatar Sep 03 '21 21:09 polx

PS: I don't understand why the issue talks about admin... but I guess it means just about any editor... I've never met admins in UMap.

polx avatar Sep 03 '21 21:09 polx