- Patch #38981 by Robert: add permission to enable/disbale theme switching by users.

1 files changed, 26 insertions, 21 deletions

diff --git a/modules/system.module b/modules/system.module
index 248ae68a4..e22afafed 100644
--- a/modules/system.module
+++ b/modules/system.module
@@ -47,7 +47,7 @@ function system_help($section) {
  * Implementation of hook_perm().
  */
 function system_perm() {
-  return array('administer site configuration', 'access administration pages');
+  return array('administer site configuration', 'access administration pages', 'select different theme');
 }
 
 /**
@@ -159,30 +159,35 @@ function system_test() {
  */
 function system_user($type, $edit, &$user, $category = NULL) {
   if ($type == 'form' && $category == 'account') {
-    $themes = list_themes();
-    ksort($themes);
-
-    // Reset to follow site default theme if user selects the site default
-    if ($key == variable_get('theme_default', 'bluemarine')) {
-      $key = '';
-      if ($edit['theme'] == variable_get('theme_default', 'bluemarine')) {
-        $edit['theme'] = '';
-      }
-    }
+    if (user_access('select different theme')) {
+      $themes = list_themes();
 
-    $form['themes'] = array(
-      '#type' => 'fieldset', '#title' => t('Theme configuration'), '#description' => t('Selecting a different theme will change the look and feel of the site.'), '#weight' => 2, '#collapsible' => TRUE, '#theme' => 'system_user');
+      if (count($themes) > 1) {
+        ksort($themes);
 
-    foreach ($themes as $info) {
-      $info->screenshot = dirname($info->filename) . '/screenshot.png';
-      $screenshot = file_exists($info->screenshot) ? theme('image', $info->screenshot, t('Screenshot for %theme theme', array('%theme' => $info->name)), '', array('class' => 'screenshot'), false) : t('no screenshot');
+        // Reset to follow site default theme if user selects the site default
+        if ($key == variable_get('theme_default', 'bluemarine')) {
+          $key = '';
+          if ($edit['theme'] == variable_get('theme_default', 'bluemarine')) {
+            $edit['theme'] = '';
+          }
+        }
 
-      $form['themes'][$info->name]['screenshot'] = array('#type' => 'markup', '#value' => $screenshot);
-      $form['themes'][$info->name]['description'] = array('#type' => 'item', '#title' => $info->name,  '#value' => dirname($info->filename));
-      $options[$info->name] = '';
-    }
+        $form['themes'] = array(
+          '#type' => 'fieldset', '#title' => t('Theme configuration'), '#description' => t('Selecting a different theme will change the look and feel of the site.'), '#weight' => 2, '#collapsible' => TRUE, '#theme' => 'system_user');
 
-    $form['themes']['theme'] = array('#type' => 'radios', '#options' => $options, '#default_value' => $edit['theme'] ? $edit['theme'] : variable_get('theme_default', 'bluemarine'));
+        foreach ($themes as $info) {
+          $info->screenshot = dirname($info->filename) . '/screenshot.png';
+          $screenshot = file_exists($info->screenshot) ? theme('image', $info->screenshot, t('Screenshot for %theme theme', array('%theme' => $info->name)), '', array('class' => 'screenshot'), false) : t('no screenshot');
+
+         $form['themes'][$info->name]['screenshot'] = array('#type' => 'markup', '#value' => $screenshot);
+          $form['themes'][$info->name]['description'] = array('#type' => 'item', '#title' => $info->name,  '#value' => dirname($info->filename));
+          $options[$info->name] = '';
+        }
+
+        $form['themes']['theme'] = array('#type' => 'radios', '#options' => $options, '#default_value' => $edit['theme'] ? $edit['theme'] : variable_get('theme_default', 'bluemarine'));
+      }
+    }
 
     if (variable_get('configurable_timezones', 1)) {
       $zones = _system_zonelist();