mahara-contributors team mailing list archive

[Mahara Bot <1249123@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://reviews.mahara.org/2786
Committed: http://gitorious.org/mahara/mahara/commit/751d3e162e0fb821eeb893727c0ecb0553f6200e
Submitter: Son Nguyen (son.nguyen@xxxxxxxxxxxxxxx)
Branch:    1.8_STABLE

commit 751d3e162e0fb821eeb893727c0ecb0553f6200e
Author: Aaron Wells <aaronw@xxxxxxxxxxxxxxx>
Date:   Tue Nov 12 16:55:58 2013 +1300

Display the skins checkbox for the default institution config page

Bug 1249123

Change-Id: I959a291d13a32132aa1cd791f1b8f094c8bfd5c4

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1249123

Title:
  Users who are in "No Institution" can't use skins

Status in Mahara ePortfolio:
  Fix Committed
Status in Mahara 1.8 series:
  In Progress
Status in Mahara 1.9 series:
  Fix Committed

Bug description:
  To replicate:

  1. Set $cfg->skins = true;
  2. Go to the configuration screen for the default institution ("No Institution")
  3. Note that unlike other institutions, there is no skins checkbox displayed
  4. Click "Submit" on the configuration screen
  5. Log in as a user in the default institution
  6. Navigate to their Portfolio section

  Expected result: They should have a "Portfolio -> Skins" option.
  Actual result: They do not.

  The problem is that, although we set institution.skins to 1 by default
  during installation and upgrade, the absence of the skins checkbox on
  the default institution's config screen means that it gets set to 0
  the first time you save anything on that screen.

  Looking at the code, I realize that the reason the skins checkbox is
  hidden form the default institution was not on purpose, but because
  the checkbox made the most logical sense placed adjacent to the other
  theming-related settings, and those all happen to be settings that
  can't be overridden for the default institution.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1249123/+subscriptions