From 321a8cb0ca72fe7e9d0753da1b5381eea14db651 Mon Sep 17 00:00:00 2001
From: Paul Holden <paulh@moodle.com>
Date: Wed, 27 Jul 2022 08:12:34 +0100
Subject: [PATCH] MDL-74656 reportbuilder: cast fields to char on mssql during
 concat.

---
 reportbuilder/classes/local/aggregation/base.php | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/reportbuilder/classes/local/aggregation/base.php b/reportbuilder/classes/local/aggregation/base.php
index 3997e02bef7..c6b3b1d3706 100644
--- a/reportbuilder/classes/local/aggregation/base.php
+++ b/reportbuilder/classes/local/aggregation/base.php
@@ -96,6 +96,9 @@ abstract class base {
 
             // We need to ensure all values are char (this ought to be done in the DML drivers, see MDL-72184).
             switch ($DB->get_dbfamily()) {
+                case 'mssql' :
+                    $sqlfield = $DB->sql_concat("''", $sqlfield);
+                    break;
                 case 'postgres' :
                     $sqlfield = "CAST({$sqlfield} AS VARCHAR)";
                     break;