ACP2E-1287: log exception to exception.log for all levels to preserve backward compatibility

Author: alzota

lib/internal/Magento/Framework/Logger/LoggerProxy.php

@@ -126,6 +126,7 @@ public function critical($message, array $context = [])
      */
     public function error($message, array $context = [])
     {
+        $context = $this->addExceptionToContext($message, $context);
         $this->getLogger()->error($message, $context);
     }
 
@@ -134,6 +135,7 @@ public function error($message, array $context = [])
      */
     public function warning($message, array $context = [])
     {
+        $context = $this->addExceptionToContext($message, $context);
         $this->getLogger()->warning($message, $context);
     }
 
@@ -142,6 +144,7 @@ public function warning($message, array $context = [])
      */
     public function notice($message, array $context = [])
     {
+        $context = $this->addExceptionToContext($message, $context);
         $this->getLogger()->notice($message, $context);
     }
 
@@ -150,6 +153,7 @@ public function notice($message, array $context = [])
      */
     public function info($message, array $context = [])
     {
+        $context = $this->addExceptionToContext($message, $context);
         $this->getLogger()->info($message, $context);
     }
 
@@ -158,6 +162,7 @@ public function info($message, array $context = [])
      */
     public function debug($message, array $context = [])
     {
+        $context = $this->addExceptionToContext($message, $context);
         $this->getLogger()->debug($message, $context);
     }
 
@@ -166,6 +171,7 @@ public function debug($message, array $context = [])
      */
     public function log($level, $message, array $context = [])
     {
+        $context = $this->addExceptionToContext($message, $context);
         $this->getLogger()->log($level, $message, $context);
     }
 

lib/internal/Magento/Framework/Logger/Test/Unit/LoggerProxyTest.php

@@ -124,11 +124,8 @@ public function logException($method): void
             ->withConsecutive([DeploymentConfig::class], [Monolog::class])
             ->willReturnOnConsecutiveCalls($deploymentConfig, $logger);
 
-        if (in_array($method, ['critical', 'emergency', 'alert'])) {
-            $message = new \Exception('This is an exception.');
-        } else {
-            $message = 'test';
-        }
+        $message = new \Exception('This is an exception.');
+
         $logger->expects($this->once())->method($method)->with($message);
 
         $loggerProxy = new LoggerProxy($objectManager);