--- lib/PXElator/httpd.pm	2009/08/29 00:06:32	335
+++ lib/PXElator/httpd.pm	2009/08/29 17:03:35	347
@@ -239,26 +239,25 @@
 		}
 
 		if ( $ip && $ip ne $server::ip ) {
-			my $hostname = client::conf( $ip, 'hostname' => $param->{hostname} );
 
+			my @editable = ( qw/hostname config homepage/ );
+
+			client::conf( $ip, $_ => $param->{$_} ) foreach @editable;
+
+			my $conf = client::all_conf( $ip );
+			my $config = delete $conf->{config};
+
+			my $nmap = qq|<a href=/nmap?scan=$ip>nmap</a>|;
 			my @table = (
+				'ping' => ping::host($ip) ? qq|<span style="color:green">up</span> $nmap| : qq|<span style="color: red">down</span> <a href=/wol/$ip>wol</a> $nmap|,
 				'ip' => qq|<input type=text name=change_ip value="$ip" onChange="document.getElementById('old_ip').style.display = '';"><span id=old_ip style="display: none; color: #888;">old: $ip<span>|,
-				'hostname' => qq|<input type=text name=hostname value="$hostname">|,
-				'ping' => ping::host($ip) ? qq|<span style="color:green">up</span>| : qq|<a href=/wol/$ip title=wol>down</a>|,
+				'mac' => format::mac( delete $conf->{mac}, 'html' ),
+				'hostname' => qq|<input type=text name=hostname value="| . delete($conf->{hostname}) . qq|">|,
+				'config' => html::select( 'config', $config, config::available ),
+				html::conf( $ip, $conf, 'edit', @editable )
 			);
 
-			my $config;
-
-			if ( my $mac = client::mac_from_ip( $ip ) ) {
-				my $deploy = client::conf( $ip, 'deploy' => $param->{deploy} );
-				push @table, (
-					'mac' => format::mac( $mac => 'html' ),
-					'deploy' => html::select( 'deploy', $deploy, config::available )
-				);
-				if ( $config = config::for_ip( $ip ) ) {
-					$config = qq|<h2>config</h2>| . html::pre( $config );
-				}
-			}
+warn "XX conf ",dump( $conf );
 
 			print $client ok
 				, qq|<form method=get>|
@@ -267,9 +266,14 @@
 					<input type=submit name=action value=change>
 					<input type=submit name=action value=remove style="color: red">
 					</form>|
-				, $config
 				;
 
+			if ( $config ) {
+				if ( my $for_ip = config::for_ip( $ip ) ) {
+					print $client qq|<h2>config::for_ip</h2>| . html::pre( $for_ip );
+				}
+			}
+
 			if ( my $amt = client::conf( $ip, 'amt' ) ) {
 				print $client qq|<h2>AMT</h2>|, amt::info( $amt, $ip );
 			}
@@ -288,8 +292,8 @@
 
 			print $client ok
 				, qq|<h2>Clients on $server::ip</h2>|
-				, html::table( -6,
-					'ip', 'mac', 'dev', 'hostname', 'deploy', 'conf',
+				, html::table( -5,
+					'ip', 'mac', 'dev', 'hostname', 'conf',
 					map {
 						my $ip = $_;
 						my $conf = client::all_conf( $ip );
@@ -306,9 +310,8 @@
 							, format::mac( $mac => 'html' )
 							, $arp->{$mac}
 							, delete $conf->{hostname}
-							, delete $conf->{deploy}
-							, ( %$conf ? html::pre_dump( $conf ) : qq|<a href=/nmap?scan=$ip>nmap</a>| )
-						);
+							, html::conf( $ip, $conf, 'inline' )
+						)
 					} client::all_ips
 				)
 				;