ondrej-bouda / ivory

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;

src/Ivory/Value/TimestampBase.php

Doc Comments +1 added lines, -1 removed lines

@@ -206,7 +206,7 @@
      * Adds a given number of seconds (1 by default) to this date/time and returns the result. Only affects finite
      * dates.
      *
-     * @param int|float $seconds
+     * @param integer $seconds
      * @return static the date <tt>$seconds</tt> seconds after (or before, if negative) this date/time
      */
     public function addSecond($seconds = 1)

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Value;
 
@@ -37,7 +37,7 @@ 
     protected static function isoStringToDateTime(
         string $isoDateTimeString,
         ?\DateTimeZone $forcedTimezone = null
-    ): \DateTimeImmutable {
+    ) : \DateTimeImmutable {
         // check out for more than 4 digits for the year - something date_create_immutable() does not handle properly
         $addYears = 0;
         $dateCreateInput = preg_replace_callback(

src/Ivory/Lang/SqlPattern/SqlPattern.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Lang\SqlPattern;
 

src/Ivory/Lang/SqlPattern/SqlPatternParser.php

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Lang\SqlPattern;
 
@@ -97,7 +97,7 @@ 
      * @param bool|null $quoted
      * @return string|null <tt>null</tt> iff <tt>$str</tt> is <tt>null</tt>
      */
-    private function unquoteString(?string $str, ?bool &$quoted = null): ?string
+    private function unquoteString(?string $str, ?bool &$quoted = null) : ?string
     {
         if ($str && $str[0] == '"') {
             assert($str[strlen($str) - 1] == '"');

src/Ivory/Lang/SqlPattern/SqlPatternPlaceholder.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Lang\SqlPattern;
 

src/Ivory/Lang/SqlPattern/CachingSqlPatternParser.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Lang\SqlPattern;
 

src/Ivory/Lang/Sql/Keywords.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Lang\Sql;
 

src/Ivory/Lang/Sql/Types.php

Spacing +2 added lines, -2 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Ivory\Lang\Sql;
 
@@ -9,7 +9,7 @@ 
 class Types
 {
     const BIGINT_MIN = '-9223372036854775808'; // NOTE: written as string not to lose precision on platforms where this
-    const BIGINT_MAX = '999999999999';  //       would be converted to float
+    const BIGINT_MAX = '999999999999'; //       would be converted to float
 
     /**
      * Lists names of types defined by SQL as reserved ones.