launchpad-dev team mailing list archive

[James Westby <jw+debian@xxxxxxxxxxxxxxx>]

On Sun, 22 Aug 2010 20:17:40 +1200, Robert Collins <robert.collins@xxxxxxxxxxxxx> wrote:
> Amongst other migration issues, __nonzero__ on SQLObjectResultSet
> makes converting SQLObject to Storm code tricky, because its hard to
> grep for, silently fails (bool(now-storm-resultset) -> always true).

Just to note that since the last storm release you can now use
.is_empty() on either result set, with the same behaviour.

I would suggest that no new code does bool(SQLObjectResultSet), and
people convert any instances that they see to .is_empty().

That will allow a smoother migration, without having to port every use
by changing the base class.

Thanks,

James