uses a non-entity as target entity in relationship attribute

[sonia pandit]

This is the error message I get when I try to run my sample app.

For a one to many bidirectional mapping with many to one on the child. I have both the classes in the same package and listed in persistence.xml.
I am stuck. Any help would be appreciated.

Thanks,

[James Sutherland]

Ensure the target object is annotated with @Entity, is in the same jar file as the persistence.xml, is listed with the correct package in the persistence.xml, and that you have correctly deployed your application.

If still having issues include the fullexception stack trace and source of the class and persistence.xml.

[sonia pandit]

Now I have a problem with a one to many bidirectional mapping. I get a java.sql.SQLIntegrigyConstraintViolationException Cannot insert NULL into

example of what I have

@JoinColumn(name = "AuthorNo", referencedColumnName = "AuthorNo")
@ManyToOne
private Authors authorNo;


@OneToMany(mappedBy = "authorNo", cascade=CascadeType.ALL)
private List<Books> booksCollection= new ArrayList<Books>();

I would really appreciate help in fixing this problem

[Bogdan Baraila]

Sonia please post you entity classes entirely and also the part of code where you make the insertion. It will be easier for us and for you to see exactly instead of making assumption of what your problem is.

[sonia pandit]

Phones.java

package testonetoone;

import java.io.Serializable;

import java.math.BigDecimal;

import javax.persistence.CascadeType;
import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.FetchType;
import javax.persistence.GeneratedValue;
import javax.persistence.Id;
import javax.persistence.NamedQueries;
import javax.persistence.NamedQuery;
import javax.persistence.OneToOne;
import javax.persistence.SequenceGenerator;


@Entity
@NamedQueries( { @NamedQuery(name = "Phones.findAll", query = "select o from Phones o") })
public class Phones implements Serializable {
    
    @Column(name = "CUSTOMER_ID")
    private BigDecimal customerId;
    
    @Id
    @Column(name = "PHONES_ID", nullable = false)
    @GeneratedValue(generator = "PhonesSeq")
    @SequenceGenerator(name = "PhonesSeq", sequenceName = "ISATS.PHONES_SEQ", allocationSize = 1)
    private BigDecimal phonesId;
    
    @Column(name = "PHONE_NUMBER", length = 20)
    private String phoneNumber;
    
    @OneToOne(cascade = CascadeType.ALL, fetch = FetchType.EAGER)
    private Customer customer;
    
    public Phones() {
    }

    public Phones(BigDecimal customerId, String phoneNumber, BigDecimal phonesId) {
        this.customerId = customerId;
        this.phoneNumber = phoneNumber;
        this.phonesId = phonesId;
    }

    public BigDecimal getCustomerId() {
        return customerId;
    }

    public void setCustomerId(BigDecimal customerId) {
        this.customerId = customerId;
    }

    public BigDecimal getPhonesId() {
        return phonesId;
    }

    public void setPhonesId(BigDecimal phonesId) {
        this.phonesId = phonesId;
    }

    public String getPhoneNumber() {
        return phoneNumber;
    }

    public void setPhoneNumber(String phoneNumber) {
        this.phoneNumber = phoneNumber;
    }

    public void setCustomer(Customer customer) {
        this.customer = customer;
    }

    public Customer getCustomer() {
        return customer;
    }
}

Customer.java

package testonetoone;

import java.io.Serializable;

import java.math.BigDecimal;

import javax.persistence.CascadeType;
import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.FetchType;
import javax.persistence.GeneratedValue;
import javax.persistence.Id;
import javax.persistence.NamedQueries;
import javax.persistence.NamedQuery;
import javax.persistence.OneToOne;
import javax.persistence.SequenceGenerator;


@Entity
@NamedQueries( { @NamedQuery(name = "Customer.findAll", query = "select o from Customer o") })
public class Customer implements Serializable {
    
    @Id
    @Column(name = "CUSTOMER_ID", nullable = false)
    @GeneratedValue(generator = "CustomerSeq")
    @SequenceGenerator(name = "CustomerSeq", sequenceName = "ISATS.CUSTOMER_SEQ", allocationSize = 1)
    private BigDecimal customerId;
    
    @Column(name = "CUSTOMER_NAME", length = 20)
    private String customerName;
    
    @Column(name = "PHONES_ID", nullable = false)
    private BigDecimal phonesId;
    
    @OneToOne(cascade = CascadeType.ALL, fetch = FetchType.EAGER, mappedBy = "customer")
    private Phones telephoneNumberMandatory;
    
    public Customer() {
    }

    public Customer(BigDecimal customerId, String customerName, BigDecimal phonesId) {
        this.customerId = customerId;
        this.customerName = customerName;
        this.phonesId = phonesId;
    }

    public BigDecimal getCustomerId() {
        return customerId;
    }

    public void setCustomerId(BigDecimal customerId) {
        this.customerId = customerId;
    }

    public String getCustomerName() {
        return customerName;
    }

    public void setCustomerName(String customerName) {
        this.customerName = customerName;
    }

    public BigDecimal getPhonesId() {
        return phonesId;
    }

    public void setPhonesId(BigDecimal phonesId) {
        this.phonesId = phonesId;
    }

    public void setTelephoneNumberMandatory(Phones telephoneNumberMandatory) {
        this.telephoneNumberMandatory = telephoneNumberMandatory;
    }

    public Phones getTelephoneNumberMandatory() {
        return telephoneNumberMandatory;
    }
}

TestOneToOne.java

package testonetoone;

import javax.persistence.EntityManager;
import javax.persistence.EntityManagerFactory;
import javax.persistence.Persistence;

public class TestOneToOne {
    public TestOneToOne() {
        super();
    }

    public static void main(String[] args) {
        TestOneToOne testOneToOne = new TestOneToOne();
        EntityManagerFactory emf = null;
        
        emf = Persistence.createEntityManagerFactory("TestOneToOne", new java.util.HashMap());
        // 1) Create customer,  create phones,  set add mapping 
        EntityManager em = emf.createEntityManager();

        
        Customer c = new Customer();
        Phones p = new Phones();
        c.setCustomerName("Chrysler");
        p.setCustomer(c);
        p.setPhoneNumber("5716120000");
        c.setTelephoneNumberMandatory(p);
        try {
            em.getTransaction().begin();
            em.persist(c);
            em.getTransaction().commit();


        } finally {

            em.close();
        }
    }
}

persistence.xml

<?xml version="1.0" encoding="windows-1252" ?>
<persistence xmlns="http://java.sun.com/xml/ns/persistence" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
             xsi:schemaLocation="http://java.sun.com/xml/ns/persistence http://java.sun.com/xml/ns/persistence/persistence_2_0.xsd"
             version="2.0">
    <persistence-unit name="TestOneToOne">
        <provider>org.eclipse.persistence.jpa.PersistenceProvider</provider>
        <jta-data-source>java:/app/jdbc/jdbc/idev01DS</jta-data-source>
        <class>testonetoone.Customer</class>
        <class>testonetoone.Phones</class>
        <properties>
            <property name="eclipselink.target-server" value="WebLogic_10"/>
            <property name="javax.persistence.jtaDataSource" value="java:/app/jdbc/jdbc/idev01DS"/>
            <property name="eclipselink.jdbc.driver" value="oracle.jdbc.OracleDriver"/>
            <property name="eclipselink.jdbc.url" value="jdbc:oracle:thin:@idev01.XXXX.com:1521:dev"/>
            <property name="eclipselink.jdbc.user" value="XXXX"/>
            <property name="eclipselink.jdbc.password" value="password"/>
        </properties>
    </persistence-unit>
</persistence>


[code=java]Error Message:

[EL Warning]: 2011-06-21 09:12:14.261--UnitOfWork(24485138)--Exception [EclipseLink-4002] (Eclipse Persistence Services - 2.1.3.v20110304-r9073): org.eclipse.persistence.exceptions.DatabaseException
Internal Exception: java.sql.SQLIntegrityConstraintViolationException: ORA-01400: cannot insert NULL into ("XXXX"."CUSTOMER"."PHONES_ID")

Error Code: 1400
Call: INSERT INTO CUSTOMER (CUSTOMER_ID, CUSTOMER_NAME, PHONES_ID) VALUES (?, ?, ?)
	bind => [2, Chrysler, null]
Query: InsertObjectQuery(testonetoone.Customer@2c9103)
Exception in thread "main" javax.persistence.RollbackException: Exception [EclipseLink-4002] (Eclipse Persistence Services - 2.1.3.v20110304-r9073): org.eclipse.persistence.exceptions.DatabaseException
Internal Exception: java.sql.SQLIntegrityConstraintViolationException: ORA-01400: cannot insert NULL into ("XXXX"."CUSTOMER"."PHONES_ID")

Error Code: 1400
Call: INSERT INTO CUSTOMER (CUSTOMER_ID, CUSTOMER_NAME, PHONES_ID) VALUES (?, ?, ?)
	bind => [2, Chrysler, null]
Query: InsertObjectQuery(testonetoone.Customer@2c9103)
	at org.eclipse.persistence.internal.jpa.transaction.EntityTransactionImpl.commitInternal(EntityTransactionImpl.java:102)
	at org.eclipse.persistence.internal.jpa.transaction.EntityTransactionImpl.commit(EntityTransactionImpl.java:63)
	at testonetoone.TestOneToOne.main(TestOneToOne.java:30)
Caused by: Exception [EclipseLink-4002] (Eclipse Persistence Services - 2.1.3.v20110304-r9073): org.eclipse.persistence.exceptions.DatabaseException
Internal Exception: java.sql.SQLIntegrityConstraintViolationException: ORA-01400: cannot insert NULL into ("XXXX"."CUSTOMER"."PHONES_ID")

Error Code: 1400
Call: INSERT INTO CUSTOMER (CUSTOMER_ID, CUSTOMER_NAME, PHONES_ID) VALUES (?, ?, ?)
	bind => [2, Chrysler, null]
Query: InsertObjectQuery(testonetoone.Customer@2c9103)

[/code]

[joy b chakravarty]

Try making Customer your driving entity with the following change..

Phones.java

 @OneToOne(cascade = CascadeType.ALL, fetch = FetchType.EAGER, mappedBy = "telephoneNumberMandatory")  
 private Customer customer;  

and
Customer.java

@OneToOne(cascade = CascadeType.ALL, fetch = FetchType.EAGER)  
private Phones telephoneNumberMandatory;

[sonia pandit]

caused by: java.sql.SQLSyntaxErrorException: ORA-00904: "TELEPHONENUMBERMANDATORY_PHONES_ID": invalid identifier

[joy b chakravarty]

out of curiosity..does this work
try {
em.getTransaction().begin();
em.persist(p);
em.getTransaction().commit();

[sonia pandit]

No. I am still getting an error

... 2 more
Caused by: java.sql.SQLSyntaxErrorException: ORA-00904: "TELEPHONENUMBERMANDATORY_PHONES_ID": invalid identifier

[joy b chakravarty]

oops .. i hope when you tried em.persist(p); you used your original code

[sonia pandit]

yes

[joy b chakravarty]

Had a closer look at your code.
The Customer class has
@Column(name = "PHONES_ID", nullable = false)
private BigDecimal phonesId;

and your Phones class has

@Column(name = "CUSTOMER_ID")
private BigDecimal customerId;

1> Even in bidirectional relationships (an ORM concept) you dont need to have foreign keys in both tables.
2> You dont specify foreign keys in this fashion, you do it using JoinColumn

[joy b chakravarty]

One to One

[sonia pandit]

I took it out after I posted the code.

See:

http://forums.oracle.com/forums/thread.jspa?messageID=9645937�

[joy b chakravarty]

From the link you mentioned ...

CUSTOMER.java
@OneToOne(cascade = CascadeType.ALL, fetch = FetchType.EAGER)
@JoinColumn(name="CUSTOMER_ID", insertable = false, updatable = false)
private Phones telephoneNumberMandatory;

Phones.java
@OneToOne(cascade = CascadeType.ALL, fetch = FetchType.EAGER, mappedBy = "telephoneNumberMandatory")
private Customer customer;

The Customer table has the column Phone_ID
The Phone table shouldn't and doesn't not have the column Customer_Id (as its not needed)

So you the way these two tables need to be joined is by the Phone_Id (pk in Phones table and fk in Customer table)

[sonia pandit]

IT is a bidirectional oneToOne. That is not where my error is though.

Caused by: java.sql.SQLSyntaxErrorException: ORA-00904: "TELEPHONENUMBER_PHONES_ID": invalid identifier

at oracle.jdbc.driver.T4CTTIoer.processError(T4CTTIoer.java:440)


I changed the TelephoneNumberMandatory to TELEPHONENUMBER

[joy b chakravarty]

Yes it is bidirectional OnetoOne, but the tables in database will have just one foreign key.

CUSTOMER.java
@OneToOne(cascade = CascadeType.ALL, fetch = FetchType.EAGER)
@JoinColumn(name="CUSTOMER_ID", insertable = false, updatable = false)
private Phones telephoneNumberMandatory

The above indicates that the Customer and Phones needs to be joined by CUSTOMER_ID, in that case the PHONE_ID column in Customer is redundant.
Please try with
@JoinColumn(name="PHONE_ID", insertable = false, updatable = false)

[sonia pandit]

@OneToOne(optional = false,cascade = CascadeType.ALL, fetch = FetchType.EAGER)
@JoinColumn(name = "PHONE_ID", unique = true, nullable = false, updatable = false)
private Phones telephoneNumber;


is what I have now, I am getting the following error:


eption [EclipseLink-4002] (Eclipse Persistence Services - 2.1.3.v20110304-r9073): org.eclipse.persistence.exceptions.DatabaseException
Internal Exception: java.sql.SQLSyntaxErrorException: ORA-00904: "TELEPHONENUMBER_PHONES_ID": invalid identifier

Error Code: 904
Call: INSERT INTO PHONES (TELEPHONENUMBER_PHONES_ID, PHONE_NUMBER) VALUES (?, ?)
bind => [101, 5716120000]
Query: InsertObjectQuery(testonetoone.Phones@1ea0105)
at org.eclipse.persistence.exceptions.DatabaseException.sqlException(DatabaseException.java:324)


I really appreciate. need to get it resolved ASAP

[sonia pandit]

fixed it. onetoone works! hurrah

[sonia pandit]

http://forums.oracle.com/forums/thread.jspa?threadID=2235014

The onetoone mapping stopped working

[himanshu rai]

im having a similar problem with my foreign keys.this is a sample appi have been trying using this particular link

http://netbeans.org/kb/67/java/gui-db-custom.html



but after making changes in the customers and orders class if i try to run the program i get the following error

[class customerrecords.Customers] uses a non-entity [class customerrecords.Countries] as target entity in the relationship attribute [private customerrecords.Countries customerrecords.Customers.countryId].