[Bug] Fix class and interface check with nullable value (#39)

* fix class and interface check with nullable value * use same fix as in 6.1.0 --------- Co-authored-by: Léa Crémier <[email protected]>
FrDeamon · Jan 3, 2024 · 09ac134 · 09ac134
1 parent 11d6903
commit 09ac134
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/Processors/Monolog/DeamonLoggerExtraWebProcessor.php b/Processors/Monolog/DeamonLoggerExtraWebProcessor.php
@@ -129,7 +129,7 @@ private function addRequestInfo(): void
      */
     private function addUserInfo(): void
     {
-        if (!$this->configShowExtraInfo('user')) {
+        if (!$this->configShowExtraInfo('user') || empty($this->userClass)) {
             return;
         }
 

diff --git a/Tests/Processors/Monolog/DeamonLoggerExtraWebProcessorTest.php b/Tests/Processors/Monolog/DeamonLoggerExtraWebProcessorTest.php
@@ -118,6 +118,19 @@ public function testAddUserInfoWithNotExistingClass()
         $this->assertArrayNotHasKey('user_name', $record['extra']);
     }
 
+
+    public function testAddUserInfoWithEmptyUserClass()
+    {
+        $config = $this->getDisplayConfig([
+            'user' => true,
+        ], null, '');
+
+        $processor = new DeamonLoggerExtraWebProcessor($config);
+        $record = $processor->__invoke($this->getRecord());
+
+        $this->assertArrayNotHasKey('user_name', $record['extra']);
+    }
+
     public function testAddUserinfoWithNoTokenStorage()
     {
         $config = $this->getDisplayConfig([
-Original file line number
+Diff line change
@@ Expand Up / @@ -129,7 +129,7 @@ private function addRequestInfo(): void @@
          */
         private function addUserInfo(): void
         {
-            if (!$this->configShowExtraInfo('user')) {
+            if (!$this->configShowExtraInfo('user') || empty($this->userClass)) {
                 return;
             }
@@ Expand Down @@