Skip to content
Snippets Groups Projects

Fix for #1520 - Eliminate MySQL CAST on datetime columns when expected value type is datetime.

Merged Fix for #1520 - Eliminate MySQL CAST on datetime columns when expected value type is datetime.
Merged Akshath requested to merge akshath/vtigercrm:fix_1520 into 7.4.0
+ 15
15
Compare changes
  • Side-by-side
  • Inline
Files
@@ -726,7 +726,7 @@ class EnhancedQueryGenerator extends QueryGenerator {
$startDateValue = explode(' ', $values[0]);
$endDateValue = explode(' ', $values[1]);
if (count($startDateValue) == 2 && count($endDateValue) == 2) {
$fieldSql .= " CAST(CONCAT($dateFieldColumnName,' ',$timeFieldColumnName) AS DATETIME) $valueSql";
$fieldSql .= " CONCAT($dateFieldColumnName,' ',$timeFieldColumnName) $valueSql";
} else {
$fieldSql .= "$dateFieldColumnName $valueSql";
}
@@ -736,7 +736,7 @@ class EnhancedQueryGenerator extends QueryGenerator {
}
$values = explode(' ', $value);
if (count($values) == 2) {
$fieldSql .= "$fieldGlue CAST(CONCAT($dateFieldColumnName,' ',$timeFieldColumnName) AS DATETIME) $valueSql ";
$fieldSql .= "$fieldGlue CONCAT($dateFieldColumnName,' ',$timeFieldColumnName) $valueSql ";
} else {
$fieldSql .= "$fieldGlue $dateFieldColumnName $valueSql";
}
Loading