Discussion :

[chacalpuant1987]

Bonjour,

J'ai le mapping suivant :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
@Entity
@Table(name = "authorities")
public class Authorities implements Serializable {
 
    @EmbeddedId
    protected AuthoritiesPK authoritiesPK;
 
    @ManyToOne
    @JoinColumn(name = "username", insertable = false, updatable = false)
    private Users users;

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
@Embeddable
public class AuthoritiesPK implements Serializable {
 
    @Column(name = "username")
    private String username;
    @Column(name = "authority")
    private String authority;

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
@Entity
@Table(name = "users")
public class Users implements Serializable {
 
    @Id
    @Column(name = "username")
    private String username;
    @Column(name = "password")
    private String password;
    @Column(name = "enabled")
    private Boolean enabled;
    @Column(name = "email")
    private String email;
 
    @OneToMany(mappedBy = "users", cascade = CascadeType.ALL, fetch = FetchType.LAZY)
    private List<Authorities> authoritiesList;

J'ai l’erreur suivante quand je modifie un utilisateur ?

org.hibernate.TransientObjectException: object references an unsaved transient instance - save the transient instance before flushing: entity.Authorities.users -> entity.Users

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
    public Users modifyUser(Users pUser, List<String> listAuthorities) {
 
        EntityTransaction tx = getEM().getTransaction();
        tx.begin();
        if (pUser.getAuthoritiesList() != null) {
 
            List<Authorities> authoritieses = pUser.getAuthoritiesList();
            for (Authorities a : authoritieses) {
                a.setUsers(null);
                getEM().remove(a);
            }
            pUser.getAuthoritiesList().clear();
            for (String authoritie : listAuthorities) {
                addAuthorities(authoritie, pUser);
            }
            pUser = getEM().merge(pUser);
 
        }
        getEM().remove(pUser);
        tx.commit();
        return pUser;
    }

[tchize_]

a quoi ressemble addAuthorities ?

[chacalpuant1987]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
private void addAuthorities(String authoritie, Users users) {
        AuthoritiesPK authoritiesPK = new AuthoritiesPK(users.getUsername(), authoritie);
        Authorities authorities = new Authorities(authoritiesPK);
        authorities.setUsers(users);
        getEM().persist(authorities);
}

[tchize_]

tu as l'erreur à quelle ligne?

[chacalpuant1987]

at dao.UserDao.addAuthorities(UserDao.java:158)
at dao.UserDao.modifyUser(UserDao.java:75)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

addAuthorities(authoritie, pUser);

(75)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

getEM().persist(authorities);

(158)

[jeffray03]

salut une petite question pourquoi fais-tu:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 getEM().remove(pUser);

dans la method

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

public Users modifyUser(Users pUser, List<String> listAuthorities)

en passant chez moi, j´ai repris ton code, et je n´ai pas de probleme, j´ai modifié le mode de persistence, mais pas de changement explicite du code, et je n´ai aucun probleme meme pour faire un delete, car ta methode

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

modifyUser

delete le user. mais la aussi je n´ai aucune erreur.

Eric

[chacalpuant1987]

Oula c'es une erreur je veux le modifier !

donc je dois faire getEM().merge(pUser); ?

Peux-tu m'expliquer ton changement de persistance ?

Merci

[jeffray03]

en principe tu ne dois fonctionner qu´avec les Id´s,
je m´explique:
MyBean.java

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
  @PersistenceContext(unitName = "nom_de_ton_unit")
  private EntityManager em;
public Users modifyUser(String username, List<String> listAuthorities) {
 
       public Users modifyUser(Users pUser, List<String> listAuthorities) {
 
		if (pUser.getAuthoritiesList() != null) {
 
			List<Authorities> authoritieses = pUser.getAuthoritiesList();
			for (Authorities a : authoritieses) {
				a.setUsers(null);
				em.remove(a);
			}
			pUser.getAuthoritiesList().clear();
			for (String authoritie : listAuthorities) {
				addAuthorities(authoritie, pUser);
			}
			pUser = em.merge(pUser);
 
		}
		return pUser;
	}

et voici son appel:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
  Users pUser = myBean.getUserById("testusername");
  List<String> listAuthorities = new ArrayList<String>();
   listAuthorities.add("Control");
   listAuthorities.add("Admin");
 
   myBean.modifyUser(pUser, listAuthorities);

si tu n´arrive pas a le calquer sur le tiens, je pourrais t´envoyer tout mon code.
eric