From c0d01d4a4681a59b229bc4c0abdd27e671e09463 Mon Sep 17 00:00:00 2001 From: Maxime Dagnicourt Date: Tue, 15 Jan 2008 17:03:25 +0000 Subject: [PATCH] Policy failures: Code warning - failed on resource UserManagerBean.java. Reason: Type safety: The expression of type List needs unchecked conversion to conform to List, line 71 - failed on resource ConferenceManagerBean.java. Reason: Type safety: The expression of type List needs unchecked conversion to conform to List, line 8 Override reason: d --- .../org/yacos/core/article/ArticleManagerBean.java | 6 ++++-- .../org/yacos/core/article/IArticleManager.java | 5 +++-- .../yacos/core/conferences/ConferenceManagerBean.java | 3 +++ YACOSCore/ejbModule/org/yacos/core/users/RolePK.java | 10 ++++++++++ .../org/yacos/core/users/UserManagerBean.java | 2 +- 5 files changed, 21 insertions(+), 5 deletions(-) diff --git a/YACOSCore/ejbModule/org/yacos/core/article/ArticleManagerBean.java b/YACOSCore/ejbModule/org/yacos/core/article/ArticleManagerBean.java index 6a39c9e..3fd830a 100644 --- a/YACOSCore/ejbModule/org/yacos/core/article/ArticleManagerBean.java +++ b/YACOSCore/ejbModule/org/yacos/core/article/ArticleManagerBean.java @@ -7,7 +7,7 @@ import javax.ejb.Stateless; import javax.persistence.EntityManager; import javax.persistence.NoResultException; import javax.persistence.PersistenceContext; -import javax.persistence.Query; + import org.yacos.core.users.User; @@ -26,7 +26,9 @@ public class ArticleManagerBean implements IArticleManager, Serializable { } - public Article addArticle(Article a) { + public Article addArticle(int reference, String title, String topic,String url_article, + String mainAuthor, List secondaryAuthor, int state) { + Article a = new Article(reference, title, topic, url_article, mainAuthor, secondaryAuthor, state); em.persist(a); return a; } diff --git a/YACOSCore/ejbModule/org/yacos/core/article/IArticleManager.java b/YACOSCore/ejbModule/org/yacos/core/article/IArticleManager.java index 66b7da0..3250dd1 100644 --- a/YACOSCore/ejbModule/org/yacos/core/article/IArticleManager.java +++ b/YACOSCore/ejbModule/org/yacos/core/article/IArticleManager.java @@ -2,12 +2,13 @@ package org.yacos.core.article; import java.util.List; import javax.ejb.Remote; -import javax.persistence.EntityManager; + @Remote public interface IArticleManager { public Article getArticle(Integer id); - public Article addArticle(Article a); + public Article addArticle(int reference, String title, String topic,String url_article, + String mainAuthor, List secondaryAuthor, int state); public void removeArticle(Article a); public void updateArticle(Article article); public List
getArticles(); diff --git a/YACOSCore/ejbModule/org/yacos/core/conferences/ConferenceManagerBean.java b/YACOSCore/ejbModule/org/yacos/core/conferences/ConferenceManagerBean.java index 0515598..aeef941 100644 --- a/YACOSCore/ejbModule/org/yacos/core/conferences/ConferenceManagerBean.java +++ b/YACOSCore/ejbModule/org/yacos/core/conferences/ConferenceManagerBean.java @@ -65,6 +65,7 @@ public class ConferenceManagerBean implements IConferenceManager { public void addRole(Role.RoleType roleType, User user, Conference conf) { Role role = new Role(roleType, user, conf); + System.out.println(role); em.persist(role); } @@ -76,6 +77,8 @@ public class ConferenceManagerBean implements IConferenceManager { em.merge(role); } + //FIXME + //methode utile? public List getRoles(){ return em.createQuery("from Role role").getResultList(); diff --git a/YACOSCore/ejbModule/org/yacos/core/users/RolePK.java b/YACOSCore/ejbModule/org/yacos/core/users/RolePK.java index eb50fed..c13aaa0 100644 --- a/YACOSCore/ejbModule/org/yacos/core/users/RolePK.java +++ b/YACOSCore/ejbModule/org/yacos/core/users/RolePK.java @@ -2,6 +2,10 @@ package org.yacos.core.users; import java.io.Serializable; +import javax.persistence.Id; +import javax.persistence.JoinColumn; +import javax.persistence.ManyToOne; + import org.yacos.core.conferences.Conference; import org.yacos.core.users.Role.RoleType; @@ -19,10 +23,16 @@ public class RolePK implements Serializable{ /** * Public key, as described here : http://www.jpox.org/docs/1_2/primary_key.html */ + @Id + @ManyToOne(targetEntity=User.class,optional=false) + @JoinColumn(name="user_id",nullable=false) public User user; /** * Public key, as described here : http://www.jpox.org/docs/1_2/primary_key.html */ + @Id + @ManyToOne(targetEntity=Conference.class,optional=false) + @JoinColumn(name="conference_id",nullable=false) public Conference conference; @Override diff --git a/YACOSCore/ejbModule/org/yacos/core/users/UserManagerBean.java b/YACOSCore/ejbModule/org/yacos/core/users/UserManagerBean.java index 6bf0839..6a75015 100644 --- a/YACOSCore/ejbModule/org/yacos/core/users/UserManagerBean.java +++ b/YACOSCore/ejbModule/org/yacos/core/users/UserManagerBean.java @@ -68,7 +68,7 @@ public class UserManagerBean implements IUserManager{ public List getRoles() { // TODO Auto-generated method stub - return null; + return em.createQuery("from Role").getResultList(); } }