Mahara

default skin error: 403 Forbidden

Bug #1720260 reported by Cecilia Vela Gurovic
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
High
Robert Lyon
Mahara 17.10.0

Bug Description

Having skins enabled, when creating a new page if we go to setting/ Skins we will not be able to see the default skin icon and there will be an error in the browser's console:
GET http://maharatest/skin/thumb.php [HTTP/1.0 403 Forbidden 32ms]

Cecilia Vela Gurovic (ceciliavg)
Changed in mahara:
status: New → Confirmed
importance: Undecided → Low
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/8076

Revision history for this message
Robert Lyon (robertl-9) wrote :

This is a regression

Changed in mahara:
status: Confirmed → In Progress
importance: Low → Medium
importance: Medium → High
milestone: none → 17.10.0
Robert Lyon (robertl-9)
Changed in mahara:
milestone: 17.10.0 → 17.10rc2
assignee: nobody → Robert Lyon (robertl-9)
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/8076
Committed: https://git.mahara.org/mahara/mahara/commit/137666bc26daf382359b3d38b6acc61a98005209
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 137666bc26daf382359b3d38b6acc61a98005209
Author: Robert Lyon <email address hidden>
Date: Fri Sep 29 15:19:17 2017 +1300

Bug 1720260: no-skin image missing

This is due switching the default behaviour of a new Skin object to
being private rather than public.

But as we are just wanting to display the holder image we can get it
directly.

behatnotneeded

Change-Id: I26da51afb34a63bfc0cf097cf7819d4c503b9d74
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.10_STABLE" branch: https://reviews.mahara.org/8114

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

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

commit f0749ea636198b09c50999559fd13e64dec59764
Author: Robert Lyon <email address hidden>
Date: Fri Sep 29 15:19:17 2017 +1300

Bug 1720260: no-skin image missing

This is due switching the default behaviour of a new Skin object to
being private rather than public.

But as we are just wanting to display the holder image we can get it
directly.

behatnotneeded

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

Robert Lyon (robertl-9)
Changed in mahara:
milestone: 17.10rc2 → 17.10.0
Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.