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

Navigation issue with user messaging preferences page

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.1.6, 2.2.3
    • Component/s: Messages
    • Labels:
      None
    • Database:
      PostgreSQL
    • Testing Instructions:
      Hide
      1. Go to 'my home' or 'site home'
      2. Open "My profile settings > Messaging"
      3. Check that the nav breadcrumbs say Home ► My profile settings ► Messaging"
      4. Check that "Messaging" is highlighted in the navigation block
      Show
      Go to 'my home' or 'site home' Open "My profile settings > Messaging" Check that the nav breadcrumbs say Home ► My profile settings ► Messaging" Check that "Messaging" is highlighted in the navigation block
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:
      MDL-28187_messaging_nav

      Description

      When "My profile settings > Messaging" page is open, the top navigation menu bar reads "Home ► My profile settings" instead of "Home ► My profile settings ► Messaging". Also corresponding Messaging item is not highlighted in the Settings navigation menu.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            This issue has been resolved by the linked issue.

            Show
            salvetore Michael de Raadt added a comment - This issue has been resolved by the linked issue.
            Hide
            kabalin Ruslan Kabalin added a comment -

            Hello Michael, I object, the issue has not been resolved by the linked one. The linked one solved "Enable messaging" and menu item appearance. This issue is about navigation.

            Show
            kabalin Ruslan Kabalin added a comment - Hello Michael, I object, the issue has not been resolved by the linked one. The linked one solved "Enable messaging" and menu item appearance. This issue is about navigation.
            Hide
            kabalin Ruslan Kabalin added a comment -

            In fact, the linked one should not be linked to this as they are sort of independent.

            Show
            kabalin Ruslan Kabalin added a comment - In fact, the linked one should not be linked to this as they are sort of independent.
            Hide
            andyjdavis Andrew Davis added a comment -

            Adding a branch that fixes this. We were just unnecessarily excluding some URL parameters.

            Show
            andyjdavis Andrew Davis added a comment - Adding a branch that fixes this. We were just unnecessarily excluding some URL parameters.
            Hide
            poltawski Dan Poltawski added a comment -

            Looks good to me.

            Ruslan, are you able to verify this?

            Show
            poltawski Dan Poltawski added a comment - Looks good to me. Ruslan, are you able to verify this?
            Hide
            kabalin Ruslan Kabalin added a comment -

            Tested. Works fine and solves the issue. Ready for integration.

            Show
            kabalin Ruslan Kabalin added a comment - Tested. Works fine and solves the issue. Ready for integration.
            Hide
            kabalin Ruslan Kabalin added a comment -

            Thanks Andrew

            Show
            kabalin Ruslan Kabalin added a comment - Thanks Andrew
            Hide
            andyjdavis Andrew Davis added a comment -

            Added branches for 2.2 and 2.1 stable. Putting this up for integration.

            Show
            andyjdavis Andrew Davis added a comment - Added branches for 2.2 and 2.1 stable. Putting this up for integration.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Andrew this has been integrated now

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Andrew this has been integrated now
            Hide
            salvetore Michael de Raadt added a comment -

            Test result: Success. Quickest test ever!

            Tested on 2.1, 2.2 and master.

            Show
            salvetore Michael de Raadt added a comment - Test result: Success. Quickest test ever! Tested on 2.1, 2.2 and master.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Congratulations are in order, you've made it, or at least your code has!
            It's now part of Moodle and both the git and cvs repositories have been updated.

            This issue is being marked as fixed and closed.

            Thank you.

            Show
            samhemelryk Sam Hemelryk added a comment - Congratulations are in order, you've made it, or at least your code has! It's now part of Moodle and both the git and cvs repositories have been updated. This issue is being marked as fixed and closed. Thank you.

              People

              • Votes:
                1 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

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