using Documentbase.Update()

[fabulousduck]

I am trying to update a defined model with the following piece of code

	connection := context.MustGet("dbConnection").(*mongodm.Connection)
	Reservation := connection.Model("reservation")
	reservation := &models.Reservation{}

	err := context.BindJSON(r)
	if err != nil {
		context.AbortWithStatusJSON(500, apierrors.JsonBindingError)
		return
	}

	err = Reservation.FindId(id).Exec(reservation)
	if err != nil {
		context.AbortWithStatusJSON(500, apierrors.DBLookupFailedError)
	}

	err, _ = reservation.Update(bson.M{
		"IsSharedRoom":      r.IsSharedRoom,
		"Beds":              r.Beds,
		"PartySize":         r.PartySize,
		"notes":             r.Notes,
		"ConfirmedGuest":    r.ConfirmedGuest,
		"RoomType":          r.RoomType,
		"CheckinDate":       r.CheckinDate,
		"CheckoutDate":      r.CheckoutDate,
		"ReservationNumber": r.ReservationNumber,
	})

however, when this code is ran, nothing in the database is updated on the model, is there something I am missing ? I also could not find any documentation on how to use this function other than that it can be used on an IDocumentBase

Regards Duck

[moehlone]

As long as I can remember the Update method is not that generic as it should be.. you can either pass a type byte[] where the main content has to be wrapped in the name of a collection (failed and static by a previous design) OR it tries to cast a type of map[string]interface{}. Mainly bson.M is only an "alias" for map[string]interface{}, but mongodm wouldn`t detect it.. Can you try the following?

err, _ = reservation.Update(map[string]interface{}{
		"IsSharedRoom":      r.IsSharedRoom,
		"Beds":              r.Beds,
		"PartySize":         r.PartySize,
		"notes":             r.Notes,
		"ConfirmedGuest":    r.ConfirmedGuest,
		"RoomType":          r.RoomType,
		"CheckinDate":       r.CheckinDate,
		"CheckoutDate":      r.CheckoutDate,
		"ReservationNumber": r.ReservationNumber,
	})

And dont forget to call Save() afterwards - this method does not persist anything.

All in all:

• the Update method is not very good implemented (shame on me)
• there is missing an else for this type condition check (this is why you should get only nil, nil with your snippet)
• there is missing the whole documentation and an example for this

Definitely something to fix, thanks!