Driver::escapeDate() & escapeDateTime() accepts only DateTimeInterface (BC break)

src/Dibi/Drivers/FirebirdDriver.php

@@ -247,26 +247,14 @@ class FirebirdDriver implements Dibi\Driver
 	}
 
 
-	/**
+	public function escapeDate(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDate($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $value->format("'Y-m-d'");
 	}
 
 
-	/**
+	public function escapeDateTime(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDateTime($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return "'" . substr($value->format('Y-m-d H:i:s.u'), 0, -2) . "'";
 	}
 

src/Dibi/Drivers/MySqliDriver.php

@@ -290,26 +290,14 @@ class MySqliDriver implements Dibi\Driver
 	}
 
 
-	/**
+	public function escapeDate(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDate($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $value->format("'Y-m-d'");
 	}
 
 
-	/**
+	public function escapeDateTime(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDateTime($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $value->format("'Y-m-d H:i:s.u'");
 	}
 

src/Dibi/Drivers/OdbcDriver.php

@@ -226,26 +226,14 @@ class OdbcDriver implements Dibi\Driver
 	}
 
 
-	/**
+	public function escapeDate(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDate($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $value->format('#m/d/Y#');
 	}
 
 
-	/**
+	public function escapeDateTime(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDateTime($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $value->format($this->microseconds ? '#m/d/Y H:i:s.u#' : '#m/d/Y H:i:s#');
 	}
 

src/Dibi/Drivers/OracleDriver.php

@@ -241,28 +241,16 @@ class OracleDriver implements Dibi\Driver
 	}
 
 
-	/**
+	public function escapeDate(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDate($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $this->nativeDate
 			? "to_date('" . $value->format('Y-m-d') . "', 'YYYY-mm-dd')"
 			: $value->format('U');
 	}
 
 
-	/**
+	public function escapeDateTime(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDateTime($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $this->nativeDate
 			? "to_date('" . $value->format('Y-m-d G:i:s') . "', 'YYYY-mm-dd hh24:mi:ss')"
 			: $value->format('U');

src/Dibi/Drivers/PdoDriver.php

@@ -289,26 +289,14 @@ class PdoDriver implements Dibi\Driver
 	}
 
 
-	/**
+	public function escapeDate(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDate($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $value->format($this->driverName === 'odbc' ? '#m/d/Y#' : "'Y-m-d'");
 	}
 
 
-	/**
+	public function escapeDateTime(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDateTime($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		switch ($this->driverName) {
 			case 'odbc':
 				return $value->format('#m/d/Y H:i:s.u#');

src/Dibi/Drivers/PostgreDriver.php

@@ -292,26 +292,14 @@ class PostgreDriver implements Dibi\Driver
 	}
 
 
-	/**
+	public function escapeDate(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDate($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $value->format("'Y-m-d'");
 	}
 
 
-	/**
+	public function escapeDateTime(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDateTime($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $value->format("'Y-m-d H:i:s.u'");
 	}
 

src/Dibi/Drivers/SqliteDriver.php

@@ -230,26 +230,14 @@ class SqliteDriver implements Dibi\Driver
 	}
 
 
-	/**
+	public function escapeDate(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDate($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $value->format($this->fmtDate);
 	}
 
 
-	/**
+	public function escapeDateTime(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDateTime($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $value->format($this->fmtDateTime);
 	}
 

src/Dibi/Drivers/SqlsrvDriver.php

@@ -219,26 +219,14 @@ class SqlsrvDriver implements Dibi\Driver
 	}
 
 
-	/**
+	public function escapeDate(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDate($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return $value->format("'Y-m-d'");
 	}
 
 
-	/**
+	public function escapeDateTime(\DateTimeInterface $value): string
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	public function escapeDateTime($value): string
 	{
-		if (!$value instanceof \DateTimeInterface) {
-			$value = new Dibi\DateTime($value);
-		}
 		return 'CONVERT(DATETIME2(7), ' . $value->format("'Y-m-d H:i:s.u'") . ')';
 	}
 

src/Dibi/Translator.php

@@ -381,10 +381,11 @@ final class Translator
 				case 'dt': // datetime
 					if ($value === null) {
 						return 'NULL';
-					} else {
+					} elseif (!$value instanceof \DateTimeInterface) {
-						return $modifier === 'd' ? $this->driver->escapeDate($value) : $this->driver->escapeDateTime($value);
+						$value = new DateTime($value);
 					}
-					// break omitted
+					return $modifier === 'd' ? $this->driver->escapeDate($value) : $this->driver->escapeDateTime($value);
+
 				case 'by':
 				case 'n':  // composed identifier name
 					return $this->identifiers->$value;

src/Dibi/interfaces.php

@@ -87,15 +87,9 @@ interface Driver
 
 	function escapeBool(bool $value): string;
 
-	/**
+	function escapeDate(\DateTimeInterface $value): string;
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	function escapeDate($value): string;
 
-	/**
+	function escapeDateTime(\DateTimeInterface $value): string;
-	 * @param  \DateTimeInterface|string|int  $value
-	 */
-	function escapeDateTime($value): string;
 
 	/**
 	 * Encodes string for use in a LIKE statement.