diff options
| author | Sarven Capadisli <csarven@status.net> | 2010-03-24 21:34:53 +0100 |
|---|---|---|
| committer | Sarven Capadisli <csarven@status.net> | 2010-03-25 22:20:20 +0100 |
| commit | b4a4438abd5780fbbdd70a02a4772db2473ec8a1 (patch) | |
| tree | 4630c14617dd7f48c7b74f3d2beb3f1cbac6f460 | |
| parent | 5e79ef58e458d8df9eb6315d3b96e1338652df51 (diff) | |
Using unique @for, @id pair for jabber and sms options in subscriptions
| -rw-r--r-- | actions/subscriptions.php | 22 |
1 files changed, 20 insertions, 2 deletions
diff --git a/actions/subscriptions.php b/actions/subscriptions.php index ba6171ef4..7b10b3425 100644 --- a/actions/subscriptions.php +++ b/actions/subscriptions.php @@ -196,12 +196,30 @@ class SubscriptionsListItem extends SubscriptionListItem $this->out->hidden('token', common_session_token()); $this->out->hidden('profile', $this->profile->id); if (common_config('xmpp', 'enabled')) { - $this->out->checkbox('jabber', _('Jabber'), $sub->jabber); + $attrs = array('name' => 'jabber', + 'type' => 'checkbox', + 'class' => 'checkbox', + 'id' => 'jabber-'.$this->profile->id); + if ($sub->jabber) { + $attrs['checked'] = 'checked'; + } + + $this->out->element('input', $attrs); + $this->out->element('label', array('for' => 'jabber-'.$this->profile->id), _('Jabber')); } else { $this->out->hidden('jabber', $sub->jabber); } if (common_config('sms', 'enabled')) { - $this->out->checkbox('sms', _('SMS'), $sub->sms); + $attrs = array('name' => 'sms', + 'type' => 'checkbox', + 'class' => 'checkbox', + 'id' => 'sms-'.$this->profile->id); + if ($sub->sms) { + $attrs['checked'] = 'checked'; + } + + $this->out->element('input', $attrs); + $this->out->element('label', array('for' => 'sms-'.$this->profile->id), _('SMS')); } else { $this->out->hidden('sms', $sub->sms); } |