Group pages not shown to non-group members even when shared

Bug #1696696 reported by Kristina Hoeppner
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Cecilia Vela Gurovic
17.04
Fix Released
High
Unassigned
17.10
Fix Released
High
Cecilia Vela Gurovic

Bug Description

Mahara 17.04+

To replicate:

1. Create a page in a group.
2. Share the page with "Registered users" on top of it being shared with the group.
3. Leave the group.
4. Click "Pages and collections" in the group and click the page.

Expected result: You are taken to the page.
Actual result: You get "invalid parameter" A required parameter is missing or malformed. The 'id' parameter is not an integer

You can still access the page via "Shared with me" though when you tick "Registered users".

The URL in the group turned into:

/collection/views.php?id= (id parameter is missing)

Whereas the URL is /view/view.php?id=16 when I view it via "Shared with me"/

Tags: regression
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/7839

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

Reviewed: https://reviews.mahara.org/7839
Committed: https://git.mahara.org/mahara/mahara/commit/4ff43e572d4646c62ad5264241f4257efc575afe
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 4ff43e572d4646c62ad5264241f4257efc575afe
Author: Cecilia Vela Gurovic <email address hidden>
Date: Wed Jun 28 16:28:00 2017 +1200

Bug 1696696: fix broken link in shared group page

behatnotneeded

Change-Id: I48b95600099ddcc9d89841eef4ec5a75c9bc52a9

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

Patch for "17.04_STABLE" branch: https://reviews.mahara.org/7843

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

Reviewed: https://reviews.mahara.org/7843
Committed: https://git.mahara.org/mahara/mahara/commit/d05a6c137632117c88457250601782900d1b9b9f
Submitter: Robert Lyon (<email address hidden>)
Branch: 17.04_STABLE

commit d05a6c137632117c88457250601782900d1b9b9f
Author: Cecilia Vela Gurovic <email address hidden>
Date: Wed Jun 28 16:28:00 2017 +1200

Bug 1696696: fix broken link in shared group page

behatnotneeded

Change-Id: I48b95600099ddcc9d89841eef4ec5a75c9bc52a9
(cherry picked from commit 4ff43e572d4646c62ad5264241f4257efc575afe)

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

Patch for "17.04_STABLE" branch: https://reviews.mahara.org/7864

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

Reviewed: https://reviews.mahara.org/7864
Committed: https://git.mahara.org/mahara/mahara/commit/57e566af3e530506d48f82160745d0f8b5f7ce66
Submitter: Aaron Wells (<email address hidden>)
Branch: 17.04_STABLE

commit 57e566af3e530506d48f82160745d0f8b5f7ce66
Author: Robert Lyon <email address hidden>
Date: Wed Jul 12 06:35:31 2017 +1200

Bug 1696696: fix broken link in shared group page

Same fix for the old_raw branch in 17.04

behatnotneeded

Change-Id: Ib5a574429c716b40188a9f6f2ccb1bb91dee8c35
Signed-off-by: Robert Lyon <email address hidden>

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/8089

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

Reviewed: https://reviews.mahara.org/8089
Committed: https://git.mahara.org/mahara/mahara/commit/912600e165cff08c2806f6212683d01735f9935d
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 912600e165cff08c2806f6212683d01735f9935d
Author: Cecilia Vela Gurovic <email address hidden>
Date: Wed Oct 4 16:09:35 2017 +1300

Bug 1720269: old_raw patches for bugs: (groups)

Bug 1714830
Bug 1696696
Bug 1681952
Bug 1666685

behatnotneeded

Change-Id: I4ffbabb28cc00451760134b478758a5c20b86819

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

Patch for "17.10_STABLE" branch: https://reviews.mahara.org/8098

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

Reviewed: https://reviews.mahara.org/8098
Committed: https://git.mahara.org/mahara/mahara/commit/96d9cdc77833bf039d6a6680080fc94dc93c9468
Submitter: Robert Lyon (<email address hidden>)
Branch: 17.10_STABLE

commit 96d9cdc77833bf039d6a6680080fc94dc93c9468
Author: Cecilia Vela Gurovic <email address hidden>
Date: Wed Oct 4 16:09:35 2017 +1300

Bug 1720269: old_raw patches for bugs: (groups)

Bug 1714830
Bug 1696696
Bug 1681952
Bug 1666685

behatnotneeded

Change-Id: I4ffbabb28cc00451760134b478758a5c20b86819
(cherry picked from commit 912600e165cff08c2806f6212683d01735f9935d)

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.