pass profile id for subscriptions rather than user name

darcs-hash:20081208031312-5ed1f-b570b86b28e8eda763c51be1ae091034f76bd984.gz
This commit is contained in:
Evan Prodromou 2008-12-07 22:13:12 -05:00
parent 81a81baf83
commit fd6df5402e
2 changed files with 15 additions and 8 deletions

View File

@ -45,9 +45,16 @@ class SubscribeAction extends Action {
return; return;
} }
$other_nickname = $this->arg('subscribeto'); $other_id = $this->arg('subscribeto');
$result=subs_subscribe_user($user, $other_nickname); $other = User::staticGet('id', $other_id);
if (!$other) {
$this->client_error(_('Not a local user.'));
return;
}
$result = subs_subscribe_to($user, $other);
if($result != true) { if($result != true) {
common_user_error($result); common_user_error($result);

View File

@ -2050,15 +2050,15 @@ function common_nudge_response() {
} }
function common_subscribe_form($profile) { function common_subscribe_form($profile) {
common_element_start('form', array('id' => 'subscribe-' . $profile->nickname, common_element_start('form', array('id' => 'subscribe-' . $profile->id,
'method' => 'post', 'method' => 'post',
'class' => 'subscribe', 'class' => 'subscribe',
'action' => common_local_url('subscribe'))); 'action' => common_local_url('subscribe')));
common_hidden('token', common_session_token()); common_hidden('token', common_session_token());
common_element('input', array('id' => 'subscribeto-' . $profile->nickname, common_element('input', array('id' => 'subscribeto-' . $profile->id,
'name' => 'subscribeto', 'name' => 'subscribeto',
'type' => 'hidden', 'type' => 'hidden',
'value' => $profile->nickname)); 'value' => $profile->id));
common_element('input', array('type' => 'submit', common_element('input', array('type' => 'submit',
'class' => 'submit', 'class' => 'submit',
'value' => _('Subscribe'))); 'value' => _('Subscribe')));