r4171 bugfix: creating vlan template was not possible due to invalid arg assertion.
authorAlexey Andriyanov <alan@al-an.info>
Sun, 30 Jan 2011 12:18:04 +0000 (12:18 +0000)
committerAlexey Andriyanov <alan@al-an.info>
Sun, 30 Jan 2011 12:18:04 +0000 (12:18 +0000)
removed max_local_vlans DB field completely (it was not used)

install.php: removed max_local_vlans DB field from VLANSwitchTemplate table
addVLANSwitchTemplate: removed check of max_local_vlans
renderVSTListEditor: removed form input named vst_maxvlans
-renderVSTEditor: unused
getVSTStats: removed max_local_vlans field from select query
upgrade.php: updating SQL schema respectively

inc/database.php
inc/interface.php
inc/ophandlers.php
install.php
upgrade.php

index c7f5f94..040cd39 100644 (file)
@@ -4183,7 +4183,7 @@ function getVSTStats()
 {
        $result = usePreparedSelectBlade
        (
-               'SELECT id, max_local_vlans, description, ' .
+               'SELECT id, description, ' .
                '(SELECT COUNT(object_id) FROM VLANSwitch WHERE template_id = id) AS switchc, ' .
                '(SELECT COUNT(rule_no) FROM VLANSTRule WHERE vst_id = id) AS rulec ' .
                'FROM VLANSwitchTemplate ORDER BY description'
index d7cac33..f464050 100644 (file)
@@ -8834,7 +8834,7 @@ function renderVSTListEditor()
 {
        function printNewItemTR ()
        {
-               printOpFormIntro ('add', array ('vst_maxvlans' => 0));
+               printOpFormIntro ('add');
                echo '<tr>';
                echo '<td>' . getImageHREF ('create', 'create template', TRUE, 104) . '</td>';
                echo '<td><input type=text name=vst_descr tabindex=101></td>';
@@ -8847,7 +8847,7 @@ function renderVSTListEditor()
                printNewItemTR();
        foreach (getVSTStats() as $vst_id => $vst_info)
        {
-               printOpFormIntro ('upd', array ('vst_id' => $vst_id, 'vst_maxvlans' => 0));
+               printOpFormIntro ('upd', array ('vst_id' => $vst_id));
                echo '<tr><td>';
                if ($vst_info['switchc'])
                        printImageHREF ('nodestroy', 'template used elsewhere');
@@ -8922,22 +8922,6 @@ function renderVST ($vst_id)
        echo '</td></tr></table>';
 }
 
-function renderVSTEditor ($vst_id)
-{
-       $vst = getVLANSwitchTemplate ($vst_id);
-       echo '<center><h1>' . niftyString ($vst['description']) . '</h1></center>';
-       echo '<table border=0 cellpadding=10 cellpadding=1 align=center>';
-       printOpFormIntro ('upd');
-       echo '<tr><td class=tdright><label for=input1>Description:</label></td>';
-       echo "<td class=tdleft><input type=text name=vst_descr id=input1 maxlength=255 value='";
-       echo niftyString ($vst['description'], 0) . "'></td></tr>";
-       echo '<tr><td class=tdright><label for=input2>Max local VLANs:</label></td>';
-       echo "<td class=tdleft><input type=text name=vst_maxvlans id=input2 value=";
-       echo $vst['max_local_vlans'] . '></td></tr>';
-       echo '<tr><td colspan=2 class=tdcenter>' . getImageHREF ('SAVE', 'Save changes', TRUE) . '</td></tr>';
-       echo '</form></table>';
-}
-
 function renderVSTRulesEditor ($vst_id)
 {
        $vst = getVLANSwitchTemplate ($vst_id);
index 3221ca8..e87f564 100644 (file)
@@ -337,7 +337,6 @@ $opspec_list['8021q-vstlist-upd'] = array
        '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
@@ -2648,18 +2647,11 @@ function addVLANSwitchTemplate()
 {
        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 = usePreparedInsertBlade
        (
                'VLANSwitchTemplate',
                array
                (
-                       'max_local_vlans' => $max_local_vlans,
                        'description' => $sic['vst_descr'],
                )
        );
index cf9a5da..b9a2a6f 100644 (file)
@@ -885,7 +885,6 @@ CREATE TABLE `VLANSwitch` (
 
 CREATE TABLE `VLANSwitchTemplate` (
   `id` int(10) unsigned NOT NULL auto_increment,
-  `max_local_vlans` int(10) unsigned default NULL,
   `description` char(255) default NULL,
   `mutex_rev` int(10) NOT NULL,
   `saved_by` char(64) NOT NULL,
index d0315a7..ca418be 100644 (file)
@@ -836,7 +836,7 @@ CREATE TABLE `EntityLink` (
                        $query[] = 'ALTER TABLE RackSpace ADD CONSTRAINT `RackSpace-FK-rack_id` FOREIGN KEY (rack_id) REFERENCES Rack (id)';
                        $query[] = "ALTER TABLE `IPv4Allocation` ADD CONSTRAINT `IPv4Allocation-FK-object_id` FOREIGN KEY (`object_id`) REFERENCES `RackObject` (`id`) ON DELETE CASCADE";
                        $query[] = "INSERT INTO `Config` (varname, varvalue, vartype, emptyok, is_hidden, description) VALUES ('SYNCDOMAIN_MAX_PROCESSES','0','uint','yes','no', 'How many worker proceses syncdomain cron script should create')";
-                       $query[] = "ALTER TABLE `VLANSwitchTemplate` ADD COLUMN `mutex_rev` int(10) NOT NULL AFTER `max_local_vlans`";
+                       $query[] = "ALTER TABLE `VLANSwitchTemplate` ADD COLUMN `mutex_rev` int(10) NOT NULL AFTER `id`";
                        $query[] = "ALTER TABLE `VLANSwitchTemplate` ADD COLUMN `saved_by` char(64) NOT NULL AFTER `description`";
                        $query[] = "INSERT INTO `Attribute` (`id`, `type`, `name`) VALUES (26,'dict','Hypervisor')";
                        $query[] = "INSERT INTO `Chapter` (`id`, `sticky`, `name`) VALUES (29,'no','Yes/No')";
@@ -894,6 +894,7 @@ CREATE TABLE `EntityLink` (
                        $query[] = "UPDATE Config SET varvalue = '8' WHERE varname = 'MASSCOUNT'";
                        $query[] = "UPDATE RackObject SET label = NULL WHERE label = ''";
                        $query[] = "UPDATE Config SET varvalue = '0.19.0' WHERE varname = 'DB_VERSION'";
+                       $query[] = 'ALTER TABLE `VLANSwitchTemplate` DROP COLUMN `max_local_vlans`';
                        break;
                default:
                        showError ("executeUpgradeBatch () failed, because batch '${batchid}' isn't defined");