use a few more control helpers for checkboxes

This commit is contained in:
Andrew Dolgov 2021-02-16 15:19:42 +03:00
parent af4b3e7df0
commit 1f5d81b77c
3 changed files with 8 additions and 16 deletions

View File

@ -649,10 +649,6 @@ class Feeds extends Handler_Protected {
print "Access denied.";
return;
}
$refetch_checked = isset($_REQUEST["force_refetch"]) ? "checked" : "";
$rehash_checked = isset($_REQUEST["force_rehash"]) ? "checked" : "";
?>
<!DOCTYPE html>
<html>
@ -712,11 +708,11 @@ class Feeds extends Handler_Protected {
</fieldset>
<fieldset>
<label class="checkbox"><input dojoType="dijit.form.CheckBox" type="checkbox" name="force_refetch" value="1" <?= $refetch_checked ?>> Force refetch</label>
<label class="checkbox"><?= \Controls\checkbox_tag("force_refetch", isset($_REQUEST["force_refetch"])) ?> Force refetch</label>
</fieldset>
<fieldset class="narrow">
<label class="checkbox"><input dojoType="dijit.form.CheckBox" type="checkbox" name="force_rehash" value="1" <?= $rehash_checked ?>> Force rehash</label>
<label class="checkbox"><?= \Controls\checkbox_tag("force_rehash", isset($_REQUEST["force_rehash"])) ?> Force rehash</label>
</fieldset>
<?= \Controls\submit_tag("Continue") ?>

View File

@ -663,7 +663,7 @@ class Pref_Feeds extends Handler_Protected {
print "<fieldset>";
print "<input dojoType='dijit.form.TextBox' id='feedEditDlg_login'
print "<input dojoType='dijit.form.TextBox'
placeHolder='".__("Login")."'
autocomplete='new-password'
name='auth_login' value=\"$auth_login\">";
@ -675,10 +675,6 @@ class Pref_Feeds extends Handler_Protected {
placeHolder='".__("Password")."'
value=\"$auth_pass\">";
print "<div dojoType='dijit.Tooltip' connectId='feedEditDlg_login' position='below'>
".__('<b>Hint:</b> you need to fill in your login information if your feed requires authentication, except for Twitter feeds.')."
</div>";
print "</fieldset>";
print "</section></div>";

View File

@ -775,12 +775,12 @@ class Pref_Filters extends Handler_Protected {
$reg_exp = htmlspecialchars($rule["reg_exp"]);
$filter_type = $rule["filter_type"];
$feed_id = $rule["feed_id"];
$inverse_checked = isset($rule["inverse"]) ? "checked" : "";
$inverse_checked = !empty($rule["inverse"]);
} else {
$reg_exp = "";
$filter_type = 1;
$feed_id = ["0"];
$inverse_checked = "";
$inverse_checked = false;
}
print "<form name='filter_new_rule_form' id='filter_new_rule_form' onsubmit='return false;'>";
@ -808,9 +808,9 @@ class Pref_Filters extends Handler_Protected {
print "<div dojoType='dijit.Tooltip' id='filterDlg_regExp_tip' connectId='filterDlg_regExp' position='below'></div>";
print "<fieldset>";
print "<label class='checkbox'><input id='filterDlg_inverse' dojoType='dijit.form.CheckBox'
name='inverse' $inverse_checked/> ".
__("Inverse regular expression matching")."</label>";
print "<label class='checkbox'>".
\Controls\checkbox_tag("inverse", $inverse_checked) .
__("Inverse regular expression matching")."</label>";
print "</fieldset>";
print "<fieldset>";