[Document] We should have a "safe" method to update documents

[johnsudaar]

A method that will add the bson.M{"$set": query}

To prevent dumm errors like:

document.Update(ctx, Collection, bson.M{"field": "value"}, doc)

[EtienneM]

We could just modify this condition:

go-utils/mongo/document/document.go

Lines 212 to 215 in 8396dba

	doc.setUpdatedAt(now)
	if _, ok := update["$set"]; ok {
	update["$set"].(bson.M)["updated_at"] = now
	}

Replace it with:

 if _, ok := update["$set"]; !ok { 
 	update["$set"] = update
} 
update["$set"].(bson.M)["updated_at"] = now 

?

[Soulou]

Or we could just check that the keys of update are all starting with a $ character and returning an error if there is none.

As a result it won't break code using '$inc' for instance.

[johnsudaar]

Or a method like:

document.Set(ctx, Collection, bson.M{"field": "value"}, doc)