Fork me on GitHub

pnrender, pngetstatusmsg and nocache problems  Bottom

  • Hi

    When testing the pnRender & xanthia cache, i found a glitch i think.
    pngetstatusmsg and nocache means the Smarty commands.

    The point is when you enable pnrender caching. Thats on a 900 second cache.

    The Smarty command pngetstatusmsg is for example in the News module template.

    The bug can happen in this way:
    - some pngetstatusmsg was generated elsewhere on my page from me but not shown
    on that page (because that template don't had a pngetstatusmsg cmd or it redirects to homepage).
    - the News module (or some other module) is showing the messages
    - because the module is triggered first time is it cached
    - because pngetstatusmsg is not marked as nocache our messages stay now 900 seconds

    Conclusion: the pngetstatusmsg Smarty tag should ALWAYS enclosed by nocache or even better
    trigger it by default.

    I think this can be also a problem with authkey command from Smarty?

    I am not 200& sure about this problem, so i don't have added it to the buglist ATM.



    edited by: michtoen, Jan 14, 2008 - 06:37 PM
  • Another point to the cache:

    I was setting the pnrender cache to 15 seconds and the xanthia cache for
    the Wiki and the main page to 3 seconds.

    That works pretty good when someone is hammering the site.
    Using small cache times for caching on default seems me more
    senseful as the big numbers, because they avoid when a bug like
    above happens that this bug "stays" so a long time.

  • It's my belief that all modules authors should have pngetstatusmsg in their module templates... after all, they should be setting the messages and therefore deciding how the messages are displayed.

    --
    Under Construction!
  • Hm, that was not my point - the point is that in the current way a user can
    run in a screen showing the wrong because cached status message, which is
    not what the system should do.
  • Yep, Status messages, AuthKeys and more stuff can't to be cached...
    That bug with Theme caching must be resolved icon_wink

    --
    - Mateo T. -
    Mis principios... son mis fines

This list is based on users active over the last 60 minutes.