From: mka <mka@opencascade.com>
Date: Mon, 18 Mar 2013 08:14:36 +0000 (+0000)
Subject: Add/Remove contributors functionalities are modified
X-Git-Tag: Root_Delivery2_2013_04_22~91
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=c62f135ddbce1dd13507260a1a7d019fc02fb6b9;p=tools%2Fsiman.git

Add/Remove contributors functionalities are modified
---

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<User> contributor = getModifiableContributors(aStudy); // Initializes contributor
 		for (Iterator<User> 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<User> contributor = getModifiableContributors(aStudy); // Initializes contributor
 		Boolean done = false;