Skip to content

Commit 121c37b

Browse files
committed
Fix an issue with reviewed_by user names incorrectly mapping to false
1 parent 1863ad5 commit 121c37b

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

index.php

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11
<?php
22
require_once('init.php');
33

4-
$users = Database::find('review')->query('SELECT DISTINCT IFNULL(reviewed_by, "") FROM '.Database::escapeField($reviewhost['review_table']));
4+
$users = Database::find('review')->query('SELECT DISTINCT reviewed_by FROM '.Database::escapeField($reviewhost['review_table']).' WHERE reviewed_by IS NOT NULL');
55
$Reviewers = "[ 'None' ";
6-
while($user = $users->fetch_col())
7-
$Reviewers .= ",'$user' ";
6+
while(($user = $users->fetch_col()) !== false)
7+
if (strlen($user))
8+
$Reviewers .= ",'$user' ";
89
$Reviewers .= " ]";
910
unset($users);
1011

0 commit comments

Comments
 (0)