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

Incorporate section name into course navigation when editing

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Course, Navigation, Usability
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      The attached patch adds support for using section names in the "Add a resource..." and "Add an activity..." navigation sections that are displayed when editing a course. After this patch is applied, custom section names will be displayed if they are set instead of generic names such as "Topic 2".

      MDL-22647 adds support for displaying custom section names throughout various parts of Moodle. Sorry, I missed this spot when writing the original patch for MDL-22647.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              blt04 Brandon Turner added a comment -

              This replaces the original patch in order to apply cleanly after MDL-22732

              Show
              blt04 Brandon Turner added a comment - This replaces the original patch in order to apply cleanly after MDL-22732
              Hide
              blt04 Brandon Turner added a comment -

              MDL-22732 removes the "Add a resource" and "Add an activity" links from the navigation, however the function to generate them was left behind in case it is needed in the future. As long as that code is around, I still think this patch applies. It should help prevent a regression should the add resources and add activity links ever be re-added.

              Show
              blt04 Brandon Turner added a comment - MDL-22732 removes the "Add a resource" and "Add an activity" links from the navigation, however the function to generate them was left behind in case it is needed in the future. As long as that code is around, I still think this patch applies. It should help prevent a regression should the add resources and add activity links ever be re-added.
              Hide
              skodak Petr Skoda added a comment -

              Sam: was this already discussed somewhere?

              Show
              skodak Petr Skoda added a comment - Sam: was this already discussed somewhere?
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Hi guys,

              Petr, thank you for bringing this to my attention.

              Brandon I think you are quite right, the code was specifically left there in case someone in the future wanted to bring it back.
              I've just reviewing the patch, applied and commit it.

              Cheers
              Sam

              Show
              samhemelryk Sam Hemelryk added a comment - Hi guys, Petr, thank you for bringing this to my attention. Brandon I think you are quite right, the code was specifically left there in case someone in the future wanted to bring it back. I've just reviewing the patch, applied and commit it. Cheers Sam

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    24/Nov/10