mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-13 10:22:42 +01:00
b3a63a62a2
Summary: It's dumb to execute a query which we know will return an empty result. Test Plan: Looked at comment preview with "11", didn't see "1 = 0" in DarkConsole. Reviewers: epriestley Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D5177
110 lines
2.6 KiB
PHP
110 lines
2.6 KiB
PHP
<?php
|
|
|
|
/**
|
|
* @task config Configuring the Query
|
|
* @task exec Query Execution
|
|
*/
|
|
final class PhabricatorNotificationQuery
|
|
extends PhabricatorCursorPagedPolicyAwareQuery {
|
|
|
|
private $userPHID;
|
|
private $keys;
|
|
private $unread;
|
|
|
|
|
|
/* -( Configuring the Query )---------------------------------------------- */
|
|
|
|
|
|
public function setUserPHID($user_phid) {
|
|
$this->userPHID = $user_phid;
|
|
return $this;
|
|
}
|
|
|
|
public function withKeys(array $keys) {
|
|
$this->keys = $keys;
|
|
return $this;
|
|
}
|
|
|
|
|
|
/**
|
|
* Filter results by read/unread status. Note that `true` means to return
|
|
* only unread notifications, while `false` means to return only //read//
|
|
* notifications. The default is `null`, which returns both.
|
|
*
|
|
* @param mixed True or false to filter results by read status. Null to remove
|
|
* the filter.
|
|
* @return this
|
|
* @task config
|
|
*/
|
|
public function withUnread($unread) {
|
|
$this->unread = $unread;
|
|
return $this;
|
|
}
|
|
|
|
|
|
/* -( Query Execution )---------------------------------------------------- */
|
|
|
|
|
|
protected function loadPage() {
|
|
if (!$this->userPHID) {
|
|
throw new Exception("Call setUser() before executing the query");
|
|
}
|
|
|
|
$story_table = new PhabricatorFeedStoryData();
|
|
$notification_table = new PhabricatorFeedStoryNotification();
|
|
|
|
$conn = $story_table->establishConnection('r');
|
|
|
|
$data = queryfx_all(
|
|
$conn,
|
|
"SELECT story.*, notif.hasViewed FROM %T notif
|
|
JOIN %T story ON notif.chronologicalKey = story.chronologicalKey
|
|
%Q
|
|
ORDER BY notif.chronologicalKey DESC
|
|
%Q",
|
|
$notification_table->getTableName(),
|
|
$story_table->getTableName(),
|
|
$this->buildWhereClause($conn),
|
|
$this->buildLimitClause($conn));
|
|
|
|
$viewed_map = ipull($data, 'hasViewed', 'chronologicalKey');
|
|
|
|
$stories = PhabricatorFeedStory::loadAllFromRows(
|
|
$data,
|
|
$this->getViewer());
|
|
|
|
foreach ($stories as $key => $story) {
|
|
$story->setHasViewed($viewed_map[$key]);
|
|
}
|
|
|
|
return $stories;
|
|
}
|
|
|
|
private function buildWhereClause(AphrontDatabaseConnection $conn_r) {
|
|
$where = array();
|
|
|
|
if ($this->userPHID) {
|
|
$where[] = qsprintf(
|
|
$conn_r,
|
|
'notif.userPHID = %s',
|
|
$this->userPHID);
|
|
}
|
|
|
|
if ($this->unread !== null) {
|
|
$where[] = qsprintf(
|
|
$conn_r,
|
|
'notif.hasViewed = %d',
|
|
(int)!$this->unread);
|
|
}
|
|
|
|
if ($this->keys) {
|
|
$where[] = qsprintf(
|
|
$conn_r,
|
|
'notif.chronologicalKey IN (%Ls)',
|
|
$this->keys);
|
|
}
|
|
|
|
return $this->formatWhereClause($where);
|
|
}
|
|
|
|
}
|