Loading...

XML

Word

Printable

Affected Branches:
MOODLE_28_STABLE
Fixed Branches:
MOODLE_28_STABLE
Testing Instructions:
Hide

Toggle subscription to discussions with JS

Confirm that your changes were respected and no errors thrown

Try to (deliberately) remove the sesskey

Confirm that you get an error about missing sesskey checks
Show
Toggle subscription to discussions with JS Confirm that your changes were respected and no errors thrown Try to (deliberately) remove the sesskey Confirm that you get an error about missing sesskey checks

where is sesskey check? There is no need for optional_param('sesskey')

Also the \mod_forum\subscriptions::subscribe_user_to_discussion() should check if user is guest account.

has been marked as being related by

MDL-48029 subscribe_user_to_discussion should check for guests

Participants:: Andrew Lyons, Dan Poltawski, Eloy Lafuente (stronk7), Frédéric Massart, Petr Skoda, Simey Lameze