X-Git-Url: http://git.ipfire.org/?p=people%2Fpmueller%2Fipfire-2.x.git;a=blobdiff_plain;f=html%2Fcgi-bin%2Foutgoingfw.cgi;h=295bde477c4a9069fe986116642bf0d94e0a39c9;hp=ed87e0d8a3ff813787d220b69350b69056ff731e;hb=98c474a4aa6802cc07088eafb60eb47a8b315d10;hpb=c19bb5d9f62417d884e8af4181158cdafb8ff620

diff --git a/html/cgi-bin/outgoingfw.cgi b/html/cgi-bin/outgoingfw.cgi
index ed87e0d8a3..295bde477c 100644
--- a/html/cgi-bin/outgoingfw.cgi
+++ b/html/cgi-bin/outgoingfw.cgi
@@ -2,7 +2,7 @@
 ###############################################################################
 #                                                                             #
 # IPFire.org - A linux based firewall                                         #
-# Copyright (C) 2010  Michael Tremer & Christian Schmidt                      #
+# Copyright (C) 2005-2010  IPFire Team                                        #
 #                                                                             #
 # This program is free software: you can redistribute it and/or modify        #
 # it under the terms of the GNU General Public License as published by        #
@@ -41,6 +41,7 @@ my @p2ps = ();
 my @p2pline = ();
 
 my $configfile = "/var/ipfire/outgoing/rules";
+my $configpath = "/var/ipfire/outgoing/groups/";
 my $p2pfile = "/var/ipfire/outgoing/p2protocols";
 my $servicefile = "/var/ipfire/outgoing/defaultservices";
 
@@ -145,7 +146,7 @@ if ( $outfwsettings{'TIME_MON'} eq "" &&
 		$outfwsettings{'TIME_SUN'} = "on";
 	 }
 
-&Header::openpage('Ausgehende Firewall', 1, '');
+&Header::openpage($Lang::tr{'outgoing firewall'}, 1, '');
 &Header::openbigbox('100%', 'left', '', $errormessage);
 
 ############################################################################################################################
@@ -309,6 +310,11 @@ if ($outfwsettings{'ACTION'} eq $Lang::tr{'delete'})
 if ($outfwsettings{'ACTION'} eq $Lang::tr{'add'})
 {
 	if ( $outfwsettings{'VALID'} eq 'yes' ) {
+
+		if ( $outfwsettings{'SNET'} eq "all" ) {
+			$outfwsettings{'SIP'} ="";
+			$outfwsettings{'SMAC'}="";
+		}
 		open( FILE, ">> $configfile" ) or die "Unable to write $configfile";
 		print FILE <<END
 $outfwsettings{'STATE'};$outfwsettings{'ENABLED'};$outfwsettings{'SNET'};$outfwsettings{'PROT'};$outfwsettings{'NAME'};$outfwsettings{'SIP'};$outfwsettings{'SMAC'};$outfwsettings{'DIP'};$outfwsettings{'DPORT'};$outfwsettings{'LOG'};$outfwsettings{'TIME_MON'};$outfwsettings{'TIME_TUE'};$outfwsettings{'TIME_WED'};$outfwsettings{'TIME_THU'};$outfwsettings{'TIME_FRI'};$outfwsettings{'TIME_SAT'};$outfwsettings{'TIME_SUN'};$outfwsettings{'TIME_FROM'};$outfwsettings{'TIME_TO'};
@@ -320,7 +326,7 @@ END
 		$outfwsettings{'ACTION'} = 'Add rule';
 	}
 }
-if ($outfwsettings{'ACTION'} eq 'Add rule')
+if ($outfwsettings{'ACTION'} eq $Lang::tr{'Add Rule'})
 {
 	&addrule();
 	exit
@@ -342,7 +348,7 @@ if ($outfwsettings{'POLICY'} ne 'MODE0'){
 	&Header::openbox('100%', 'center', 'Rules');
 		print <<END
 	<form method='post' action='$ENV{'SCRIPT_NAME'}'>
-		<input type='submit' name='ACTION' value='Add rule' />
+		<input type='submit' name='ACTION' value='$Lang::tr{'Add Rule'}' />
 	</form>
 END
 ;
@@ -402,8 +408,12 @@ END
 				if ($outfwsettings{'STATE'} eq 'DENY'){ $outfwsettings{'DISPLAY_STATE'} = "<img src='/images/stock_stop.png' alt='DENY' />"; }
 				if ($outfwsettings{'STATE'} eq 'ALLOW'){ $outfwsettings{'DISPLAY_STATE'} = "<img src='/images/stock_ok.png' alt='ALLOW' />"; }
 				if ((($outfwsettings{'POLICY'} eq 'MODE1') && ($outfwsettings{'STATE'} eq 'ALLOW')) || (($outfwsettings{'POLICY'} eq 'MODE2') && ($outfwsettings{'STATE'} eq 'DENY'))){
-			  		print <<END
-					<tr bgcolor='$color{'color20'}'>
+				if ( $outfwsettings{'ENABLED'} eq "on" ){
+					print "<tr bgcolor='$color{'color20'}'>";
+				} else {
+					print "<tr bgcolor='$color{'color18'}'>";
+				}
+					print <<END
 					    <td align='center'>$outfwsettings{'PROT'}
 					    <td align='center'>$outfwsettings{'SNET'}
 					    <td align='center'>$outfwsettings{'DISPLAY_DIP'}:$outfwsettings{'DISPLAY_DPORT'}
@@ -461,17 +471,25 @@ END
 END
 ;
 					if (($outfwsettings{'SIP'}) || ($outfwsettings{'SMAC'})) {
-						unless ($outfwsettings{'SIP'}) { $outfwsettings{'DISPLAY_SIP'} = 'ALL'; } else { $outfwsettings{'DISPLAY_SIP'} = $outfwsettings{'SIP'}; }
-						unless ($outfwsettings{'SMAC'}) { $outfwsettings{'DISPLAY_SMAC'} = 'ALL'; } else { $outfwsettings{'DISPLAY_SMAC'} = $outfwsettings{'SMAC'}; }
-						print <<END
-						<tr><td width='14%' align='right'>$Lang::tr{'source ip'}:
-						    <td width='14%' align='left'>$outfwsettings{'DISPLAY_SIP'}
-						    <td width='44%' colspan='2' align='center'>
-END
-;
+
+						unless ($outfwsettings{'SIP'}) {
+							$outfwsettings{'DISPLAY_SIP'} = 'ALL';
+						} else {
+							$outfwsettings{'DISPLAY_SIP'} = $outfwsettings{'SIP'};
+						}
+
+						unless ($outfwsettings{'SMAC'}) {
+							$outfwsettings{'DISPLAY_SMAC'} = 'ALL';
+							print "<tr><td /><td align='left'>$Lang::tr{'source ip or net'}: </td>";
+							print "<td align='left' colspan='2'>$outfwsettings{'DISPLAY_SIP'}</td>";
+						} else {
+							$outfwsettings{'DISPLAY_SMAC'} = $outfwsettings{'SMAC'};
+							print "<tr><td /><td align='left'>$Lang::tr{'source'} $Lang::tr{'mac address'}: </td>";
+							print "<td align='left' colspan='2'>$outfwsettings{'DISPLAY_SMAC'}</td>";
+						}
 					}
 						print <<END
-						<tr><td width='14%' align='right'>$Lang::tr{'time'} - </td>
+						<tr><td width='14%' align='right'>$Lang::tr{'time'} -  </td>
 						    <td width='14%' align='left'>
 END
 ;
@@ -558,13 +576,13 @@ END
 		if ($p2pline[2] eq 'on') {
 			print <<END
 				<input type='hidden' name='ACTION' value='disable' />
-				<input type='image' name='submit' src='/images/stock_ok.png' alt='' />
+				<input type='image' name='submit' src='/images/stock_ok.png' alt='$Lang::tr{'outgoing firewall p2p allow'}' title='$Lang::tr{'outgoing firewall p2p allow'}'/>
 END
 ;
 		} else {
 			print <<END
 				<input type='hidden' name='ACTION' value='enable' />
-				<input type='image' name='submit' src='/images/stock_stop.png' alt='' />
+				<input type='image' name='submit' src='/images/stock_stop.png' alt='$Lang::tr{'outgoing firewall p2p deny'}' title='$Lang::tr{'outgoing firewall p2p deny'}' />
 END
 ;
 		}
@@ -575,7 +593,7 @@ END
 	}
 	print <<END
 	</table>
-  <br />$Lang::tr{'outgoingfw p2p description'}
+  <br />$Lang::tr{'outgoing firewall p2p description 1'} <img src='/images/stock_ok.png' align='absmiddle' alt='$Lang::tr{'outgoing firewall p2p deny'}'> $Lang::tr{'outgoing firewall p2p description 2'} <img src='/images/stock_stop.png' align='absmiddle' alt='$Lang::tr{'outgoing firewall p2p deny'}'> $Lang::tr{'outgoing firewall p2p description 3'}
 END
 ;
 	&Header::closebox();
@@ -585,9 +603,9 @@ END
 print <<END
 	<form method='post' action='$ENV{'SCRIPT_NAME'}'>
 	<table width='100%'>
-		<tr><td width='10%' align='left'><b>$Lang::tr{'mode'} 0:</b><td width='90%' align='left' colspan='2'>$Lang::tr{'outgoingfw mode0'}</td></tr>
-		<tr><td width='10%' align='left'><b>$Lang::tr{'mode'} 1:</b><td width='90%' align='left' colspan='2'>$Lang::tr{'outgoingfw mode1'}</td></tr>
-		<tr><td width='10%' align='left'><b>$Lang::tr{'mode'} 2:</b><td width='90%' align='left' colspan='2'>$Lang::tr{'outgoingfw mode2'}</td></tr>
+		<tr><td width='10%' align='left'><b>$Lang::tr{'mode'} 0:</b><td width='90%' align='left' colspan='2'>$Lang::tr{'outgoing firewall mode0'}</td></tr>
+		<tr><td width='10%' align='left'><b>$Lang::tr{'mode'} 1:</b><td width='90%' align='left' colspan='2'>$Lang::tr{'outgoing firewall mode1'}</td></tr>
+		<tr><td width='10%' align='left'><b>$Lang::tr{'mode'} 2:</b><td width='90%' align='left' colspan='2'>$Lang::tr{'outgoing firewall mode2'}</td></tr>
 		<tr><td colspan='3'><hr /></td></tr>
 		<tr><td width='10%' align='left'>	<select name='POLICY' style="width: 85px"><option value='MODE0' $selected{'POLICY'}{'MODE0'}>$Lang::tr{'mode'} 0</option><option value='MODE1' $selected{'POLICY'}{'MODE1'}>$Lang::tr{'mode'} 1</option><option value='MODE2' $selected{'POLICY'}{'MODE2'}>$Lang::tr{'mode'} 2</option></select>
 		    <td width='45%' align='left'><input type='submit' name='ACTION' value=$Lang::tr{'save'} />
@@ -596,7 +614,7 @@ END
 ;
 	if ($outfwsettings{'POLICY'} ne 'MODE0') {
 		print <<END
-		    $Lang::tr{'outgoingfw reset'}: <input type='submit' name='ACTION' value=$Lang::tr{'reset'} />
+		    $Lang::tr{'outgoing firewall reset'}: <input type='submit' name='ACTION' value=$Lang::tr{'reset'} />
 END
 ;
 	}
@@ -607,89 +625,144 @@ END
 ;
 &Header::closebox();
 
-&Header::closebigbox();
-&Header::closepage();
-
 ############################################################################################################################
 ############################################################################################################################
 
 sub addrule
 {
-	&Header::openbox('100%', 'center', 'Rules hinzufuegen');
+	&Header::openbox('100%', 'center', $Lang::tr{'Add Rule'});
 	if ($outfwsettings{'EDIT'} eq 'no') { $selected{'ENABLED'} = 'checked'; }
 	$selected{'TIME_FROM'}{$outfwsettings{'TIME_FROM'}} = "selected='selected'";
 	$selected{'TIME_TO'}{$outfwsettings{'TIME_TO'}} = "selected='selected'";
-	print <<END
+print <<END
 	<form method='post' action='$ENV{'SCRIPT_NAME'}'>
 	<table width='80%'>
-		<tr><td width='20%' align='right'>$Lang::tr{'description'}: <img src='/blob.gif' />
-		    <td width='30%' align='left'><input type='text' name='NAME' maxlength='30' value='$outfwsettings{'NAME'}' />
-		    <td width='20%' align='right' colspan='2'>$Lang::tr{'active'}:
-		    <td width='30%' align='left' colspan='2'><input type='checkbox' name='ENABLED' $selected{'ENABLED'} />
-		<tr><td width='20%' align='right'>$Lang::tr{'protocol'}:
-                   <td width='30%' align='left'>
-                       <select name='PROT'>
-                               <option value='all' $selected{'PROT'}{'all'}>All</option>
-                               <option value='tcp' $selected{'PROT'}{'tcp'}>TCP</option>
-                               <option value='udp' $selected{'PROT'}{'udp'}>UDP</option>
-                               <option value='gre' $selected{'PROT'}{'gre'}>GRE</option>
-                               <option value='esp' $selected{'PROT'}{'esp'}>ESP</option>
-                               <option value='tcp&udp' $selected{'PROT'}{'tcp&udp'}>TCP & UDP</option>
-                       </select>
-		    <td width='20%' align='right' colspan='2'>$Lang::tr{'policy'}:
-		    <td width='30%' align='left' colspan='2'>
+		<tr>
+			<td width='20%' align='right'>$Lang::tr{'description'}: <img src='/blob.gif' /></td>
+			<td width='30%' align='left'><input type='text' name='NAME' maxlength='30' value='$outfwsettings{'NAME'}' /></td>
+			<td width='20%' align='right' colspan='2'>$Lang::tr{'active'}:</td>
+			<td width='30%' align='left' colspan='2'><input type='checkbox' name='ENABLED' $selected{'ENABLED'} /></td>
+		</tr>
+		<tr>
+			<td width='20%' align='right'>$Lang::tr{'protocol'}</td>
+			<td width='30%' align='left'>
+				<select name='PROT'>
+					<option value='all' $selected{'PROT'}{'all'}>All</option>
+					<option value='tcp' $selected{'PROT'}{'tcp'}>TCP</option>
+					<option value='udp' $selected{'PROT'}{'udp'}>UDP</option>
+					<option value='gre' $selected{'PROT'}{'gre'}>GRE</option>
+					<option value='esp' $selected{'PROT'}{'esp'}>ESP</option>
+				</select>
+			</td>
+			<td width='20%' align='right' colspan='2'>$Lang::tr{'policy'}:</td>
+			<td width='30%' align='left' colspan='2'>
 END
 ;
 	if ($outfwsettings{'POLICY'} eq 'MODE1'){
-		print "\t\t\tALLOW<input type='hidden' name='STATE' value='ALLOW' />\n";
+		print "\t\t\t\tALLOW<input type='hidden' name='STATE' value='ALLOW' />\n";
 	} elsif ($outfwsettings{'POLICY'} eq 'MODE2'){
-		print "\t\t\tDENY<input type='hidden' name='STATE' value='DENY' />\n";
+		print "\t\t\t\tDENY<input type='hidden' name='STATE' value='DENY' />\n";
 	}
 	print <<END
-		<tr><td width='20%' align='right'>$Lang::tr{'source net'}:
-		    <td width='30%' align='left'><select name='SNET'>
-			<option value='all' $selected{'SNET'}{'ALL'}>$Lang::tr{'all'}</option>
-			<option value='ip' $selected{'SNET'}{'ip'}>$Lang::tr{'source ip'}</option>
-			<option value='red' $selected{'SNET'}{'red'}>$Lang::tr{'red'} IP</option>
-			<option value='green' $selected{'SNET'}{'green'}>$Lang::tr{'green'}</option>
+			</td>
+		</tr>
+		<tr>
+			<td width='20%' align='right'>$Lang::tr{'source'}:</td>
+			<td width='30%' align='left'>
+				<select name='SNET'>
+					<optgroup label='---'>
+						<option value='all' $selected{'SNET'}{'ALL'}>$Lang::tr{'all'}</option>
+					<optgroup label='$Lang::tr{'mac address'}'>
+						<option value='mac' $selected{'SNET'}{'mac'}>$Lang::tr{'source'} $Lang::tr{'mac address'}</option>
+					</optgroup>
+					<optgroup label='$Lang::tr{'ip address'}'>
+						<option value='ip' $selected{'SNET'}{'ip'}>$Lang::tr{'source ip or net'}</option>
+						<option value='red' $selected{'SNET'}{'red'}>$Lang::tr{'red'} IP</option>
+					</optgroup>
+					<optgroup label='$Lang::tr{'alt vpn'}'>
+						<option value='ovpn' $selected{'SNET'}{'ovpn'}>OpenVPN $Lang::tr{'interface'}</option>
+					</optgroup>
+					<optgroup label='$Lang::tr{'network'}'>
+						<option value='green' $selected{'SNET'}{'green'}>$Lang::tr{'green'}</option>
 END
 ;
 	if (&Header::blue_used()){
-		print "\t\t\t<option value='blue' $selected{'SNET'}{'blue'}>$Lang::tr{'wireless'}</option>\n";
+		print "\t\t\t\t\t<option value='blue' $selected{'SNET'}{'blue'}>$Lang::tr{'wireless'}</option>\n";
 	}
 	if (&Header::orange_used()){
-		print "\t\t\t<option value='orange' $selected{'SNET'}{'orange'}>$Lang::tr{'dmz'}</option>\n";
+		print "\t\t\t\t\t<option value='orange' $selected{'SNET'}{'orange'}>$Lang::tr{'dmz'}</option>\n";
 	}
 	print <<END
-			</select>
-		    <td width='20%' align='right' colspan='2'>$Lang::tr{'source ip'}: <img src='/blob.gif' />
-		    <td width='30%' align='left' colspan='2'><input type='text' name='SIP' maxlength='15' value='$outfwsettings{'SIP'}' />
-		<tr><td width='20%' align='right'>$Lang::tr{'logging'}:
-			<td width='30%' align='left'><select name='LOG'>
-										<option value='$Lang::tr{'active'}' $selected{'LOG'}{$Lang::tr{'active'}}>$Lang::tr{'active'}</option>
-										<option value='$Lang::tr{'inactive'}' $selected{'LOG'}{$Lang::tr{'inactive'}}>$Lang::tr{'inactive'}</option>
-										</select></td>";
-		    <td width='20%' align='right' colspan='2' />
-		    <td width='30%' align='left' colspan='2' />
-		<tr><td width='20%' align='right'>$Lang::tr{'destination ip'}: <img src='/blob.gif' />
-		    <td width='30%' align='left'><input type='text' name='DIP' maxlength='15' value='$outfwsettings{'DIP'}' />
-		    <td width='20%' align='right' colspan='2'>$Lang::tr{'destination port'}: <img src='/blob.gif' />
-		    <td width='30%' align='left' colspan='2'><input type='text' name='DPORT' maxlength='11' value='$outfwsettings{'DPORT'}' />
-		<tr><td width='20%' align='right'>$Lang::tr{'time'}:</td>
+					</optgroup>
+					<optgroup label='IP $Lang::tr{'advproxy NCSA group'}'>
+END
+;
+	my @ipgroups = qx(ls $configpath/ipgroups/);
+	foreach (sort @ipgroups){
+		chomp($_);
+		print "\t\t\t\t\t<option value='$_' $selected{'SNET'}{$_}>$_</option>\n";
+	}
+	print <<END
+					</optgroup>
+					<optgroup label='MAC $Lang::tr{'advproxy NCSA group'}'>
+END
+;
+	my @macgroups = qx(ls $configpath/macgroups/);
+	foreach (sort @macgroups){
+		chomp($_);
+		print "\t\t\t\t\t<option value='$_' $selected{'SNET'}{$_}>$_</option>\n";
+	}
+	print <<END
+					</optgroup>
+				</select>
+			</td>
+			<td align='right' colspan='4'><font color='red'>$Lang::tr{'outgoing firewall warning'}</font></td>
+		</tr>
+		<tr>
+			<td align='right' colspan='4' >$Lang::tr{'source ip or net'}<img src='/blob.gif' /></td>
+			<td align='left' colspan='4' ><input type='text' name='SIP' value='$outfwsettings{'SIP'}' /></td>
+		</tr>
+		<tr>
+			<td align='right' colspan='4' >$Lang::tr{'source'} $Lang::tr{'mac address'}: <img src='/blob.gif' />
+			<td align='left' colspan='4' ><input type='text' name='SMAC' maxlength='23' value='$outfwsettings{'SMAC'}' />
+		</tr>
+		<tr>
+			<td width='20%' align='right'>$Lang::tr{'logging'}:</td>
+			<td width='30%' align='left'>
+				<select name='LOG'>
+					<option value='$Lang::tr{'active'}' $selected{'LOG'}{$Lang::tr{'active'}}>$Lang::tr{'active'}</option>
+					<option value='$Lang::tr{'inactive'}' $selected{'LOG'}{$Lang::tr{'inactive'}}>$Lang::tr{'inactive'}</option>
+				</select>
+			</td>
+			<td width='20%' align='right' colspan='2' />
+			<td width='30%' align='left' colspan='2' />
+		<tr>
+			<td width='20%' align='right'>$Lang::tr{'destination ip or net'}: <img src='/blob.gif' /></td>
+			<td width='30%' align='left'><input type='text' name='DIP'  value='$outfwsettings{'DIP'}' /></td>
+			<td width='20%' align='right' colspan='2'>$Lang::tr{'destination port'}(s) <img src='/blob.gif' /></td>
+			<td width='30%' align='left' colspan='2'><input type='text' name='DPORT' value='$outfwsettings{'DPORT'}' /></td>
+		</tr>
+		<tr>
+			<td width='20%' align='right'>$Lang::tr{'time'}:</td>
 			<td width='30%' align='left'>$Lang::tr{'advproxy monday'} $Lang::tr{'advproxy tuesday'} $Lang::tr{'advproxy wednesday'} $Lang::tr{'advproxy thursday'} $Lang::tr{'advproxy friday'} $Lang::tr{'advproxy saturday'} $Lang::tr{'advproxy sunday'}</td>
 			<td width='20%' align='right' colspan='2' />
 			<td width='15%' align='left'>$Lang::tr{'advproxy from'}</td>
-			<td width='15%' align='left'>$Lang::tr{'advproxy to'}</td></tr>
-		<tr><td width='20%' align='right'></td>
-			<td width='30%' align='left'><input type='checkbox' name='TIME_MON' $checked{'TIME_MON'}{'on'} />
-										 <input type='checkbox' name='TIME_TUE' $checked{'TIME_TUE'}{'on'} />
-										 <input type='checkbox' name='TIME_WED' $checked{'TIME_WED'}{'on'} />
-										 <input type='checkbox' name='TIME_THU' $checked{'TIME_THU'}{'on'} />
-										 <input type='checkbox' name='TIME_FRI' $checked{'TIME_FRI'}{'on'} />
-										 <input type='checkbox' name='TIME_SAT' $checked{'TIME_SAT'}{'on'} />
-										 <input type='checkbox' name='TIME_SUN' $checked{'TIME_SUN'}{'on'} /></td>
+			<td width='15%' align='left'>$Lang::tr{'advproxy to'}</td>
+		</tr>
+		<tr>
+			<td width='20%' align='right'></td>
+			<td width='30%' align='left'>
+				<input type='checkbox' name='TIME_MON' $checked{'TIME_MON'}{'on'} />
+				<input type='checkbox' name='TIME_TUE' $checked{'TIME_TUE'}{'on'} />
+				<input type='checkbox' name='TIME_WED' $checked{'TIME_WED'}{'on'} />
+				<input type='checkbox' name='TIME_THU' $checked{'TIME_THU'}{'on'} />
+				<input type='checkbox' name='TIME_FRI' $checked{'TIME_FRI'}{'on'} />
+				<input type='checkbox' name='TIME_SAT' $checked{'TIME_SAT'}{'on'} />
+				<input type='checkbox' name='TIME_SUN' $checked{'TIME_SUN'}{'on'} />
+			</td>
 			<td width='20%' align='right' colspan='2' />
-			<td width='15%' align='left'><select name='TIME_FROM'>
+			<td width='15%' align='left'>
+				<select name='TIME_FROM'>
 END
 ;
 for (my $i=0;$i<=23;$i++) {
@@ -697,11 +770,12 @@ for (my $i=0;$i<=23;$i++) {
 	for (my $j=0;$j<=45;$j+=15) {
 		$j = sprintf("%02s",$j);
 		my $time = $i.":".$j;
-		print "<option $selected{'TIME_FROM'}{$time}>$i:$j</option>\n";
+		print "\t\t\t\t\t<option $selected{'TIME_FROM'}{$time}>$i:$j</option>\n";
 	}
 }
 print <<END	
-			</select></td>
+				</select>
+			</td>
 			<td width='15%' align='left'><select name='TIME_TO'>
 END
 ;
@@ -710,14 +784,19 @@ for (my $i=0;$i<=23;$i++) {
 	for (my $j=0;$j<=45;$j+=15) {
 		$j = sprintf("%02s",$j);
 		my $time = $i.":".$j;
-		print "<option $selected{'TIME_TO'}{$time}>$i:$j</option>\n";
+		print "\t\t\t\t\t<option $selected{'TIME_TO'}{$time}>$i:$j</option>\n";
 	}
 }
 print <<END	
-			</select></td></tr>
-		<tr><td colspan='6'>
-		<tr><td width='40%' align='right' colspan='2'><img src='/blob.gif' />$Lang::tr{'this field may be blank'}
-		    <td width='60%' align='left' colspan='4'><input type='submit' name='ACTION' value=$Lang::tr{'add'} />
+				</select>
+			</td>
+		</tr>
+		<tr>
+			<td colspan='6' />
+		<tr>
+		<tr>
+			<td width='40%' align='right' colspan='2'><img src='/blob.gif' />$Lang::tr{'this field may be blank'}</td>
+			<td width='60%' align='left' colspan='4'><input type='submit' name='ACTION' value=$Lang::tr{'add'} /></td>
 	</table></form>
 END
 ;
@@ -764,4 +843,7 @@ END
 	print "</table>";
 	&Header::closebox();
   }
-}
\ No newline at end of file
+}
+
+&Header::closebigbox();
+&Header::closepage();