Skip to content

Drop useless stsl() calls#27

Open
cmb69 wants to merge 1 commit intomasterfrom
cmb/no-stsl
Open

Drop useless stsl() calls#27
cmb69 wants to merge 1 commit intomasterfrom
cmb/no-stsl

Conversation

@cmb69
Copy link
Member

@cmb69 cmb69 commented Mar 4, 2025

The function is soft-deprecated and basically a no-op since CMSimple_XH 1.7.0, and triggers a deprecation notice since CMSimple_XH 1.8. Thus we get rid of it.


@lck-git, in your mod, you've replaced stsl() with htmspecialchar(). Is that necessary? If so, that would be an important bug fix, but it may also lead to double escaping (e.g. &).

The function is soft-deprecated and basically a no-op since CMSimple_XH
1.7.0, and triggers a deprecation notice since CMSimple_XH 1.8.
Thus we get rid of it.
@cmb69 cmb69 added this to the 1.4.12 milestone Mar 4, 2025
@cmb69 cmb69 mentioned this pull request Mar 4, 2025
@lck-git
Copy link

lck-git commented Mar 5, 2025

@lck-git, in your mod, you've replaced stsl() with htmspecialchar(). Is that necessary? If so, that would be an important bug fix, but it may also lead to double escaping (e.g. &).

Well, you're asking me a question. I couldn't ask you, so I asked the KI and it presented me with this solution, among others.
It's good that you looked over it again. 👍

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.

2 participants