Link text is always displayed due to using isset() for checking config option.

Comments

How about:

<?php
isset($this->options['timeline_config']['link_text_enabled']) && $this->options['timeline_config']['link_text_enabled']
?>

This way, it handles unset or no value gracefully and then checks for a non false/null value.

Sounds fine. I guess we shouldn't assume even a mandatory field has a value. Although it will have a value one way or the other as soon as the config form is saved for the first time.

StatusFileSize
new782 bytes

Done.

Status:Needs review» Fixed

And this is what you get when you fix old issues in a hurry :(
Good catch, applied the patch and attributed imclean.

Status:Fixed» Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.