[BUGFIX] Allow translation of invalid input errors 10/56510/3
authorStanislas Rolland <typo3@sjbr.ca>
Sun, 1 Apr 2018 20:27:02 +0000 (16:27 -0400)
committerStanislas Rolland <typo3@sjbr.ca>
Mon, 2 Apr 2018 16:46:30 +0000 (18:46 +0200)
Note: Thanks to Christian Toffolo

Resolves: #83808
Change-Id: I2de59231419a202a39014dfbd62235597f1249eb
Reviewed-on: https://review.typo3.org/56510
Reviewed-by: Stanislas Rolland <typo3@sjbr.ca>
Tested-by: Stanislas Rolland <typo3@sjbr.ca>
ChangeLog
Classes/Validation/Validator/CaptchaValidator.php
Classes/Validation/Validator/TtfFileValidator.php

index 6dd5d2f..b8f4e35 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,6 @@
+2018-04-01  Stanislas Rolland  <typo(arobas)sjbr.ca>
+       * Resolves #83808: Allow translation of invalid input error (Thanks to Christian Toffolo)
+
 2017-12-20  Stanislas Rolland  <typo(arobas)sjbr.ca>
        * Update manual
 
index 2d46fee..a790ba6 100644 (file)
@@ -4,11 +4,9 @@ namespace SJBR\SrFreecap\Validation\Validator;
 /***************************************************************
  *  Copyright notice
  *
- *  (c) 2012-2017 Stanislas Rolland <typo3@sjbr.ca>
+ *  (c) 2012-2018 Stanislas Rolland <typo3@sjbr.ca>
  *  All rights reserved
  *
- *  All credits go to the v5 team.
- *
  *  This script is part of the TYPO3 project. The TYPO3 project is
  *  free software; you can redistribute it and/or modify
  *  it under the terms of the GNU General Public License as published by
@@ -39,8 +37,8 @@ class CaptchaValidator extends AbstractValidator
        /**
         * Specifies whether this validator accepts empty values.
         *
-        * If this is TRUE, the validators isValid() method is not called in case of an empty value
-        * Note: A value is considered empty if it is NULL or an empty string!
+        * If this is true, the validators isValid() method is not called in case of an empty value
+        * Note: A value is considered empty if it is null or an empty string!
         * By default all validators except for NotEmpty and the Composite Validators accept empty values
         *
         * @var bool
@@ -54,10 +52,10 @@ class CaptchaValidator extends AbstractValidator
 
        /**
         * Check the word that was entered against the hashed value
-        * Returns TRUE, if the given property ($word) matches the session captcha value.
+        * Returns true, if the given property ($word) matches the session captcha value.
         *
         * @param string $word: the word that was entered and should be validated
-        * @return boolean TRUE, if the word entered matches the hash value, FALSE if an error occured
+        * @return boolean true, if the word entered matches the hash value, false if an error occured
         */
        public function isValid($word)
        {
@@ -87,7 +85,14 @@ class CaptchaValidator extends AbstractValidator
                        $isValid = empty($word);
                }
                if (!$isValid) {
-                       $this->addError('Entered word does not match the image', 9221561048);
+                       // Please enter the word or number as it appears in the image. The entered value was incorrect.
+                       $this->addError(
+                               $this->translateErrorMessage(
+                                       '9221561048',
+                                       'sr_freecap'
+                               ),
+                               9221561048
+                       );
                }
                return $isValid;
        }
index 046611a..1210d90 100644 (file)
@@ -1,63 +1,76 @@
 <?php
 namespace SJBR\SrFreecap\Validation\Validator;
+
 /***************************************************************
-*  Copyright notice
-*
-*  (c) 2012 Stanislas Rolland <typo3@sjbr.ca>
-*  All rights reserved
-*
-*  This class is a backport of the corresponding class of FLOW3. 
-*  All credits go to the v5 team.
-*
-*  This script is part of the TYPO3 project. The TYPO3 project is
-*  free software; you can redistribute it and/or modify
-*  it under the terms of the GNU General Public License as published by
-*  the Free Software Foundation; either version 2 of the License, or
-*  (at your option) any later version.
-*
-*  The GNU General Public License can be found at
-*  http://www.gnu.org/copyleft/gpl.html.
-*
-*  This script is distributed in the hope that it will be useful,
-*  but WITHOUT ANY WARRANTY; without even the implied warranty of
-*  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-*  GNU General Public License for more details.
-*
-*  This copyright notice MUST APPEAR in all copies of the script!
-***************************************************************/
+ *  Copyright notice
+ *
+ *  (c) 2012-2018 Stanislas Rolland <typo3@sjbr.ca>
+ *  All rights reserved
+ *
+ *  This script is part of the TYPO3 project. The TYPO3 project is
+ *  free software; you can redistribute it and/or modify
+ *  it under the terms of the GNU General Public License as published by
+ *  the Free Software Foundation; either version 2 of the License, or
+ *  (at your option) any later version.
+ *
+ *  The GNU General Public License can be found at
+ *  http://www.gnu.org/copyleft/gpl.html.
+ *
+ *  This script is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU General Public License for more details.
+ *
+ *  This copyright notice MUST APPEAR in all copies of the script!
+ ***************************************************************/
+
+use TYPO3\CMS\Core\Utility\GeneralUtility;
+use TYPO3\CMS\Extbase\Validation\Validator\AbstractValidator;
 
 /**
  * Validator for True Type Font file existence
  *
  */
-class TtfFileValidator extends \TYPO3\CMS\Extbase\Validation\Validator\AbstractValidator {
-
+class TtfFileValidator extends AbstractValidator
+{
        /**
-        * Returns TRUE, if the given property ($propertyValue) is a valid number in the given range.
+        * Returns true, if the given property ($propertyValue) is a valid number in the given range.
         *
-        * If at least one error occurred, the result is FALSE.
+        * If at least one error occurred, the result is false.
         *
         * @param mixed $value The value that should be validated
-        * @return boolean TRUE if the value is within the range, otherwise FALSE
+        * @return boolean true if the value is within the range, otherwise false
         */
-       protected function isValid ($value) {
-               $isValid = TRUE;
-
-               $absoluteFileName = \TYPO3\CMS\Core\Utility\GeneralUtility::getFileAbsFileName($value);
-
+       protected function isValid($value)
+       {
+               $isValid = true;
+               $absoluteFileName = GeneralUtility::getFileAbsFileName($value);
                // Check file existence
                if (!is_file($absoluteFileName)) {
-                       $this->addError('A file with the given name could not be found.', 9221561046);
-                       $isValid = FALSE;
+                       // A file with the given name could not be found.
+                       $this->addError(
+                               $this->translateErrorMessage(
+                                       '9221561046',
+                                       'sr_freecap'
+                               ),
+                               9221561046
+                       );
+                       $isValid = false;
                } else {
                        // Check file extension
                        $pathInfo = pathinfo($absoluteFileName);
-                       if (strtolower($pathInfo['extension']) != 'ttf') {
-                               $this->addError('The specified file is not a True Type Font file.', 9221561047);
-                               $isValid = FALSE;                               
+                       if (strtolower($pathInfo['extension']) !== 'ttf') {
+                               // The specified file is not a True Type Font file.
+                               $this->addError(
+                                       $this->translateErrorMessage(
+                                               '9221561047',
+                                               'sr_freecap'
+                                       ),
+                                       9221561046
+                               );
+                               $isValid = false;
                        }
                }
                return $isValid;
        }
-}
-?>
\ No newline at end of file
+}
\ No newline at end of file