Page Menu
Home
Phabricator
Search
Configure Global Search
Log In
Files
F6330212
raw.txt
No One
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Award Token
Flag For Later
Authored By
jbrownEP
Apr 2 2019, 1:16 AM
2019-04-02 01:16:51 (UTC+0)
Size
2 KB
Referenced Files
None
Subscribers
None
raw.txt
View Options
diff --git a/src/applications/dashboard/paneltype/PhabricatorDashboardQueryPanelType.php b/src/applications/dashboard/paneltype/PhabricatorDashboardQueryPanelType.php
index a71263b27..95324ef26 100644
--- a/src/applications/dashboard/paneltype/PhabricatorDashboardQueryPanelType.php
+++ b/src/applications/dashboard/paneltype/PhabricatorDashboardQueryPanelType.php
@@ -136,30 +136,32 @@ final class PhabricatorDashboardQueryPanelType
$results_view->setContent($content);
}
- if ($pager->getHasMoreResults()) {
- $item_list = $results_view->getObjectList();
-
- $more_href = $engine->getQueryResultsPageURI($key);
- if ($item_list) {
- $item_list->newTailButton()
- ->setHref($more_href);
- } else {
- // For search engines that do not return an object list, add a fake
- // one to the end so we can render a "View All Results" button that
- // looks like it does in normal applications. At time of writing,
- // several major applications like Maniphest (which has group headers)
- // and Feed (which uses custom rendering) don't return simple lists.
-
- $content = $results_view->getContent();
-
- $more_list = id(new PHUIObjectItemListView())
- ->setAllowEmptyList(true);
-
- $more_list->newTailButton()
- ->setHref($more_href);
-
- $content = array($content, $more_list);
- $results_view->setContent($content);
+ if (!$engine->shouldUseOffsetPaging()) {
+ if ($pager->getHasMoreResults()) {
+ $item_list = $results_view->getObjectList();
+
+ $more_href = $engine->getQueryResultsPageURI($key);
+ if ($item_list) {
+ $item_list->newTailButton()
+ ->setHref($more_href);
+ } else {
+ // For search engines that do not return an object list, add a fake
+ // one to the end so we can render a "View All Results" button that
+ // looks like it does in normal applications. At time of writing,
+ // several major applications like Maniphest (which has group headers)
+ // and Feed (which uses custom rendering) don't return simple lists.
+
+ $content = $results_view->getContent();
+
+ $more_list = id(new PHUIObjectItemListView())
+ ->setAllowEmptyList(true);
+
+ $more_list->newTailButton()
+ ->setHref($more_href);
+
+ $content = array($content, $more_list);
+ $results_view->setContent($content);
+ }
}
}
File Metadata
Details
Attached
Mime Type
text/plain; charset=utf-8
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
1531555
Default Alt Text
raw.txt (2 KB)
Attached To
Mode
P2115 panel_fix_pagination.patch
Attached
Detach File
Event Timeline
Log In to Comment