Moodle

[Patch] Sort courses on profile

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.9.3, 1.9.4, 1.9.5, 1.9.6
  • Fix Version/s: 1.9.6
  • Component/s: Other, Usability
  • Labels:
    None
  • Affected Branches:
    MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE

Description

still working on a project at Federal University of Santa Catarina, in Brasil. Some users reported that the course list on profiles isn't on the same order of my moodle.
The attached patch correct this, passing 'visible DESC, sortorder ASC' to $sort param instead of null. Maybe the function should be corrected to handle null differently from an empty string, but i don't have much time to test it now.

Activity

Hide
Helen Foster added a comment -

Daniel, thanks for your report and patch.

Reassigning to moodle.com for consideration.

Show
Helen Foster added a comment - Daniel, thanks for your report and patch. Reassigning to moodle.com for consideration.
Hide
Daniel Neis added a comment -

here is the patch to 1.9.5 version

Show
Daniel Neis added a comment - here is the patch to 1.9.5 version
Hide
Martin Dougiamas added a comment -

Thanks, Daniel. Great little patches!

I've checked this in.

Show
Martin Dougiamas added a comment - Thanks, Daniel. Great little patches! I've checked this in.
Hide
Helen Foster added a comment -

Daniel and Martin, thanks again for your attention to detail. I tested the latest 1.9.5+ and found that courses are correctly listed in the same order on the profile page and front page.

Show
Helen Foster added a comment - Daniel and Martin, thanks again for your attention to detail. I tested the latest 1.9.5+ and found that courses are correctly listed in the same order on the profile page and front page.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: