Merge pull request #3103 from briannesbitt/fix/issue-3102-setUnitNoOverflow-immutable

Fix immutable return for setUnitNoOverflow

Author: kylekatarnls

src/Carbon/Traits/Date.php

@@ -2965,7 +2965,7 @@ private static function floorZeroPad(int|float $value, int $length): string
     private function mutateIfMutable(CarbonInterface $date): CarbonInterface
     {
         return $this instanceof DateTimeImmutable
-            ? $this
+            ? $date
             : $this->modify('@'.$date->rawFormat('U.u'))->setTimezone($date->getTimezone());
     }
 }

tests/Carbon/SettersTest.php

@@ -629,6 +629,14 @@ public function testMidDayAtSetter()
         $this->assertSame(12, $d->getMidDayAt());
     }
 
+    public function testSetUnitNoOverflowFebruary()
+    {
+        $d = Carbon::parse('2024-02-29')->setUnitNoOverFlow('day', 31, 'month');
+
+        $this->assertInstanceOf(Carbon::class, $d);
+        $this->assertSame('2024-02-29 23:59:59.999999', $d->format('Y-m-d H:i:s.u'));
+    }
+
     public function testSetUnitNoOverflow()
     {
         $results = [

tests/CarbonImmutable/SettersTest.php

@@ -423,4 +423,12 @@ public function testSetter()
         $d->setMidDayAt(12);
         $this->assertSame(12, $d->getMidDayAt());
     }
+
+    public function testSetUnitNoOverflowFebruary()
+    {
+        $d = Carbon::parse('2024-02-29')->setUnitNoOverFlow('day', 31, 'month');
+
+        $this->assertInstanceOf(Carbon::class, $d);
+        $this->assertSame('2024-02-29 23:59:59.999999', $d->format('Y-m-d H:i:s.u'));
+    }
 }