Skip to content

[Translation] Fix default-path XML config #18872

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

Merged

Conversation

HeahDude
Copy link
Contributor

@HeahDude HeahDude commented Sep 16, 2023

Note: we should wait for symfony/symfony#51671 to be merged first.

@carsonbot carsonbot added this to the 5.4 milestone Sep 16, 2023
@carsonbot carsonbot changed the title [Translator] Fix default-path XML config [Translation] Fix default-path XML config Sep 16, 2023
@OskarStark OskarStark added the Waiting Code Merge Docs for features pending to be merged label Sep 16, 2023
@carsonbot carsonbot modified the milestones: 5.4, next Sep 16, 2023
nicolas-grekas added a commit to symfony/symfony that referenced this pull request Sep 19, 2023
…` in XML (HeahDude)

This PR was merged into the 5.4 branch.

Discussion
----------

[FrameworkBundle] Fix support for `translator.default_path` in XML

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | ~
| License       | MIT
| Doc PR        | symfony/symfony-docs#18872

The docs is currently wrong assuming there is an element `default-path` using XML config,  https://symfony.com/doc/current/translation.html#configuration.

It appears it's missing in the XSD, but IMO it should be added as an attribute.

Commits
-------

3534118 [Translator] Fix support for `default_path` in XML
@HeahDude
Copy link
Contributor Author

symfony/symfony#51671 has been merged.

@javiereguiluz javiereguiluz removed the Waiting Code Merge Docs for features pending to be merged label Sep 19, 2023
@javiereguiluz javiereguiluz modified the milestones: next, 6.4, 5.4 Sep 19, 2023
@javiereguiluz
Copy link
Member

It's great that you could fix this bug that you found while contributing to docs. Thanks Jules!

@javiereguiluz javiereguiluz merged commit 4eaf1d9 into symfony:5.4 Sep 19, 2023
@HeahDude HeahDude deleted the fix/translation-default-path-config branch September 19, 2023 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants