[MDL-63974] Messaging: Search updates don't return non-contact users unless a conversation exists - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Blocker
Resolution: Fixed
Affects Version/s: 3.6
Fix Version/s: 3.6
Component/s: Messages
Labels:
- ci
- triaged

Affected Branches:
MOODLE_36_STABLE
Fixed Branches:
MOODLE_36_STABLE
Epic Link:
Add the ability to message a group of users
Pull from Repository:
https://github.com/snake/moodle.git
Pull Main Branch:
~~MDL-63974~~-master
Pull Main Diff URL:
https://github.com/moodle/moodle/compare/dccda6546...snake:MDL-63974-master
Testing Instructions:
Hide

CI Covers the new search method, this is just a test to make sure the old deprecated method has been restored to its former glory.

Make sure you have a site with a range of users set up

Go to the legacy search page (message/index.php) as any user type

Make sure you're on the contacts tab and type 'a' or 'e' or 'i' in the search field

Verify you see a bunch of users come up in the results.
Show
CI Covers the new search method, this is just a test to make sure the old deprecated method has been restored to its former glory. Make sure you have a site with a range of users set up Go to the legacy search page (message/index.php) as any user type Make sure you're on the contacts tab and type 'a' or 'e' or 'i' in the search field Verify you see a bunch of users come up in the results.

Description

This was introduced by ~~MDL-63282~~. Beforehand, non contacts would be returned, regardless, but now, we don't get any unless a conversation exists. This is a problem because the search results are what allow us to create conversations with people. If we don't see users, then we can't init a conversation. This needs to be fixed.

Attachments

- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

MDL-63974.PNG
126 kB
15/Nov/18 1:48 PM

Issue Links

blocks

MDL-63303 Messaging: UI: Base version of new UI for 3.6

Closed

will help resolve

MDL-63288 Messaging: Extend user search API to properly check user visibility in course/site

Closed

Activity

People

Assignee:: Jake Dallimore

Reporter:: Jake Dallimore

Peer reviewer:: Damyon Wiese

Integrator:: Jun Pataleta

Tester:: Anna Carissa Sadia

Participants:: Anna Carissa Sadia, CiBoT, Damyon Wiese, David Monllaó, Jake Dallimore, Jun Pataleta

Votes:: 0 Vote for this issue

Watchers:: 6 Start watching this issue

Dates

Created:: 12/Nov/18 3:41 PM

Updated:: 17/Nov/18 12:42 AM

Resolved:: 17/Nov/18 12:42 AM

Fix Release Date:: 3/Dec/18

Time Tracking

Estimated:

Remaining:

Logged:

10m