Merge lp:~voluntatefaber/lingo-dictionary/recents-fixes into lp:lingo-dictionary

Proposed by Andrea Basso
Status: Merged
Approved by: Cody Garver
Approved revision: 127
Merged at revision: 153
Proposed branch: lp:~voluntatefaber/lingo-dictionary/recents-fixes
Merge into: lp:lingo-dictionary
Diff against target: 12 lines (+1/-1)
1 file modified
src/lingo-database.vala (+1/-1)
To merge this branch: bzr merge lp:~voluntatefaber/lingo-dictionary/recents-fixes
Reviewer Review Type Date Requested Status
Cody Garver (community) Approve
Review via email: mp+95754@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Cody Garver (codygarver) :
review: Approve
Revision history for this message
Cody Garver (codygarver) wrote :

This has been superseded by commit 152.

review: Disapprove
Revision history for this message
Cody Garver (codygarver) wrote :

Ignore that comment, it's on the wrong branch..

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/lingo-database.vala'
--- src/lingo-database.vala 2012-02-11 19:39:21 +0000
+++ src/lingo-database.vala 2012-03-03 22:37:37 +0000
@@ -303,7 +303,7 @@
303 execute_statement(stmt);303 execute_statement(stmt);
304304
305 // Deep magic begins here ...305 // Deep magic begins here ...
306 execute_sql ("DELETE FROM recent WHERE id = CASE WHEN (SELECT COUNT(word) FROM recent) > 10 THEN (SELECT MIN(id) FROM recent) ELSE (SELECT (MIN(word) + 1) FROM recent) END");306 execute_sql ("DELETE FROM recent WHERE id = CASE WHEN (SELECT COUNT(word) FROM recent) > 10 THEN (SELECT MIN(id) FROM recent) ELSE (SELECT (-1) FROM recent) END");
307 }307 }
308308
309 public void clear_recents () {309 public void clear_recents () {

Subscribers

People subscribed via source and target branches

to all changes: