From c648077625e3f95926c661783cead54e7eba48c9 Mon Sep 17 00:00:00 2001 From: epriestley Date: Wed, 17 Apr 2019 17:23:19 -0700 Subject: [PATCH] Don't warn about a locked database value after users run "bin/auth lock" Summary: Ref T7667. Also one text fix. Test Plan: Ran `bin/auth lock`, didn't get a replacement setup issue. Reviewers: amckinley Reviewed By: amckinley Maniphest Tasks: T7667 Differential Revision: https://secure.phabricator.com/D20447 --- .../config/check/PhabricatorExtraConfigSetupCheck.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/applications/config/check/PhabricatorExtraConfigSetupCheck.php b/src/applications/config/check/PhabricatorExtraConfigSetupCheck.php index b676063e8c..a8a883ca96 100644 --- a/src/applications/config/check/PhabricatorExtraConfigSetupCheck.php +++ b/src/applications/config/check/PhabricatorExtraConfigSetupCheck.php @@ -121,6 +121,7 @@ final class PhabricatorExtraConfigSetupCheck extends PhabricatorSetupCheck { 'phabricator.uninstalled-applications', 'phabricator.application-settings', 'config.ignore-issues', + 'auth.lock-config', ); $soft_locks = array_fuse($soft_locks); if (isset($soft_locks[$key])) { @@ -146,7 +147,7 @@ final class PhabricatorExtraConfigSetupCheck extends PhabricatorSetupCheck { 'that it was previously not locked, you set it using the web UI, '. 'and it later became locked.'. "\n\n". - 'You should copy this configuration value in a local configuration '. + 'You should copy this configuration value to a local configuration '. 'source (usually by using %s) and then remove it from the database '. 'with the command below.'. "\n\n".