Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-45524

Better calculation of expandable courses in the navigation

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5.6, 2.6.3, 2.7
    • Fix Version/s: 2.6.4, 2.7.1
    • Component/s: Navigation
    • Labels:
    • Testing Instructions:
      Hide
      1. Run behat tests
      2. On a new site create a single course and log out.
      3. Without logging in expand "Courses" in the navigation block
      4. Check that the course you created is shown, but with a square and not an arrow (really check it isn't expandable)
      Show
      Run behat tests On a new site create a single course and log out. Without logging in expand "Courses" in the navigation block Check that the course you created is shown, but with a square and not an arrow (really check it isn't expandable)
    • Affected Branches:
      MOODLE_25_STABLE, MOODLE_26_STABLE, MOODLE_27_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE, MOODLE_27_STABLE
    • Pull Master Branch:
      45524-28
    • Pull Master Diff URL:

      Description

      This is a follow on issue from MDL-44711.

      In that issue there was an execption shown to users when they expanded a course that they couldn't access.
      In that issue we fixed the exception handling so that the user was shown an empty node instead of an exception.
      However the courses are still shown as expandable (not a biggy).

      This issue is about better display of the courses so that courses are shown as expandable only if you can access them.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              samhemelryk Sam Hemelryk
              Reporter:
              samhemelryk Sam Hemelryk
              Integrator:
              Damyon Wiese Damyon Wiese
              Tester:
              Simey Lameze Simey Lameze
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                14/Jul/14