Hide suspended users' pages from the "Latest Pages" block
Bug #1252885 reported by
Aaron Wells
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mahara |
Fix Released
|
Medium
|
Aaron Wells | ||
| 1.6 |
Won't Fix
|
Medium
|
Unassigned | ||
| 1.7 |
Won't Fix
|
Medium
|
Unassigned | ||
| 1.8 |
Won't Fix
|
Medium
|
Unassigned | ||
Bug Description
I noticed this one on mahara.org. When you suspend a user, their Pages are no longer accessible (going to one gives you an "access denied" error). BUT, they still show up in the "Latest Pages" block.
| Changed in mahara: | |
| milestone: | none → 1.9.0 |
| assignee: | nobody → Aaron Wells (u-aaronw) |
| importance: | Undecided → Medium |
| status: | New → Confirmed |
Revision history for this message
| Aaron Wells (u-aaronw) wrote | #1 |
Revision history for this message
| Aaron Wells (u-aaronw) wrote | #3 |
Revision history for this message
| Mahara Bot (dev-mahara) wrote A change has been merged | #4 |
Revision history for this message
| Aaron Wells (u-aaronw) wrote | #5 |
| no longer affects: | mahara/1.6 |
| no longer affects: | mahara/1.7 |
| Changed in mahara: | |
| status: | Confirmed → Fix Committed |
| Changed in mahara: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
The data in the newviews block comes from view_search(), a function which is meant to return only views that the specified user has access to. So, it would probably make the most sense to add a filter there to prevent the inclusion of views from suspended users, since the user does not have access to them.
On the other hand, would this cause performance issues? Adding one more table to the already-costly latest views query?