r3696 process.php: get rid of RuntimeException
authorDenis Ovsienko <infrastation@yandex.ru>
Sun, 30 May 2010 15:14:17 +0000 (15:14 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Sun, 30 May 2010 15:14:17 +0000 (15:14 +0000)
inc/config.php
process.php

index 1044f45e782c7b309b16fbb5c3ab2774b7e3011e..d60fdf90546acb95df3fe4ee5c5cd86255187f7f 100644 (file)
@@ -51,6 +51,7 @@ define ('E_8021Q_PULL_REMOTE_ERROR', 102);
 define ('E_8021Q_PUSH_REMOTE_ERROR', 103);
 define ('E_8021Q_SYNC_DISABLED', 104);
 define ('E_BAD_RACKCODE', 1);
+define ('E_INTERNAL', 2);
 define ('VLAN_MIN_ID', 1);
 define ('VLAN_MAX_ID', 4094);
 define ('VLAN_DFL_ID', 1);
index d139361e3695eb0b335cbacf77955dccf4e9272c..5a75b0b5923559e1409ec575b85fa78ac5281726 100644 (file)
@@ -11,23 +11,12 @@ assertStringArg ('op');
 $op = $_REQUEST['op'];
 prepareNavigation();
 // FIXME: find a better way to handle this error
-if ($op == 'addFile' && !isset($_FILES['file']['error'])) {
-       throw new RuntimeException("File upload error, it's size probably exceeds upload_max_filesize directive in php.ini");
-}
+if ($op == 'addFile' && !isset($_FILES['file']['error']))
+       throw new Exception ('File upload error, check upload_max_filesize in php.ini', E_INTERNAL);
 fixContext();
 
-
-if (!isset ($ophandler[$pageno][$tabno][$op]))
-{
-       throw new RuntimeException("Invalid request in operation broker: page '${pageno}', tab '${tabno}', op '${op}'");
-}
-
-if (!function_exists ($ophandler[$pageno][$tabno][$op]))
-       throw new RuntimeException
-       (
-               "Dispatching error at position '${pageno}-${tabno}-${op}': function '" .
-               $ophandler[$pageno][$tabno][$op] . "' does not exist."
-       );
+if (!isset ($ophandler[$pageno][$tabno][$op]) or !function_exists ($ophandler[$pageno][$tabno][$op]))
+       throw new Exception ("Invalid navigation data for '${pageno}-${tabno}-${op}'", E_INTERNAL);
 
 // We have a chance to handle an error before starting HTTP header.
 if (!isset ($delayauth[$pageno][$tabno][$op]) and !permitted())
@@ -36,9 +25,7 @@ else
 {
        $location = call_user_func ($ophandler[$pageno][$tabno][$op]);
        if (!strlen ($location))
-       {
-               throw new RuntimeException('Operation handler failed to return its status');
-       }
+               throw new Exception ('Operation handler failed to return its status', E_INTERNAL);
 }
 header ("Location: " . $location);
 ob_end_flush();