diff options
author | Eric Helgeson <erichelgeson@gmail.com> | 2009-11-18 15:53:33 -0500 |
---|---|---|
committer | Eric Helgeson <erichelgeson@gmail.com> | 2009-11-18 15:54:24 -0500 |
commit | 199ccdb53fbd732eeced3edf734e39687729da9b (patch) | |
tree | 9b32c2c2de01b8a339da9c6ea276189f0be2b917 /actions/newgroup.php | |
parent | 8f2db3820f6ab2c8881521c0adc9a6427b5ea86b (diff) |
Consolidate group creation into static function in User_group
Diffstat (limited to 'actions/newgroup.php')
-rw-r--r-- | actions/newgroup.php | 46 |
1 files changed, 7 insertions, 39 deletions
diff --git a/actions/newgroup.php b/actions/newgroup.php index 80da9861a..25da7f8fc 100644 --- a/actions/newgroup.php +++ b/actions/newgroup.php @@ -186,45 +186,13 @@ class NewgroupAction extends Action assert(!is_null($cur)); - $group = new User_group(); - - $group->query('BEGIN'); - - $group->nickname = $nickname; - $group->fullname = $fullname; - $group->homepage = $homepage; - $group->description = $description; - $group->location = $location; - $group->created = common_sql_now(); - - $result = $group->insert(); - - if (!$result) { - common_log_db_error($group, 'INSERT', __FILE__); - $this->serverError(_('Could not create group.')); - } - - $result = $group->setAliases($aliases); - - if (!$result) { - $this->serverError(_('Could not create aliases.')); - } - - $member = new Group_member(); - - $member->group_id = $group->id; - $member->profile_id = $cur->id; - $member->is_admin = 1; - $member->created = $group->created; - - $result = $member->insert(); - - if (!$result) { - common_log_db_error($member, 'INSERT', __FILE__); - $this->serverError(_('Could not set group membership.')); - } - - $group->query('COMMIT'); + $group = User_group::register(array('nickname' => $nickname, + 'fullname' => $fullname, + 'homepage' => $homepage, + 'description' => $description, + 'location' => $location, + 'aliases' => $aliases, + 'userid' => $cur->id)); common_redirect($group->homeUrl(), 303); } |