Hi Abe, On Apr 7, 2008, at 8:57 AM, Abe White wrote:
I believe parts of OpenJPA may rely on being able to stringify an id and then reconstruct it using the id class's Class,string constructor. (Andif not OpenJPA, then certainly Kodo's JDO bindings).
Right, so the String should be the more detailed String but with the actual subclass name instead of the superclass.
So isn't this a bug, for which we would dearly love to see a patch? CraigP.S. As Patrick pointed out, anyone can play with the code via svn and see where the problem is and how to fix it.
It might be good to start with a test case and we can discuss whether the behavior currently implemented is correctly designed.
On Fri, 2008-04-04 at 17:32 -0700, [EMAIL PROTECTED] wrote:I dont think anyone responded to this request yet - so I am asking againI would like OpenJPA to remove all the toString() method on all childrenclass of OpenJPAId.Reason: OpenJPAId class already has the toString() and it handles it well.It puts the type and the id together - like this: com.myCompany.Account-2345This is very useful behavior when managing a cache (or even looking intothe log file).Currently all the children have their own toString() which just outputs the id - This is not useful becuase it is not possible to distinguish between the types inside one cache. All types in the cache are DataCachePCData.FredericPS: Can someone give me check in right in the code base... ;) - Have a goodweek end everyone.Notice: This email message, together with any attachments, may contain information of BEA Systems, Inc., its subsidiaries and affiliated entities, that may be confidential, proprietary, copyrighted and/or legally privileged, and is intended solely for the use of the individual or entity named in this message. If you are not the intended recipient, and have received this message in error, please immediately return this by email and then delete it.
Craig Russell Architect, Sun Java Enterprise System http://java.sun.com/products/jdo 408 276-5638 mailto:[EMAIL PROTECTED] P.S. A good JDO? O, Gasp!
smime.p7s
Description: S/MIME cryptographic signature
