Changeset 2226

Timestamp:

02/29/08 00:41:47 (5 years ago)

Author:

rub

Message:

0000809: Use more php classes implementation

Use class for tabsheet like grum class
Change way for tabsheet

Location:

trunk

Files:

• admin/cat_options.php (modified) (2 diffs)
• admin/configuration.php (modified) (3 diffs)
• admin/include/functions_history.inc.php (modified) (3 diffs)
• admin/include/functions_waiting.inc.php (modified) (3 diffs)
• admin/include/tabsheet.class.php (moved) (moved from trunk/admin/include/functions_tabsheet.inc.php) (2 diffs)
• admin/notification_by_mail.php (modified) (3 diffs)
• template/yoga/admin/cat_options.tpl (modified) (1 diff)
• template/yoga/admin/tabsheet.tpl (modified) (1 diff)

trunk/admin/cat_options.php

@@ -31 +31 @@
 
 include_once(PHPWG_ROOT_PATH.'admin/include/functions.php');
-include_once(PHPWG_ROOT_PATH.'admin/include/functions_tabsheet.inc.php');
+include_once(PHPWG_ROOT_PATH.'admin/include/tabsheet.class.php');
 
 // +-----------------------------------------------------------------------+
@@ -160 +160 @@
  );
 
+// TabSheet
+$tabsheet = new tabsheet();
 // TabSheet initialization
 $opt_link = $link_start.'cat_options&section=';
-$tabsheet = array
-(
-  'status' => array
-   (
-    'caption' => l10n('cat_security'),
-    'url' => $opt_link.'status'
-   ),
-  'visible' => array
-   (
-    'caption' => l10n('lock'),
-    'url' => $opt_link.'visible'
-   ),
-  'upload' => array
-   (
-    'caption' => l10n('upload'),
-    'url' => $opt_link.'upload'
-   ),
-  'comments' => array
-   (
-    'caption' => l10n('comments'),
-    'url' => $opt_link.'comments'
-   ),
-);
-
+$tabsheet->add('status', l10n('cat_security'), $opt_link.'status');
+$tabsheet->add('visible', l10n('lock'), $opt_link.'visible');
+$tabsheet->add('upload', l10n('upload'), $opt_link.'upload');
+$tabsheet->add('comments', l10n('comments'), $opt_link.'comments');
 if ($conf['allow_random_representative'])
 {
-  $tabsheet['representative'] =
-    array
-    (
-      'caption' => l10n('Representative'),
-      'url' => $opt_link.'representative'
-    );
-}
-$tabsheet[$page['section']]['selected'] = true;
-
+  $tabsheet->add('representative', l10n('Representative'), $opt_link.'representative');
+}
+// TabSheet selection
+$tabsheet->select($page['section']);
 // Assign tabsheet to template
-$template->assign(
-    array(
-      'tabsheet' => $tabsheet,
-      'TABSHEET_TITLE' => $tabsheet[$page['section']]['caption']
-    )
-  );
+$tabsheet->assign();
 
 // +-----------------------------------------------------------------------+

trunk/admin/configuration.php

@@ -3 +3 @@
 // | PhpWebGallery - a PHP based picture gallery                           |
 // | Copyright (C) 2002-2003 Pierrick LE GALL - pierrick@phpwebgallery.net |
-// | Copyright (C) 2003-2007 PhpWebGallery Team - http://phpwebgallery.net |
+// | Copyright (C) 2003-2008 PhpWebGallery Team - http://phpwebgallery.net |
 // +-----------------------------------------------------------------------+
 // | branch        : BSF (Best So Far)
@@ -32 +32 @@
 
 include_once(PHPWG_ROOT_PATH.'admin/include/functions.php');
-include_once(PHPWG_ROOT_PATH.'admin/include/functions_tabsheet.inc.php');
+include_once(PHPWG_ROOT_PATH.'admin/include/tabsheet.class.php');
 
 // +-----------------------------------------------------------------------+
@@ -158 +158 @@
 $template->set_filename('config', 'admin/configuration.tpl');
 
+// TabSheet
+$tabsheet = new tabsheet();
 // TabSheet initialization
-$page['tabsheet'] = array
-(
-  'main' => array
-   (
-    'caption' => l10n('conf_main_title'),
-    'url' => $conf_link.'main'
-   ),
-  'history' => array
-   (
-    'caption' => l10n('conf_history_title'),
-    'url' => $conf_link.'history'
-   ),
-  'comments' => array
-   (
-    'caption' => l10n('conf_comments_title'),
-    'url' => $conf_link.'comments'
-   ),
-  'default' => array
-   (
-    'caption' => l10n('conf_display'),
-    'url' => $conf_link.'default'
-   )
-);
-
-$page['tabsheet'][$page['section']]['selected'] = true;
-
+$tabsheet->add('main', l10n('conf_main_title'), $conf_link.'main');
+$tabsheet->add('history', l10n('conf_history_title'), $conf_link.'history');
+$tabsheet->add('comments', l10n('conf_comments_title'), $conf_link.'comments');
+$tabsheet->add('default', l10n('conf_display'), $conf_link.'default');
+// TabSheet selection
+$tabsheet->select($page['section']);
 // Assign tabsheet to template
-template_assign_tabsheet();
+$tabsheet->assign();
 
 $action = PHPWG_ROOT_PATH.'admin.php?page=configuration';

trunk/admin/include/functions_history.inc.php

@@ -3 +3 @@
 // | PhpWebGallery - a PHP based picture gallery                           |
 // | Copyright (C) 2002-2003 Pierrick LE GALL - pierrick@phpwebgallery.net |
-// | Copyright (C) 2003-2007 PhpWebGallery Team - http://phpwebgallery.net |
+// | Copyright (C) 2003-2008 PhpWebGallery Team - http://phpwebgallery.net |
 // +-----------------------------------------------------------------------+
 // | file          : $Id$
@@ -25 +25 @@
 // +-----------------------------------------------------------------------+
 
-include_once(PHPWG_ROOT_PATH.'admin/include/functions_tabsheet.inc.php');
+include_once(PHPWG_ROOT_PATH.'admin/include/tabsheet.class.php');
 
 function history_tabsheet()
@@ -31 +31 @@
   global $page, $link_start;
 
+  // TabSheet
+  $tabsheet = new tabsheet();
   // TabSheet initialization
-  $page['tabsheet'] = array
-  (
-    'stats' => array
-     (
-      'caption' => l10n('Statistics'),
-      'url' => $link_start.'stats'
-     ),
-    'history' => array
-     (
-      'caption' => l10n('Search'),
-      'url' => $link_start.'history'
-     )
-  );
-
-  $page['tabsheet'][$page['page']]['selected'] = true;
-
+  $tabsheet->add('stats', l10n('Statistics'), $link_start.'stats');
+  $tabsheet->add('history', l10n('Search'), $link_start.'history');
+  // TabSheet selection
+  $tabsheet->select($page['page']);
   // Assign tabsheet to template
-  template_assign_tabsheet();
+  $tabsheet->assign();
 }
 

trunk/admin/include/functions_waiting.inc.php

@@ -3 +3 @@
 // | PhpWebGallery - a PHP based picture gallery                           |
 // | Copyright (C) 2002-2003 Pierrick LE GALL - pierrick@phpwebgallery.net |
-// | Copyright (C) 2003-2007 PhpWebGallery Team - http://phpwebgallery.net |
+// | Copyright (C) 2003-2008 PhpWebGallery Team - http://phpwebgallery.net |
 // +-----------------------------------------------------------------------+
 // | file          : $Id$
@@ -25 +25 @@
 // +-----------------------------------------------------------------------+
 
-include_once(PHPWG_ROOT_PATH.'admin/include/functions_tabsheet.inc.php');
+include_once(PHPWG_ROOT_PATH.'admin/include/tabsheet.class.php');
 
 function waiting_tabsheet()
@@ -31 +31 @@
   global $page, $link_start;
 
+  // TabSheet
+  $tabsheet = new tabsheet();
   // TabSheet initialization
-  $page['tabsheet'] = array
-  (
-    'comments' => array
-     (
-      'caption' => l10n('comments'),
-      'url' => $link_start.'comments'
-     ),
-    'upload' => array
-     (
-      'caption' => l10n('upload_pictures'),
-      'url' => $link_start.'upload'
-     )
-  );
-
-  $page['tabsheet'][$page['page']]['selected'] = true;
-
+  $tabsheet->add('comments', l10n('comments'), $link_start.'comments');
+  $tabsheet->add('upload', l10n('upload_pictures'), $link_start.'upload');
+  // TabSheet selection
+  $tabsheet->select($page['page']);
   // Assign tabsheet to template
-  template_assign_tabsheet();
+  $tabsheet->assign();
 }
 

trunk/admin/include/tabsheet.class.php

@@ -3 +3 @@
 // | PhpWebGallery - a PHP based picture gallery                           |
 // | Copyright (C) 2002-2003 Pierrick LE GALL - pierrick@phpwebgallery.net |
-// | Copyright (C) 2003-2007 PhpWebGallery Team - http://phpwebgallery.net |
+// | Copyright (C) 2003-2008 PhpWebGallery Team - http://phpwebgallery.net |
 // +-----------------------------------------------------------------------+
 // | file          : $Id$
@@ -25 +25 @@
 // +-----------------------------------------------------------------------+
 
-/*
- * Build TabSheet and assign this content to current page
- *
- * Uses $page['tabsheet'], it's an array of array
- *
- * $page['tabsheet'] description:
- *  $page['tabsheet']'[url'] : Tab link
- *  $page['tabsheet']['Caption'] : Tab caption
- *  $page['tabsheet']['selected'] : Is the selected tab (default value false)
- *
- * Fill {TABSHEET} with HTML code for tabshette
- * Fill {U_TABSHEET_TITLE} with formated caption of the selected tab
- */
+class tabsheet
+{
+  var $sheets;
+  var $name;
+  var $titlename;
+  var $selected;
 
-function template_assign_tabsheet()
-{
-  global $page, $template;
-//THIS SHOULD BE DEPRECATED ?
-  
-  $template->set_filename('tabsheet', 'admin/tabsheet.tpl');
-  $template->assign('tabsheet', $page['tabsheet']);
+  /*
+    $name is the tabsheet's name inside the template .tpl file
+    $titlename in the template is affected by $titlename value
+  */
+  function tabsheet($name = 'TABSHEET', $titlename = 'TABSHEET_TITLE')
+  {
+    $this->sheets = array();
+    $this->name = $name;
+    $this->titlename = $titlename;
+    $this->selected = "";
+  }
 
-  foreach ($page['tabsheet'] as $tab_name => $tab)
+  /*
+     add a tab
+  */
+  function add($name, $caption, $url, $selected = false)
   {
-    $is_selected = isset($tab['selected']) and $tab['selected'] === true;
-    if ($is_selected)
+    if (!isset($this->sheets[$name]))
     {
-      $template->assign_vars(
-        array('TABSHEET_TITLE' => '['.$tab['caption'].']'));
-      break;
+      $this->sheets[$name] = array('caption' => $caption,
+                                   'url' => $url);
+      if($selected)
+      {
+        $this->selected=$name;
+      }
+      return true;
+    }
+    return false;
+  }
+
+  /*
+     remove a tab
+  */
+  function delete($name)
+  {
+    if (isset($this->sheets[$name]))
+    {
+      array_splice($this->sheets, $name, 1);
+
+      if ($this->selected == $name)
+      {
+        $this->selected = "";
+      }
+      return true;
+    }
+    return false;
+  }
+
+  /*
+     select a tab to be active
+  */
+  function select($name)
+  {
+    $this->selected = $name;
+  }
+
+  /*
+    set $titlename value
+  */
+  function set_titlename($titlename)
+  {
+    $this->titlename = $titlename;
+    return $this->titlename;
+  }
+
+  /*
+    returns $titlename value
+  */
+  function get_titlename()
+  {
+    return $this->titlename;
+  }
+
+  /*
+    returns properties of selected tab
+  */
+  function get_selected()
+  {
+    if (!empty($this->selected))
+    {
+      return $this->sheets[$this->selected];
+    }
+    else
+    {
+      return null;
     }
   }
 
-  $template->assign_var_from_handle('TABSHEET', 'tabsheet');
-  $template->clear_assign('tabsheet');
+  /*
+   * Build TabSheet and assign this content to current page
+   *
+   * Fill $this->$name {default value = TABSHEET} with HTML code for tabsheet
+   * Fill $this->titlename {default value = TABSHEET_TITLE} with formated caption of the selected tab
+   */
+  function assign()
+  {
+    global $template;
+
+    $template->set_filename('tabsheet', 'admin/tabsheet.tpl');
+    $template->assign('tabsheet', $this->sheets);
+    $template->assign('tabsheet_selected', $this->selected);
+
+    $selected_tab = $this->get_selected();
+    
+    if (isset($selected_tab))
+    {
+      $template->assign_vars(
+        array($this->titlename => '['.$selected_tab['caption'].']'));
+    }
+
+    $template->assign_var_from_handle($this->name, 'tabsheet');
+    $template->clear_assign('tabsheet');
+  }
 }
 

trunk/admin/notification_by_mail.php

@@ -3 +3 @@
 // | PhpWebGallery - a PHP based picture gallery                           |
 // | Copyright (C) 2002-2003 Pierrick LE GALL - pierrick@phpwebgallery.net |
-// | Copyright (C) 2003-2007 PhpWebGallery Team - http://phpwebgallery.net |
+// | Copyright (C) 2003-2008 PhpWebGallery Team - http://phpwebgallery.net |
 // +-----------------------------------------------------------------------+
 // | file          : $Id$
@@ -36 +36 @@
 include_once(PHPWG_ROOT_PATH.'admin/include/functions.php');
 include_once(PHPWG_ROOT_PATH.'admin/include/functions_notification_by_mail.inc.php');
-include_once(PHPWG_ROOT_PATH.'admin/include/functions_tabsheet.inc.php');
+include_once(PHPWG_ROOT_PATH.'admin/include/tabsheet.class.php');
 include_once(PHPWG_ROOT_PATH.'include/common.inc.php');
 include_once(PHPWG_ROOT_PATH.'include/functions_notification.inc.php');
@@ -611 +611 @@
 if (is_autorize_status(ACCESS_WEBMASTER))
 {
+  // TabSheet
+  $tabsheet = new tabsheet();
   // TabSheet initialization
-  $page['tabsheet'] = array
-  (
-    'param' => array
-     (
-      'caption' => l10n('nbm_param_mode'),
-      'url' => add_url_params($base_url.get_query_string_diff(array('mode', 'select')),
-        array('mode' => 'param'))
-     ),
-    'subscribe' => array
-     (
-      'caption' => l10n('nbm_subscribe_mode'),
-      'url' => add_url_params($base_url.get_query_string_diff(array('mode', 'select')), array('mode' => 'subscribe')),
-     ),
-    'send' => array
-     (
-      'caption' => l10n('nbm_send_mode'),
-      'url' => add_url_params($base_url.get_query_string_diff(array('mode', 'select')), array('mode' => 'send'))
-     )
-  );
-
-  $page['tabsheet'][$page['mode']]['selected'] = true;
-
+  $tabsheet->add('param', l10n('nbm_param_mode'),
+    add_url_params($base_url.get_query_string_diff(array('mode', 'select')),
+        array('mode' => 'param')));
+  $tabsheet->add('subscribe', l10n('nbm_subscribe_mode'),
+    add_url_params($base_url.get_query_string_diff(array('mode', 'select')),
+      array('mode' => 'subscribe')));
+  $tabsheet->add('send', l10n('nbm_send_mode'),
+    add_url_params($base_url.get_query_string_diff(array('mode', 'select')),
+      array('mode' => 'send')));
+  // TabSheet selection
+  $tabsheet->select($page['mode']);
   // Assign tabsheet to template
-  template_assign_tabsheet();
+  $tabsheet->assign();
 }
 

trunk/template/yoga/admin/cat_options.tpl

@@ -5 +5 @@
   </ul>
   <h2>{'cat_options_title'|@translate} {$TABSHEET_TITLE}</h2>
-  {include file='admin/tabsheet.tpl'}
+  {$TABSHEET}
 </div>
 

trunk/template/yoga/admin/tabsheet.tpl

@@ -2 +2 @@
 {if isset($tabsheet) and count($tabsheet)}
 <ul class="tabsheet">
-{foreach from=$tabsheet item=tab}
-  <li class="{if isset($tab.selected) and $tab.selected}selected_tab{else}normal_tab{/if}"><a href="{$tab.url}">{$tab.caption}</a></li>
+{foreach from=$tabsheet key=name item=sheet}
+  <li class="{if ($name == $tabsheet_selected)}selected_tab{else}normal_tab{/if}"><a href="{$sheet.url}">{$sheet.caption}</a></li>
 {/foreach}
 </ul>