From f5caccc8276252edcffa7fb3dc778cba7bc50d0a Mon Sep 17 00:00:00 2001
From: Evan Prodromou <evan@controlezvous.ca>
Date: Thu, 26 Jun 2008 14:27:05 -0400
Subject: [PATCH] show checkbox as checked or not

darcs-hash:20080626182705-34904-893d72f480c330beb2d8f871da0d2cd9b902ffbf.gz
---
 actions/imsettings.php | 6 ++++--
 lib/util.php           | 8 +++++---
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/actions/imsettings.php b/actions/imsettings.php
index a5e9be307e..18666e1708 100644
--- a/actions/imsettings.php
+++ b/actions/imsettings.php
@@ -79,9 +79,11 @@ class ImsettingsAction extends SettingsAction {
 		common_element('h2', NULL, _t('Preferences'));
 
 		common_checkbox('jabbernotify',
-		                _t('Send me notices through Jabber/GTalk.'));
+		                _t('Send me notices through Jabber/GTalk.'),
+		                $user->jabbernotify);
 		common_checkbox('updatefrompresence',
-		                _t('Post a notice when my Jabber/GTalk status changes.'));
+		                _t('Post a notice when my Jabber/GTalk status changes.'),
+		                $user->updatefrompresence);
 		common_submit('save', _t('Save'));
 
 		common_element_end('form');
diff --git a/lib/util.php b/lib/util.php
index a20f64c85f..e3c00b31d9 100644
--- a/lib/util.php
+++ b/lib/util.php
@@ -319,16 +319,18 @@ function common_input($id, $label, $value=NULL,$instructions=NULL) {
 	common_element_end('p');
 }
 
-function common_checkbox($id, $label, $instructions=NULL, $value='true')
+function common_checkbox($id, $label, $checked=false, $instructions=NULL, $value='true')
 {
 	common_element_start('p');
 	$attrs = array('name' => $id,
 				   'type' => 'checkbox',
-				   'id' => $id,
-				   'value' => $value);
+				   'id' => $id);
 	if ($value) {
 		$attrs['value'] = htmlspecialchars($value);
 	}
+	if ($checked) {
+		$attrs['checked'] = 'checked';
+	}
 	common_element('input', $attrs);
 	# XXX: use a <label>
 	common_text(' ');