Fix the debug_result convenience function

The point of a convenience function is to be convenient, not to have an
unused first parameter and always be surrounded by the same boilerplate.

1 files changed, 4 insertions, 18 deletions

diff --git a/templates/show_install_config.inc.php b/templates/show_install_config.inc.php
index bae4d3ba..9ec17b1f 100644
--- a/templates/show_install_config.inc.php
+++ b/templates/show_install_config.inc.php
@@ -72,33 +72,19 @@ require $prefix . '/templates/install_header.inc.php';
         <table>
 <tr>
         <td class="align"><?php echo T_('ampache.cfg.php exists?'); ?></td>
-        <td>[
-        <?php
-                if (!is_readable($configfile)) {
-            echo debug_result('',false);
-                }
-                else {
-            echo debug_result('',true);
-                }
-        ?>
-        ]
+        <td>
+        <?php echo debug_result(is_readable($configfile)); ?>
         </td>
 </tr>
 <tr>
         <td class="align">
                 <?php echo T_('ampache.cfg.php configured?'); ?>
         </td>
-        <td>[
+        <td>
         <?php
                 $results = @parse_ini_file($configfile);
-                if (!check_config_values($results)) {
-            echo debug_result('',false);
-                }
-                else {
-            echo debug_result('',true);
-                }
+                echo debug_result(check_config_values($results));
         ?>
-        ]
         </td>
 </tr>
 <tr>