#31325 - Do not show TOC if it is disabled

[padvincenzo]

Hi, I've added this condition before the method isTocNecessary() so that the TOC won't be displayed if it is deactivated from the "Menu" settings of the module.

Also, I think that the method isTocNecessary() is somehow messed up, I didn't get it.

[klees]

Hi @padvincenzo,

I have classified this as "improvement", since there is no ticket linked here.

If your PR contains a bugfix please reference the number of the mantis ticket in the title 12345 - To many spaces, link the ticket in the description and label the ticket with bugfix.

Rules for Contributors

Thanks a lot for contributing to ILIAS!

[padvincenzo]

Hi @klees, This PR fix the bug mentioned on mantis: https://mantis.ilias.de/view.php?id=31325

[kergomard]

Dear @alex40724 While checking on old open PRs in the frame of our Technical Board Meeting we stumbled over this one that is getting a little long in the tooth. Could you please either close it with an explanation, give feedback or merge? Thank you very much! @kergomard in the name of the TB

[alex40724]

Sorry for the delay. See my comment in https://mantis.ilias.de/view.php?id=30027

The mentioned setting never controlled the side/slate tree. It is a menu setting, which became obsolete with ILIAS 6. See also the comment in the bug report.