diff options
| author | cash <cash.costello@gmail.com> | 2013-01-22 14:31:22 -0500 |
|---|---|---|
| committer | cash <cash.costello@gmail.com> | 2013-01-22 14:31:22 -0500 |
| commit | 87995e28bab9a291db54da3faca90e2336066cbe (patch) | |
| tree | 6242855d2f564262b1c754518f41aea026264a46 | |
| parent | 718cddc89e3b4901a641e175e86b4208436f2089 (diff) | |
| download | elgg-87995e28bab9a291db54da3faca90e2336066cbe.tar.gz elgg-87995e28bab9a291db54da3faca90e2336066cbe.tar.bz2 | |
addresses pull request 472 - group not having an owner is a serious problem - try exception approach - maybe we need a mechanism for displaying an error page besides exceptions
| -rw-r--r-- | mod/groups/views/default/groups/profile/summary.php | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/mod/groups/views/default/groups/profile/summary.php b/mod/groups/views/default/groups/profile/summary.php index 6eca9745e..261a746d1 100644 --- a/mod/groups/views/default/groups/profile/summary.php +++ b/mod/groups/views/default/groups/profile/summary.php @@ -15,8 +15,10 @@ if (!isset($vars['entity']) || !$vars['entity']) { $group = $vars['entity']; $owner = $group->getOwnerEntity(); -if (!$owner) { - return true; +if ($owner) { + // not having an owner is very bad so we throw an exception + $msg = elgg_echo('InvalidParameterException:IdNotExistForGUID', array('group owner', $group->guid)); + throw new InvalidParameterException($msg); } ?> |