tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

BUGFIX Added specific border case for array form data in RequiredFields::php()

Browse Source 
git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/branches/2.4@108811 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
Sean Harvey 2010-08-03 01:40:53 +00:00 committed by Sam Minnee
parent 9d305bc94a
commit e8e3ea6680
1 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
forms/RequiredFields.php
View File

@ -101,9 +101,16 @@ JS;
foreach($this->required as $fieldName) { foreach($this->required as $fieldName) {
$formField = $fields->dataFieldByName($fieldName); $formField = $fields->dataFieldByName($fieldName);
// we need to check if $data[$fieldName] is an array (FileFiled case) $error = true;
if(($formField && $data[$fieldName] == "") || // submitted data for file upload fields come back as an array
(is_array($data[$fieldName]) && $data[$fieldName]['name'] == "")) { if(is_array($data[$fieldName])) {
$error = ($data[$fieldName]) ? false : true;
} else {
// assume a string or integer
$error = (strlen($data[$fieldName])) ? false : true;
}
if($formField && $error) {
$errorMessage = sprintf(_t('Form.FIELDISREQUIRED', '%s is required').'.', strip_tags('"' . ($formField->Title() ? $formField->Title() : $fieldName) . '"')); $errorMessage = sprintf(_t('Form.FIELDISREQUIRED', '%s is required').'.', strip_tags('"' . ($formField->Title() ? $formField->Title() : $fieldName) . '"'));
if($msg = $formField->getCustomValidationMessage()) { if($msg = $formField->getCustomValidationMessage()) {
$errorMessage = $msg; $errorMessage = $msg;