Bastian-Greshake-Tzovaras/MapComplete
1
0
Fork 0
forked from MapComplete/MapComplete
Code Pull requests Activity

Various changes after handling many error reports

Browse source
This commit is contained in:
Pieter Vander Vennet 2024-08-09 20:38:13 +02:00
parent 51fa48a01f
commit 9adc91fbae
4 changed files with 31 additions and 24 deletions
Download patch file Download diff file Expand all files Collapse all files

20
src/Logic/Osm/Changes.ts
View file

@ -440,15 +440,17 @@ export class Changes {
}
})
console.debug(
"Calculated the pending changes: ",
result.newObjects.length,
"new; ",
result.modifiedObjects.length,
"modified;",
result.deletedObjects,
"deleted"
)
if(!(result.newObjects.length === 0 && result.modifiedObjects.length === 0 && result.deletedObjects.length === 0)) {
console.debug(
"Calculated the pending changes: ",
result.newObjects.length,
"new; ",
result.modifiedObjects.length,
"modified;",
result.deletedObjects.length,
"deleted"
)
}
return result
}

6
src/Logic/Osm/ChangesetHandler.ts
View file

@ -154,6 +154,10 @@ export class ChangesetHandler {
if (this._reportError) {
this._reportError(e)
}
if((<XMLHttpRequest> e).status === 400){
// This request is invalid. We simply drop the changes and hope that someone will analyze what went wrong with it in the upload; we pretend everything went fine
return
}
console.warn(
"Could not open/upload changeset due to ",
e,
@ -195,7 +199,7 @@ export class ChangesetHandler {
"Could not reuse changeset " +
csId +
", might be closed: " +
(e.stacktrace ?? "" + e)
(e.stacktrace ?? e.status ?? "" + e)
)
}
console.warn("Could not upload, changeset is probably closed: ", e)