feat(appframework): ⌚ Make ITimeFactory extend \PSR\Clock\ClockInterface

Signed-off-by: Joas Schilling <coding@schilljs.com>

2 files changed, 26 insertions, 1 deletions

diff --git a/lib/private/AppFramework/Utility/TimeFactory.php b/lib/private/AppFramework/Utility/TimeFactory.php
index 27117ed3cfc..1e4655dd1cd 100644
--- a/lib/private/AppFramework/Utility/TimeFactory.php
+++ b/lib/private/AppFramework/Utility/TimeFactory.php
@@ -3,6 +3,7 @@
 declare(strict_types=1);
 
 /**
+ * @copyright Copyright (c) 2022, Joas Schilling <coding@schilljs.com>
  * @copyright Copyright (c) 2016, ownCloud, Inc.
  *
  * @author Bernhard Posselt <dev@bernhard-posselt.com>
@@ -30,11 +31,23 @@ namespace OC\AppFramework\Utility;
 use OCP\AppFramework\Utility\ITimeFactory;
 
 /**
- * Needed to mock calls to time()
+ * Use this to get a timestamp or DateTime object in code to remain testable
+ *
+ * @since 8.0.0
+ * @since 26.0.0 Extends the \Psr\Clock\ClockInterface interface
+ * @ref https://www.php-fig.org/psr/psr-20/#21-clockinterface
  */
 class TimeFactory implements ITimeFactory {
+	protected \DateTimeZone $timezone;
+
+	public function __construct() {
+		$this->timezone = new \DateTimeZone('UTC');
+	}
+
 	/**
 	 * @return int the result of a call to time()
+	 * @since 8.0.0
+	 * @deprecated 26.0.0 {@see ITimeFactory::now()}
 	 */
 	public function getTime(): int {
 		return time();
@@ -45,8 +58,19 @@ class TimeFactory implements ITimeFactory {
 	 * @param \DateTimeZone $timezone
 	 * @return \DateTime
 	 * @since 15.0.0
+	 * @deprecated 26.0.0 {@see ITimeFactory::now()}
 	 */
 	public function getDateTime(string $time = 'now', \DateTimeZone $timezone = null): \DateTime {
 		return new \DateTime($time, $timezone);
 	}
+
+	public function now(): \DateTimeImmutable {
+		return new \DateTimeImmutable('now', $this->timezone);
+	}
+	public function withTimeZone(\DateTimeZone $timezone): static {
+		$clone = clone $this;
+		$clone->timezone = $timezone;
+
+		return $clone;
+	}
 }
diff --git a/lib/private/Server.php b/lib/private/Server.php
index fbb86711b41..f9fc585e74d 100644
--- a/lib/private/Server.php
+++ b/lib/private/Server.php
@@ -1387,6 +1387,7 @@ class Server extends ServerContainer implements IServerContainer {
 		$this->registerDeprecatedAlias('ControllerMethodReflector', \OCP\AppFramework\Utility\IControllerMethodReflector::class);
 
 		$this->registerAlias(\OCP\AppFramework\Utility\ITimeFactory::class, \OC\AppFramework\Utility\TimeFactory::class);
+		$this->registerAlias(\Psr\Clock\ClockInterface::class, \OCP\AppFramework\Utility\ITimeFactory::class);
 		/** @deprecated 19.0.0 */
 		$this->registerDeprecatedAlias('TimeFactory', \OCP\AppFramework\Utility\ITimeFactory::class);