r2063 + fix half of the ticket:157
authorDenis Ovsienko <infrastation@yandex.ru>
Wed, 16 Jul 2008 17:33:30 +0000 (17:33 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Wed, 16 Jul 2008 17:33:30 +0000 (17:33 +0000)
inc/database.php
inc/ophandlers.php

index 178e889..dff3981 100644 (file)
@@ -775,7 +775,10 @@ function commitAddPort ($object_id = 0, $port_name, $port_type_id, $port_label,
                return 'SQL query failed';
 }
 
-function commitUpdatePort ($port_id, $port_name, $port_label, $port_l2address, $port_reservation_comment)
+// The fifth argument may be either explicit 'NULL' or some (already quoted by the upper layer)
+// string value. In case it is omitted, we just assign it its current value.
+// It would be nice to simplify this semantics later.
+function commitUpdatePort ($port_id, $port_name, $port_label, $port_l2address, $port_reservation_comment = 'reservation_comment')
 {
        global $dbxlink;
        $port_l2address = l2addressForDatabase ($port_l2address);
index 960ad9f..52ad8ae 100644 (file)
@@ -176,6 +176,7 @@ function addPortForObject ()
 function editPortForObject ()
 {
        assertUIntArg ('port_id', __FUNCTION__);
+       // tolerate empty value now to produce custom informative message later
        assertStringArg ('name', __FUNCTION__, TRUE);
        if (empty ($_REQUEST['name']))
                return buildRedirectURL_ERR ('Port name cannot be empty');