-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixed file visitor for extracting exception messages #232
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Given the logic here is quite complicated, maybe we can have unit test coverage to fully understand what's going on?
src/lib/MVC/Symfony/Translation/ExceptionMessageTemplateFileVisitor.php
Outdated
Show resolved
Hide resolved
src/lib/MVC/Symfony/Translation/ExceptionMessageTemplateFileVisitor.php
Outdated
Show resolved
Hide resolved
tests/lib/MVC/Symfony/Translation/ExceptionMessageTemplateFileVisitorTest.php
Outdated
Show resolved
Hide resolved
src/lib/MVC/Symfony/Translation/ExceptionMessageTemplateFileVisitor.php
Outdated
Show resolved
Hide resolved
tests/lib/MVC/Symfony/Translation/ExceptionMessageTemplateFileVisitorTest.php
Outdated
Show resolved
Hide resolved
tests/lib/MVC/Symfony/Translation/fixtures/NoTranslationToExtract.php
Outdated
Show resolved
Hide resolved
tests/lib/MVC/Symfony/Translation/fixtures/NoTranslationToExtract.php
Outdated
Show resolved
Hide resolved
tests/lib/MVC/Symfony/Translation/fixtures/SetMessageTemplate.php
Outdated
Show resolved
Hide resolved
tests/lib/MVC/Symfony/Translation/fixtures/WrongTranslationId.php
Outdated
Show resolved
Hide resolved
b11853f
to
d946511
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
d946511
to
88d052f
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
v4.5
Because of this error wrong translation domain was created (messages) during the process of translation extraction in the
ibexa/seo
package. The correct domain should berepository_exceptions
. The file in theseo
package could be found here https://github.com/ibexa/seo/blob/main/src/bundle/Resources/translations/repository_exceptions.en.xliff.Checklist:
$ composer fix-cs
).@ibexa/engineering
).