Messages Sending ignoring user preference

Bug #1830302 reported by Ghada El-Zoghbi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Ghada El-Zoghbi
19.04
Fix Released
High
Unassigned
19.10
Fix Released
High
Ghada El-Zoghbi

Bug Description

Mahara: 19.04.0
OS: Linux
DB: Postgres
Browser: FF

There is a bug that was introduced with the isolated institution. It is ignoring the user message preferences.

Scenario:

* User1: set User > Settings > Preferences - 'Messages from other users' = 'Allow people on my friends list to send me messages'

* User2: set User > Settings > Preferences - 'Messages from other users' = 'Allow people on my friends list to send me messages'

* Neither User1 nor User2 are friends of each other

* The site is not set up as isolated institution ($cfg->isolatedinstitutions = false;)

Log in as User1 and attempt to create a message to User1. Mahara allows the message to be sent.

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/10041

Changed in mahara:
assignee: nobody → Ghada El-Zoghbi (ghada-z)
Revision history for this message
Steven (stevens-q) wrote :

Commit hash: 28fc80f6ffeb5691ce292682771dbfb7736fed6d
Environment tested: Master
Browser tested: Chrome
Theme used: Raw | Default raw | Ocean | Modern | Primary school

PRECONDITIONS:
------------------------
1) Mahara User1 and User2 exist and both have the following set in their preferences
     a) 'Messages from other users' = 'Allow people on my friends list to send me messages'
2) Neither User1 nor User2 are friends of each other
3) The site is not set up as isolated institution ($cfg->isolatedinstitutions = false;)

TEST STEPS:
------------------------
1) Log in as User1 and attempt to create a message to User2. Mahara allows the message to be sent.
2) Verify that user cannot sent a message to User2 who is not a friend

Catalyst QA Approved ✔

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/10041
Committed: https://git.mahara.org/mahara/mahara/commit/28fc80f6ffeb5691ce292682771dbfb7736fed6d
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 28fc80f6ffeb5691ce292682771dbfb7736fed6d
Author: Ghada El-Zoghbi <email address hidden>
Date: Fri May 24 11:50:58 2019 +1000

Bug 1830302: Check if site is set for isolated institutions

When sending a message to another user, the check was
assuming that the site is set up as an isolated
institution and ignoring the user preference for
messages.

Sponsored by The Australian National University

behatnotneeded

Change-Id: I52936cd153336580146a9ac3eaf6e150a0e3affb

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "19.04_STABLE" branch: https://reviews.mahara.org/10044

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/10044
Committed: https://git.mahara.org/mahara/mahara/commit/ec0b86db6ec01f4573d3850db32f48664504096b
Submitter: Robert Lyon (<email address hidden>)
Branch: 19.04_STABLE

commit ec0b86db6ec01f4573d3850db32f48664504096b
Author: Ghada El-Zoghbi <email address hidden>
Date: Fri May 24 11:50:58 2019 +1000

Bug 1830302: Check if site is set for isolated institutions

When sending a message to another user, the check was
assuming that the site is set up as an isolated
institution and ignoring the user preference for
messages.

Sponsored by The Australian National University

behatnotneeded

Change-Id: I52936cd153336580146a9ac3eaf6e150a0e3affb
(cherry picked from commit 28fc80f6ffeb5691ce292682771dbfb7736fed6d)

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/10065

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "master" branch: https://reviews.mahara.org/10067

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.