Yeah, that's it. For 5.4.3, that would fix this problem.

ms

On Feb 2, 2011, at 8:38 PM, Chuck Hill wrote:

> 
> On Feb 2, 2011, at 5:33 PM, Lachlan Deck wrote:
> 
>> On 03/02/2011, at 8:59 AM, Mike Schrag wrote:
>> 
>>> i still would like to see what entity.primaryKeyRootName returns for each 
>>> ... i don't recall what the impl was in 5.4.3 and I don't have it checked 
>>> out at the moment, but i'm pretty sure that method is the only thing that 
>>> matters here.
>> 
>> The impl in 5.4.3 is wrong. Essentially 5.4.3 introduced a regression, which 
>> I recall checking in a fix for in wonder,
> 
> I see this, but nothing else in ERXEntity related to this:
> 
>       public boolean hasExternalName() {
>               // (ldeck) radar://6592526 fix for 5.4.3 regression which 
> assumed that any parent entity that is abstract has no external name!
>               return externalName() != null && externalName().trim().length() 
> > 0;
>       }
> 
> 
> Chuck
> 
> 
>> and I believe it's fixed in 5.4.3.1 :) like this:
>> if (parent is abstract) {
>>      return my external name.
>> }
>> 
>> Which is just wrong. This was particularly a problem for Vertical 
>> Inheritance where various fields ought to have resolved to the parent 
>> entity's table (but didn't).
>> 
>> I've never tested horizontal inheritance but you can fix it by:
>> a) subclassing EOEntity and overriding primaryKeyRootName etc, or better
>> b) use Wonder and if Horizontal Inheritance is a problem, fix it there for 
>> all :)
>> 
>> with regards,
>> -
>> 
>> Lachlan Deck
>> 
>> 
>> 
>> _______________________________________________
>> Do not post admin requests to the list. They will be ignored.
>> Webobjects-dev mailing list      ([email protected])
>> Help/Unsubscribe/Update your Subscription:
>> http://lists.apple.com/mailman/options/webobjects-dev/chill%40global-village.net
>> 
>> This email sent to [email protected]
> 
> -- 
> Chuck Hill             Senior Consultant / VP Development
> 
> Practical WebObjects - for developers who want to increase their overall 
> knowledge of WebObjects or who are trying to solve specific problems.    
> http://www.global-village.net/products/practical_webobjects
> 
> 
> 
> 
> 
> 
> 
> _______________________________________________
> Do not post admin requests to the list. They will be ignored.
> Webobjects-dev mailing list      ([email protected])
> Help/Unsubscribe/Update your Subscription:
> http://lists.apple.com/mailman/options/webobjects-dev/mschrag%40pobox.com
> 
> This email sent to [email protected]

 _______________________________________________
Do not post admin requests to the list. They will be ignored.
Webobjects-dev mailing list      ([email protected])
Help/Unsubscribe/Update your Subscription:
http://lists.apple.com/mailman/options/webobjects-dev/archive%40mail-archive.com

This email sent to [email protected]

Reply via email to