Skip to content

Fix bug in most active courses.xml #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

icsbcn
Copy link

@icsbcn icsbcn commented Feb 20, 2019

When you try to import and execute this xml report in a Moodle 3.5.2 an error appears:

Debug info: ERROR: column "c.fullname" must appear in the GROUP BY clause or be used in an aggregate function
LINE 1: SELECT COUNT(l.id) hits, l.course courseId, c.fullname cours...
^
SELECT COUNT(l.id) hits, l.course courseId, c.fullname coursename
FROM mdl_log l INNER JOIN mdl_course c ON l.course = c.id
GROUP BY courseId
ORDER BY hits DESC LIMIT 5000
[array (
)]
Error code: dmlreadexception

With this change, error is solved.

When you try to import and execute this xml report in a Moodle 3.5.2 an error appears: 

Debug info: ERROR: column "c.fullname" must appear in the GROUP BY clause or be used in an aggregate function
LINE 1: SELECT COUNT(l.id) hits, l.course courseId, c.fullname cours...
^
SELECT COUNT(l.id) hits, l.course courseId, c.fullname coursename
FROM mdl_log l INNER JOIN mdl_course c ON l.course = c.id
GROUP BY courseId
ORDER BY hits DESC LIMIT 5000
[array (
)]
Error code: dmlreadexception 

With this change, error is solved.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant