From 6fa348a2cbf989524c34458cf2a37fb5a7cabb6a Mon Sep 17 00:00:00 2001 From: Sam Minnee Date: Tue, 19 Oct 2010 01:31:30 +0000 Subject: [PATCH] BUGFIX Fixed boundary PHP notice case in RequiredFields::php() where a field name may not be defined in the $data array when a Form is submitted (from r110463) git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@112842 467b73ca-7a2a-4603-9d3b-597d59a354a9 --- forms/RequiredFields.php | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/forms/RequiredFields.php b/forms/RequiredFields.php index f1981e437..950618c67 100755 --- a/forms/RequiredFields.php +++ b/forms/RequiredFields.php @@ -103,11 +103,12 @@ JS; $error = true; // submitted data for file upload fields come back as an array - if(is_array($data[$fieldName])) { - $error = ($data[$fieldName]) ? false : true; + $value = isset($data[$fieldName]) ? $data[$fieldName] : null; + if(is_array($value)) { + $error = ($value) ? false : true; } else { // assume a string or integer - $error = (strlen($data[$fieldName])) ? false : true; + $error = (strlen($value)) ? false : true; } if($formField && $error) {