fix harmless code leaking out onto the page, hehe, closes: Bug#3253, also remove...

diff --git a/httemplate/misc/catchall.cgi b/httemplate/misc/catchall.cgi
index 2094494..240f34d 100755 (executable)
--- a/httemplate/misc/catchall.cgi
+++ b/httemplate/misc/catchall.cgi
@@ -4,10 +4,9 @@
 
 <FORM ACTION="<%$p1%>process/catchall.cgi" METHOD=POST>
 
-<PRE>
-
 <INPUT TYPE="hidden" NAME="svcnum" VALUE="<% $svcnum |h %>">
 Service #<FONT SIZE=+1><B><% $svcnum ? $svcnum : ' (NEW)' |h %></B></FONT>
+<BR><BR>
 
 <INPUT TYPE="hidden" NAME="pkgnum" VALUE="<% $pkgnum |h %>">
 
@@ -23,8 +22,7 @@ Mail to <I>(anything)</I>@<B><% $domain |h %></B> forwards to <SELECT NAME="catc
     <OPTION<% $_ eq $catchall ? ' SELECTED' : '' %> VALUE="<% $_ %>"><% $email{$_} %>
 % }
 </SELECT>
-
-</PRE>
+<BR><BR>
 
 <INPUT TYPE="submit" VALUE="Submit">
 

diff --git a/httemplate/view/svc_domain.cgi b/httemplate/view/svc_domain.cgi
index a58d75e..d6565db 100755 (executable)
--- a/httemplate/view/svc_domain.cgi
+++ b/httemplate/view/svc_domain.cgi
@@ -10,10 +10,9 @@
 Service #<% $svcnum %>
 <BR>Service: <B><% $part_svc->svc %></B>
 <BR>Domain name: <B><% $domain %></B>
-<BR>Catch all email 
 % if ( $FS::CurrentUser::CurrentUser->access_right('Edit domain catchall') ) {
-    <BR>Catch all email<A HREF="<% ${p} %>misc/catchall.cgi?<% $svcnum %>">(change)</A>:
-} else {
+    <BR>Catch all email <A HREF="<% ${p} %>misc/catchall.cgi?<% $svcnum %>">(change)</A>:
+% } else {
     <BR>Catch all email:
 % }