Quellcode durchsuchen

Make ChromePHPHandler and FirePHPHandler sendHeaders flag static so that when multiple instances of the handler class exist they will all respect the one-time check that determines whether the handler should append its logs to the response header

Ryan LaHue vor 13 Jahren
Ursprung
Commit
e0d18f454b

+ 3 - 3
src/Monolog/Handler/ChromePHPHandler.php

@@ -38,7 +38,7 @@ class ChromePHPHandler extends AbstractProcessingHandler
         'rows' => array(),
     );
 
-    protected $sendHeaders = true;
+    protected static $sendHeaders = true;
 
     /**
      * {@inheritdoc}
@@ -91,7 +91,7 @@ class ChromePHPHandler extends AbstractProcessingHandler
     protected function send()
     {
         if (!self::$initialized) {
-            $this->sendHeaders = $this->headersAccepted();
+            self::$sendHeaders = $this->headersAccepted();
             self::$json['request_uri'] = isset($_SERVER['REQUEST_URI']) ? $_SERVER['REQUEST_URI'] : '';
 
             self::$initialized = true;
@@ -108,7 +108,7 @@ class ChromePHPHandler extends AbstractProcessingHandler
      */
     protected function sendHeader($header, $content)
     {
-        if (!headers_sent() && $this->sendHeaders) {
+        if (!headers_sent() && self::$sendHeaders) {
             header(sprintf('%s: %s', $header, $content));
         }
     }

+ 3 - 3
src/Monolog/Handler/FirePHPHandler.php

@@ -51,7 +51,7 @@ class FirePHPHandler extends AbstractProcessingHandler
      */
     protected static $messageIndex = 1;
 
-    protected $sendHeaders = true;
+    protected static $sendHeaders = true;
 
     /**
      * Base header creation function used by init headers & record headers
@@ -117,7 +117,7 @@ class FirePHPHandler extends AbstractProcessingHandler
      */
     protected function sendHeader($header, $content)
     {
-        if (!headers_sent() && $this->sendHeaders) {
+        if (!headers_sent() && self::$sendHeaders) {
             header(sprintf('%s: %s', $header, $content));
         }
     }
@@ -133,7 +133,7 @@ class FirePHPHandler extends AbstractProcessingHandler
     {
         // WildFire-specific headers must be sent prior to any messages
         if (!self::$initialized) {
-            $this->sendHeaders = $this->headersAccepted();
+            self::$sendHeaders = $this->headersAccepted();
 
             foreach ($this->getInitHeaders() as $header => $content) {
                 $this->sendHeader($header, $content);