diff options
author | cash <cash.costello@gmail.com> | 2011-11-05 16:22:10 -0400 |
---|---|---|
committer | cash <cash.costello@gmail.com> | 2011-11-05 16:22:10 -0400 |
commit | f19305b23bba276ef2601227d1e12f6beceef8ae (patch) | |
tree | 086599d2d4bd3eee41418ccbfb3b0e00753c84de /engine/lib | |
parent | 42e3a2567409713910694f9eafedfc053594d1dc (diff) | |
download | elgg-f19305b23bba276ef2601227d1e12f6beceef8ae.tar.gz elgg-f19305b23bba276ef2601227d1e12f6beceef8ae.tar.bz2 |
Fixes #4018 Refs #3722 merged up to 1.7.14 into master
Diffstat (limited to 'engine/lib')
-rw-r--r-- | engine/lib/entities.php | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/engine/lib/entities.php b/engine/lib/entities.php index 0e89b34bd..711fcbbd7 100644 --- a/engine/lib/entities.php +++ b/engine/lib/entities.php @@ -468,12 +468,10 @@ function can_write_to_container($user_guid = 0, $container_guid = 0, $type = 'al $return = true; } - // Basics, see if the user is a member of the group. + // If still not approved, see if the user is a member of the group // @todo this should be moved to the groups plugin/library - if ($user && $container instanceof ElggGroup) { - if (!$container->isMember($user)) { - $return = false; - } else { + if (!$return && $user && $container instanceof ElggGroup) { + if ($container->isMember($user)) { $return = true; } } |