Fix PHP 8.3 compatibility

phpstan.neon

@@ -3,6 +3,8 @@ parameters:
         - lazy/Carbon/MessageFormatter/MessageFormatterMapperStrongType.php
         - lazy/Carbon/PHPStan/MacroStrongType.php
         - lazy/Carbon/TranslatorStrongType.php
+        - tests/Fixtures/DateMalformedIntervalStringException.php
+        - tests/Fixtures/DateMalformedStringException.php
         - vendor/autoload.php
     level: 3
     paths:

readme.md

@@ -2,7 +2,7 @@
 
 [![Latest Stable Version](https://img.shields.io/packagist/v/nesbot/carbon.svg?style=flat-square)](https://packagist.org/packages/nesbot/carbon)
 [![Total Downloads](https://img.shields.io/packagist/dt/nesbot/carbon.svg?style=flat-square)](https://packagist.org/packages/nesbot/carbon)
-[![GitHub Actions](https://img.shields.io/endpoint.svg?url=https%3A%2F%2Factions-badge.atrox.dev%2Fbriannesbitt%2FCarbon%2Fbadge&style=flat-square&label=Build&logo=none)](https://actions-badge.atrox.dev/briannesbitt/Carbon/goto)
+[![GitHub Actions](https://img.shields.io/endpoint.svg?url=https%3A%2F%2Factions-badge.atrox.dev%2Fbriannesbitt%2FCarbon%2Fbadge&style=flat-square&label=Build&logo=none)](https://github.com/briannesbitt/Carbon/actions)
 [![codecov.io](https://img.shields.io/codecov/c/github/briannesbitt/Carbon.svg?style=flat-square)](https://codecov.io/github/briannesbitt/Carbon?branch=master)
 [![Tidelift](https://tidelift.com/badges/github/briannesbitt/Carbon)](https://tidelift.com/subscription/pkg/packagist-nesbot-carbon?utm_source=packagist-nesbot-carbon&utm_medium=referral&utm_campaign=readme)
 

src/Carbon/CarbonInterval.php

@@ -28,6 +28,7 @@
 use Carbon\Traits\ToStringFormat;
 use Closure;
 use DateInterval;
+use DateMalformedIntervalStringException;
 use DateTimeInterface;
 use DateTimeZone;
 use Exception;
@@ -1004,8 +1005,14 @@ protected static function makeFromString(string $interval)
             return static::fromString($interval);
         }
 
-        /** @var static $interval */
-        $interval = static::createFromDateString($interval);
+        // @codeCoverageIgnoreStart
+        try {
+            /** @var static $interval */
+            $interval = static::createFromDateString($interval);
+        } catch (DateMalformedIntervalStringException $e) {
+            return null;
+        }
+        // @codeCoverageIgnoreEnd
 
         return !$interval || $interval->isEmpty() ? null : $interval;
     }

src/Carbon/Traits/Creator.php

@@ -19,6 +19,7 @@
 use Carbon\Exceptions\OutOfRangeException;
 use Carbon\Translator;
 use Closure;
+use DateMalformedStringException;
 use DateTimeInterface;
 use DateTimeZone;
 use Exception;
@@ -184,7 +185,13 @@ public static function rawParse($time = null, $tz = null)
         try {
             return new static($time, $tz);
         } catch (Exception $exception) {
-            $date = @static::now($tz)->change($time);
+            // @codeCoverageIgnoreStart
+            try {
+                $date = @static::now($tz)->change($time);
+            } catch (DateMalformedStringException $ignoredException) {
+                $date = null;
+            }
+            // @codeCoverageIgnoreEnd
 
             if (!$date) {
                 throw new InvalidFormatException("Could not parse '$time': ".$exception->getMessage(), 0, $exception);

src/Carbon/Traits/Units.php

@@ -17,6 +17,7 @@
 use Carbon\Exceptions\UnitException;
 use Closure;
 use DateInterval;
+use DateMalformedStringException;
 use ReturnTypeWillChange;
 
 /**
@@ -304,12 +305,17 @@ public function addUnit($unit, $value = 1, $overflow = null)
             $unit = 'second';
             $value = $second;
         }
-        $date = $date->modify("$value $unit");
 
-        if (isset($timeString)) {
-            $date = $date->setTimeFromTimeString($timeString);
-        } elseif (isset($canOverflow, $day) && $canOverflow && $day !== $date->day) {
-            $date = $date->modify('last day of previous month');
+        try {
+            $date = $date->modify("$value $unit");
+
+            if (isset($timeString)) {
+                $date = $date->setTimeFromTimeString($timeString);
+            } elseif (isset($canOverflow, $day) && $canOverflow && $day !== $date->day) {
+                $date = $date->modify('last day of previous month');
+            }
+        } catch (DateMalformedStringException $ignoredException) { // @codeCoverageIgnore
+            $date = null; // @codeCoverageIgnore
         }
 
         if (!$date) {

tests/Carbon/ModifyTest.php

@@ -14,6 +14,8 @@
 namespace Tests\Carbon;
 
 use Carbon\Carbon;
+use Closure;
+use DateMalformedStringException;
 use InvalidArgumentException;
 use Tests\AbstractTestCase;
 
@@ -256,9 +258,34 @@ public function testNextAndPrevious()
 
     public function testInvalidModifier(): void
     {
-        $this->assertFalse(@Carbon::parse('2000-01-25')->change('invalid'));
-        $this->assertFalse(@Carbon::now()->next('invalid'));
-        $this->assertFalse(@Carbon::now()->previous('invalid'));
+        $this->checkInvalid('invalid', static function () {
+            return @Carbon::parse('2000-01-25')->change('invalid');
+        });
+        $this->checkInvalid('next invalid', static function () {
+            return @Carbon::now()->next('invalid');
+        });
+        $this->checkInvalid('last invalid', static function () {
+            return @Carbon::now()->previous('invalid');
+        });
+    }
+
+    private function checkInvalid(string $message, Closure $callback): void
+    {
+        if (PHP_VERSION < 8.3) {
+            $this->assertFalse($callback());
+
+            return;
+        }
+
+        try {
+            $callback();
+        } catch (DateMalformedStringException $exception) {
+            $this->assertStringContainsString("Failed to parse time string ($message)", $exception->getMessage());
+
+            return;
+        }
+
+        $this->fail('This should throw a DateMalformedStringException in PHP 8.3');
     }
 
     public function testImplicitCast(): void

tests/Fixtures/DateMalformedIntervalStringException.php

@@ -0,0 +1,14 @@
+<?php
+
+/**
+ * This file is part of the Carbon package.
+ *
+ * (c) Brian Nesbitt <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+class DateMalformedIntervalStringException extends Exception
+{
+}

tests/Fixtures/DateMalformedStringException.php

@@ -0,0 +1,14 @@
+<?php
+
+/**
+ * This file is part of the Carbon package.
+ *
+ * (c) Brian Nesbitt <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+class DateMalformedStringException extends Exception
+{
+}