zeitgeist team mailing list archive

[Seif Lotfy <seif@xxxxxxxxx>]

The proposal to merge lp:~seif/zeitgeist/fix-redundant-statement into lp:zeitgeist has been updated.

Description changed to:

Basically I found this statement. For me we could get rid of one of them. Right now using only one statement of both works for all our test cases.

if return_mode == 0:
    sql = "SELECT DISTINCT id FROM event_view"
elif return_mode == 1:
    sql = "SELECT id FROM event_view"

Do we really need both? In any case here is a little merge proposal


-- 
https://code.launchpad.net/~seif/zeitgeist/fix-redundant-statement/+merge/41751
Your team Zeitgeist Framework Team is requested to review the proposed merge of lp:~seif/zeitgeist/fix-redundant-statement into lp:zeitgeist.