Commit b2bac26e authored by Shalu Garg's avatar Shalu Garg Committed by Robert Lyon

Bug 1839499: Site pages and collections, Site unavailable error after enabling lti plugin

Fixing the issue of sql exception on pages & collections page after logging in as a non admin
user on enabling lti_assessment plugin. This is caused by calling a select statement for
lti_assessmentid column with aggregate function and group by condition,
but missing group by condition for lti_assessmentid.

Change-Id: I0a040018fadcf543452b24174964e929f286e552
(cherry picked from commit 1d2bad4e)
parent 0f5a0f71
......@@ -4166,8 +4166,9 @@ class View {
$collfrom .= $fromstr;
if (!empty($groupby)) {
$groupby .= ', g.id, h.wwwroot';
$collgroupby .= ', g.id, h.wwwroot';
// Adding groupby condition for lti_assessment id column.
$groupby .= ', g.id, h.wwwroot' . ($haslti ? ', a.id' : '');
$collgroupby .= ', g.id, h.wwwroot' . ($haslti ? ', a.id' : '');
}
$sort = '
ORDER BY ' . $order . ' vtitle, vid';
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment