r4126 renderVSTListEditor(): update to always have the hidden parameter
authorDenis Ovsienko <infrastation@yandex.ru>
Thu, 13 Jan 2011 21:13:17 +0000 (21:13 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Thu, 13 Jan 2011 21:13:17 +0000 (21:13 +0000)
commitLinkFile(): dismiss in favour of usePreparedInsertBlade()
addFileToEntity(): update respectively
linkFileToEntity(): idem
commitUpdateLB(): dismiss
updateLoadBalancer(): dismiss
commitUpdateRSPool(): dismiss
updateRSPool(): dismiss
commitUpdateFile(): dismiss
updateFile(): dismiss
commitUpdateVLANDomain(): dismiss
updateVLANDomain(): dismiss
commitUpdateVST(): dismiss
updVLANSwitchTemplate(): dismiss
upd8021QPort(): employ usePreparedUpdateBlade()

inc/database.php
inc/interface.php
inc/navigation.php
inc/ophandlers.php

index 8dbd7cc..ae1809a 100644 (file)
@@ -2677,23 +2677,6 @@ function commitUpdateRS ($rsid = 0, $rsip = '', $rsport = 0, $rsconfig = '')
        );
 }
 
-function commitUpdateLB ($object_id = 0, $pool_id = 0, $vs_id = 0, $vsconfig = '', $rsconfig = '', $prio = '')
-{
-       return usePreparedExecuteBlade
-       (
-               'UPDATE IPv4LB SET vsconfig=?, rsconfig=?, prio=? WHERE object_id=? AND rspool_id=? AND vs_id=?',
-               array
-               (
-                       !strlen ($vsconfig) ? NULL : $vsconfig,
-                       !strlen ($rsconfig) ? NULL : $rsconfig,
-                       !strlen ($prio) ? NULL : $prio,
-                       $object_id,
-                       $pool_id,
-                       $vs_id,
-               )
-       );
-}
-
 function commitUpdateVS ($vsid = 0, $vip = '', $vport = 0, $proto = '', $name = '', $vsconfig = '', $rsconfig = '')
 {
        if (!strlen ($vip))
@@ -2788,21 +2771,6 @@ function commitDeleteRSPool ($pool_id = 0)
        return usePreparedDeleteBlade ('IPv4RSPool', array ('id' => $pool_id)) && destroyTagsForEntity ('ipv4rspool', $pool_id);
 }
 
-function commitUpdateRSPool ($pool_id = 0, $name = '', $vsconfig = '', $rsconfig = '')
-{
-       return usePreparedExecuteBlade
-       (
-               'UPDATE IPv4RSPool SET name=?, vsconfig=?, rsconfig=? WHERE id=?',
-               array
-               (
-                       !strlen ($name) ? NULL : $name,
-                       !strlen ($vsconfig) ? NULL : $vsconfig,
-                       !strlen ($rsconfig) ? NULL : $rsconfig,
-                       $pool_id,
-               )
-       );
-}
-
 function getRSList ()
 {
        $result = usePreparedSelectBlade
@@ -3511,15 +3479,6 @@ function commitAddFile ($name, $type, $size, $contents, $comment)
        }
 }
 
-function commitLinkFile ($file_id, $entity_type, $entity_id)
-{
-       return usePreparedExecuteBlade
-       (
-               'INSERT INTO FileLink (file_id, entity_type, entity_id) VALUES (?, ?, ?)',
-               array ($file_id, $entity_type, $entity_id)
-       );
-}
-
 function commitReplaceFile ($file_id = 0, $contents)
 {
        global $dbxlink;
@@ -3537,19 +3496,6 @@ function commitReplaceFile ($file_id = 0, $contents)
        }
 }
 
-function commitUpdateFile ($file_id = 0, $new_name = '', $new_type = '', $new_comment = '')
-{
-       if (!strlen ($new_name))
-               throw new InvalidArgException ('$new_name', $new_name);
-       if (!strlen ($new_type))
-               throw new InvalidArgException ('$new_type', $new_type);
-       return usePreparedExecuteBlade
-       (
-               'UPDATE File SET name = ?, type = ?, comment = ? WHERE id = ?',
-               array ($new_name, $new_type, $new_comment, $file_id)
-       );
-}
-
 function commitUnlinkFile ($link_id)
 {
        return usePreparedDeleteBlade ('FileLink', array ('id' => $link_id));
@@ -3849,15 +3795,6 @@ function commitUpdateVLANDescription ($vdom_id, $vlan_id, $vlan_type, $vlan_desc
        );
 }
 
-function commitUpdateVLANDomain ($vdom_id, $vdom_descr)
-{
-       return usePreparedExecuteBlade
-       (
-               'UPDATE VLANDomain SET description=? WHERE id=?',
-               array ($vdom_descr, $vdom_id)
-       );
-}
-
 function getVLANSwitches()
 {
        $ret = array();
@@ -4116,10 +4053,11 @@ function upd8021QPort ($instance = 'desired', $object_id, $port_name, $port)
        // A record on a port with none VLANs allowed makes no sense regardless of port mode.
        if ($port['mode'] != 'trunk' and !count ($port['allowed']))
                return 0;
-       usePreparedExecuteBlade
+       usePreparedUpdateBlade
        (
-               'UPDATE ' . $tablemap_8021q[$instance]['pvm'] . ' SET vlan_mode=? WHERE object_id=? AND port_name=?',
-               array ($port['mode'], $object_id, $port_name)
+               $tablemap_8021q[$instance]['pvm'],
+               array ('vlan_mode' => $port['mode']),
+               array ('object_id' => $object_id, 'port_name' => $port_name)
        );
        if (FALSE === usePreparedDeleteBlade ($tablemap_8021q[$instance]['pav'], array ('object_id' => $object_id, 'port_name' => $port_name)))
                throw new RackTablesError ('', RackTablesError::DB_WRITE_FAILED);
@@ -4203,15 +4141,6 @@ function getVLANSwitchTemplate ($vst_id)
        return $ret;
 }
 
-function commitUpdateVST ($vst_id, $max_local_vlans, $description)
-{
-       return usePreparedExecuteBlade
-       (
-               'UPDATE VLANSwitchTemplate SET max_local_vlans=?, description=? WHERE id=?',
-               array ($max_local_vlans, $description, $vst_id)
-       );
-}
-
 function commitUpdateVSTRules ($vst_id, $mutex_rev, $rules)
 {
        global $dbxlink, $remote_username;
index f48eb62..c810334 100644 (file)
@@ -8649,7 +8649,7 @@ function renderVSTListEditor()
 {
        function printNewItemTR ()
        {
-               printOpFormIntro ('add');
+               printOpFormIntro ('add', array ('vst_maxvlans' => 0));
                echo '<tr>';
                echo '<td>' . getImageHREF ('create', 'create template', TRUE, 104) . '</td>';
                echo '<td><input type=text name=vst_descr tabindex=101></td>';
@@ -8662,7 +8662,7 @@ function renderVSTListEditor()
                printNewItemTR();
        foreach (getVSTStats() as $vst_id => $vst_info)
        {
-               printOpFormIntro ('upd', array ('vst_id' => $vst_id));
+               printOpFormIntro ('upd', array ('vst_id' => $vst_id, 'vst_maxvlans' => 0));
                echo '<tr><td>';
                if ($vst_info['switchc'])
                        printImageHREF ('nodestroy', 'template used elsewhere');
index 621c9ff..22ba665 100644 (file)
@@ -179,7 +179,7 @@ $ophandler['object']['files']['linkFile'] = 'linkFileToEntity';
 $ophandler['object']['files']['unlinkFile'] = 'unlinkFile';
 $ophandler['object']['editrspvs']['addLB'] = 'addLoadBalancer';
 $ophandler['object']['editrspvs']['delLB'] = 'tableHandler';
-$ophandler['object']['editrspvs']['updLB'] = 'updateLoadBalancer';
+$ophandler['object']['editrspvs']['updLB'] = 'tableHandler';
 $ophandler['object']['lvsconfig']['submitSLBConfig'] = 'submitSLBConfig';
 $ophandler['object']['snmpportfinder']['querySNMPData'] = 'querySNMPData';
 $ophandler['object']['8021qorder']['add'] = 'add8021QOrder';
@@ -319,7 +319,7 @@ $trigger['ipv4vs']['tags'] = 'trigger_tags';
 $ophandler['ipv4vs']['edit']['updIPv4VS'] = 'updateVService';
 $ophandler['ipv4vs']['editlblist']['addLB'] = 'addLoadBalancer';
 $ophandler['ipv4vs']['editlblist']['delLB'] = 'tableHandler';
-$ophandler['ipv4vs']['editlblist']['updLB'] = 'updateLoadBalancer';
+$ophandler['ipv4vs']['editlblist']['updLB'] = 'tableHandler';
 $ophandler['ipv4vs']['tags']['saveTags'] = 'saveEntityTags';
 $ophandler['ipv4vs']['files']['addFile'] = 'addFileToEntity';
 $ophandler['ipv4vs']['files']['linkFile'] = 'linkFileToEntity';
@@ -353,14 +353,14 @@ $tabhandler['ipv4rspool']['editlblist'] = 'renderRSPoolLBForm';
 $tabhandler['ipv4rspool']['rsinservice'] = 'renderRSPoolRSInServiceForm';
 $tabhandler['ipv4rspool']['tags'] = 'renderEntityTags';
 $tabhandler['ipv4rspool']['files'] = 'renderFilesForEntity';
-$ophandler['ipv4rspool']['edit']['updIPv4RSP'] = 'updateRSPool';
+$ophandler['ipv4rspool']['edit']['updIPv4RSP'] = 'tableHandler';
 $ophandler['ipv4rspool']['editrslist']['addRS'] = 'addRealServer';
 $ophandler['ipv4rspool']['editrslist']['delRS'] = 'tableHandler';
 $ophandler['ipv4rspool']['editrslist']['updRS'] = 'updateRealServer';
 $ophandler['ipv4rspool']['editrslist']['addMany'] = 'addRealServers';
 $ophandler['ipv4rspool']['editlblist']['addLB'] = 'addLoadBalancer';
 $ophandler['ipv4rspool']['editlblist']['delLB'] = 'tableHandler';
-$ophandler['ipv4rspool']['editlblist']['updLB'] = 'updateLoadBalancer';
+$ophandler['ipv4rspool']['editlblist']['updLB'] = 'tableHandler';
 $ophandler['ipv4rspool']['rsinservice']['upd'] = 'updateRSInService';
 $ophandler['ipv4rspool']['tags']['saveTags'] = 'saveEntityTags';
 $ophandler['ipv4rspool']['files']['addFile'] = 'addFileToEntity';
@@ -548,7 +548,7 @@ $tabhandler['file']['tags'] = 'renderEntityTags';
 $tabhandler['file']['editText'] = 'renderTextEditor';
 $tabhandler['file']['replaceData'] = 'renderFileReuploader';
 $tabhandler['file']['download'] = 'renderFileDownloader';
-$ophandler['file']['edit']['updateFile'] = 'updateFile';
+$ophandler['file']['edit']['updateFile'] = 'tableHandler';
 $ophandler['file']['tags']['saveTags'] = 'saveEntityTags';
 $ophandler['file']['editText']['updateFileText'] = 'updateFileText';
 $ophandler['file']['replaceData']['replaceFile'] = 'replaceFile';
@@ -563,10 +563,10 @@ $tabhandler['8021q']['vdlist'] = 'renderVLANDomainListEditor';
 $tabhandler['8021q']['vstlist'] = 'renderVSTListEditor';
 $ophandler['8021q']['vdlist']['add'] = 'createVLANDomain';
 $ophandler['8021q']['vdlist']['del'] = 'destroyVLANDomain';
-$ophandler['8021q']['vdlist']['upd'] = 'updateVLANDomain';
+$ophandler['8021q']['vdlist']['upd'] = 'tableHandler';
 $ophandler['8021q']['vstlist']['add'] = 'addVLANSwitchTemplate';
 $ophandler['8021q']['vstlist']['del'] = 'delVLANSwitchTemplate';
-$ophandler['8021q']['vstlist']['upd'] = 'updVLANSwitchTemplate';
+$ophandler['8021q']['vstlist']['upd'] = 'tableHandler';
 
 $page['vlandomain']['parent'] = '8021q';
 $page['vlandomain']['bypass'] = 'vdom_id';
index 2f5eb39..0575f2e 100644 (file)
@@ -86,6 +86,25 @@ $opspec_list['object-editrspvs-delLB'] = array
                array ('url_argname' => 'vs_id', 'assertion' => 'uint'),
        ),
 );
+$opspec_list['ipv4vs-editlblist-updLB'] =
+$opspec_list['ipv4rspool-editlblist-updLB'] =
+$opspec_list['object-editrspvs-updLB'] = array
+(
+       'table' => 'IPv4LB',
+       'action' => 'UPDATE',
+       'set_arglist' => array
+       (
+               array ('url_argname' => 'vsconfig', 'assertion' => 'string0', 'if_empty' => 'NULL'),
+               array ('url_argname' => 'rsconfig', 'assertion' => 'string0', 'if_empty' => 'NULL'),
+               array ('url_argname' => 'prio', 'assertion' => 'uint0', 'if_empty' => 'NULL'),
+       ),
+       'where_arglist' => array
+       (
+               array ('url_argname' => 'object_id', 'assertion' => 'uint'),
+               array ('url_argname' => 'pool_id', 'table_colname' => 'rspool_id', 'assertion' => 'uint'),
+               array ('url_argname' => 'vs_id', 'assertion' => 'uint'),
+       ),
+);
 $opspec_list['ipv4net-properties-editRange'] = array
 (
        'table' => 'IPv4Network',
@@ -123,6 +142,36 @@ $opspec_list['ipv4rspool-editrslist-delRS'] = array
                array ('url_argname' => 'id', 'assertion' => 'uint'),
        ),
 );
+$opspec_list['ipv4rspool-edit-updIPv4RSP'] = array
+(
+       'table' => 'IPv4RSPool',
+       'action' => 'UPDATE',
+       'set_arglist' => array
+       (
+               array ('url_argname' => 'name', 'assertion' => 'string0', 'if_empty' => 'NULL'),
+               array ('url_argname' => 'vsconfig', 'assertion' => 'string0', 'if_empty' => 'NULL'),
+               array ('url_argname' => 'rsconfig', 'assertion' => 'string0', 'if_empty' => 'NULL'),
+       ),
+       'where_arglist' => array
+       (
+               array ('url_argname' => 'pool_id', 'table_colname' => 'id', 'assertion' => 'uint')
+       ),
+);
+$opspec_list['file-edit-updateFile'] = array
+(
+       'table' => 'File',
+       'action' => 'UPDATE',
+       'set_arglist' => array
+       (
+               array ('url_argname' => 'file_name', 'table_colname' => 'name', 'assertion' => 'string'),
+               array ('url_argname' => 'file_type', 'table_colname' => 'type', 'assertion' => 'string'),
+               array ('url_argname' => 'file_comment', 'table_colname' => 'comment', 'assertion' => 'string0', 'if_empty' => 'NULL'),
+       ),
+       'where_arglist' => array
+       (
+               array ('url_argname' => 'file_id', 'table_colname' => 'id', 'assertion' => 'uint')
+       ),
+);
 $opspec_list['portmap-edit-add'] = array
 (
        'table' => 'PortCompat',
@@ -247,6 +296,33 @@ $opspec_list['tagtree-edit-updateTag'] = array
                array ('url_argname' => 'tag_id', 'table_colname' => 'id', 'assertion' => 'uint'),
        ),
 );
+$opspec_list['8021q-vstlist-upd'] = array
+(
+       'table' => 'VLANSwitchTemplate',
+       'action' => 'UPDATE',
+       'set_arglist' => array
+       (
+               array ('url_argname' => 'vst_maxvlans', 'table_colname' => 'max_local_vlans', 'assertion' => 'uint0', 'if_empty' => 'NULL'),
+               array ('url_argname' => 'vst_descr', 'table_colname' => 'description', 'assertion' => 'string'),
+       ),
+       'where_arglist' => array
+       (
+               array ('url_argname' => 'vst_id', 'table_colname' => 'id', 'assertion' => 'uint'),
+       ),
+);
+$opspec_list['8021q-vdlist-upd'] = array
+(
+       'table' => 'VLANDomain',
+       'action' => 'UPDATE',
+       'set_arglist' => array
+       (
+               array ('url_argname' => 'vdom_descr', 'table_colname' => 'description', 'assertion' => 'string'),
+       ),
+       'where_arglist' => array
+       (
+               array ('url_argname' => 'vdom_id', 'table_colname' => 'id', 'assertion' => 'uint'),
+       ),
+);
 $opspec_list['vlandomain-vlanlist-add'] = array
 (
        'table' => 'VLANDescription',
@@ -1477,31 +1553,6 @@ function updateRealServer ()
                return buildRedirectURL (__FUNCTION__, 'OK');
 }
 
-$msgcode['updateLoadBalancer']['OK'] = 51;
-$msgcode['updateLoadBalancer']['ERR'] = 109;
-function updateLoadBalancer ()
-{
-       assertUIntArg ('object_id');
-       assertUIntArg ('pool_id');
-       assertUIntArg ('vs_id');
-       assertStringArg ('vsconfig', TRUE);
-       assertStringArg ('rsconfig', TRUE);
-       if (! empty($_REQUEST['prio']))
-               assertUIntArg('prio', TRUE);
-
-       if (FALSE === commitUpdateLB (
-               $_REQUEST['object_id'],
-               $_REQUEST['pool_id'],
-               $_REQUEST['vs_id'],
-               $_REQUEST['vsconfig'],
-               $_REQUEST['rsconfig'],
-               $_REQUEST['prio']
-       ))
-               return buildRedirectURL (__FUNCTION__, 'ERR');
-       else
-               return buildRedirectURL (__FUNCTION__, 'OK');
-}
-
 $msgcode['updateVService']['OK'] = 51;
 $msgcode['updateVService']['ERR'] = 109;
 function updateVService ()
@@ -1583,26 +1634,6 @@ function deleteRSPool ()
                return buildRedirectURL (__FUNCTION__, 'OK');
 }
 
-$msgcode['updateRSPool']['OK'] = 51;
-$msgcode['updateRSPool']['ERR'] = 109;
-function updateRSPool ()
-{
-       assertStringArg ('name', TRUE);
-       assertStringArg ('vsconfig', TRUE);
-       assertStringArg ('rsconfig', TRUE);
-       if (FALSE === commitUpdateRSPool
-               (
-                       $_REQUEST['pool_id'],
-                       $_REQUEST['name'],
-                       $_REQUEST['vsconfig'],
-                       $_REQUEST['rsconfig']
-               )
-       )
-               return buildRedirectURL (__FUNCTION__, 'ERR');
-       else
-               return buildRedirectURL (__FUNCTION__, 'OK');
-}
-
 $msgcode['updateRSInService']['OK'] = 26;
 $msgcode['updateRSInService']['ERR'] = 141;
 function updateRSInService ()
@@ -2021,7 +2052,7 @@ function addFileToEntity ()
        global $sic;
        if (FALSE === commitAddFile ($_FILES['file']['name'], $_FILES['file']['type'], $_FILES['file']['size'], $fp, $sic['comment']))
                return buildRedirectURL (__FUNCTION__, 'ERR3');
-       if (FALSE === commitLinkFile (lastInsertID(), $realm, $entity_id))
+       if (FALSE === usePreparedInsertBlade ('FileLink', array ('file_id' => lastInsertID(), 'entity_type' => $realm, 'entity_id' => $entity_id)))
                return buildRedirectURL (__FUNCTION__, 'ERR3');
 
        return buildRedirectURL (__FUNCTION__, 'OK', array (htmlspecialchars ($_FILES['file']['name'])));
@@ -2032,12 +2063,12 @@ $msgcode['linkFileToEntity']['ERR2'] = 110;
 function linkFileToEntity ()
 {
        assertUIntArg ('file_id');
-       global $pageno, $etype_by_pageno;
+       global $pageno, $etype_by_pageno, $sic;
        if (!isset ($etype_by_pageno[$pageno]))
                throw new RackTablesError ('key not found in etype_by_pageno', RackTablesError::INTERNAL);
 
-       $fi = spotEntity ('file', $_REQUEST['file_id']);
-       if (FALSE === commitLinkFile ($_REQUEST['file_id'], $etype_by_pageno[$pageno], getBypassValue()))
+       $fi = spotEntity ('file', $sic['file_id']);
+       if (FALSE === usePreparedInsertBlade ('FileLink', array ('file_id' => $sic['file_id'], 'entity_type' => $etype_by_pageno[$pageno], 'entity_id' => getBypassValue())))
                return buildRedirectURL (__FUNCTION__, 'ERR2');
 
        return buildRedirectURL (__FUNCTION__, 'OK', array (htmlspecialchars ($fi['name'])));
@@ -2071,19 +2102,6 @@ function replaceFile ()
        return buildRedirectURL (__FUNCTION__, 'OK', array (htmlspecialchars ($shortInfo['name'])));
 }
 
-$msgcode['updateFile']['OK'] = 70;
-$msgcode['updateFile']['ERR'] = 109;
-function updateFile ()
-{
-       assertStringArg ('file_name');
-       assertStringArg ('file_type');
-       assertStringArg ('file_comment', TRUE);
-       global $sic;
-       if (FALSE === commitUpdateFile ($sic['file_id'], $sic['file_name'], $sic['file_type'], $sic['file_comment']))
-               return buildRedirectURL (__FUNCTION__, 'ERR');
-       return buildRedirectURL (__FUNCTION__, 'OK', array ($_REQUEST['file_name']));
-}
-
 $msgcode['unlinkFile']['OK'] = 72;
 $msgcode['unlinkFile']['ERR'] = 111;
 function unlinkFile ()
@@ -2294,17 +2312,6 @@ function destroyVLANDomain ()
        return buildRedirectURL (__FUNCTION__, $result ? 'OK' : 'ERR');
 }
 
-$msgcode['updateVLANDomain']['OK'] = 51;
-$msgcode['updateVLANDomain']['ERR'] = 109;
-function updateVLANDomain ()
-{
-       assertUIntArg ('vdom_id');
-       assertStringArg ('vdom_descr');
-       global $sic;
-       $result = FALSE !== commitUpdateVLANDomain ($sic['vdom_id'], $sic['vdom_descr']);
-       return buildRedirectURL (__FUNCTION__, $result ? 'OK' : 'ERR');
-}
-
 $msgcode['save8021QPorts']['OK1'] = 63;
 $msgcode['save8021QPorts']['OK2'] = 41;
 $msgcode['save8021QPorts']['ERR2'] = 109;
@@ -2612,23 +2619,6 @@ function delVLANSwitchTemplate()
        return buildRedirectURL (__FUNCTION__, $result ? 'OK' : 'ERR');
 }
 
-$msgcode['updVLANSwitchTemplate']['OK'] = 51;
-$msgcode['updVLANSwitchTemplate']['ERR'] = 109;
-function updVLANSwitchTemplate()
-{
-       assertUIntArg ('vst_id');
-       assertStringArg ('vst_descr');
-       global $sic;
-       $max_local_vlans = NULL;
-       if (array_key_exists ('vst_maxvlans', $sic) && mb_strlen ($sic['vst_maxvlans']))
-       {
-               assertUIntArg ('vst_maxvlans');
-               $max_local_vlans = $sic['vst_maxvlans'];
-       }
-       $result = commitUpdateVST ($sic['vst_id'], $max_local_vlans, $sic['vst_descr']);
-       return buildRedirectURL (__FUNCTION__, $result !== FALSE ? 'OK' : 'ERR');
-}
-
 $msgcode['cloneVSTRule']['OK'] = 48;
 $msgcode['cloneVSTRule']['ERR'] = 179;
 function cloneVSTRule()