Skip to content
Permalink
Browse files

fix(groups): do not attempt to populate groups_entity table if base e…

…ntity fails to save

ElggGroup::create() now terminates early if ElggEntity::create() fails to save the entity.
  • Loading branch information...
hypeJunction committed May 11, 2015
1 parent 6af9814 commit f2cbb23722aa60ab9449eadd958d98e7b585cab1
Showing with 6 additions and 0 deletions.
  1. +6 −0 engine/classes/ElggGroup.php
@@ -523,6 +523,12 @@ protected function create() {
global $CONFIG;
$guid = parent::create();
if (!$guid) {
// @todo this probably means permission to create entity was denied
// Is returning false the correct thing to do
return false;
}
$name = sanitize_string($this->name);
$description = sanitize_string($this->description);

0 comments on commit f2cbb23

Please sign in to comment.
You can’t perform that action at this time.