Validated XHMTL for preferences

2 files changed, 21 insertions, 21 deletions

diff --git a/preferences.php b/preferences.php
index aed1b9b..821a25e 100644
--- a/preferences.php
+++ b/preferences.php
@@ -79,9 +79,9 @@ while ($file = readdir($dir_handle)) {
 	if (substr($file, -8) == ".inc.php") {
 		$language_tmp = urlencode(ucfirst(substr($file, 0, -8)));
 		if ($language_tmp == $cookie_language) {
-			$language_select .= "<option value=\"$language_tmp\" selected>$language_tmp</option>\n";
+			$language_select .= '<option value="'.$language_tmp.'" selected="selected">'.$language_tmp.'</option>';
 		} else {
-			$language_select .= "<option value=\"$language_tmp\">$language_tmp</option>\n";
+			$language_select .= '<option value="'.$language_tmp.'">'.$language_tmp.'</option>';
 		}
 	}
 }
@@ -91,16 +91,16 @@ closedir($dir_handle);
 $calendar_select = display_ical_list(availableCalendars($username, $password, $ALL_CALENDARS_COMBINED));
 
 // select for dayview
-$view_select 	= ($cookie_view == 'day') ? '<option value="day" selected>{L_DAY}</option>' : '<option value="day">{L_DAY}</option>';
-$view_select    .= ($cookie_view == 'week') ? '<option value="week" selected>{L_WEEK}</option>' : '<option value="week">{L_WEEK}</option>';
-$view_select    .= ($cookie_view == 'month') ? '<option value="month" selected>{L_MONTH}</option>' : '<option value="month">{L_MONTH}</option>';
+$view_select 	= ($cookie_view == 'day') ? '<option value="day" selected="selected">{L_DAY}</option>' : '<option value="day">{L_DAY}</option>';
+$view_select    .= ($cookie_view == 'week') ? '<option value="week" selected="selected">{L_WEEK}</option>' : '<option value="week">{L_WEEK}</option>';
+$view_select    .= ($cookie_view == 'month') ? '<option value="month" selected="selected">{L_MONTH}</option>' : '<option value="month">{L_MONTH}</option>';
 
 // select for time
 for ($i = 000; $i <= 1200; $i += 100) {
 	$s = sprintf("%04d", $i);
-	$time_select .= "<option value=\"$s\"";
+	$time_select .= '<option value="'.$s.'"';
 	if ($s == $cookie_time) {
-		$time_select .= " selected";
+		$time_select .= ' selected="selected"';
 	}
 	$time_select .= ">$s</option>\n";
 }
@@ -108,10 +108,10 @@ for ($i = 000; $i <= 1200; $i += 100) {
 // select for day of week
 $i=0;
 foreach ($daysofweek_lang as $daysofweek) {
-	if ($startdays[$i] == "$cookie_startday") {
-		$startday_select .= "<option value=\"$startdays[$i]\" selected>$daysofweek</option>\n";
+	if ($startdays[$i] == $cookie_startday) {
+		$startday_select .= '<option value="'.$startdays[$i].'" selected="selected">'.$daysofweek.'</option>';
 	} else {
-		$startday_select .= "<option value=\"$startdays[$i]\">$daysofweek</option>\n";
+		$startday_select .= '<option value="'.$startdays[$i].'">'.$daysofweek.'</option>';
 	}
 	$i++;
 }
@@ -121,7 +121,7 @@ while ($file = readdir($dir_handle)) {
 	if (($file != ".") && ($file != "..") && ($file != "CVS")) {
 		if (!is_file($file)) {
 			$file_disp = ucfirst($file);
-			$style_select .= ($file == "$cookie_style") ? "<option value=\"$file\" selected>$file_disp</option>\n" : "<option value=\"$file\">$file_disp</option>\n";
+			$style_select .= ($file == "$cookie_style") ? "<option value=\"$file\" selected=\"selected\">$file_disp</option>\n" : "<option value=\"$file\">$file_disp</option>\n";
 		}
 	}
 }
diff --git a/templates/default/preferences.tpl b/templates/default/preferences.tpl
index 1cac501..0140b66 100644
--- a/templates/default/preferences.tpl
+++ b/templates/default/preferences.tpl
@@ -35,48 +35,48 @@
 				<!-- switch message off -->
 				<tr>
 					<td valign="top" align="left">
-					<form action="preferences.php?action=setcookie" METHOD="post">
+					<form action="preferences.php?action=setcookie" method="post">
 					<table border="0" width="100%" cellspacing="2" cellpadding="2" align="center">
 						<tr align="left" valign="top">
-							<td width="80%" nowrap>{L_SELECT_LANG}:</td>
+							<td width="80%" nowrap="nowrap">{L_SELECT_LANG}:</td>
 							<td width="10%"><img src="images/spacer.gif" alt=" " width="20" height="1" border="0" /></td>
 							<td width="10%"><select name="cookie_language" class="query_style">{LANGUAGE_SELECT}</select></td>
 						</tr>
 						<tr align="left" valign="top">
-							<td nowrap>{L_SELECT_CAL}:</td>
+							<td nowrap="nowrap">{L_SELECT_CAL}:</td>
 							<td><img src="images/spacer.gif" alt=" " width="20" height="1" border="0" /></td>
 							<td><select name="cookie_calendar" class="query_style">{CALENDAR_SELECT}</select></td>
 						</tr>
 						<tr align="left" valign="top">
-							<td nowrap>{L_SELECT_VIEW}:</td>
+							<td nowrap="nowrap">{L_SELECT_VIEW}:</td>
 							<td><img src="images/spacer.gif" alt=" " width="20" height="1" border="0" /></td>
 							<td><select name="cookie_view" class="query_style">{VIEW_SELECT}</select></td>
 						</tr>
 						<tr align="left" valign="top">
-							<td nowrap>{L_SELECT_TIME}:</td>
+							<td nowrap="nowrap">{L_SELECT_TIME}:</td>
 							<td><img src="images/spacer.gif" alt=" " width="20" height="1" border="0" /></td>
 							<td><select name="cookie_time" class="query_style">{TIME_SELECT}</select></td>
 						</tr>
 						<tr align="left" valign="top">
-							<td nowrap>{L_SELECT_DAY}:</td>
+							<td nowrap="nowrap">{L_SELECT_DAY}:</td>
 							<td><img src="images/spacer.gif" alt=" " width="20" height="1" border="0" /></td>
 							<td><select name="cookie_startday" class="query_style">{STARTDAY_SELECT}</select></td>
 						</tr>
 						<tr align="left" valign="top">
-							<td nowrap>{L_SELECT_STYLE}:</td>
+							<td nowrap="nowrap">{L_SELECT_STYLE}:</td>
 							<td><img src="images/spacer.gif" alt=" " width="20" height="1" border="0" /></td>
 							<td><select name="cookie_style" class="query_style">{STYLE_SELECT}</select></td>
 						</tr>
 						<!-- switch cookie_already_set on -->
 						<tr align="left" valign="top">
-							<td nowrap>{L_UNSET_PREFS}:</td>
+							<td nowrap="nowrap">{L_UNSET_PREFS}:</td>
 							<td><img src="images/spacer.gif" alt=" " width="20" height="1" border="0" /></td>
-							<td><INPUT TYPE="checkbox" NAME="unset" VALUE="true"></td>
+							<td><input type="checkbox" name="unset" value="true" /></td>
 						</tr>
 						<!-- switch cookie_already_set off -->
 						<!-- switch cookie_not_set on -->
 						<tr align="left" valign="top">
-							<td nowrap>&nbsp;</td>
+							<td nowrap="nowrap">&nbsp;</td>
 							<td><img src="images/spacer.gif" alt=" " width="20" height="1" border="0" /></td>
 							<td><input type="submit" name="set" value="{L_SET_PREFS}" /></td>
 						</tr>