summaryrefslogtreecommitdiff
path: root/actions/newgroup.php
diff options
context:
space:
mode:
authorEric Helgeson <erichelgeson@gmail.com>2009-11-18 15:53:33 -0500
committerEric Helgeson <erichelgeson@gmail.com>2009-11-18 15:54:24 -0500
commit199ccdb53fbd732eeced3edf734e39687729da9b (patch)
tree9b32c2c2de01b8a339da9c6ea276189f0be2b917 /actions/newgroup.php
parent8f2db3820f6ab2c8881521c0adc9a6427b5ea86b (diff)
Consolidate group creation into static function in User_group
Diffstat (limited to 'actions/newgroup.php')
-rw-r--r--actions/newgroup.php46
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);
}