Forum OpenACS Development: Re: should the supertype of 'acs_object' be 'acs_object' ?

Collapse
2: Re: should the supertype of 'acs_object' be 'acs_object' ? (response to 1)
Posted by Malte Sussdorff on 07/12/04 10:55 AM
We should not break things for existing packages, but adding a flag / special reserved word for a "null" supertype would be okay I think.

What I wonder is this: Which types of objects should *not* have acs_object as a supertype. And why does the object type hierarchy not sense (acs_object is the root type) ?

Collapse
3: Re: should the supertype of 'acs_object' be 'acs_object' ? (response to 2)
Posted by russ m on 07/12/04 11:28 AM
Which types of objects should *not* have acs_object as a supertype
*acs_object* itself should not have acs_object as it's supertype. acs_object should have a null supertype.
And why does the object type hierarchy not sense
Because it doesn't currently have a well-defined root. Do a
select object_type, supertype from acs_object_types;
and then try manually climbing the object hierarchy. How do you know that you've reached the root?
Collapse
4: Re: should the supertype of 'acs_object' be 'acs_object' ? (response to 3)
Posted by Malte Sussdorff on 07/12/04 12:13 PM
I would do a "select object_type, supertype from acs_object_types where object_type != supertype" and then start climbing :). Okay, it is not semantically correct but should achieve the goal.

Either way, I'm fine with the proposal to change acs_object into having a "null" supertype, not sure though if the upgrade is as trivial as I'd hope. Let's wait and hear what the others say. This needs to be Tiped anyway in the end.

Collapse
5: Re: should the supertype of 'acs_object' be 'acs_object' ? (response to 4)
Posted by Jeff Davis on 07/12/04 01:13 PM
I think the reason the root supertype was not NULL was because oracle does not index nulls. The acs_object_types table is so small that it does not matter one way or the other though.

Russell, is there any reason you can't just use the code as is using supertype = type where you want to create a new object type root?

Collapse
6: Re: should the supertype of 'acs_object' be 'acs_object' ? (response to 5)
Posted by russ m on 07/12/04 01:56 PM
Jeff - I'm not wanting to create a new root to the object hierarchy, I'm just climbing from a specified object_type to the root to see what attributes the type and it's parents have. I can easily enough special-case 'acs_object' and stop climbing then (that is in fact what I'm doing now), but it seemed to me that the current situation was not semantically correct and was probably just an unforseen side-effect of making acs_object_type__create_type more "friendly".
Collapse
7: Re: should the supertype of 'acs_object' be 'acs_object' ? (response to 6)
Posted by Jeff Davis on 07/12/04 02:29 PM
Oh, I see.

Actually I was talking out my ass anyway, the reason it's not null is the pg port is borken with respect to the oracle version. On oracle the supertype param is defaulted to 'acs_object' but when acs_object is added as an object type the param is set explicitly to null.

On pg, you don't have parameter defaulting so it can't distinguish a supertype you want to have be null from one where you just want the default.

I think we should special case the acs_object object_type create to leave it null in pg as well.