From 33678d2e4b7e594b0a6c1752e0a8ab20c76252d2 Mon Sep 17 00:00:00 2001
From: oleonard <oleonard>
Date: Tue, 30 Dec 2003 16:16:26 +0000
Subject: [PATCH] Changes to generated HTML to allow proper validation (&amp;,
 quoted attributes, etc.)

---
 admin/marctagstructure.pl | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/admin/marctagstructure.pl b/admin/marctagstructure.pl
index 5f25f808c8..c6a1ad95d2 100755
--- a/admin/marctagstructure.pl
+++ b/admin/marctagstructure.pl
@@ -105,10 +105,10 @@ if ($op eq 'add_form') {
 
 	if ($searchfield) {
 		$template->param(action => "Modify tag",
-								searchfield => "<input type=hidden name=tagfield value='$searchfield'>$searchfield");
+								searchfield => "<input type=\"hidden\" name=\"tagfield\" value=\"$searchfield\" />$searchfield");
 	} else {
 		$template->param(action => "Add tag",
-								searchfield => "<input type=text name=tagfield size=5 maxlength=3>");
+								searchfield => "<input type=\"text\" name=\"tagfield\" size=\"5\" maxlength=\"3\" />");
 	}
 	$template->param(liblibrarian => $data->{'liblibrarian'},
 							libopac => $data->{'libopac'},
@@ -165,7 +165,7 @@ if ($op eq 'add_form') {
 ################## DEFAULT ##################################
 } else { # DEFAULT
 	if  ($searchfield ne '') {
-		 $template->param(searchfield => "You Searched for <b>$searchfield<b><p>");
+		 $template->param(searchfield => "<p>You Searched for <strong>$searchfield<strong></p>");
 	}
 	my $env;
 	my ($count,$results)=StringSearch($env,$searchfield,'web');
@@ -184,8 +184,8 @@ if ($op eq 'add_form') {
 		$row_data{mandatory} = $results->[$i]{'mandatory'};
 		$row_data{authorised_value} = $results->[$i]{'authorised_value'};
 		$row_data{subfield_link} ="marc_subfields_structure.pl?tagfield=".$results->[$i]{'tagfield'};
-		$row_data{edit} = "$script_name?op=add_form&searchfield=".$results->[$i]{'tagfield'};
-		$row_data{delete} = "$script_name?op=delete_confirm&searchfield=".$results->[$i]{'tagfield'};
+		$row_data{edit} = "$script_name?op=add_form&amp;searchfield=".$results->[$i]{'tagfield'};
+		$row_data{delete} = "$script_name?op=delete_confirm&amp;searchfield=".$results->[$i]{'tagfield'};
 		$row_data{bgcolor} = $toggle;
 		push(@loop_data, \%row_data);
 	}
-- 
2.20.1