Automatically set the value type for the case statement

Author: Walther Lalk

src/Database/Expression/CaseExpression.php

@@ -113,10 +113,7 @@ protected function _addExpressions($conditions, $trueValues) {
 			if ($trueValue === 'literal') {
 				$trueValue = $k;
 			} elseif (is_string($trueValue) || is_numeric($trueValue)) {
-				$trueValue = [
-					'value' => $trueValue,
-					'type' => null
-				];
+				$trueValue = $this->_parseValue($trueValue);
 			}
 
 			$this->_conditions[] = $c;
@@ -150,7 +147,7 @@ protected function _parseValue($value) {
 		if (is_string($value) || is_numeric($value)) {
 			$value = [
 				'value' => $value,
-				'type' => null
+				'type' => is_string($value) ? null : $this->_getType($value)
 			];
 		} elseif (is_array($value) && !isset($value['value'])) {
 			$value = array_keys($value);
@@ -159,6 +156,23 @@ protected function _parseValue($value) {
 		return $value;
 	}
 
+/**
+ * Gets the correct type for the value
+ *
+ * @param mixed $value The value to test
+ *
+ * @return null|string
+ */
+	protected function _getType($value) {
+		if (is_integer($value)) {
+			return 'integer';
+		} elseif (is_float($value)) {
+			return 'float';
+		}
+
+		return null;
+	}
+
 /**
  * Compiles the relevant parts into sql
  *

tests/TestCase/Database/QueryTest.php

@@ -2722,8 +2722,6 @@ public function testDirectIsNull() {
  * @return void
  */
 	public function testSqlCaseStatement() {
-		$convert = $this->connection->driver() instanceof \Cake\Database\Driver\Sqlserver;
-
 		$query = new Query($this->connection);
 		$publishedCase = $query
 			->newExpr()
@@ -2738,20 +2736,6 @@ public function testSqlCaseStatement() {
 					->add(['published' => 'N'])
 			);
 
-		//SQLServer requires the case statements to be converted to int
-		if ($convert) {
-			$publishedCase = $query->func()
-				->convert([
-					'INT' => 'literal',
-					$publishedCase
-				]);
-			$notPublishedCase = $query->func()
-				->convert([
-					'INT' => 'literal',
-					$notPublishedCase
-				]);
-		}
-
 		$results = $query
 			->select([
 				'published' => $query->func()->sum($publishedCase),