From 7fe908c50c8dbc5cc05f571dbe11d66141caacd4 Mon Sep 17 00:00:00 2001 From: Marius Cramer <m.cramer@pixcept.de> Date: Thu, 14 Nov 2013 09:01:22 -0500 Subject: [PATCH] Cleaning up code to match coding guidelines --- server/lib/classes/tpl_cache.inc.php | 287 +++++++++++++++++++++++++++++---------------------------- 1 files changed, 147 insertions(+), 140 deletions(-) diff --git a/server/lib/classes/tpl_cache.inc.php b/server/lib/classes/tpl_cache.inc.php index a399946..4bf75fa 100644 --- a/server/lib/classes/tpl_cache.inc.php +++ b/server/lib/classes/tpl_cache.inc.php @@ -20,167 +20,174 @@ * @access public */ + class tplc extends tpl { -/*-----------------------------------------------------------------------------\ + /*-----------------------------------------------------------------------------\ | DO NOT TOUCH ANYTHING IN THIS CLASS, IT MAY NOT WORK OTHERWISE | \-----------------------------------------------------------------------------*/ - var $_cache = 1; // tells vlibTemplate that we're caching - var $_cachefile; // full path to current cache file (even if it doesn't yet exist) - var $_cacheexists; // has this file been cached before - var $_cachefilelocked; // is this file currently locked whilst writing - var $_cachefiledir; // dir of current cache file - var $_clearcache = 0; + var $_cache = 1; // tells vlibTemplate that we're caching + var $_cachefile; // full path to current cache file (even if it doesn't yet exist) + var $_cacheexists; // has this file been cached before + var $_cachefilelocked; // is this file currently locked whilst writing + var $_cachefiledir; // dir of current cache file + var $_clearcache = 0; - /** - * FUNCTION: clearCache - * will unset a file, and set $this->_cacheexists to 0. - * - * @access public - * @return boolean - */ - function clearCache() { - $this->_clearcache = 1; - return true; - } - - /** - * FUNCTION: recache - * alias for clearCache(). - * - * @access public - * @return boolean - */ - function recache() { - return $this->clearCache(); - } - - /** - * FUNCTION: setCacheLifeTime - * sets the lifetime of the cached file - * - * @param int $int number of seconds to set lifetime to - * @access public - * @return boolean - */ - function setCacheLifeTime($int = null) { - if ($int == null || !is_int($int)) return false; - if ($int == 0) $int = 60; - if ($int == -1) $int = 157680000; // set to 5 yrs time - $this->OPTIONS['CACHE_LIFETIME'] = $int; - return true; - } - - /** - * FUNCTION: setCacheExtension - * sets the extention of the cache file - * - * @param str $str name of new cache extention - * @access public - * @return boolean - */ - function setCacheExtension($str = null) { - if ($str == null || !preg_match('/^[a-z0-9]+$/', strtolower($str))) return false; - $this->OPTIONS['CACHE_EXTENSION'] = strtolower($str); - return true; - } + /** + * FUNCTION: clearCache + * will unset a file, and set $this->_cacheexists to 0. + * + * @access public + * @return boolean + */ + function clearCache() { + $this->_clearcache = 1; + return true; + } -/*----------------------------------------\ + /** + * FUNCTION: recache + * alias for clearCache(). + * + * @access public + * @return boolean + */ + function recache() { + return $this->clearCache(); + } + + + /** + * FUNCTION: setCacheLifeTime + * sets the lifetime of the cached file + * + * @param int $int number of seconds to set lifetime to + * @access public + * @return boolean + */ + function setCacheLifeTime($int = null) { + if ($int == null || !is_int($int)) return false; + if ($int == 0) $int = 60; + if ($int == -1) $int = 157680000; // set to 5 yrs time + $this->OPTIONS['CACHE_LIFETIME'] = $int; + return true; + } + + + /** + * FUNCTION: setCacheExtension + * sets the extention of the cache file + * + * @param str $str name of new cache extention + * @access public + * @return boolean + */ + function setCacheExtension($str = null) { + if ($str == null || !preg_match('/^[a-z0-9]+$/', strtolower($str))) return false; + $this->OPTIONS['CACHE_EXTENSION'] = strtolower($str); + return true; + } + + + /*----------------------------------------\ Private Functions -----------------------------------------*/ - /** - * FUNCTION: _checkCache - * checks if there's a cache, if there is then it will read the cache file as the template. - */ - function _checkCache ($tmplfile) { - $this->_cachefile = $this->_getFilename($tmplfile); - if ($this->_clearcache) { - if (file_exists($this->_cachefile)) unlink($this->_cachefile); - return false; - } - if (file_exists($this->_cachefile)) { - $this->_cacheexists = 1; + /** + * FUNCTION: _checkCache + * checks if there's a cache, if there is then it will read the cache file as the template. + */ + function _checkCache ($tmplfile) { + $this->_cachefile = $this->_getFilename($tmplfile); + if ($this->_clearcache) { + if (file_exists($this->_cachefile)) unlink($this->_cachefile); + return false; + } - // if it's expired - if ((filemtime($this->_cachefile) + $this->OPTIONS['CACHE_LIFETIME']) < date ('U') - || filectime($this->_cachefile) < filemtime($tmplfile)) { - $this->_cacheexists = 0; - return false; // so that we know to recache - } - else { - return true; - } + if (file_exists($this->_cachefile)) { + $this->_cacheexists = 1; - } else { - $this->_cacheexists = 0; - return false; - } - } + // if it's expired + if ((filemtime($this->_cachefile) + $this->OPTIONS['CACHE_LIFETIME']) < date('U') + || filectime($this->_cachefile) < filemtime($tmplfile)) { + $this->_cacheexists = 0; + return false; // so that we know to recache + } + else { + return true; + } + + } else { + $this->_cacheexists = 0; + return false; + } + } - /** - * FUNCTION: _getFilename - * gets the full pathname for the cached file - * - */ - function _getFilename($tmplfile) { - return $this->OPTIONS['CACHE_DIRECTORY'].'/'.md5('vlibCachestaR'.realpath($tmplfile)).'.'.$this->OPTIONS['CACHE_EXTENSION']; - } + /** + * FUNCTION: _getFilename + * gets the full pathname for the cached file + * + */ + function _getFilename($tmplfile) { + return $this->OPTIONS['CACHE_DIRECTORY'].'/'.md5('vlibCachestaR'.realpath($tmplfile)).'.'.$this->OPTIONS['CACHE_EXTENSION']; + } - /** - * FUNCTION: _createCache - * creates the cached file - * - */ - function _createCache($data) { - $cache_file = $this->_cachefile; - if(!$this->_prepareDirs($cache_file)) return false; // prepare all of the directories - $f = fopen ($cache_file, "w"); - flock($f, 2); // set an EXclusive lock - if (!$f) vlibTemplateError::raiseError('VT_ERROR_NO_CACHE_WRITE',KILL,$cache_file); - fputs ($f, $data); // write the parsed string from vlibTemplate - flock($f, 3); // UNlock file - fclose ($f); - touch ($cache_file); - return true; - } + /** + * FUNCTION: _createCache + * creates the cached file + * + */ + function _createCache($data) { + $cache_file = $this->_cachefile; + if(!$this->_prepareDirs($cache_file)) return false; // prepare all of the directories - /** - * FUNCTION: _prepareDirs - * prepares the directory structure - * - */ - function _prepareDirs($file) { - if (empty($file)) die('no filename'); //do error in future - $filepath = dirname($file); - if (is_dir($filepath)) return true; + $f = fopen($cache_file, "w"); + flock($f, 2); // set an EXclusive lock + if (!$f) vlibTemplateError::raiseError('VT_ERROR_NO_CACHE_WRITE', KILL, $cache_file); + fputs($f, $data); // write the parsed string from vlibTemplate + flock($f, 3); // UNlock file + fclose($f); + touch($cache_file); + return true; + } - $dirs = preg_split('/[\\/]/', $filepath); - $currpath; - foreach ($dirs as $dir) { - $currpath .= $dir .'/'; - $type = @filetype($currpath); - ($type=='link') and $type = 'dir'; - if ($type != 'dir' && $type != false && !empty($type)) { - vlibTemplateError::raiseError('VT_ERROR_WRONG_CACHE_TYPE',KILL,'directory: '.$currpath.', type: '.$type); - } - if ($type == 'dir') { - continue; - } - else { - $s = @mkdir($currpath, 0775); - if (!$s) vlibTemplateError::raiseError('VT_ERROR_CACHE_MKDIR_FAILURE',KILL,'directory: '.$currpath); - } - } - return true; - } + /** + * FUNCTION: _prepareDirs + * prepares the directory structure + * + */ + function _prepareDirs($file) { + if (empty($file)) die('no filename'); //do error in future + $filepath = dirname($file); + if (is_dir($filepath)) return true; + + $dirs = preg_split('/[\\/]/', $filepath); + $currpath; + foreach ($dirs as $dir) { + $currpath .= $dir .'/'; + $type = @filetype($currpath); + + ($type=='link') and $type = 'dir'; + if ($type != 'dir' && $type != false && !empty($type)) { + vlibTemplateError::raiseError('VT_ERROR_WRONG_CACHE_TYPE', KILL, 'directory: '.$currpath.', type: '.$type); + } + if ($type == 'dir') { + continue; + } + else { + $s = @mkdir($currpath, 0775); + if (!$s) vlibTemplateError::raiseError('VT_ERROR_CACHE_MKDIR_FAILURE', KILL, 'directory: '.$currpath); + } + } + return true; + } } // -- end vlibTemplateCache class -?> \ No newline at end of file +?> -- Gitblit v1.9.1