Skip to content

[FIX] UICore: encode cmd- and base-class parameters - #11967

Open
thibsy wants to merge 1 commit into
ILIAS-eLearning:trunkfrom
srsolutionsag:fix/12/uicore-encode-parameters
Open

[FIX] UICore: encode cmd- and base-class parameters#11967
thibsy wants to merge 1 commit into
ILIAS-eLearning:trunkfrom
srsolutionsag:fix/12/uicore-encode-parameters

Conversation

@thibsy

@thibsy thibsy commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

This works around the fact the Init component strips some
characters from the request URL for security reasons. It
uses a very hideous way to replace \ of PSR-4 namespaces
to ensure we can still use them with ilCtrl.

@marvimarv

Copy link
Copy Markdown

@thibsy could you please add the Mantis reference to this PR? According to contributing.md, bugfix PRs should carry the ticket number in the title (for example 12345 - encode cmd- and base-class parameters) and link the ticket in the description. If there is no Mantis ticket for this yet, please let us know. Thanks!

For the Technical Board of the ILIAS association

@marvimarv marvimarv added bugfix php Pull requests that update Php code labels Aug 25, 2026

@chfsx chfsx left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks a lot!

@thibsy

thibsy commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

@chfsx wait, that one's not finished yet! apparently this still gets stripped by init... probably have to change delimiter.

This works around the fact the Init component strips some
characters from the request URL for security reasons. It
uses a very hideous way to replace `\` of PSR-4 namespaces
to ensure we can still use them with `ilCtrl`.
@thibsy
thibsy force-pushed the fix/12/uicore-encode-parameters branch from c05645e to aa341a0 Compare August 25, 2026 14:10
@thibsy thibsy added improvement and removed bugfix labels Aug 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement php Pull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants