r2646 - renderAddMultipleObjectsForm(): display message log
authorDenis Ovsienko <infrastation@yandex.ru>
Wed, 15 Apr 2009 18:00:52 +0000 (18:00 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Wed, 15 Apr 2009 18:00:52 +0000 (18:00 +0000)
 - addMultipleObjects(): fix message generation
 - addLotOfObjects(): idem
 - renderRackObject(): revert to original (before changeset:2339) style for stickers and tags
 - renderUser(): idem
 - printTagTRs(): idem
 - renderAccessDenied(): idem

inc/interface.php
inc/ophandlers.php
pi.css

index b68b6f3..70e1192 100644 (file)
@@ -855,7 +855,7 @@ function renderRackObject ($object_id = 0)
                echo "<tr><td colspan=2 class=msg_error>Has problems</td></tr>\n";
        foreach (getAttrValues ($object_id, TRUE) as $record)
                if (!empty ($record['value']))
-                       echo "<tr><th width='50%' class=tdright><span class=sticker>${record['name']}</span>:</th><td class=tdleft>${record['a_value']}</td></tr>\n";
+                       echo "<tr><th width='50%' class=sticker>${record['name']}:</th><td class=sticker>${record['a_value']}</td></tr>\n";
        printTagTRs (makeHref(array('page'=>'objgroup', 'tab'=>'default', 'group_id'=>$info['objtype_id']))."&");
        echo "</table><br>\n";
        finishPortlet();
@@ -1436,6 +1436,7 @@ function printLog ($log)
                                77 => array ('code' => 'success', 'format' => 'Row "%s" was deleted successfully'),
                                78 => array ('code' => 'success', 'format' => 'File "%s" saved Ok'),
                                79 => array ('code' => 'success', 'format' => 'Rack "%s" was deleted successfully'),
+                               80 => array ('code' => 'success', 'format' => "Added new object '%s'"),
 
 // records 100~199 with fatal error messages
                                100 => array ('code' => 'error', 'format' => '%s'),
@@ -1522,6 +1523,9 @@ function printLog ($log)
                                181 => array ('code' => 'error', 'format' => "file uploads not allowed, change 'file_uploads' parameter in php.ini"),
                                182 => array ('code' => 'error', 'format' => 'SQL query failed: %s'),
                                183 => array ('code' => 'error', 'format' => "Tag id '%s' does not exist."),
+                               184 => array ('code' => 'error', 'format' => 'Submitted form is invalid at line %u'),
+                               185 => array ('code' => 'error', 'format' => "Failed to add object '%s'"),
+                               186 => array ('code' => 'error', 'format' => 'Incomplete form has been ignored. Cheers.'),
 
 // records 200~299 with warnings
                                200 => array ('code' => 'warning', 'format' => '%s'),
@@ -1532,6 +1536,7 @@ function printLog ($log)
                                205 => array ('code' => 'warning', 'format' => '%u change request(s) have been ignored'),
                                206 => array ('code' => 'warning', 'format' => 'Rack is not empty'),
                                207 => array ('code' => 'warning', 'format' => 'Ignored empty request'),
+
                        );
                        // Handle the arguments. Is there any better way to do it?
                        foreach ($log['m'] as $record)
@@ -2913,6 +2918,7 @@ function renderNATv4ForObject ($object_id = 0)
 
 function renderAddMultipleObjectsForm ()
 {
+       showMessageOrError();
        $typelist = getObjectTypeList();
        $typelist[0] = 'select type...';
        $max = getConfigVar ('MASSCOUNT');
@@ -5093,22 +5099,22 @@ function printTagTRs ($baseurl = '')
        global $expl_tags, $impl_tags, $auto_tags, $target_given_tags;
        if (getConfigVar ('SHOW_EXPLICIT_TAGS') == 'yes' and count ($target_given_tags))
        {
-               echo "<tr><th width='50%' class=tdright><span class=tagheader>Given explicit tags</span>:</th><td class=tdleft>";
+               echo "<tr><th width='50%' class=tagchain>Given explicit tags:</th><td class=tagchain>";
                echo serializeTags ($target_given_tags, $baseurl) . "</td></tr>\n";
        }
        if (getConfigVar ('SHOW_EXPLICIT_TAGS') == 'yes' and count ($expl_tags))
        {
-               echo "<tr><th width='50%' class=tdright><span class=tagheader>Effective explicit tags</span>:</th><td class=tdleft>";
+               echo "<tr><th width='50%' class=tagchain>Effective explicit tags:</th><td class=tagchain>";
                echo serializeTags ($expl_tags, $baseurl) . "</td></tr>\n";
        }
        if (getConfigVar ('SHOW_IMPLICIT_TAGS') == 'yes' and count ($impl_tags))
        {
-               echo "<tr><th width='50%' class=tdright>Effective implicit tags:</th><td class=tdleft>";
+               echo "<tr><th width='50%' class=tagchain>Effective implicit tags:</th><td class=tagchain>";
                echo serializeTags ($impl_tags, $baseurl) . "</td></tr>\n";
        }
        if (getConfigVar ('SHOW_AUTOMATIC_TAGS') == 'yes' and count ($auto_tags))
        {
-               echo "<tr><th width='50%' class=tdright>Automatic tags:</th><td class=tdleft>";
+               echo "<tr><th width='50%' class=tagchain>Automatic tags:</th><td class=tagchain>";
                echo serializeTags ($auto_tags) . "</td></tr>\n";
        }
 }
@@ -5412,19 +5418,19 @@ function renderUser ($user_id)
        $baseurl = makeHref(array('page'=>'userlist', 'tab'=>'default'))."&";
        if (getConfigVar ('SHOW_EXPLICIT_TAGS') == 'yes' and count ($target_given_tags))
        {
-               echo "<tr><th width='50%' class=tdright><span class=tagheader>Given explicit tags</span>:</th><td class=tdleft>";
+               echo "<tr><th width='50%' class=tagchain>Given explicit tags:</th><td class=tagchain>";
                echo serializeTags ($target_given_tags, $baseurl) . "</td></tr>\n";
        }
        $target_shadow = getImplicitTags ($target_given_tags);
        if (getConfigVar ('SHOW_IMPLICIT_TAGS') == 'yes' and count ($target_shadow))
        {
-               echo "<tr><th width='50%' class=tdright><span class=tagheader>Given implicit tags</span>:</th><td class=tdleft>";
+               echo "<tr><th width='50%' class=tagchain>Given implicit tags:</th><td class=tagchain>";
                echo serializeTags ($target_shadow, $baseurl) . "</td></tr>\n";
        }
        $target_auto_tags = generateEntityAutoTags ('user', $username);
        if (getConfigVar ('SHOW_AUTOMATIC_TAGS') == 'yes' and count ($target_auto_tags))
        {
-               echo "<tr><th width='50%' class=tdright><span class=tagheader>Automatic tags</span>:</th><td class=tdleft>";
+               echo "<tr><th width='50%' class=tagchain>Automatic tags:</th><td class=tagchain>";
                echo serializeTags ($target_auto_tags) . "</td></tr>\n";
        }
        echo '</table>';
@@ -5466,15 +5472,15 @@ function renderAccessDenied ()
        echo ' access denied ';
        printImageHREF ('DENIED');
        echo '</h3></th></tr>';
-       echo "<tr><th width='50%' class=tdright><span class=tagheader>User given tags</span>:</th><td class=tdleft>";
+       echo "<tr><th width='50%' class=tagchain>User given tags:</th><td class=tagchain>";
        echo serializeTags ($user_given_tags) . "&nbsp;</td></tr>\n";
-       echo "<tr><th width='50%' class=tdright><span class=tagheader>Target given tags</span>:</th><td class=tdleft>";
+       echo "<tr><th width='50%' class=tagchain>Target given tags:</th><td class=tagchain>";
        echo serializeTags ($target_given_tags) . "&nbsp;</td></tr>\n";
-       echo "<tr><th width='50%' class=tdright><span class=tagheader>Effective explicit tags</span>:</th><td class=tdleft>";
+       echo "<tr><th width='50%' class=tagchain>Effective explicit tags:</th><td class=tagchain>";
        echo serializeTags ($expl_tags) . "&nbsp;</td></tr>\n";
-       echo "<tr><th width='50%' class=tdright><span class=tagheader>Effective implicit tags</span>:</th><td class=tdleft>";
+       echo "<tr><th width='50%' class=tagchain>Effective implicit tags:</th><td class=tagchain>";
        echo serializeTags ($impl_tags) . "&nbsp;</td></tr>\n";
-       echo "<tr><th width='50%' class=tdright><span class=tagheader>Automatic tags</span>:</th><td class=tdleft>";
+       echo "<tr><th width='50%' class=tagchain>Automatic tags:</th><td class=tagchain>";
        echo serializeTags ($auto_tags) . "&nbsp;</td></tr>\n";
        echo "<tr><th width='50%' class=tdright>Requested page:</th><td class=tdleft>${pageno}</td></tr>\n";
        echo "<tr><th width='50%' class=tdright>Requested tab:</th><td class=tdleft>${tabno}</td></tr>\n";
index c4dc80f..83dfc92 100644 (file)
@@ -795,7 +795,7 @@ function addMultipleObjects()
        {
                if (!isset ($_REQUEST["${i}_object_type_id"]))
                {
-                       $log['m'][] = array ('code' => 'error', 'message' => "Submitted form is invalid at line " . $i + 1);
+                       $log = mergeLogs ($log, oneLiner (184, array ($i + 1)));
                        break;
                }
                assertUIntArg ("${i}_object_type_id", __FUNCTION__, TRUE);
@@ -817,11 +817,10 @@ function addMultipleObjects()
                        $_REQUEST["${i}_object_asset_no"],
                        $taglist
                ) === TRUE)
-                       $log['m'][] = array ('code' => 'success', 'message' => "Added new object '${name}'");
+                       $log = mergeLogs ($log, oneLiner (80, array ($name)));
                else
-                       $log['m'][] = array ('code' => 'error', 'message' => __FUNCTION__ . ': commitAddObject() failed');
+                       $log = mergeLogs ($log, oneLiner (185, array ($name)));
        }
-
        return buildWideRedirectURL ($log);
 }
 
@@ -833,7 +832,7 @@ function addLotOfObjects()
        assertStringArg ('namelist', __FUNCTION__, TRUE);
        $global_type_id = $_REQUEST['global_type_id'];
        if ($global_type_id == 0 or empty ($_REQUEST['namelist']))
-               $log['m'][] = array ('code' => 'warning', 'message' => 'Incomplete form has been ignored. Cheers.');
+               $log = mergeLogs ($log, oneLiner (186));
        else
        {
                // The name extractor below was stolen from ophandlers.php:addMultiPorts()
@@ -848,11 +847,11 @@ function addLotOfObjects()
                        else
                                $names2[] = rtrim ($parts[0]);
                }
-               foreach ($names2 as $cname)
-                       if (commitAddObject ($cname, '', '', $global_type_id, '', $taglist) === TRUE)
-                               $log['m'][] = array ('code' => 'success', 'message' => "Added new object '${cname}'");
+               foreach ($names2 as $name)
+                       if (commitAddObject ($name, '', '', $global_type_id, '', $taglist) === TRUE)
+                               $log = mergeLogs ($log, oneLiner (80, array ($name)));
                        else
-                               $log['m'][] = array ('code' => 'error', 'message' => "Could not add '${cname}'");
+                               $log = mergeLogs ($log, oneLiner (185, array ($name)));
        }
        return buildWideRedirectURL ($log);
 }
diff --git a/pi.css b/pi.css
index cd1e07d..01b4b1d 100644 (file)
--- a/pi.css
+++ b/pi.css
@@ -109,12 +109,22 @@ td.pcright {
        font-weight: bold;
 }
 
-span.sticker {
+th.tagchain {
+       background-color: #a0ffa0;
+       text-align: right;
+}
+
+td.tagchain {
+       text-align: left;
+}
+
+th.sticker {
        background-color: #ffffa0;
+       text-align: right;
 }
 
-span.tagheader {
-       background-color: #a0ffa0;
+td.sticker {
+       text-align: left;
 }
 
 .widetable td {