On Tuesday 02 November 2010 20:09:53 Lukas Fleischer wrote:
> On Tue, Nov 02, 2010 at 05:47:12PM +0100, Andrea Scarpino wrote:
> >  <?php
> > 
> > -           $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

Reply via email to