diff options
| author | delvh <dev.lh@web.de> | 2023-12-25 21:25:29 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-12-25 21:25:29 +0100 |
| commit | 778ad795fd4a19dc15723b59a846a250034c7c3a (patch) | |
| tree | 8d64a21734517a9826655955d77cb0bcc187f784 /models/org.go | |
| parent | b41925cee3d67a1fe546c7a219174e4a8b2302b7 (diff) | |
| download | gitea-778ad795fd4a19dc15723b59a846a250034c7c3a.tar.gz gitea-778ad795fd4a19dc15723b59a846a250034c7c3a.zip | |
Refactor deletion (#28610)
Introduce the new generic deletion methods
- `func DeleteByID[T any](ctx context.Context, id int64) (int64, error)`
- `func DeleteByIDs[T any](ctx context.Context, ids ...int64) error`
- `func Delete[T any](ctx context.Context, opts FindOptions) (int64,
error)`
So, we no longer need any specific deletion method and can just use
the generic ones instead.
Replacement of #28450
Closes #28450
---------
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Diffstat (limited to 'models/org.go')
| -rw-r--r-- | models/org.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/models/org.go b/models/org.go index 5e0deeb8de..5f61f05b16 100644 --- a/models/org.go +++ b/models/org.go @@ -57,7 +57,7 @@ func RemoveOrgUser(ctx context.Context, orgID, userID int64) error { } defer committer.Close() - if _, err := db.GetEngine(ctx).ID(ou.ID).Delete(ou); err != nil { + if _, err := db.DeleteByID[organization.OrgUser](ctx, ou.ID); err != nil { return err } else if _, err = db.Exec(ctx, "UPDATE `user` SET num_members=num_members-1 WHERE id=?", orgID); err != nil { return err |