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

Show course full name in navigation block

    Details

    • Testing Instructions:
      Hide
      1. Log in as an admin.
      2. Expand some things in the navigation and check that courses are shown using their short names.
      3. Browse to Settings > Appearance > Navigation
      4. Enable navshowfullcoursenames.
      5. Expand some things in the navigation and check that courses are new shown using their full names.
      Show
      Log in as an admin. Expand some things in the navigation and check that courses are shown using their short names. Browse to Settings > Appearance > Navigation Enable navshowfullcoursenames. Expand some things in the navigation and check that courses are new shown using their full names.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull Master Branch:
      wip-MDL-37644-m25

      Description

      As MDL-26004 was not implemented, and following Aparup Banerjee suggested, I am renewing this issue, which has been requested from one of our clients:

      Course short names are often somewhat cryptic as they must be unique and often use codes from other sources e.g. prospectus or amalgamations of various elements e.g. award, date, version, instance in their construction.

      Where these appear in isolation i.e. in the navigation block without a supporting full name they are not very useful to students who use the navigation block to navigate the site.

      Suggestions:

      The full name should appear in the block.

      Full name or short name should be an admin configurable item.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            samhemelryk Sam Hemelryk added a comment -

            Hi Antonio,

            Thanks for creating this issue.
            I've created a simple little patch to implement this request now.
            It's up for peer-review presently, perhaps if you or someone on your team has a few moments they could peer-review this for me and we can put it up for integration. If not no probs someone will get to it.

            Many thanks
            Sam

            Show
            samhemelryk Sam Hemelryk added a comment - Hi Antonio, Thanks for creating this issue. I've created a simple little patch to implement this request now. It's up for peer-review presently, perhaps if you or someone on your team has a few moments they could peer-review this for me and we can put it up for integration. If not no probs someone will get to it. Many thanks Sam
            Hide
            rwijaya Rossiani Wijaya added a comment -

            This looks great Sam.

            [y] Syntax
            [y] Output
            [y] Whitespace
            [y] Language
            [-] Databases
            [-] Testing
            [-] Security
            [-] Documentation
            [y] Git
            [y] Sanity check

            +1 for integration.

            Show
            rwijaya Rossiani Wijaya added a comment - This looks great Sam. [y] Syntax [y] Output [y] Whitespace [y] Language [-] Databases [-] Testing [-] Security [-] Documentation [y] Git [y] Sanity check +1 for integration.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Rosie, putting it up for integration now.

            Antonio, please have a look if you are able to still. It'd be great to know this is what you had in mind.

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Rosie, putting it up for integration now. Antonio, please have a look if you are able to still. It'd be great to know this is what you had in mind.
            Hide
            avilela Antonio Vilela added a comment -

            Thanks Sam,

            It is exactly what we needed!! Our customer will be very happy

            Thanks for your quick answer. This was the fastest answer to an issue I can remember

            Show
            avilela Antonio Vilela added a comment - Thanks Sam, It is exactly what we needed!! Our customer will be very happy Thanks for your quick answer. This was the fastest answer to an issue I can remember
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            TIA and ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Note this is only for master (upcoming 2.5).

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Note this is only for master (upcoming 2.5).
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Only one (unsure) objection. Is it 'navshowfullcoursenames_help' or 'navshowfullcoursenames_desc' ? I think the later is the preferred one, although really we have a nice mess there.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Only one (unsure) objection. Is it 'navshowfullcoursenames_help' or 'navshowfullcoursenames_desc' ? I think the later is the preferred one, although really we have a nice mess there.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated, some day we'll normalize all them, but surely not today.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, some day we'll normalize all them, but surely not today.
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Sorry guys it seems this is not integrated.

            Show
            rajeshtaneja Rajesh Taneja added a comment - Sorry guys it seems this is not integrated.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Wow, apologies! It's now available.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Wow, apologies! It's now available.
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Thanks Eloy and Sam,

            I can see course fullname now.

            Cheers.

            Show
            rajeshtaneja Rajesh Taneja added a comment - Thanks Eloy and Sam, I can see course fullname now. Cheers.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            A brilliant future is awaiting us out there, better with your code. Let's look towards the future together, this is now closed.

            (and won't be revisiting it unless some regression is found)

            Thanks and ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - A brilliant future is awaiting us out there, better with your code. Let's look towards the future together, this is now closed. (and won't be revisiting it unless some regression is found) Thanks and ciao
            Hide
            eder.exe Eder dos Santos added a comment -

            Hello! How could I download this parch and apply it into a 2.4 installation? Thanks in advance.

            Show
            eder.exe Eder dos Santos added a comment - Hello! How could I download this parch and apply it into a 2.4 installation? Thanks in advance.
            Hide
            marycooch Mary Cooch added a comment -

            Removing docs_required label as this is documented here http://docs.moodle.org/25/en/Navigation

            Show
            marycooch Mary Cooch added a comment - Removing docs_required label as this is documented here http://docs.moodle.org/25/en/Navigation

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/13