Wrong language used when requesting sb else's comment to be made public

Bug #1950067 reported by Kristina Hoeppner
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Doris Tam
21.10
Fix Released
High
Doris Tam
22.04
Fix Released
High
Doris Tam

Bug Description

This is a regression introduced - I believe - in bug #1940800.

When a portfolio author requests that a comment author make their comment public, the text still says 'This comment is private. The comment author requested that you make their comment public.' even though it should be 'This comment is private. You have requested that this comment be made public.' See number 7 in the screenshot at https://manual.mahara.org/en/21.04/portfolio/feedback.html

To replicate:

1. Create a page as Learner, allow comments, and share the page with a Educator. Leave moderation on 'No'.
2. Educator logs in and leaves a private comment on the page.
3. Learner logs in and views the comment and clicks 'Make comment public'. The Learner sees 'You have requested that this comment be made public.'
4. The Educator logs in and views the page

Expected result: The text below the comment now reads 'This comment is private. The portfolio owner requested that you make your comment public.'

Actual result: Instead, it reads 'This comment is private. The comment author requested that you make their comment public.' which suggests that the comment author (Educator) asked to have the comment made public when it was actually the portfolio author (Learner).

Tags: regression
Doris Tam (doristam)
description: updated
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "main" branch: https://reviews.mahara.org/12297

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

Reviewed: https://reviews.mahara.org/12297
Committed: https://git.mahara.org/mahara/mahara/commit/4652c975fcdbc7228766ada737f4da81f33ba824
Submitter: Robert Lyon (<email address hidden>)
Branch: main

commit 4652c975fcdbc7228766ada737f4da81f33ba824
Author: Doris Tam <email address hidden>
Date: Wed Dec 8 16:55:44 2021 +1300

Bug 1950067: Fix string for requesting sb elses's comment be public

Change-Id: I27d56a53efe631ffaaefa9e719bba0e5ac058708

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

Patch for "21.10_DEV" branch: https://reviews.mahara.org/12389

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

Reviewed: https://reviews.mahara.org/12389
Committed: https://git.mahara.org/mahara/mahara/commit/cde330e0e646cf3f23ba41dd174b72b5138282a3
Submitter: Robert Lyon (<email address hidden>)
Branch: 21.10_DEV

commit cde330e0e646cf3f23ba41dd174b72b5138282a3
Author: Doris Tam <email address hidden>
Date: Wed Dec 8 16:55:44 2021 +1300

Bug 1950067: Fix string for requesting sb elses's comment be public

Change-Id: I27d56a53efe631ffaaefa9e719bba0e5ac058708
(cherry picked from commit 4652c975fcdbc7228766ada737f4da81f33ba824)

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.