Fix: Inconsistencies due to ALTREP parameter (#1916)

max65482 · web-flow · commit bffcd4554676 · 2022-01-31T17:21:23.000+01:00
* delete all parameters of description property upon modification

Signed-off-by: Maximilian Martin &lt;maximilian_martin@gmx.de&gt;
diff --git a/src/models/task.js b/src/models/task.js
@@ -366,7 +366,9 @@ export default class Task {
 	}
 
 	set note(note) {
-		this.vtodo.updatePropertyWithValue('description', note)
+		// To avoid inconsistent property parameters (bug #3863 in nextcloud/calendar), delete the property, then recreate
+		this.vtodo.removeProperty('description')
+		this.vtodo.addPropertyWithValue('description', note)
 		this.updateLastModified()
 		this._note = this.vtodo.getFirstPropertyValue('description') || ''
 	}

Original file line number	Diff line number	Diff line change
`@@ -366,7 +366,9 @@ export default class Task {`
`366`	`366`	`}`
`367`	`367`
`368`	`368`	`set note(note) {`
`369`		`- this.vtodo.updatePropertyWithValue('description', note)`
	`369`	`+ // To avoid inconsistent property parameters (bug #3863 in nextcloud/calendar), delete the property, then recreate`
	`370`	`+ this.vtodo.removeProperty('description')`
	`371`	`+ this.vtodo.addPropertyWithValue('description', note)`
`370`	`372`	`this.updateLastModified()`
`371`	`373`	`this._note = this.vtodo.getFirstPropertyValue('description') \|\| ''`
`372`	`374`	`}`