make sure FlagprofileAction without Profile_flag

This commit is contained in:
Evan Prodromou 2009-11-07 13:22:07 -05:00
parent 285745d2fd
commit 509109a71a

View File

@ -81,19 +81,12 @@ class FlagprofileAction extends Action
return false; return false;
} }
$this->flag = $this->trimmed('flag');
if (empty($this->flag)) {
$this->flag = Profile_flag::DEFAULTFLAG;
}
$user = common_current_user(); $user = common_current_user();
assert(!empty($user)); // checked above assert(!empty($user)); // checked above
if (User_flag_profile::exists($this->profile->id, if (User_flag_profile::exists($this->profile->id,
$user->id, $user->id))
$this->flag))
{ {
$this->clientError(_('Flag already exists.')); $this->clientError(_('Flag already exists.'));
return false; return false;
@ -134,13 +127,11 @@ class FlagprofileAction extends Action
assert(!empty($user)); assert(!empty($user));
assert(!empty($this->profile)); assert(!empty($this->profile));
assert(!empty($this->flag));
$ufp = new User_flag_profile(); $ufp = new User_flag_profile();
$ufp->profile_id = $this->profile->id; $ufp->profile_id = $this->profile->id;
$ufp->user_id = $user->id; $ufp->user_id = $user->id;
$ufp->flag = $this->flag;
$ufp->created = common_sql_now(); $ufp->created = common_sql_now();
if (!$ufp->insert()) { if (!$ufp->insert()) {