Skip to content
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

EZP-32338: List url aliases for all available translations #1711

Merged
merged 1 commit into from
Feb 22, 2021
Merged

Conversation

adamwojs
Copy link
Member

Question Answer
Tickets https://issues.ibexa.co/browse/EZP-32338
Bug fix? yes
New feature? no
BC breaks? no
Tests pass? yes
Doc needed? no
License GPL-2.0

Currently user is able to manage translations in all available languages but the list of url aliases is filter out (incompletely^) to ezpublish.system.<scope>.languages, which is misleading for users (see linked JIRA issue). This PR changes this behaviour to always show all available url aliases.

^url alias for language out side of the ezpublish.system.<scope>.languages will be displayed on the list when it's exactly the same as one of the configured languages:

image

Checklist:

  • Coding standards ($ composer fix-cs)
  • Ready for Code Review

@adamwojs adamwojs added the Bug label Feb 22, 2021
@adamwojs adamwojs self-assigned this Feb 22, 2021
@micszo micszo self-assigned this Feb 22, 2021
Copy link
Member

@micszo micszo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The table in the back-office now lists all URLs.
Tested on eZ Platform EE v2.5.16 with diff.

@micszo micszo removed their assignment Feb 22, 2021
@lserwatka lserwatka merged commit ff9011e into 1.5 Feb 22, 2021
@lserwatka lserwatka deleted the ezp_32338 branch February 22, 2021 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
6 participants