ondrej-bouda / ivory

src/Ivory/Relation/IRelation.php

Doc Comments +2 added lines, -2 removed lines

@@ -311,12 +311,12 @@
     function tuple(int $offset = 0): ITuple;
 
     /**
-     * @param int|string|ITupleEvaluator|\Closure $colOffsetOrNameOrEvaluator
+     * @param integer $colOffsetOrNameOrEvaluator
      *                                  Specification of column from which to get the value. See {@link col()} for more
      *                                    details on the column specification.
      * @param int $tupleOffset zero-based offset of the tuple to get;
      *                         if negative, the <tt>-$tupleOffset</tt>'th tuple from the end is returned
-     * @return mixed
+     * @return string
      * @throws \OutOfBoundsException when this relation has fewer than <tt>$tupleOffset+1</tt> tuples, or fewer than
      *                                 <tt>-$tupleOffset</tt> tuples if <tt>$tupleOffset</tt> is negative
      * @throws UndefinedColumnException if no column matches the specification

Spacing +1 added lines, -1 removed lines

@@ -264,7 +264,7 @@
      * @throws UndefinedColumnException if no column matches the specification
      * @throws AmbiguousException if referring to the column by its name, which is used by multiple columns
      */
-    function toSet($colOffsetOrNameOrEvaluator, ?ISet $set = null): ISet;
+    function toSet($colOffsetOrNameOrEvaluator, ?ISet $set = null) : ISet;
 
     /**
      * Reduces the relation only to unique tuples.

src/Ivory/Connection/Config/ICachingConnConfig.php

Doc Comments +3 added lines

@@ -35,6 +35,7 @@ 
      *
      * @param string|string[] one or more names of parameters the cached values of which to invalidate;
      *                        <tt>null</tt> invalidates everything
+     * @return void
      */
     function invalidateCache($paramName = null): void;
 
@@ -47,6 +48,7 @@ 
      *
      * @param string|string[] $paramName one or more names of parameters to disabled caching values of;
      *                                   <tt>null</tt> disables the caching altogether
+     * @return void
      */
     function disableCaching($paramName = null): void;
 
@@ -58,6 +60,7 @@ 
      * argument (which is the default).
      *
      * @param string|string[] $paramName
+     * @return void
      */
     function enableCaching($paramName = null): void;
 }

src/Ivory/Connection/IConnectionControl.php

Doc Comments +3 added lines

@@ -100,6 +100,7 @@ 
      * {@link connectWait()}, the hooks are *not* called.
      *
      * @param \Closure $closure closure to call, given no arguments
+     * @return void
      */
     function registerConnectStartHook(\Closure $closure): void;
 
@@ -112,6 +113,7 @@ 
      * method, or when the connection gets destroyed before freeing from memory.
      *
      * @param \Closure $closure closure to call, given no arguments
+     * @return void
      */
     function registerPreDisconnectHook(\Closure $closure): void;
 
@@ -124,6 +126,7 @@ 
      * method, or when the connection gets destroyed before freeing from memory.
      *
      * @param \Closure $closure closure to call, given no arguments
+     * @return void
      */
     function registerPostDisconnectHook(\Closure $closure): void;
 }

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      *              <tt>false</tt> if the connection has already been open or started opening and thus this was a no-op
      * @throws ConnectionException on error connecting to the database
      */
-    function connect(?\Closure $initProcedure = null): bool;
+    function connect(?\Closure $initProcedure = null) : bool;
 
     /**
      * Establishes a connection with the database and waits for the connection to be established.

src/Ivory/Connection/ITypeControl.php

Doc Comments +1 added lines

@@ -22,6 +22,7 @@
      * Flushes the type dictionary currently in use, leading to loading a fresh new type dictionary.
      *
      * Useful when the data types change during the script execution in such a way Ivory is unable to detect it.
+     * @return void
      */
     function flushTypeDictionary(): void;
 }

src/Ivory/Exception/StatementExceptionFactory.php

Doc Comments +4 added lines

@@ -116,6 +116,10 @@
         return new $exClass($resultHandler, $query);
     }
 
+    /**
+     * @param resource $resultHandler
+     * @param StatementExceptionFactory $fallbackFactory
+     */
     private function inferExceptionClass($resultHandler, ?StatementExceptionFactory $fallbackFactory = null): string
     {
         if ($this->bySqlStateCodeAndMessage || $this->bySqlStateCode || $this->bySqlStateClass) {

Spacing +3 added lines, -3 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Exception;
 
@@ -110,13 +110,13 @@ 
         $resultHandler,
         string $query,
         ?StatementExceptionFactory $fallbackFactory = null
-    ): StatementException {
+    ) : StatementException {
         $exClass = $this->inferExceptionClass($resultHandler, $fallbackFactory);
         assert(is_a($exClass, StatementException::class, true));
         return new $exClass($resultHandler, $query);
     }
 
-    private function inferExceptionClass($resultHandler, ?StatementExceptionFactory $fallbackFactory = null): string
+    private function inferExceptionClass($resultHandler, ?StatementExceptionFactory $fallbackFactory = null) : string
     {
         if ($this->bySqlStateCodeAndMessage || $this->bySqlStateCode || $this->bySqlStateClass) {
             $sqlStateCode = pg_result_error_field($resultHandler, PGSQL_DIAG_SQLSTATE);

src/Ivory/Ivory.php

Doc Comments +1 added lines, -1 removed lines

@@ -204,7 +204,7 @@
     }
 
     /**
-     * @param IConnection|string $conn (name of) connection to use as the default connection
+     * @param IConnection $conn (name of) connection to use as the default connection
      * @throws \RuntimeException if the requested connection is not defined
      */
     public static function useConnectionAsDefault($conn): void

Spacing +4 added lines, -4 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory;
 
@@ -118,7 +118,7 @@ 
      * @param CacheItemPoolInterface|null $cacheItemPool the cache implementation to use for connections which do not
      *                                                     set their own cache, or <tt>null</tt> for no default cache
      */
-    public static function setDefaultCacheImpl(?CacheItemPoolInterface $cacheItemPool): void
+    public static function setDefaultCacheImpl(?CacheItemPoolInterface $cacheItemPool) : void
     {
         self::$defaultCacheImpl = $cacheItemPool;
     }
@@ -149,7 +149,7 @@ 
      * @throws ConnectionException if connection name is explicitly specified but a connection with the same name
      *                               already exists
      */
-    public static function setupNewConnection($params, ?string $connName = null): IConnection
+    public static function setupNewConnection($params, ?string $connName = null) : IConnection
     {
         if (!$params instanceof ConnectionParameters) {
             $params = ConnectionParameters::create($params);
@@ -186,7 +186,7 @@ 
      * @throws \RuntimeException if the default connection is requested but no connection has been setup yet, or if the
      *                             requested connection is not defined
      */
-    public static function getConnection(?string $connName = null): IConnection
+    public static function getConnection(?string $connName = null) : IConnection
     {
         if ($connName === null) {
             if (self::$defaultConn) {

src/Ivory/Type/ITypeDictionary.php

Doc Comments +8 added lines

@@ -51,10 +51,16 @@ 
      */
     function getValueSerializer(string $name): ?IValueSerializer;
 
+    /**
+     * @param \Ivory\Connection\anonymous//src/Ivory/Connection/TypeControl.php$0 $undefinedTypeHandler
+     *
+     * @return void
+     */
     function setUndefinedTypeHandler(?ITypeDictionaryUndefinedHandler $undefinedTypeHandler);
 
     /**
      * @param string[] $schemaList
+     * @return void
      */
     function setTypeSearchPath(array $schemaList): void;
 
@@ -62,6 +68,7 @@ 
      * Attach connection-dependent objects in this dictionary to a connection.
      *
      * @param IConnection $connection connection to attach the dictionary to
+     * @return void
      */
     function attachToConnection(IConnection $connection): void;
 
@@ -69,6 +76,7 @@ 
      * Detach any objects in this dictionary from the database connection.
      *
      * After this operation, the dictionary must be safe for serialization.
+     * @return void
      */
     function detachFromConnection(): void;
 }

src/Ivory/Type/Postgresql/RangeType.php

Doc Comments +4 added lines

@@ -151,6 +151,10 @@
         }
     }
 
+    /**
+     * @param boolean|null $aIsInc
+     * @param boolean|null $bIsInc
+     */
     private function compareBounds(int $sgn, $aVal, ?bool $aIsInc, $bVal, ?bool $bIsInc): int
     {
         if ($aVal === null && $bVal === null) {

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Type\Postgresql;
 
@@ -151,7 +151,7 @@ 
         }
     }
 
-    private function compareBounds(int $sgn, $aVal, ?bool $aIsInc, $bVal, ?bool $bIsInc): int
+    private function compareBounds(int $sgn, $aVal, ?bool $aIsInc, $bVal, ?bool $bIsInc) : int
     {
         if ($aVal === null && $bVal === null) {
             return 0;

src/Ivory/Type/TypeDictionary.php

Doc Comments +3 added lines

@@ -306,6 +306,9 @@
         }
     }
 
+    /**
+     * @param IType $item
+     */
     private static function removeAll(array &$array, $item): void
     {
         while (($key = array_search($item, $array, true)) !== false) {

Spacing +7 added lines, -7 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Type;
 
@@ -108,7 +108,7 @@ 
         throw new UndefinedTypeException("There is no type defined for converting value of type \"$typeName\"");
     }
 
-    private function recognizeType($value, $ruleSet): ?array
+    private function recognizeType($value, $ruleSet): ? array
     {
         static $gettypeMap = [
             'boolean' => 'bool',
@@ -136,7 +136,7 @@ 
         }
     }
 
-    private function recognizeObjectType($value, $ruleSet): ?array
+    private function recognizeObjectType($value, $ruleSet): ? array
     {
         $valueClass = new \ReflectionClass($value);
         $class = $valueClass;
@@ -156,7 +156,7 @@ 
         return null;
     }
 
-    private function recognizeArrayType($value, $ruleSet): ?array
+    private function recognizeArrayType($value, $ruleSet): ? array
     {
         $element = $this->findFirstSignificantElement($value);
 
@@ -243,7 +243,7 @@ 
 
     //region content management
 
-    public function defineType(IType $type, ?int $oid = null): void
+    public function defineType(IType $type, ?int $oid = null) : void
     {
         if ($oid !== null) {
             $this->oidTypeMap[$oid] = $type;
@@ -324,7 +324,7 @@ 
             // necessary for the forward reference to be valid once the aliased type finally appears in the type map
             $this->qualNameTypeMap[$schemaName] = [];
         }
-        $this->typeAliases[$alias] =& $this->qualNameTypeMap[$schemaName][$typeName];
+        $this->typeAliases[$alias] = & $this->qualNameTypeMap[$schemaName][$typeName];
     }
 
     /**
@@ -341,7 +341,7 @@ 
         foreach ($this->typeSearchPath as $schemaName) {
             foreach (($this->qualNameTypeMap[$schemaName] ?? []) as $typeName => $type) {
                 if (!isset($this->searchedNameCache[$typeName])) {
-                    $this->searchedNameCache[$typeName] =& $this->qualNameTypeMap[$schemaName][$typeName];
+                    $this->searchedNameCache[$typeName] = & $this->qualNameTypeMap[$schemaName][$typeName];
                 }
             }
         }