Fix var_export warnings on logs (#23)

* Fix var_export warnings on logs

* Force finaly string cast for service logging

Author: samizdam

CHANGELOG.md

@@ -6,6 +6,11 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [Unreleased]
 
+## [4.0.1] - 2024-10-09
+
+### Fixed
+- Use simple string instead var_export on logging, prevent `Warning: var_export does not handle circular references in LoggerHelper`
+
 ## [4.0.0] - 2024-06-03
 
 ### Added
@@ -110,7 +115,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 ### Added
 - All classes. 
 
-[Unreleased]: https://github.com/FreeElephants/php-di/compare/4.0.0...HEAD
+[Unreleased]: https://github.com/FreeElephants/php-di/compare/4.0.1...HEAD
+[4.0.1]: https://github.com/FreeElephants/php-di/compare/4.0.0...4.0.1
 [4.0.0]: https://github.com/FreeElephants/php-di/compare/3.1.1...4.0.0
 [3.1.1]: https://github.com/FreeElephants/php-di/compare/3.1.0...3.1.1
 [3.1.0]: https://github.com/FreeElephants/php-di/compare/3.0.0...3.1.0

src/LoggerHelper.php

@@ -33,14 +33,15 @@ public function logLazyLoading(string $interface, $service): void
         $debugMsg = 'Set service type  ' . $interface . ' instance by lazy load. ';
         $context = [
             'typeName' => $interface,
-            'instance' => $service
+            'instance' => $this->stringifyService($service),
         ];
         $this->logger->debug($debugMsg, $context);
     }
 
     public function logServiceRegistration($implementation, string $interface): void
     {
-        $msg = 'Service with type ' . $interface . ' and implementation ' . $this->stringifyImplementation($implementation) . ' register. ';
+        $implementation = $this->stringifyService($implementation);
+        $msg = 'Service with type ' . $interface . ' and implementation ' . $implementation . ' register. ';
         $context = [
             'interface'      => $interface,
             'implementation' => $implementation,
@@ -52,7 +53,7 @@ public function logNotMatchedTypeInstance(string $typeName, $instance): void
     {
         $context = [
             'typeName' => $typeName,
-            'instance' => $instance
+            'instance' => $this->stringifyService($instance)
         ];
         $this->logger->critical('Given instance not belong to this type. Exception will be thrown. ', $context);
     }
@@ -62,8 +63,8 @@ public function logServiceInstanceReplacing(string $typeName, $service, $previou
         $debugMsg = 'Replace service type  ' . $typeName . ' instance with another. ';
         $context = [
             'typeName'    => $typeName,
-            'instance'    => $service,
-            'oldInstance' => $previousServiceInstance,
+            'instance'    => $this->stringifyService($service),
+            'oldInstance' => $this->stringifyService($previousServiceInstance),
         ];
         $this->logger->debug($debugMsg, $context);
         return [$debugMsg, $context];
@@ -74,8 +75,8 @@ public function logRegisterServiceReplacing($implementation, string $interface,
         $msg = 'Replace registered service type ' . $interface . ' with another. ';
         $context = [
             'interface'         => $interface,
-            'newImplementation' => $implementation,
-            'oldImplementation' => $oldImplementation,
+            'newImplementation' => $this->stringifyService($implementation),
+            'oldImplementation' => $this->stringifyService($oldImplementation),
         ];
         $this->logger->debug($msg, $context);
     }
@@ -85,18 +86,19 @@ public function logServiceSetting(string $typeName, $service): void
         $debugMsg = 'Instance for service type ' . $typeName . ' was set. ';
         $context = [
             'typeName' => $typeName,
-            'instance' => $service
+            'instance' => $this->stringifyService($service),
         ];
         $this->logger->debug($debugMsg, $context);
     }
 
-    private function stringifyImplementation($implementation): string
+    private function stringifyService($implementation): string
     {
-        // var_export does not handle circular references
-        // handle this case
         if($implementation instanceof CallableBeanContainer) {
             $implementation = 'user defined callable';
+        } elseif(is_object($implementation)) {
+            $implementation = get_class($implementation);
         }
-        return var_export($implementation, true);
+
+        return (string) $implementation;
     }
 }