@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
Fix an issue with handling of `null` feed URIs
Summary: Ref T8658. Caught this in the logs. This value may be set to `null`. Handle that gracefully.
Test Plan: Will check logs.
Reviewers: btrahan
Reviewed By: btrahan
Subscribers: epriestley
Maniphest Tasks: T8658
Differential Revision: https://secure.phabricator.com/D13432
+10
-7
+10
-7
src/applications/feed/worker/FeedPublisherWorker.php
+10
-7
src/applications/feed/worker/FeedPublisherWorker.php
···
6
6
$story = $this->loadFeedStory();
7
7
8
8
$uris = PhabricatorEnv::getEnvConfig('feed.http-hooks');
9
-
foreach ($uris as $uri) {
10
-
$this->queueTask(
11
-
'FeedPublisherHTTPWorker',
12
-
array(
13
-
'key' => $story->getChronologicalKey(),
14
-
'uri' => $uri,
15
-
));
9
+
10
+
if ($uris) {
11
+
foreach ($uris as $uri) {
12
+
$this->queueTask(
13
+
'FeedPublisherHTTPWorker',
14
+
array(
15
+
'key' => $story->getChronologicalKey(),
16
+
'uri' => $uri,
17
+
));
18
+
}
16
19
}
17
20
18
21
$argv = array(