Merge pull request #84 from spenserhale/master

ArturMoczulski · web-flow · commit 7582a2a28b2f · 2019-05-24T05:32:22.000+02:00
fix(initPhpLogging): Moving fetch settings to before settings check.
diff --git 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 {
             

Original file line number	Diff line number	Diff line change
`@@ -265,13 +265,13 @@ public static function buildIncludedErrno($cutoff)`
`265`	`265`
`266`	`266`	`public function initPhpLogging()`
`267`	`267`	`{`
	`268`	`+ $this->fetchSettings();`
	`269`	`+`
`268`	`270`	`// Return if logging is not enabled`
`269`	`271`	`if ( $this->settings['php_logging_enabled'] === 0 ) {`
`270`	`272`	`return;`
`271`	`273`	`}`
`272`	`274`
`273`		`- $this->fetchSettings();`
`274`		`-`
`275`	`275`	`// installs global error and exception handlers`
`276`	`276`	`try {`
`277`	`277`