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
    • Rank:
      47337

      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.

        Activity

        Hide
        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
        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
        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
        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
        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
        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
        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
        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
        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
        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
        Eloy Lafuente (stronk7) added a comment -

        Note this is only for master (upcoming 2.5).

        Show
        Eloy Lafuente (stronk7) added a comment - Note this is only for master (upcoming 2.5).
        Hide
        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
        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
        Eloy Lafuente (stronk7) added a comment -

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

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

        Sorry guys it seems this is not integrated.

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

        Wow, apologies! It's now available.

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

        Thanks Eloy and Sam,

        I can see course fullname now.

        Cheers.

        Show
        Rajesh Taneja added a comment - Thanks Eloy and Sam, I can see course fullname now. Cheers.
        Hide
        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
        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 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 dos Santos added a comment - Hello! How could I download this parch and apply it into a 2.4 installation? Thanks in advance.
        Hide
        Mary Cooch added a comment -

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

        Show
        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: