Major rewrite of screen handling code.

git-svn-id: svn://svn.zabbix.com/trunk@918 97f52cf1-0a1b-0410-bd0e-c28be96e8082

1 files changed, 23 insertions, 55 deletions

diff --git a/frontends/php/screenedit.php b/frontends/php/screenedit.php
index 1e314c4c..18493884 100644
--- a/frontends/php/screenedit.php
+++ b/frontends/php/screenedit.php
@@ -49,14 +49,7 @@
 //				delete_screen_item($HTTP_GET_VARS["screenitemid"]);
 //				unset($HTTP_GET_VARS["screenitemid"]);
 //			}
-			if($HTTP_GET_VARS["resource"]==0)
-			{
-				$result=add_screen_graph($HTTP_GET_VARS["screenid"],$HTTP_GET_VARS["x"],$HTTP_GET_VARS["y"],$HTTP_GET_VARS["itemid"],$HTTP_GET_VARS["width"],$HTTP_GET_VARS["height"]);
-			}
-			if($HTTP_GET_VARS["resource"]==1)
-			{
-				$result=add_screen_item($HTTP_GET_VARS["screenid"],$HTTP_GET_VARS["x"],$HTTP_GET_VARS["y"],$HTTP_GET_VARS["graphid"],$HTTP_GET_VARS["width"],$HTTP_GET_VARS["height"]);
-			}
+			$result=add_screen_item($HTTP_GET_VARS["resource"],$HTTP_GET_VARS["screenid"],$HTTP_GET_VARS["x"],$HTTP_GET_VARS["y"],$HTTP_GET_VARS["resourceid"],$HTTP_GET_VARS["width"],$HTTP_GET_VARS["height"]);
 			unset($HTTP_GET_VARS["x"]);
 			show_messages($result,"Item added","Cannot add item");
 		}
@@ -97,10 +90,8 @@
 
 		{
                 	$screenitemid=0;
-                	$screengraphid=0;
-                	$graphid=0;
-                	$sysmapid=0;
-                	$itemid=0;
+                	$resource=0;
+                	$resourceid=0;
                 	$width=500;
                 	$height=100;
 			$resource=@iif(isset($HTTP_GET_VARS["resource"]),$HTTP_GET_VARS["resource"],0);
@@ -109,26 +100,15 @@
 		$iresult=DBSelect("select * from screens_items where screenid=$screenid and x=$c and y=$r");
         	if(DBnum_rows($iresult)>0)
         	{
-			$resource=1;
         		$irow=DBfetch($iresult);
         		$screenitemid=$irow["screenitemid"];
-			$graphid=$irow["graphid"];
+			$resource=$irow["resource"];
+			$resourceid=$irow["resourceid"];
 			$width=$irow["width"];
 			$height=$irow["height"];
 			$found=1;
         	}
 
-		$iresult=DBSelect("select * from screens_graphs where screenid=$screenid and x=$c and y=$r");
-        	if(DBnum_rows($iresult)>0)
-        	{
-			$resource=0;
-        		$irow=DBfetch($iresult);
-        		$screengraphid=$irow["screengraphid"];
-			$itemid=$irow["itemid"];
-			$width=$irow["width"];
-			$height=$irow["height"];
-        	}
-
 		if(isset($HTTP_GET_VARS["x"])&&($HTTP_GET_VARS["x"]==$c)&&($HTTP_GET_VARS["y"]==$r))
 		{
         		show_table2_header_begin();
@@ -137,63 +117,51 @@
         		echo "<input name=\"screenid\" type=\"hidden\" value=$screenid>";
 			echo "<input name=\"x\" type=\"hidden\" value=$c>";
 			echo "<input name=\"y\" type=\"hidden\" value=$r>";
-			echo "<input name=\"screenitemid\" type=\"hidden\" value=$screenitemid>";
-			echo "<input name=\"screengraphid\" type=\"hidden\" value=$screengraphid>";
-			echo "<input name=\"resource\" type=\"hidden\" value=$resource>";
+//			echo "<input name=\"resourceid\" type=\"hidden\" value=$resourceid>";
+//			echo "<input name=\"resource\" type=\"hidden\" value='$resource'>";
 
 			show_table2_v_delimiter();
 			echo "Resource";
 			show_table2_h_delimiter();
 			echo "<select name=\"resource\" size=1 onChange=\"submit()\">";
-			echo "<OPTION VALUE='0' ".iif($resource==0,"selected","").">Simple graph";
-			echo "<OPTION VALUE='1' ".iif($resource==1,"selected","").">Graph";
-//			echo "<OPTION VALUE='2' ".iif($resource==2,"selected","").">Network map";
+			echo "<OPTION VALUE='1' ".iif($resource==1,"selected","").">Simple graph";
+			echo "<OPTION VALUE='0' ".iif($resource==0,"selected","").">Graph";
 			echo "</SELECT>";
 
-			if($resource == 0)
+			if($resource == 1)
 			{
 				show_table2_v_delimiter();
 				echo nbsp("Graph name");
 				show_table2_h_delimiter();
 				$result=DBselect("select h.host,i.description,i.itemid from hosts h,items i where h.hostid=i.hostid and h.status in (0,2) and i.status=0 order by h.host,i.description");
-				echo "<select name=\"itemid\" size=1>";
+				echo "<select name=\"resourceid\" size=1>";
 				echo "<OPTION VALUE='0'>(none)";
 				for($i=0;$i<DBnum_rows($result);$i++)
 				{
 					$host_=DBget_field($result,$i,0);
 					$description_=DBget_field($result,$i,1);
 					$itemid_=DBget_field($result,$i,2);
-					echo "<OPTION VALUE='$itemid_' ".iif($itemid==$itemid_,"selected","")."'>$host_: $description_";
+					echo "<OPTION VALUE='$itemid_' ".iif($resourceid==$itemid_,"selected","")."'>$host_: $description_";
 				}
 				echo "</SELECT>";
 			}
-			else
-			{
-				echo "<input class=\"biginput\" name=\"itemid\" type=\"hidden\" size=1 value=\"$itemid\">";
-			}
-
-			if($resource == 1)
+			else if($resource == 0)
 			{
 				show_table2_v_delimiter();
 				echo nbsp("Graph name");
 				show_table2_h_delimiter();
 				$result=DBselect("select graphid,name from graphs order by name");
-				echo "<select name=\"graphid\" size=1>";
+				echo "<select name=\"resourceid\" size=1>";
 				echo "<OPTION VALUE='0'>(none)";
 				for($i=0;$i<DBnum_rows($result);$i++)
 				{
 					$name_=DBget_field($result,$i,1);
 					$graphid_=DBget_field($result,$i,0);
-					echo "<OPTION VALUE='$graphid_' ".iif($graphid==$graphid_,"selected","").">$name_";
+					echo "<OPTION VALUE='$graphid_' ".iif($resourceid==$graphid_,"selected","").">$name_";
 				}
 				echo "</SELECT>";
 			}
-			else
-			{
-				echo "<input class=\"biginput\" name=\"graphid\" type=\"hidden\" size=1 value=\"$graphid\">";
-			}
-
-			if($resource == 2)
+			else if($resource == 2)
 			{
 				show_table2_v_delimiter();
 				echo "Map";
@@ -205,13 +173,13 @@
 				{
 					$name_=DBget_field($result,$i,1);
 					$sysmapid_=DBget_field($result,$i,0);
-					echo "<OPTION VALUE='$sysmapid_' ".iif($sysmapid==$sysmapid_,"selected","").">$name_";
+					echo "<OPTION VALUE='$sysmapid_' ".iif($resourceid==$sysmapid_,"selected","").">$name_";
 				}
 				echo "</SELECT>";
 			}
 			else
 			{
-				echo "<input class=\"biginput\" name=\"sysmapid\" type=\"hidden\" size=1 value=\"$sysmapid\">";
+				echo "<input class=\"biginput\" name=\"resourceid\" type=\"hidden\" size=1 value=\"$resourceid\">";
 			}
 
 			if($resource!=2)
@@ -233,7 +201,7 @@
 
 			show_table2_v_delimiter2();
 			echo "<input type=\"submit\" name=\"register\" value=\"add\">";
-			if(($screenitemid!=0)||($screengraphid!=0)) 
+			if($resourceid!=0) 
 			{ 
 				echo "<input type=\"submit\" name=\"register\" value=\"update\">";
 			}
@@ -241,13 +209,13 @@
 
 			show_table2_header_end();
 		}
-		else if($graphid!=0)
+		else if( ($screenitemid!=0) && ($resource==0) )
 		{
-			echo "<a href=screenedit.php?register=edit&screenid=$screenid&x=$c&y=$r><img src='chart2.php?graphid=$graphid&width=$width&height=$height&period=3600' border=0></a>";
+			echo "<a href=screenedit.php?register=edit&screenid=$screenid&x=$c&y=$r><img src='chart2.php?graphid=$resourceid&width=$width&height=$height&period=3600' border=0></a>";
 		}
-		else if($itemid!=0)
+		else if( ($screenitemid!=0) && ($resource==1) )
 		{
-			echo "<a href=screenedit.php?register=edit&screenid=$screenid&x=$c&y=$r><img src='chart.php?itemid=$itemid&width=$width&height=$height&period=3600' border=0></a>";
+			echo "<a href=screenedit.php?register=edit&screenid=$screenid&x=$c&y=$r><img src='chart.php?itemid=$resourceid&width=$width&height=$height&period=3600' border=0></a>";
 		}
 		else
 		{