Merge pull request #1254 from ONLYOFFICE/bugfix/chage-user-type
Bugfix/chage user type
This commit is contained in:
commit
f04408ace1
@ -15,6 +15,7 @@ const ChangeUserTypeEvent = ({
|
||||
peopleFilter,
|
||||
updateUserType,
|
||||
getUsersList,
|
||||
onClose,
|
||||
}) => {
|
||||
const {
|
||||
toType,
|
||||
@ -58,7 +59,7 @@ const ChangeUserTypeEvent = ({
|
||||
};
|
||||
|
||||
const onChangeUserType = () => {
|
||||
onClose();
|
||||
onClosePanel();
|
||||
updateUserType(toType, userIDs, peopleFilter, fromType)
|
||||
.then(() => {
|
||||
toastr.success(t("SuccessChangeUserType"));
|
||||
@ -83,14 +84,15 @@ const ChangeUserTypeEvent = ({
|
||||
});
|
||||
};
|
||||
|
||||
const onClose = () => {
|
||||
const onClosePanel = () => {
|
||||
setVisible(false);
|
||||
onClose();
|
||||
};
|
||||
|
||||
const onCloseAction = async () => {
|
||||
await getUsersList(peopleFilter);
|
||||
abortCallback && abortCallback();
|
||||
onClose();
|
||||
onClosePanel();
|
||||
};
|
||||
|
||||
const getType = (type) => {
|
||||
@ -99,6 +101,8 @@ const ChangeUserTypeEvent = ({
|
||||
return t("Common:DocSpaceAdmin");
|
||||
case "manager":
|
||||
return t("Common:RoomAdmin");
|
||||
case "collaborator":
|
||||
return t("Common:Collaborator");
|
||||
case "user":
|
||||
default:
|
||||
return t("Common:User");
|
||||
|
@ -90,6 +90,8 @@ const User = ({
|
||||
? "admin"
|
||||
: option.key === "roomAdmin"
|
||||
? "manager"
|
||||
: option.key === "collaborator"
|
||||
? "collaborator"
|
||||
: "user";
|
||||
|
||||
const successCallback = () => {
|
||||
@ -98,7 +100,13 @@ const User = ({
|
||||
|
||||
setIsLoading(true);
|
||||
|
||||
if (!changeUserType(userType, [user], successCallback, abortCallback)) {
|
||||
const needChangeUserType =
|
||||
((user.isVisitor || user.isCollaborator) && userType === "manager") ||
|
||||
(user.isVisitor && userType === "collaborator");
|
||||
|
||||
if (needChangeUserType) {
|
||||
changeUserType(userType, [user], successCallback, abortCallback);
|
||||
} else {
|
||||
updateRole(option);
|
||||
}
|
||||
};
|
||||
|
Loading…
Reference in New Issue
Block a user