Fixes

class/class.modules.php

@@ -14,17 +14,28 @@
 	{
 		public $_modules = [];
 
+		private static $instance = null;
 
-		function __construct()
+
+		function __construct ()
 		{
 			$this->_modules = $this->getModules();
 		}
 
 
+		public static function init ()
+		{
+			if (is_null(self::$instance))
+				self::$instance = new AVE_Module;
+
+			return self::$instance;
+		}
+
+
 		/**
 		 * Метод, который обрабатывает все module.php и записывает как свойство класса списки модулей
 		 */
-		function getModules()
+		function getModules ()
 		{
 			$modules = [];
 
@@ -61,7 +72,7 @@
 				if (! is_dir($module_dir))
 					continue;
 
-				$module = array();
+				$module = array ();
 
 				// Если не удалось подключить основной файл модуля module.php - Фиксируем ошибку
 				if (! (is_file($module_dir . '/info.php') && @include_once($module_dir . '/info.php')))
@@ -122,23 +133,25 @@
 		 * Метод, преданзначеный для выода модулей
 		 *
 		 */
-		function moduleList()
+		function moduleList ()
 		{
 			global $AVE_DB, $AVE_Template;
 
-			$assign				= array(); // Массив для передачи в Smarty
-			$errors				= array(); // Массив с ошибками
+			$assign				= []; // Массив для передачи в Smarty
+			$errors				= []; // Массив с ошибками
 
 			// Получаем список всех шаблонов
-			$sql = $AVE_DB->Query("
+			$sql = "
 				SELECT
 					Id,
 					template_title
 				FROM
 					" . PREFIX . "_templates
-			");
+			";
+
+			$query = $AVE_DB->Query($sql);
 
-			while ($row = $sql->FetchRow())
+			while ($row = $query->FetchRow())
 				$assign['all_templates'][$row->Id] = htmlspecialchars($row->template_title, ENT_QUOTES);
 
 			// Заголовок: Автор модуля
@@ -147,7 +160,7 @@
 			// Получаем список всех модулей
 			$modules = $this->_modules;
 
-			foreach ($modules as $module)
+			foreach ($modules AS $module)
 			{
 				$module['info'] = $module['ModuleDescription'] . (! $module['ModuleAutor']
 					? '<br /><br />'
@@ -246,7 +259,7 @@
 		 * Метод, предназначенный для обновления в БД информации о шаблонах модулей
 		 *
 		 */
-		function moduleOptionsSave()
+		function moduleOptionsSave ()
 		{
 			global $AVE_DB;
 
@@ -264,7 +277,7 @@
 				");
 			}
 
-			$this->clearModulesCache();
+			$this->clearModulesCache ();
 
 			// Выполянем обновление страницы со списком модулей
 			header('Location:index.php?do=modules&cp=' . SESSION);
@@ -276,7 +289,7 @@
 		 * Метод, предназанченный для установки или переустановки модуля
 		 *
 		 */
-		function moduleInstall()
+		function moduleInstall ()
 		{
 			global $AVE_DB, $AVE_Template;
 
@@ -359,12 +372,14 @@
 		 * Метод, предназначенный для обновления модуля при увеличении номера версии модуля
 		 *
 		 */
-		function moduleUpdate()
+		function moduleUpdate ()
 		{
 			global $AVE_DB, $AVE_Template;
 
 			// Подключаем файл с запросами к БД для данного модуля
-			$module_sql_update = array();
+			$module_sql_update = [];
+
+			$module = [];
 
 			$sql_file = BASE_DIR . '/modules/' . MODULE_PATH . '/sql.php';
 
@@ -378,16 +393,16 @@
 				// Выполняем запросы обновления модуля
 				// из массива $module_sql_update файла sql.php
 				foreach ($module_sql_update as $sql)
-				{
 					$AVE_DB->Query(str_replace('%%PRFX%%', PREFIX, $sql));
 			}
-			}
 			// Обновляем модуль, если в нем не применяется (отсутствует) файл sql.php
 			elseif (file_exists($mod_file) && file_exists($sql_file) === false)
 			{
-				include_once ($mod_file);
+				include ($mod_file);
 
-				$AVE_DB->Query("
+				if (isset ($module) && ! empty($module))
+				{
+					$sql = "
 						UPDATE
 							" . PREFIX . "_module
 						SET
@@ -402,7 +417,11 @@
 							ModuleStatus		= '1'
 						WHERE
 							ModuleSysName = '" . MODULE_PATH . "'
-				");
+					";
+
+					$AVE_DB->Query($sql);
+				}
+
 			}
 			// Сохраняем системное сообщение в журнал
 			reportLog ($AVE_Template->get_config_vars('MODULES_ACTION_UPDATE') . ' (' . MODULE_PATH . ')');
@@ -419,7 +438,7 @@
 		 * Метод, предназанченный для удаление модуля
 		 *
 		 */
-		function moduleDelete()
+		function moduleDelete ()
 		{
 			global $AVE_DB, $AVE_Template;
 
@@ -462,7 +481,7 @@
 		 * Метод, предназначенный для отключения/включение модуля в Панели управления
 		 *
 		 */
-		function moduleStatusChange()
+		function moduleStatusChange ()
 		{
 			global $AVE_DB, $AVE_Template;
 
@@ -490,7 +509,7 @@
 					ModuleSysName = '" . MODULE_PATH . "'
 			");
 
-			$this->clearModulesCache();
+			$this->clearModulesCache ();
 
 			// Сохраняем системное сообщение в журнал
 			reportLog ((($ModuleStatus == "0")
@@ -503,7 +522,7 @@
 		}
 
 
-		function moduleRemove($dir)
+		function moduleRemove ($dir)
 		{
 			global $AVE_DB, $AVE_Template;
 
@@ -514,7 +533,7 @@
 			foreach ($files as $file)
 			{
 				if(substr($file, -1) == '/')
-					moduleRemove($file);
+					$this->moduleRemove($file);
 				else
 					unlink($file);
 			}

inc/init.php

@@ -433,5 +433,5 @@
 
 	//-- Класс Модулей
 	require (BASE_DIR . '/class/class.modules.php');
-	$AVE_Module = new AVE_Module;
+	$AVE_Module = AVE_Module::init();
 ?>

lib/Smarty/Config_File.class.php

@@ -22,14 +22,14 @@
  * smarty-discussion-subscribe@googlegroups.com 
  *
  * @link http://www.smarty.net/
- * @version 2.6.26
+ * @version 2.6.25-dev
  * @copyright Copyright: 2001-2005 New Digital Group, Inc.
  * @author Andrei Zmievski <andrei@php.net>
  * @access public
  * @package Smarty
  */
 
-/* $Id: Config_File.class.php 3149 2009-05-23 20:59:25Z monte.ohrt $ */
+/* $Id$ */
 
 /**
  * Config file reading class
@@ -73,7 +73,7 @@ class Config_File {
      *
      * @param string $config_path (optional) path to the config files
      */
-	function __construct($config_path = NULL)
+    public function __construct($config_path = NULL)
     {
         if (isset($config_path))
             $this->set_path($config_path);

lib/Smarty/Smarty.class.php

@@ -27,7 +27,7 @@
  * @author Monte Ohrt <monte at ohrt dot com>
  * @author Andrei Zmievski <andrei@php.net>
  * @package Smarty
- * @version 2.6.30
+ * @version 2.6.31-dev
  */
 
 /* $Id$ */
@@ -465,7 +465,7 @@ class Smarty
      *
      * @var string
      */
-    var $_version              = '2.6.30';
+    var $_version              = '2.6.31';
 
     /**
      * current template inclusion depth

lib/Smarty/Smarty_Compiler.class.php

@@ -259,14 +259,9 @@ class Smarty_Compiler extends Smarty {
 
         preg_match_all($search, $source_content, $match,  PREG_SET_ORDER);
         $this->_folded_blocks = $match;
-        reset($this->_folded_blocks);
 
         /* replace special blocks by "{php}" */
-        $source_content = preg_replace_callback($search, create_function ('$matches', "return '"
-                                       . $this->_quote_replace($this->left_delimiter) . 'php'
-                                       . "' . str_repeat(\"\n\", substr_count('\$matches[1]', \"\n\")) .'"
-                                       . $this->_quote_replace($this->right_delimiter)
-                                       . "';")
+        $source_content = preg_replace_callback($search, array($this,'_preg_callback')
                                        , $source_content);
 
         /* Gather all template tags. */
@@ -556,7 +551,7 @@ class Smarty_Compiler extends Smarty {
 
             case 'php':
                 /* handle folded tags replaced by {php} */
-                list(, $block) = each($this->_folded_blocks);
+                $block = array_shift($this->_folded_blocks);
                 $this->_current_line_no += substr_count($block[0], "\n");
                 /* the number of matched elements in the regexp in _compile_file()
                    determins the type of folded tag that was found */
@@ -754,7 +749,12 @@ class Smarty_Compiler extends Smarty {
         return true;
     }
 
-
+    function _preg_callback ($matches) {
+    return $this->_quote_replace($this->left_delimiter)
+           . 'php'
+           . str_repeat("\n", substr_count($matches[1], "\n"))
+           . $this->_quote_replace($this->right_delimiter);
+    }
     /**
      * compile custom function tag
      *

lib/Smarty/plugins/function.math.php

@@ -18,11 +18,11 @@
  * @author   Monte Ohrt <monte at ohrt dot com>
  *
  * @param array                    $params   parameters
- * @param Smarty_Internal_Template $template template object
+ * @param Smarty
  *
  * @return string|null
  */
-function smarty_function_math($params, $template)
+function smarty_function_math($params, &$smarty)
 {
     static $_allowed_funcs =
         array('int' => true, 'abs' => true, 'ceil' => true, 'cos' => true, 'exp' => true, 'floor' => true,
@@ -58,12 +58,28 @@ function smarty_function_math($params, $template)
         return;
     }
 
+    foreach ($params as $key => $val) {
+        if ($key != "equation" && $key != "format" && $key != "assign") {
+            // make sure value is not empty
+            if (strlen($val) == 0) {
+                trigger_error("math: parameter '{$key}' is empty", E_USER_WARNING);
+
+                return;
+            }
+            if (!is_numeric($val)) {
+                trigger_error("math: parameter '{$key}' is not numeric", E_USER_WARNING);
+
+                return;
+            }
+        }
+    }
+
     // match all vars in equation, make sure all are passed
     preg_match_all('!(?:0x[a-fA-F0-9]+)|([a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)!', $equation, $match);
 
     foreach ($match[ 1 ] as $curr_var) {
         if ($curr_var && !isset($params[ $curr_var ]) && !isset($_allowed_funcs[ $curr_var ])) {
-            trigger_error("math: function call $curr_var not allowed", E_USER_WARNING);
+            trigger_error("math: function call '{$curr_var}' not allowed, or missing parameter '{$curr_var}'", E_USER_WARNING);
 
             return;
         }
@@ -71,17 +87,6 @@ function smarty_function_math($params, $template)
 
     foreach ($params as $key => $val) {
         if ($key != "equation" && $key != "format" && $key != "assign") {
-            // make sure value is not empty
-            if (strlen($val) == 0) {
-                trigger_error("math: parameter $key is empty", E_USER_WARNING);
-
-                return;
-            }
-            if (!is_numeric($val)) {
-                trigger_error("math: parameter $key: is not numeric", E_USER_WARNING);
-
-                return;
-            }
             $equation = preg_replace("/\b$key\b/", " \$params['$key'] ", $equation);
         }
     }
@@ -92,13 +97,13 @@ function smarty_function_math($params, $template)
         if (empty($params[ 'assign' ])) {
             return $smarty_math_result;
         } else {
-            $template->assign($params[ 'assign' ], $smarty_math_result);
+            $smarty->assign($params[ 'assign' ], $smarty_math_result);
         }
     } else {
         if (empty($params[ 'assign' ])) {
             printf($params[ 'format' ], $smarty_math_result);
         } else {
-            $template->assign($params[ 'assign' ], sprintf($params[ 'format' ], $smarty_math_result));
+            $smarty->assign($params[ 'assign' ], sprintf($params[ 'format' ], $smarty_math_result));
         }
     }
 }