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

Forum render breaks when viewing split discussion with missing author

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.7
    • 3.7
    • Forum
    • MOODLE_37_STABLE
    • MOODLE_37_STABLE
    • MDL-65574-master
    • Hide
      1. Log in as admin
      2. Create course with 2 standard forums
      3. Create 2 users and enrol them as students in the course
      4. Create a discussion in one of the forums
      5. Log in as one of the users
      6. View the discussion and add a reply
      7. Log in as the other user
      8. View the discussion and add a reply to the previous user's reply (not the original post)
      9. Log back in as admin
      10. View the discussion
      11. Split the post at the first user's reply
      12. After the split has created the new discussion move that discussion into the other forum
      13. View the first forum
      14. CONFIRM that it shows the original discussion
      15. CONFIRM the author in the "started by" column is the admin user
      16. CONFIRM the author in the "last post" column is the admin user
      17. View the second forum
      18. CONFIRM that it shows the newly created discussion from the split
      19. CONFIRM the author in the "started by" column is the first user
      20. CONFIRM the author in the "last post" column is the second user
      Show
      Log in as admin Create course with 2 standard forums Create 2 users and enrol them as students in the course Create a discussion in one of the forums Log in as one of the users View the discussion and add a reply Log in as the other user View the discussion and add a reply to the previous user's reply (not the original post) Log back in as admin View the discussion Split the post at the first user's reply After the split has created the new discussion move that discussion into the other forum View the first forum CONFIRM that it shows the original discussion CONFIRM the author in the "started by" column is the admin user CONFIRM the author in the "last post" column is the admin user View the second forum CONFIRM that it shows the newly created discussion from the split CONFIRM the author in the "started by" column is the first user CONFIRM the author in the "last post" column is the second user

    Description

      Due to MDL-65573 we can have discussion records who's "userid" values don't match any the discussion's post's authors.

      This mismatch causes the new rendering to break because it assumes that the "userid" field will always refer to an author of a post within the discussion.

      Attachments

        Issue Links

          Activity

            People

              ryanwyllie Ryan Wyllie
              ryanwyllie Ryan Wyllie
              Simey Lameze Simey Lameze
              Andrew Lyons Andrew Lyons
              Janelle Barcega Janelle Barcega
              Adrian Greeve, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                20/May/19

                Time Tracking

                  Estimated:
                  Original Estimate - 0 minutes
                  0m
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 5 hours, 6 minutes
                  5h 6m