aboutsummaryrefslogtreecommitdiff
path: root/mod/groups
diff options
context:
space:
mode:
authorcash <cash.costello@gmail.com>2013-01-22 14:31:22 -0500
committercash <cash.costello@gmail.com>2013-01-22 14:31:22 -0500
commit87995e28bab9a291db54da3faca90e2336066cbe (patch)
tree6242855d2f564262b1c754518f41aea026264a46 /mod/groups
parent718cddc89e3b4901a641e175e86b4208436f2089 (diff)
downloadelgg-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
Diffstat (limited to 'mod/groups')
-rw-r--r--mod/groups/views/default/groups/profile/summary.php6
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);
}
?>