Commit bb8865c3 authored by Piotr Gawron's avatar Piotr Gawron
Browse files

Merge branch '659-updateUser-fix-final' into 'devel_12.1.x'

Resolve "The privileges of a new user are not saved in some cases"

See merge request minerva/core!636
parents 854ff587 67de09df
Pipeline #8677 passed with stages
in 14 minutes and 25 seconds
minerva (12.1.8) stable; urgency=medium minerva (12.1.8) stable; urgency=medium
* Bug fix: add a project reset users custom overlays limit to default (#679) * Bug fix: add a project reset users custom overlays limit to default (#679)
* Bug fix: edit/remove project button was disabled based on Configuration * Bug fix: edit/remove project button was disabled based on Configuration
Manage Privilege, but should be based on Project Manage Privilege (#681) Manage Privilege, but should be based on Project Manage Privilege (#681)
* Bug fix: another minor problem with updating privileges introduced in
12.1.6 fixed (#659)
-- Piotr Gawron <piotr.gawron@uni.lu> Fri, 8 Feb 2019 15:30:00 +0200 -- Piotr Gawron <piotr.gawron@uni.lu> Fri, 8 Feb 2019 15:30:00 +0200
......
This diff is collapsed.
...@@ -1244,8 +1244,7 @@ ServerConnector.updateUser = function (user) { ...@@ -1244,8 +1244,7 @@ ServerConnector.updateUser = function (user) {
.then(function (response) { .then(function (response) {
canModifyPrivileges = JSON.parse(response).privileges.some(function (privilege) { canModifyPrivileges = JSON.parse(response).privileges.some(function (privilege) {
return privilege.type === PrivilegeType.USER_MANAGEMENT return (privilege.type === PrivilegeType.USER_MANAGEMENT) && parseInt(privilege.value) === 1;
|| privilege.type === PrivilegeType.PROJECT_MANAGEMENT;
}); });
return self.getConfiguration(); return self.getConfiguration();
}) })
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment