diff --git a/controllers/user.go b/controllers/user.go index 7836a79f..7d8754bb 100644 --- a/controllers/user.go +++ b/controllers/user.go @@ -448,6 +448,10 @@ func (c *ApiController) SetPassword() { } targetUser, err := object.GetUser(userId) + if targetUser == nil { + c.ResponseError(fmt.Sprintf(c.T("general:The user: %s doesn't exist"), userId)) + return + } if err != nil { c.ResponseError(err.Error()) return diff --git a/web/src/UserEditPage.js b/web/src/UserEditPage.js index bb0b187b..f0e90cba 100644 --- a/web/src/UserEditPage.js +++ b/web/src/UserEditPage.js @@ -402,7 +402,7 @@ class UserEditPage extends React.Component { {Setting.getLabel(i18next.t("general:Password"), i18next.t("general:Password - Tooltip"))} : - + ); diff --git a/web/src/common/modal/PasswordModal.js b/web/src/common/modal/PasswordModal.js index a240d0d1..cd3882d0 100644 --- a/web/src/common/modal/PasswordModal.js +++ b/web/src/common/modal/PasswordModal.js @@ -26,6 +26,7 @@ export const PasswordModal = (props) => { const [newPassword, setNewPassword] = React.useState(""); const [rePassword, setRePassword] = React.useState(""); const {user} = props; + const {userName} = props; const {organization} = props; const {account} = props; @@ -90,7 +91,7 @@ export const PasswordModal = (props) => { return; } - UserBackend.setPassword(user.owner, user.name, oldPassword, newPassword) + UserBackend.setPassword(user.owner, userName, oldPassword, newPassword) .then((res) => { if (res.status === "ok") { Setting.showMessage("success", i18next.t("user:Password set successfully"));