Test: Rename variable for ass logging

[mjansenDatabay]

This PR suggests picking a commit from release_8 to solve Mantis Issue https://mantis.ilias.de/view.php?id=47705 .

The original fix cleaned up variable naming when creating logging entries in the test component. Unfortunately, during the great PHP 8 compliance campaign of ILIAS 8, this tiny but noble commit apparently became collateral damage somewhere between strict types, deprecated warnings, and existential null checks.

The variables have now safely returned from the battlefield and are once again reporting for logging duty in release_9.

[dsstrassner]

Hi @mjansenDatabay,

thanks for your PR.
@ILIAS-eLearning/technical-board We (as in T&A code authorities) will review this PR if the TB agrees to merge this into ILIAS 9. I hope this doesn't open a can of worms. Because it's not really a security fix.

Kind regards

@dsstrassner

[rfalkenstein]

Hi @dsstrassner , we would highly appreciate a review by the authorities. Afterwards we, as TB, would take over the merge into release_9. This fix is a cherry-pick from ILIAS 8 btw.
Thanks for the immediate reaction!
Best regards,
@rfalkenstein

For the Technical Board of the ILIAS Society

[kergomard]

Hi all

I looked at the changes and they look correct and I do not see any problem with it.

I explicitly do not approve this PR as I will not take any responsibility for any regressions caused on a security only branch by introducing fixes that have nothing to do with security.

Best,
@kergomard