[BUGFIX] Rename setLocale() in DateTimeViewHelperTest 13/42113/2
authorAndreas Fernandez <a.fernandez@scripting-base.de>
Wed, 29 Jul 2015 18:07:00 +0000 (20:07 +0200)
committerAndreas Fernandez <typo3@scripting-base.de>
Wed, 29 Jul 2015 18:07:29 +0000 (20:07 +0200)
The method setLocale($locale) in the DateTimeViewHelperTest
was implemented to test against several locales. However,
as there is a setLocale() in the parent class from PHPUnit that
does not take any parameter, PHP7 throws a warning as the
subclass does not have the same method signature as the parent
class.

The patch changes the method to be named differently thus,
solving the method signature conflict inside the test.

Resolves: #67092
Releases: 6.2, master
Change-Id: Ia05d90c64a3a8e2fc599935d2435192392e5a667
Reviewed-on: http://review.typo3.org/42113
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
typo3/sysext/fluid/Tests/Unit/ViewHelpers/Format/DateViewHelperTest.php

index 9ddd7a7..2909070 100644 (file)
@@ -258,14 +258,14 @@ class DateViewHelperTest extends \TYPO3\CMS\Core\Tests\UnitTestCase {
                if (!setlocale(LC_COLLATE, $locale)) {
                        $this->markTestSkipped('Locale ' . $locale . ' is not available.');
                }
-               $this->setLocale($locale);
+               $this->setCustomLocale($locale);
                $this->assertEquals($expected, $viewHelper->render($timestamp, $format));
        }
 
        /**
         * @param string $locale
         */
-       protected function setLocale($locale) {
+       protected function setCustomLocale($locale) {
                setlocale(LC_CTYPE, $locale);
                setlocale(LC_MONETARY, $locale);
                setlocale(LC_TIME, $locale);