From db947a148e577fb27c1b81b54cdec713be714154 Mon Sep 17 00:00:00 2001 From: Spenser Hale Date: Tue, 21 May 2019 18:02:18 -0700 Subject: [PATCH] fix(initPhpLogging): Moving fetch settings to before settings check. There is the bug that you cannot disable PhpLogging since the settings property is not set until fetchSettings() is called. So I reordered. --- src/Plugin.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Plugin.php b/src/Plugin.php index 85e6f0f6..3bf54aa1 100644 --- a/src/Plugin.php +++ b/src/Plugin.php @@ -265,13 +265,13 @@ public static function buildIncludedErrno($cutoff) public function initPhpLogging() { + $this->fetchSettings(); + // Return if logging is not enabled if ( $this->settings['php_logging_enabled'] === 0 ) { return; } - $this->fetchSettings(); - // installs global error and exception handlers try {