Ed Leafe wrote: > 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.
The failure is at the point of the problem, making it very easy to understand what's going on. But my real question was more general: do we enforce correct types to be passed, or do we coerce to the expected type? > 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 Could be part of the silent-error removal, or it could be something else between 4180 and HEAD. I agree it is likely the former though. Paul _______________________________________________ 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]
