Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Addressing] Make sure the CountryNameExtension::translateCountryIsoCode() always returns a string#10805

Merged
Zales0123 merged 2 commits into
Sylius:1.5from
vvasiloi:10799
Nov 8, 2019
Merged

[Addressing] Make sure the CountryNameExtension::translateCountryIsoCode() always returns a string#10805
Zales0123 merged 2 commits into
Sylius:1.5from
vvasiloi:10799

Conversation

@vvasiloi
Copy link
Copy Markdown
Contributor

Fallback to code when there's no translation.
Fallback to empty string when there's no code.

Q A
Branch? 1.5
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets fixes #10799
License MIT

@vvasiloi vvasiloi requested a review from a team as a code owner October 29, 2019 14:26
Comment thread src/Sylius/Bundle/AddressingBundle/Twig/CountryNameExtension.php Outdated
Comment thread src/Sylius/Bundle/AddressingBundle/Twig/CountryNameExtension.php
Comment thread src/Sylius/Bundle/AddressingBundle/spec/Twig/CountryNameExtensionSpec.php Outdated
Comment thread src/Sylius/Bundle/AddressingBundle/spec/Twig/CountryNameExtensionSpec.php Outdated
@Zales0123 Zales0123 added the Enhancement Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.). label Oct 30, 2019
@Zales0123 Zales0123 merged commit 6075261 into Sylius:1.5 Nov 8, 2019
@Zales0123
Copy link
Copy Markdown
Contributor

Thank you, Victor! 🎉

@vvasiloi vvasiloi deleted the 10799 branch November 8, 2019 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants