[12.x] Do not dispatch MessageLogged twice (#56713)

* failing test

* passing test

* Update Logger.php

---------

Co-authored-by: Taylor Otwell <[email protected]>

Author: cosmastech

src/Illuminate/Log/Logger.php

@@ -245,6 +245,12 @@ public function listen(Closure $callback)
      */
     protected function fireLogEvent($level, $message, array $context = [])
     {
+        // Avoid dispatching the event multiple times if our logger instance is the LogManager...
+        if ($this->logger instanceof LogManager && 
+            $this->logger->getEventDispatcher() !== null) {
+            return;
+        }
+
         // If the event dispatcher is set, we will pass along the parameters to the
         // log listeners. These are useful for building profilers or other tools
         // that aggregate all of the log messages for a given "request" cycle.

tests/Integration/Log/LoggingIntegrationTest.php

@@ -2,6 +2,9 @@
 
 namespace Illuminate\Tests\Integration\Log;
 
+use Illuminate\Log\Events\MessageLogged;
+use Illuminate\Log\Logger;
+use Illuminate\Support\Facades\Event;
 use Illuminate\Support\Facades\Log;
 use Orchestra\Testbench\TestCase;
 
@@ -13,4 +16,13 @@ public function testLoggingCanBeRunWithoutEncounteringExceptions()
 
         Log::info('Hello World');
     }
+
+    public function testCallingLoggerDirectlyDispatchesOneEvent()
+    {
+        Event::fake([MessageLogged::class]);
+
+        $this->app->make(Logger::class)->debug('my debug message');
+
+        Event::assertDispatchedTimes(MessageLogged::class, 1);
+    }
 }