- 🇳🇱Netherlands rudy de kok
Did anyone find a solution for this? I'm also running into the issue that a rewritten label cannot be translated.
- Status changed to Needs review
8 months ago 8:33pm 18 April 2024 - 🇵🇱Poland ad0z
I faced the same issue and I dived into the topic. The problem is that view exposed form, which is altered by Better Exposed Filters contains translated options already, so we are not able to read original labels in the alter function..
Another option would be using keys to determine options to overwrite, but I see some points that it users labels now, as using keys may be tricky sometimes as well when you have some ids as keys value. I guess there is no clear choice.I've prepared a patch which adds "Rewrite the text displayed based on key" checkbox which allows using key|Label values to overwrite labels.
- Status changed to Needs work
8 months ago 2:08pm 19 April 2024 - Assigned to ad0z
- Merge request !68Issue #3253994: Rewriting the text displayed for "- All -" label should occur before translation → (Closed) created by ad0z
- Issue was unassigned.
- Status changed to Needs review
8 months ago 10:23pm 19 April 2024 - 🇵🇱Poland ad0z
@smustgrave pushed mentioned changes + unit tests to cover it to issue fork, created merge request as well.
- Status changed to Needs work
8 months ago 11:55pm 19 April 2024 - 🇺🇸United States smustgrave
Test looks great! But if we are adding a new configuration key it'll need a schema entry and update hook.
- Status changed to Needs review
8 months ago 11:37am 20 April 2024 - 🇵🇱Poland ad0z
> But if we are adding a new configuration key it'll need a schema entry and update hook.
I've applied mentioned changes.
-
smustgrave →
committed fbdd6cf8 on 6.0.x
Issue #3253994 by ad0z: Rewriting the text displayed for "- All -" label...
-
smustgrave →
committed fbdd6cf8 on 6.0.x
- 🇺🇸United States smustgrave
@ad0z excellent test cases! Ran the test-only feature and it shows the coverage is good.
Tested manually and can confirm the replacement is working excellently.
Update ran fine too.
- Status changed to Fixed
8 months ago 11:12pm 30 April 2024 -
smustgrave →
committed fbdd6cf8 on 7.0.x
Issue #3253994 by ad0z: Rewriting the text displayed for "- All -" label...
-
smustgrave →
committed fbdd6cf8 on 7.0.x
Automatically closed - issue fixed for 2 weeks with no activity.