extern/egroupware
1
0
Fork 1
mirror of https://github.com/EGroupware/egroupware.git synced 2024-11-08 17:14:36 +01:00
Code Issues Packages Projects Releases Wiki Activity

* Api: keep session small / quick to load by limiting number of cached link-titles

Browse Source
This commit is contained in:
Ralf Becker 2017-04-04 09:58:18 +02:00
parent d15597224a
commit 4aafcf7e2b
1 changed files with 25 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
api/src/Link.php
View File

@ -167,6 +167,11 @@ class Link extends Link\Storage
),
),
);
/**
* Max. number of titles stored in session (older once get removed)
*/
const TITLE_CACHE_SIZE = 500;
/**
* Caches link titles for a better performance
*
@ -174,6 +179,10 @@ class Link extends Link\Storage
*/
private static $title_cache = array();
/**
* Max. number of titles stored in session (older once get removed)
*/
const FILE_ACCESS_CACHE_SIZE = 1000;
/**
* Cache file access permissions
*
@ -301,7 +310,17 @@ class Link extends Link\Storage
if (isset($GLOBALS['egw']->session)) // eg. cron-jobs use it too, without any session
{
//error_log(__METHOD__.'() items in title-cache: '.count(self::$title_cache).' file-access-cache: '.count(self::$file_access_cache));
if (count(self::$title_cache > self::TITLE_CACHE_SIZE))
{
self::$title_cache = array_slice(self::$title_cache, -self::TITLE_CACHE_SIZE);
}
Cache::setSession(__CLASS__, 'link_title_cache', self::$title_cache);
if (count(self::$file_access_cache > self::FILE_ACCESS_CACHE_SIZE))
{
self::$file_access_cache = array_slice(self::$file_access_cache, -self::FILE_ACCESS_CACHE_SIZE);
}
Cache::setSession(__CLASS__, 'link_file_access_cache', self::$file_access_cache);
}
}
@ -1538,6 +1557,10 @@ class Link extends Link\Storage
switch($type)
{
case 'title':
if ($app == self::VFS_APPNAME)
{
return null; // do not cache file titles, they are just the names
}
return self::$title_cache[$app.':'.$id];
case 'file_access':
return self::$file_access_cache[$app.':'.$id];
@ -1561,7 +1584,8 @@ class Link extends Link\Storage
public static function set_cache($app,$id,$title,$file_access=null)
{
//error_log(__METHOD__."($app,$id,$title,$file_access)");
if (!is_null($title))
// do not cache file titles, they are just the names
if (!is_null($title) && $app != self::VFS_APPNAME)
{
self::$title_cache[$app.':'.$id] = $title;
}