Skip to content

fix: Unescape HTML entities in component index checkbox tree#948

Merged
moustachu merged 2 commits intodevelopfrom
fix/unescape-html-entities-filter-checkbox-tree
Feb 24, 2026
Merged

fix: Unescape HTML entities in component index checkbox tree#948
moustachu merged 2 commits intodevelopfrom
fix/unescape-html-entities-filter-checkbox-tree

Conversation

@moustachu
Copy link
Member

🎩 Description

When creating a category name with special characters (like " or '), the checkbox filter in the component index on the front end are labeled escape HTML entities (like ").

image

see example here (admin sign in needed)

This fix add a deface override on the checkbox tree shared template to unescape these entities.

Stef-Rousset
Stef-Rousset previously approved these changes Feb 23, 2026
Copy link
Contributor

@Stef-Rousset Stef-Rousset left a comment

Choose a reason for hiding this comment

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

LGTM !

Copy link
Collaborator

@BarbaraOliveira13 BarbaraOliveira13 left a comment

Choose a reason for hiding this comment

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

I tested locally and It's seems not fix the bug.
@Stef-Rousset Can/Have you test locally ?

Image

@Stef-Rousset
Copy link
Contributor

hello @BarbaraOliveira13 , yes I tested it locally and it worked. As it is a deface override, may be you should compile deface and test again

@BarbaraOliveira13
Copy link
Collaborator

just for tracking: I can't reproduce but @Stef-Rousset can so we decided to bypass = ⚠️ Focus Test after deployment !

@moustachu moustachu merged commit 9975118 into develop Feb 24, 2026
10 checks passed
@moustachu moustachu deleted the fix/unescape-html-entities-filter-checkbox-tree branch February 24, 2026 16:29
@moustachu moustachu restored the fix/unescape-html-entities-filter-checkbox-tree branch February 24, 2026 16:29
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.

3 participants