Yes, I did notice that, but the problem is whether you are going to use a boolean value or the "on" and "off" values (since currently it's mixed). I would prefer that boolean values be used for all such variables and the variables defined in the MySQL database changed accordingly. In that case something like the following would be used for each file:
newreply.php
newpost.php
sendprivate.php
mess_handler.php
mess_reply.php
if ($Preview == NULL) {
$Preview = $config[preview];
}
if ($Preview) {
$Pselected = "checked";
}
=======================================================
and in editdisplay.php:
// Find out if they preview or not
if ($Preview == NULL) {
$Preview = $config[preview];
}
if ($Preview) {
$previewon = "selected";
}
else {
$previewoff = "selected";
}
------------------------------------------------------------------------------------
<select name = "Preview" class="formboxes">
<option value = "0" $previewoff>$lang[TEXT_OFF]</option>
<option value = "1" $previewon>$lang[TEXT_ON]</option>
</select>