From: arjen <arjen>
Date: Thu, 14 Aug 2003 10:31:57 +0000 (+0000)
Subject: BUGFIX: Removed parameters were somtimes shown on the wrong side
X-Git-Tag: V0_0_7~4
X-Git-Url: http://www.andromeda.nl/gitweb/?p=gnucomo.git;a=commitdiff_plain;h=a4725bb3d0e870a394ecabfe47a2f0371d3528bc

BUGFIX: Removed parameters were somtimes shown on the wrong side
of the parameter difference page.
---

diff --git a/src/web/parameter_compare.php b/src/web/parameter_compare.php
index 9cc64bd..a977b6f 100644
--- a/src/web/parameter_compare.php
+++ b/src/web/parameter_compare.php
@@ -9,7 +9,7 @@
 ***********************
 **      FILE NAME      : parameter_compare.php
 **      SYSTEM NAME    : Gnucomo - Gnu Computer Monitoring
-**      VERSION NUMBER : $Revision: 1.4 $
+**      VERSION NUMBER : $Revision: 1.5 $
 **
 **  DESCRIPTION      : 
 **
@@ -27,7 +27,11 @@
 
 /*****************************
    $Log: parameter_compare.php,v $
-   Revision 1.4  2003-07-15 11:06:45  arjen
+   Revision 1.5  2003-08-14 10:31:57  arjen
+   BUGFIX: Removed parameters were somtimes shown on the wrong side
+   of the parameter difference page.
+
+   Revision 1.4  2003/07/15 11:06:45  arjen
    Removed parameters are displayed in a shaded style.
 
    Revision 1.3  2003/02/13 09:01:29  arjen
@@ -38,7 +42,7 @@
 
 ******************************/
 
-// RCSID = "$Id: parameter_compare.php,v 1.4 2003-07-15 11:06:45 arjen Exp $";
+// RCSID = "$Id: parameter_compare.php,v 1.5 2003-08-14 10:31:57 arjen Exp $";
 
 ini_set('include_path', '.:./classes:../phpclasses');
 
@@ -129,10 +133,12 @@ class param_diff extends page
    {
    if (!empty($_POST['oid']))
    {
-      $res = pg_exec($this->database, "SELECT objectid, objectname FROM object WHERE objectid=" . $_POST['oid']);
+      $res = pg_exec($this->database, "SELECT objectid, objectname FROM object
+                                       WHERE objectid=" . $_POST['oid']);
       $obj = pg_fetch_object($res, 0);
       echo "<h1>" . $_POST['class'] . " parameters for " . $obj->objectname;
-      $res = pg_exec($this->database, "SELECT objectid, objectname FROM object WHERE objectid=" . $_POST['compare_to']);
+      $res = pg_exec($this->database, "SELECT objectid, objectname FROM object
+                                       WHERE objectid=" . $_POST['compare_to']);
       $cmp_obj = pg_fetch_object($res, 0);
       echo " compared to " . $cmp_obj->objectname . "</h1><hr>";
 
@@ -164,25 +170,26 @@ class param_diff extends page
             $prnext = param_properties($this->database, $nextpar->paramid);
             if (property_compare($pr, $prnext))
             {
-               display_parameter($par->name, $pr, "", $this->is_removed($par->paramid));
-               display_parameter($nextpar->name, $prnext, "", $this->is_removed($nextpar->paramid));
+               $Style = "";
+            }
+            else
+            {
+               $Style = "both";
+            }
+            // We want the parameters of $obj on the left, so we need
+            // to swap the left and right sides if appropriate.
+
+            if ($par->objectid == $obj->objectid)
+            {
+               display_parameter($par->name, $pr, $Style, $this->is_removed($par->paramid));
+               display_parameter($nextpar->name, $prnext, $Style, $this->is_removed($nextpar->paramid));
             }
             else
             {
-               // We want the parameters of $obj on the left, so we need
-               // to swap the left and right sides if appropriate.
-
-               if ($par->objectid == $obj->objectid)
-               {
-                  display_parameter($par->name, $pr, "both", $this->is_removed($par->paramid));
-                  display_parameter($nextpar->name, $prnext, "both", $this->is_removed($nextpar->paramid));
-               }
-               else
-               {
-                  display_parameter($nextpar->name, $prnext, "both", $this->is_removed($nextpar->paramid));
-                  display_parameter($par->name, $pr, "both", $this->is_removed($par->paramid));
-               }
+               display_parameter($nextpar->name, $prnext, $Style, $this->is_removed($nextpar->paramid));
+               display_parameter($par->name, $pr, $Style, $this->is_removed($par->paramid));
             }
+ 
          }
          else
          {