Merge pull request #28 from chilianyi/refactor

Fix bind user role: unbind all the roles of the user
openpitrix · Feb 28, 2019 · c829d85 · c829d85
2 parents 11486b1 + 38b9581
commit c829d85
Showing 1 changed file with 0 additions and 1 deletion.
diff --git a/pkg/service/am/resource/user_role_binding_control.go b/pkg/service/am/resource/user_role_binding_control.go
@@ -75,7 +75,6 @@ func BindUserRole(ctx context.Context, req *pb.BindUserRoleRequest) (*pb.BindUse
 
 	// Unbind first
 	if err := global.Global().Database.
-		Where(constants.ColumnRoleId+" in (?)", req.RoleId).
 		Where(constants.ColumnUserId+" in (?)", req.UserId).
 		Delete(models.UserRoleBinding{}).Error; err != nil {
 		return nil, gerr.NewWithDetail(ctx, gerr.Internal, err, gerr.ErrorInternalError)