DateTime: modify() and modifyClone throw exception on error (#293)

Author: lukaspijak

src/Utils/DateTime.php

@@ -129,6 +129,17 @@ public function __toString(): string
 	}
 
 
+	public function modify(string $modifier = ''): static
+	{
+		$datetime = @parent::modify($modifier);
+
+		if ($datetime === false) {
+			throw new Nette\InvalidArgumentException("Failed to parse time string '$modifier'");
+		}
+		return $datetime;
+	}
+
+
 	/**
 	 * Creates a copy with a modified time.
 	 */

tests/Utils/DateTime.modifyClone.phpt

@@ -25,3 +25,8 @@ $dolly2 = $date->modifyClone('+1 hour');
 Assert::type(DateTime::class, $dolly2);
 Assert::notSame($date, $dolly2);
 Assert::notSame((string) $date, (string) $dolly2);
+
+Assert::exception(
+	fn() => $date->modifyClone('xx'),
+	Nette\InvalidArgumentException::class,
+);