jsfunc.validateform.js does not verify checkbox fields
authorMartin Kutschker <martin.t.kutschker@blackbox.net>
Fri, 18 May 2007 18:17:33 +0000 (18:17 +0000)
committerMartin Kutschker <martin.t.kutschker@blackbox.net>
Fri, 18 May 2007 18:17:33 +0000 (18:17 +0000)
git-svn-id: https://svn.typo3.org/TYPO3v4/Core/trunk@2343 709f56b5-9817-0410-a4d7-c38de5d9e867

ChangeLog
t3lib/jsfunc.validateform.js
typo3/sysext/cms/tslib/class.tslib_content.php

index 133a500..8dc0832 100755 (executable)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,10 +1,10 @@
 
-
 2007-05-18  Martin Kutschker  <martin.t.kutschker@blackbox.net>
 
        * Fixed bug #5645: Missing Slovenian flag
        * Fixed bug #5628: Missing Slovak flag
        * Fixed bug #5578: config setting SYS[requestURIvar] doesn't work
+       * Fixed bug #4509/3925: jsfunc.validateform.js does not verify checkbox fields (thanks to Jascha Schubert and Eicke Godehardt)
 
 2007-05-17 Andreas Otto <andreas.otto@dkd.de>
        * Feature request: If TYPO3 is configured to send a notification email about install tool logins this email discloses the used password if the login fails. Since this behaviour might not be wanted only the last 5 characters of the md5 hash of the used password are disclosed.
index ca633e3..e9f6672 100755 (executable)
@@ -87,6 +87,7 @@ function validateForm(theFormname,theFieldlist,goodMess,badMess,emailMess)    {
                                                }
                                        break;
                                        case "radio":
+                                       case "checkbox":
                                                var len=fObj.length;
                                                if (len)        {
                                                        for (a=0;a<len;a++)     {
index f3d5c15..ad51f2b 100755 (executable)
@@ -1945,7 +1945,7 @@ class tslib_cObj {
                                                        $confData['required'] = 1;      // Setting this so "required" layout is used.
                                                break;
                                                default:
-                                                       if ($confData['required'] && $confData['type']!='check')        {
+                                                       if ($confData['required'])      {
                                                                $fieldlist[] = rawurlencode($confData['fieldname']);
                                                                $fieldlist[] = rawurlencode($confData['label']);
                                                        }