Menu

#1261 de]

XOOPS_2.5.x
pending
4
2014-02-16
2013-02-28
irmtfan
No

there is a bug in XOOPS255/header.php
exit() should be commented otherwise foooter will not be loaded.
[code]
$xoopsPreload->triggerEvent('core.header.checkcache');
if ($xoTheme->checkCache()) {
$xoopsPreload->triggerEvent('core.header.cacheend');
//exit();
}
[/code]

Related

Bugs: #1261

Discussion

  • Michael Beck

    Michael Beck - 2013-03-01
    • assigned_to: trabis
     
  • Richard Griffith

    It appears the exit() would only be reached if the page had already been rendered from cache, so it seems appropriate. It has been in there for years, at least 2008 in version 2.3 (when I stopped looking.)

    Under what circumstances would footer still need to be loaded?

     
  • irmtfan

    irmtfan - 2014-02-03

    Hi Richard
    I catch your reply right now.
    Sorry for my late reply.
    The bug is occured under below condition:
    When no module is set as homepage footer.php will not be loaded in SITE/index.php
    Specifically I faced this issue in writing userlog module. Plz look at userlog/docs/readme.txt under known bugs section


    On Thu, Nov 21, 2013 12:57 AM IRST Richard Griffith wrote:

    It appears the exit() would only be reached if the page had already been rendered from cache, so it seems appropriate. It has been in there for years, at least 2008 in version 2.3 (when I stopped looking.)

    Under what circumstances would footer still need to be loaded?


    [bugs:#1261] de]

    Status: open
    Labels: xoops header preloads
    Created: Thu Feb 28, 2013 04:00 AM UTC by irmtfan
    Last Updated: Fri Mar 01, 2013 05:39 PM UTC
    Owner: trabis

    there is a bug in XOOPS255/header.php
    exit() should be commented otherwise foooter will not be loaded.
    [code]
    $xoopsPreload->triggerEvent('core.header.checkcache');
    if ($xoTheme->checkCache()) {
    $xoopsPreload->triggerEvent('core.header.cacheend');
    //exit();
    }
    [/code]


    Sent from sourceforge.net because you indicated interest in https://sourceforge.net/p/xoops/bugs/1261/

    To unsubscribe from further messages, please visit https://sourceforge.net/auth/subscriptions/

     

    Related

    Bugs: #1261

  • irmtfan

    irmtfan - 2014-02-03

    Hi Richard
    I catch your reply right now.
    Sorry for my late reply.
    The bug is occured under below condition:
    When no module is set as homepage footer.php will not be loaded in SITE/index.php
    Specifically I faced this issue in writing userlog module. Plz look at userlog/docs/readme.txt under known bugs section


    On Thu, Nov 21, 2013 12:57 AM IRST Richard Griffith wrote:

    It appears the exit() would only be reached if the page had already been rendered from cache, so it seems appropriate. It has been in there for years, at least 2008 in version 2.3 (when I stopped looking.)

    Under what circumstances would footer still need to be loaded?


    [bugs:#1261] de]

    Status: open
    Labels: xoops header preloads
    Created: Thu Feb 28, 2013 04:00 AM UTC by irmtfan
    Last Updated: Fri Mar 01, 2013 05:39 PM UTC
    Owner: trabis

    there is a bug in XOOPS255/header.php
    exit() should be commented otherwise foooter will not be loaded.
    [code]
    $xoopsPreload->triggerEvent('core.header.checkcache');
    if ($xoTheme->checkCache()) {
    $xoopsPreload->triggerEvent('core.header.cacheend');
    //exit();
    }
    [/code]


    Sent from sourceforge.net because you indicated interest in https://sourceforge.net/p/xoops/bugs/1261/

    To unsubscribe from further messages, please visit https://sourceforge.net/auth/subscriptions/

     

    Related

    Bugs: #1261

  • Richard Griffith

    • status: open --> pending
    • assigned_to: trabis --> Richard Griffith
     
  • Richard Griffith

    Accepted as suggested.

     

Log in to post a comment.