diff options
author | chris <chris@jalakai.co.uk> | 2006-09-11 04:14:18 +0200 |
---|---|---|
committer | chris <chris@jalakai.co.uk> | 2006-09-11 04:14:18 +0200 |
commit | 4b5f4f4ed319790fe7f0729560616b55c4e64715 (patch) | |
tree | 6c80db4c05de0fa757005ca9298a106b0521da34 /inc/parserutils.php | |
parent | c6ea345da89cfdb4479d7e472e4dcfab8547ff2c (diff) | |
download | rpg-4b5f4f4ed319790fe7f0729560616b55c4e64715.tar.gz rpg-4b5f4f4ed319790fe7f0729560616b55c4e64715.tar.bz2 |
parser caching update
This patch primarily updates p_cached_xhtml() and p_cached_instructions() to
allow their caching logic to be surrounded by an event trigger.
p_cached_xhtml() has been rewritten as the more general p_cached_output() to
support other render output formats besides 'xhtml'. All calls to
p_cached_xhtml() have been changed to refer to the new function.
New event:
name: PARSER_CACHE_USE
data: cache object (see below)
action: determine if cache file can be used
preventable: yes
result: bool, true to use cache file, false otherwise
Cache operations have been generalised in a new class, cache, extended to
cache_parser, cache_renderer & cache_instructions. Details can be found in
inc/cache.php
For handling of above event, key properties are:
- page, if present the wiki page id,
may not always be present, e.g. when called for locale xhtml files
- file, source file
- mode, renderer mode (e.g. 'xhtml') or 'i' for instructions
Other changes:
- cache class counts cache hits against attempts, results are stored in
{cache_dir}/cache_stats.txt
- adds metadata dependency to renderer page cache
- replaces purgefile dependency for renderer cache with metadata
'relation references' (internal link) dependency for wiki pages only
darcs-hash:20060911021418-9b6ab-19601ed194b8c8e45236ab72c3e23d78bf777e6c.gz
Diffstat (limited to 'inc/parserutils.php')
-rw-r--r-- | inc/parserutils.php | 88 |
1 files changed, 35 insertions, 53 deletions
diff --git a/inc/parserutils.php b/inc/parserutils.php index f48580b0f..4b3c8c209 100644 --- a/inc/parserutils.php +++ b/inc/parserutils.php @@ -12,6 +12,7 @@ require_once(DOKU_INC.'inc/confutils.php'); require_once(DOKU_INC.'inc/pageutils.php'); require_once(DOKU_INC.'inc/pluginutils.php'); + require_once(DOKU_INC.'inc/cache.php'); /** * Returns the parsed Wikitext in XHTML for the given id and revision. @@ -38,7 +39,7 @@ function p_wiki_xhtml($id, $rev='', $excuse=true){ } }else{ if(@file_exists($file)){ - $ret = p_cached_xhtml($file); + $ret = p_cached_output($file,'xhtml',$id); }elseif($excuse){ $ret = p_locale_xhtml('newpage'); } @@ -114,48 +115,48 @@ function p_wiki_xhtml_summary($id, &$title, $rev='', $excuse=true){ */ function p_locale_xhtml($id){ //fetch parsed locale - $html = p_cached_xhtml(localeFN($id)); + $html = p_cached_output(localeFN($id)); return $html; } /** + * *** DEPRECATED *** + * + * use p_cached_output() + * * Returns the given file parsed to XHTML * * Uses and creates a cachefile * + * @deprecated * @author Andreas Gohr <andi@splitbrain.org> * @todo rewrite to use mode instead of hardcoded XHTML */ function p_cached_xhtml($file){ + return p_cached_output($file); +} + +/** + * Returns the given file parsed into the requested output format + * + * @author Andreas Gohr <andi@splitbrain.org> + * @author Chris Smith <chris@jalakai.co.uk> + */ +function p_cached_output($file, $format='xhtml', $id='') { global $conf; - $cache = getCacheName($file.$_SERVER['HTTP_HOST'].$_SERVER['SERVER_PORT'],'.xhtml'); - $purge = $conf['cachedir'].'/purgefile'; - - // check if cache can be used - $cachetime = @filemtime($cache); // 0 if not exists - - if( @file_exists($file) // does the source exist - && $cachetime > @filemtime($file) // cache is fresh - && ((time() - $cachetime) < $conf['cachetime']) // and is cachefile young enough - && !isset($_REQUEST['purge']) // no purge param was set - && ($cachetime > @filemtime($purge)) // and newer than the purgefile - && ($cachetime > @filemtime(DOKU_CONF.'dokuwiki.php')) // newer than the config file - && ($cachetime > @filemtime(DOKU_CONF.'local.php')) // newer than the local config file - && ($cachetime > @filemtime(DOKU_INC.'inc/parser/xhtml.php')) // newer than the renderer - && ($cachetime > @filemtime(DOKU_INC.'inc/parser/parser.php')) // newer than the parser - && ($cachetime > @filemtime(DOKU_INC.'inc/parser/handler.php')))// newer than the handler - { - //well then use the cache - $parsed = io_readfile($cache); - if($conf['allowdebug']) $parsed .= "\n<!-- cachefile $cache used -->\n"; - }else{ - $parsed = p_render('xhtml', p_cached_instructions($file),$info); //try to use cached instructions - if($info['cache']){ - io_saveFile($cache,$parsed); //save cachefile + $cache = new cache_renderer($id, $file, $format); + if ($cache->useCache()) { + $parsed = $cache->retrieveCache(); + if($conf['allowdebug']) $parsed .= "\n<!-- cachefile {$cache->cache} used -->\n"; + } else { + $parsed = p_render($format, p_cached_instructions($file,false,$id), $info); + + if ($info['cache']) { + $cache->storeCache($parsed); //save cachefile if($conf['allowdebug']) $parsed .= "\n<!-- no cachefile used, but created -->\n"; }else{ - @unlink($cache); //try to delete cachefile + $cache->removeCache(); //try to delete cachefile if($conf['allowdebug']) $parsed .= "\n<!-- no cachefile used, caching forbidden -->\n"; } } @@ -170,36 +171,17 @@ function p_cached_xhtml($file){ * * @author Andreas Gohr <andi@splitbrain.org> */ -function p_cached_instructions($file,$cacheonly=false){ +function p_cached_instructions($file,$cacheonly=false,$id='') { global $conf; - $cache = getCacheName($file.$_SERVER['HTTP_HOST'].$_SERVER['SERVER_PORT'],'.i'); - // check if cache can be used - $cachetime = @filemtime($cache); // 0 if not exists - - // cache forced? - if($cacheonly){ - if($cachetime){ - return unserialize(io_readfile($cache,false)); - }else{ - return array(); - } - } + $cache = new cache_instructions($id, $file); - if( @file_exists($file) // does the source exist - && $cachetime > @filemtime($file) // cache is fresh - && !isset($_REQUEST['purge']) // no purge param was set - && ($cachetime > @filemtime(DOKU_CONF.'dokuwiki.php')) // newer than the config file - && ($cachetime > @filemtime(DOKU_CONF.'local.php')) // newer than the local config file - && ($cachetime > @filemtime(DOKU_INC.'inc/parser/parser.php')) // newer than the parser - && ($cachetime > @filemtime(DOKU_INC.'inc/parser/handler.php')))// newer than the handler - { - //well then use the cache - return unserialize(io_readfile($cache,false)); - }elseif(@file_exists($file)){ + if ($cacheonly || $cache->useCache()) { + return $cache->retrieveCache(); + } else if (@file_exists($file)) { // no cache - do some work $ins = p_get_instructions(io_readfile($file)); - io_savefile($cache,serialize($ins)); + $cache->storeCache($ins); return $ins; } @@ -313,7 +295,7 @@ function p_render_metadata($id, $orig){ require_once DOKU_INC."inc/parser/metadata.php"; // get instructions - $instructions = p_cached_instructions(wikiFN($id)); + $instructions = p_cached_instructions(wikiFN($id),false,$id); // set up the renderer $renderer = & new Doku_Renderer_metadata(); |