Peer review role prevents site / institution admin from seeing all content on page during objectionable review
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mahara |
Fix Released
|
Medium
|
Kevin Dibble | ||
| 18.10 |
Fix Released
|
Medium
|
Unassigned | ||
| 19.04 |
Fix Released
|
Medium
|
Kevin Dibble | ||
Bug Description
Bug Summary: Content author has a public portfolio page -> Another user reports objectionable material = author sees warning on page -> does not edit page but adds the site admin as a peer reviewer -> who can not see the page details but only the headings
-------
Environment tested: Master
Browser tested: Chrome
Preconditions:
1. Content author creates a portfolio page
2. Another site user reports objectionable material on the page
Test Steps:
1. Content author does not change any content on the page
2. Content author shares the page with Site admin and assigns the role of peer for peer review
3. Content author notify site admin stuff has been changed
4. Site admin logs in and browses to the page with objectionable material
Actual result: The site admin only sees headings on the page and cannot see the actual page content
Expected result: Site admin should be able to see all page content even if they are assigned the role of peer for peer review or site admins should not be in the list of users that can do peer reviews
| Kristina Hoeppner (kris-hoeppner) wrote | #1 |
| summary: |
- Peer review role prevents site admin from seeing all content on page + Peer review role prevents site / institution admin from seeing all + content on page during objectionable review |
| Changed in mahara: | |
| status: | New → Confirmed |
| importance: | Undecided → Medium |
| Kristina Hoeppner (kris-hoeppner) wrote | #2 |
| Changed in mahara: | |
| milestone: | none → 18.10rc2 |
| Changed in mahara: | |
| milestone: | 18.10rc2 → 18.10.1 |
| tags: | added: bite-sized |
| Mahara Bot (dev-mahara) wrote A patch has been submitted for review | #3 |
| Mahara Bot (dev-mahara) wrote A change has been merged | #4 |
| Mahara Bot (dev-mahara) wrote A patch has been submitted for review | #5 |
| Mahara Bot (dev-mahara) wrote A change has been merged | #6 |
| Mahara Bot (dev-mahara) wrote A patch has been submitted for review | #7 |
| Mahara Bot (dev-mahara) wrote A change has been merged | #8 |
This is a tricky one. The site or institution admin can be peer assessor. In that role, they should only be able to see what the peer assessor is allowed to see.
However, when objectionable content is reported, the institution and site admin need to be able to see everything on the page that the portfolio author can see. That means any published peer assessments. They should not be able to see peer assessments that are in draft status as the portfolio author cannot see them either.