mirror of
https://we.phorge.it/source/phorge.git
synced 2024-12-30 01:10:58 +01:00
c1c897b961
Summary: Ref T8424. No UI or interesting behavior yet, but integrates Spaces checks: - `PolicyFilter` now checks Spaces. - `PolicyAwareQuery` now automatically adds Spaces constraints. There's one interesting design decision here: **spaces are stronger than automatic capabilities**. That means that you can't see a task in a space you don't have permission to access, //even if you are the owner//. I //think// this is desirable. Particularly, we need to do this in order to exclude objects at the query level, which potentially makes policy filtering for spaces hugely more efficient. I also like Spaces being very strong, conceptually. It's possible that we might want to change this; this would reduce our access to optimizations but might be a little friendlier or make more sense to users later on. For now, at least, I'm pursuing the more aggressive line. If we stick with this, we probably need to make some additional UI affordances (e.g., show when an owner can't see a task). This also means that you get a hard 404 instead of a policy exception when you try to access something in a space you can't see. I'd slightly prefer to show you a policy exception instead, but think this is generally a reasonable tradeoff to get the high-performance filtering at the Query layer. Test Plan: - Added and executed unit tests. - Put objects in spaces and viewed them with multiple users. - Made the default space visible/invisible, viewed objects. - Checked the services panel and saw `spacePHID` constraints. - Verified that this adds only one query to each page. Reviewers: btrahan, chad Reviewed By: btrahan Subscribers: chad, epriestley Maniphest Tasks: T8424 Differential Revision: https://secure.phabricator.com/D13156
271 lines
6.1 KiB
PHP
271 lines
6.1 KiB
PHP
<?php
|
|
|
|
final class PhabricatorPasteQuery
|
|
extends PhabricatorCursorPagedPolicyAwareQuery {
|
|
|
|
private $ids;
|
|
private $phids;
|
|
private $authorPHIDs;
|
|
private $parentPHIDs;
|
|
|
|
private $needContent;
|
|
private $needRawContent;
|
|
private $languages;
|
|
private $includeNoLanguage;
|
|
private $dateCreatedAfter;
|
|
private $dateCreatedBefore;
|
|
|
|
public function withIDs(array $ids) {
|
|
$this->ids = $ids;
|
|
return $this;
|
|
}
|
|
|
|
public function withPHIDs(array $phids) {
|
|
$this->phids = $phids;
|
|
return $this;
|
|
}
|
|
|
|
public function withAuthorPHIDs(array $phids) {
|
|
$this->authorPHIDs = $phids;
|
|
return $this;
|
|
}
|
|
|
|
public function withParentPHIDs(array $phids) {
|
|
$this->parentPHIDs = $phids;
|
|
return $this;
|
|
}
|
|
|
|
public function needContent($need_content) {
|
|
$this->needContent = $need_content;
|
|
return $this;
|
|
}
|
|
|
|
public function needRawContent($need_raw_content) {
|
|
$this->needRawContent = $need_raw_content;
|
|
return $this;
|
|
}
|
|
|
|
public function withLanguages(array $languages) {
|
|
$this->includeNoLanguage = false;
|
|
foreach ($languages as $key => $language) {
|
|
if ($language === null) {
|
|
$languages[$key] = '';
|
|
continue;
|
|
}
|
|
}
|
|
$this->languages = $languages;
|
|
return $this;
|
|
}
|
|
|
|
public function withDateCreatedBefore($date_created_before) {
|
|
$this->dateCreatedBefore = $date_created_before;
|
|
return $this;
|
|
}
|
|
|
|
public function withDateCreatedAfter($date_created_after) {
|
|
$this->dateCreatedAfter = $date_created_after;
|
|
return $this;
|
|
}
|
|
|
|
protected function newResultObject() {
|
|
return new PhabricatorPaste();
|
|
}
|
|
|
|
protected function loadPage() {
|
|
$table = new PhabricatorPaste();
|
|
$conn_r = $table->establishConnection('r');
|
|
|
|
$data = queryfx_all(
|
|
$conn_r,
|
|
'SELECT paste.* FROM %T paste %Q %Q %Q',
|
|
$table->getTableName(),
|
|
$this->buildWhereClause($conn_r),
|
|
$this->buildOrderClause($conn_r),
|
|
$this->buildLimitClause($conn_r));
|
|
|
|
$pastes = $table->loadAllFromArray($data);
|
|
|
|
return $pastes;
|
|
}
|
|
|
|
protected function didFilterPage(array $pastes) {
|
|
if ($this->needRawContent) {
|
|
$pastes = $this->loadRawContent($pastes);
|
|
}
|
|
|
|
if ($this->needContent) {
|
|
$pastes = $this->loadContent($pastes);
|
|
}
|
|
|
|
return $pastes;
|
|
}
|
|
|
|
protected function buildWhereClauseParts(AphrontDatabaseConnection $conn) {
|
|
$where = parent::buildWhereClauseParts($conn);
|
|
|
|
if ($this->ids) {
|
|
$where[] = qsprintf(
|
|
$conn,
|
|
'id IN (%Ld)',
|
|
$this->ids);
|
|
}
|
|
|
|
if ($this->phids) {
|
|
$where[] = qsprintf(
|
|
$conn,
|
|
'phid IN (%Ls)',
|
|
$this->phids);
|
|
}
|
|
|
|
if ($this->authorPHIDs) {
|
|
$where[] = qsprintf(
|
|
$conn,
|
|
'authorPHID IN (%Ls)',
|
|
$this->authorPHIDs);
|
|
}
|
|
|
|
if ($this->parentPHIDs) {
|
|
$where[] = qsprintf(
|
|
$conn,
|
|
'parentPHID IN (%Ls)',
|
|
$this->parentPHIDs);
|
|
}
|
|
|
|
if ($this->languages) {
|
|
$where[] = qsprintf(
|
|
$conn,
|
|
'language IN (%Ls)',
|
|
$this->languages);
|
|
}
|
|
|
|
if ($this->dateCreatedAfter) {
|
|
$where[] = qsprintf(
|
|
$conn,
|
|
'dateCreated >= %d',
|
|
$this->dateCreatedAfter);
|
|
}
|
|
|
|
if ($this->dateCreatedBefore) {
|
|
$where[] = qsprintf(
|
|
$conn,
|
|
'dateCreated <= %d',
|
|
$this->dateCreatedBefore);
|
|
}
|
|
|
|
return $where;
|
|
}
|
|
|
|
private function getContentCacheKey(PhabricatorPaste $paste) {
|
|
return implode(
|
|
':',
|
|
array(
|
|
'P'.$paste->getID(),
|
|
$paste->getFilePHID(),
|
|
$paste->getLanguage(),
|
|
));
|
|
}
|
|
|
|
private function loadRawContent(array $pastes) {
|
|
$file_phids = mpull($pastes, 'getFilePHID');
|
|
$files = id(new PhabricatorFileQuery())
|
|
->setParentQuery($this)
|
|
->setViewer($this->getViewer())
|
|
->withPHIDs($file_phids)
|
|
->execute();
|
|
$files = mpull($files, null, 'getPHID');
|
|
|
|
foreach ($pastes as $key => $paste) {
|
|
$file = idx($files, $paste->getFilePHID());
|
|
if (!$file) {
|
|
unset($pastes[$key]);
|
|
continue;
|
|
}
|
|
try {
|
|
$paste->attachRawContent($file->loadFileData());
|
|
} catch (Exception $ex) {
|
|
// We can hit various sorts of file storage issues here. Just drop the
|
|
// paste if the file is dead.
|
|
unset($pastes[$key]);
|
|
continue;
|
|
}
|
|
}
|
|
|
|
return $pastes;
|
|
}
|
|
|
|
private function loadContent(array $pastes) {
|
|
$cache = new PhabricatorKeyValueDatabaseCache();
|
|
|
|
$cache = new PhutilKeyValueCacheProfiler($cache);
|
|
$cache->setProfiler(PhutilServiceProfiler::getInstance());
|
|
|
|
$keys = array();
|
|
foreach ($pastes as $paste) {
|
|
$keys[] = $this->getContentCacheKey($paste);
|
|
}
|
|
|
|
$caches = $cache->getKeys($keys);
|
|
|
|
$need_raw = array();
|
|
$have_cache = array();
|
|
foreach ($pastes as $paste) {
|
|
$key = $this->getContentCacheKey($paste);
|
|
if (isset($caches[$key])) {
|
|
$paste->attachContent(phutil_safe_html($caches[$key]));
|
|
$have_cache[$paste->getPHID()] = true;
|
|
} else {
|
|
$need_raw[$key] = $paste;
|
|
}
|
|
}
|
|
|
|
if (!$need_raw) {
|
|
return $pastes;
|
|
}
|
|
|
|
$write_data = array();
|
|
|
|
$have_raw = $this->loadRawContent($need_raw);
|
|
$have_raw = mpull($have_raw, null, 'getPHID');
|
|
foreach ($pastes as $key => $paste) {
|
|
$paste_phid = $paste->getPHID();
|
|
if (isset($have_cache[$paste_phid])) {
|
|
continue;
|
|
}
|
|
|
|
if (empty($have_raw[$paste_phid])) {
|
|
unset($pastes[$key]);
|
|
continue;
|
|
}
|
|
|
|
$content = $this->buildContent($paste);
|
|
$paste->attachContent($content);
|
|
$write_data[$this->getContentCacheKey($paste)] = (string)$content;
|
|
}
|
|
|
|
if ($write_data) {
|
|
$cache->setKeys($write_data);
|
|
}
|
|
|
|
return $pastes;
|
|
}
|
|
|
|
private function buildContent(PhabricatorPaste $paste) {
|
|
$language = $paste->getLanguage();
|
|
$source = $paste->getRawContent();
|
|
|
|
if (empty($language)) {
|
|
return PhabricatorSyntaxHighlighter::highlightWithFilename(
|
|
$paste->getTitle(),
|
|
$source);
|
|
} else {
|
|
return PhabricatorSyntaxHighlighter::highlightWithLanguage(
|
|
$language,
|
|
$source);
|
|
}
|
|
}
|
|
|
|
public function getQueryApplicationClass() {
|
|
return 'PhabricatorPasteApplication';
|
|
}
|
|
|
|
}
|