Fix Delte group

Emyrk · Emyrk · commit 874e9daa8591 · 2023-02-02T09:50:22.000-06:00
diff --git a/coderd/authzquery/group.go b/coderd/authzquery/group.go
@@ -14,9 +14,12 @@ func (q *AuthzQuerier) DeleteGroupByID(ctx context.Context, id uuid.UUID) error
 	return authorizedDelete(q.logger, q.authorizer, q.database.GetGroupByID, q.database.DeleteGroupByID)(ctx, id)
 }
 
-func (q *AuthzQuerier) DeleteGroupMember(ctx context.Context, userID uuid.UUID) error {
+func (q *AuthzQuerier) DeleteGroupMemberFromGroup(ctx context.Context, arg database.DeleteGroupMemberFromGroupParams) error {
 	// Deleting a group member counts as updating a group.
-	return authorizedUpdate(q.logger, q.authorizer, q.database.GetGroupByID, q.database.DeleteGroupMember)(ctx, userID)
+	fetch := func(ctx context.Context, arg database.DeleteGroupMemberFromGroupParams) (database.Group, error) {
+		return q.database.GetGroupByID(ctx, arg.GroupID)
+	}
+	return authorizedUpdate(q.logger, q.authorizer, fetch, q.database.DeleteGroupMemberFromGroup)(ctx, arg)
 }
 
 func (q *AuthzQuerier) GetGroupByID(ctx context.Context, id uuid.UUID) (database.Group, error) {

Original file line number	Diff line number	Diff line change
`@@ -14,9 +14,12 @@ func (q *AuthzQuerier) DeleteGroupByID(ctx context.Context, id uuid.UUID) error`
`14`	`14`	`return authorizedDelete(q.logger, q.authorizer, q.database.GetGroupByID, q.database.DeleteGroupByID)(ctx, id)`
`15`	`15`	`}`
`16`	`16`
`17`		`-func (q *AuthzQuerier) DeleteGroupMember(ctx context.Context, userID uuid.UUID) error {`
	`17`	`+func (q *AuthzQuerier) DeleteGroupMemberFromGroup(ctx context.Context, arg database.DeleteGroupMemberFromGroupParams) error {`
`18`	`18`	`// Deleting a group member counts as updating a group.`
`19`		`- return authorizedUpdate(q.logger, q.authorizer, q.database.GetGroupByID, q.database.DeleteGroupMember)(ctx, userID)`
	`19`	`+ fetch := func(ctx context.Context, arg database.DeleteGroupMemberFromGroupParams) (database.Group, error) {`
	`20`	`+ return q.database.GetGroupByID(ctx, arg.GroupID)`
	`21`	`+ }`
	`22`	`+ return authorizedUpdate(q.logger, q.authorizer, fetch, q.database.DeleteGroupMemberFromGroup)(ctx, arg)`
`20`	`23`	`}`
`21`	`24`
`22`	`25`	`func (q *AuthzQuerier) GetGroupByID(ctx context.Context, id uuid.UUID) (database.Group, error) {`