r1803 + move assertion functions to where they are mostly used (ophandlers.php)
authorDenis Ovsienko <infrastation@yandex.ru>
Thu, 20 Mar 2008 08:36:16 +0000 (08:36 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Thu, 20 Mar 2008 08:36:16 +0000 (08:36 +0000)
+ replace handler_portmap with 2 standard tab handlers in interface.php
+ drop pagehandler.php

inc/init.php
inc/interface.php
inc/navigation.php
inc/ophandlers.php
inc/pagehandlers.php [deleted file]

index 83f2a7d7069dd4a69b73f5b0705bbfea056aa3ef..6532b20c64c45b5a630ab6a6ad37767454e6b041 100644 (file)
@@ -114,7 +114,6 @@ $tabno = (isset ($_REQUEST['tab'])) ? $_REQUEST['tab'] : 'default';
 
 require_once 'inc/navigation.php';
 require_once 'inc/pagetitles.php';
 
 require_once 'inc/navigation.php';
 require_once 'inc/pagetitles.php';
-require_once 'inc/pagehandlers.php';
 require_once 'inc/ophandlers.php';
 require_once 'inc/triggers.php';
 require_once 'inc/gateways.php';
 require_once 'inc/ophandlers.php';
 require_once 'inc/triggers.php';
 require_once 'inc/gateways.php';
index 87618d9afc868fd061b1a716f15c267c1481c827..80932cc5a966401cf018f38676d8d955e53c2ae0 100644 (file)
@@ -2703,6 +2703,16 @@ function renderPermissionsEditForm ()
        finishPortlet();
 }
 
        finishPortlet();
 }
 
+function renderPortMapViewer ()
+{
+       renderPortMap (FALSE);
+}
+
+function renderPortMapEditor ()
+{
+       renderPortMap (TRUE);
+}
+
 function renderPortMap ($editable = FALSE)
 {
        global $nextorder, $root, $pageno, $tabno;
 function renderPortMap ($editable = FALSE)
 {
        global $nextorder, $root, $pageno, $tabno;
index 67a5ae9863d51f9359751fa23a0f0cfb85376bf3..8a5089aae859873ab64b68be9108588363b10863 100644 (file)
@@ -250,10 +250,11 @@ $ophandler['perms']['edit']['revoke'] = 'revokePermission';
 $ophandler['perms']['edit']['grant'] = 'grantPermission';
 
 $page['portmap']['title'] = 'Port compatibility map';
 $ophandler['perms']['edit']['grant'] = 'grantPermission';
 
 $page['portmap']['title'] = 'Port compatibility map';
-$page['portmap']['handler'] = 'handler_portmap';
 $page['portmap']['parent'] = 'config';
 $tab['portmap']['default'] = 'View';
 $tab['portmap']['edit'] = 'Change';
 $page['portmap']['parent'] = 'config';
 $tab['portmap']['default'] = 'View';
 $tab['portmap']['edit'] = 'Change';
+$tabhandler['portmap']['default'] = 'renderPortMapViewer';
+$tabhandler['portmap']['edit'] = 'renderPortMapEditor';
 $ophandler['portmap']['edit']['save'] = 'savePortMap';
 
 $page['attrs']['title'] = 'Attributes';
 $ophandler['portmap']['edit']['save'] = 'savePortMap';
 
 $page['attrs']['title'] = 'Attributes';
index 4682e87045c28a0f07f140394e45b3a25e33f3f6..3dab40a3576f97ac87ac8916431820b8ade78895 100644 (file)
@@ -5,6 +5,82 @@
 *
 */
 
 *
 */
 
+// This function assures that specified argument was passed
+// and is a number greater than zero.
+function assertUIntArg ($argname, $allow_zero = FALSE)
+{
+       if (!isset ($_REQUEST[$argname]))
+       {
+               showError ("Parameter '${argname}' is missing.", __FUNCTION__);
+               die();
+       }
+       if (!is_numeric ($_REQUEST[$argname]))
+       {
+               showError ("Parameter '${argname}' is not a number.", __FUNCTION__);
+               die();
+       }
+       if ($_REQUEST[$argname] < 0)
+       {
+               showError ("Parameter '${argname}' is less than zero.", __FUNCTION__);
+               die();
+       }
+       if (!$allow_zero and $_REQUEST[$argname] == 0)
+       {
+               showError ("Parameter '${argname}' is equal to zero.", __FUNCTION__);
+               die();
+       }
+}
+
+// This function assures that specified argument was passed
+// and is a non-empty string.
+function assertStringArg ($argname, $ok_if_empty = FALSE)
+{
+       if (!isset ($_REQUEST[$argname]))
+       {
+               showError ("Parameter '${argname}' is missing.", __FUNCTION__);
+               die();
+       }
+       if (!is_string ($_REQUEST[$argname]))
+       {
+               showError ("Parameter '${argname}' is not a string.", __FUNCTION__);
+               die();
+       }
+       if (!$ok_if_empty and empty ($_REQUEST[$argname]))
+       {
+               showError ("Parameter '${argname}' is an empty string.", __FUNCTION__);
+               die();
+       }
+}
+
+function assertBoolArg ($argname, $ok_if_empty = FALSE)
+{
+       if (!isset ($_REQUEST[$argname]))
+       {
+               showError ("Parameter '${argname}' is missing.", __FUNCTION__);
+               die();
+       }
+       if (!is_string ($_REQUEST[$argname]) or $_REQUEST[$argname] != 'on')
+       {
+               showError ("Parameter '${argname}' is not a string.", __FUNCTION__);
+               die();
+       }
+       if (!$ok_if_empty and empty ($_REQUEST[$argname]))
+       {
+               showError ("Parameter '${argname}' is an empty string.", __FUNCTION__);
+               die();
+       }
+}
+
+function assertIPv4Arg ($argname, $ok_if_empty = FALSE)
+{
+       assertStringArg ($argname, $ok_if_empty);
+       if (!empty ($_REQUEST[$argname]) and long2ip (ip2long ($_REQUEST[$argname])) !== $_REQUEST[$argname])
+       {
+               showError ("IPv4 address validation failed for value '" . $_REQUEST[$argname] . "'", __FUNCTION__);
+               die();
+       }
+}
+
 function addPortForwarding ()
 {
        global $root, $pageno, $tabno;
 function addPortForwarding ()
 {
        global $root, $pageno, $tabno;
diff --git a/inc/pagehandlers.php b/inc/pagehandlers.php
deleted file mode 100644 (file)
index cf128ac..0000000
+++ /dev/null
@@ -1,99 +0,0 @@
-<?php
-/*
-*
-*  This file is a library of page handlers for RackTables.
-*
-*/
-
-// This function assures that specified argument was passed
-// and is a number greater than zero.
-function assertUIntArg ($argname, $allow_zero = FALSE)
-{
-       if (!isset ($_REQUEST[$argname]))
-       {
-               showError ("Parameter '${argname}' is missing.", __FUNCTION__);
-               die();
-       }
-       if (!is_numeric ($_REQUEST[$argname]))
-       {
-               showError ("Parameter '${argname}' is not a number.", __FUNCTION__);
-               die();
-       }
-       if ($_REQUEST[$argname] < 0)
-       {
-               showError ("Parameter '${argname}' is less than zero.", __FUNCTION__);
-               die();
-       }
-       if (!$allow_zero and $_REQUEST[$argname] == 0)
-       {
-               showError ("Parameter '${argname}' is equal to zero.", __FUNCTION__);
-               die();
-       }
-}
-
-// This function assures that specified argument was passed
-// and is a non-empty string.
-function assertStringArg ($argname, $ok_if_empty = FALSE)
-{
-       if (!isset ($_REQUEST[$argname]))
-       {
-               showError ("Parameter '${argname}' is missing.", __FUNCTION__);
-               die();
-       }
-       if (!is_string ($_REQUEST[$argname]))
-       {
-               showError ("Parameter '${argname}' is not a string.", __FUNCTION__);
-               die();
-       }
-       if (!$ok_if_empty and empty ($_REQUEST[$argname]))
-       {
-               showError ("Parameter '${argname}' is an empty string.", __FUNCTION__);
-               die();
-       }
-}
-
-function assertBoolArg ($argname, $ok_if_empty = FALSE)
-{
-       if (!isset ($_REQUEST[$argname]))
-       {
-               showError ("Parameter '${argname}' is missing.", __FUNCTION__);
-               die();
-       }
-       if (!is_string ($_REQUEST[$argname]) or $_REQUEST[$argname] != 'on')
-       {
-               showError ("Parameter '${argname}' is not a string.", __FUNCTION__);
-               die();
-       }
-       if (!$ok_if_empty and empty ($_REQUEST[$argname]))
-       {
-               showError ("Parameter '${argname}' is an empty string.", __FUNCTION__);
-               die();
-       }
-}
-
-function assertIPv4Arg ($argname, $ok_if_empty = FALSE)
-{
-       assertStringArg ($argname, $ok_if_empty);
-       if (!empty ($_REQUEST[$argname]) and long2ip (ip2long ($_REQUEST[$argname])) !== $_REQUEST[$argname])
-       {
-               showError ("IPv4 address validation failed for value '" . $_REQUEST[$argname] . "'", __FUNCTION__);
-               die();
-       }
-}
-
-function handler_portmap ($tabno)
-{
-       switch ($tabno)
-       {
-               case 'default':
-                       renderPortMap (FALSE);
-                       break;
-               case 'edit':
-                       renderPortMap (TRUE);
-                       break;
-               default:
-                       showError ("Invalid tab '${tabno}' requested.", __FUNCTION__);
-       }
-}
-
-?>