Page MenuHomePhabricator
Files F15459149

D10822.id25977.diff
No OneTemporary
Actions

D10822.id25977.diff
View Options

diff --git a/src/applications/phriction/controller/PhrictionDocumentController.php b/src/applications/phriction/controller/PhrictionDocumentController.php
--- a/src/applications/phriction/controller/PhrictionDocumentController.php
+++ b/src/applications/phriction/controller/PhrictionDocumentController.php
@@ -358,33 +358,23 @@
}
private function renderDocumentChildren($slug) {
- $document_dao = new PhrictionDocument();
- $content_dao = new PhrictionContent();
- $conn = $document_dao->establishConnection('r');
- $limit = 250;
$d_child = PhabricatorSlug::getDepth($slug) + 1;
$d_grandchild = PhabricatorSlug::getDepth($slug) + 2;
+ $limit = 250;
- // Select children and grandchildren.
- $children = queryfx_all(
- $conn,
- 'SELECT d.slug, d.depth, c.title FROM %T d JOIN %T c
- ON d.contentID = c.id
- WHERE d.slug LIKE %> AND d.depth IN (%d, %d)
- AND d.status IN (%Ld)
- ORDER BY d.depth, c.title LIMIT %d',
- $document_dao->getTableName(),
- $content_dao->getTableName(),
- ($slug == '/' ? '' : $slug),
- $d_child,
- $d_grandchild,
- array(
+ $query = id(new PhrictionDocumentQuery())
+ ->setViewer($this->getRequest()->getUser())
+ ->withDepths(array($d_child, $d_grandchild))
+ ->withSlugPrefix($slug == '/' ? '' : $slug)
+ ->withStatuses(array(
PhrictionDocumentStatus::STATUS_EXISTS,
PhrictionDocumentStatus::STATUS_STUB,
- ),
- $limit);
+ ))
+ ->setLimit($limit)
+ ->needContent(true);
+ $children = $query->execute();
if (!$children) {
return;
}
@@ -405,7 +395,7 @@
if (count($children) == $limit) {
$more_children = true;
foreach ($children as $child) {
- if ($child['depth'] == $d_grandchild) {
+ if ($child->getDepth() == $d_grandchild) {
$more_children = false;
}
}
@@ -415,24 +405,30 @@
$more_children = false;
}
- $grandchildren = array();
+ $children_dicts = array();
+ $grandchildren_dicts = array();
foreach ($children as $key => $child) {
- if ($child['depth'] == $d_child) {
+ $child_dict = array(
+ 'slug' => $child->getSlug(),
+ 'depth' => $child->getDepth(),
+ 'title' => $child->getContent()->getTitle(),);
+ if ($child->getDepth() == $d_child) {
+ $children_dicts[] = $child_dict;
continue;
} else {
unset($children[$key]);
if ($show_grandchildren) {
- $ancestors = PhabricatorSlug::getAncestry($child['slug']);
- $grandchildren[end($ancestors)][] = $child;
+ $ancestors = PhabricatorSlug::getAncestry($child->getSlug());
+ $grandchildren_dicts[end($ancestors)][] = $child_dict;
}
}
}
// Fill in any missing children.
- $known_slugs = ipull($children, null, 'slug');
- foreach ($grandchildren as $slug => $ignored) {
+ $known_slugs = mpull($children, null, 'getSlug');
+ foreach ($grandchildren_dicts as $slug => $ignored) {
if (empty($known_slugs[$slug])) {
- $children[] = array(
+ $children_dicts[] = array(
'slug' => $slug,
'depth' => $d_child,
'title' => PhabricatorSlug::getDefaultTitle($slug),
@@ -441,13 +437,13 @@
}
}
- $children = isort($children, 'title');
+ $children_dicts = isort($children_dicts, 'title');
$list = array();
- foreach ($children as $child) {
+ foreach ($children_dicts as $child) {
$list[] = hsprintf('<li>');
$list[] = $this->renderChildDocumentLink($child);
- $grand = idx($grandchildren, $child['slug'], array());
+ $grand = idx($grandchildren_dicts, $child['slug'], array());
if ($grand) {
$list[] = hsprintf('<ul>');
foreach ($grand as $grandchild) {
diff --git a/src/applications/phriction/query/PhrictionDocumentQuery.php b/src/applications/phriction/query/PhrictionDocumentQuery.php
--- a/src/applications/phriction/query/PhrictionDocumentQuery.php
+++ b/src/applications/phriction/query/PhrictionDocumentQuery.php
@@ -6,6 +6,10 @@
private $ids;
private $phids;
private $slugs;
+ private $depths;
+ private $slugPrefix;
+ private $statuses;
+
private $needContent;
@@ -33,6 +37,21 @@
return $this;
}
+ public function withDepths(array $depths) {
+ $this->depths = $depths;
+ return $this;
+ }
+
+ public function withSlugPrefix($slug_prefix) {
+ $this->slugPrefix = $slug_prefix;
+ return $this;
+ }
+
+ public function withStatuses(array $statuses) {
+ $this->statuses = $statuses;
+ return $this;
+ }
+
public function withStatus($status) {
$this->status = $status;
return $this;
@@ -182,6 +201,27 @@
$this->slugs);
}
+ if ($this->statuses) {
+ $where[] = qsprintf(
+ $conn,
+ 'status IN (%Ld)',
+ $this->statuses);
+ }
+
+ if ($this->slugPrefix) {
+ $where[] = qsprintf(
+ $conn,
+ 'slug LIKE %>',
+ $this->slugPrefix);
+ }
+
+ if ($this->depths) {
+ $where[] = qsprintf(
+ $conn,
+ 'depth IN (%Ld)',
+ $this->depths);
+ }
+
switch ($this->status) {
case self::STATUS_OPEN:
$where[] = qsprintf(

File Metadata

Mime Type
text/plain
Expires
Tue, Apr 1, 10:13 AM (3 d, 23 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7728820
Default Alt Text
D10822.id25977.diff (5 KB)

Event Timeline

Phabricator · Built by Phacility