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

          Attachments

            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