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: explicit batch instead of blindly on add data

Browse source 
Co-authored-by: Yohan Boniface <yohanboniface@free.fr>
This commit is contained in:
David Larlet 2025-04-04 11:45:26 -04:00
parent d6f591b365
commit b342480f3b
No known key found for this signature in database
GPG key ID: 3E2953A359E7E7BD
1 changed files with 14 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
umap/static/umap/js/modules/data/layer.js
View file

@ -327,7 +327,7 @@ export class DataLayer {
this.clear()
return this._umap.formatter
.parse(raw, this.options.remoteData.format)
.then((geojson) => this.fromGeoJSON(geojson))
.then((geojson) => this.fromGeoJSON(geojson, false))
.catch((error) => {
console.debug(error)
Alert.error(
@ -460,10 +460,7 @@ export class DataLayer {
try {
// Do not fail if remote data is somehow invalid,
// otherwise the layer becomes uneditable.
this.sync.startBatch()
const features = this.makeFeatures(geojson, sync)
this.sync.commitBatch()
return features
return this.makeFeatures(geojson, sync)
} catch (err) {
console.debug('Error with DataLayer', this.id)
console.error(err)
@ -529,7 +526,11 @@ export class DataLayer {
async importRaw(raw, format) {
return this._umap.formatter
.parse(raw, format)
.then((geojson) => this.addData(geojson))
.then((geojson) => {
this.sync.startBatch()
this.addData(geojson)
this.sync.commitBatch()
})
.catch((error) => {
console.debug(error)
Alert.error(translate('Import failed: invalid data'))
@ -926,8 +927,13 @@ export class DataLayer {
this.sync.update('options', this.options, oldOptions)
}
this.empty()
if (this.isRemoteLayer()) this.fetchRemoteData()
else this.addData(geojson)
if (this.isRemoteLayer()) {
this.fetchRemoteData()
} else {
this.sync.startBatch()
this.addData(geojson)
this.sync.commitBatch()
}
}
})
}