Moodle

A use profile's course list is arbitarily limited with no way of seeing complete list

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 1.8, 1.8.1, 2.3
  • Fix Version/s: None
  • Component/s: Administration
  • Labels:
    None
  • Affected Branches:
    MOODLE_18_STABLE, MOODLE_23_STABLE

Description

When viewing a user's profile, if the list of their courses exceeds 21 (I think) an ellipsis is displayed. This seems a bit useless because...
a) You can't click on the ellipsis to see the rest of the courses
b) The list isn't even in alphabetic order so it becomes even more arbitrary
c) the ellipsis is easy to miss, so you may not realise the list is truncated

Suggest either making this work properly, i.e., the ellipsis is clickable OR if the list is not considered important just put "too many courses to show" if it exceeds a certain amount. It would seem sensible to display them in alphabetic order too.

Without this it is difficult for an administrator to see all the courses a user is in if (for example) checking role assignments.

Issue Links

Activity

Hide
Howard Miller added a comment -

Illustrating the point...

Show
Howard Miller added a comment - Illustrating the point...
Hide
Matthew Davidson added a comment -

This will do a drop box style course selector....I've been trying to get Moodle to do this for some time.

Show
Matthew Davidson added a comment - This will do a drop box style course selector....I've been trying to get Moodle to do this for some time.
Hide
Charlie Owen (SonniesEdge) added a comment -

I added updated Matthew's code - hope it's useful...

http://tracker.moodle.org/browse/MDL-9446

Show
Charlie Owen (SonniesEdge) added a comment - I added updated Matthew's code - hope it's useful... http://tracker.moodle.org/browse/MDL-9446
Hide
Amanda Doughty added a comment -

We need this for all the same reasons stated by Howard. I have fixed it in 1.9 by changing the fixed value and using $CFG->mycoursesperpage instead. So:

if ($mycourses = get_my_courses($user->id, 'visible DESC,sortorder ASC', null, false, 21)) {

becomes:

if ($mycourses = get_my_courses($user->id, 'visible DESC,sortorder ASC', null, false, $CFG->mycoursesperpage)) {

and:

if($shown==20) {

becomes:

if($shown==($CFG->mycoursesperpage - 1)) {

BUT we really need it for Moodle 2.x

Is it ok to request that version 2.x is added to this issue or should I raise a new one linked to this?

Show
Amanda Doughty added a comment - We need this for all the same reasons stated by Howard. I have fixed it in 1.9 by changing the fixed value and using $CFG->mycoursesperpage instead. So: if ($mycourses = get_my_courses($user->id, 'visible DESC,sortorder ASC', null, false, 21)) { becomes: if ($mycourses = get_my_courses($user->id, 'visible DESC,sortorder ASC', null, false, $CFG->mycoursesperpage)) { and: if($shown==20) { becomes: if($shown==($CFG->mycoursesperpage - 1)) { BUT we really need it for Moodle 2.x Is it ok to request that version 2.x is added to this issue or should I raise a new one linked to this?
Hide
Howard Miller added a comment -

If the issue is substantially the same in Moodle 2 (I admit I haven't looked) then I think it's appropriate to add the additional versions.

Show
Howard Miller added a comment - If the issue is substantially the same in Moodle 2 (I admit I haven't looked) then I think it's appropriate to add the additional versions.
Hide
Amanda Doughty added a comment -

As the reporter, you have permission to edit this issue and include 2.1. Could you do it for me please? It is exactly the same in 2.1

Show
Amanda Doughty added a comment - As the reporter, you have permission to edit this issue and include 2.1. Could you do it for me please? It is exactly the same in 2.1

People

Dates

  • Created:
    Updated: