Fixes bug when upserting widget with blanked out id

pixlise · Oct 10, 2024 · 8d68e54 · 8d68e54
1 parent d3c9959
commit 8d68e54
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/api/ws/handlers/screen-configuration.go b/api/ws/handlers/screen-configuration.go
@@ -175,11 +175,9 @@ func writeScreenConfiguration(screenConfig *protos.ScreenConfiguration, hctx wsH
 						widget.Id = formWidgetId(widget, screenConfig.Id, i)
 						if widget.Data != nil {
 							// We have widget data, but no ID, so write it to the database with a new ID
-							_, err := hctx.Svcs.MongoDB.Collection(dbCollections.WidgetDataName).UpdateOne(sessCtx, bson.M{
-								"_id": widget.Id,
-							}, bson.M{
-								"$set": widget.Data,
-							}, options.Update().SetUpsert(true))
+							widget.Data.Id = widget.Id
+							_, err := hctx.Svcs.MongoDB.Collection(dbCollections.WidgetDataName).InsertOne(sessCtx, widget.Data)
+
 							if err != nil {
 								return nil, err
 							}