From: Jason Woofenden Date: Fri, 22 Dec 2006 02:59:24 +0000 (-0500) Subject: run.php works even through symbolic links X-Git-Url: https://jasonwoof.com/gitweb/?p=wfpl.git;a=commitdiff_plain;h=1b701682c788072eb0d469fc77b2925fc8ef16b5 run.php works even through symbolic links running sub-templates clears their sub-templates --- diff --git a/run.php b/run.php index 716e364..1b5b19b 100644 --- a/run.php +++ b/run.php @@ -56,9 +56,20 @@ function run_php($basename = false) { $html_file = "$basename.html"; $php_file = "$basename.php"; } else { - chdir('../..'); - $html_file = $_SERVER['REDIRECT_URL']; - $html_file = ereg_replace('.*/', '', $html_file); + + # first we must find the file and directory that was actually requested + # (before we messed it up with mod_rewrite) + $fs_path = $_SERVER['DOCUMENT_ROOT'] . $_SERVER['REDIRECT_URL']; + $split = strrpos($fs_path, '/'); # find the last slash + $directory = substr($fs_path, 0, $split); + $file = substr($fs_path, $split + 1); + + if($file == '') { + $file = 'index.html'; + } + + chdir($directory); + $html_file = $file; $php_file = ereg_replace('\.html$', '.php', $html_file); } if($php_file != $html_file && file_exists($php_file)) { diff --git a/template.php b/template.php index 862f7a6..3c0e1e0 100644 --- a/template.php +++ b/template.php @@ -41,13 +41,14 @@ require_once('code/wfpl/misc.php'); # to get read_whole_file() class tem { var $keyval; # an array containing key/value pairs - var $filename; # template filename (sometimes unset) + var $filename; # template filename (sometimes not set) var $template; # contents of template var $sub_templates; # tag-name/template-string pairs + var $sub_subs; # key: sub-template name value: array of names of the sub-templates of this one # initialize variables function tem() { - $this->keyval = array('' => '~'); + $this->keyval = array('' => '~'); # so that ~~ in the template creates a single ~ $this->sub_templates = array(); } @@ -56,6 +57,11 @@ class tem { $this->keyval[$key] = $value; } + # clear a value. Functionally equivalent to set($key, '') but cleaner and more efficient + function clear($key) { + unset($this->keyval[$key]); + } + # grab a value you stuck in earlier with set() function get($key) { return $this->keyval[$key]; @@ -66,10 +72,17 @@ class tem { # this can be used. function sub($sub_template_name) { $this->keyval[$sub_template_name] .= template_run($this->sub_templates[$sub_template_name], $this->keyval); + + # after running a sub-template, clear its sub-templates + if(isset($this->sub_subs[$sub_template_name])) { + foreach($this->sub_subs[$sub_template_name] as $sub_sub) { + $this->clear($sub_sub); + } + } } # this is used by tem::load() and should be otherwise useless - function _load(&$in, &$out) { + function _load(&$in, &$out, &$parents, &$parent) { while($in) { # scan for one of: 1) the begining of a sub-template 2) the end of this one 3) the end of the file $n = strpos($in, '', substr($in, 0, 12)) == 0) { $in = substr($in, 12); + $parent = array_pop($parents); return; } @@ -95,10 +109,19 @@ class tem { # this limits sub_template names to 50 chars if(ereg('^', substr($in, 0, 65), $matches)) { list($start_tag, $tag_name) = $matches; + + # keep track of the tree + if(!isset($this->sub_subs[$parent])) { + $this->sub_subs[$parent] = array(); + } + array_push($this->sub_subs[$parent], $tag_name); + array_push($parents, $parent); + $parent = $tag_name; + $out .= '~' . $tag_name . '~'; $in = substr($in, strlen($start_tag)); $this->sub_templates[$tag_name] = ''; - $this->_load($in, $this->sub_templates[$tag_name]); + $this->_load($in, $this->sub_templates[$tag_name], $parents, $parent); } else { # it's not a start tag or end tag, so let's pass it through: $out .= substr($in, 0, 5); @@ -114,7 +137,9 @@ class tem { $this->filename = $filename; $tmp = read_whole_file($filename); $this->template = ''; - $this->_load($tmp, $this->template); + $parents = array('top_level_subs' => array()); + $parent = 'top_level_subs'; + $this->_load($tmp, $this->template, $parents, $parent); } # Run the template. Pass a filename, or a string, unless you've already