Oleg Broytmann wrote:
> Hello!
> 
> On Tue, Aug 15, 2006 at 04:26:35PM -0300, Johan Dahlin wrote:
>> InheritableSQLObject.addColumn is not updating sqlmeta.columns, meaning that
>> you cannot use sqlbuilder to access this column.
> 
>    It should not update columns list. It only should update the q-magic
> reference to the existing column.
> 
>> FWIW here's the patch.
> 
>    The patch is incorrect but it inspired me to dive into those waters
> deeper. I extended your test and found that addColumn() didn't do the
> q-magic and delColumn() didn't work at all. I fixed them both. Revisions
> 1988-1989.

Thanks

Okay, this seems to work for the use cases I submitted.
Recently I've begun to use selectBy quite a bit, which excepted the
attribute to be in the column list.

So, the following code is broken:

Employee.selectBy(person_attribute=...)

ForeignKey are also broken, neither obj or objID are queryable.

I'll see what I can do to solve that on top of your patch.

-- 
Johan Dahlin <[EMAIL PROTECTED]>
Async Open Source

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
sqlobject-discuss mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sqlobject-discuss

Reply via email to