ondrej-bouda / ivory

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];
                 }
             }
         }

src/Ivory/Value/Decimal.php

Doc Comments +6 added lines, -6 removed lines

@@ -256,7 +256,7 @@ 
      * which might or might not be desired. Such a difference only arises in the trailing fractional zero digits,
      * though.
      *
-     * @param string|int|float|Decimal|object $number number to compare this number with
+     * @param Decimal $number number to compare this number with
      * @return bool|null whether this number numerically equals to <tt>$number</tt>;
      *                   <tt>null</tt> for <tt>null</tt> input
      */
@@ -278,7 +278,7 @@ 
     }
 
     /**
-     * @param string|int|float|Decimal|object $number
+     * @param Decimal $number
      * @return bool <tt>true</tt> iff this number is numerically greater than <tt>$number</tt>
      */
     public function greaterThan($number): bool
@@ -363,7 +363,7 @@ 
     }
 
     /**
-     * @param string|int|float|Decimal|object $augend
+     * @param integer $augend
      * @return Decimal a new decimal number, representing the result of sum of this and the given number
      */
     public function add($augend): Decimal
@@ -380,7 +380,7 @@ 
     }
 
     /**
-     * @param string|int|float|Decimal|object $subtrahend
+     * @param Decimal $subtrahend
      * @return Decimal a new decimal number, representing the result of subtraction of this and the given number
      */
     public function subtract($subtrahend): Decimal
@@ -397,7 +397,7 @@ 
     }
 
     /**
-     * @param string|int|float|Decimal|object $multiplicand
+     * @param Decimal $multiplicand
      * @return Decimal a new decimal number, representing the result of multiplication of this and the given number
      */
     public function multiply($multiplicand): Decimal
@@ -414,7 +414,7 @@ 
     }
 
     /**
-     * @param string|int|float|Decimal|object $divisor
+     * @param Decimal $divisor
      * @return Decimal a new decimal number, representing the result of division of this number with the given number
      * @throws \RuntimeException if <tt>$divisor</tt> is zero
      */

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Value;
 
@@ -63,7 +63,7 @@ 
      *                        if not given, it is computed from <tt>$decimalNumber</tt> automatically
      * @return Decimal
      */
-    public static function fromNumber($decimalNumber, ?int $scale = null): Decimal
+    public static function fromNumber($decimalNumber, ?int $scale = null) : Decimal
     {
         if ($decimalNumber === null) {
             throw new \InvalidArgumentException('decimalNumber');

src/Ivory/Value/Quantity.php

Doc Comments +4 added lines

@@ -99,6 +99,10 @@
         return new Quantity($val, $unit);
     }
 
+    /**
+     * @param string $value
+     * @param string $unit
+     */
     public static function fromValue($value, ?string $unit = null): Quantity
     {
         if (!is_numeric($value)) {

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Value;
 
@@ -99,7 +99,7 @@ 
         return new Quantity($val, $unit);
     }
 
-    public static function fromValue($value, ?string $unit = null): Quantity
+    public static function fromValue($value, ?string $unit = null) : Quantity
     {
         if (!is_numeric($value)) {
             throw new \InvalidArgumentException('$value');

src/Ivory/Value/Range.php

Doc Comments +6 added lines

@@ -168,6 +168,9 @@ 
         return new Range($subtype, null, true, null, null, null, null);
     }
 
+    /**
+     * @param boolean $upperInc
+     */
     private static function processBoundSpec($boundsOrLowerInc = '[)', ?bool $upperInc = null)
     {
         if (is_string($boundsOrLowerInc)) {
@@ -194,6 +197,9 @@ 
     }
 
 
+    /**
+     * @param IRangeCanonicalFunc|null $canonicalFunc
+     */
     private function __construct(
         ITotallyOrderedType $subtype,
         ?IRangeCanonicalFunc $canonicalFunc,

Spacing +9 added lines, -9 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Value;
 
@@ -107,7 +107,7 @@ 
         $upper,
         $boundsOrLowerInc = '[)',
         ?bool $upperInc = null
-    ): Range {
+    ) : Range {
         list($loInc, $upInc) = self::processBoundSpec($boundsOrLowerInc, $upperInc);
 
         if ($lower !== null && $upper !== null) {
@@ -320,7 +320,7 @@ 
      * @return array|null pair of the lower and upper bound, or <tt>null</tt> if the range is empty
      * @throws UnsupportedException if the range subtype is not an {@link IDiscreteType}
      */
-    public function toBounds($boundsOrLowerInc, ?bool $upperInc = null): ?array
+    public function toBounds($boundsOrLowerInc, ?bool $upperInc = null) : ? array
     {
         if (!$this->subtype instanceof IDiscreteType) {
             throw new UnsupportedException('Range subtype is not ' . IDiscreteType::class . ', cannot convert bounds');
@@ -455,7 +455,7 @@ 
      *                   an empty range is considered to be contained in any range, even an empty one;
      *                   <tt>null</tt> on <tt>null</tt> input
      */
-    public function containsRange(?Range $other): ?bool
+    public function containsRange(?Range $other) : ?bool
     {
         if ($other === null) {
             return null;
@@ -498,7 +498,7 @@ 
      *                   an empty range is considered to be contained in any range, even an empty one;
      *                   <tt>null</tt> on <tt>null</tt> input
      */
-    public function containedInRange(?Range $other): ?bool
+    public function containedInRange(?Range $other) : ?bool
     {
         if ($other === null) {
             return null;
@@ -511,7 +511,7 @@ 
      * @return bool|null whether this and the other range overlap, i.e., have a non-empty intersection;
      *                   <tt>null</tt> on <tt>null</tt> input
      */
-    public function overlaps(?Range $other): ?bool
+    public function overlaps(?Range $other) : ?bool
     {
         if ($other === null) {
             return null;
@@ -542,7 +542,7 @@ 
      * @return Range|null intersection of this and the other range
      *                    <tt>null</tt> on <tt>null</tt> input
      */
-    public function intersect(?Range $other): ?Range
+    public function intersect(?Range $other) : ?Range
     {
         if ($other === null) {
             return null;
@@ -613,7 +613,7 @@ 
      *                   <tt>false</tt> otherwise, especially if either range is empty;
      *                   <tt>null</tt> on <tt>null</tt> input
      */
-    public function strictlyLeftOf(?Range $other): ?bool
+    public function strictlyLeftOf(?Range $other) : ?bool
     {
         if ($other === null) {
             return null;
@@ -636,7 +636,7 @@ 
      *                   <tt>false</tt> otherwise, especially if either range is empty;;
      *                   <tt>null</tt> on <tt>null</tt> input
      */
-    public function strictlyRightOf(?Range $other): ?bool
+    public function strictlyRightOf(?Range $other) : ?bool
     {
         if ($other === null) {
             return null;