From 5ca959fa688255a8de61f89fe2751eb4d24a6912 Mon Sep 17 00:00:00 2001 From: Florian Schaal <florian@schaal-24.de> Date: Tue, 22 Mar 2016 09:22:07 -0400 Subject: [PATCH] fixed typo --- interface/lib/classes/tools_monitor.inc.php | 114 +++++++++++++++++++++++++++++++------------------------- 1 files changed, 63 insertions(+), 51 deletions(-) diff --git a/interface/lib/classes/tools_monitor.inc.php b/interface/lib/classes/tools_monitor.inc.php index 19f2ccd..bd1a969 100644 --- a/interface/lib/classes/tools_monitor.inc.php +++ b/interface/lib/classes/tools_monitor.inc.php @@ -33,7 +33,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'server_load' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'server_load' AND server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $data = unserialize($record['data']); @@ -80,7 +80,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'disk_usage' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'disk_usage' AND server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $data = unserialize($record['data']); @@ -91,16 +91,18 @@ $html = '<div class="systemmonitor-state state-'.$record['state'].'"> <div class="systemmonitor-content icons32 ico-'.$record['state'].'"> - <table> + <table class="table"> + <thead class="dark"> <tr> - <td>'.$app->lng("monitor_diskusage_filesystem_txt").'</td> - <td>'.$app->lng("monitor_diskusage_type_txt").'</td> - <td>'.$app->lng("monitor_diskusage_size_txt").'</td> - <td>'.$app->lng("monitor_diskusage_used_txt").'</td> - <td>'.$app->lng("monitor_diskusage_available_txt").'</td> - <td>'.$app->lng("monitor_diskusage_usage_txt").'</td> - <td>'.$app->lng("monitor_diskusage_mounted_txt").'</td> - </tr>'; + <th>'.$app->lng("monitor_diskusage_filesystem_txt").'</th> + <th class="small-col">'.$app->lng("monitor_diskusage_type_txt").'</th> + <th class="tiny-col">'.$app->lng("monitor_diskusage_size_txt").'</th> + <th class="tiny-col">'.$app->lng("monitor_diskusage_used_txt").'</th> + <th class="tiny-col">'.$app->lng("monitor_diskusage_available_txt").'</th> + <th class="tiny-col">'.$app->lng("monitor_diskusage_usage_txt").'</th> + <th>'.$app->lng("monitor_diskusage_mounted_txt").'</th> + </tr></thead> + <tbody>'; foreach($data as $line) { $html .= '<tr>'; foreach ($line as $item) { @@ -108,7 +110,7 @@ } $html .= '</tr>'; } - $html .= '</table>'; + $html .= '</tbody></table>'; $html .= '</div></div>'; } else { $html = '<p>'.$app->lng("no_data_diskusage_txt").'</p>'; @@ -121,35 +123,41 @@ function showDatabaseSize () { global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'database_size' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'database_size' AND server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $data = unserialize($record['data']); - /* - Format the data - */ + //* format the data $html = '<div class="systemmonitor-state state-'.$record['state'].'"> <div class="systemmonitor-content icons32 ico-'.$record['state'].'"> - <table> - <thead> - <tr> - <td>'.$app->lng("monitor_database_name_txt").'</td> - <td>'.$app->lng("monitor_database_size_txt").'</td> - <td>'.$app->lng("monitor_database_client_txt").'</td> - </tr>'; + <table class="table"><thead class="dark"><tr> + <th>'.$app->lng("monitor_database_name_txt").'</th> + <th class="tiny-col">'.$app->lng("monitor_database_size_txt").'</th> + <th>'.$app->lng("monitor_database_client_txt").'</th> + <th>'.$app->lng("monitor_database_domain_txt").'</th> + </tr></thead> + <tbody>'; foreach($data as $line) { $html .= '<tr>'; if ($line['size'] > 0) $line['size'] = $app->functions->formatBytes($line['size']); - $t=$app->db->queryOneRecord("SELECT username FROM client WHERE sys_groupid = ".$line['client_id']); - $line['client_id']=$t['username']; - unset($t); + + //* get the client + $tmp = $app->db->queryOneRecord("SELECT client.username FROM web_database, sys_group, client WHERE web_database.sys_groupid = sys_group.groupid AND sys_group.client_id = client.client_id AND web_database.database_name=?", $line['database_name']); + $line['client'] = $tmp['username']; + + //* get the domain + $tmp = $app->db->queryOneRecord("SELECT domain FROM web_domain WHERE domain_id=(SELECT parent_domain_id FROM web_database WHERE database_name=?", $line['database_name']); + $line['domain'] = $tmp['domain']; + + //* remove the sys_groupid from output + unset($line['sys_groupid']); + foreach ($line as $item) { $html .= '<td>' . $item . '</td>'; } - $html .= '</tr></tmpl loop>'; + $html .= '</tr>'; } - $html .= '</tbody></table>'; - $html .= '</div></div>'; + $html .= '</tbody></table></div></div>'; } else { $html = '<p>'.$app->lng("no_data_database_size_txt").'</p>'; } @@ -160,7 +168,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'mem_usage' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'mem_usage' and server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $data = unserialize($record['data']); @@ -171,7 +179,8 @@ $html = '<div class="systemmonitor-state state-'.$record['state'].'"> <div class="systemmonitor-content icons32 ico-'.$record['state'].'"> - <table>'; + <table class="table"> + <tbody>'; foreach($data as $key => $value) { if ($key != '') { @@ -181,7 +190,7 @@ </tr>'; } } - $html .= '</table>'; + $html .= '</tbody></table>'; $html .= '</div></div>'; } else { @@ -195,7 +204,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'cpu_info' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'cpu_info' and server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $data = unserialize($record['data']); @@ -206,7 +215,8 @@ $html = '<div class="systemmonitor-state state-'.$record['state'].'"> <div class="systemmonitor-content icons32 ico-'.$record['state'].'"> - <table>'; + <table class="table"> + <tbody>'; foreach($data as $key => $value) { if ($key != '') { $html .= '<tr> @@ -215,7 +225,7 @@ </tr>'; } } - $html .= '</table>'; + $html .= '</tbody></table>'; $html .= '</div></div>'; } else { $html = '<p>'.$app->lng("no_data_cpuinfo_txt").'</p>'; @@ -228,7 +238,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'services' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'services' and server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $data = unserialize($record['data']); @@ -239,7 +249,8 @@ $html = '<div class="systemmonitor-state state-'.$record['state'].'"> <div class="systemmonitor-content icons32 ico-'.$record['state'].'"> - <table>'; + <table class="table"> + <tbody>'; if($data['webserver'] != -1) { if($data['webserver'] == 1) { @@ -327,7 +338,7 @@ } - $html .= '</table></div></div>'; + $html .= '</tbody></table></div></div>'; } else { $html = '<p>'.$app->lng("no_data_services_txt").'</p>'; } @@ -340,7 +351,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'system_update' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'system_update' and server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $html = @@ -370,7 +381,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'openvz_beancounter' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'openvz_beancounter' and server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $html = @@ -399,7 +410,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'raid_state' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'raid_state' and server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $html = @@ -415,7 +426,9 @@ } else { $data = unserialize($record['data']); - $html .= nl2br($data['output']); + // improve view @Author <info@typoworx.de> + //-- $html .= nl2br($data['output']); + $html .= '<xmp>' . $data['output'] . '</xmp>'; } $html .= '</div></div>'; @@ -430,7 +443,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'rkhunter' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'rkhunter' and server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $html = @@ -461,7 +474,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'log_fail2ban' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'log_fail2ban' and server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $html = @@ -475,7 +488,7 @@ $data = unserialize($record['data']); if ($data == '') { $html .= '<p>'. - 'fail2ban is not installed at this server.<br />' . + 'fail2ban is not installed on this server.<br />' . 'See more (for debian) <a href="http://www.howtoforge.com/fail2ban_debian_etch" target="htf">here...</a>'. '</p>'; } @@ -495,7 +508,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'log_mongodb' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'log_mongodb' and server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $html = @@ -509,8 +522,7 @@ $data = unserialize($record['data']); if ($data == '') { $html .= '<p>'. - 'MongoDB is not installed at this server.<br />' . - 'See more (for debian) <a href="http://www.howtoforge.com/fail2ban_debian_etch" target="htf">here...</a>'. + 'MongoDB is not installed on this server.<br />' . '</p>'; } else { @@ -527,7 +539,7 @@ function showIPTables() { global $app; - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'iptables_rules' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'iptables_rules' and server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $html = '<div class="systemmonitor-state state-'.$record['state'].'"> @@ -551,7 +563,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'mailq' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT data, state FROM monitor_data WHERE type = 'mailq' and server_id = ? ORDER BY created DESC", $_SESSION['monitor']['server_id']); if(isset($record['data'])) { $data = unserialize($record['data']); @@ -567,7 +579,7 @@ global $app; /* fetch the Data from the DB */ - $record = $app->db->queryOneRecord("SELECT created FROM monitor_data WHERE type = '" . $type . "' and server_id = " . $_SESSION['monitor']['server_id'] . " order by created desc"); + $record = $app->db->queryOneRecord("SELECT created FROM monitor_data WHERE type = ? and server_id = ? ORDER BY created DESC", $type, $_SESSION['monitor']['server_id']); /* TODO: datetimeformat should be set somewhat other way */ $dateTimeFormat = $app->lng("monitor_settings_datetimeformat_txt"); -- Gitblit v1.9.1