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

Messaging: visual artifact present when deleting a group message

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.8
    • Fix Version/s: 3.8
    • Component/s: Messages, Themes
    • Labels:
    • Testing Instructions:
      Hide

      Testing required for Boost only

      • Login as admin
      • Create a new course C1
      • Create a new user U1 and enrol into course C1
      • Create a new group G1 in course C1 en add user U1 and admin
      • Open the message drawer and send a few messages in group G1
      • Click one of the messages and click the Delete icon in the message drawer footer
      • On the delete dialogue verify the checkbox icon looks like this:
      Show
      Testing required for Boost only Login as admin Create a new course C1 Create a new user U1 and enrol into course C1 Create a new group G1 in course C1 en add user U1 and admin Open the message drawer and send a few messages in group G1 Click one of the messages and click the Delete icon in the message drawer footer On the delete dialogue verify the checkbox icon looks like this:
    • Affected Branches:
      MOODLE_38_STABLE
    • Fixed Branches:
      MOODLE_38_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-66812-master

      Description

      1. As admin
      2. Log in and create a course
      3. Enrol yourself, and some users.
      4. Create a group with all course participants
      5. Enable group messaging in the group
      6. Now, open messaging and select the group
      7. Send a message to the group
      8. Click the message to select it
      9. Click the trash icon
      10. Notice the confirmation pane looks like the attached screenshot:

      A git bisect says this was caused by MDL-64887, and reverting that issue does remedy this, however, that's hardly a solution.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              basbrands Bas Brands
              Reporter:
              jaked Jake Dallimore
              Peer reviewer:
              Simey Lameze
              Integrator:
              Jake Dallimore
              Tester:
              Janelle Barcega
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón, Bas Brands
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                18/Nov/19

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 1 hour, 37 minutes
                  1h 37m