Skip to content

Mail: Use non broken tag input options#11034

Open
lscharmer wants to merge 1 commit intoILIAS-eLearning:release_11from
lscharmer:11/mail-use-nonbroken-tag
Open

Mail: Use non broken tag input options#11034
lscharmer wants to merge 1 commit intoILIAS-eLearning:release_11from
lscharmer:11/mail-use-nonbroken-tag

Conversation

@lscharmer
Copy link
Contributor

@lscharmer lscharmer commented Feb 2, 2026

This PR fixes Mantis Bug: https://mantis.ilias.de/view.php?id=46596
This PR must be merged after PR #10798 is merged.

@lscharmer lscharmer added bugfix php Pull requests that update Php code labels Feb 2, 2026
@mjansenDatabay
Copy link
Contributor

Hi @kergomard ,

do you think changing this in "User Search API" is the right way, or do you prefer to use withoutStripTags in the consumer code (here: "Mail")?

Best regards,
Michael

@kergomard
Copy link
Contributor

Hi @mjansenDatabay

I'm unsure about the right way to go:

  • If we are sure that the value still always gets escaped when it is used in the interface (if I'm understanding it right, it is), then we can do this here.
  • If we cannot, we should not.

My understanding right now is, that the first is the case and then we can keep this here.

Best,
@kergomard

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants