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

Forum pages don't always redirect properly after an error

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.9.1
    • Fix Version/s: 2.8.9, 2.9.3
    • Component/s: Forum
    • Labels:
    • Testing Instructions:
      Hide

      Create a discussion in a forum as User 1 (not an admin user)
      Reply to the discussion as User 2
      Try to delete the post as User 1
      On error message, click continue, should be redirected back to home page

      Show
      Create a discussion in a forum as User 1 (not an admin user) Reply to the discussion as User 2 Try to delete the post as User 1 On error message, click continue, should be redirected back to home page
    • Affected Branches:
      MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE, MOODLE_29_STABLE
    • Pull Master Branch:
      MDL-51253_forum_urls

      Description

      GIVEN I am a student
      AND have posted a discussion to a forum
      AND someone has replied to that discussion
      WHEN I go to delete my original post (that has a reply)
      THEN I should receive an error
      AND be redirected back to the discussion post

      Currently it is being redirected back to '/' the site/dashboard page

      this is because it throws an exception and get_exception_info expects a full qualified URL, whereas most of forum calls exceptions and redirects using for e.g. "discuss.php?d=132"

      A patch is provided changing urls in mod/forum used for exceptions/redirects to moodle_urls

        Attachments

          Activity

            People

            Assignee:
            berserkk Ben Kelada
            Reporter:
            berserkk Ben Kelada
            Peer reviewer:
            Jun Pataleta Jun Pataleta
            Integrator:
            David Monllaó David Monllaó
            Tester:
            cameron1729 cameron1729
            Participants:
            Component watchers:
            Andrew Lyons, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              9/Nov/15