@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 exception when trying to save global setting mail tags
Summary: Ref T11098. Template preferences don't have a user, but this codepath didn't get fully updated to account for that.
Test Plan: Saved mail tags in global prefernces.
Reviewers: avivey, chad
Reviewed By: chad
Maniphest Tasks: T11098
Differential Revision: https://secure.phabricator.com/D16050
+1
-1
+1
-1
src/applications/settings/panel/PhabricatorEmailPreferencesSettingsPanel.php
+1
-1
src/applications/settings/panel/PhabricatorEmailPreferencesSettingsPanel.php
···
160
160
return $editors;
161
161
}
162
162
163
-
private function getAllTags(PhabricatorUser $user) {
163
+
private function getAllTags(PhabricatorUser $user = null) {
164
164
$tags = array();
165
165
foreach ($this->getAllEditorsWithTags($user) as $editor) {
166
166
$tags += $editor->getMailTagsMap();