From 04dc03440ce25edaff4bfbaf16c4ebc1e9d78394 Mon Sep 17 00:00:00 2001
From: Maxime Dagnicourt <maxime.dagnicourt@gmail.com>
Date: Mon, 4 Feb 2008 14:15:07 +0000
Subject: [PATCH] Policy failures: Code warning - failed on resource
 ArticleManagerBean.java. Reason: The field ArticleManagerBean.sc is never
 read locally, line 36 - failed on resource ArticleManagerBean.java. Reason:
 The import javax.ejb.TransactionAttribute is never used, line 9 - failed on
 resource ArticleManagerBean.java. Reason: The import
 javax.ejb.TransactionAttributeType is never used, line 1 Override reason: x

---
 .../core/article/ArticleManagerBean.java      | 22 ++++++++++++++++---
 1 file changed, 19 insertions(+), 3 deletions(-)

diff --git a/YACOSCore/ejbModule/org/yacos/core/article/ArticleManagerBean.java b/YACOSCore/ejbModule/org/yacos/core/article/ArticleManagerBean.java
index 561d420..cc1f22a 100644
--- a/YACOSCore/ejbModule/org/yacos/core/article/ArticleManagerBean.java
+++ b/YACOSCore/ejbModule/org/yacos/core/article/ArticleManagerBean.java
@@ -3,7 +3,13 @@ package org.yacos.core.article;
 import java.io.Serializable;
 import java.util.List;
 
+import javax.annotation.Resource;
+import javax.ejb.SessionContext;
 import javax.ejb.Stateless;
+import javax.ejb.TransactionAttribute;
+import javax.ejb.TransactionAttributeType;
+import javax.ejb.TransactionManagement;
+import javax.ejb.TransactionManagementType;
 import javax.persistence.EntityManager;
 import javax.persistence.NoResultException;
 import javax.persistence.PersistenceContext;
@@ -15,6 +21,7 @@ import org.yacos.core.exceptions.ConferenceDoesntExistException;
 import org.yacos.core.users.User;
 
 @Stateless
+@TransactionManagement(TransactionManagementType.CONTAINER)
 public class ArticleManagerBean implements IArticleManager, Serializable {
 
 	/**
@@ -25,6 +32,9 @@ public class ArticleManagerBean implements IArticleManager, Serializable {
 	@PersistenceContext
 	private EntityManager em;
 
+	@Resource
+	private SessionContext sc;
+	
 	public ArticleManagerBean() {
 		
 	}
@@ -96,12 +106,18 @@ public class ArticleManagerBean implements IArticleManager, Serializable {
 		Article article = this.getArticle(articleId);
 		User PCMember = em.find(User.class, memberId);
 		article.assignToPCMember(PCMember);
-		em.flush();
+		this.updateArticle(article);
+		
 	}
 
+	
 	public List<User> getArticlePCMembers(Integer articleId) {
-		Article article = this.getArticle(articleId);
-		return (List<User>) article.getPCMembers();
+			
+			Article article = this.getArticle(articleId);
+			List<User> users = (List<User>)article.getPCMembers();
+			return users;	
+		
+	
 	}
 	
 	public void delegateArticleToReferee(Integer articleId, String refereeId, String memberId){