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

Ill formed breadcrumb on "This user's role assignments" page

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.2
    • Fix Version/s: 2.1.7, 2.2.4
    • Component/s: Navigation
    • Labels:
    • Testing Instructions:
      Hide
      1. Log in as an admin
      2. Browse to Settings > My profile settings > Roles > This users role assignments and check the navbar makes sense.
      3. Browse to a course
      4. Browse to Settings > My profile settings > Roles > This users role assignments (will have a URL relative to the course now) and check the navbar makes sense.
      5. In the course participants list click to view a users course profile.
      6. Browse to Settings > Profile settings for USERS_FULL_NAME_HERE > Roles > This users role assignments and check the navbar makes sense.
      7. In the URL change course=X to course=1 (the id of the front page) and check the navbar makes sense.
      Show
      Log in as an admin Browse to Settings > My profile settings > Roles > This users role assignments and check the navbar makes sense. Browse to a course Browse to Settings > My profile settings > Roles > This users role assignments (will have a URL relative to the course now) and check the navbar makes sense. In the course participants list click to view a users course profile. Browse to Settings > Profile settings for USERS_FULL_NAME_HERE > Roles > This users role assignments and check the navbar makes sense. In the URL change course=X to course=1 (the id of the front page) and check the navbar makes sense.
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:
      wip-MDL-32872-m23

      Description

      Breadcrumb of the page "This user's role assignments" is ill formed.

      Text "This user's role assignments" appears twice and the last one is not a link. Earlier one is a link and will take you to the current page.

      To replicate:

      1. Go to some course.
      2. In settings block go to: My profile settings -> Roles -> This user's role assignments
      3. See the breadcrumb

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks for the report!
            I've created a patch for this now and am putting it straight up for integration (simple changes here).

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks for the report! I've created a patch for this now and am putting it straight up for integration (simple changes here).
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Up for integration now

            Show
            samhemelryk Sam Hemelryk added a comment - Up for integration now
            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
            juonio Juho Viitasalo added a comment -

            Thanks for working on this!

            Show
            juonio Juho Viitasalo added a comment - Thanks for working on this!
            Hide
            poltawski Dan Poltawski added a comment -

            Thanks Sam, integrated this now

            Show
            poltawski Dan Poltawski added a comment - Thanks Sam, integrated this now
            Hide
            fred Frédéric Massart added a comment -

            Test successful on 2.1, 2.2 and master

            Show
            fred Frédéric Massart added a comment - Test successful on 2.1, 2.2 and master
            Hide
            juonio Juho Viitasalo added a comment -

            I noticed same kind of problem on wiki pages (edit, comment...) are these related?

            Show
            juonio Juho Viitasalo added a comment - I noticed same kind of problem on wiki pages (edit, comment...) are these related?
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            U P S T R E A M I Z E D !

            Many thanks for the hard work, closing this as fixed.

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - U P S T R E A M I Z E D ! Many thanks for the hard work, closing this as fixed. Ciao

              People

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

                Dates

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