template_compile

Complexity

Total Complexity

149

Size/Duplication

Total Lines	867
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	2	1
C	compile_tag_define()	0	47	12
A	generate_block_varref()	0	13	1
A	_tpl_load_file()	0	33	4
A	compile_write()	0	18	3
C	compile_var_tags()	0	69	10
A	remove_php_tags()	0	10	1
A	compile_tag_include()	0	9	2
A	compile_tag_include_php()	0	3	1
A	minify()	0	26	4
A	generate_block_data_ref()	0	31	5
F	compile()	0	165	34
B	compile_tag_block()	0	100	9
F	compile_tag_if()	0	151	53
B	_parse_is_expr()	0	61	9

<?php
/**
*
* @package phpBB3
* @version $Id$
* @copyright (c) 2005 phpBB Group, sections (c) 2001 ispi of Lincoln Inc
* @license http://opensource.org/licenses/gpl-license.php GNU Public License
*
* Modified by Gorlum to work within http://supernova.ws
*
*/

/**
* @ignore
*/
if (!defined('INSIDE'))
{
  exit;
}

/**
* Extension of template class - Functions needed for compiling templates only.
*
* psoTFX, phpBB Development Team - Completion of file caching, decompilation
* routines and implementation of conditionals/keywords and associated changes
*
* The interface was inspired by PHPLib templates,  and the template file (formats are
* quite similar)
*
* The keyword/conditional implementation is currently based on sections of code from
* the Smarty templating engine (c) 2001 ispi of Lincoln, Inc. which is released
* (on its own and in whole) under the LGPL. Section 3 of the LGPL states that any code
* derived from an LGPL application may be relicenced under the GPL, this applies
* to this source
*
* DEFINE directive inspired by a request by Cyberalien
*
* @package phpBB3
*/
class template_compile
{
  var $template;

  // Various storage arrays
  var $block_names = array();
  var $block_else_level = array();

  /**
   * template_compile constructor.
   *
   * @param template $template
   */
  public function __construct($template) {
    $this->template = $template;
  }

  /**
  * Load template source from file
  * @access private
  */
  function _tpl_load_file($handle, $store_in_db = false)

It is generally recommended to explicitly declare the visibility for methods.

  {
    // Try and open template for read
    if (!file_exists($this->template->files[$handle]))
    {
      if (!file_exists($this->template->files_inherit[$handle]))
      {
//        pdump($handle);
////        global $debug;
//        var_dump((debug_backtrace()));
//        die();

        SN::$gc->debug->warning("template->_tpl_load_file(): File {$this->template->files[$handle]} does not exist or is empty");

        return;
        trigger_error("template->_tpl_load_file(): File {$this->template->files[$handle]} does not exist or is empty", E_USER_ERROR);

trigger_error('template->_tpl_load_file(): File '.$this->template->files[$handle].' does not exist or is empty', E_USER_ERROR) is not reachable.

      }
      else
      {
        $this->template->files[$handle] = $this->template->files_inherit[$handle];
      }
    }

    $html = $this->minify(@file_get_contents($this->template->files[$handle]));

    $this->template->compiled_code[$handle] = $this->compile(trim($html));

It seems like $html can also be of type false; however, parameter $string of trim() does only seem to accept string, maybe add an additional type check?

    // Actually compile the code now.
    $this->compile_write($handle, $this->template->compiled_code[$handle]);

    // Store in database if required...
    if ($store_in_db)
    {
//      global $db, $user;
//
//      $sql_ary = array(
//        'template_id'     => $this->template->files_template[$handle],
//        'template_filename'   => $this->template->filename[$handle],
//        'template_included'   => '',
//        'template_mtime'    => time(),
//        'template_data'     => trim(@file_get_contents($this->template->files[$handle])),
//      );
//
//      $sql = 'INSERT INTO ' . STYLES_TEMPLATE_DATA_TABLE . ' ' . $db->sql_build_array('INSERT', $sql_ary);
//      $db->sql_query($sql);
    }
  }

  /**
  * Remove any PHP tags that do not belong, these regular expressions are derived from
  * the ones that exist in zend_language_scanner.l
  * @access private
  */
  function remove_php_tags(&$code)

It is generally recommended to explicitly declare the visibility for methods.

  {
    // This matches the information gathered from the internal PHP lexer
    $match = array(
      '#<([\?%])=?.*?\1>#s',
      '#<script\s+language\s*=\s*(["\']?)php\1\s*>.*?</script\s*>#s',
      '#<\?php(?:\r\n?|[ \n\t]).*?\?>#s'
    );

    $code = preg_replace($match, '', $code);
  }

  /**
  * The all seeing all doing compile method. Parts are inspired by or directly from Smarty
  * @access private
  */
  function compile($code, $no_echo = false, $echo_var = '')

It is generally recommended to explicitly declare the visibility for methods.

  {
    if ($echo_var)
    {
      global $$echo_var;
    }

    // Remove any "loose" php ... we want to give admins the ability
    // to switch on/off PHP for a given template. Allowing unchecked
    // php is a no-no. There is a potential issue here in that non-php
    // content may be removed ... however designers should use entities
    // if they wish to display < and >
    $this->remove_php_tags($code);

    // Pull out all block/statement level elements and separate plain text
    preg_match_all('#<!-- PHP -->(.*?)<!-- ENDPHP -->#s', $code, $matches);
    $php_blocks = $matches[1];
    $code = preg_replace('#<!-- PHP -->.*?<!-- ENDPHP -->#s', '<!-- PHP -->', $code);

    preg_match_all('#<!-- INCLUDE (\{\$?[A-Z0-9\-_]+\}|[a-zA-Z0-9\_\-\+\./]+) -->#', $code, $matches);
    $include_blocks = $matches[1];
    if($include_blocks)
    {
      foreach($include_blocks as &$included_file)
      {
        $included_file .= '.tpl.html';
      }
    }
    $code = preg_replace('#<!-- INCLUDE (?:\{\$?[A-Z0-9\-_]+\}|[a-zA-Z0-9\_\-\+\./]+) -->#', '<!-- INCLUDE -->', $code);

    preg_match_all('#<!-- INCLUDEPHP ([a-zA-Z0-9\_\-\+\./]+) -->#', $code, $matches);
    $includephp_blocks = $matches[1];
    $code = preg_replace('#<!-- INCLUDEPHP [a-zA-Z0-9\_\-\+\./]+ -->#', '<!-- INCLUDEPHP -->', $code);

    preg_match_all('#<!-- ([^<].*?) (.*?)? ?-->#', $code, $blocks, PREG_SET_ORDER);

    $text_blocks = preg_split('#<!-- [^<].*? (?:.*?)? ?-->#', $code);

    for ($i = 0, $j = sizeof($text_blocks); $i < $j; $i++)
    {
      $this->compile_var_tags($text_blocks[$i]);
    }
    $compile_blocks = array();

    for ($curr_tb = 0, $tb_size = sizeof($blocks); $curr_tb < $tb_size; $curr_tb++)
    {
      $block_val = &$blocks[$curr_tb];

      switch ($block_val[1])
      {
        case 'BEGIN':
          $this->block_else_level[] = false;
          $compile_blocks[] = '<?php ' . $this->compile_tag_block($block_val[2]) . ' ?>';
        break;

        case 'BEGINELSE':
          $this->block_else_level[sizeof($this->block_else_level) - 1] = true;
          $compile_blocks[] = '<?php }} else { ?>';
        break;

        case 'END':
          array_pop($this->block_names);
          $compile_blocks[] = '<?php ' . ((array_pop($this->block_else_level)) ? '}' : '}}') . ' ?>';
        break;

        case 'IF':
          $compile_blocks[] = '<?php ' . $this->compile_tag_if($block_val[2], false) . ' ?>';
        break;

        case 'ELSE':
          $compile_blocks[] = '<?php } else { ?>';
        break;

        case 'ELSEIF':
          $compile_blocks[] = '<?php ' . $this->compile_tag_if($block_val[2], true) . ' ?>';
        break;

        case 'ENDIF':
          $compile_blocks[] = '<?php } ?>';
        break;

        case 'DEFINE':
          $compile_blocks[] = '<?php ' . $this->compile_tag_define($block_val[2], true) . ' ?>';
        break;

        case 'UNDEFINE':
          $compile_blocks[] = '<?php ' . $this->compile_tag_define($block_val[2], false) . ' ?>';
        break;

        case 'INCLUDE':
          $temp = array_shift($include_blocks);

          // Dynamic includes
          // Cheap match rather than a full blown regexp, we already know
          // the format of the input so just use string manipulation.
          if ($temp[0] == '{')
          {
            $file = false;

            if ($temp[1] == '$')
            {
              $var = substr($temp, 2, -1);
              //$file = $this->template->_tpldata['DEFINE']['.'][$var];
              $temp = "\$this->_tpldata['DEFINE']['.']['$var']";
            }
            else
            {
              $var = substr($temp, 1, -1);
              //$file = $this->template->_rootref[$var];
              $temp = "\$this->_rootref['$var']";
            }
          }
          else
          {
            $file = $temp;
          }

          $compile_blocks[] = '<?php ' . $this->compile_tag_include($temp) . ' ?>';

          // No point in checking variable includes
          if ($file)
          {
            $this->template->_tpl_include($file, false);
          }
        break;

        case 'INCLUDEPHP':
          $compile_blocks[] = (SN::$config->tpl_allow_php) ? '<?php ' . $this->compile_tag_include_php(array_shift($includephp_blocks)) . ' ?>' : '';
        break;

        case 'PHP':
          $compile_blocks[] = (SN::$config->tpl_allow_php) ? '<?php ' . array_shift($php_blocks) . ' ?>' : '';
        break;

        default:
          $this->compile_var_tags($block_val[0]);
          $trim_check = trim($block_val[0]);
          $compile_blocks[] = (!$no_echo) ? ((!empty($trim_check)) ? $block_val[0] : '') : ((!empty($trim_check)) ? $block_val[0] : '');
        break;
      }
    }

    $template_php = '';
    for ($i = 0, $size = sizeof($text_blocks); $i < $size; $i++)
    {
      $trim_check_text = trim($text_blocks[$i]);
      $template_php .= (!$no_echo) ? (($trim_check_text != '') ? $text_blocks[$i] : '') . ((isset($compile_blocks[$i])) ? $compile_blocks[$i] : '') : (($trim_check_text != '') ? $text_blocks[$i] : '') . ((isset($compile_blocks[$i])) ? $compile_blocks[$i] : '');
    }

    // Remove unused opening/closing tags
    $template_php = str_replace(' ?><?php ', ' ', $template_php);

    // Now add a newline after each php closing tag which already has a newline
    // PHP itself strips a newline if a closing tag is used (this is documented behaviour) and it is mostly not intended by style authors to remove newlines
    $template_php = preg_replace('#\?\>([\r\n])#', '?>\1\1', $template_php);

    // There will be a number of occasions where we switch into and out of
    // PHP mode instantaneously. Rather than "burden" the parser with this
    // we'll strip out such occurences, minimising such switching
    if ($no_echo)
    {
      return "\$$echo_var .= '" . $template_php . "'";
    }

    return $template_php;
  }

  /**
  * Compile variables
  * @access private
  */
  function compile_var_tags(&$text_blocks)

It is generally recommended to explicitly declare the visibility for methods.

  {
    // including $lang variable
    // global $lang, $config; // NOT NEDEED - $lang now is global!

    // change template varrefs into PHP varrefs
    $varrefs = array();

    // This one will handle varrefs WITH namespaces
    preg_match_all('#\{((?:[a-z0-9\-_]+\.)+)(\$)?([A-Z0-9\-_]+)(?:(\|.+?)*)?\}#', $text_blocks, $varrefs, PREG_SET_ORDER);

    foreach ($varrefs as $var_val)
    {
      $namespace = $var_val[1];
      $varname = $var_val[3];
      $new = $this->generate_block_varref($namespace, $varname, $var_val[2]);

      if(!empty($var_val[4])) {
        $new = \Ptl\PtlVariableDecorator::decorate($var_val[0], $new, $this->template);
      }

      $new = "<?php echo $new; ?>";

      $text_blocks = str_replace($var_val[0], $new, $text_blocks);
    }

    // This will handle the remaining root-level varrefs

    // Prefix R_ means "render this block again". Only one level of rendering supported to avoid circular references
    if (strpos($text_blocks, '{R_') !== false) {
      $text_blocks = preg_replace(/** @lang RegExp */'#\{R_([a-zA-Z0-9\-_\.\$\[\]]+)\}#', /** @lang PHP */'<?php $this->reRender(\'\\1\'); ?>', $text_blocks);
    }

    // transform vars prefixed by I_ into skin-specific images with context
    if (strpos($text_blocks, '{I_') !== false && is_callable(array('SkinV2', 'image_url'))) {
      $text_blocks = preg_replace(/** @lang RegExp */'#\{I_(.+?)\}#', /** @lang PHP */'<?php echo SkinV2::image_url(\'\\1\', $this); ?>', $text_blocks);
    }

    // transform vars prefixed by C_ into global config value
    if (strpos($text_blocks, '{C_') !== false)
    {
      $text_blocks = preg_replace(/** @lang RegExp */'#\{C_([a-zA-Z0-9\-_]+)\[([a-zA-Z0-9\-_]*?)\]\}#', /** @lang PHP */'<?php echo ((isset($this->_rootref[\'C_\\1\'][\'\\2\'])) ? $this->_rootref[\'C_\\1\'][\'\\2\'] : ((isset(SN::$config[\'\\1\'][\'\\2\'])) ? SN::$config[\'\\1\'][\'\\2\'] : \'{ \\1[\\2] }\')); ?>', $text_blocks);
      $text_blocks = preg_replace(/** @lang RegExp */'#\{C_([a-zA-Z0-9\-_]+)\}#', /** @lang PHP */'<?php echo ((isset($this->_rootref[\'C_\\1\'])) ? $this->_rootref[\'C_\\1\'] : ((isset(SN::$config[\'\\1\'])) ? SN::$config[\'\\1\'] : \'{ C_\\1 }\')); ?>', $text_blocks);
    }
    // transform vars prefixed by D_ into global defined constant
    if (strpos($text_blocks, '{D_') !== false)
    {
      $text_blocks = preg_replace(/** @lang RegExp */'#\{D_([a-zA-Z0-9\-_]+)\}#', /** @lang PHP */'<?php echo ((isset($this->_rootref[\'D_\\1\'])) ? $this->_rootref[\'D_\\1\'] : ((defined(\'\\1\')) ? \\1 : \'{ D_\\1 }\')); ?>', $text_blocks);
    }
    // transform vars prefixed by L_ into their language variable pendant if nothing is set within the tpldata array
    if (strpos($text_blocks, '{L_') !== false)
    {
      $text_blocks = preg_replace(/** @lang RegExp */'#\{L_([a-zA-Z0-9\-_]+)\[D_([a-zA-Z0-9\-_]*?)\]\}#', /** @lang PHP */'<?php echo ((isset($this->_rootref[\'L_\\1\'][\\2])) ? $this->_rootref[\'L_\\1\'][\\2] : ((isset(SN::$lang[\'\\1\'][\\2])) ? SN::$lang[\'\\1\'][\\2] : \'{ \\1[\\2] }\')); ?>', $text_blocks);
      $text_blocks = preg_replace(/** @lang RegExp */'#\{L_([a-zA-Z0-9\-_]+)\[([a-zA-Z0-9\-_]*?)\]\}#', /** @lang PHP */'<?php echo ((isset($this->_rootref[\'L_\\1\'][\'\\2\'])) ? $this->_rootref[\'L_\\1\'][\'\\2\'] : ((isset(SN::$lang[\'\\1\'][\'\\2\'])) ? SN::$lang[\'\\1\'][\'\\2\'] : \'{ \\1[\\2] }\')); ?>', $text_blocks);
      $text_blocks = preg_replace(/** @lang RegExp */'#\{L_([a-zA-Z0-9\-_]+)\}#', /** @lang PHP */'<?php echo ((isset($this->_rootref[\'L_\\1\'])) ? $this->_rootref[\'L_\\1\'] : ((isset(SN::$lang[\'\\1\'])) ? SN::$lang[\'\\1\'] : \'{ L_\\1 }\')); ?>', $text_blocks);
    }

    // Handle addslashed language variables prefixed with LA_
    // If a template variable already exist, it will be used in favor of it...
    if (strpos($text_blocks, '{LA_') !== false)
    {
      $text_blocks = preg_replace(/** @lang RegExp */'#\{LA_([a-zA-Z0-9\-_]+)\}#', /** @lang PHP */'<?php echo ((isset($this->_rootref[\'LA_\\1\'])) ? $this->_rootref[\'LA_\\1\'] : ((isset($this->_rootref[\'L_\\1\'])) ? addslashes($this->_rootref[\'L_\\1\']) : ((isset(SN::$lang[\'\\1\'])) ? addslashes(SN::$lang[\'\\1\']) : \'{ LA_\\1 }\'))); ?>', $text_blocks);
    }

    // Handle remaining varrefs
    $text_blocks = preg_replace(/** @lang RegExp */'#\{([a-zA-Z0-9\-_]+)\}#', /** @lang PHP */'<?php echo (isset($this->_rootref[\'\\1\'])) ? $this->_rootref[\'\\1\'] : \'\'; ?>', $text_blocks);
    $text_blocks = preg_replace(/** @lang RegExp */'#\{\$([a-zA-Z0-9\-_]+)\}#', /** @lang PHP */'<?php echo (isset($this->_tpldata[\'DEFINE\'][\'.\'][\'\\1\'])) ? $this->_tpldata[\'DEFINE\'][\'.\'][\'\\1\'] : \'\'; ?>', $text_blocks);

    return;
  }

  /**
  * Compile blocks
  * @access private
  */
  function compile_tag_block($tag_args)

It is generally recommended to explicitly declare the visibility for methods.

  {
    $no_nesting = false;

    // Is the designer wanting to call another loop in a loop?
    if (strpos($tag_args, '!') === 0)
    {
      // Count the number if ! occurrences (not allowed in vars)
      $no_nesting = substr_count($tag_args, '!');
      $tag_args = substr($tag_args, $no_nesting);
    }

    // Allow for control of looping (indexes start from zero):
    // foo(2)    : Will start the loop on the 3rd entry
    // foo(-2)   : Will start the loop two entries from the end
    // foo(3,4)  : Will start the loop on the fourth entry and end it on the fifth
    // foo(3,-4) : Will start the loop on the fourth entry and end it four from last
    if (preg_match('#^([^()]*)\(([\-\d]+)(?:,([\-\d]+))?\)$#', $tag_args, $match))
    {
      $tag_args = $match[1];

      if ($match[2] < 0)
      {
        $loop_start = '($_' . $tag_args . '_count ' . $match[2] . ' < 0 ? 0 : $_' . $tag_args . '_count ' . $match[2] . ')';
      }
      else
      {
        $loop_start = '($_' . $tag_args . '_count < ' . $match[2] . ' ? $_' . $tag_args . '_count : ' . $match[2] . ')';
      }

      if (strlen($match[3]) < 1 || $match[3] == -1)
      {
        $loop_end = '$_' . $tag_args . '_count';
      }
      else if ($match[3] >= 0)
      {
        $loop_end = '(' . ($match[3] + 1) . ' > $_' . $tag_args . '_count ? $_' . $tag_args . '_count : ' . ($match[3] + 1) . ')';
      }
      else //if ($match[3] < -1)
      {
        $loop_end = '$_' . $tag_args . '_count' . ($match[3] + 1);
      }
    }
    else
    {
      $loop_start = 0;
      $loop_end = '$_' . $tag_args . '_count';
    }

    $tag_template_php = '';
    array_push($this->block_names, $tag_args);

    if ($no_nesting !== false)
    {
      // We need to implode $no_nesting times from the end...
      $block = array_slice($this->block_names, -$no_nesting);
    }
    else
    {
      $block = $this->block_names;
    }

    if (sizeof($block) < 2)
    {
      // Block is not nested.
      $tag_template_php = '$_' . $tag_args . "_count = (isset(\$this->_tpldata['$tag_args'])) ? sizeof(\$this->_tpldata['$tag_args']) : 0;";
      $varref = "\$this->_tpldata['$tag_args']";
    }
    else
    {
      // This block is nested.
      // Generate a namespace string for this block.
      $namespace = implode('.', $block);

      // Get a reference to the data array for this block that depends on the
      // current indices of all parent blocks.
      $varref = $this->generate_block_data_ref($namespace, false);

      // Create the for loop code to iterate over this block.
      $tag_template_php = '$_' . $tag_args . '_count = (isset(' . $varref . ')) ? sizeof(' . $varref . ') : 0;';
    }

    $tag_template_php .= 'if ($_' . $tag_args . '_count) {';

    /**
    * The following uses foreach for iteration instead of a for loop, foreach is faster but requires PHP to make a copy of the contents of the array which uses more memory
    * <code>
    * if (!$offset)
    * {
    *   $tag_template_php .= 'foreach (' . $varref . ' as $_' . $tag_args . '_i => $_' . $tag_args . '_val){';
    * }
    * </code>
    */

    $tag_template_php .= 'for ($_' . $tag_args . '_i = ' . $loop_start . '; $_' . $tag_args . '_i < ' . $loop_end . '; ++$_' . $tag_args . '_i){';
//    $tag_template_php .= '$this->_block_counter["'. $tag_args . '"] = $_' . $tag_args . '_i;';
    $tag_template_php .= '$_'. $tag_args . '_val = &' . $varref . '[$_'. $tag_args. '_i];';
    $tag_template_php .= '$this->_block_value["'. $tag_args . '"] = &' . $varref . '[$_'. $tag_args. '_i];';

    return $tag_template_php;
  }

  /**
  * Compile IF tags - much of this is from Smarty with
  * some adaptions for our block level methods
  * @access private
  */
  function compile_tag_if($tag_args, $elseif)

It is generally recommended to explicitly declare the visibility for methods.

  {
    // Tokenize args for 'if' tag.
    preg_match_all('/(?:
      "[^"\\\\]*(?:\\\\.[^"\\\\]*)*"         |
      \'[^\'\\\\]*(?:\\\\.[^\'\\\\]*)*\'     |
      [(),]                                  |
      [^\s(),]+)/x', $tag_args, $match);

    $tokens = $match[0];
    $is_arg_stack = array();

    for ($i = 0, $size = sizeof($tokens); $i < $size; $i++)
    {
      $token = &$tokens[$i];

      switch ($token)
      {
        case '!==':
        case '===':
        case '<<':
        case '>>':
        case '|':
        case '^':
        case '&':
        case '~':
        case ')':
        case ',':
        case '+':
        case '-':
        case '*':
        case '/':
        case '@':
        break;

        case '==':
        case 'eq':
          $token = '==';
        break;

        case '!=':
        case '<>':
        case 'ne':
        case 'neq':
          $token = '!=';
        break;

        case '<':
        case 'lt':
          $token = '<';
        break;

        case '<=':
        case 'le':
        case 'lte':
          $token = '<=';
        break;

        case '>':
        case 'gt':
          $token = '>';
        break;

        case '>=':
        case 'ge':
        case 'gte':
          $token = '>=';
        break;

        case '&&':
        case 'and':
          $token = '&&';
        break;

        case '||':
        case 'or':
          $token = '||';
        break;

        case '!':
        case 'not':
          $token = '!';
        break;

        case '%':
        case 'mod':
          $token = '%';
        break;

        case '(':
          array_push($is_arg_stack, $i);
        break;

        case 'is':
          $is_arg_start = ($tokens[$i-1] == ')') ? array_pop($is_arg_stack) : $i-1;
          $is_arg = implode(' ', array_slice($tokens, $is_arg_start, $i - $is_arg_start));

          $new_tokens = $this->_parse_is_expr($is_arg, array_slice($tokens, $i+1));

          array_splice($tokens, $is_arg_start, sizeof($tokens), $new_tokens);

          $i = $is_arg_start;

        // no break

        default:
          if (preg_match('#^((?:[a-z0-9\-_]+\.)+)?(\$)?(?=[A-Za-z])([A-Za-z0-9\-_]+)#s', $token, $varrefs))
          {
            $token = (!empty($varrefs[1])) ? $this->generate_block_data_ref(substr($varrefs[1], 0, -1), true, $varrefs[2]) . '[\'' . $varrefs[3] . '\']' : (($varrefs[2]) ? '$this->_tpldata[\'DEFINE\'][\'.\'][\'' . $varrefs[3] . '\']' : '$this->_rootref[\'' . $varrefs[3] . '\']');
          }
          else if (preg_match('#^\.((?:[a-z0-9\-_]+\.?)+)$#s', $token, $varrefs))
          {
            // Allow checking if loops are set with .loopname
            // It is also possible to check the loop count by doing <!-- IF .loopname > 1 --> for example
            $blocks = explode('.', $varrefs[1]);

            // If the block is nested, we have a reference that we can grab.
            // If the block is not nested, we just go and grab the block from _tpldata
            if (sizeof($blocks) > 1)
            {
              $block = array_pop($blocks);
              $namespace = implode('.', $blocks);
              $varref = $this->generate_block_data_ref($namespace, true);

              // Add the block reference for the last child.
              $varref .= "['" . $block . "']";
            }
            else
            {
              $varref = '$this->_tpldata';

              // Add the block reference for the last child.
              $varref .= "['" . $blocks[0] . "']";
            }
            $token = "(empty($varref) ? 0 : sizeof($varref))";
          }
          else if (!empty($token))
          {
            $token = '(' . $token . ')';
          }

        break;
      }
    }

    // If there are no valid tokens left or only control/compare characters left, we do skip this statement
    if (!sizeof($tokens) || str_replace(array(' ', '=', '!', '<', '>', '&', '|', '%', '(', ')'), '', implode('', $tokens)) == '')
    {
      $tokens = array('false');
    }
    return (($elseif) ? '} else if (' : 'if (') . (implode(' ', $tokens) . ') { ');
  }

  /**
  * Compile DEFINE tags
  * @access private
  */
  function compile_tag_define($tag_args, $op)

It is generally recommended to explicitly declare the visibility for methods.

  {
    preg_match('#^((?:[a-z0-9\-_]+\.)+)?\$(?=[A-Z])([A-Z0-9_\-]*)(?: = (\'?)([^\']*)(\'?))?$#', $tag_args, $match);

    if (empty($match[2]) || (!isset($match[4]) && $op))
    {
      return '';
    }

    if (!$op)
    {
      return 'unset(' . (($match[1]) ? $this->generate_block_data_ref(substr($match[1], 0, -1), true, true) . '[\'' . $match[2] . '\']' : '$this->_tpldata[\'DEFINE\'][\'.\'][\'' . $match[2] . '\']') . ');';
    }

    // Are we a string?
    if ($match[3] && $match[5])
    {
      $match[4] = str_replace(array('\\\'', '\\\\', '\''), array('\'', '\\', '\\\''), $match[4]);

      // Compile reference, we allow template variables in defines...
      $match[4] = $this->compile($match[4]);

      // Now replace the php code
      $match[4] = "'" . str_replace(array('<?php echo ', '; ?>'), array("' . ", " . '"), $match[4]) . "'";
    }
    else
    {
      preg_match('#true|false|\.#i', $match[4], $type);

      switch (strtolower($type[0]))
      {
        case 'true':
        case 'false':
          $match[4] = strtoupper($match[4]);
        break;

        case '.':
          $match[4] = doubleval($match[4]);
        break;

        default:
          $match[4] = intval($match[4]);
        break;
      }
    }

    return (($match[1]) ? $this->generate_block_data_ref(substr($match[1], 0, -1), true, true) . '[\'' . $match[2] . '\']' : '$this->_tpldata[\'DEFINE\'][\'.\'][\'' . $match[2] . '\']') . ' = ' . $match[4] . ';';
  }

  /**
  * Compile INCLUDE tag
  * @access private
  */
  function compile_tag_include($tag_args)

It is generally recommended to explicitly declare the visibility for methods.

  {
    // Process dynamic includes
    if ($tag_args[0] == '$')
    {
      return "if (isset($tag_args)) { \$this->_tpl_include($tag_args); }";
    }

    return "\$this->_tpl_include('$tag_args');";
  }

  /**
  * Compile INCLUDE_PHP tag
  * @access private
  */
  function compile_tag_include_php($tag_args)

It is generally recommended to explicitly declare the visibility for methods.

  {
    return "\$this->_php_include('$tag_args');";
  }

  /**
  * parse expression
  * This is from Smarty
  * @access private
  */
  function _parse_is_expr($is_arg, $tokens)

It is generally recommended to explicitly declare the visibility for methods.

  {
    $expr_end = 0;
    $negate_expr = false;

    if (($first_token = array_shift($tokens)) == 'not')
    {
      $negate_expr = true;
      $expr_type = array_shift($tokens);
    }
    else
    {
      $expr_type = $first_token;
    }

    switch ($expr_type)
    {
      case 'even':
        if (@$tokens[$expr_end] == 'by')
        {
          $expr_end++;
          $expr_arg = $tokens[$expr_end++];
          $expr = "!(($is_arg / $expr_arg) % $expr_arg)";
        }
        else
        {
          $expr = "!($is_arg & 1)";
        }
      break;

      case 'odd':
        if (@$tokens[$expr_end] == 'by')
        {
          $expr_end++;
          $expr_arg = $tokens[$expr_end++];
          $expr = "(($is_arg / $expr_arg) % $expr_arg)";
        }
        else
        {
          $expr = "($is_arg & 1)";
        }
      break;

      case 'div':
        if (@$tokens[$expr_end] == 'by')
        {
          $expr_end++;
          $expr_arg = $tokens[$expr_end++];
          $expr = "!($is_arg % $expr_arg)";
        }
      break;
    }

    if ($negate_expr)
    {
      $expr = "!($expr)";

The variable $expr does not seem to be defined for all execution paths leading up to this point.

    }

    array_splice($tokens, 0, $expr_end, $expr);

    return $tokens;
  }

  /**
   * Generates a reference to the given variable inside the given (possibly nested)
   * block namespace. This is a string of the form:
   * ' . $this->_tpldata['parent'][$_parent_i]['$child1'][$_child1_i]['$child2'][$_child2_i]...['varname'] . '
   * It's ready to be inserted into an "echo" line in one of the templates.
   * NOTE: expects a trailing "." on the namespace.
   *
   * @param string $namespace
   * @param string $varname
   * @param bool   $defop
   *
   * @return string
   */
  private function generate_block_varref($namespace, $varname, $defop = false)
  {
    // Strip the trailing period.
    $namespace = substr($namespace, 0, -1);

    // Get a reference to the data block for this namespace.
    $varref = $this->generate_block_data_ref($namespace, true, $defop);
    // Prepend the necessary code to stick this in an echo line.

    // Append the variable reference.
    $varref .= "['$varname']";

    return $varref;
  }

  /**
  * Generates a reference to the array of data values for the given
  * (possibly nested) block namespace. This is a string of the form:
  * $this->_tpldata['parent'][$_parent_i]['$child1'][$_child1_i]['$child2'][$_child2_i]...['$childN']
  *
  * If $include_last_iterator is true, then [$_childN_i] will be appended to the form shown above.
  * NOTE: does not expect a trailing "." on the blockname.
  * @access private
  */
  function generate_block_data_ref($blockname, $include_last_iterator, $defop = false)

It is generally recommended to explicitly declare the visibility for methods.

  {
    // Get an array of the blocks involved.
    $blocks = explode('.', $blockname);
    $blockcount = sizeof($blocks) - 1;

    // DEFINE is not an element of any referenced variable, we must use _tpldata to access it
    if ($defop)
    {
      $varref = '$this->_tpldata[\'DEFINE\']';
      // Build up the string with everything but the last child.
      for ($i = 0; $i < $blockcount; $i++)
      {
        $varref .= "['" . $blocks[$i] . "'][\$_" . $blocks[$i] . '_i]';
      }
      // Add the block reference for the last child.
      $varref .= "['" . $blocks[$blockcount] . "']";
      // Add the iterator for the last child if requried.
      if ($include_last_iterator)
      {
        $varref .= '[$_' . $blocks[$blockcount] . '_i]';
      }
      return $varref;
    }
    else if ($include_last_iterator)
    {
      return '$_'. $blocks[$blockcount] . '_val';
    }
    else
    {
      return '$_'. $blocks[$blockcount - 1] . '_val[\''. $blocks[$blockcount]. '\']';
    }
  }

  /**
  * Write compiled file to cache directory
  * @access private
  */
  function compile_write($handle, $data)

It is generally recommended to explicitly declare the visibility for methods.

  {
    $filename = $this->template->cachepath . str_replace('/', '.', $this->template->filename[$handle]) . DOT_PHP_EX;

    $data = "<?php if (!defined('INSIDE')) exit;" . ((strpos($data, '<?php') === 0) ? substr($data, 5) : ' ?>' . $data);

    if ($fp = @fopen($filename, 'wb'))
    {
      @flock($fp, LOCK_EX);

It seems like you do not handle an error condition for flock(). This can introduce security issues, and is generally not recommended.

      @fwrite ($fp, $data);

It seems like you do not handle an error condition for fwrite(). This can introduce security issues, and is generally not recommended.

      @flock($fp, LOCK_UN);
      @fclose($fp);

It seems like you do not handle an error condition for fclose(). This can introduce security issues, and is generally not recommended.

      //phpbb_chmod($filename, CHMOD_READ | CHMOD_WRITE);
      chmod($filename, 0710);
    }

    return;
  }

  // Gorlum's minifier BOF
  /**
  * Minifies template w/i PHP code by removing extra spaces
  * @access private
  */
  function minify($html)

It is generally recommended to explicitly declare the visibility for methods.

  {
    if(!SN::$config->tpl_minifier)
    {
      return $html;
    }

    // TODO: Match <code> and <pre> too - in separate arrays
    preg_match_all('/(<script[^>]*?>.*?<\/script>)/si', $html, $pre);
    $html = preg_replace('/(<script[^>]*?>.*?<\/script>)/si', '#pre#', $html);
    //$html = preg_replace('#<!-[^\[].+->#', '', $html);
    //$html = preg_replace('/[\r\n\t]+/', ' ', $html);
    $html = preg_replace('/>[\s]*</', '><', $html); // Strip spacechars between tags
    $html = preg_replace('/[\s]+/', ' ', $html); // Replace several spacechars with one space
    if(!empty($pre[0]))
    {
      foreach($pre[0] as $tag)
      {
        $tag = preg_replace('/^\ *\/\/[^\<]*?$/m', ' ', $tag); // Strips comments - except those that contains HTML comment inside
        $tag = preg_replace('/[\ \t]{2,}/', ' ', $tag); // Replace several spaces by one
        $tag = preg_replace('/\s{2,}/', "\r\n", $tag); // Replace several linefeeds by one
        $html = preg_replace('/#pre#/', $tag, $html,1);
      }
    }

    return $html;
  }
  // Gorlum's minifier EOF

}