Small bug(?) in serendipity_event_suggest

[yellowled]

Hey guys,

disclaimer: I don't use serendipity_event_suggest myself, this was reported to me by a user via email. According to this user, the plugin configuration for serendipity_event_suggest does not offer an option to format the page generated by this plugin as an article (yet?), but plugin_suggest.tpl includes the following code:

Code: Select all

{if $plugin_suggest_articleformat}
<div class="serendipity_Entry_Date">
    <h3 class="serendipity_date">{$plugin_suggest_name}</h3>
{/if}

This <div> apparently is never opened since $plugin_suggest_articleformat can't be set in the configuration, but it is closed with a </div> which is not inside an {if}-statement later. This results in a spare </div> which can easily screw up layouts. So this {if}-statement should be commented out until the plugin actually gets that feature.

YL

[garvinhicking]

Hi!

Ah, great catch. Thanks a lot, just committed a fix.

Regards,
Garvin