[aur-dev] [PATCH] Do not display current votes in All Votes
Subject says everything. -- Andrea Scarpino Arch Linux Developer
On Tue, Nov 02, 2010 at 05:47:12PM +0100, Andrea Scarpino wrote:
$type = __("All Votes");
Looks good in general but why should current votes not be listed in the "All Votes" section? I'd suggest renaming "All Votes" to "Closed Votes" or something similar then.
On Tuesday 02 November 2010 20:09:53 Lukas Fleischer wrote:
On Tue, Nov 02, 2010 at 05:47:12PM +0100, Andrea Scarpino wrote:
- $q = "SELECT * FROM TU_VoteInfo ORDER BY Submitted " . $order . $lim; + $q = "SELECT * FROM TU_VoteInfo WHERE End < " . time() . " ORDER BY Submitted " . $order . $lim;
$result = db_query($q, $dbh);
$type = __("All Votes");
Looks good in general but why should current votes not be listed in the "All Votes" section? I'd suggest renaming "All Votes" to "Closed Votes" or something similar then. You are right. I suggest to rename it as "Closed Votes" and apply this patch.
-- Andrea Scarpino Arch Linux Developer
On Tuesday 02 November 2010 20:39:14 Andrea Scarpino wrote:
You are right. I suggest to rename it as "Closed Votes" and apply this patch. Do not display current votes in All Votes and rename it as Past Votes.
From d417ea0ba7b48fe685b1932c5bda7acbe0230e7f Mon Sep 17 00:00:00 2001
From: Andrea Scarpino
On Thu 18 Nov 2010 09:52 +0100, Andrea Scarpino wrote:
On Tuesday 02 November 2010 20:39:14 Andrea Scarpino wrote:
You are right. I suggest to rename it as "Closed Votes" and apply this patch. Do not display current votes in All Votes and rename it as Past Votes.
From d417ea0ba7b48fe685b1932c5bda7acbe0230e7f Mon Sep 17 00:00:00 2001 From: Andrea Scarpino
Date: Thu, 18 Nov 2010 09:49:37 +0100 Subject: [PATCH] Do not display current votes in All Votes, and rename it as Thanks. I pushed this.
participants (3)
-
Andrea Scarpino
-
Loui Chang
-
Lukas Fleischer