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

Link sent in message email for changing message preferences error

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6.6, 2.7.3, 2.8.1, 2.9
    • Fix Version/s: 2.7.5, 2.8.3
    • Component/s: Messages
    • Labels:
    • Testing Instructions:
      Hide
      1. Make sure you are logged out
      2. Request the URL http://xxx/message/edit.php
      3. Enter your username
      4. Make sure that you are redirected to message/edit.php and the page is correctly displayed
      Show
      Make sure you are logged out Request the URL http://xxx/message/edit.php Enter your username Make sure that you are redirected to message/edit.php and the page is correctly displayed
    • Workaround:
      Hide

      To fix the issue, page message/edit.php:
      L28, change to $userid = optional_param('id', 0, PARAM_INT);
      L30:
      if ($userid == 0)

      { $userid = $USER->id; }
      Show
      To fix the issue, page message/edit.php: L28, change to $userid = optional_param('id', 0, PARAM_INT); L30: if ($userid == 0) { $userid = $USER->id; }
    • Affected Branches:
      MOODLE_26_STABLE, MOODLE_27_STABLE, MOODLE_28_STABLE, MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE, MOODLE_28_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-48776-master

      Description

      At the bottom of any email to the user is the following link:

      "To change your preferences for receiving these emails, go to My Profile Settings -> Messaging (here: /message/edit.php)"

      If the user is logged out of the website and clicks on the link the the user is directed to the login page, after logging in the following error message is displayed:

      "!Invalid user id"

        Attachments

          Activity

            People

            Assignee:
            moodle.com moodle.com
            Reporter:
            oleg Oleg Demeshev
            Peer reviewer:
            Dave Cooper
            Integrator:
            David Monllaó
            Tester:
            Damyon Wiese
            Participants:
            Component watchers:
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              2/Feb/15