Update request for MONITOR plugin

[AndyStewart]

Can the images for the hosts be hyperlinked to their graph pages?

[rony]

Moved to Addons

[cigamit]

Seems a reasonable request.

Add a ticket as a Feature Request to the bug tracker located here

http://cactiusers.org/bugs/

and I will get to it when I get the chance.

[bdiez]

It's not difficult to reach this with some simple changes. I managed to do it in a short time during this afternoon and now is working for me. So in the meantime that cigamit publish a new official version of the monitor plugin with this feature you can try to do the following steps:


1. Open for editing the "monitor.php" file under the /plugins/monitor folder

(it's always a good idea to maintain a backup copy of the original file).


2. Locate the line with the following code:

Code: Select all

$sql = "SELECT description, status, hostname, cur_time, status_rec_date, availability FROM host where disabled = '' and monitor = 'on' ORDER BY description";

Then replace it with the following line:

Code: Select all

$sql = "SELECT id, description, status, hostname, cur_time, status_rec_date, availability FROM host where disabled = '' and monitor = 'on' ORDER BY description";

(note that we're simply adding the id field to the select statement).


3. Locate the lines with the following code:

Code: Select all

while ($row = mysql_fetch_array($result, MYSQL_ASSOC)) {
	// Loop through each host one by one
	$name = $row['description'];
	$status = $row['status'];
	...
	...
	etc.

...and insert the following extra line:

Code: Select all

while ($row = mysql_fetch_array($result, MYSQL_ASSOC)) {
	// Loop through each host one by one
	$anchor = $config['url_path'] . "graph_view.php?action=preview&host_id=" . $row['id'];
	$name = $row['description'];
	$status = $row['status'];
	...
	...
	etc.

(we create a new $anchor variable to be used later).


4. And finally, locate the line who says:

Code: Select all

	print "<td valign=top><center><img src='images/$i' name='$name' id='$name' title='$title'><br>$s$name$s2</center></td>\n";

...then replace it with the following code:

Code: Select all

	print "<td valign=top>
	         <center>
	           <a href=$anchor>
	             <img src='images/$i' name='$name' id='$name' title='$title' border='0'><br>
	             <font color='black'>$s$name$s2</font>
	           </a>
	         </center>
	       </td>\n";

(here is when we're creating the anchor corresponding to each icon, but trying to maintain the original aspect of the page).


Save your modified file (as "monitor.php", do not rename it), refresh your monitor tab ...et voila! It works for me

For sure you're thinking that it would be easier if I post my modified monitor.php file, but I prefer not to do this, because I think my installed monitor plugin is an outdated version and maybe it can cause compatibility problems to others.


Good luck.

Bernardo

[cigamit]

Added this to my SVN copy, works perfectly for me.

Thanks!