document.go

@@ -21,7 +21,6 @@ type IDocument interface {
 	Delete() error
 	Remove() error
 	Save() error
-	SaveWith(opts *SaveOptions) error
 	setSelf(arg interface{})
 	getExists() bool
 	setExists(n bool)
@@ -34,10 +33,6 @@ type IDocument interface {
 	setModel(m Model)
 }
 
-type SaveOptions struct {
-	SetTimestamps bool
-}
-
 func (d *Document) getCreated() time.Time {
 	return d.Created
 }
@@ -94,33 +89,25 @@ func (d *Document) Remove() error {
 	return d.Delete()
 }
 
-// SaveWith - updates this Model in the database,
-// or inserts it if it doesn't exist, using the provided
-// SaveOptions
-func (d *Document) SaveWith(opts *SaveOptions) error {
+// Save - updates this Model in the database,
+// or inserts it if it doesn't exist
+func (d *Document) Save() error {
 	val := valueOf(d.self)
 	if val.Kind() == reflect.Slice {
 		for i := 0; i < val.Len(); i++ {
 			cur := val.Index(i)
-			if err := doSave(d.model.getColl(), !d.exists, opts, cur.Interface()); err != nil {
+			asHId := asId(cur.Interface())
+			if err := doSave(d.model.getColl(), !d.exists && reflect.ValueOf(asHId.Id()).IsZero(), cur.Interface()); err != nil {
 				return err
 			}
 		}
 		return nil
 	} else {
-		return doSave(d.model.getColl(), !d.exists, opts, d.self)
+		asHId := asId(val.Interface())
+		return doSave(d.model.getColl(), !d.exists && reflect.ValueOf(asHId.Id()).IsZero(), d.self)
 	}
 }
 
-// Save - updates this Model in the database,
-// or inserts it if it doesn't exist, using
-// default SaveOptions
-func (d *Document) Save() error {
-	return d.SaveWith(&SaveOptions{
-		SetTimestamps: true,
-	})
-}
-
 func (d *Document) serializeToStore() any {
 	return serializeIDs((d).self)
 }

document_internals.go

@@ -84,11 +84,7 @@ func serializeIDs(input interface{}) interface{} {
 					}
 
 				} else {
-					if fv.Type() == reflect.TypeFor[time.Time]() {
-						ret0[bbson.Name] = fv.Interface()
-					} else {
-						ret0[bbson.Name] = serializeIDs(fv.Interface())
-					}
+					ret0[bbson.Name] = serializeIDs(fv.Interface())
 				}
 			}
 
@@ -106,7 +102,7 @@ func serializeIDs(input interface{}) interface{} {
 	}
 	return ret
 }
-func doSave(c *mongo.Collection, isNew bool, opts *SaveOptions, arg interface{}) error {
+func doSave(c *mongo.Collection, isNew bool, arg interface{}) error {
 	var err error
 	d, ok := arg.(IDocument)
 	if !ok {
@@ -122,12 +118,10 @@ func doSave(c *mongo.Collection, isNew bool, opts *SaveOptions, arg interface{})
 	}
 	var asHasId = vp.Interface().(HasID)
 	var asModel = vp.Interface().(IDocument)
-	if isNew && opts.SetTimestamps {
+	if isNew {
 		d.setCreated(now)
 	}
-	if opts.SetTimestamps {
-		d.setModified(now)
-	}
+	d.setModified(now)
 	idxs := d.getModel().getIdxs()
 	for _, i := range idxs {
 		_, err = c.Indexes().CreateOne(context.TODO(), *i)

util.go

@@ -45,12 +45,7 @@ func asId(i interface{}) HasID {
 	var ok bool
 	switch v.Kind() {
 	case reflect.Struct:
-		asHasId, ok = v.Interface().(HasID)
-		if ok {
-			return asHasId
-		}
-		v = reflect.New(v.Type())
-		v.Elem().Set(reflect.ValueOf(i))
+		v = reflect.New(reflect.PointerTo(v.Type()))
 		fallthrough
 	case reflect.Pointer:
 		asHasId, ok = v.Interface().(HasID)