r2644 - addMultipleObjects(): removed obsolete code, employed log format 2
authorDenis Ovsienko <infrastation@yandex.ru>
Wed, 15 Apr 2009 16:59:49 +0000 (16:59 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Wed, 15 Apr 2009 16:59:49 +0000 (16:59 +0000)
 - addLotOfObjects(): idem

inc/ophandlers.php

index 4157dea7b72d0ab0070e66581d9242fd82f4c74e..c4dc80fa4f7631907e4b1d798094eddbe68e5fb5 100644 (file)
@@ -788,16 +788,14 @@ function updateObject ()
 
 function addMultipleObjects()
 {
-       
-       $log = array();
+       $log = emptyLog();
        $taglist = isset ($_REQUEST['taglist']) ? $_REQUEST['taglist'] : array();
-       $keepvalues1 = TRUE;
        $max = getConfigVar ('MASSCOUNT');
        for ($i = 0; $i < $max; $i++)
        {
                if (!isset ($_REQUEST["${i}_object_type_id"]))
                {
-                       $log[] = array ('code' => 'error', 'message' => "Submitted form is invalid at line " . $i + 1);
+                       $log['m'][] = array ('code' => 'error', 'message' => "Submitted form is invalid at line " . $i + 1);
                        break;
                }
                assertUIntArg ("${i}_object_type_id", __FUNCTION__, TRUE);
@@ -805,40 +803,37 @@ function addMultipleObjects()
                assertStringArg ("${i}_object_label", __FUNCTION__, TRUE);
                assertStringArg ("${i}_object_asset_no", __FUNCTION__, TRUE);
                assertStringArg ("${i}_object_barcode", __FUNCTION__, TRUE);
-               $type_id[$i] = $_REQUEST["${i}_object_type_id"];
-               // Save user input for possible rendering.
-               $name[$i] = $_REQUEST["${i}_object_name"];
-               $label[$i] = $_REQUEST["${i}_object_label"];
-               $asset_no[$i] = $_REQUEST["${i}_object_asset_no"];
-               $barcode[$i] = $_REQUEST["${i}_object_barcode"];
+               $name = $_REQUEST["${i}_object_name"];
 
                // It's better to skip silently, than to print a notice.
-               if ($type_id[$i] == 0)
+               if ($_REQUEST["${i}_object_type_id"] == 0)
                        continue;
-               if (commitAddObject ($name[$i], $label[$i], $barcode[$i], $type_id[$i], $asset_no[$i], $taglist) === TRUE)
-                       $log[] = array ('code' => 'success', 'message' => "Added new object '${name[$i]}'");
+               if (commitAddObject
+               (
+                       $name,
+                       $_REQUEST["${i}_object_label"],
+                       $_REQUEST["${i}_object_barcode"],
+                       $_REQUEST["${i}_object_type_id"],
+                       $_REQUEST["${i}_object_asset_no"],
+                       $taglist
+               ) === TRUE)
+                       $log['m'][] = array ('code' => 'success', 'message' => "Added new object '${name}'");
                else
-                       $log[] = array ('code' => 'error', 'message' => __FUNCTION__ . ': commitAddObject() failed');
+                       $log['m'][] = array ('code' => 'error', 'message' => __FUNCTION__ . ': commitAddObject() failed');
        }
 
-       return buildWideRedirectURL($log);
+       return buildWideRedirectURL ($log);
 }
 
 function addLotOfObjects()
 {
-       $log = array();
+       $log = emptyLog();
        $taglist = isset ($_REQUEST['taglist']) ? $_REQUEST['taglist'] : array();
-       $keepvalues2 = TRUE;
        assertUIntArg ('global_type_id', __FUNCTION__, TRUE);
        assertStringArg ('namelist', __FUNCTION__, TRUE);
        $global_type_id = $_REQUEST['global_type_id'];
-       if ($global_type_id == 0)
-       {
-               if (!empty ($_REQUEST['namelist']))
-                       $log[] = array ('code' => 'error', 'message' => 'Object type is not selected, check the form below');
-               else
-                       $log[] = array ('code' => 'error', 'message' => 'Empty form has been ignored. Cheers.');
-       }
+       if ($global_type_id == 0 or empty ($_REQUEST['namelist']))
+               $log['m'][] = array ('code' => 'warning', 'message' => 'Incomplete form has been ignored. Cheers.');
        else
        {
                // The name extractor below was stolen from ophandlers.php:addMultiPorts()
@@ -855,14 +850,13 @@ function addLotOfObjects()
                }
                foreach ($names2 as $cname)
                        if (commitAddObject ($cname, '', '', $global_type_id, '', $taglist) === TRUE)
-                               $log[] = array ('code' => 'success', 'message' => "Added new object '${cname}'");
+                               $log['m'][] = array ('code' => 'success', 'message' => "Added new object '${cname}'");
                        else
-                               $log[] = array ('code' => 'error', 'message' => "Could not add '${cname}'");
+                               $log['m'][] = array ('code' => 'error', 'message' => "Could not add '${cname}'");
        }
-       return buildWideRedirectURL($log);
+       return buildWideRedirectURL ($log);
 }
 
-
 function deleteObject ()
 {
        assertUIntArg ('object_id', __FUNCTION__);