Slight wording changes on menus
[people/meteger/rom-o-matic/.git] / utils.php
index a2c4b14..4855c41 100644 (file)
--- a/utils.php
+++ b/utils.php
@@ -119,14 +119,22 @@ function parse_nic_file ()
     }
     fclose ( $fd );
 
-    // add special cases
-    $nics["undi"] = "undionly";
-    $nics["gpxe-all-drivers"] = "gpxe";
-
-    // put NICs in nice alpha order for menu
+    // put most NICs in nice alpha order for menu
     ksort ( $nics );
 
-    return ( array ( $nics, $roms ) );
+    // add special cases to the top
+
+       $new_nics = array ( "all-drivers" => "gpxe",
+                                               "undi" => "undionly" );
+
+       foreach ( $nics as $key => $value ) {
+               // skip the undi driver
+               if ( $key != "undi" ) {
+                       $new_nics[$key] = $value;
+               }
+       }
+
+       return ( array ( $new_nics, $roms ) );
 }
 
 ////
@@ -198,12 +206,8 @@ function default_flags ()
         $type  = $props["type"];
 
         // Fields like headers have no "value" property
-        $value = isset ( $props["value"] ) ? $props['value'] : '';
-
-        if ( $value != '' ) {
-            if ( $type != "on/off" || $value == "on" ) {
-                $flags[$flag] = $props["value"];
-            }
+        if ( isset ( $props["value"] ) ) {
+            $flags[$flag] = $props["value"];
         }
     }
     return $flags;
@@ -238,13 +242,12 @@ function pprint_flag ( $key, $value )
     } else if ( $type == "on/off" && $value != "on" ) {
         $s = "#undef $key";
     } else if ( $type == "string" ) {
-        if ( $value != "" ) $s = ( "#define $key \"" . cleanstring($value) . "\"" );
+        $s = ( "#define $key \"" . cleanstring ( $value ) . "\"" );
     } else if ($type == "qstring" ) {
-        if ( $value != "" ) $s = ( "#define $key \\\"" . cleanstring($value) . "\\\"" );
+        $s = ( "#define $key \\\"" . cleanstring ( $value ) . "\\\"" );
     } else {
         $s = "#define $key " . cleanstring($value);
     }
-    // print ( $s . "<br>" );
     
     return $s;
 }
@@ -253,9 +256,8 @@ function pprint_flags ( $flags )
 {
     $s = "";
        foreach ( $flags as $key => $value ) {
-        $f = pprint_flag($key, $value);
-        if ($f != "" )
-            $s .= $f . " ";
+        $f = pprint_flag ( $key, $value );
+        $s .= $f . " ";
     }
     return $s;
 }
@@ -295,11 +297,11 @@ function echo_flags ( $flags )
 
             } else if ($type == "on/off" ) {
 
-                echo "<td>", checkbox ( $flag, $value ), "</td><td><b>$flag</b></td>";
+                echo "<td>", checkbox ( $flag, $value ), "</td><td><strong>$flag</strong></td>";
 
             } else {   // don't display checkbox for non-on/off flags
 
-                echo "<td>&nbsp;</td><td><b>$flag: </b>";
+                echo "<td>&nbsp;</td><td><strong>$flag: </strong>";
 
                 if ($type == "choice" ) {
                     $options = $props["options"];
@@ -307,24 +309,26 @@ function echo_flags ( $flags )
 
                 } else {
 
-                    echo textbox($flag, $value, ($type == "integer" ? 5 : 25));
+                    echo textbox($flag, $value, ($type == "integer" ? 7 : 25));
                 }
                 echo "</td>";
             }
             echo "</tr>\n";
 
             if ( $type != "header" ) {
-                echo "<tr><td>&nbsp;</td>";
-                echo "<td>\n";
-                include_once "doc/$flag.html";
-                echo "\n</td></tr>\n";
+                               echo "<tr><td>&nbsp;</td>";
+                               echo "<td>\n";
+                               if ( is_file ( "doc/$flag.html" ) ) { 
+                                       include_once "doc/$flag.html";
+                               }
+                               echo "\n</td></tr>\n";
             }
         }
     }
     echo "</table>";
 }
 
-function get_cfgsecs ()
+function get_flag_cfgsecs ()
 {
     global $flag_table;
     $cfgsecs = array ();
@@ -349,7 +353,7 @@ function mktempcopy ( $src, $dst )
     }
 
     $dir = tempnam ( $dst, "MDCROM" );
-    unlink( $dir );
+    unlink ( $dir );
 
     exec ( "/bin/cp -a '$src' '$dir' 2>&1", $cpytxt, $status );  
     if ( $status != 0 ) {
@@ -362,7 +366,7 @@ function write_gpxe_config_files ( $config_dir, $flags )
 {
     global $flag_table;
 
-    $cfgsecs = get_cfgsecs ();
+    $cfgsecs = get_flag_cfgsecs ();
 
     foreach ( $cfgsecs as $cfgsec ) {
 
@@ -414,7 +418,7 @@ function deltempdir ( $dir )
                 deltempdir ( "$dir/$file" );
             }
         } else {
-            unlink( "$dir/$file" );
+            unlink ( "$dir/$file" );
         }
     }
     closedir ( $dp );