remove arguments overview_plugins and other_plugins from plugins_list function

Those can be gathered within the function.

1 files changed, 24 insertions, 11 deletions

diff --git a/inc/html.inc.php b/inc/html.inc.php
index fd32b38..a9e6a94 100644
--- a/inc/html.inc.php
+++ b/inc/html.inc.php
@@ -63,29 +63,42 @@ function plugin_header($host, $plugin) {
         return printf("<h3><a href='%shost.php?h=%s&p=%s'>%s</a></h3>\n", $CONFIG['weburl'], $host, $plugin, $plugin);
 }
 
-function plugins_list($host, $overview_plugins, $other_plugins, $selected_plugins = array()) {
+function plugins_list($host, $selected_plugins = array()) {
         global $CONFIG;
 
+        $plugins = collectd_plugins($host);
+
         echo '<div class="plugins">';
         echo '<h3>Plugins</h3>';
         echo '<ul>';
 
-        $selected = selected_overview($selected_plugins);
-        printf("<li><a %s href='%shost.php?h=%s'>%s</a></li>\n", $selected, $CONFIG['weburl'], $host, 'overview');
+        printf("<li><a %s href='%shost.php?h=%s'>overview</a></li>\n",
+                selected_overview($selected_plugins),
+                $CONFIG['weburl'],
+                $host
+        );
 
         # first the ones defined as ordered
-        foreach($overview_plugins as $plugin) {
-                if (in_array($plugin, $other_plugins)) {
-                        $selected = selected_plugin($plugin, $selected_plugins);
-                        printf("<li><a %s href='%shost.php?h=%s&p=%s'>%s</a></li>\n", $selected, $CONFIG['weburl'], $host, $plugin, $plugin);
+        foreach($CONFIG['overview'] as $plugin) {
+                if (in_array($plugin, $plugins)) {
+                        printf("<li><a %s href='%shost.php?h=%s&p=%s'>%4\$s</a></li>\n",
+                                selected_plugin($plugin, $selected_plugins),
+                                $CONFIG['weburl'],
+                                $host,
+                                $plugin
+                        );
                 }
         }
 
         # other plugins
-        foreach($other_plugins as $plugin) {
-                if (!in_array($plugin, $overview_plugins)) {
-                        $selected = selected_plugin($plugin, $selected_plugins);
-                        printf("<li><a %s href='%shost.php?h=%s&p=%s'>%s</a></li>\n", $selected, $CONFIG['weburl'], $host, $plugin, $plugin);
+        foreach($plugins as $plugin) {
+                if (!in_array($plugin, $CONFIG['overview'])) {
+                        printf("<li><a %s href='%shost.php?h=%s&p=%s'>%4\$s</a></li>\n",
+                                selected_plugin($plugin, $selected_plugins),
+                                $CONFIG['weburl'],
+                                $host,
+                                $plugin
+                        );
                 }
         }