almet/umap
1
0
Fork 0
mirror of https://github.com/umap-project/umap.git synced 2025-04-28 19:42:36 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

fix(sync): Allow cloning of features

Browse source 
Removing the `id` from the feature when cloning makes it generate a new
one. Without this change, the cloned feature keep the already existing
`id`, and during sync, the original object is lost (replaced by the
clone).

Nobody wants to be replaced by a clone.
This commit is contained in:
Alexis Métaireau 2024-05-09 11:14:43 +02:00
parent e4bcb53c6f
commit 28a359e03f
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
umap/static/umap/js/umap.features.js
View file

@ -595,7 +595,10 @@ U.FeatureMixin = {
},
clone: function () {
const layer = this.datalayer.geojsonToFeatures(this.toGeoJSON())
const geoJSON = this.toGeoJSON()
delete geoJSON.id
delete geoJSON.properties.id
const layer = this.datalayer.geojsonToFeatures(geoJSON)
layer.isDirty = true
layer.edit()
return layer
@ -652,7 +655,7 @@ U.Marker = L.Marker.extend({
},
this
)
this.on('editable:drawing:end', this.onCommit)
this.on('editable:drawing:commit', this.onCommit)
if (!this.isReadOnly()) this.on('mouseover', this._enableDragging)
this.on('mouseout', this._onMouseOut)
this._popupHandlersAdded = true // prevent Leaflet from binding event on bindPopup
@ -1275,6 +1278,8 @@ U.Polygon = L.Polygon.extend({
toPolyline: function () {
const geojson = this.toGeoJSON()
delete geojson.id
delete geojson.properties.id
geojson.geometry.type = 'LineString'
geojson.geometry.coordinates = U.Utils.flattenCoordinates(
geojson.geometry.coordinates