summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEvan Prodromou <evan@controlyourself.ca>2009-01-22 22:02:39 +0000
committerEvan Prodromou <evan@controlyourself.ca>2009-01-22 22:02:39 +0000
commit04d498784ff1892607a3aee65d76104192fe4af2 (patch)
tree7378405ca8b323658f2fb3dc438c65ec3bdc96b0
parentbaaf55e9d6e6acdf1d2ee0a4099bb8824520ebb4 (diff)
convert tagother action
-rw-r--r--actions/tagother.php122
1 files changed, 63 insertions, 59 deletions
diff --git a/actions/tagother.php b/actions/tagother.php
index e60eb8b58..5daf612fb 100644
--- a/actions/tagother.php
+++ b/actions/tagother.php
@@ -23,71 +23,85 @@ require_once(INSTALLDIR.'/lib/settingsaction.php');
class TagotherAction extends Action
{
+ var $profile = null;
+ var $error = null;
- function handle($args)
+ function prepare($args)
{
-
- parent::handle($args);
-
+ parent::prepare($args);
if (!common_logged_in()) {
$this->clientError(_('Not logged in'), 403);
- return;
+ return false;
+ }
+
+ $id = $this->trimmed('id');
+ if (!$id) {
+ $this->clientError(_('No id argument.'));
+ return false;
}
+ $this->profile = Profile::staticGet('id', $id);
+
+ if (!$this->profile) {
+ $this->clientError(_('No profile with that ID.'));
+ return false;
+ }
+
+ return true;
+ }
+
+ function handle($args)
+ {
+ parent::handle($args);
if ($_SERVER['REQUEST_METHOD'] == 'POST') {
- $this->save_tags();
+ $this->saveTags();
} else {
- $id = $this->trimmed('id');
- if (!$id) {
- $this->clientError(_('No id argument.'));
- return;
- }
- $profile = Profile::staticGet('id', $id);
- if (!$profile) {
- $this->clientError(_('No profile with that ID.'));
- return;
- }
- $this->show_form($profile);
+ $this->showForm($profile);
}
}
- function show_form($profile, $error=null)
+ function title()
{
+ return sprintf(_('Tag %s'), $this->profile->nickname);
+ }
- $user = common_current_user();
-
- common_show_header(_('Tag a person'),
- null, array($profile, $error), array($this, 'show_top'));
+ function showForm($error=null)
+ {
+ $this->error = $error;
+ $this->showPage();
+ }
- $avatar = $profile->getAvatar(AVATAR_PROFILE_SIZE);
+ function showContent()
+ {
+ $avatar = $this->profile->getAvatar(AVATAR_PROFILE_SIZE);
$this->element('img', array('src' => ($avatar) ? common_avatar_display_url($avatar) : common_default_avatar(AVATAR_PROFILE_SIZE),
'class' => 'avatar stream',
'width' => AVATAR_PROFILE_SIZE,
'height' => AVATAR_PROFILE_SIZE,
'alt' =>
- ($profile->fullname) ? $profile->fullname :
- $profile->nickname));
+ ($this->profile->fullname) ? $this->profile->fullname :
+ $this->profile->nickname));
- $this->element('a', array('href' => $profile->profileurl,
+ $this->element('a', array('href' => $this->profile->profileurl,
'class' => 'external profile nickname'),
- $profile->nickname);
+ $this->profile->nickname);
- if ($profile->fullname) {
+ if ($this->profile->fullname) {
$this->elementStart('div', 'fullname');
- if ($profile->homepage) {
- $this->element('a', array('href' => $profile->homepage),
- $profile->fullname);
+ if ($this->profile->homepage) {
+ $this->element('a', array('href' => $this->profile->homepage),
+ $this->profile->fullname);
} else {
- $this->text($profile->fullname);
+ $this->text($this->profile->fullname);
}
$this->elementEnd('div');
}
- if ($profile->location) {
- $this->element('div', 'location', $profile->location);
+ if ($this->profile->location) {
+ $this->element('div', 'location', $this->profile->location);
}
- if ($profile->bio) {
- $this->element('div', 'bio', $profile->bio);
+ if ($this->profile->bio) {
+ $this->element('div', 'bio', $this->profile->bio);
}
$this->elementStart('form', array('method' => 'post',
@@ -95,33 +109,24 @@ class TagotherAction extends Action
'name' => 'tagother',
'action' => $this->selfUrl()));
$this->hidden('token', common_session_token());
- $this->hidden('id', $profile->id);
+ $this->hidden('id', $this->profile->id);
$this->input('tags', _('Tags'),
- ($this->arg('tags')) ? $this->arg('tags') : implode(' ', Profile_tag::getTags($user->id, $profile->id)),
+ ($this->arg('tags')) ? $this->arg('tags') : implode(' ', Profile_tag::getTags($user->id, $this->profile->id)),
_('Tags for this user (letters, numbers, -, ., and _), comma- or space- separated'));
$this->submit('save', _('Save'));
$this->elementEnd('form');
- common_show_footer();
-
}
- function save_tags()
+ function saveTags()
{
-
$id = $this->trimmed('id');
$tagstring = $this->trimmed('tags');
$token = $this->trimmed('token');
if (!$token || $token != common_session_token()) {
- $this->show_form(_('There was a problem with your session token. Try again, please.'));
- return;
- }
-
- $profile = Profile::staticGet('id', $id);
-
- if (!$profile) {
- $this->clientError(_('No such profile.'));
+ $this->showForm(_('There was a problem with your session token.'.
+ ' Try again, please.'));
return;
}
@@ -132,7 +137,7 @@ class TagotherAction extends Action
foreach ($tags as $tag) {
if (!common_valid_profile_tag($tag)) {
- $this->show_form($profile, sprintf(_('Invalid tag: "%s"'), $tag));
+ $this->showForm(sprintf(_('Invalid tag: "%s"'), $tag));
return;
}
}
@@ -143,22 +148,22 @@ class TagotherAction extends Action
$user = common_current_user();
if (!Subscription::pkeyGet(array('subscriber' => $user->id,
- 'subscribed' => $profile->id)) &&
- !Subscription::pkeyGet(array('subscriber' => $profile->id,
+ 'subscribed' => $this->profile->id)) &&
+ !Subscription::pkeyGet(array('subscriber' => $this->profile->id,
'subscribed' => $user->id)))
{
$this->clientError(_('You can only tag people you are subscribed to or who are subscribed to you.'));
return;
}
- $result = Profile_tag::setTags($user->id, $profile->id, $tags);
+ $result = Profile_tag::setTags($user->id, $this->profile->id, $tags);
if (!$result) {
$this->clientError(_('Could not save tags.'));
return;
}
- $action = $user->isSubscribed($profile) ? 'subscriptions' : 'subscribers';
+ $action = $user->isSubscribed($this->profile) ? 'subscriptions' : 'subscribers';
if ($this->boolean('ajax')) {
common_start_html('text/xml');
@@ -182,11 +187,10 @@ class TagotherAction extends Action
}
}
- function show_top($arr = null)
+ function showPageNotice()
{
- list($profile, $error) = $arr;
- if ($error) {
- $this->element('p', 'error', $error);
+ if ($this->error) {
+ $this->element('p', 'error', $this->error);
} else {
$this->elementStart('div', 'instructions');
$this->element('p', null,