revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-26 08:42:41 +01:00
Code Issues Releases Wiki Activity

Fix a small bug with empty user queries

Browse source 
Summary: This does the wrong thing (fatals) if there are no passed PHIDs.

Test Plan: No more fatal.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Differential Revision: https://secure.phabricator.com/D5940
This commit is contained in:
epriestley 2013-05-17 03:55:45 -07:00
parent c967141f92
commit 27a0265367
1 changed files with 5 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/applications/people/storage/PhabricatorUserStatus.php
View file

@ -47,9 +47,14 @@ final class PhabricatorUserStatus extends PhabricatorUserDAO {
} }
public function loadCurrentStatuses($user_phids) { public function loadCurrentStatuses($user_phids) {
if (!$user_phids) {
return array();
}
$statuses = $this->loadAllWhere( $statuses = $this->loadAllWhere(
'userPHID IN (%Ls) AND UNIX_TIMESTAMP() BETWEEN dateFrom AND dateTo', 'userPHID IN (%Ls) AND UNIX_TIMESTAMP() BETWEEN dateFrom AND dateTo',
$user_phids); $user_phids);
return mpull($statuses, null, 'getUserPHID'); return mpull($statuses, null, 'getUserPHID');
} }