Changeset 5258

Timestamp:

Mar 22, 2010, 3:11:10 PM (14 years ago)

Author:

plg

Message:

feature 1514: improvement, impossible to delete a theme that is required
by another installed theme.

Location:

trunk/admin

Files:

• include/themes.class.php (modified) (3 diffs)
• themes/default/template/footer.tpl (modified) (1 diff)
• themes/default/template/themes_installed.tpl (modified) (4 diffs)
• themes_installed.php (modified) (2 diffs)

trunk/admin/include/themes.class.php

@@ -141 +141 @@
           break;
         }
+
+        $children = $this->get_children_themes($theme_id);
+        if (count($children) > 0)
+        {
+          array_push(
+            $errors,
+            sprintf(
+              l10n('Impossible to delete this theme. Other themes depends on it: %s'),
+              implode(', ', $children)
+              )
+            );
+          break;
+        }
         
         if (!$this->deltree(PHPWG_THEMES_PATH.$theme_id))
@@ -155 +168 @@
     return $errors;
   }
+
+  function get_children_themes($theme_id)
+  {
+    $children = array();
+    
+    foreach ($this->fs_themes as $test_child)
+    {
+      if (isset($test_child['parent']) and $test_child['parent'] == $theme_id)
+      {
+        array_push($children, $test_child['name']);
+      }
+    }
+
+    return $children;
+  } 
 
   function set_default_theme($theme_id)
@@ -267 +295 @@
             list( , $extension) = explode('extension_view.php?eid=', $theme['uri']);
             if (is_numeric($extension)) $theme['extension'] = $extension;
+          }
+          if (preg_match('/["\']parent["\'][^"\']+["\']([^"\']+)["\']/', $theme_data, $val))
+          {
+            $theme['parent'] = $val[1];
           }
 

trunk/admin/themes/default/template/footer.tpl

@@ -46 +46 @@
 <script type='text/javascript'>
   $(function() {
-    $('#pwgHead A, #footer A, .themeActions A').tipTip({
+    $('#pwgHead A, #footer A, .themeActions A, .themeActions SPAN').tipTip({
         'delay' : 0,
         'fadeIn' : 200,

trunk/admin/themes/default/template/themes_installed.tpl

@@ -5 +5 @@
 <div id="themesContent">
 
-<h3>Active Themes</h3>
+<fieldset>
+<legend>Active Themes</legend>
 {if isset($active_themes)}
 <div class="themeBoxes">
@@ -22 +23 @@
 </div> <!-- themeBoxes -->
 {/if}
+</fieldset>
 
 {if isset($inactive_themes)}
-<h3>Inactive Themes</h3>
+<fieldset>
+<legend>Inactive Themes</legend>
 <div class="themeBoxes">
 {foreach from=$inactive_themes item=theme}
@@ -32 +35 @@
     <div class="themeActions">
       <a href="{$activate_baseurl}{$theme.id}" title="{'Make this theme available to users'|@translate}">{'Activate'|@translate}</a>
-      | <a href="{$delete_baseurl}{$theme.id}" onclick="return confirm('{'Are you sure?'|@translate|@escape:javascript}');" title="{'Delete this theme'|@translate}">{'Delete'|@translate}</a>
+      |
+  {if $theme.deletable}
+      <a href="{$delete_baseurl}{$theme.id}" onclick="return confirm('{'Are you sure?'|@translate|@escape:javascript}');" title="{'Delete this theme'|@translate}">{'Delete'|@translate}</a>
+  {else}
+      <span title="{$theme.delete_tooltip}">{'Delete'|@translate}</span>
+  {/if}
     </div>
     
@@ -38 +46 @@
 {/foreach}
 </div> <!-- themeBoxes -->
+</fieldset>
 {/if}
 

trunk/admin/themes_installed.php

@@ -69 +69 @@
 $inactive_themes = array();
 
-foreach($themes->fs_themes as $theme_id => $fs_theme)
+foreach ($themes->fs_themes as $theme_id => $fs_theme)
 {
   if ($theme_id == 'default')
@@ -90 +90 @@
   else
   {
+    $children = $themes->get_children_themes($theme_id);
+    
+    if (count($children) > 0)
+    {
+      $fs_theme['deletable'] = false;
+      
+      $fs_theme['delete_tooltip'] = sprintf(
+        l10n('Impossible to delete this theme. Other themes depends on it: %s'),
+        implode(', ', $children)
+        );
+    }
+    else
+    {
+      $fs_theme['deletable'] = true;
+    }
+    
     array_push($inactive_themes, $fs_theme);
   }