Homec4science

Correct `notification.pid` to `notification.pidfile`

Authored by Elias Probst <mail@eliasprobst.eu> on Feb 14 2015, 23:33.

Description

Correct notification.pid to notification.pidfile

Summary:
The correct conf variable is notification.pidfile, not notification.pid.
See also:

  • src/applications/config/option/PhabricatorNotificationConfigOptions.php:58
  • src/applications/aphlict/management/PhabricatorAphlictManagementWorkflow.php:34

Test Plan:
Grepped the whole source of Phabricator for occurences of notification.pid,
but all matches pointed to notification.pidfile instead.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: epriestley

Differential Revision: https://secure.phabricator.com/D11768

Details

Committed
epriestley <git@epriestley.com>Feb 14 2015, 23:33
Pushed
aubortJan 31 2017, 17:16
Parents
rPH05377ef48c7a: Expand Subscription handles slightly
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHa8cc1c05e528: Correct `notification.pid` to `notification.pidfile` (authored by Elias Probst <mail@eliasprobst.eu>).Feb 14 2015, 23:33