error when creating a site skins

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

Bug Description

Having Skins enabled in the site

- Log in as site admin
- Go to Admin menu->Configure site->Skins
- Click on 'Create skin'
- Open the browser console
- Set title to 'Skin title' and save

Expected result: The site skin is saved and we are redirtected to the list of site skins
Actual result: The form gets stuck in loading and we see the following error message in the browsers console:

 [Show/hide message details.] ReferenceError: designskinform_body_background_image is not defined[Learn More] design.php:50:7
designskinform_callback
http://mahara/skin/design.php:50:7
Pieform/this.processForm/window.pieformHandlers[self.data.name]
http://mahara/lib/pieforms/static/core/pieforms.js:239:25
sendResult
http://mahara/skin/design.php:4:13
onload
http://mahara/skin/design.php:1:1

I think this could be related to not allowing to set a background image for a site skin and the changes made for https://bugs.launchpad.net/mahara/+bug/1782729

Changed in mahara:
milestone: none → 18.10.0
Changed in mahara:
importance: Undecided → High
status: New → Confirmed
Robert Lyon (robertl-9)
Changed in mahara:
milestone: 18.10.0 → 18.10rc1
Changed in mahara:
assignee: nobody → Cecilia Vela Gurovic (ceciliavg)
Changed in mahara:
status: Confirmed → In Progress
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/9202

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

Reviewed: https://reviews.mahara.org/9202
Committed: https://git.mahara.org/mahara/mahara/commit/5444b4ab4da13eb2f7df47524fddd1dcb4ec3048
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: master

commit 5444b4ab4da13eb2f7df47524fddd1dcb4ec3048
Author: Cecilia Vela Gurovic <email address hidden>
Date: Mon Oct 8 09:41:01 2018 +1300

Bug 1795758: site skin update not submitted by js

personal skins are submitted by js because
a background image can be added.
We dont allow this for a site skin,
so the submission method has to be changed.

behatnotneeded

Change-Id: I28f83d4bbc6318ec9ff1cc9dc899833d6ba0dcfb

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

Patch for "18.10_STABLE" branch: https://reviews.mahara.org/9206

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

Reviewed: https://reviews.mahara.org/9206
Committed: https://git.mahara.org/mahara/mahara/commit/d583d13699e7223d22103ea90bc8885404d313d2
Submitter: Cecilia Vela Gurovic (<email address hidden>)
Branch: 18.10_STABLE

commit d583d13699e7223d22103ea90bc8885404d313d2
Author: Cecilia Vela Gurovic <email address hidden>
Date: Mon Oct 8 09:41:01 2018 +1300

Bug 1795758: site skin update not submitted by js

personal skins are submitted by js because
a background image can be added.
We dont allow this for a site skin,
so the submission method has to be changed.

behatnotneeded

Change-Id: I28f83d4bbc6318ec9ff1cc9dc899833d6ba0dcfb
(cherry picked from commit 5444b4ab4da13eb2f7df47524fddd1dcb4ec3048)

Changed in mahara:
milestone: 18.10rc1 → 18.10.0
Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
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.

Other bug subscribers

Remote bug watches

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