On Jun 30, 2008, at 2:13 PM, Paul McNett wrote:

> Are you of the opinion that we should assert isinstance(val, int) in
> _setOrder?

        I don't see what the assertion buys us. If a non-int is passed, we  
get a failure either way.

        If the column's Order value is not getting restored properly, this is  
something that needs to be investigated. It's been working great all  
along, and now we get rid of silent error suppression, and it throws  
an error. That tells me that there is a long-hidden bug that needs to  
be fixed; that bug is either in the code itself, or one of the  
exception handlers. Since the cdxml code was running correctly before,  
odds are that it is in the exception handlers.


-- Ed Leafe





_______________________________________________
Post Messages to: [email protected]
Subscription Maintenance: http://leafe.com/mailman/listinfo/dabo-dev
Searchable Archives: http://leafe.com/archives/search/dabo-dev
This message: http://leafe.com/archives/byMID/[EMAIL PROTECTED]

Reply via email to