Use ??= more

Author: nicolas-grekas

Lock.php

@@ -184,10 +184,7 @@ public function acquireRead(bool $blocking = false): bool
 
     public function refresh(float $ttl = null)
     {
-        if (null === $ttl) {
-            $ttl = $this->ttl;
-        }
-        if (!$ttl) {
+        if (!$ttl ??= $this->ttl) {
             throw new InvalidArgumentException('You have to define an expiration duration.');
         }
 

Store/DoctrineDbalPostgreSqlStore.php

@@ -262,6 +262,6 @@ private function getInternalStore(): SharedLockStoreInterface
     {
         $namespace = spl_object_hash($this->conn);
 
-        return self::$storeRegistry[$namespace] ?? self::$storeRegistry[$namespace] = new InMemoryStore();
+        return self::$storeRegistry[$namespace] ??= new InMemoryStore();
     }
 }

Store/FlockStore.php

@@ -39,10 +39,7 @@ class FlockStore implements BlockingStoreInterface, SharedLockStoreInterface
      */
     public function __construct(string $lockPath = null)
     {
-        if (null === $lockPath) {
-            $lockPath = sys_get_temp_dir();
-        }
-        if (!is_dir($lockPath)) {
+        if (!is_dir($lockPath ??= sys_get_temp_dir())) {
             if (false === @mkdir($lockPath, 0777, true) && !is_dir($lockPath)) {
                 throw new InvalidArgumentException(sprintf('The FlockStore directory "%s" does not exists and cannot be created.', $lockPath));
             }

Store/PostgreSqlStore.php

@@ -285,6 +285,6 @@ private function getInternalStore(): SharedLockStoreInterface
     {
         $namespace = spl_object_hash($this->getConnection());
 
-        return self::$storeRegistry[$namespace] ?? self::$storeRegistry[$namespace] = new InMemoryStore();
+        return self::$storeRegistry[$namespace] ??= new InMemoryStore();
     }
 }