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

context_course::get_context_name should respect $CFG->courselistshortnames

    Details

    • Testing Instructions:
      Hide

      This is about what happens when the setting "Display extended course names" setting on Site administration > Appearance > Courses is turned on. Really you should test both with that on, to make sure it works, and with it off, to make sure there are no regressions.

      1. Go to a user's profile.

      2. Click the Roles > This user's role assignments link in the setting block.

      3. Verify that any course names are shown with short-name first (if the admin setting is on).

      4. Check index.php, course/index.php and course/category.php, to make sure that course names there are also shown with the short-name first (if the admin setting is on).

      Show
      This is about what happens when the setting "Display extended course names" setting on Site administration > Appearance > Courses is turned on. Really you should test both with that on, to make sure it works, and with it off, to make sure there are no regressions. 1. Go to a user's profile. 2. Click the Roles > This user's role assignments link in the setting block. 3. Verify that any course names are shown with short-name first (if the admin setting is on). 4. Check index.php, course/index.php and course/category.php, to make sure that course names there are also shown with the short-name first (if the admin setting is on).
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      This got reported because admin/roles/usersroles.php does not display the course short name even if this config options is turned on.

      I think it makes sense to respect that option whenever we print the non-short version of the course context name.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  5 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    10/Sep/12