*/
function showContent()
{
- $profile = $this->scoped;
$user = $this->scoped->getUser();
$this->elementStart('form', array('method' => 'post',
$this->elementStart('li');
// TRANS: Field label in form for profile settings.
$this->input('nickname', _('Nickname'),
- $this->arg('nickname') ?: $profile->nickname,
+ $this->trimmed('nickname') ?: $this->scoped->getNickname(),
// TRANS: Tooltip for field label in form for profile settings.
_('1-64 lowercase letters or numbers, no punctuation or spaces.'),
null, false, // "name" (will be set to id), then "required"
$this->elementStart('li');
// TRANS: Field label in form for profile settings.
$this->input('fullname', _('Full name'),
- ($this->arg('fullname')) ? $this->arg('fullname') : $profile->fullname);
+ $this->trimmed('fullname') ?: $this->scoped->getFullname());
$this->elementEnd('li');
$this->elementStart('li');
// TRANS: Field label in form for profile settings.
$this->input('homepage', _('Homepage'),
- ($this->arg('homepage')) ? $this->arg('homepage') : $profile->homepage,
+ $this->trimmed('homepage') ?: $this->scoped->getHomepage(),
// TRANS: Tooltip for field label in form for profile settings.
_('URL of your homepage, blog, or profile on another site.'));
$this->elementEnd('li');
// TRANS: Text area label in form for profile settings where users can provide
// TRANS: their biography.
$this->textarea('bio', _('Bio'),
- ($this->arg('bio')) ? $this->arg('bio') : $profile->bio,
+ $this->trimmed('bio') ?: $this->scoped->getDescription(),
$bioInstr);
$this->elementEnd('li');
$this->elementStart('li');
// TRANS: Field label in form for profile settings.
$this->input('location', _('Location'),
- ($this->arg('location')) ? $this->arg('location') : $profile->location,
+ $this->trimmed('location') ?: $this->scoped->location,
// TRANS: Tooltip for field label in form for profile settings.
_('Where you are, like "City, State (or Region), Country".'));
$this->elementEnd('li');
// TRANS: Checkbox label in form for profile settings.
$this->checkbox('sharelocation', _('Share my current location when posting notices'),
($this->arg('sharelocation')) ?
- $this->arg('sharelocation') : $this->scoped->shareLocation());
+ $this->boolean('sharelocation') : $this->scoped->shareLocation());
$this->elementEnd('li');
}
Event::handle('EndProfileFormData', array($this));
$this->elementStart('li');
// TRANS: Field label in form for profile settings.
$this->input('tags', _('Tags'),
- ($this->arg('tags')) ? $this->arg('tags') : implode(' ', $user->getSelfTags()),
+ $this->trimmed('tags') ?: implode(' ', Profile_tag::getSelfTagsArray($this->scoped)),
// TRANS: Tooltip for field label in form for profile settings.
_('Tags for yourself (letters, numbers, -, ., and _), comma- or space- separated.'));
$this->elementEnd('li');
*
* @return void
*/
- function handlePost()
+ protected function doPost()
{
- // CSRF protection
- $token = $this->trimmed('token');
- if (!$token || $token != common_session_token()) {
- // TRANS: Form validation error.
- $this->showForm(_('There was a problem with your session token. '.
- 'Try again, please.'));
- return;
- }
-
if (Event::handle('StartProfileSaveForm', array($this))) {
// $nickname will only be set if this changenick value is true.
try {
$nickname = Nickname::normalize($this->trimmed('nickname'), true);
} catch (NicknameTakenException $e) {
- // Abort only if the nickname is occupied by another local profile
- if ($e->profile->id != $this->scoped->id) {
- $this->showForm($e->getMessage());
- return;
+ // Abort only if the nickname is occupied by _another_ local user profile
+ if (!$this->scoped->sameAs($e->profile)) {
+ throw $e;
}
- $nickname = Nickname::normalize($this->trimmed('nickname')); // without in-use check this time
- } catch (NicknameException $e) {
- $this->showForm($e->getMessage());
- return;
+ // Since the variable wasn't set before the exception was thrown, let's run
+ // the normalize sequence again, but without in-use check this time.
+ $nickname = Nickname::normalize($this->trimmed('nickname'));
}
}
if (!is_null($homepage) && (strlen($homepage) > 0) &&
!common_valid_http_url($homepage)) {
// TRANS: Validation error in form for profile settings.
- $this->showForm(_('Homepage is not a valid URL.'));
- return;
+ throw new ClientException(_('Homepage is not a valid URL.'));
} else if (!is_null($fullname) && mb_strlen($fullname) > 191) {
// TRANS: Validation error in form for profile settings.
- $this->showForm(_('Full name is too long (maximum 191 characters).'));
- return;
+ throw new ClientException(_('Full name is too long (maximum 191 characters).'));
} else if (Profile::bioTooLong($bio)) {
// TRANS: Validation error in form for profile settings.
// TRANS: Plural form is used based on the maximum number of allowed
// TRANS: characters for the biography (%d).
- $this->showForm(sprintf(_m('Bio is too long (maximum %d character).',
+ throw new ClientException(sprintf(_m('Bio is too long (maximum %d character).',
'Bio is too long (maximum %d characters).',
Profile::maxBio()),
Profile::maxBio()));
- return;
} else if (!is_null($location) && mb_strlen($location) > 191) {
// TRANS: Validation error in form for profile settings.
- $this->showForm(_('Location is too long (maximum 191 characters).'));
- return;
+ throw new ClientException(_('Location is too long (maximum 191 characters).'));
} else if (is_null($timezone) || !in_array($timezone, DateTimeZone::listIdentifiers())) {
// TRANS: Validation error in form for profile settings.
- $this->showForm(_('Timezone not selected.'));
- return;
+ throw new ClientException(_('Timezone not selected.'));
} else if (!is_null($language) && strlen($language) > 50) {
// TRANS: Validation error in form for profile settings.
- $this->showForm(_('Language is too long (maximum 50 characters).'));
- return;
+ throw new ClientException(_('Language is too long (maximum 50 characters).'));
}
$tags = array();
if (!common_valid_profile_tag($tag)) {
// TRANS: Validation error in form for profile settings.
// TRANS: %s is an invalid tag.
- $this->showForm(sprintf(_('Invalid tag: "%s".'), $tag));
- return;
+ throw new ClientException(sprintf(_('Invalid tag: "%s".'), $tag));
}
$tag_priv[$tag] = $private;
}
}
- $user = common_current_user();
+ $user = $this->scoped->getUser();
$user->query('BEGIN');
// $user->nickname is updated through Profile->update();
$result = $user->update($original);
if ($result === false) {
common_log_db_error($user, 'UPDATE', __FILE__);
+ $user->query('ROLLBACK');
// TRANS: Server error thrown when user profile settings could not be updated to
// TRANS: automatically subscribe to any subscriber.
- $this->serverError(_('Could not update user for autosubscribe or subscribe_policy.'));
+ throw new ServerException(_('Could not update user for autosubscribe or subscribe_policy.'));
}
// Re-initialize language environment if it changed
common_init_language();
}
- $profile = $user->getProfile();
-
- $orig_profile = clone($profile);
+ $original = clone($this->scoped);
- if (common_config('profile', 'changenick') == true && $profile->nickname !== $nickname) {
+ if (common_config('profile', 'changenick') == true && $this->scoped->getNickname() !== $nickname) {
assert(Nickname::normalize($nickname)===$nickname);
- common_debug("Changing user nickname from '{$profile->nickname}' to '{$nickname}'.");
- $profile->nickname = $nickname;
- $profile->profileurl = common_profile_url($profile->nickname);
+ common_debug("Changing user nickname from '{$this->scoped->getNickname()}' to '{$nickname}'.");
+ $this->scoped->nickname = $nickname;
+ $this->scoped->profileurl = common_profile_url($this->scoped->getNickname());
}
- $profile->fullname = $fullname;
- $profile->homepage = $homepage;
- $profile->bio = $bio;
- $profile->location = $location;
+ $this->scoped->fullname = $fullname;
+ $this->scoped->homepage = $homepage;
+ $this->scoped->bio = $bio;
+ $this->scoped->location = $location;
$loc = Location::fromName($location);
if (empty($loc)) {
- $profile->lat = null;
- $profile->lon = null;
- $profile->location_id = null;
- $profile->location_ns = null;
+ $this->scoped->lat = null;
+ $this->scoped->lon = null;
+ $this->scoped->location_id = null;
+ $this->scoped->location_ns = null;
} else {
- $profile->lat = $loc->lat;
- $profile->lon = $loc->lon;
- $profile->location_id = $loc->location_id;
- $profile->location_ns = $loc->location_ns;
+ $this->scoped->lat = $loc->lat;
+ $this->scoped->lon = $loc->lon;
+ $this->scoped->location_id = $loc->location_id;
+ $this->scoped->location_ns = $loc->location_ns;
}
if (common_config('location', 'share') == 'user') {
$exists = false;
- $prefs = User_location_prefs::getKV('user_id', $user->id);
+ $prefs = User_location_prefs::getKV('user_id', $this->scoped->getID());
if (empty($prefs)) {
$prefs = new User_location_prefs();
- $prefs->user_id = $user->id;
+ $prefs->user_id = $this->scoped->getID();
$prefs->created = common_sql_now();
} else {
$exists = true;
if ($result === false) {
common_log_db_error($prefs, ($exists) ? 'UPDATE' : 'INSERT', __FILE__);
+ $user->query('ROLLBACK');
// TRANS: Server error thrown when user profile location preference settings could not be updated.
- $this->serverError(_('Could not save location prefs.'));
+ throw new ServerException(_('Could not save location prefs.'));
}
}
- common_debug('Old profile: ' . common_log_objstring($orig_profile), __FILE__);
- common_debug('New profile: ' . common_log_objstring($profile), __FILE__);
+ common_debug('Old profile: ' . common_log_objstring($original), __FILE__);
+ common_debug('New profile: ' . common_log_objstring($this->scoped), __FILE__);
- $result = $profile->update($orig_profile);
+ $result = $this->scoped->update($original);
if ($result === false) {
- common_log_db_error($profile, 'UPDATE', __FILE__);
+ common_log_db_error($this->scoped, 'UPDATE', __FILE__);
+ $user->query('ROLLBACK');
// TRANS: Server error thrown when user profile settings could not be saved.
- $this->serverError(_('Could not save profile.'));
+ throw new ServerException(_('Could not save profile.'));
}
// Set the user tags
- $result = $user->setSelfTags($tags, $tag_priv);
-
- if (!$result) {
- // TRANS: Server error thrown when user profile settings tags could not be saved.
- $this->serverError(_('Could not save tags.'));
- }
+ $result = Profile_tag::setSelfTags($this->scoped, $tags, $tag_priv);
$user->query('COMMIT');
Event::handle('EndProfileSaveForm', array($this));
// TRANS: Confirmation shown when user profile settings are saved.
- $this->showForm(_('Settings saved.'), true);
+ return _('Settings saved.');
}
}
function showAside() {
- $user = common_current_user();
-
$this->elementStart('div', array('id' => 'aside_primary',
'class' => 'aside'));
'class' => 'section'));
$this->elementStart('ul');
if (Event::handle('StartProfileSettingsActions', array($this))) {
- if ($user->hasRight(Right::BACKUPACCOUNT)) {
+ if ($this->scoped->hasRight(Right::BACKUPACCOUNT)) {
$this->elementStart('li');
$this->element('a',
array('href' => common_local_url('backupaccount')),
_('Backup account'));
$this->elementEnd('li');
}
- if ($user->hasRight(Right::DELETEACCOUNT)) {
+ if ($this->scoped->hasRight(Right::DELETEACCOUNT)) {
$this->elementStart('li');
$this->element('a',
array('href' => common_local_url('deleteaccount')),
_('Delete account'));
$this->elementEnd('li');
}
- if ($user->hasRight(Right::RESTOREACCOUNT)) {
+ if ($this->scoped->hasRight(Right::RESTOREACCOUNT)) {
$this->elementStart('li');
$this->element('a',
array('href' => common_local_url('restoreaccount')),
/**
* Table Definition for profile_tag
*/
-require_once INSTALLDIR.'/classes/Memcached_DataObject.php';
class Profile_tag extends Managed_DataObject
{
- ###START_AUTOCODE
- /* the code below is auto generated do not remove the above tag */
-
public $__table = 'profile_tag'; // table name
public $tagger; // int(4) primary_key not_null
public $tagged; // int(4) primary_key not_null
public $tag; // varchar(64) primary_key not_null
public $modified; // timestamp() not_null default_CURRENT_TIMESTAMP
- /* the code above is auto generated do not remove the tag below */
- ###END_AUTOCODE
-
public static function schemaDef()
{
return array(
return Profile_list::pkeyGet(array('tagger' => $this->tagger, 'tag' => $this->tag));
}
+ static function getSelfTagsArray(Profile $target)
+ {
+ return self::getTagsArray($target->getID(), $target->getID(), $target);
+ }
+
+ static function setSelfTags(Profile $target, array $newtags, array $privacy=array())
+ {
+ return self::setTags($target->getID(), $target->getID(), $newtags, $privacy);
+ }
+
static function getTags($tagger, $tagged, $auth_user=null) {
$profile_list = new Profile_list();
return $profile_list;
}
- static function getTagsArray($tagger, $tagged, $auth_user_id=null)
+ static function getTagsArray($tagger, $tagged, Profile $scoped=null)
{
$ptag = new Profile_tag();
'and profile_tag.tagged = %d ',
$tagger, $tagged);
- if ($auth_user_id != $tagger) {
+ if (!$scoped instanceof Profile || $scoped->getID() !== $tagger) {
$qry .= 'and profile_list.private = 0';
}
return $tags;
}
- static function setTags($tagger, $tagged, $newtags, $privacy=array()) {
+ static function setTags($tagger, $tagged, array $newtags, array $privacy=array()) {
$newtags = array_unique($newtags);
- $oldtags = self::getTagsArray($tagger, $tagged, $tagger);
+ $oldtags = self::getTagsArray($tagger, $tagged, Profile::getByID($tagger));
$ptag = new Profile_tag();
'tag' => $tag));
# if tag already exists, return it
- if(!empty($ptag)) {
+ if ($ptag instanceof Profile_tag) {
return $ptag;
}
- $tagger_profile = Profile::getKV('id', $tagger);
- $tagged_profile = Profile::getKV('id', $tagged);
+ $tagger_profile = Profile::getByID($tagger);
+ $tagged_profile = Profile::getByID($tagged);
if (Event::handle('StartTagProfile', array($tagger_profile, $tagged_profile, $tag))) {
if (!$tagger_profile->canTag($tagged_profile)) {
// TRANS: Client exception thrown trying to set a tag for a user that cannot be tagged.
throw new ClientException(_('You cannot tag this user.'));
- return false;
}
$tags = new Profile_list();
'which is the maximum allowed number of tags. ' .
'Try using or deleting some existing tags.'),
common_config('peopletag', 'maxtags')));
- return false;
}
$plist = new Profile_list();
'which is the maximum allowed number. ' .
'Try unlisting others first.'),
common_config('peopletag', 'maxpeople'), $tag));
- return false;
}
$newtag = new Profile_tag();
$result = $newtag->insert();
-
if (!$result) {
common_log_db_error($newtag, 'INSERT', __FILE__);
+ $plist->query('ROLLBACK');
return false;
}
$newtag->delete();
$profile_list->delete();
throw $e;
- return false;
}
$profile_list->taggedCount(true);
if (Event::handle('StartUntagProfile', array($ptag))) {
$orig = clone($ptag);
$result = $ptag->delete();
- if (!$result) {
+ if ($result === false) {
common_log_db_error($this, 'DELETE', __FILE__);
return false;
}
Event::handle('EndUntagProfile', array($orig));
- if ($result) {
- $profile_list = Profile_list::pkeyGet(array('tag' => $tag, 'tagger' => $tagger));
- if (!empty($profile_list)) {
- $profile_list->taggedCount(true);
- }
- self::blowCaches($tagger, $tagged);
- return true;
+ $profile_list = Profile_list::pkeyGet(array('tag' => $tag, 'tagger' => $tagger));
+ if (!empty($profile_list)) {
+ $profile_list->taggedCount(true);
}
- return false;
+ self::blowCaches($tagger, $tagged);
+ return true;
}
}
projects / quix0rs-gnu-social.git / commitdiff