Skip to content

Fixed bug with #switch parser function #default#370

Merged
xxyzz merged 1 commit intomainfrom
switch
Mar 5, 2025
Merged

Fixed bug with #switch parser function #default#370
xxyzz merged 1 commit intomainfrom
switch

Conversation

@kristian-clausal
Copy link
Collaborator

@kristian-clausal kristian-clausal commented Mar 4, 2025

Fixes #369

Turns out that if you have a switch statement like

{{#switch:|a=one|#default|b=three|c=two}}

the default option will be "three". Previously, we assumed that #default would only appear with an assignment, like #default=foo, but it can be part of a chain.

Turns out that if you have a switch statement like

`{{#switch:|a=one|#default|b=three|c=two}}`

the default option will be "three". Previously, we
assumed that `#default` would only appear with an assignment,
like `#default=foo`, but it can be part of a chain.
@kristian-clausal kristian-clausal requested a review from xxyzz March 4, 2025 10:50
@xxyzz xxyzz merged commit 633533e into main Mar 5, 2025
10 checks passed
@xxyzz xxyzz deleted the switch branch March 5, 2025 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[el] Template:επώνυμο returns only whitespace

2 participants