Skip to content

Commit 930640b

Browse files
committed
chore(user): set default BusinessWritePermission for system admins in AddUser function.
1 parent 91cb204 commit 930640b

1 file changed

Lines changed: 12 additions & 1 deletion

File tree

internal/dms/service/user.go

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,17 @@ func (d *DMSService) AddUser(ctx context.Context, currentUserUid string, req *dm
9696
defer func() {
9797
d.log.Infof("AddUsers.req=%v;reply=%v;error=%v", req, reply, err)
9898
}()
99+
// 如果 BusinessWritePermission 为 nil,如果为系统管理员权限,默认有业务写权限
100+
businessWritePermission := req.User.BusinessWritePermission
101+
if businessWritePermission == nil {
102+
for _, uid := range req.User.OpPermissionUids {
103+
if uid == pkgConst.UIDOfOpPermissionGlobalManagement {
104+
t := true
105+
businessWritePermission = &t
106+
break
107+
}
108+
}
109+
}
99110

100111
args := &biz.CreateUserArgs{
101112
UID: req.User.UID,
@@ -111,7 +122,7 @@ func (d *DMSService) AddUser(ctx context.Context, currentUserUid string, req *dm
111122
ThirdPartyUserID: req.User.ThirdPartyUserID,
112123
ThirdPartyUserInfo: req.User.ThirdPartyUserInfo,
113124
UserAuthenticationType: biz.UserAuthenticationType(req.User.UserAuthenticationType),
114-
BusinessWritePermission: req.User.BusinessWritePermission,
125+
BusinessWritePermission: businessWritePermission,
115126
}
116127

117128
uid, err := d.UserUsecase.AddUser(ctx, currentUserUid, args)

0 commit comments

Comments
 (0)