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
This commit is contained in:
Maxime Dagnicourt
2008-02-04 14:15:07 +00:00
parent af13dfaded
commit 04dc03440c

View File

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