Skip to content

Commit

Permalink
rename build parameters which were removed in 6.3
Browse files Browse the repository at this point in the history
  • Loading branch information
yceruto committed Aug 8, 2023
1 parent 53f6072 commit 2f7079f
Show file tree
Hide file tree
Showing 6 changed files with 15 additions and 15 deletions.
2 changes: 1 addition & 1 deletion config/currencies.php
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@
->args([abstract_arg('currencies')])

->set(CurrencyList::class)
->args([param('.money_currencies')])
->args([param('money_currencies')])
->tag(CurrenciesPass::TAG)

->set(ISOCurrencies::class)
Expand Down
2 changes: 1 addition & 1 deletion config/exchangers.php
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
])

->set(FixedExchange::class)
->args([param('.money_exchange_fixed')])
->args([param('money_exchange_fixed')])

->set(IndirectExchange::class)
->args([
Expand Down
8 changes: 4 additions & 4 deletions config/formatters.php
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,9 @@
->set(IntlNumberFormatterFactory::class)
->args([
service('request_stack')->nullOnInvalid(),
param('.money_formatter_number_locale'),
param('.money_formatter_number_style'),
param('.money_formatter_number_pattern'),
param('money_formatter_number_locale'),
param('money_formatter_number_style'),
param('money_formatter_number_pattern'),
])

->set('money.intl.number_formatter', \NumberFormatter::class)
Expand Down Expand Up @@ -60,7 +60,7 @@

->set(BitcoinMoneyFormatter::class)
->args([
param('.money_formatter_fraction_digits'),
param('money_formatter_fraction_digits'),
service(Currencies::class),
])
->tag(FormattersPass::TAG, ['code' => BitcoinCurrencies::CODE])
Expand Down
2 changes: 1 addition & 1 deletion config/parsers.php
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@
->tag('money.parser')

->set(BitcoinMoneyParser::class)
->args([param('.money_formatter_fraction_digits')])
->args([param('money_formatter_fraction_digits')])
->tag('money.parser')

->alias(MoneyParser::class, AggregateMoneyParser::class)
Expand Down
12 changes: 6 additions & 6 deletions src/DependencyInjection/MoneyExtension.php
Original file line number Diff line number Diff line change
Expand Up @@ -37,12 +37,12 @@ public function load(array $configs, ContainerBuilder $container): void
{
$config = $this->processConfiguration($this->getConfiguration([], $container), $configs);

$container->setParameter('.money_currencies', $config['currencies']);
$container->setParameter('.money_formatter_fraction_digits', $config['formatters']['bitcoin']['fraction_digits']);
$container->setParameter('.money_formatter_number_locale', $config['formatters']['intl']['number_locale']);
$container->setParameter('.money_formatter_number_style', $config['formatters']['intl']['number_style']);
$container->setParameter('.money_formatter_number_pattern', $config['formatters']['intl']['number_pattern']);
$container->setParameter('.money_exchange_fixed', $config['exchanges']['fixed']);
$container->setParameter('money_currencies', $config['currencies']);
$container->setParameter('money_formatter_fraction_digits', $config['formatters']['bitcoin']['fraction_digits']);
$container->setParameter('money_formatter_number_locale', $config['formatters']['intl']['number_locale']);
$container->setParameter('money_formatter_number_style', $config['formatters']['intl']['number_style']);
$container->setParameter('money_formatter_number_pattern', $config['formatters']['intl']['number_pattern']);
$container->setParameter('money_exchange_fixed', $config['exchanges']['fixed']);

$container->registerForAutoconfiguration(Currencies::class)
->addTag(CurrenciesPass::TAG);
Expand Down
4 changes: 2 additions & 2 deletions tests/DependencyInjection/MoneyExtensionTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,8 @@ public function testCurrencyServices(): void
->addTag('money.currencies');
});

self::assertTrue($container->hasParameter('.money_currencies'));
self::assertSame(['FOO' => 3], $container->getParameter('.money_currencies'));
self::assertTrue($container->hasParameter('money_currencies'));
self::assertSame(['FOO' => 3], $container->getParameter('money_currencies'));

$currencies = $container->get(AggregateCurrencies::class);

Expand Down

0 comments on commit 2f7079f

Please sign in to comment.