From abb5088045165d6f5f69aea53176653427ce3179 Mon Sep 17 00:00:00 2001
From: felmer <felmer>
Date: Mon, 25 Mar 2013 07:46:22 +0000
Subject: [PATCH] bug: do not add space if user is 'system'
SVN: 28677
---
.../systemsx/cisd/openbis/generic/server/CommonServer.java | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/openbis/source/java/ch/systemsx/cisd/openbis/generic/server/CommonServer.java b/openbis/source/java/ch/systemsx/cisd/openbis/generic/server/CommonServer.java
index bc044558aee..c07d07de836 100644
--- a/openbis/source/java/ch/systemsx/cisd/openbis/generic/server/CommonServer.java
+++ b/openbis/source/java/ch/systemsx/cisd/openbis/generic/server/CommonServer.java
@@ -460,12 +460,13 @@ public final class CommonServer extends AbstractCommonServer<ICommonServerForInt
spaceBO.save();
// If the user who registers this space is _not_ instance admin, make him space admin for
// the freshly created space.
- if (new AuthorizationServiceUtils(getDAOFactory(), session.getUserName()).doesUserHaveRole(
- RoleCode.ADMIN.toString(), null) == false)
+ PersonPE person = session.tryGetPerson();
+ if (person.isSystemUser() == false
+ && new AuthorizationServiceUtils(getDAOFactory(), session.getUserName())
+ .doesUserHaveRole(RoleCode.ADMIN.toString(), null) == false)
{
registerSpaceRole(sessionToken, RoleCode.ADMIN, new SpaceIdentifier(spaceCode),
Grantee.createPerson(session.getUserName()));
- PersonPE person = session.tryGetPerson();
session.setPerson(getDAOFactory().getPersonDAO().getPerson(person.getId()));
}
}
--
GitLab