[geeklog-devel] Calendar bug

Mark R. Evans mevans at ecsnet.com
Fri Dec 21 08:30:10 EST 2007


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 


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

More information about the geeklog-devel mailing list