[jira] Closed: (OPENJPA-1823) Is merge() seriously broken?

["Pinaki Poddar (JIRA)" <ji...@apache.org>]

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

Pinaki Poddar closed OPENJPA-1823.
----------------------------------

    Resolution: Invalid

My silly bad. Ignore.

> Is merge() seriously broken?
> ----------------------------
>
>                 Key: OPENJPA-1823
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-1823
>             Project: OpenJPA
>          Issue Type: Bug
>          Components: Enhance, kernel
>            Reporter: Pinaki Poddar
>
> Does someone recently rewrite the critical method pcIsDetached() in PCEnhancer and broke it?
> This is a piece of code in  isDetached(Object obj)  method of BrokerImpl
>        PersistenceCapable pc = ...;
>         Boolean detached = pc.pcIsDetached();
>         if (detached != null)
>             return detached.booleanValue();
> A enhanced PersistenceCapable has the following in its pcIsDetached()
> public Boolean pcIsDetached()
>     {
>         if(pcStateManager != null)
>             if(pcStateManager.isDetached())
>                 return Boolean.TRUE;
>             else
>                 return Boolean.FALSE;
>         if(pcGetDetachedState() != null && pcGetDetachedState() != PersistenceCapable.DESERIALIZED)
>             return Boolean.TRUE;
>         if(version != 0)
>             return Boolean.TRUE;
>         if(pcVersionInit)
>         {
>             return Boolean.TRUE;
>         } else
>         {
>             return Boolean.FALSE;
>         }
>     }
> The method never returns null. But that is what BrokerImpl checks for.
> Does someone recently rewrite this critical method in PCEnhancer and broke it?

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