- compile locking by touching old compiled files to avoid concurrent compilations

change_log.txt

@@ -1,3 +1,6 @@
+31/03/2010
+- compile locking by touching old compiled files to avoid concurrent compilations
+
 29/03/2010
 - bugfix allow array definitions as modifier parameter
 - bugfix observe compile_check property when loading config files

libs/Smarty.class.php

@@ -129,6 +129,8 @@ class Smarty extends Smarty_Internal_Data {
     public $force_compile = false; 
     // check template for modifications?
     public $compile_check = true; 
+    // locking concurrent compiles
+    public $compile_locking = true; 
     // use sub dirs for compiled/cached files?
     public $use_sub_dirs = false; 
     // compile_error?

libs/sysplugins/smarty_internal_config.php

@@ -205,18 +205,26 @@ class Smarty_Internal_Config {
             // load compiler
             $this->compiler_object = new Smarty_Internal_Config_File_Compiler($this->smarty);
         } 
+        // compile locking
+        if ($this->smarty->compile_locking) {
+            if ($saved_timestamp = $this->getCompiledTimestamp()) {
+                touch($this->getCompiledFilepath());
+            } 
+        } 
         // call compiler
-            if ($this->compiler_object->compileSource($this)) {
+        try {
+            $this->compiler_object->compileSource($this);
+        } 
+        catch (Exception $e) {
+            // restore old timestamp in case of error
+            if ($this->smarty->compile_locking && $saved_timestamp) {
+                touch($this->getCompiledFilepath(), $saved_timestamp);
+            } 
+            throw $e;
+        } 
         // compiling succeded
         // write compiled template
         Smarty_Internal_Write_File::writeFile($this->getCompiledFilepath(), $this->getCompiledConfig(), $this->smarty);
-                // make template and compiled file timestamp match
-                touch($this->getCompiledFilepath(), $this->getTimestamp());
-            } else {
-                // error compiling template
-                throw new Exception("Error compiling template {$this->getConfigFilepath ()}");
-                return false;
-            } 
     } 
 
     /*

libs/sysplugins/smarty_internal_config_file_compiler.php

@@ -14,7 +14,6 @@
  * Main config file compiler class
  */
 class Smarty_Internal_Config_File_Compiler {
-    public $compile_error = false;
     /**
      * Initialize compiler
      */
@@ -59,12 +58,6 @@ class Smarty_Internal_Config_File_Compiler {
         $parser->doParse(0, 0);
 
         $config->compiled_config = serialize($this->config_data);
-        if (!$this->compile_error) {
-            return true;
-        } else {
-            // compilation error
-            return false;
-        } 
     } 
     /**
      * display compiler error messages without dying
@@ -107,8 +100,6 @@ class Smarty_Internal_Config_File_Compiler {
             $error_text .= ' - Unexpected "' . $this->lex->value . '", expected one of: ' . implode(' , ', $expect);
         } 
         throw new Exception($error_text); 
-        // set error flag
-        $this->compile_error = true;
     } 
 } 
 

libs/sysplugins/smarty_internal_smartytemplatecompiler.php

@@ -59,15 +59,9 @@ class Smarty_Internal_SmartyTemplateCompiler extends Smarty_Internal_TemplateCom
             list($_open_tag, $_data) = array_pop($this->_tag_stack);
             $this->trigger_template_error("unclosed {" . $_open_tag . "} tag");
         } 
-
-        if (!$this->compile_error) {
         // return compiled code
         // return str_replace(array("? >\n<?php","? ><?php"), array('',''), $this->parser->retvalue);
         return $this->parser->retvalue;
-        } else {
-            // compilation error
-            return false;
-        } 
     } 
 } 
 

libs/sysplugins/smarty_internal_template.php

@@ -255,18 +255,28 @@ class Smarty_Internal_Template extends Smarty_Internal_Data {
             $this->smarty->loadPlugin($this->resource_object->compiler_class);
             $this->compiler_object = new $this->resource_object->compiler_class($this->resource_object->template_lexer_class, $this->resource_object->template_parser_class, $this->smarty);
         } 
+        // compile locking
+        if ($this->smarty->compile_locking && !$this->resource_object->isEvaluated) {
+            if ($saved_timestamp = $this->getCompiledTimestamp()) {
+                touch($this->getCompiledFilepath());
+            } 
+        } 
         // call compiler
-        if ($this->compiler_object->compileTemplate($this)) {
+        try {
+            $this->compiler_object->compileTemplate($this);
+        } 
+        catch (Exception $e) {
+            // restore old timestamp in case of error
+            if ($this->smarty->compile_locking && !$this->resource_object->isEvaluated && $saved_timestamp) {
+                touch($this->getCompiledFilepath(), $saved_timestamp);
+            } 
+            throw $e;
+        }
         // compiling succeded
         if (!$this->resource_object->isEvaluated) {
             // write compiled template
             Smarty_Internal_Write_File::writeFile($this->getCompiledFilepath(), $this->compiled_template, $this->smarty);
         } 
-        } else {
-            // error compiling template
-            throw new Exception("Error compiling template {$this->getTemplateFilepath ()}");
-            return false;
-        } 
         if ($this->smarty->debugging) {
             Smarty_Internal_Debug::end_compile($this);
         } 

libs/sysplugins/smarty_internal_templatecompilerbase.php

@@ -50,8 +50,6 @@ class Smarty_Internal_TemplateCompilerBase {
         // flag for nochache sections
         $this->nocache = false;
         $this->tag_nocache = false; 
-        // assume successfull compiling
-        $this->compile_error = false; 
         // save template object in compiler class
         $this->template = $template;
         $this->smarty->_current_file = $this->template->getTemplateFilepath(); 
@@ -83,7 +81,6 @@ class Smarty_Internal_TemplateCompilerBase {
             // call compiler
             $_compiled_code = $this->doCompile($_content);
         } while ($this->abort_and_recompile); 
-        if (!$this->compile_error) {
         // return compiled code to template object
         if ($template->suppressFileDependency) {
             $template->compiled_template = $_compiled_code;
@@ -94,11 +91,6 @@ class Smarty_Internal_TemplateCompilerBase {
         if (isset($this->smarty->autoload_filters['post']) || isset($this->smarty->registered_filters['post'])) {
             $template->compiled_template = Smarty_Internal_Filter_Handler::runFilter('post', $template->compiled_template, $this->smarty, $template);
         } 
-            return true;
-        } else {
-            // compilation error
-            return false;
-        } 
     } 
 
     /**
@@ -430,8 +422,6 @@ class Smarty_Internal_TemplateCompilerBase {
             $error_text .= ' - Unexpected "' . $this->lex->value . '", expected one of: ' . implode(' , ', $expect);
         } 
         throw new Exception($error_text);
-        // set error flag
-        $this->compile_error = true;
     } 
 }