All portfolios of others are shown in the "Default" theme rather than the site or institution theme

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

Bug Description

https://mahara.org/interaction/forum/topic.php?id=7937&post=31909

Mahara 17.04+

When somebody else's page is viewed, the page theme is shown in the "Default" theme rather than the institution or site theme.

To replicate:

1. Choose "Primary school" as site theme. Don't allow page themes.
2. Log in as a user who is a member of "No institution" (but the same would also go if he were in an institution)
3. Create a page and share it with "Registered users".
4. Log in as another user who is a member of "No institution" and view the page.

Expected result: You see the page in the "Primary school" theme.
Actual result: You see the "Default" theme.

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

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

Reviewed: https://reviews.mahara.org/7724
Committed: https://git.mahara.org/mahara/mahara/commit/ace8b80f4384dbbde619bda23911670e6751a32e
Submitter: Dmitrii Metelkin (<email address hidden>)
Branch: master

commit ace8b80f4384dbbde619bda23911670e6751a32e
Author: Robert Lyon <email address hidden>
Date: Mon May 8 09:25:34 2017 +1200

Bug 1688693: Need to cast the $owner an integer

Before checking for their theme

behatnotneeded

Change-Id: I25190cb69f6ff04c160f6d8f8980a42b4360830b
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 "17.04_STABLE" branch: https://reviews.mahara.org/7732

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

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

commit 0c49935be4423c60f589fdc4e5c3045e83b937f2
Author: Robert Lyon <email address hidden>
Date: Mon May 8 09:25:34 2017 +1200

Bug 1688693: Need to cast the $owner an integer

Before checking for their theme

behatnotneeded

Change-Id: I25190cb69f6ff04c160f6d8f8980a42b4360830b
Signed-off-by: Robert Lyon <email address hidden>
(cherry picked from commit ace8b80f4384dbbde619bda23911670e6751a32e)

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.