[geeklog-devel] Calendar bug

[Mark R. Evans]

Dirk,

Yep, you are correct.  After looking at the code a little closer, maybe a 
better overall fix would be to remove the COM_siteFooter() calls from all 
the case options and simply have one prior to the echo $display.  This 
would be a bit more consistent with how the COM_siteHeader() call is made 
(only once before the case statement).

Looks like either approach will solve the issue which is the important 
thing.

Thanks!
Mark

On Fri, 21 Dec 2007, Dirk Haun wrote:

> Mark R. Evans wrote:
>
>> I believe that $display .= COM_siteFooter(); needs to go after the end of
>> the switch.
>
> Nope. Several of the other cases already have a COM_siteFooter() call
> (and would have two then) and some of the others are returning redirects
> where the COM_siteFooter() is not needed.
>
> So I think it should only be added for the 'addentry' case:
>
> case 'addentry':
>    $display .= plugin_submit_calendar($mode);
>    $display .= COM_siteFooter();
>    break;
>
> bye, Dirk
>
>
> -- 
> http://www.haun-online.de/
> http://spam.tinyweb.net/
>
> _______________________________________________
> geeklog-devel mailing list
> geeklog-devel at lists.geeklog.net
> http://eight.pairlist.net/mailman/listinfo/geeklog-devel
>
>