Merge pull request #14 from petrknap/logger

Added logger support

README.md

@@ -16,6 +16,8 @@ It is an easy way to make sure that everyone has to check if they have (not) rec
 ```php
 namespace PetrKnap\Optional;
 
+Optional::setLogger(new \Psr\Log\NullLogger());
+
 $optionalString = OptionalString::of('value');
 
 echo $optionalString->isPresent() ? $optionalString->get() : 'empty';

composer.json

@@ -28,7 +28,8 @@
   "license": "LGPL-3.0-or-later",
   "name": "petrknap/optional",
   "require": {
-    "php": ">=8.1"
+    "php": ">=8.1",
+    "psr/log": "^2.0"
   },
   "require-dev": {
     "nunomaduro/phpinsights": "^2.11",

src/AbstractOptional.php

@@ -19,6 +19,8 @@
  */
 abstract class AbstractOptional
 {
+    use LoggerTrait;
+
     private bool|null $wasPresent = null;
 
     /**
@@ -109,10 +111,7 @@ public function flatMap(callable $mapper): self
     public function get(): mixed
     {
         if ($this->wasPresent === null) {
-            trigger_error(
-                'Call `isPresent()` before accessing the value.',
-                error_level: E_USER_NOTICE,
-            );
+            self::logNotice('Call `isPresent()` before accessing the value.');
         }
         return $this->orElseThrow(static fn (): Exception\NoSuchElement => new Exception\NoSuchElement());
     }

src/LoggerTrait.php

@@ -0,0 +1,29 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PetrKnap\Optional;
+
+use Psr\Log\LoggerInterface;
+
+trait LoggerTrait
+{
+    private static ?LoggerInterface $logger = null;
+
+    public static function setLogger(LoggerInterface $logger): void
+    {
+        self::$logger = $logger;
+    }
+
+    protected static function logNotice(string $message): void
+    {
+        if (self::$logger === null) {
+            trigger_error(
+                $message,
+                error_level: E_USER_NOTICE,
+            );
+        } else {
+            self::$logger->notice($message);
+        }
+    }
+}

src/Optional.php

@@ -17,10 +17,7 @@
 {
     /* abstract */ protected static function isSupported(mixed $value): bool
     {
-        trigger_error(
-            static::class . ' does not check the type of value.',
-            error_level: E_USER_NOTICE,
-        );
+        self::logNotice(static::class . ' does not check the type of value.');
         return true;
     }
 }

src/OptionalObject.php

@@ -17,10 +17,7 @@
 {
     /* abstract */ protected static function getObjectClassName(): string
     {
-        trigger_error(
-            static::class . ' does not check the instance of object.',
-            error_level: E_USER_NOTICE,
-        );
+        self::logNotice(static::class . ' does not check the instance of object.');
         /** @var class-string */
         return '';
     }

src/OptionalResource.php

@@ -12,10 +12,7 @@
 {
     /* abstract */ protected static function getResourceType(): string
     {
-        trigger_error(
-            static::class . ' does not check the type of resource.',
-            error_level: E_USER_NOTICE,
-        );
+        self::logNotice(static::class . ' does not check the type of resource.');
         /** @var non-empty-string */
         return '';
     }

tests/OptionalTest.php

@@ -8,12 +8,19 @@
 use LogicException;
 use PHPUnit\Framework\Attributes\DataProvider;
 use PHPUnit\Framework\TestCase;
+use Psr\Log\NullLogger;
 
 class OptionalTest extends TestCase
 {
     private const VALUE = 'value';
     private const OTHER = 'other';
 
+    public static function setUpBeforeClass(): void
+    {
+        Optional::setLogger(new NullLogger());
+        parent::setUpBeforeClass();
+    }
+
     public function testMethodEmptyReturnsEmptyOptional(): void
     {
         self::assertFalse(Optional::empty()->isPresent());