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

Users's Notes in participants menu has faulty breadcrumb navigation setup and missing links

    Details

      Description

      Clicking the "Notes" link under the Participants link in the Navigation menu,
      Display the Notes page with problematic (faulty) breadcrams navigation links.
      Please see attached screen capture (or link)

      Home / ► My courses / ► SOME COURSE / ► Participants / ► Notes / ► Participants / ► Notes 

      The Extra '''Participants / ► Notes ''' is redundant.
      and the "My Courses" is missing a link.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            salvetore Michael de Raadt added a comment -

            Hi, Nadav.

            Thanks for reporting that.

            I've put it on our backlog and we'll try to get to it as soon as we can.

            In the meantime feel free to help us work on this issue. If you do add a solution, please add a 'patch' label.

            Show
            salvetore Michael de Raadt added a comment - Hi, Nadav. Thanks for reporting that. I've put it on our backlog and we'll try to get to it as soon as we can. In the meantime feel free to help us work on this issue. If you do add a solution, please add a 'patch' label.
            Hide
            dobedobedoh Andrew Nicols added a comment -

            Cherry-picks cleany to MOODLE_21_STABLE

            Show
            dobedobedoh Andrew Nicols added a comment - Cherry-picks cleany to MOODLE_21_STABLE
            Hide
            phalacee Jason Fowler added a comment -

            Code makes sense, and should work

            Show
            phalacee Jason Fowler added a comment - Code makes sense, and should work
            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
            samhemelryk Sam Hemelryk added a comment -

            Thanks Andrew - spot on!
            This has been integrated now

            Cheers
            Sam

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Andrew - spot on! This has been integrated now Cheers Sam
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Tested during integration under as many conditions as I could imagine

            Show
            samhemelryk Sam Hemelryk added a comment - Tested during integration under as many conditions as I could imagine
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks!

            Now... disconnect, relax and enjoy the next days, yay!

            Closing...ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks! Now... disconnect, relax and enjoy the next days, yay! Closing...ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  9/Jan/12