Loading...

XML

Word

Printable

Affected Branches:
MOODLE_37_STABLE
Fixed Branches:
MOODLE_35_STABLE, MOODLE_36_STABLE
Epic Link:
Group messaging follow ups for 3.7
Pull from Repository:
https://github.com/snake/moodle.git
Pull Main Branch:
~~MDL-65170~~-master
Pull Main Diff URL:
https://github.com/moodle/moodle/compare/9d4f4f005...snake:MDL-65170-master
Testing Instructions:
Hide

PLEASE TEST ON MASTER AND 36 ONLY! 35 is unit tested.

Disable site wide messaging (site admin -> messaging -> messaging settings -> messagingallusers)

Enrol 2 users as student in a course

Set course group mode to 'separate groups'

Auto create 2 groups, with 1 users in each

As student 1, try to search for student 2

Verify you don't see that user in the results.
Show
PLEASE TEST ON MASTER AND 36 ONLY! 35 is unit tested. Disable site wide messaging (site admin -> messaging -> messaging settings -> messagingallusers) Enrol 2 users as student in a course Set course group mode to 'separate groups' Auto create 2 groups, with 1 users in each As student 1, try to search for student 2 Verify you don't see that user in the results.

To replicate:

Disable site wide messaging (site admin -> messaging -> messaging settings -> messagingallusers)
Enrol 2 users as student in a course
Set course group mode to 'separate groups'
Auto create 2 groups, with 1 users in each
As student 1, try to search for student 2
Notice you can see them in the results. This shouldn't be possible because you can't view their course profile.

blocks

MDL-65169 Messaging: search should respect profile visibility when site-wide messaging is used

has been marked as being related by

MDL-64676 Fix old behat tests affected by 'messagingallusers' setting

has to be done before

MDL-64671 Add behat test for new message drawer to search

will help resolve

MDL-65023 Messaging search: course contacts are found by everyone, despite not sharing a course, nor being a contact.

Participants:: Anna Carissa Sadia, CiBoT, Eloy Lafuente (stronk7), Jake Dallimore, Mark Nelson, Sara Arjona (@sarjona)

Estimated:

Not Specified

Remaining:

Logged:

6h 20m