From f3018529ac794395049f0f4a860c38d2461559bf Mon Sep 17 00:00:00 2001 From: Ingo Schommer Date: Fri, 14 Sep 2007 17:10:37 +0000 Subject: [PATCH] elofgren: Add 2 needed isset() calls: in forms/Form.php and security/Member.php (merged from branches/gsoc) git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@41770 467b73ca-7a2a-4603-9d3b-597d59a354a9 --- forms/Form.php | 2 +- security/Member.php | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/forms/Form.php b/forms/Form.php index c87dc28aa..9a032ecf9 100644 --- a/forms/Form.php +++ b/forms/Form.php @@ -567,7 +567,7 @@ class Form extends ViewableData { if(isset($tableFieldName) && isset($subFieldName) && is_a($fields[$tableFieldName], 'TableField')) { $field = $fields[$tableFieldName]->getField($subFieldName, $fieldName); return $field->$methodName(); - } else if($fields[$fieldName]) { + } else if(isset($fields[$fieldName])) { return $fields[$fieldName]->$methodName(); } else { user_error("Form::callfieldmethod() Field '$fieldName' not found", E_USER_ERROR); diff --git a/security/Member.php b/security/Member.php index 27069a142..f56946657 100644 --- a/security/Member.php +++ b/security/Member.php @@ -290,7 +290,7 @@ class Member extends DataObject { $table = "Group_Members"; - if($collatedGroups) { + if(isset($collatedGroups)) { $collatedGroups = implode(", ", array_unique($collatedGroups)); $result = singleton('Group')->instance_get("`ID` IN ($collatedGroups)", "ID", "", "", "Member_GroupSet");