ILIAS icon indicating copy to clipboard operation
ILIAS copied to clipboard

T&A 46392: Streamline usage of getAdditionalTableName

Open thojou opened this issue 1 month ago • 2 comments

This PR relates to Mantis issue https://mantis.ilias.de/view.php?id=46392.

As discussed in the ticket, there is only one method in the core that can handle an array return type for the getAdditionalTableName method. All other usages assume the return type to be a string.
Given that, we streamline the behavior and also expect a string return type in deleteAdditionalTableData.

With this change, there may be implications for plugin developers. However, since the signature has already been changed, the main BC adjustments have already been introduced.

If a question type plugin requires multiple additional tables, the plugin developer must:

a) refactor their database structure into a single additional table
b) override methods to read and write additional data.

Best regards,
@thojou

thojou avatar Dec 03 '25 11:12 thojou

Hi, we would like to resolve the problem stated in https://mantis.ilias.de/view.php?id=46392 with this PR. We want to merge the PR after the JourFixe on 12 JAN 2026. So right now, the label is to inform all the persons involved with the T&A & Question Plugins.

dsstrassner avatar Dec 03 '25 11:12 dsstrassner

Jour Fixe, 08 DEC 2025: Suggested bug fix has been accepted. PR can be merged after next JF at 12 JAN 2026.

matthiaskunkel avatar Dec 08 '25 12:12 matthiaskunkel