[ 
https://issues.apache.org/jira/browse/OPENJPA-1569?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jerry Carter updated OPENJPA-1569:
----------------------------------

    Priority: Minor  (was: Major)

(Set priority to minor as there are a number of workarounds.)

While some argument might be made for supporting arbitrary types as recipients 
for @Strategy decoration, the spirit of JPA limits one to "user-defined types 
that implement the Serializable interface" (JPA 2.0, section 2.2).  This is 
probably why the distinction is made in JavaTypes, but the implementation might 
be wrong. 

Rather than 'type == Serializable.class', mightn't this be 
'Serializable.class.isAssignableFrom(type)'?

> @Strategy triggers an InvalidStateException for fields which are declared as 
> Java interfaces
> --------------------------------------------------------------------------------------------
>
>                 Key: OPENJPA-1569
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-1569
>             Project: OpenJPA
>          Issue Type: Bug
>    Affects Versions: 2.0.0-beta2
>         Environment: MacOS X, Spring 3.0, Maven
>            Reporter: Jerry Carter
>            Priority: Minor
>
> The @Strategy annotation works fine for classes but does not work if the 
> field is a Java interface.  Consider:
>       @Persistent
>       @Strategy("SimpleHandler")
>       @Column(name="address_packed")
>       private BaseAddress address;
>       public BaseAddress getAddress() { return address; }
>       public void setAddress(BaseAddress address) { this.address = address; }
> This works perfectly if BaseAddress is defined as a class but results in an 
> InvalidStateException if it is an interface:
> <openjpa-2.0.0-beta2-rexported fatal user error> 
> org.apache.openjpa.persistence.InvalidStateException: Encountered unmanaged 
> object in persistent field "com.example.SimpleEntity.address" during flush.  
> However, this field does not allow cascade persist. Set the cascade attribute 
> for this field to CascadeType.PERSIST or CascadeType.ALL (JPA annotations) or 
> "persist" or "all" (JPA orm.xml), or enable cascade-persist globally, or 
> manually persist the related field value prior to flushing. You cannot flush 
> unmanaged objects or graphs that have persistent associations to unmanaged 
> objects.
> FailedObject: com.example.usaddr...@4e0a39de
>       at 
> org.apache.openjpa.kernel.SingleFieldManager.preFlushPC(SingleFieldManager.java:767)
>       at 
> org.apache.openjpa.kernel.SingleFieldManager.preFlush(SingleFieldManager.java:614)
>       at 
> org.apache.openjpa.kernel.SingleFieldManager.preFlush(SingleFieldManager.java:575)
>       at 
> org.apache.openjpa.kernel.SingleFieldManager.preFlush(SingleFieldManager.java:491)
>       at 
> org.apache.openjpa.kernel.StateManagerImpl.preFlush(StateManagerImpl.java:2956)
>       at org.apache.openjpa.kernel.PNewState.beforeFlush(PNewState.java:40)
>       at 
> org.apache.openjpa.kernel.StateManagerImpl.beforeFlush(StateManagerImpl.java:1048)
>       at org.apache.openjpa.kernel.BrokerImpl.flush(BrokerImpl.java:2051)
>       at org.apache.openjpa.kernel.BrokerImpl.flushSafe(BrokerImpl.java:2011)
>       at org.apache.openjpa.kernel.BrokerImpl.flush(BrokerImpl.java:1782)
>       at 
> org.apache.openjpa.kernel.DelegatingBroker.flush(DelegatingBroker.java:1015)
>       at 
> org.apache.openjpa.persistence.EntityManagerImpl.flush(EntityManagerImpl.java:637)
>       ... etc ...
> Here com.example.USAddress is a class which implements the BaseAddress 
> interface.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to