Object: Remove deprecated list encoding - #11978
Conversation
kergomard
left a comment
There was a problem hiding this comment.
I do not agree with your assessment. Setting titles and descriptions on UI components is the domain of ILIASObject. The renderer should never sanitize anything and it was the wrong move to (partially) introduce this in the framework as it can and should not care what data it receives as long as it is printable. The problem with this approach can easily be seen by the places where we had to introduce withoutStriptags(). So, this should currently be fixed by preventing double-encoding in the corresponding htmlentities in the renderer of the item.
|
Sorry, no, I just gave this some more thought. I'm wrong, you are right: The problem with using strip-tags was strip-tags itself not the output sanitization. I will merge this. Sorry! Merged and picked ILIAS 11 and trunk. Some more of my reasoning here, but we should give this more thought, as we are doing this completely inconsistently:
|
https://mantis.ilias.de/view.php?id=48239