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

          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: