[geeklog-devel] Calendar bug
Mark R. Evans
mevans at ecsnet.com
Fri Dec 21 08:30:10 EST 2007
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
>
>
More information about the geeklog-devel
mailing list