Split Session Fields

Issue #222 resolved
Jeremy Hopkins created an issue

Checking custom profile fields requires a file include and additional data to be loaded to the user object.

This looked pretty expensive when I wrote this so added an optoin to store checks in the sessoin with a time to live.

This was originally added for the top menus and subsequent checks such as help links, news ticker etc are all looking at this single settings under "Header Menus":

  • menusession
  • menusessionttl

This is a problem for a few reasons:

  1. it is not clear in other parts of the admin settings that there is a dependency on these obscure menu settings
  2. it is not possible to isolate / test individual settings without disabling this feature across the site

There are two options here:

  1. Duplicate those settings everywhere they are needed (news ticker, help etc)
  2. Create a separate admin settings page "sessions.php" where all rules can be set, then add a note in the language pack in other pages (like help, menus etc) pointing users to that page

Comments (1)

  1. Jeremy Hopkins reporter

    This has been superceded with a move to using Moodles caching API. As that work is completed additional settings will be added.

  2. Log in to comment