fix: do not default to narrowSymbol for currencyDisplay (#42)

Author: ramsey

CHANGELOG.md

@@ -5,6 +5,28 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## 0.5.1 - 2022-02-01
+
+### Added
+
+- Nothing.
+
+### Changed
+
+- Nothing.
+
+### Deprecated
+
+- Nothing.
+
+### Removed
+
+- Nothing.
+
+### Fixed
+
+- Do not default to `narrowSymbol` for `currencyDisplay`.
+
 ## 0.5.0 - 2022-02-01
 
 ### Added

src/FormatPHP.php

@@ -152,7 +152,6 @@ public function formatCurrency($value, string $currencyCode, ?NumberFormatOption
         $options = $options ?? new NumberFormatOptions();
         $options->style = NumberFormatOptions::STYLE_CURRENCY;
         $options->currency = $currencyCode;
-        $options->currencyDisplay = $options->currencyDisplay ?? NumberFormatOptions::CURRENCY_DISPLAY_NARROW_SYMBOL;
 
         return $this->formatNumber($value, $options);
     }

tests/FormatPHPTest.php

@@ -373,17 +373,17 @@ public function testFormatNumberWithOptions(): void
 
     public function testFormatCurrency(): void
     {
-        $locale = new Locale('en');
+        $locale = new Locale('en-GB');
         $config = new Config($locale);
         $messageCollection = new MessageCollection();
         $formatphp = new FormatPHP($config, $messageCollection);
 
-        $this->assertSame('$1,234.00', $formatphp->formatCurrency(1234, 'USD'));
+        $this->assertSame('US$1,234.00', $formatphp->formatCurrency(1234, 'USD'));
     }
 
     public function testFormatCurrencyWithOptions(): void
     {
-        $locale = new Locale('en');
+        $locale = new Locale('en-US');
         $config = new Config($locale);
         $messageCollection = new MessageCollection();
         $formatphp = new FormatPHP($config, $messageCollection);