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

          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