Empty model on BeforeInsert hook

[uptrace]

Empty model on BeforeInsert hook

[vmihailenco]

Why I must use query.GetModel() all time instead use self object?

Because Bun also works with slice-based models, e.g. *[]User. And in some cases it might be useful to have access to a whole slice.

---

Overall it is best to only use hooks as a last resort when everything else fails. In your case you should create InsertUser func and have everything there.

[vmihailenco]

Also see https://bun.uptrace.dev/guide/hooks.html#disclaimer

[vmihailenco]

IDB was added in v0.2.1. See https://github.com/uptrace/bun/tree/master/example/tx-composition

[jpascal]

Overall it is best to only use hooks as a last resort when everything else fails. In your case you should create InsertUser func and have everything there.

If I will create function Insert User and for other operations like this, then I didn't see some profit from model hooks overall. ) May be model hooks in this lib must not to be? :)

[jpascal]

Another case...
I have models User and Profile.
One logic need to create user directly, other need create user and profile inside transaction. In this case I must realize InsertUser(bun.DB), InsertUserTx(bun.Tx), InsertProfile(bun.DB), InsertProfileTx(bun.Tx), ... a lot of code ... In old variant of hooks all work perfectly.

[jpascal]

Better example for https://bun.uptrace.dev/guide/hooks.html#disclaimer

func UpdateUser(ctx context.Context, query bun.UpdateQuery, user *User) error { ... }
func InsertUser(ctx context.Context, query bun.InsertQuery, user *User) error { ... }

[vmihailenco]

I am not sure sure how you managed transactions with hooks, but that does not sound right :)

A better solution would be to create a common interface for bun.DB and bun.Tx. They both have methods like NewSelect and NewInsert so you can pass a tx instead of db:

tx := db.Begin()
InsertUser(ctx, tx)
InsertProfile(ctx, tx)
tx.Commit()

I guess Bun must provide such interface.

May be model hooks in this lib must not to be? :)

Ideally, yes. Practically you should not use hooks for such common things like transactions and validation.