hibernate.PersistentObjectException: detached entity passed to persist

Question

So I am having problems creating this new MenuPortlet when it has a parent of the same type. The parent exists in the database, and it is there when I setParent (mp).

package com.kpsol.version6.model;

import java.io.Serializable;
import java.util.HashSet;
import java.util.Set;

import javax.persistence.CascadeType;
import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.FetchType;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.JoinColumn;
import javax.persistence.ManyToOne;
import javax.persistence.NamedQuery;
import javax.persistence.OneToMany;
import javax.persistence.OrderBy;
import javax.persistence.Table;
import javax.persistence.TableGenerator;

@Entity
@Table(name = "MENU_PORTLET")
@NamedQuery(name = "menuPortlet.findAll", query = "SELECT t FROM MenuPortlet t")

public class MenuPortlet extends Portlet implements Serializable {

    private static final long serialVersionUID = 1L;

    @TableGenerator(name = "MenuPortlet", table = "NRD_ID", pkColumnName = "TABLENAME", valueColumnName = "NEXTID", pkColumnValue = "MENU_PORTLET", allocationSize = 1)
    @Id
    @GeneratedValue(strategy = GenerationType.TABLE, generator = "MenuPortlet")
    @Column(name = "MENUPORTLETID")
    private Long id;

    @ManyToOne
    @JoinColumn(name = "PARENTID")
    MenuPortlet parent;

    @OneToMany(mappedBy = "parent", fetch = FetchType.LAZY, cascade = { CascadeType.ALL })
    @OrderBy("PORTLETORDER")
    Set<MenuPortlet> children = new HashSet<MenuPortlet>();

    @JoinColumn(name = "USERID")
    private Long userId;

    @Column(name = "ISUSER")
    private Boolean isUser;

    public Long getId() {
        return this.id;
    }

    public void setId(Long id) {
        this.id = id;
    }

    public static long getSerialversionuid() {
        return serialVersionUID;
    }

    public Long getUserId() {
        return this.userId;
    }

    public void setUserId(Long userId) {
        this.userId = userId;
    }

    public MenuPortlet getParent() {
        return this.parent;
    }

    public void setParent(MenuPortlet parent) {
        this.parent = parent;
    }

    public Set<MenuPortlet> getChildren() {
        return this.children;
    }

    public void setChildren(Set<MenuPortlet> children) {
        this.children = children;
    }

    public Boolean getIsUser() {
        return this.isUser;
    }

    public void setIsUser(Boolean isUser) {
        this.isUser = isUser;
    }

}

This is how I have my entity for the parent relationship. I have tried lazy and eager loads for both parent and children, neither seem to make a difference.

portlet.setParent(mp);
portlet = this.portletservice.create(portlet);

mp is the correct MenuPortlet, no problems there.

this.em.persist(portlet);

Honestly there is nothing else really to show. The annoying thing is I have done similar things in several other places and no problems, and I have literally copied down to the last part the same code, and it still does not work. What am I missing?

@XtremeBaumer by id. String ejbQL = "SELECT OBJECT(p) FROM MenuPortlet as p WHERE p.id = :portletid"; Simply ejbql, works fine. — user1817988, Jul 04 '18 at 12:05
Shouldnt' that query return a `MenuPortlet` which you then set as parent? — XtremeBaumer, Jul 04 '18 at 12:09
@XtremeBaumer Yea it does MenuPortlet result = (MenuPortlet) query.getSingleResult(); return result; — user1817988, Jul 04 '18 at 12:10
What hibernate method does `portletservice.create(portlet)` call? — XtremeBaumer, Jul 04 '18 at 12:15
And there is your problem. You didn't read the link in my second comment. Do it and your problem will be gone — XtremeBaumer, Jul 04 '18 at 12:18
@XtremeBaumer I did read it but it makes no sense to me. There is a new MenuPortlet and also in some of our other code persist has been used and it works as intended. Are you saying even though I am creating a new MenuPortlet because I am relating it to something that already exists I should use merge? — user1817988, Jul 04 '18 at 12:23
That you should definitly try. If it doesn't work, then you have to insert the new Portlet without a parent and update afterwards — XtremeBaumer, Jul 04 '18 at 12:30

hibernate.PersistentObjectException: detached entity passed to persist

0 Answers0