Dont set the locale implicitly, just use the value provided by \Locale::getDefault() if none is set

Add tests

Signed-off-by: Matthias Kühne <matthias.kuehne@ellerhold.de>

Author: MatthiasKuehneEllerhold

src/View/Helper/CurrencyFormat.php

@@ -212,7 +212,7 @@ public function setLocale($locale)
     public function getLocale()
     {
         if ($this->locale === null) {
-            $this->locale = Locale::getDefault();
+            return Locale::getDefault();
         }
 
         return $this->locale;

src/View/Helper/DateFormat.php

@@ -98,7 +98,7 @@ public function setLocale($locale)
     public function getLocale()
     {
         if ($this->locale === null) {
-            $this->locale = Locale::getDefault();
+            return Locale::getDefault();
         }
 
         return $this->locale;

src/View/Helper/NumberFormat.php

@@ -214,7 +214,7 @@ public function setLocale($locale)
     public function getLocale()
     {
         if ($this->locale === null) {
-            $this->locale = Locale::getDefault();
+            return Locale::getDefault();
         }
 
         return $this->locale;

test/View/Helper/CurrencyFormatTest.php

@@ -108,4 +108,16 @@ public static function assertMbStringEquals(string $expected, string $test, stri
         $test     = str_replace(["\xC2\xA0", ' '], '', $test);
         self::assertEquals($expected, $test, $message);
     }
+
+    public function testNoImplicitLocale(): void
+    {
+        Locale::setDefault('de');
+        self::assertEquals(Locale::getDefault(), $this->helper->getLocale());
+
+        Locale::setDefault('en');
+        self::assertEquals(Locale::getDefault(), $this->helper->getLocale());
+
+        $this->helper->setLocale('es');
+        self::assertEquals('es', $this->helper->getLocale());
+    }
 }

test/View/Helper/DateFormatTest.php

@@ -295,4 +295,16 @@ public function testIntlCalendarIsHandledAsWell(): void
             $helper->__invoke($calendar, IntlDateFormatter::FULL, IntlDateFormatter::FULL, 'it_IT', 'dd-MM-Y')
         );
     }
+
+    public function testNoImplicitLocale(): void
+    {
+        Locale::setDefault('de');
+        self::assertEquals(Locale::getDefault(), $this->helper->getLocale());
+
+        Locale::setDefault('en');
+        self::assertEquals(Locale::getDefault(), $this->helper->getLocale());
+
+        $this->helper->setLocale('es');
+        self::assertEquals('es', $this->helper->getLocale());
+    }
 }

test/View/Helper/NumberFormatTest.php

@@ -196,4 +196,16 @@ public static function assertMbStringEquals(string $expected, string $test, stri
         $test     = str_replace(["\xC2\xA0", ' '], '', $test);
         self::assertEquals($expected, $test, $message);
     }
+
+    public function testNoImplicitLocale(): void
+    {
+        Locale::setDefault('de');
+        self::assertEquals(Locale::getDefault(), $this->helper->getLocale());
+
+        Locale::setDefault('en');
+        self::assertEquals(Locale::getDefault(), $this->helper->getLocale());
+
+        $this->helper->setLocale('es');
+        self::assertEquals('es', $this->helper->getLocale());
+    }
 }