From c62f135ddbce1dd13507260a1a7d019fc02fb6b9 Mon Sep 17 00:00:00 2001 From: mka Date: Mon, 18 Mar 2013 08:14:36 +0000 Subject: [PATCH] Add/Remove contributors functionalities are modified --- .../Siman-Common/src/org/splat/service/StudyServiceImpl.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Workspace/Siman-Common/src/org/splat/service/StudyServiceImpl.java b/Workspace/Siman-Common/src/org/splat/service/StudyServiceImpl.java index 8e8e6a9..fa31d39 100644 --- a/Workspace/Siman-Common/src/org/splat/service/StudyServiceImpl.java +++ b/Workspace/Siman-Common/src/org/splat/service/StudyServiceImpl.java @@ -324,6 +324,7 @@ public class StudyServiceImpl implements StudyService { * * @see org.splat.service.StudyService#addContributor(org.splat.dal.bo.som.Study, org.splat.dal.bo.kernel.User) */ + @Transactional public boolean addContributor(final Study aStudy, final User user) { List contributor = getModifiableContributors(aStudy); // Initializes contributor for (Iterator i = contributor.iterator(); i.hasNext();) { @@ -338,7 +339,6 @@ public class StudyServiceImpl implements StudyService { if (absent) { update(aStudy); // Else, useless to re-index the study } - contributor.add(user); return true; } @@ -412,6 +412,7 @@ public class StudyServiceImpl implements StudyService { * * @see org.splat.service.StudyService#removeContributor(org.splat.dal.bo.som.Study, org.splat.dal.bo.kernel.User[]) */ + @Transactional public boolean removeContributor(final Study aStudy, final User... users) { List contributor = getModifiableContributors(aStudy); // Initializes contributor Boolean done = false; -- 2.39.2