← Back to team overview

zeitgeist team mailing list archive

Re: [Merge] lp:~thekorn/zeitgeist/fix-641100-min-max-use-index into lp:zeitgeist

 

Mikkel, thanks for doing the review,
yes, the case of an empty db is covered by our unittests and everything works just fine.

(
MIN/MAX always return sth. even if the table it's operating on is empty

In [7]: con = sqlite3.connect(":memory:")

In [8]: con.execute("CREATE TABLE boo(x)")
Out[8]: <sqlite3.Cursor object at 0x117da28>

In [9]: con.execute("SELECT MIN(x) FROM boo").fetchone()
Out[9]: (None,)

)
-- 
https://code.launchpad.net/~thekorn/zeitgeist/fix-641100-min-max-use-index/+merge/35786
Your team Zeitgeist Framework Team is requested to review the proposed merge of lp:~thekorn/zeitgeist/fix-641100-min-max-use-index into lp:zeitgeist.



References