@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Default statuses and priorities to `array()` correctly
Summary: Fixes T7904. Builtin queries won't set these to anything.
Test Plan: "Authored" builtin works again.
Reviewers: btrahan, chad
Reviewed By: chad
Subscribers: epriestley
Maniphest Tasks: T7904
Differential Revision: https://secure.phabricator.com/D12539
+2
-2
+2
-2
src/applications/maniphest/query/ManiphestTaskSearchEngine.php
+2
-2
src/applications/maniphest/query/ManiphestTaskSearchEngine.php
···
141
141
142
142
$datasource = id(new ManiphestTaskStatusFunctionDatasource())
143
143
->setViewer($this->requireViewer());
144
-
$statuses = $saved->getParameter('statuses');
144
+
$statuses = $saved->getParameter('statuses', array());
145
145
$statuses = $datasource->evaluateTokens($statuses);
146
146
if ($statuses) {
147
147
$query->withStatuses($statuses);
148
148
}
149
149
150
-
$priorities = $saved->getParameter('priorities');
150
+
$priorities = $saved->getParameter('priorities', array());
151
151
if ($priorities) {
152
152
$query->withPriorities($priorities);
153
153
}