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

Some notifications are marked as headings in user profile page

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.4, 2.3.1, 2.4
    • Fix Version/s: 2.2.5, 2.3.2
    • Component/s: General
    • Labels:
    • Testing Instructions:
      Hide

      Login as student and access this url: http://yoursite.for.moodle/user/profile.php?id=## (replace ## with other user id).
      make sure the 'The details of this user are not available to you' message is not in heading tag.

      Modified the user table. set deleted for one of the user. then use that user id for the url above.
      make sure the 'This user account has been deleted' message is not in heading tag.

      Show
      Login as student and access this url: http://yoursite.for.moodle/user/profile.php?id=## (replace ## with other user id). make sure the 'The details of this user are not available to you' message is not in heading tag. Modified the user table. set deleted for one of the user. then use that user id for the url above. make sure the 'This user account has been deleted' message is not in heading tag.
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      When you for example try to view a user profile that you don't have access the notification below is outputted as heading.
      "The details of this user are not available to you"

      It should probably be a notification. This holds also for message: "This user account has been deleted"

        Gliffy Diagrams

          Activity

          Show
          juonio Juho Viitasalo added a comment - Patch: https://github.com/jiv-e/moodle/commit/c4d92ee07590cd2fe05a7a74645417f80201b447
          Hide
          salvetore Michael de Raadt added a comment -

          Thanks for spotting that. Having these notices as notifications will also help with consistency in theming.

          Show
          salvetore Michael de Raadt added a comment - Thanks for spotting that. Having these notices as notifications will also help with consistency in theming.
          Hide
          rwijaya Rossiani Wijaya added a comment -

          Thanks Juho for proving patch for this. It looks good.

          I created patch for 2.2 and 2.3 branches.

          Sending for peer-review.

          Show
          rwijaya Rossiani Wijaya added a comment - Thanks Juho for proving patch for this. It looks good. I created patch for 2.2 and 2.3 branches. Sending for peer-review.
          Hide
          rwijaya Rossiani Wijaya added a comment -

          It seems that the master branch is not up to date. I applied the patch to the latest version of master.

          Show
          rwijaya Rossiani Wijaya added a comment - It seems that the master branch is not up to date. I applied the patch to the latest version of master.
          Hide
          ankit_frenz Ankit Agarwal added a comment -

          Hi Rosie,
          This looks good.
          Thanks

          Show
          ankit_frenz Ankit Agarwal added a comment - Hi Rosie, This looks good. Thanks
          Hide
          rwijaya Rossiani Wijaya added a comment -

          Thanks Ankit for reviewing.

          Sending for integration review.

          Show
          rwijaya Rossiani Wijaya added a comment - Thanks Ankit for reviewing. Sending for integration review.
          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
          nebgor Aparup Banerjee added a comment -

          thanks, that's integrated into 22, 23 and master for testing now.

          Show
          nebgor Aparup Banerjee added a comment - thanks, that's integrated into 22, 23 and master for testing now.
          Hide
          nebgor Aparup Banerjee added a comment -

          grr, just noting (i only just spotted after integrating) that the commit message here didn't include the MDL-xxx but the merge message above it does. Please lets pay some attention to commit and adding the MDL

          Show
          nebgor Aparup Banerjee added a comment - grr, just noting (i only just spotted after integrating) that the commit message here didn't include the MDL-xxx but the merge message above it does. Please lets pay some attention to commit and adding the MDL
          Hide
          abgreeve Adrian Greeve added a comment -

          Tested on 2.2, 2.3 and master.
          The notification is no longer a header.
          No other problems encountered.
          Test passed.

          Show
          abgreeve Adrian Greeve added a comment - Tested on 2.2, 2.3 and master. The notification is no longer a header. No other problems encountered. Test passed.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          I'm so proud...of you, many thanks!

          http://youtu.be/n64CdfDRnZY

          Closing as fixed, ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - I'm so proud...of you, many thanks! http://youtu.be/n64CdfDRnZY Closing as fixed, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                10/Sep/12