xpressengine / xe-core

config/func.inc.php

Doc Comments +21 added lines, -15 removed lines

@@ -74,7 +74,7 @@ 
  * @param string $module_name The module name to get a instance
  * @param string $type disp, proc, controller, class
  * @param string $kind admin, null
- * @return mixed Module instance
+ * @return ModuleObject Module instance
  */
 function getModule($module_name, $type = 'view', $kind = '')
 {
@@ -85,7 +85,7 @@ 
  * Create a controller instance of the module
  *
  * @param string $module_name The module name to get a controller instance
- * @return mixed Module controller instance
+ * @return ModuleObject Module controller instance
  */
 function getController($module_name)
 {
@@ -96,7 +96,7 @@ 
  * Create a admin controller instance of the module
  *
  * @param string $module_name The module name to get a admin controller instance
- * @return mixed Module admin controller instance
+ * @return ModuleObject Module admin controller instance
  */
 function getAdminController($module_name)
 {
@@ -107,7 +107,7 @@ 
  * Create a view instance of the module
  *
  * @param string $module_name The module name to get a view instance
- * @return mixed Module view instance
+ * @return ModuleObject Module view instance
  */
 function getView($module_name)
 {
@@ -118,7 +118,7 @@ 
  * Create a mobile instance of the module
  *
  * @param string $module_name The module name to get a mobile instance
- * @return mixed Module mobile instance
+ * @return ModuleObject Module mobile instance
  */
 function &getMobile($module_name)
 {
@@ -129,7 +129,7 @@ 
  * Create a admin view instance of the module
  *
  * @param string $module_name The module name to get a admin view instance
- * @return mixed Module admin view instance
+ * @return ModuleObject Module admin view instance
  */
 function getAdminView($module_name)
 {
@@ -140,7 +140,7 @@ 
  * Create a model instance of the module
  *
  * @param string $module_name The module name to get a model instance
- * @return mixed Module model instance
+ * @return ModuleObject Module model instance
  */
 function getModel($module_name)
 {
@@ -151,7 +151,7 @@ 
  * Create an admin model instance of the module
  *
  * @param string $module_name The module name to get a admin model instance
- * @return mixed Module admin model instance
+ * @return ModuleObject Module admin model instance
  */
 function getAdminModel($module_name)
 {
@@ -162,7 +162,7 @@ 
  * Create an api instance of the module
  *
  * @param string $module_name The module name to get a api instance
- * @return mixed Module api class instance
+ * @return ModuleObject Module api class instance
  */
 function getAPI($module_name)
 {
@@ -173,7 +173,7 @@ 
  * Create a wap instance of the module
  *
  * @param string $module_name The module name to get a wap instance
- * @return mixed Module wap class instance
+ * @return ModuleObject Module wap class instance
  */
 function getWAP($module_name)
 {
@@ -184,7 +184,7 @@ 
  * Create a class instance of the module
  *
  * @param string $module_name The module name to get a class instance
- * @return mixed Module class instance
+ * @return ModuleObject Module class instance
  */
 function getClass($module_name)
 {
@@ -490,7 +490,7 @@ 
  * Return if domain of the virtual site is url type or id type
  *
  * @param string $domain
- * @return bool
+ * @return integer
  */
 function isSiteID($domain)
 {
@@ -559,7 +559,7 @@ 
 /**
  * Get a time gap between server's timezone and XE's timezone
  *
- * @return int
+ * @return double
  */
 function zgap()
 {
@@ -797,7 +797,7 @@ 
  * Display $buff contents into the file ./files/_debug_message.php.
  * You can see the file on your prompt by command: tail-f./files/_debug_message.php
  *
- * @param mixed $debug_output Target object to be printed
+ * @param string $debug_output Target object to be printed
  * @param bool $display_option boolean Flag whether to print seperator (default:true)
  * @param string $file Target file name
  * @return void
@@ -904,7 +904,7 @@ 
 /**
  * @param string $type query, trigger
  * @param float $elapsed_time
- * @param object $obj
+ * @param stdClass $obj
  */
 function writeSlowlog($type, $elapsed_time, $obj)
 {
@@ -1685,6 +1685,9 @@ 
 	}
 }
 
+/**
+ * @param string $key
+ */
 function changeValueInUrl($key, $requestKey, $dbKey, $urlName = 'success_return_url')
 {
 	if($requestKey != $dbKey)
@@ -1781,6 +1784,9 @@ 
 }
 
 
+/**
+ * @param string $str
+ */
 function isDefinedLangCode($str)
 {
 	return preg_match('!\$user_lang->([a-z0-9\_]+)$!is', trim($str));

Spacing +188 added lines, -188 removed lines

@@ -6,13 +6,13 @@ 
  *
  * @author NAVER (developers@xpressengine.com)
  */
-if(!defined('__XE__'))
+if (!defined('__XE__'))
 {
 	exit();
 }
 
 // define an empty function to avoid errors when iconv function doesn't exist
-if(!function_exists('iconv'))
+if (!function_exists('iconv'))
 {
 	eval('
 		function iconv($in_charset, $out_charset, $str)
@@ -220,7 +220,7 @@ 
 {
 	$oDB = DB::getInstance();
 	$output = $oDB->executeQuery($query_id, $args, $arg_columns);
-	if(!is_array($output->data) && count($output->data) > 0)
+	if (!is_array($output->data) && count($output->data) > 0)
 	{
 		$output->data = array($output->data);
 	}
@@ -250,9 +250,9 @@ 
 function setUserSequence($seq)
 {
 	$arr_seq = array();
-	if(isset($_SESSION['seq']))
+	if (isset($_SESSION['seq']))
 	{
-		if(!is_array($_SESSION['seq'])) {
+		if (!is_array($_SESSION['seq'])) {
 			$_SESSION['seq'] = array($_SESSION['seq']);
 		}
 		$arr_seq = $_SESSION['seq'];
@@ -269,11 +269,11 @@ 
  */
 function checkUserSequence($seq)
 {
-	if(!isset($_SESSION['seq']))
+	if (!isset($_SESSION['seq']))
 	{
 		return false;
 	}
-	if(!in_array($seq, $_SESSION['seq']))
+	if (!in_array($seq, $_SESSION['seq']))
 	{
 		return false;
 	}
@@ -299,7 +299,7 @@ 
 	$num_args = func_num_args();
 	$args_list = func_get_args();
 
-	if($num_args)
+	if ($num_args)
 		$url = Context::getUrl($num_args, $args_list);
 	else
 		$url = Context::getRequestUri();
@@ -318,7 +318,7 @@ 
 	$num_args = func_num_args();
 	$args_list = func_get_args();
 
-	if($num_args)
+	if ($num_args)
 	{
 		$url = Context::getUrl($num_args, $args_list, NULL, FALSE);
 	}
@@ -341,7 +341,7 @@ 
 	$num_args = func_num_args();
 	$args_list = func_get_args();
 
-	if($num_args)
+	if ($num_args)
 	{
 		$url = Context::getUrl($num_args, $args_list, NULL, TRUE, TRUE);
 	}
@@ -363,16 +363,16 @@ 
 	$num_args = func_num_args();
 	$args_list = func_get_args();
 	$request_uri = Context::getRequestUri();
-	if(!$num_args)
+	if (!$num_args)
 	{
 		return $request_uri;
 	}
 
 	$url = Context::getUrl($num_args, $args_list);
-	if(strncasecmp('http', $url, 4) !== 0)
+	if (strncasecmp('http', $url, 4) !== 0)
 	{
 		preg_match('/^(http|https):\/\/([^\/]+)\//', $request_uri, $match);
-		return substr($match[0], 0, -1) . $url;
+		return substr($match[0], 0, -1).$url;
 	}
 	return $url;
 }
@@ -387,17 +387,17 @@ 
 	$num_args = func_num_args();
 	$args_list = func_get_args();
 	$request_uri = Context::getRequestUri();
-	if(!$num_args)
+	if (!$num_args)
 	{
 		return $request_uri;
 	}
 
 	$url = Context::getUrl($num_args, $args_list, NULL, FALSE);
-	if(strncasecmp('http', $url, 4) !== 0)
+	if (strncasecmp('http', $url, 4) !== 0)
 	{
 		preg_match('/^(http|https):\/\/([^\/]+)\//', $request_uri, $match);
 		$url = Context::getUrl($num_args, $args_list, NULL, FALSE);
-		return substr($match[0], 0, -1) . $url;
+		return substr($match[0], 0, -1).$url;
 	}
 	return $url;
 }
@@ -413,7 +413,7 @@ 
 	$num_args = func_num_args();
 	$args_list = func_get_args();
 
-	if(!$num_args)
+	if (!$num_args)
 	{
 		return Context::getRequestUri();
 	}
@@ -435,7 +435,7 @@ 
 	$num_args = func_num_args();
 	$args_list = func_get_args();
 
-	if(!$num_args)
+	if (!$num_args)
 	{
 		return Context::getRequestUri();
 	}
@@ -457,7 +457,7 @@ 
 	$args_list = func_get_args();
 
 	$request_uri = Context::getRequestUri();
-	if(!$num_args)
+	if (!$num_args)
 	{
 		return $request_uri;
 	}
@@ -466,10 +466,10 @@ 
 	$num_args = count($args_list);
 
 	$url = Context::getUrl($num_args, $args_list, $domain);
-	if(strncasecmp('http', $url, 4) !== 0)
+	if (strncasecmp('http', $url, 4) !== 0)
 	{
 		preg_match('/^(http|https):\/\/([^\/]+)\//', $request_uri, $match);
-		return substr($match[0], 0, -1) . $url;
+		return substr($match[0], 0, -1).$url;
 	}
 	return $url;
 }
@@ -482,7 +482,7 @@ 
 function getCurrentPageUrl()
 {
 	$protocol = $_SERVER['HTTPS'] == 'on' ? 'https://' : 'http://';
-	$url = $protocol . $_SERVER['HTTP_HOST'] . preg_replace('/[<>"]/', '', $_SERVER['REQUEST_URI']);
+	$url = $protocol.$_SERVER['HTTP_HOST'].preg_replace('/[<>"]/', '', $_SERVER['REQUEST_URI']);
 	return htmlspecialchars($url, ENT_COMPAT, 'UTF-8', FALSE);
 }
 
@@ -507,12 +507,12 @@ 
  */
 function cut_str($string, $cut_size = 0, $tail = '...')
 {
-	if($cut_size < 1 || !$string)
+	if ($cut_size < 1 || !$string)
 	{
 		return $string;
 	}
 
-	if($GLOBALS['use_mb_strimwidth'] || function_exists('mb_strimwidth'))
+	if ($GLOBALS['use_mb_strimwidth'] || function_exists('mb_strimwidth'))
 	{
 		$GLOBALS['use_mb_strimwidth'] = TRUE;
 		return mb_strimwidth($string, 0, $cut_size + 4, $tail, 'utf-8');
@@ -526,16 +526,16 @@ 
 	$char_count = 0;
 
 	$idx = 0;
-	while($idx < $string_length && $char_count < $cut_size && $char_width <= $max_width)
+	while ($idx < $string_length && $char_count < $cut_size && $char_width <= $max_width)
 	{
 		$c = ord(substr($string, $idx, 1));
 		$char_count++;
-		if($c < 128)
+		if ($c < 128)
 		{
 			$char_width += (int) $chars[$c - 32];
 			$idx++;
 		}
-		else if(191 < $c && $c < 224)
+		else if (191 < $c && $c < 224)
 		{
 			$char_width += $chars[4];
 			$idx += 2;
@@ -548,7 +548,7 @@ 
 	}
 
 	$output = substr($string, 0, $idx);
-	if(strlen($output) < $string_length)
+	if (strlen($output) < $string_length)
 	{
 		$output .= $tail;
 	}
@@ -564,7 +564,7 @@ 
 function zgap()
 {
 	$time_zone = $GLOBALS['_time_zone'];
-	if($time_zone < 0)
+	if ($time_zone < 0)
 	{
 		$to = -1;
 	}
@@ -577,7 +577,7 @@ 
 	$t_min = substr($time_zone, 3, 2) * $to;
 
 	$server_time_zone = date("O");
-	if($server_time_zone < 0)
+	if ($server_time_zone < 0)
 	{
 		$so = -1;
 	}
@@ -604,7 +604,7 @@ 
  */
 function ztime($str)
 {
-	if(!$str)
+	if (!$str)
 	{
 		return;
 	}
@@ -619,7 +619,7 @@ 
 	$year = (int) substr($str, 0, 4);
 	$month = (int) substr($str, 4, 2);
 	$day = (int) substr($str, 6, 2);
-	if(strlen($str) <= 8)
+	if (strlen($str) <= 8)
 	{
 		$gap = 0;
 	}
@@ -643,19 +643,19 @@ 
 	$gap = $_SERVER['REQUEST_TIME'] + zgap() - ztime($date);
 
 	$lang_time_gap = Context::getLang('time_gap');
-	if($gap < 60)
+	if ($gap < 60)
 	{
 		$buff = sprintf($lang_time_gap['min'], (int) ($gap / 60) + 1);
 	}
-	elseif($gap < 60 * 60)
+	elseif ($gap < 60 * 60)
 	{
 		$buff = sprintf($lang_time_gap['mins'], (int) ($gap / 60) + 1);
 	}
-	elseif($gap < 60 * 60 * 2)
+	elseif ($gap < 60 * 60 * 2)
 	{
 		$buff = sprintf($lang_time_gap['hour'], (int) ($gap / 60 / 60) + 1);
 	}
-	elseif($gap < 60 * 60 * 24)
+	elseif ($gap < 60 * 60 * 24)
 	{
 		$buff = sprintf($lang_time_gap['hours'], (int) ($gap / 60 / 60) + 1);
 	}
@@ -692,40 +692,40 @@ 
 function zdate($str, $format = 'Y-m-d H:i:s', $conversion = TRUE)
 {
 	// return null if no target time is specified
-	if(!$str)
+	if (!$str)
 	{
 		return;
 	}
 	// convert the date format according to the language
-	if($conversion == TRUE)
+	if ($conversion == TRUE)
 	{
-		switch(Context::getLangType())
+		switch (Context::getLangType())
 		{
 			case 'en' :
 			case 'es' :
-				if($format == 'Y-m-d')
+				if ($format == 'Y-m-d')
 				{
 					$format = 'M d, Y';
 				}
-				elseif($format == 'Y-m-d H:i:s')
+				elseif ($format == 'Y-m-d H:i:s')
 				{
 					$format = 'M d, Y H:i:s';
 				}
-				elseif($format == 'Y-m-d H:i')
+				elseif ($format == 'Y-m-d H:i')
 				{
 					$format = 'M d, Y H:i';
 				}
 				break;
 			case 'vi' :
-				if($format == 'Y-m-d')
+				if ($format == 'Y-m-d')
 				{
 					$format = 'd-m-Y';
 				}
-				elseif($format == 'Y-m-d H:i:s')
+				elseif ($format == 'Y-m-d H:i:s')
 				{
 					$format = 'H:i:s d-m-Y';
 				}
-				elseif($format == 'Y-m-d H:i')
+				elseif ($format == 'Y-m-d H:i')
 				{
 					$format = 'H:i d-m-Y';
 				}
@@ -734,7 +734,7 @@ 
 	}
 
 	// If year value is less than 1970, handle it separately.
-	if((int) substr($str, 0, 4) < 1970)
+	if ((int) substr($str, 0, 4) < 1970)
 	{
 		$hour = (int) substr($str, 8, 2);
 		$min = (int) substr($str, 10, 2);
@@ -784,9 +784,9 @@ 
 function getEncodeEmailAddress($email)
 {
 	$return = '';
-	for($i = 0, $c = strlen($email); $i < $c; $i++)
+	for ($i = 0, $c = strlen($email); $i < $c; $i++)
 	{
-		$return .= '&#' . (rand(0, 1) == 0 ? ord($email[$i]) : 'X' . dechex(ord($email[$i]))) . ';';
+		$return .= '&#'.(rand(0, 1) == 0 ? ord($email[$i]) : 'X'.dechex(ord($email[$i]))).';';
 	}
 	return $return;
 }
@@ -806,25 +806,25 @@ 
 {
 	static $debug_file;
 
-	if(!(__DEBUG__ & 1))
+	if (!(__DEBUG__ & 1))
 	{
 		return;
 	}
 
 	static $firephp;
 	$bt = debug_backtrace();
-	if(is_array($bt))
+	if (is_array($bt))
 	{
 		$bt_debug_print = array_shift($bt);
 		$bt_called_function = array_shift($bt);
 	}
 	$file_name = str_replace(_XE_PATH_, '', $bt_debug_print['file']);
 	$line_num = $bt_debug_print['line'];
-	$function = $bt_called_function['class'] . $bt_called_function['type'] . $bt_called_function['function'];
+	$function = $bt_called_function['class'].$bt_called_function['type'].$bt_called_function['function'];
 
-	if(__DEBUG_OUTPUT__ == 2 && version_compare(PHP_VERSION, '6.0.0') === -1)
+	if (__DEBUG_OUTPUT__ == 2 && version_compare(PHP_VERSION, '6.0.0') === -1)
 	{
-		if(!isset($firephp))
+		if (!isset($firephp))
 		{
 			$firephp = FirePHP::getInstance(TRUE);
 		}
@@ -833,16 +833,16 @@ 
 		$label = sprintf('[%s:%d] %s() (Memory usage: current=%s, peak=%s)', $file_name, $line_num, $function, FileHandler::filesize(memory_get_usage()), FileHandler::filesize(memory_get_peak_usage()));
 
 		// Check a FirePHP option
-		if($display_option === 'TABLE')
+		if ($display_option === 'TABLE')
 		{
 			$label = $display_option;
 		}
-		if($display_option === 'ERROR')
+		if ($display_option === 'ERROR')
 		{
 			$type = $display_option;
 		}
 		// Check if the IP specified by __DEBUG_PROTECT__ option is same as the access IP.
-		if(__DEBUG_PROTECT__ === 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
+		if (__DEBUG_PROTECT__ === 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
 		{
 			$debug_output = 'The IP address is not allowed. Change the value of __DEBUG_PROTECT_IP__ into your IP address in config/config.user.inc.php or config/config.inc.php';
 			$label = NULL;
@@ -852,52 +852,52 @@ 
 	}
 	else
 	{
-		if(__DEBUG_PROTECT__ === 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
+		if (__DEBUG_PROTECT__ === 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
 		{
 			return;
 		}
 
 		$print = array();
-		if(!$debug_file)
+		if (!$debug_file)
 		{
-			$debug_file = _XE_PATH_ . 'files/' . $file;
+			$debug_file = _XE_PATH_.'files/'.$file;
 		}
-		if(!file_exists($debug_file)) $print[] = '<?php exit() ?>';
+		if (!file_exists($debug_file)) $print[] = '<?php exit() ?>';
 
-		if($display_option === TRUE || $display_option === 'ERROR')
+		if ($display_option === TRUE || $display_option === 'ERROR')
 		{
-			$print[] = sprintf("[%s %s:%d] %s() - mem(%s)", date('Y-m-d H:i:s'), $file_name, $line_num, $function, FileHandler::filesize(memory_get_usage()));;
+			$print[] = sprintf("[%s %s:%d] %s() - mem(%s)", date('Y-m-d H:i:s'), $file_name, $line_num, $function, FileHandler::filesize(memory_get_usage())); ;
 			$print[] = str_repeat('=', 80);
 		}
 		$type = gettype($debug_output);
-		if(!in_array($type, array('array', 'object', 'resource')))
+		if (!in_array($type, array('array', 'object', 'resource')))
 		{
-			if($display_option === 'ERROR')
+			if ($display_option === 'ERROR')
 			{
-				$print[] = 'ERROR : ' . var_export($debug_output, TRUE);
+				$print[] = 'ERROR : '.var_export($debug_output, TRUE);
 			}
 			else
 			{
-				$print[] = 'DEBUG : ' . $type . '(' . var_export($debug_output, TRUE) . ')';
+				$print[] = 'DEBUG : '.$type.'('.var_export($debug_output, TRUE).')';
 			}
 		}
 		else
 		{
-			$print[] = 'DEBUG : ' . trim(preg_replace('/\r?\n/', "\n" . '        ', print_r($debug_output, true)));
+			$print[] = 'DEBUG : '.trim(preg_replace('/\r?\n/', "\n".'        ', print_r($debug_output, true)));
 		}
 		$backtrace_args = defined('\DEBUG_BACKTRACE_IGNORE_ARGS') ? \DEBUG_BACKTRACE_IGNORE_ARGS : 0;
 		$backtrace = debug_backtrace($backtrace_args);
 
-		if(count($backtrace) > 1 && $backtrace[1]['function'] === 'debugPrint' && !$backtrace[1]['class'])
+		if (count($backtrace) > 1 && $backtrace[1]['function'] === 'debugPrint' && !$backtrace[1]['class'])
 		{
 			array_shift($backtrace);
 		}
-		foreach($backtrace as $val)
+		foreach ($backtrace as $val)
 		{
-			$print[] = '        - ' . $val['file'] . ' : ' . $val['line'];
+			$print[] = '        - '.$val['file'].' : '.$val['line'];
 		}
 		$print[] = PHP_EOL;
-		@file_put_contents($debug_file, implode(PHP_EOL, $print), FILE_APPEND|LOCK_EX);
+		@file_put_contents($debug_file, implode(PHP_EOL, $print), FILE_APPEND | LOCK_EX);
 	}
 }
 
@@ -908,7 +908,7 @@ 
  */
 function writeSlowlog($type, $elapsed_time, $obj)
 {
-	if(!__LOG_SLOW_TRIGGER__ && !__LOG_SLOW_ADDON__ && !__LOG_SLOW_WIDGET__ && !__LOG_SLOW_QUERY__) return;
+	if (!__LOG_SLOW_TRIGGER__ && !__LOG_SLOW_ADDON__ && !__LOG_SLOW_WIDGET__ && !__LOG_SLOW_QUERY__) return;
 
 	static $log_filename = array(
 		'query' => 'files/_slowlog_query.php',
@@ -918,47 +918,47 @@ 
 	);
 	$write_file = true;
 
-	$log_file = _XE_PATH_ . $log_filename[$type];
+	$log_file = _XE_PATH_.$log_filename[$type];
 
 	$buff = array();
 	$buff[] = '<?php exit(); ?>';
 	$buff[] = date('c');
 
-	if($type == 'trigger' && __LOG_SLOW_TRIGGER__ > 0 && $elapsed_time > __LOG_SLOW_TRIGGER__)
+	if ($type == 'trigger' && __LOG_SLOW_TRIGGER__ > 0 && $elapsed_time > __LOG_SLOW_TRIGGER__)
 	{
-		$buff[] = "\tCaller : " . $obj->caller;
-		$buff[] = "\tCalled : " . $obj->called;
+		$buff[] = "\tCaller : ".$obj->caller;
+		$buff[] = "\tCalled : ".$obj->called;
 	}
-	else if($type == 'addon' && __LOG_SLOW_ADDON__ > 0 && $elapsed_time > __LOG_SLOW_ADDON__)
+	else if ($type == 'addon' && __LOG_SLOW_ADDON__ > 0 && $elapsed_time > __LOG_SLOW_ADDON__)
 	{
-		$buff[] = "\tAddon : " . $obj->called;
-		$buff[] = "\tCalled position : " . $obj->caller;
+		$buff[] = "\tAddon : ".$obj->called;
+		$buff[] = "\tCalled position : ".$obj->caller;
 	}
-	else if($type == 'widget' && __LOG_SLOW_WIDGET__ > 0 && $elapsed_time > __LOG_SLOW_WIDGET__)
+	else if ($type == 'widget' && __LOG_SLOW_WIDGET__ > 0 && $elapsed_time > __LOG_SLOW_WIDGET__)
 	{
-		$buff[] = "\tWidget : " . $obj->called;
+		$buff[] = "\tWidget : ".$obj->called;
 	}
-	else if($type == 'query' && __LOG_SLOW_QUERY__ > 0 && $elapsed_time > __LOG_SLOW_QUERY__)
+	else if ($type == 'query' && __LOG_SLOW_QUERY__ > 0 && $elapsed_time > __LOG_SLOW_QUERY__)
 	{
 
 		$buff[] = $obj->query;
-		$buff[] = "\tQuery ID   : " . $obj->query_id;
-		$buff[] = "\tCaller     : " . $obj->caller;
-		$buff[] = "\tConnection : " . $obj->connection;
+		$buff[] = "\tQuery ID   : ".$obj->query_id;
+		$buff[] = "\tCaller     : ".$obj->caller;
+		$buff[] = "\tConnection : ".$obj->connection;
 	}
 	else
 	{
 		$write_file = false;
 	}
 
-	if($write_file)
+	if ($write_file)
 	{
 		$buff[] = sprintf("\t%0.6f sec", $elapsed_time);
-		$buff[] = PHP_EOL . PHP_EOL;
+		$buff[] = PHP_EOL.PHP_EOL;
 		file_put_contents($log_file, implode(PHP_EOL, $buff), FILE_APPEND);
 	}
 
-	if($type != 'query')
+	if ($type != 'query')
 	{
 		$trigger_args = $obj;
 		$trigger_args->_log_type = $type;
@@ -998,11 +998,11 @@ 
  */
 function delObjectVars($target_obj, $del_obj)
 {
-	if(!is_object($target_obj))
+	if (!is_object($target_obj))
 	{
 		return;
 	}
-	if(!is_object($del_obj))
+	if (!is_object($del_obj))
 	{
 		return;
 	}
@@ -1012,7 +1012,7 @@ 
 
 	$target = array_keys($target_vars);
 	$del = array_keys($del_vars);
-	if(!count($target) || !count($del))
+	if (!count($target) || !count($del))
 	{
 		return $target_obj;
 	}
@@ -1020,10 +1020,10 @@ 
 	$return_obj = new stdClass();
 
 	$target_count = count($target);
-	for($i = 0; $i < $target_count; $i++)
+	for ($i = 0; $i < $target_count; $i++)
 	{
 		$target_key = $target[$i];
-		if(!in_array($target_key, $del))
+		if (!in_array($target_key, $del))
 		{
 			$return_obj->{$target_key} = $target_obj->{$target_key};
 		}
@@ -1036,10 +1036,10 @@ 
 {
 	$del_vars = array('error_return_url', 'success_return_url', 'ruleset', 'xe_validator_id');
 
-	foreach($del_vars as $var)
+	foreach ($del_vars as $var)
 	{
-		if(is_array($vars)) unset($vars[$var]);
-		else if(is_object($vars)) unset($vars->$var);
+		if (is_array($vars)) unset($vars[$var]);
+		else if (is_object($vars)) unset($vars->$var);
 	}
 
 	return $vars;
@@ -1056,12 +1056,12 @@ 
  */
 function handleError($errno, $errstr, $file, $line)
 {
-	if(!__DEBUG__)
+	if (!__DEBUG__)
 	{
 		return;
 	}
 	$errors = array(E_USER_ERROR, E_ERROR, E_PARSE);
-	if(!in_array($errno, $errors))
+	if (!in_array($errno, $errors))
 	{
 		return;
 	}
@@ -1081,8 +1081,8 @@ 
 function getNumberingPath($no, $size = 3)
 {
 	$mod = pow(10, $size);
-	$output = sprintf('%0' . $size . 'd/', $no % $mod);
-	if($no >= $mod)
+	$output = sprintf('%0'.$size.'d/', $no % $mod);
+	if ($no >= $mod)
 	{
 		$output .= getNumberingPath((int) $no / $mod, $size);
 	}
@@ -1102,12 +1102,12 @@ 
 
 function purifierHtml(&$content)
 {
-	require_once(_XE_PATH_ . 'classes/security/Purifier.class.php');
+	require_once(_XE_PATH_.'classes/security/Purifier.class.php');
 	$oPurifier = Purifier::getInstance();
 
 	// @see https://github.com/xpressengine/xe-core/issues/2278
 	$logged_info = Context::get('logged_info');
-	if($logged_info->is_admin !== 'Y') {
+	if ($logged_info->is_admin !== 'Y') {
 		$oPurifier->setConfig('HTML.Nofollow', true);
 	}
 
@@ -1122,7 +1122,7 @@ 
  */
 function removeHackTag($content)
 {
-	require_once(_XE_PATH_ . 'classes/security/EmbedFilter.class.php');
+	require_once(_XE_PATH_.'classes/security/EmbedFilter.class.php');
 	$oEmbedFilter = EmbedFilter::getInstance();
 	$oEmbedFilter->check($content);
 
@@ -1164,7 +1164,7 @@ 
  */
 function checkUploadedFile($file)
 {
-	require_once(_XE_PATH_ . 'classes/security/UploadFileFilter.class.php');
+	require_once(_XE_PATH_.'classes/security/UploadFileFilter.class.php');
 	return UploadFileFilter::check($file);
 }
 
@@ -1178,13 +1178,13 @@ 
 {
 	$content = preg_replace('@<(/?)xmp.*?>@i', '<\1xmp>', $content);
 
-	if(($start_xmp = strrpos($content, '<xmp>')) !== FALSE)
+	if (($start_xmp = strrpos($content, '<xmp>')) !== FALSE)
 	{
-		if(($close_xmp = strrpos($content, '</xmp>')) === FALSE)
+		if (($close_xmp = strrpos($content, '</xmp>')) === FALSE)
 		{
 			$content .= '</xmp>';
 		}
-		else if($close_xmp < $start_xmp)
+		else if ($close_xmp < $start_xmp)
 		{
 			$content .= '</xmp>';
 		}
@@ -1204,33 +1204,33 @@ 
 	$tag = strtolower($match[2]);
 
 	// xmp tag ?뺣━
-	if($tag == 'xmp')
+	if ($tag == 'xmp')
 	{
 		return "<{$match[1]}xmp>";
 	}
-	if($match[1])
+	if ($match[1])
 	{
 		return $match[0];
 	}
-	if($match[4])
+	if ($match[4])
 	{
-		$match[4] = ' ' . $match[4];
+		$match[4] = ' '.$match[4];
 	}
 
 	$attrs = array();
-	if(preg_match_all('/([\w:-]+)\s*=(?:\s*(["\']))?(?(2)(.*?)\2|([^ ]+))/s', $match[3], $m))
+	if (preg_match_all('/([\w:-]+)\s*=(?:\s*(["\']))?(?(2)(.*?)\2|([^ ]+))/s', $match[3], $m))
 	{
-		foreach($m[1] as $idx => $name)
+		foreach ($m[1] as $idx => $name)
 		{
-			if(strlen($name) >= 2 && substr_compare($name, 'on', 0, 2) === 0)
+			if (strlen($name) >= 2 && substr_compare($name, 'on', 0, 2) === 0)
 			{
 				continue;
 			}
 
-			$val = preg_replace_callback('/&#(?:x([a-fA-F0-9]+)|0*(\d+));/', function($n) {return chr($n[1] ? ('0x00' . $n[1]) : ($n[2] + 0)); }, $m[3][$idx] . $m[4][$idx]);
+			$val = preg_replace_callback('/&#(?:x([a-fA-F0-9]+)|0*(\d+));/', function($n) {return chr($n[1] ? ('0x00'.$n[1]) : ($n[2] + 0)); }, $m[3][$idx].$m[4][$idx]);
 			$val = preg_replace('/^\s+|[\t\n\r]+/', '', $val);
 
-			if(preg_match('/^[a-z]+script:/i', $val))
+			if (preg_match('/^[a-z]+script:/i', $val))
 			{
 				continue;
 			}
@@ -1241,60 +1241,60 @@ 
 
 	$filter_arrts = array('style', 'src', 'href');
 
-	if($tag === 'object') array_push($filter_arrts, 'data');
-	if($tag === 'param') array_push($filter_arrts, 'value');
+	if ($tag === 'object') array_push($filter_arrts, 'data');
+	if ($tag === 'param') array_push($filter_arrts, 'value');
 
-	foreach($filter_arrts as $attr)
+	foreach ($filter_arrts as $attr)
 	{
-		if(!isset($attrs[$attr])) continue;
+		if (!isset($attrs[$attr])) continue;
 
 		$attr_value = rawurldecode($attrs[$attr]);
 		$attr_value = htmlspecialchars_decode($attr_value, ENT_COMPAT);
 		$attr_value = preg_replace('/\s+|[\t\n\r]+/', '', $attr_value);
-		if(preg_match('@(\?|&|;)(act=(\w+))@i', $attr_value, $m) && $m[3] !== 'procFileDownload')
+		if (preg_match('@(\?|&|;)(act=(\w+))@i', $attr_value, $m) && $m[3] !== 'procFileDownload')
 		{
 			unset($attrs[$attr]);
 		}
 	}
 
-	if(isset($attrs['style']) && preg_match('@(?:/\*|\*/|\n|:\s*expression\s*\()@i', $attrs['style']))
+	if (isset($attrs['style']) && preg_match('@(?:/\*|\*/|\n|:\s*expression\s*\()@i', $attrs['style']))
 	{
 		unset($attrs['style']);
 	}
 
 	$attr = array();
-	foreach($attrs as $name => $val)
+	foreach ($attrs as $name => $val)
 	{
-		if($tag == 'object' || $tag == 'embed' || $tag == 'a')
+		if ($tag == 'object' || $tag == 'embed' || $tag == 'a')
 		{
 			$attribute = strtolower(trim($name));
-			if($attribute == 'data' || $attribute == 'src' || $attribute == 'href')
+			if ($attribute == 'data' || $attribute == 'src' || $attribute == 'href')
 			{
-				if(stripos($val, 'data:') === 0)
+				if (stripos($val, 'data:') === 0)
 				{
 					continue;
 				}
 			}
 		}
 
-		if($tag == 'img')
+		if ($tag == 'img')
 		{
 			$attribute = strtolower(trim($name));
-			if(stripos($val, 'data:') === 0)
+			if (stripos($val, 'data:') === 0)
 			{
 				continue;
 			}
 		}
 		$val = str_replace('"', '&quot;', $val);
-		$attr[] = $name . "=\"{$val}\"";
+		$attr[] = $name."=\"{$val}\"";
 	}
-	$attr = count($attr) ? ' ' . implode(' ', $attr) : '';
+	$attr = count($attr) ? ' '.implode(' ', $attr) : '';
 
 	return "<{$match[1]}{$tag}{$attr}{$match[4]}>";
 }
 
 // convert hexa value to RGB
-if(!function_exists('hexrgb'))
+if (!function_exists('hexrgb'))
 {
 
 	/**
@@ -1330,9 +1330,9 @@ 
 
 	settype($password, "string");
 
-	for($i = 0; $i < strlen($password); $i++)
+	for ($i = 0; $i < strlen($password); $i++)
 	{
-		if($password[$i] == ' ' || $password[$i] == '\t')
+		if ($password[$i] == ' ' || $password[$i] == '\t')
 		{
 			continue;
 		}
@@ -1344,11 +1344,11 @@ 
 	$result1 = sprintf("%08lx", $nr & ((1 << 31) - 1));
 	$result2 = sprintf("%08lx", $nr2 & ((1 << 31) - 1));
 
-	if($result1 == '80000000')
+	if ($result1 == '80000000')
 	{
 		$nr += 0x80000000;
 	}
-	if($result2 == '80000000')
+	if ($result2 == '80000000')
 	{
 		$nr2 += 0x80000000;
 	}
@@ -1364,7 +1364,7 @@ 
 function getScriptPath()
 {
 	static $url = NULL;
-	if($url == NULL)
+	if ($url == NULL)
 	{
 		$script_path = filter_var($_SERVER['SCRIPT_NAME'], FILTER_SANITIZE_STRING);
 		$url = str_ireplace('/tools/', '/', preg_replace('/index.php.*/i', '', str_replace('\\', '/', $script_path)));
@@ -1395,14 +1395,14 @@ 
 	$decodedStr = '';
 	$pos = 0;
 	$len = strlen($source);
-	while($pos < $len)
+	while ($pos < $len)
 	{
 		$charAt = substr($source, $pos, 1);
-		if($charAt == '%')
+		if ($charAt == '%')
 		{
 			$pos++;
 			$charAt = substr($source, $pos, 1);
-			if($charAt == 'u')
+			if ($charAt == 'u')
 			{
 				// we got a unicode character
 				$pos++;
@@ -1436,21 +1436,21 @@ 
  */
 function _code2utf($num)
 {
-	if($num < 128)
+	if ($num < 128)
 	{
 		return chr($num);
 	}
-	if($num < 2048)
+	if ($num < 2048)
 	{
-		return chr(($num >> 6) + 192) . chr(($num & 63) + 128);
+		return chr(($num >> 6) + 192).chr(($num & 63) + 128);
 	}
-	if($num < 65536)
+	if ($num < 65536)
 	{
-		return chr(($num >> 12) + 224) . chr((($num >> 6) & 63) + 128) . chr(($num & 63) + 128);
+		return chr(($num >> 12) + 224).chr((($num >> 6) & 63) + 128).chr(($num & 63) + 128);
 	}
-	if($num < 2097152)
+	if ($num < 2097152)
 	{
-		return chr(($num >> 18) + 240) . chr((($num >> 12) & 63) + 128) . chr((($num >> 6) & 63) + 128) . chr(($num & 63) + 128);
+		return chr(($num >> 18) + 240).chr((($num >> 12) & 63) + 128).chr((($num >> 6) & 63) + 128).chr(($num & 63) + 128);
 	}
 	return '';
 }
@@ -1465,7 +1465,7 @@ 
  */
 function detectUTF8($string, $return_convert = FALSE, $urldecode = TRUE)
 {
-	if($urldecode)
+	if ($urldecode)
 	{
 		$string = urldecode($string);
 	}
@@ -1473,12 +1473,12 @@ 
 	$sample = iconv('utf-8', 'utf-8', $string);
 	$is_utf8 = (md5($sample) === md5($string));
 
-	if(!$urldecode)
+	if (!$urldecode)
 	{
 		$string = urldecode($string);
 	}
 
-	if($return_convert)
+	if ($return_convert)
 	{
 		return ($is_utf8) ? $string : iconv('euc-kr', 'utf-8', $string);
 	}
@@ -1494,7 +1494,7 @@ 
  */
 function json_encode2($data)
 {
-	switch(gettype($data))
+	switch (gettype($data))
 	{
 		case 'boolean':
 			return $data ? 'true' : 'false';
@@ -1502,15 +1502,15 @@ 
 		case 'double':
 			return $data;
 		case 'string':
-			return '"' . strtr($data, array('\\' => '\\\\', '"' => '\\"')) . '"';
+			return '"'.strtr($data, array('\\' => '\\\\', '"' => '\\"')).'"';
 		case 'object':
 			$data = get_object_vars($data);
 		case 'array':
 			$rel = FALSE; // relative array?
 			$key = array_keys($data);
-			foreach($key as $v)
+			foreach ($key as $v)
 			{
-				if(!is_int($v))
+				if (!is_int($v))
 				{
 					$rel = TRUE;
 					break;
@@ -1518,12 +1518,12 @@ 
 			}
 
 			$arr = array();
-			foreach($data as $k => $v)
+			foreach ($data as $k => $v)
 			{
-				$arr[] = ($rel ? '"' . strtr($k, array('\\' => '\\\\', '"' => '\\"')) . '":' : '') . json_encode2($v);
+				$arr[] = ($rel ? '"'.strtr($k, array('\\' => '\\\\', '"' => '\\"')).'":' : '').json_encode2($v);
 			}
 
-			return $rel ? '{' . join(',', $arr) . '}' : '[' . join(',', $arr) . ']';
+			return $rel ? '{'.join(',', $arr).'}' : '['.join(',', $arr).']';
 		default:
 			return '""';
 	}
@@ -1537,7 +1537,7 @@ 
  */
 function isCrawler($agent = NULL)
 {
-	if(!$agent)
+	if (!$agent)
 	{
 		$agent = $_SERVER['HTTP_USER_AGENT'];
 	}
@@ -1547,9 +1547,9 @@ 
 		/*'211.245.21.110-211.245.21.119' mixsh is closed */
 	);
 
-	foreach($check_agent as $str)
+	foreach ($check_agent as $str)
 	{
-		if(stristr($agent, $str) != FALSE)
+		if (stristr($agent, $str) != FALSE)
 		{
 			return TRUE;
 		}
@@ -1567,7 +1567,7 @@ 
  */
 function stripEmbedTagForAdmin(&$content, $writer_member_srl)
 {
-	if(!Context::get('is_logged'))
+	if (!Context::get('is_logged'))
 	{
 		return;
 	}
@@ -1575,18 +1575,18 @@ 
 	$oModuleModel = getModel('module');
 	$logged_info = Context::get('logged_info');
 
-	if($writer_member_srl != $logged_info->member_srl && ($logged_info->is_admin == "Y" || $oModuleModel->isSiteAdmin($logged_info)))
+	if ($writer_member_srl != $logged_info->member_srl && ($logged_info->is_admin == "Y" || $oModuleModel->isSiteAdmin($logged_info)))
 	{
-		if($writer_member_srl)
+		if ($writer_member_srl)
 		{
 			$oMemberModel = getModel('member');
 			$member_info = $oMemberModel->getMemberInfoByMemberSrl($writer_member_srl);
-			if($member_info->is_admin == "Y")
+			if ($member_info->is_admin == "Y")
 			{
 				return;
 			}
 		}
-		$security_msg = "<div style='border: 1px solid #DDD; background: #FAFAFA; text-align:center; margin: 1em 0;'><p style='margin: 1em;'>" . Context::getLang('security_warning_embed') . "</p></div>";
+		$security_msg = "<div style='border: 1px solid #DDD; background: #FAFAFA; text-align:center; margin: 1em 0;'><p style='margin: 1em;'>".Context::getLang('security_warning_embed')."</p></div>";
 		$content = preg_replace('/<object[^>]+>(.*?<\/object>)?/is', $security_msg, $content);
 		$content = preg_replace('/<embed[^>]+>(\s*<\/embed>)?/is', $security_msg, $content);
 		$content = preg_replace('/<img[^>]+editor_component="multimedia_link"[^>]*>(\s*<\/img>)?/is', $security_msg, $content);
@@ -1603,18 +1603,18 @@ 
 function requirePear()
 {
 	static $required = false;
-	if($required)
+	if ($required)
 	{
 		return;
 	}
 
-	if(version_compare(PHP_VERSION, "5.3.0") < 0)
+	if (version_compare(PHP_VERSION, "5.3.0") < 0)
 	{
-		set_include_path(_XE_PATH_ . "libs/PEAR" . PATH_SEPARATOR . get_include_path());
+		set_include_path(_XE_PATH_."libs/PEAR".PATH_SEPARATOR.get_include_path());
 	}
 	else
 	{
-		set_include_path(_XE_PATH_ . "libs/PEAR.1.9.5" . PATH_SEPARATOR . get_include_path());
+		set_include_path(_XE_PATH_."libs/PEAR.1.9.5".PATH_SEPARATOR.get_include_path());
 	}
 
 	$required = true;
@@ -1622,7 +1622,7 @@ 
 
 function checkCSRF()
 {
-	if($_SERVER['REQUEST_METHOD'] != 'POST')
+	if ($_SERVER['REQUEST_METHOD'] != 'POST')
 	{
 		return FALSE;
 	}
@@ -1630,9 +1630,9 @@ 
 	$default_url = Context::getDefaultUrl();
 	$referer = $_SERVER["HTTP_REFERER"];
 
-	if(strpos($default_url, 'xn--') !== FALSE && strpos($referer, 'xn--') === FALSE)
+	if (strpos($default_url, 'xn--') !== FALSE && strpos($referer, 'xn--') === FALSE)
 	{
-		require_once(_XE_PATH_ . 'libs/idna_convert/idna_convert.class.php');
+		require_once(_XE_PATH_.'libs/idna_convert/idna_convert.class.php');
 		$IDN = new idna_convert(array('idn_version' => 2008));
 		$referer = $IDN->encode($referer);
 	}
@@ -1643,9 +1643,9 @@ 
 	$oModuleModel = getModel('module');
 	$siteModuleInfo = $oModuleModel->getDefaultMid();
 
-	if($siteModuleInfo->site_srl == 0)
+	if ($siteModuleInfo->site_srl == 0)
 	{
-		if($default_url['host'] !== $referer['host'])
+		if ($default_url['host'] !== $referer['host'])
 		{
 			return FALSE;
 		}
@@ -1653,7 +1653,7 @@ 
 	else
 	{
 		$virtualSiteInfo = $oModuleModel->getSiteInfo($siteModuleInfo->site_srl);
-		if(strtolower($virtualSiteInfo->domain) != strtolower(Context::get('vid')) && !strstr(strtolower($virtualSiteInfo->domain), strtolower($referer['host'])))
+		if (strtolower($virtualSiteInfo->domain) != strtolower(Context::get('vid')) && !strstr(strtolower($virtualSiteInfo->domain), strtolower($referer['host'])))
 		{
 			return FALSE;
 		}
@@ -1669,15 +1669,15 @@ 
  */
 function recurciveExposureCheck(&$menu)
 {
-	if(is_array($menu))
+	if (is_array($menu))
 	{
-		foreach($menu AS $key=>$value)
+		foreach ($menu AS $key=>$value)
 		{
-			if(!$value['isShow'])
+			if (!$value['isShow'])
 			{
 				unset($menu[$key]);
 			}
-			if(is_array($value['list']) && count($value['list']) > 0)
+			if (is_array($value['list']) && count($value['list']) > 0)
 			{
 				recurciveExposureCheck($menu[$key]['list']);
 			}
@@ -1687,14 +1687,14 @@ 
 
 function changeValueInUrl($key, $requestKey, $dbKey, $urlName = 'success_return_url')
 {
-	if($requestKey != $dbKey)
+	if ($requestKey != $dbKey)
 	{
 		$arrayUrl = parse_url(Context::get('success_return_url'));
-		if($arrayUrl['query'])
+		if ($arrayUrl['query'])
 		{
 			parse_str($arrayUrl['query'], $parsedStr);
 
-			if(isset($parsedStr[$key]))
+			if (isset($parsedStr[$key]))
 			{
 				$parsedStr[$key] = $requestKey;
 				$successReturnUrl .= $arrayUrl['path'].'?'.http_build_query($parsedStr);
@@ -1737,14 +1737,14 @@ 
  */
 function alertScript($msg)
 {
-	if(!$msg)
+	if (!$msg)
 	{
 		return;
 	}
 
 	echo '<script type="text/javascript">
 //<![CDATA[
-alert("' . $msg . '");
+alert("' . $msg.'");
 //]]>
 </script>';
 }
@@ -1775,7 +1775,7 @@ 
 
 	echo '<script type="text/javascript">
 //<![CDATA[
-' . $reloadScript . '
+' . $reloadScript.'
 //]]>
 </script>';
 }
@@ -1798,7 +1798,7 @@ 
  */
 function escape($str, $double_escape = true, $escape_defined_lang_code = false)
 {
-	if(!$escape_defined_lang_code && isDefinedLangCode($str)) return $str;
+	if (!$escape_defined_lang_code && isDefinedLangCode($str)) return $str;
 
 	$flags = ENT_QUOTES | ENT_SUBSTITUTE;
 	return htmlspecialchars($str, $flags, 'UTF-8', $double_escape);
@@ -1830,7 +1830,7 @@ 
 function escape_js($str)
 {
 	$flags = JSON_HEX_TAG | JSON_HEX_AMP | JSON_HEX_APOS | JSON_HEX_QUOT | JSON_UNESCAPED_UNICODE;
-	$str = json_encode((string)$str, $flags);
+	$str = json_encode((string) $str, $flags);
 	return substr($str, 1, strlen($str) - 2);
 }
 
@@ -1882,12 +1882,12 @@ 
 {
 	if ($limit < 1) $limit = null;
 	$result = array();
-	$split = preg_split('/(?<!' . preg_quote($escape_char, '/') . ')' . preg_quote($delimiter, '/') . '/', $str, $limit);
+	$split = preg_split('/(?<!'.preg_quote($escape_char, '/').')'.preg_quote($delimiter, '/').'/', $str, $limit);
 	foreach ($split as $piece)
 	{
 		if (trim($piece) !== '')
 		{
-			$result[] = trim(str_replace($escape_char . $delimiter, $delimiter, $piece));
+			$result[] = trim(str_replace($escape_char.$delimiter, $delimiter, $piece));
 		}
 	}
 	return $result;

Braces +59 added lines, -69 removed lines

@@ -299,10 +299,11 @@ 
 	$num_args = func_num_args();
 	$args_list = func_get_args();
 
-	if($num_args)
-		$url = Context::getUrl($num_args, $args_list);
-	else
-		$url = Context::getRequestUri();
+	if($num_args) {
+			$url = Context::getUrl($num_args, $args_list);
+	} else {
+			$url = Context::getRequestUri();
+	}
 
 	return preg_replace('@\berror_return_url=[^&]*|\w+=(?:&|$)@', '', $url);
 }
@@ -321,8 +322,7 @@ 
 	if($num_args)
 	{
 		$url = Context::getUrl($num_args, $args_list, NULL, FALSE);
-	}
-	else
+	} else
 	{
 		$url = Context::getRequestUri();
 	}
@@ -344,8 +344,7 @@ 
 	if($num_args)
 	{
 		$url = Context::getUrl($num_args, $args_list, NULL, TRUE, TRUE);
-	}
-	else
+	} else
 	{
 		$url = Context::getRequestUri();
 	}
@@ -534,13 +533,11 @@ 
 		{
 			$char_width += (int) $chars[$c - 32];
 			$idx++;
-		}
-		else if(191 < $c && $c < 224)
+		} else if(191 < $c && $c < 224)
 		{
 			$char_width += $chars[4];
 			$idx += 2;
-		}
-		else
+		} else
 		{
 			$char_width += $chars[0];
 			$idx += 3;
@@ -567,8 +564,7 @@ 
 	if($time_zone < 0)
 	{
 		$to = -1;
-	}
-	else
+	} else
 	{
 		$to = 1;
 	}
@@ -580,8 +576,7 @@ 
 	if($server_time_zone < 0)
 	{
 		$so = -1;
-	}
-	else
+	} else
 	{
 		$so = 1;
 	}
@@ -622,8 +617,7 @@ 
 	if(strlen($str) <= 8)
 	{
 		$gap = 0;
-	}
-	else
+	} else
 	{
 		$gap = zgap();
 	}
@@ -646,20 +640,16 @@ 
 	if($gap < 60)
 	{
 		$buff = sprintf($lang_time_gap['min'], (int) ($gap / 60) + 1);
-	}
-	elseif($gap < 60 * 60)
+	} elseif($gap < 60 * 60)
 	{
 		$buff = sprintf($lang_time_gap['mins'], (int) ($gap / 60) + 1);
-	}
-	elseif($gap < 60 * 60 * 2)
+	} elseif($gap < 60 * 60 * 2)
 	{
 		$buff = sprintf($lang_time_gap['hour'], (int) ($gap / 60 / 60) + 1);
-	}
-	elseif($gap < 60 * 60 * 24)
+	} elseif($gap < 60 * 60 * 24)
 	{
 		$buff = sprintf($lang_time_gap['hours'], (int) ($gap / 60 / 60) + 1);
-	}
-	else
+	} else
 	{
 		$buff = zdate($date, $format);
 	}
@@ -706,12 +696,10 @@ 
 				if($format == 'Y-m-d')
 				{
 					$format = 'M d, Y';
-				}
-				elseif($format == 'Y-m-d H:i:s')
+				} elseif($format == 'Y-m-d H:i:s')
 				{
 					$format = 'M d, Y H:i:s';
-				}
-				elseif($format == 'Y-m-d H:i')
+				} elseif($format == 'Y-m-d H:i')
 				{
 					$format = 'M d, Y H:i';
 				}
@@ -720,12 +708,10 @@ 
 				if($format == 'Y-m-d')
 				{
 					$format = 'd-m-Y';
-				}
-				elseif($format == 'Y-m-d H:i:s')
+				} elseif($format == 'Y-m-d H:i:s')
 				{
 					$format = 'H:i:s d-m-Y';
-				}
-				elseif($format == 'Y-m-d H:i')
+				} elseif($format == 'Y-m-d H:i')
 				{
 					$format = 'H:i d-m-Y';
 				}
@@ -761,8 +747,7 @@ 
 		);
 
 		$string = strtr($format, $trans);
-	}
-	else
+	} else
 	{
 		// if year value is greater than 1970, get unixtime by using ztime() for date() function's argument. 
 		$string = date($format, ztime($str));
@@ -849,8 +834,7 @@ 
 		}
 
 		$firephp->fb($debug_output, $label, $type);
-	}
-	else
+	} else
 	{
 		if(__DEBUG_PROTECT__ === 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
 		{
@@ -862,7 +846,9 @@ 
 		{
 			$debug_file = _XE_PATH_ . 'files/' . $file;
 		}
-		if(!file_exists($debug_file)) $print[] = '<?php exit() ?>';
+		if(!file_exists($debug_file)) {
+			$print[] = '<?php exit() ?>';
+		}
 
 		if($display_option === TRUE || $display_option === 'ERROR')
 		{
@@ -875,13 +861,11 @@ 
 			if($display_option === 'ERROR')
 			{
 				$print[] = 'ERROR : ' . var_export($debug_output, TRUE);
-			}
-			else
+			} else
 			{
 				$print[] = 'DEBUG : ' . $type . '(' . var_export($debug_output, TRUE) . ')';
 			}
-		}
-		else
+		} else
 		{
 			$print[] = 'DEBUG : ' . trim(preg_replace('/\r?\n/', "\n" . '        ', print_r($debug_output, true)));
 		}
@@ -908,7 +892,9 @@ 
  */
 function writeSlowlog($type, $elapsed_time, $obj)
 {
-	if(!__LOG_SLOW_TRIGGER__ && !__LOG_SLOW_ADDON__ && !__LOG_SLOW_WIDGET__ && !__LOG_SLOW_QUERY__) return;
+	if(!__LOG_SLOW_TRIGGER__ && !__LOG_SLOW_ADDON__ && !__LOG_SLOW_WIDGET__ && !__LOG_SLOW_QUERY__) {
+		return;
+	}
 
 	static $log_filename = array(
 		'query' => 'files/_slowlog_query.php',
@@ -928,25 +914,21 @@ 
 	{
 		$buff[] = "\tCaller : " . $obj->caller;
 		$buff[] = "\tCalled : " . $obj->called;
-	}
-	else if($type == 'addon' && __LOG_SLOW_ADDON__ > 0 && $elapsed_time > __LOG_SLOW_ADDON__)
+	} else if($type == 'addon' && __LOG_SLOW_ADDON__ > 0 && $elapsed_time > __LOG_SLOW_ADDON__)
 	{
 		$buff[] = "\tAddon : " . $obj->called;
 		$buff[] = "\tCalled position : " . $obj->caller;
-	}
-	else if($type == 'widget' && __LOG_SLOW_WIDGET__ > 0 && $elapsed_time > __LOG_SLOW_WIDGET__)
+	} else if($type == 'widget' && __LOG_SLOW_WIDGET__ > 0 && $elapsed_time > __LOG_SLOW_WIDGET__)
 	{
 		$buff[] = "\tWidget : " . $obj->called;
-	}
-	else if($type == 'query' && __LOG_SLOW_QUERY__ > 0 && $elapsed_time > __LOG_SLOW_QUERY__)
+	} else if($type == 'query' && __LOG_SLOW_QUERY__ > 0 && $elapsed_time > __LOG_SLOW_QUERY__)
 	{
 
 		$buff[] = $obj->query;
 		$buff[] = "\tQuery ID   : " . $obj->query_id;
 		$buff[] = "\tCaller     : " . $obj->caller;
 		$buff[] = "\tConnection : " . $obj->connection;
-	}
-	else
+	} else
 	{
 		$write_file = false;
 	}
@@ -1038,8 +1020,11 @@ 
 
 	foreach($del_vars as $var)
 	{
-		if(is_array($vars)) unset($vars[$var]);
-		else if(is_object($vars)) unset($vars->$var);
+		if(is_array($vars)) {
+			unset($vars[$var]);
+		} else if(is_object($vars)) {
+			unset($vars->$var);
+		}
 	}
 
 	return $vars;
@@ -1183,8 +1168,7 @@ 
 		if(($close_xmp = strrpos($content, '</xmp>')) === FALSE)
 		{
 			$content .= '</xmp>';
-		}
-		else if($close_xmp < $start_xmp)
+		} else if($close_xmp < $start_xmp)
 		{
 			$content .= '</xmp>';
 		}
@@ -1241,12 +1225,18 @@ 
 
 	$filter_arrts = array('style', 'src', 'href');
 
-	if($tag === 'object') array_push($filter_arrts, 'data');
-	if($tag === 'param') array_push($filter_arrts, 'value');
+	if($tag === 'object') {
+		array_push($filter_arrts, 'data');
+	}
+	if($tag === 'param') {
+		array_push($filter_arrts, 'value');
+	}
 
 	foreach($filter_arrts as $attr)
 	{
-		if(!isset($attrs[$attr])) continue;
+		if(!isset($attrs[$attr])) {
+			continue;
+		}
 
 		$attr_value = rawurldecode($attrs[$attr]);
 		$attr_value = htmlspecialchars_decode($attr_value, ENT_COMPAT);
@@ -1410,16 +1400,14 @@ 
 				$unicode = hexdec($unicodeHexVal);
 				$decodedStr .= _code2utf($unicode);
 				$pos += 4;
-			}
-			else
+			} else
 			{
 				// we have an escaped ascii character
 				$hexVal = substr($source, $pos, 2);
 				$decodedStr .= chr(hexdec($hexVal));
 				$pos += 2;
 			}
-		}
-		else
+		} else
 		{
 			$decodedStr .= $charAt;
 			$pos++;
@@ -1611,8 +1599,7 @@ 
 	if(version_compare(PHP_VERSION, "5.3.0") < 0)
 	{
 		set_include_path(_XE_PATH_ . "libs/PEAR" . PATH_SEPARATOR . get_include_path());
-	}
-	else
+	} else
 	{
 		set_include_path(_XE_PATH_ . "libs/PEAR.1.9.5" . PATH_SEPARATOR . get_include_path());
 	}
@@ -1649,8 +1636,7 @@ 
 		{
 			return FALSE;
 		}
-	}
-	else
+	} else
 	{
 		$virtualSiteInfo = $oModuleModel->getSiteInfo($siteModuleInfo->site_srl);
 		if(strtolower($virtualSiteInfo->domain) != strtolower(Context::get('vid')) && !strstr(strtolower($virtualSiteInfo->domain), strtolower($referer['host'])))
@@ -1798,7 +1784,9 @@ 
  */
 function escape($str, $double_escape = true, $escape_defined_lang_code = false)
 {
-	if(!$escape_defined_lang_code && isDefinedLangCode($str)) return $str;
+	if(!$escape_defined_lang_code && isDefinedLangCode($str)) {
+		return $str;
+	}
 
 	$flags = ENT_QUOTES | ENT_SUBSTITUTE;
 	return htmlspecialchars($str, $flags, 'UTF-8', $double_escape);
@@ -1880,7 +1868,9 @@ 
  */
 function explode_with_escape($delimiter, $str, $limit = 0, $escape_char = '\\')
 {
-	if ($limit < 1) $limit = null;
+	if ($limit < 1) {
+		$limit = null;
+	}
 	$result = array();
 	$split = preg_split('/(?<!' . preg_quote($escape_char, '/') . ')' . preg_quote($delimiter, '/') . '/', $str, $limit);
 	foreach ($split as $piece)

modules/module/module.model.php

Spacing +368 added lines, -368 removed lines

@@ -19,26 +19,26 @@ 
 	 */
 	function isIDExists($id, $site_srl = 0)
 	{
-		if(!preg_match('/^[a-z]{1}([a-z0-9_]+)$/i',$id)) return true;
+		if (!preg_match('/^[a-z]{1}([a-z0-9_]+)$/i', $id)) return true;
 		// directory and rss/atom/api reserved checking, etc.
 		$dirs = FileHandler::readDir(_XE_PATH_);
 		$dirs[] = 'rss';
 		$dirs[] = 'atom';
 		$dirs[] = 'api';
-		if(in_array($id, $dirs)) return true;
+		if (in_array($id, $dirs)) return true;
 		// mid test
 		$args = new stdClass();
 		$args->mid = $id;
 		$args->site_srl = $site_srl;
 		$output = executeQuery('module.isExistsModuleName', $args);
-		if($output->data->count) return true;
+		if ($output->data->count) return true;
 		// vid test (check mid != vid if site_srl=0, which means it is not a virtual site)
-		if(!$site_srl)
+		if (!$site_srl)
 		{
 			$site_args = new stdClass();
 			$site_args->domain = $id;
 			$output = executeQuery('module.isExistsSiteDomain', $site_args);
-			if($output->data->count) return true;
+			if ($output->data->count) return true;
 		}
 
 		return false;
@@ -82,10 +82,10 @@ 
 	function getDefaultMid()
 	{
 		$default_url = Context::getDefaultUrl();
-		if($default_url && substr_compare($default_url, '/', -1) === 0) $default_url = substr($default_url, 0, -1);
+		if ($default_url && substr_compare($default_url, '/', -1) === 0) $default_url = substr($default_url, 0, -1);
 
 		$request_url = Context::getRequestUri();
-		if($request_url && substr_compare($request_url, '/', -1) === 0) $request_url = substr($request_url, 0, -1);
+		if ($request_url && substr_compare($request_url, '/', -1) === 0) $request_url = substr($request_url, 0, -1);
 
 		$default_url_parse = parse_url($default_url);
 		$request_url_parse = parse_url($request_url);
@@ -95,20 +95,20 @@ 
 		// Set up
 		$domain = '';
 		$site_info = NULL;
-		if($default_url && $default_url_parse['host'] != $request_url_parse['host'])
+		if ($default_url && $default_url_parse['host'] != $request_url_parse['host'])
 		{
 			$hostname = $request_url_parse['host'];
 			$path = $request_url_parse['path'];
 			$port = $request_url_parse['port'];
-			if(strlen($path) >= 1 && substr_compare($path, '/', -1) === 0) $path = substr($path, 0, -1);
+			if (strlen($path) >= 1 && substr_compare($path, '/', -1) === 0) $path = substr($path, 0, -1);
 
-			$domain = sprintf('%s%s%s', $hostname, $port && ($port != 80 )? ':'.$port  : '', $path);
+			$domain = sprintf('%s%s%s', $hostname, $port && ($port != 80) ? ':'.$port : '', $path);
 		}
 
-		if($domain === '')
+		if ($domain === '')
 		{
-			if(!$vid) $vid = $mid;
-			if($vid)
+			if (!$vid) $vid = $mid;
+			if ($vid)
 			{
 				$domain = $vid;
 			}
@@ -116,89 +116,89 @@ 
 
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
 		// If domain is set, look for subsite
-		if($domain !== '')
+		if ($domain !== '')
 		{
 			$site_info = false;
-			if($oCacheHandler->isSupport())
+			if ($oCacheHandler->isSupport())
 			{
-				$object_key = 'site_info:' . md5($domain);
+				$object_key = 'site_info:'.md5($domain);
 				$domain_cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 				$site_info = $oCacheHandler->get($domain_cache_key);
 			}
 
-			if($site_info === false)
+			if ($site_info === false)
 			{
 				$args = new stdClass();
 				$args->domain = $domain;
 				$output = executeQuery('module.getSiteInfoByDomain', $args);
 				$site_info = $output->data;
 
-				if($oCacheHandler->isSupport()) $oCacheHandler->put($domain_cache_key, $site_info);
+				if ($oCacheHandler->isSupport()) $oCacheHandler->put($domain_cache_key, $site_info);
 			}
 
-			if($site_info && $vid)
+			if ($site_info && $vid)
 			{
 				Context::set('vid', $site_info->domain, true);
-				if(strtolower($mid)==strtolower($site_info->domain)) Context::set('mid', $site_info->mid,true);
+				if (strtolower($mid) == strtolower($site_info->domain)) Context::set('mid', $site_info->mid, true);
 			}
-			if(!$site_info || !$site_info->domain) { $domain = ''; unset($site_info); }
+			if (!$site_info || !$site_info->domain) { $domain = ''; unset($site_info); }
 		}
 
 		// If no virtual website was found, get default website
-		if($domain === '')
+		if ($domain === '')
 		{
 			$site_info = false;
-			if($oCacheHandler->isSupport())
+			if ($oCacheHandler->isSupport())
 			{
 				$object_key = 'default_site';
 				$default_site_cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 				$site_info = $oCacheHandler->get($default_site_cache_key);
 			}
 
-			if($site_info === false)
+			if ($site_info === false)
 			{
 				$args = new stdClass();
 				$args->site_srl = 0;
 				$output = executeQuery('module.getSiteInfo', $args);
 				// Update the related informaion if there is no default site info
-				if(!$output->data)
+				if (!$output->data)
 				{
 					// Create a table if sites table doesn't exist
 					$oDB = &DB::getInstance();
-					if(!$oDB->isTableExists('sites')) $oDB->createTableByXmlFile(_XE_PATH_.'modules/module/schemas/sites.xml');
-					if(!$oDB->isTableExists('sites')) return;
+					if (!$oDB->isTableExists('sites')) $oDB->createTableByXmlFile(_XE_PATH_.'modules/module/schemas/sites.xml');
+					if (!$oDB->isTableExists('sites')) return;
 
 					// Get mid, language
 					$mid_output = $oDB->executeQuery('module.getDefaultMidInfo', $args);
 					$db_info = Context::getDBInfo();
 					$domain = Context::getDefaultUrl();
 					$url_info = parse_url($domain);
-					$domain = $url_info['host'].( (!empty($url_info['port'])&&$url_info['port']!=80)?':'.$url_info['port']:'').$url_info['path'];
+					$domain = $url_info['host'].((!empty($url_info['port']) && $url_info['port'] != 80) ? ':'.$url_info['port'] : '').$url_info['path'];
 
 					$site_args = new stdClass;
 					$site_args->site_srl = 0;
-					$site_args->index_module_srl  = $mid_output->data->module_srl;
+					$site_args->index_module_srl = $mid_output->data->module_srl;
 					$site_args->domain = $domain;
 					$site_args->default_language = $db_info->lang_type;
 
-					if($output->data && !$output->data->index_module_srl)
+					if ($output->data && !$output->data->index_module_srl)
 					{
 						$output = executeQuery('module.updateSite', $site_args);
 					}
 					else
 					{
 						$output = executeQuery('module.insertSite', $site_args);
-						if(!$output->toBool()) return $output;
+						if (!$output->toBool()) return $output;
 					}
 					$output = executeQuery('module.getSiteInfo', $args);
 				}
 				$site_info = $output->data;
-				if($oCacheHandler->isSupport()) $oCacheHandler->put($default_site_cache_key, $site_info);
+				if ($oCacheHandler->isSupport()) $oCacheHandler->put($default_site_cache_key, $site_info);
 			}
 		}
 
-		if(!$site_info->module_srl) return $site_info;
-		if(is_array($site_info) && $site_info->data[0]) $site_info = $site_info[0];
+		if (!$site_info->module_srl) return $site_info;
+		if (is_array($site_info) && $site_info->data[0]) $site_info = $site_info[0];
 		return $this->addModuleExtraVars($site_info);
 	}
 
@@ -207,48 +207,48 @@ 
 	 */
 	function getModuleInfoByMid($mid, $site_srl = 0, $columnList = array())
 	{
-		if(!$mid || ($mid && !preg_match("/^[a-z][a-z0-9_]+$/i", $mid)))
+		if (!$mid || ($mid && !preg_match("/^[a-z][a-z0-9_]+$/i", $mid)))
 		{
 			return;
 		}
 
 		$args = new stdClass();
 		$args->mid = $mid;
-		$args->site_srl = (int)$site_srl;
+		$args->site_srl = (int) $site_srl;
 
 		$module_srl = false;
 		$module_info = false;
 
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$object_key = 'module_srl:'.$mid.'_'.$site_srl;
 			$module_srl_cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 			$module_srl = $oCacheHandler->get($module_srl_cache_key);
-			if($module_srl)
+			if ($module_srl)
 			{
-				$object_key = 'mid_info:' . $module_srl;
+				$object_key = 'mid_info:'.$module_srl;
 				$module_info_cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 				$module_info = $oCacheHandler->get($module_info_cache_key);
 			}
 		}
 
-		if($module_info === false)
+		if ($module_info === false)
 		{
 			$output = executeQuery('module.getMidInfo', $args);
 			$module_info = $output->data;
-			if($oCacheHandler->isSupport())
+			if ($oCacheHandler->isSupport())
 			{
 				$oCacheHandler->put($module_srl_cache_key, $module_info->module_srl);
 
-				$object_key = 'mid_info:' . $module_info->module_srl;
+				$object_key = 'mid_info:'.$module_info->module_srl;
 				$module_info_cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 				$oCacheHandler->put($module_info_cache_key, $module_info);
 			}
 		}
 
 		$this->applyDefaultSkin($module_info);
-		if(!$module_info->module_srl && $module_info->data[0]) $module_info = $module_info->data[0];
+		if (!$module_info->module_srl && $module_info->data[0]) $module_info = $module_info->data[0];
 		return $this->addModuleExtraVars($module_info);
 	}
 
@@ -264,7 +264,7 @@ 
 		$menuItemSrl = Context::get('menu_item_srl');
 		$menuItemSrl = (!$menuItemSrl) ? $menu_item_srl : $menuItemSrl;
 
-		if(!$menuItemSrl)
+		if (!$menuItemSrl)
 		{
 			$this->stop(-1, 'msg_invalid_request');
 			return;
@@ -273,7 +273,7 @@ 
 		$args = new stdClass();
 		$args->menu_item_srl = $menuItemSrl;
 		$output = executeQuery('module.getModuleInfoByMenuItemSrl', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -297,12 +297,12 @@ 
 		$layoutInfoMobile = $layoutSrlMobile ? $oLayoutModel->getLayoutRawData($layoutSrlMobile, array('title')) : NULL;
 		$skinInfoPc = $this->loadSkinInfo(Modulehandler::getModulePath($moduleInfo->module), $skinNamePc);
 		$skinInfoMobile = $this->loadSkinInfo(Modulehandler::getModulePath($moduleInfo->module), $skinNameMobile, 'm.skins');
-		if(!$skinInfoPc)
+		if (!$skinInfoPc)
 		{
 			$skinInfoPc = new stdClass();
 			$skinInfoPc->title = $skinNamePc;
 		}
-		if(!$skinInfoMobile)
+		if (!$skinInfoMobile)
 		{
 			$skinInfoMobile = new stdClass();
 			$skinInfoMobile->title = $skinNameMobile;
@@ -321,23 +321,23 @@ 
 		$mid_info = false;
 
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$object_key = 'module_srl:'.$mid.'_'.$site_srl;
 			$module_srl_cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 			$module_srl = $oCacheHandler->get($module_srl_cache_key);
-			if($module_srl)
+			if ($module_srl)
 			{
-				$object_key = 'mid_info:' . $module_srl;
+				$object_key = 'mid_info:'.$module_srl;
 				$module_info_cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 				$mid_info = $oCacheHandler->get($module_info_cache_key);
 			}
 
-			if($mid_info === false)
+			if ($mid_info === false)
 			{
 				$oCacheHandler->put($module_srl_cache_key, $output->data->module_srl);
 
-				$object_key = 'mid_info:' . $output->data->module_srl;
+				$object_key = 'mid_info:'.$output->data->module_srl;
 				$module_info_cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 				$oCacheHandler->put($module_info_cache_key, $moduleInfo);
 			}
@@ -350,7 +350,7 @@ 
 
 		$moduleInfo = $this->addModuleExtraVars($moduleInfo);
 
-		if($moduleInfo->module == 'page' && $moduleInfo->page_type != 'ARTICLE')
+		if ($moduleInfo->module == 'page' && $moduleInfo->page_type != 'ARTICLE')
 		{
 			unset($moduleInfo->skin);
 			unset($moduleInfo->mskin);
@@ -369,32 +369,32 @@ 
 		$mid_info = false;
 
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
-			$object_key = 'mid_info:' . $module_srl;
+			$object_key = 'mid_info:'.$module_srl;
 			$cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 			$mid_info = $oCacheHandler->get($cache_key);
 		}
 
-		if($mid_info === false)
+		if ($mid_info === false)
 		{
 			// Get data
 			$args = new stdClass();
 			$args->module_srl = $module_srl;
 			$output = executeQuery('module.getMidInfo', $args);
-			if(!$output->toBool()) return;
+			if (!$output->toBool()) return;
 
 			$mid_info = $output->data;
 			$this->applyDefaultSkin($mid_info);
-			if($oCacheHandler->isSupport()) $oCacheHandler->put($cache_key, $mid_info);
+			if ($oCacheHandler->isSupport()) $oCacheHandler->put($cache_key, $mid_info);
 		}
 
-		if($mid_info && count($columnList))
+		if ($mid_info && count($columnList))
 		{
 			$module_info = new stdClass();
-			foreach($mid_info as $key => $item)
+			foreach ($mid_info as $key => $item)
 			{
-				if(in_array($key, $columnList))
+				if (in_array($key, $columnList))
 				{
 					$module_info->$key = $item;
 				}
@@ -415,12 +415,12 @@ 
 	 */
 	private function applyDefaultSkin(&$moduleInfo)
 	{
-		if($moduleInfo->is_skin_fix == 'N')
+		if ($moduleInfo->is_skin_fix == 'N')
 		{
 			$moduleInfo->skin = '/USE_DEFAULT/';
 		}
 
-		if($moduleInfo->is_mskin_fix == 'N')
+		if ($moduleInfo->is_mskin_fix == 'N')
 		{
 			$moduleInfo->mskin = '/USE_DEFAULT/';
 		}
@@ -438,7 +438,7 @@ 
 		$count = count($output->data);
 
 		$modules = array();
-		for($i=0;$i<$count;$i++)
+		for ($i = 0; $i < $count; $i++)
 		{
 			$modules[] = $output->data[$i];
 		}
@@ -450,11 +450,11 @@ 
 	 */
 	function getModulesInfo($module_srls, $columnList = array())
 	{
-		if(is_array($module_srls)) $module_srls = implode(',',$module_srls);
+		if (is_array($module_srls)) $module_srls = implode(',', $module_srls);
 		$args = new stdClass();
 		$args->module_srls = $module_srls;
 		$output = executeQueryArray('module.getModulesInfo', $args, $columnList);
-		if(!$output->toBool()) return;
+		if (!$output->toBool()) return;
 		return $this->addModuleExtraVars($output->data);
 	}
 
@@ -464,31 +464,31 @@ 
 	function addModuleExtraVars($module_info)
 	{
 		// Process although one or more module informaion is requested
-		if(!is_array($module_info)) $target_module_info = array($module_info);
+		if (!is_array($module_info)) $target_module_info = array($module_info);
 		else $target_module_info = $module_info;
 		// Get module_srl
 		$module_srls = array();
-		foreach($target_module_info as $key => $val)
+		foreach ($target_module_info as $key => $val)
 		{
 			$module_srl = $val->module_srl;
-			if(!$module_srl) continue;
+			if (!$module_srl) continue;
 			$module_srls[] = $val->module_srl;
 		}
 		// Extract extra information of the module and skin
 		$extra_vars = $this->getModuleExtraVars($module_srls);
-		if(!count($module_srls) || !count($extra_vars)) return $module_info;
+		if (!count($module_srls) || !count($extra_vars)) return $module_info;
 
-		foreach($target_module_info as $key => $val)
+		foreach ($target_module_info as $key => $val)
 		{
-			if(!$extra_vars[$val->module_srl] || !count($extra_vars[$val->module_srl])) continue;
-			foreach($extra_vars[$val->module_srl] as $k => $v)
+			if (!$extra_vars[$val->module_srl] || !count($extra_vars[$val->module_srl])) continue;
+			foreach ($extra_vars[$val->module_srl] as $k => $v)
 			{
-				if($target_module_info[$key]->{$k}) continue;
+				if ($target_module_info[$key]->{$k}) continue;
 				$target_module_info[$key]->{$k} = $v;
 			}
 		}
 
-		if(is_array($module_info)) return $target_module_info;
+		if (is_array($module_info)) return $target_module_info;
 		return $target_module_info[0];
 	}
 
@@ -499,37 +499,37 @@ 
 	{
 		$list = false;
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
-			if(count($args) === 1 && isset($args->site_srl))
+			if (count($args) === 1 && isset($args->site_srl))
 			{
-				$object_key = 'module:mid_list_' . $args->site_srl;
+				$object_key = 'module:mid_list_'.$args->site_srl;
 				$cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 				$list = $oCacheHandler->get($cache_key);
 			}
 		}
 
-		if($list === false)
+		if ($list === false)
 		{
-			if($oCacheHandler->isSupport() && count($args) === 1 && isset($args->site_srl))
+			if ($oCacheHandler->isSupport() && count($args) === 1 && isset($args->site_srl))
 			{
 				$columnList = array();
 			}
 
 			$output = executeQuery('module.getMidList', $args, $columnList);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 			$list = $output->data;
 
-			if($oCacheHandler->isSupport() && count($args) === 1 && isset($args->site_srl))
+			if ($oCacheHandler->isSupport() && count($args) === 1 && isset($args->site_srl))
 			{
 				$oCacheHandler->put($cache_key, $list);
 			}
 		}
-		if(!$list) return;
+		if (!$list) return;
 
-		if(!is_array($list)) $list = array($list);
+		if (!is_array($list)) $list = array($list);
 
-		foreach($list as $val)
+		foreach ($list as $val)
 		{
 			$mid_list[$val->mid] = $val;
 		}
@@ -543,10 +543,10 @@ 
 	function getModuleSrlList($args = null, $columnList = array())
 	{
 		$output = executeQueryArray('module.getMidList', $args, $columnList);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$list = $output->data;
-		if(!$list) return;
+		if (!$list) return;
 
 		return $list;
 	}
@@ -556,22 +556,22 @@ 
 	 */
 	function getModuleSrlByMid($mid)
 	{
-		if($mid && !is_array($mid)) $mid = explode(',',$mid);
-		if(is_array($mid)) $mid = "'".implode("','",$mid)."'";
+		if ($mid && !is_array($mid)) $mid = explode(',', $mid);
+		if (is_array($mid)) $mid = "'".implode("','", $mid)."'";
 
 		$site_module_info = Context::get('site_module_info');
 
 		$args = new stdClass;
 		$args->mid = $mid;
-		if($site_module_info) $args->site_srl = $site_module_info->site_srl;
+		if ($site_module_info) $args->site_srl = $site_module_info->site_srl;
 		$output = executeQuery('module.getModuleSrlByMid', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$list = $output->data;
-		if(!$list) return;
-		if(!is_array($list)) $list = array($list);
+		if (!$list) return;
+		if (!is_array($list)) $list = array($list);
 
-		foreach($list as $key => $val)
+		foreach ($list as $key => $val)
 		{
 			$module_srl_list[] = $val->module_srl;
 		}
@@ -587,33 +587,33 @@ 
 		$action_forward = false;
 		// cache controll
 		$oCacheHandler = CacheHandler::getInstance('object', NULL, TRUE);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$cache_key = 'action_forward';
 			$action_forward = $oCacheHandler->get($cache_key);
 		}
 
 		// retrieve and caching all registered action_forward
-		if($action_forward === false)
+		if ($action_forward === false)
 		{
 			$args = new stdClass();
-			$output = executeQueryArray('module.getActionForward',$args);
-			if(!$output->toBool()) return new stdClass;
-			if(!$output->data) $output->data = array();
+			$output = executeQueryArray('module.getActionForward', $args);
+			if (!$output->toBool()) return new stdClass;
+			if (!$output->data) $output->data = array();
 
 			$action_forward = array();
-			foreach($output->data as $item)
+			foreach ($output->data as $item)
 			{
 				$action_forward[$item->act] = $item;
 			}
 
-			if($oCacheHandler->isSupport())
+			if ($oCacheHandler->isSupport())
 			{
 				$oCacheHandler->put($cache_key, $action_forward);
 			}
 		}
 
-		if($action_forward[$act])
+		if ($action_forward[$act])
 		{
 			return $action_forward[$act];
 		}
@@ -628,25 +628,25 @@ 
 	 */
 	function getTriggers($trigger_name, $called_position)
 	{
-		if(is_null($GLOBALS['__triggers__']))
+		if (is_null($GLOBALS['__triggers__']))
 		{
 			$triggers = FALSE;
 			$oCacheHandler = CacheHandler::getInstance('object', NULL, TRUE);
-			if($oCacheHandler->isSupport())
+			if ($oCacheHandler->isSupport())
 			{
 				$cache_key = 'triggers';
 				$triggers = $oCacheHandler->get($cache_key);
 			}
-			if($triggers === FALSE)
+			if ($triggers === FALSE)
 			{
 				$output = executeQueryArray('module.getTriggers');
 				$triggers = $output->data;
-				if($output->toBool() && $oCacheHandler->isSupport())
+				if ($output->toBool() && $oCacheHandler->isSupport())
 				{
 					$oCacheHandler->put($cache_key, $triggers);
 				}
 			}
-			foreach($triggers as $item)
+			foreach ($triggers as $item)
 			{
 				$GLOBALS['__triggers__'][$item->trigger_name][$item->called_position][] = $item;
 			}
@@ -662,11 +662,11 @@ 
 	{
 		$triggers = $this->getTriggers($trigger_name, $called_position);
 
-		if($triggers && is_array($triggers))
+		if ($triggers && is_array($triggers))
 		{
-			foreach($triggers as $item)
+			foreach ($triggers as $item)
 			{
-				if($item->module == $module && $item->type == $type && $item->called_method == $called_method)
+				if ($item->module == $module && $item->type == $type && $item->called_method == $called_method)
 				{
 					return $item;
 				}
@@ -679,12 +679,12 @@ 
 	/**
 	 * @brief Get module extend
 	 */
-	function getModuleExtend($parent_module, $type, $kind='')
+	function getModuleExtend($parent_module, $type, $kind = '')
 	{
 		$key = $parent_module.'.'.$kind.'.'.$type;
 
 		$module_extend_info = $this->loadModuleExtends();
-		if(array_key_exists($key, $module_extend_info))
+		if (array_key_exists($key, $module_extend_info))
 		{
 			return $module_extend_info[$key];
 		}
@@ -700,29 +700,29 @@ 
 		$cache_file = './files/config/module_extend.php';
 		$cache_file = FileHandler::getRealPath($cache_file);
 
-		if(!isset($GLOBALS['__MODULE_EXTEND__']))
+		if (!isset($GLOBALS['__MODULE_EXTEND__']))
 		{
 			// check pre install
-			if(file_exists(FileHandler::getRealPath('./files')) && !file_exists($cache_file))
+			if (file_exists(FileHandler::getRealPath('./files')) && !file_exists($cache_file))
 			{
 				$arr = array();
 				$output = executeQueryArray('module.getModuleExtend');
-				if($output->data)
+				if ($output->data)
 				{
-					foreach($output->data as $v)
+					foreach ($output->data as $v)
 					{
 						$arr[] = sprintf("'%s.%s.%s' => '%s'", $v->parent_module, $v->kind, $v->type, $v->extend_module);
 					}
 				}
 
 				$str = '<?PHP return array(%s); ?>';
-				$str = sprintf($str, join(',',$arr));
+				$str = sprintf($str, join(',', $arr));
 
 				FileHandler::writeFile($cache_file, $str);
 			}
 
 
-			if(file_exists($cache_file))
+			if (file_exists($cache_file))
 			{
 				$GLOBALS['__MODULE_EXTEND__'] = include($cache_file);
 			}
@@ -742,20 +742,20 @@ 
 	{
 		// Get a path of the requested module. Return if not exists.
 		$module_path = ModuleHandler::getModulePath($module);
-		if(!$module_path) return;
+		if (!$module_path) return;
 		// Read the xml file for module skin information
 		$xml_file = sprintf("%s/conf/info.xml", $module_path);
-		if(!file_exists($xml_file)) return;
+		if (!file_exists($xml_file)) return;
 
 		$oXmlParser = new XmlParser();
 		$tmp_xml_obj = $oXmlParser->loadXmlFile($xml_file);
 		$xml_obj = $tmp_xml_obj->module;
 
-		if(!$xml_obj) return;
+		if (!$xml_obj) return;
 
 		// Module Information
 		$module_info = new stdClass();
-		if($xml_obj->version && $xml_obj->attrs->version == '0.2')
+		if ($xml_obj->version && $xml_obj->attrs->version == '0.2')
 		{
 			// module format 0.2
 			$module_info->title = $xml_obj->title->body;
@@ -763,16 +763,16 @@ 
 			$module_info->version = $xml_obj->version->body;
 			$module_info->homepage = $xml_obj->link->body;
 			$module_info->category = $xml_obj->category->body;
-			if(!$module_info->category) $module_info->category = 'service';
+			if (!$module_info->category) $module_info->category = 'service';
 			sscanf($xml_obj->date->body, '%d-%d-%d', $date_obj->y, $date_obj->m, $date_obj->d);
 			$module_info->date = sprintf('%04d%02d%02d', $date_obj->y, $date_obj->m, $date_obj->d);
 			$module_info->license = $xml_obj->license->body;
 			$module_info->license_link = $xml_obj->license->attrs->link;
 
-			if(!is_array($xml_obj->author)) $author_list[] = $xml_obj->author;
+			if (!is_array($xml_obj->author)) $author_list[] = $xml_obj->author;
 			else $author_list = $xml_obj->author;
 
-			foreach($author_list as $author)
+			foreach ($author_list as $author)
 			{
 				$author_obj = new stdClass();
 				$author_obj->name = $author->name->body;
@@ -788,7 +788,7 @@ 
 			$module_info->description = $xml_obj->author->description->body;
 			$module_info->version = $xml_obj->attrs->version;
 			$module_info->category = $xml_obj->attrs->category;
-			if(!$module_info->category) $module_info->category = 'service';
+			if (!$module_info->category) $module_info->category = 'service';
 			sscanf($xml_obj->author->attrs->date, '%d. %d. %d', $date_obj->y, $date_obj->m, $date_obj->d);
 			$module_info->date = sprintf('%04d%02d%02d', $date_obj->y, $date_obj->m, $date_obj->d);
 			$author_obj = new stdClass();
@@ -817,17 +817,17 @@ 
 	{
 		// Get a path of the requested module. Return if not exists.
 		$class_path = ModuleHandler::getModulePath($module);
-		if(!$class_path) return;
+		if (!$class_path) return;
 
 		// Check if module.xml exists in the path. Return if not exist
 		$xml_file = sprintf("%sconf/module.xml", $class_path);
-		if(!file_exists($xml_file)) return;
+		if (!file_exists($xml_file)) return;
 
 		// Check if cached file exists
-		$cache_file = sprintf(_XE_PATH_ . "files/cache/module_info/%s.%s.%s.php", $module, Context::getLangType(), __XE_VERSION__);
+		$cache_file = sprintf(_XE_PATH_."files/cache/module_info/%s.%s.%s.php", $module, Context::getLangType(), __XE_VERSION__);
 
 		// Update if no cache file exists or it is older than xml file
-		if(!file_exists($cache_file) || filemtime($cache_file) < filemtime($xml_file) || $re_cache)
+		if (!file_exists($cache_file) || filemtime($cache_file) < filemtime($xml_file) || $re_cache)
 		{
 			$info = new stdClass();
 			$buff = array(); // /< Set buff variable to use in the cache file
@@ -840,7 +840,7 @@ 
 
 			$xml_obj = XmlParser::loadXmlFile($xml_file); // /< Read xml file and convert it to xml object
 
-			if(!count($xml_obj->module)) return; // /< Error occurs if module tag doesn't included in the xml
+			if (!count($xml_obj->module)) return; // /< Error occurs if module tag doesn't included in the xml
 
 			$grants = $xml_obj->module->grants->grant; // /< Permission information
 			$permissions = $xml_obj->module->permissions->permission; // /<  Acting permission
@@ -850,17 +850,17 @@ 
 			$default_index = $admin_index = '';
 
 			// Arrange permission information
-			if($grants)
+			if ($grants)
 			{
-				if(is_array($grants)) $grant_list = $grants;
+				if (is_array($grants)) $grant_list = $grants;
 				else $grant_list[] = $grants;
 
 				$info->grant = new stdClass();
 				$buff[] = '$info->grant = new stdClass;';
-				foreach($grant_list as $grant)
+				foreach ($grant_list as $grant)
 				{
 					$name = $grant->attrs->name;
-					$default = $grant->attrs->default?$grant->attrs->default:'guest';
+					$default = $grant->attrs->default ? $grant->attrs->default : 'guest';
 					$title = $grant->title->body;
 
 					$info->grant->{$name} = new stdClass();
@@ -873,15 +873,15 @@ 
 				}
 			}
 			// Permissions to grant
-			if($permissions)
+			if ($permissions)
 			{
-				if(is_array($permissions)) $permission_list = $permissions;
+				if (is_array($permissions)) $permission_list = $permissions;
 				else $permission_list[] = $permissions;
 
 				$buff[] = '$info->permission = new stdClass;';
 
 				$info->permission = new stdClass();
-				foreach($permission_list as $permission)
+				foreach ($permission_list as $permission)
 				{
 					$action = $permission->attrs->action;
 					$target = $permission->attrs->target;
@@ -892,14 +892,14 @@ 
 				}
 			}
 			// for admin menus
-			if($menus)
+			if ($menus)
 			{
-				if(is_array($menus)) $menu_list = $menus;
+				if (is_array($menus)) $menu_list = $menus;
 				else $menu_list[] = $menus;
 
 				$buff[] = '$info->menu = new stdClass;';
 				$info->menu = new stdClass();
-				foreach($menu_list as $menu)
+				foreach ($menu_list as $menu)
 				{
 					$menu_name = $menu->attrs->name;
 					$menu_title = is_array($menu->title) ? $menu->title[0]->body : $menu->title->body;
@@ -917,23 +917,23 @@ 
 			}
 
 			// actions
-			if($actions)
+			if ($actions)
 			{
-				if(is_array($actions)) $action_list = $actions;
+				if (is_array($actions)) $action_list = $actions;
 				else $action_list[] = $actions;
 
 				$buff[] = '$info->action = new stdClass;';
 				$info->action = new stdClass();
-				foreach($action_list as $action)
+				foreach ($action_list as $action)
 				{
 					$name = $action->attrs->name;
 
 					$type = $action->attrs->type;
-					$grant = $action->attrs->grant?$action->attrs->grant:'guest';
-					$standalone = $action->attrs->standalone=='false'?'false':'true';
-					$ruleset = $action->attrs->ruleset?$action->attrs->ruleset:'';
-					$method = $action->attrs->method?$action->attrs->method:'';
-					$check_csrf = $action->attrs->check_csrf=='false'?'false':'true';
+					$grant = $action->attrs->grant ? $action->attrs->grant : 'guest';
+					$standalone = $action->attrs->standalone == 'false' ? 'false' : 'true';
+					$ruleset = $action->attrs->ruleset ? $action->attrs->ruleset : '';
+					$method = $action->attrs->method ? $action->attrs->method : '';
+					$check_csrf = $action->attrs->check_csrf == 'false' ? 'false' : 'true';
 					$meta_noindex = $action->attrs->{'meta-noindex'} === 'true' ? 'true' : 'false';
 
 					$index = $action->attrs->index;
@@ -950,14 +950,14 @@ 
 					$info->action->{$name}->method = $method;
 					$info->action->{$name}->check_csrf = $check_csrf;
 					$info->action->{$name}->meta_noindex = $meta_noindex;
-					if($action->attrs->menu_name)
+					if ($action->attrs->menu_name)
 					{
-						if($menu_index == 'true')
+						if ($menu_index == 'true')
 						{
 							$info->menu->{$action->attrs->menu_name}->index = $name;
 							$buff[] = sprintf('$info->menu->%s->index=\'%s\';', $action->attrs->menu_name, $name);
 						}
-						if(is_array($info->menu->{$action->attrs->menu_name}->acts))
+						if (is_array($info->menu->{$action->attrs->menu_name}->acts))
 						{
 							$info->menu->{$action->attrs->menu_name}->acts[] = $name;
 							$currentKey = array_search($name, $info->menu->{$action->attrs->menu_name}->acts);
@@ -976,22 +976,22 @@ 
 					$buff[] = sprintf('$info->action->%s->check_csrf=\'%s\';', $name, $check_csrf);
 					$buff[] = sprintf('$info->action->%s->meta_noindex=\'%s\';', $name, $meta_noindex);
 
-					if($index=='true')
+					if ($index == 'true')
 					{
 						$default_index_act = $name;
 						$info->default_index_act = $name;
 					}
-					if($admin_index=='true')
+					if ($admin_index == 'true')
 					{
 						$admin_index_act = $name;
 						$info->admin_index_act = $name;
 					}
-					if($setup_index=='true')
+					if ($setup_index == 'true')
 					{
 						$setup_index_act = $name;
 						$info->setup_index_act = $name;
 					}
-					if($simple_setup_index=='true')
+					if ($simple_setup_index == 'true')
 					{
 						$simple_setup_index_act = $name;
 						$info->simple_setup_index_act = $name;
@@ -1012,7 +1012,7 @@ 
 			return $info;
 		}
 
-		if(file_exists($cache_file)) return include($cache_file);
+		if (file_exists($cache_file)) return include($cache_file);
 	}
 
 	/**
@@ -1023,7 +1023,7 @@ 
 	{
 		$module = Context::get('module_type');
 
-		if($module == 'ARTICLE')
+		if ($module == 'ARTICLE')
 		{
 			$module = 'page';
 		}
@@ -1043,26 +1043,26 @@ 
 	 */
 	function getSkins($path, $dir = 'skins')
 	{
-		if(substr($path, -1) == '/')
+		if (substr($path, -1) == '/')
 		{
 			$path = substr($path, 0, -1);
 		}
 
 		$skin_path = sprintf("%s/%s/", $path, $dir);
 		$list = FileHandler::readDir($skin_path);
-		if(!count($list)) return;
+		if (!count($list)) return;
 
 		natcasesort($list);
 
-		foreach($list as $skin_name)
+		foreach ($list as $skin_name)
 		{
-			if(!is_dir($skin_path . $skin_name))
+			if (!is_dir($skin_path.$skin_name))
 			{
 				continue;
 			}
 			unset($skin_info);
 			$skin_info = $this->loadSkinInfo($path, $skin_name, $dir);
-			if(!$skin_info)
+			if (!$skin_info)
 			{
 				$skin_info = new stdClass();
 				$skin_info->title = $skin_name;
@@ -1075,15 +1075,15 @@ 
 		$tmpPath = trim($tmpPath);
 		$module = array_pop(explode(' ', $tmpPath));
 
-		if($dir == 'skins')
+		if ($dir == 'skins')
 		{
 			$oAdminModel = getAdminModel('admin');
 			$themesInfo = $oAdminModel->getThemeList();
 
-			foreach($themesInfo as $themeName => $info)
+			foreach ($themesInfo as $themeName => $info)
 			{
 				$skinInfos = $info->skin_infos;
-				if(isset($skinInfos[$module]) && $skinInfos[$module]->is_theme)
+				if (isset($skinInfos[$module]) && $skinInfos[$module]->is_theme)
 				{
 					$themeSkinInfo = $GLOBALS['__ThemeModuleSkin__'][$module]['skins'][$skinInfos[$module]->name];
 					$skin_list[$skinInfos[$module]->name] = $themeSkinInfo;
@@ -1095,13 +1095,13 @@ 
 		$oMenuAdminModel = getAdminModel('menu');
 		$installedMenuTypes = $oMenuAdminModel->getModuleListInSitemap($siteInfo->site_srl);
 		$moduleName = $module;
-		if($moduleName === 'page')
+		if ($moduleName === 'page')
 		{
 			$moduleName = 'ARTICLE';
 		}
-		if(array_key_exists($moduleName, $installedMenuTypes))
+		if (array_key_exists($moduleName, $installedMenuTypes))
 		{
-			if($dir == 'skins')
+			if ($dir == 'skins')
 			{
 				$type = 'P';
 			}
@@ -1111,12 +1111,12 @@ 
 			}
 			$defaultSkinName = $this->getModuleDefaultSkin($module, $type, $site_info->site_srl);
 
-			if(isset($defaultSkinName))
+			if (isset($defaultSkinName))
 			{
 				$defaultSkinInfo = $this->loadSkinInfo($path, $defaultSkinName, $dir);
 
 				$useDefault = new stdClass();
-				$useDefault->title = Context::getLang('use_site_default_skin') . ' (' . $defaultSkinInfo->title . ')';
+				$useDefault->title = Context::getLang('use_site_default_skin').' ('.$defaultSkinInfo->title.')';
 
 				$useDefaultList['/USE_DEFAULT/'] = $useDefault;
 
@@ -1133,20 +1133,20 @@ 
 	function loadSkinInfo($path, $skin, $dir = 'skins')
 	{
 		// Read xml file having skin information
-		if(substr($path,-1)!='/') $path .= '/';
+		if (substr($path, -1) != '/') $path .= '/';
 		$skin_xml_file = sprintf("%s%s/%s/skin.xml", $path, $dir, $skin);
-		if(!file_exists($skin_xml_file)) return;
+		if (!file_exists($skin_xml_file)) return;
 		// Create XmlParser object
 		$oXmlParser = new XmlParser();
 		$_xml_obj = $oXmlParser->loadXmlFile($skin_xml_file);
 		// Return if no skin information is
-		if(!$_xml_obj->skin) return;
+		if (!$_xml_obj->skin) return;
 		$xml_obj = $_xml_obj->skin;
 		// Skin Name
 		$skin_info = new stdClass();
 		$skin_info->title = $xml_obj->title->body;
 		// Author information
-		if($xml_obj->version && $xml_obj->attrs->version == '0.2')
+		if ($xml_obj->version && $xml_obj->attrs->version == '0.2')
 		{
 			// skin format v0.2
 			sscanf($xml_obj->date->body, '%d-%d-%d', $date_obj->y, $date_obj->m, $date_obj->d);
@@ -1157,10 +1157,10 @@ 
 			$skin_info->license_link = $xml_obj->license->attrs->link;
 			$skin_info->description = $xml_obj->description->body;
 
-			if(!is_array($xml_obj->author)) $author_list[] = $xml_obj->author;
+			if (!is_array($xml_obj->author)) $author_list[] = $xml_obj->author;
 			else $author_list = $xml_obj->author;
 
-			foreach($author_list as $author)
+			foreach ($author_list as $author)
 			{
 				$author_obj = new stdClass();
 				$author_obj->name = $author->name->body;
@@ -1169,25 +1169,25 @@ 
 				$skin_info->author[] = $author_obj;
 			}
 			// List extra vars
-			if($xml_obj->extra_vars)
+			if ($xml_obj->extra_vars)
 			{
 				$extra_var_groups = $xml_obj->extra_vars->group;
-				if(!$extra_var_groups) $extra_var_groups = $xml_obj->extra_vars;
-				if(!is_array($extra_var_groups)) $extra_var_groups = array($extra_var_groups);
+				if (!$extra_var_groups) $extra_var_groups = $xml_obj->extra_vars;
+				if (!is_array($extra_var_groups)) $extra_var_groups = array($extra_var_groups);
 
-				foreach($extra_var_groups as $group)
+				foreach ($extra_var_groups as $group)
 				{
 					$extra_vars = $group->var;
-					if(!$extra_vars)
+					if (!$extra_vars)
 					{
 						continue;
 					}
-					if(!is_array($group->var)) $extra_vars = array($group->var);
+					if (!is_array($group->var)) $extra_vars = array($group->var);
 
-					foreach($extra_vars as $key => $val)
+					foreach ($extra_vars as $key => $val)
 					{
 						$obj = new stdClass();
-						if(!$val->attrs->type) { $val->attrs->type = 'text'; }
+						if (!$val->attrs->type) { $val->attrs->type = 'text'; }
 
 						$obj->group = $group->title->body;
 						$obj->name = $val->attrs->name;
@@ -1196,14 +1196,14 @@ 
 						$obj->description = $val->description->body;
 						$obj->value = $extra_vals->{$obj->name};
 						$obj->default = $val->attrs->default;
-						if(strpos($obj->value, '|@|') != false) { $obj->value = explode('|@|', $obj->value); }
-						if($obj->type == 'mid_list' && !is_array($obj->value)) { $obj->value = array($obj->value); }
+						if (strpos($obj->value, '|@|') != false) { $obj->value = explode('|@|', $obj->value); }
+						if ($obj->type == 'mid_list' && !is_array($obj->value)) { $obj->value = array($obj->value); }
 						// Get an option list from 'select'type
-						if(is_array($val->options))
+						if (is_array($val->options))
 						{
 							$option_count = count($val->options);
 
-							for($i = 0; $i < $option_count; $i++)
+							for ($i = 0; $i < $option_count; $i++)
 							{
 								$obj->options[$i] = new stdClass();
 								$obj->options[$i]->title = $val->options[$i]->title->body;
@@ -1240,18 +1240,18 @@ 
 			$skin_info->author[0]->homepage = $xml_obj->maker->attrs->link;
 			// Variables used in the skin
 			$extra_var_groups = $xml_obj->extra_vars->group;
-			if(!$extra_var_groups) $extra_var_groups = $xml_obj->extra_vars;
-			if(!is_array($extra_var_groups)) $extra_var_groups = array($extra_var_groups);
+			if (!$extra_var_groups) $extra_var_groups = $xml_obj->extra_vars;
+			if (!is_array($extra_var_groups)) $extra_var_groups = array($extra_var_groups);
 
-			foreach($extra_var_groups as $group)
+			foreach ($extra_var_groups as $group)
 			{
 				$extra_vars = $group->var;
 
-				if($extra_vars)
+				if ($extra_vars)
 				{
-					if(!is_array($extra_vars)) $extra_vars = array($extra_vars);
+					if (!is_array($extra_vars)) $extra_vars = array($extra_vars);
 
-					foreach($extra_vars as $var)
+					foreach ($extra_vars as $var)
 					{
 						unset($obj);
 						unset($options);
@@ -1262,11 +1262,11 @@ 
 						$title = $var->title->body;
 						$description = $var->description->body;
 						// Get an option list from 'select'type.
-						if(is_array($var->default))
+						if (is_array($var->default))
 						{
 							$option_count = count($var->default);
 
-							for($i = 0; $i < $option_count; $i++)
+							for ($i = 0; $i < $option_count; $i++)
 							{
 								$options[$i]->title = $var->default[$i]->body;
 								$options[$i]->value = $var->default[$i]->body;
@@ -1300,19 +1300,19 @@ 
 
 		// colorset
 		$colorset = $xml_obj->colorset->color;
-		if($colorset)
+		if ($colorset)
 		{
-			if(!is_array($colorset)) $colorset = array($colorset);
+			if (!is_array($colorset)) $colorset = array($colorset);
 
-			foreach($colorset as $color)
+			foreach ($colorset as $color)
 			{
 				$name = $color->attrs->name;
 				$title = $color->title->body;
 				$screenshot = $color->attrs->src;
-				if($screenshot)
+				if ($screenshot)
 				{
 					$screenshot = sprintf("%s%s/%s/%s", $path, $dir, $skin, $screenshot);
-					if(!file_exists($screenshot)) $screenshot = "";
+					if (!file_exists($screenshot)) $screenshot = "";
 				}
 				else $screenshot = "";
 
@@ -1324,19 +1324,19 @@ 
 			}
 		}
 		// Menu type (settings for layout)
-		if($xml_obj->menus->menu)
+		if ($xml_obj->menus->menu)
 		{
 			$menus = $xml_obj->menus->menu;
-			if(!is_array($menus)) $menus = array($menus);
+			if (!is_array($menus)) $menus = array($menus);
 
 			$menu_count = count($menus);
 			$skin_info->menu_count = $menu_count;
-			for($i=0;$i<$menu_count;$i++)
+			for ($i = 0; $i < $menu_count; $i++)
 			{
 				unset($obj);
 
 				$obj->name = $menus[$i]->attrs->name;
-				if($menus[$i]->attrs->default == "true") $obj->default = true;
+				if ($menus[$i]->attrs->default == "true") $obj->default = true;
 				$obj->title = $menus[$i]->title->body;
 				$obj->maxdepth = $menus[$i]->maxdepth->body;
 
@@ -1345,7 +1345,7 @@ 
 		}
 
 		$thumbnail = sprintf("%s%s/%s/thumbnail.png", $path, $dir, $skin);
-		$skin_info->thumbnail = (file_exists($thumbnail))?$thumbnail:null;
+		$skin_info->thumbnail = (file_exists($thumbnail)) ? $thumbnail : null;
 		return $skin_info;
 	}
 
@@ -1356,7 +1356,7 @@ 
 	{
 		$args = new stdClass;
 		$args->site_srl = $site_srl;
-		if(!is_null($module)) $args->module = $module;
+		if (!is_null($module)) $args->module = $module;
 		$output = executeQuery('module.getModuleCount', $args);
 		return $output->data->count;
 	}
@@ -1370,26 +1370,26 @@ 
 		$config = false;
 		// cache controll
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
-			$object_key = 'module_config:' . $module . '_' . $site_srl;
+			$object_key = 'module_config:'.$module.'_'.$site_srl;
 			$cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 			$config = $oCacheHandler->get($cache_key);
 		}
 
-		if($config === false)
+		if ($config === false)
 		{
-			if(!$GLOBALS['__ModuleConfig__'][$site_srl][$module])
+			if (!$GLOBALS['__ModuleConfig__'][$site_srl][$module])
 			{
 				$args = new stdClass();
 				$args->module = $module;
 				$args->site_srl = $site_srl;
 				$output = executeQuery('module.getModuleConfig', $args);
-				if($output->data->config) $config = unserialize($output->data->config);
+				if ($output->data->config) $config = unserialize($output->data->config);
 				else $config = null;
 
 				//insert in cache
-				if($oCacheHandler->isSupport())
+				if ($oCacheHandler->isSupport())
 				{
 					$oCacheHandler->put($cache_key, $config);
 				}
@@ -1410,26 +1410,26 @@ 
 		$config = false;
 		// cache controll
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$object_key = 'module_part_config:'.$module.'_'.$module_srl;
 			$cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 			$config = $oCacheHandler->get($cache_key);
 		}
 
-		if($config === false)
+		if ($config === false)
 		{
-			if(!isset($GLOBALS['__ModulePartConfig__'][$module][$module_srl]))
+			if (!isset($GLOBALS['__ModulePartConfig__'][$module][$module_srl]))
 			{
 				$args = new stdClass();
 				$args->module = $module;
 				$args->module_srl = $module_srl;
 				$output = executeQuery('module.getModulePartConfig', $args);
-				if($output->data->config) $config = unserialize($output->data->config);
+				if ($output->data->config) $config = unserialize($output->data->config);
 				else $config = null;
 
 				//insert in cache
-				if($oCacheHandler->isSupport())
+				if ($oCacheHandler->isSupport())
 				{
 					$oCacheHandler->put($cache_key, $config);
 				}
@@ -1448,11 +1448,11 @@ 
 	{
 		$args = new stdClass();
 		$args->module = $module;
-		if($site_srl) $args->site_srl = $site_srl;
+		if ($site_srl) $args->site_srl = $site_srl;
 		$output = executeQueryArray('module.getModulePartConfigs', $args);
-		if(!$output->toBool() || !$output->data) return array();
+		if (!$output->toBool() || !$output->data) return array();
 
-		foreach($output->data as $key => $val)
+		foreach ($output->data as $key => $val)
 		{
 			$result[$val->module_srl] = unserialize($val->config);
 		}
@@ -1468,12 +1468,12 @@ 
 		$args->moduleCategorySrl = $moduleCategorySrl;
 		// Get data from the DB
 		$output = executeQuery('module.getModuleCategories', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 		$list = $output->data;
-		if(!$list) return;
-		if(!is_array($list)) $list = array($list);
+		if (!$list) return;
+		if (!is_array($list)) $list = array($list);
 
-		foreach($list as $val)
+		foreach ($list as $val)
 		{
 			$category_list[$val->module_category_srl] = $val;
 		}
@@ -1489,7 +1489,7 @@ 
 		$args = new stdClass;
 		$args->module_category_srl = $module_category_srl;
 		$output = executeQuery('module.getModuleCategory', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 		return $output->data;
 	}
 
@@ -1501,10 +1501,10 @@ 
 		// Get a list of downloaded and installed modules
 		$searched_list = FileHandler::readDir('./modules');
 		$searched_count = count($searched_list);
-		if(!$searched_count) return;
+		if (!$searched_count) return;
 		sort($searched_list);
 
-		for($i=0;$i<$searched_count;$i++)
+		for ($i = 0; $i < $searched_count; $i++)
 		{
 			// Module name
 			$module_name = $searched_list[$i];
@@ -1514,7 +1514,7 @@ 
 			$info = $this->getModuleInfoXml($module_name);
 			unset($obj);
 
-			if(!isset($info)) continue;
+			if (!isset($info)) continue;
 			$info->module = $module_name;
 			$info->created_table_count = $created_table_count;
 			$info->table_count = $table_count;
@@ -1531,19 +1531,19 @@ 
 		$info = null;
 
 		$moduledir = ModuleHandler::getModulePath($module_name);
-		if(file_exists(FileHandler::getRealPath($moduledir."schemas")))
+		if (file_exists(FileHandler::getRealPath($moduledir."schemas")))
 		{
 			$tmp_files = FileHandler::readDir($moduledir."schemas", '/(\.xml)$/');
 			$table_count = count($tmp_files);
 			// Check if the table is created
 			$created_table_count = 0;
-			for($j=0;$j<count($tmp_files);$j++)
+			for ($j = 0; $j < count($tmp_files); $j++)
 			{
-				list($table_name) = explode(".",$tmp_files[$j]);
-				if($oDB->isTableExists($table_name)) $created_table_count ++;
+				list($table_name) = explode(".", $tmp_files[$j]);
+				if ($oDB->isTableExists($table_name)) $created_table_count++;
 			}
 			// Check if DB is installed
-			if($table_count > $created_table_count) return true;
+			if ($table_count > $created_table_count) return true;
 			else return false;
 		}
 		return false;
@@ -1553,7 +1553,7 @@ 
 	{
 		// Check if it is upgraded to module.class.php on each module
 		$oDummy = getModule($module_name, 'class');
-		if($oDummy && method_exists($oDummy, "checkUpdate"))
+		if ($oDummy && method_exists($oDummy, "checkUpdate"))
 		{
 			return $oDummy->checkUpdate();
 		}
@@ -1567,15 +1567,15 @@ 
 	 */
 	public function needUpdate($update_id)
 	{
-		if(!is_array($update_id)) $update_id = array($update_id);
+		if (!is_array($update_id)) $update_id = array($update_id);
 
 		$args = new stdClass();
 		$args->update_id = implode(',', $update_id);
 		$output = executeQueryArray('module.getModuleUpdateLog', $args);
 
-		if(!!$output->error) return false;
-		if(!$output->data) $output->data = array();
-		if(count($update_id) === count($output->data)) return false;
+		if (!!$output->error) return false;
+		if (!$output->data) $output->data = array();
+		if (count($update_id) === count($output->data)) return false;
 
 		return true;
 	}
@@ -1592,31 +1592,31 @@ 
 		sort($searched_list);
 
 		$searched_count = count($searched_list);
-		if(!$searched_count) return;
+		if (!$searched_count) return;
 
-		for($i=0;$i<$searched_count;$i++)
+		for ($i = 0; $i < $searched_count; $i++)
 		{
 			// module name
 			$module_name = $searched_list[$i];
 
 			$path = ModuleHandler::getModulePath($module_name);
-			if(!is_dir(FileHandler::getRealPath($path))) continue;
+			if (!is_dir(FileHandler::getRealPath($path))) continue;
 
 			// Get the number of xml files to create a table in schemas
 			$tmp_files = FileHandler::readDir($path.'schemas', '/(\.xml)$/');
 			$table_count = count($tmp_files);
 			// Check if the table is created
 			$created_table_count = 0;
-			for($j=0;$j<$table_count;$j++)
+			for ($j = 0; $j < $table_count; $j++)
 			{
-				list($table_name) = explode('.',$tmp_files[$j]);
-				if($oDB->isTableExists($table_name)) $created_table_count ++;
+				list($table_name) = explode('.', $tmp_files[$j]);
+				if ($oDB->isTableExists($table_name)) $created_table_count++;
 			}
 			// Get information of the module
 			$info = NULL;
 			$info = $this->getModuleInfoXml($module_name);
 
-			if(!$info) continue;
+			if (!$info) continue;
 
 			$info->module = $module_name;
 			$info->category = $info->category;
@@ -1625,12 +1625,12 @@ 
 			$info->path = $path;
 			$info->admin_index_act = $info->admin_index_act;
 			// Check if DB is installed
-			if($table_count > $created_table_count) $info->need_install = true;
+			if ($table_count > $created_table_count) $info->need_install = true;
 			else $info->need_install = false;
 			// Check if it is upgraded to module.class.php on each module
 			$oDummy = null;
 			$oDummy = getModule($module_name, 'class');
-			if($oDummy && method_exists($oDummy, "checkUpdate"))
+			if ($oDummy && method_exists($oDummy, "checkUpdate"))
 			{
 				$info->need_update = $oDummy->checkUpdate();
 			}
@@ -1651,15 +1651,15 @@ 
 	 */
 	function syncModuleToSite(&$data)
 	{
-		if(!$data) return;
+		if (!$data) return;
 
-		if(is_array($data))
+		if (is_array($data))
 		{
-			foreach($data as $key => $val)
+			foreach ($data as $key => $val)
 			{
 				$module_srls[] = $val->module_srl;
 			}
-			if(!count($module_srls)) return;
+			if (!count($module_srls)) return;
 		}
 		else
 		{
@@ -1667,17 +1667,17 @@ 
 		}
 
 		$args = new stdClass();
-		$args->module_srls = implode(',',$module_srls);
+		$args->module_srls = implode(',', $module_srls);
 		$output = executeQueryArray('module.getModuleSites', $args);
-		if(!$output->data) return array();
-		foreach($output->data as $key => $val)
+		if (!$output->data) return array();
+		foreach ($output->data as $key => $val)
 		{
 			$modules[$val->module_srl] = $val;
 		}
 
-		if(is_array($data))
+		if (is_array($data))
 		{
-			foreach($data as $key => $val)
+			foreach ($data as $key => $val)
 			{
 				$data[$key]->domain = $modules[$val->module_srl]->domain;
 			}
@@ -1693,14 +1693,14 @@ 
 	 */
 	function isSiteAdmin($member_info, $site_srl = null)
 	{
-		if(!$member_info->member_srl) return false;
-		if($member_info->is_admin == 'Y') return true;
+		if (!$member_info->member_srl) return false;
+		if ($member_info->is_admin == 'Y') return true;
 
 		$args = new stdClass();
-		if(!isset($site_srl))
+		if (!isset($site_srl))
 		{
 			$site_module_info = Context::get('site_module_info');
-			if(!$site_module_info) return;
+			if (!$site_module_info) return;
 			$args->site_srl = $site_module_info->site_srl;
 		}
 		else
@@ -1710,7 +1710,7 @@ 
 
 		$args->member_srl = $member_info->member_srl;
 		$output = executeQuery('module.isSiteAdmin', $args);
-		if($output->data->member_srl == $args->member_srl) return true;
+		if ($output->data->member_srl == $args->member_srl) return true;
 		return false;
 	}
 
@@ -1733,7 +1733,7 @@ 
 		$obj = new stdClass();
 		$obj->module_srl = $module_srl;
 		$output = executeQueryArray('module.getAdminID', $obj);
-		if(!$output->toBool() || !$output->data) return;
+		if (!$output->toBool() || !$output->data) return;
 
 		return $output->data;
 	}
@@ -1746,20 +1746,20 @@ 
 	{
 		$extra_vars = array();
 		$get_module_srls = array();
-		if(!is_array($list_module_srl)) $list_module_srl = array($list_module_srl);
+		if (!is_array($list_module_srl)) $list_module_srl = array($list_module_srl);
 
 		$vars = false;
 		// cache controll
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
-			foreach($list_module_srl as $module_srl)
+			foreach ($list_module_srl as $module_srl)
 			{
 				$object_key = 'module_extra_vars:'.$module_srl;
 				$cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 				$vars = $oCacheHandler->get($cache_key);
 
-				if($vars)
+				if ($vars)
 				{
 					$extra_vars[$module_srl] = $vars;
 				}
@@ -1774,35 +1774,35 @@ 
 			$get_module_srls = $list_module_srl;
 		}
 
-		if(count($get_module_srls) > 0)
+		if (count($get_module_srls) > 0)
 		{
 			$args = new stdClass();
 			$args->module_srl = implode(',', $get_module_srls);
 			$output = executeQueryArray('module.getModuleExtraVars', $args);
 
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				return;
 			}
 
-			if(!$output->data)
+			if (!$output->data)
 			{
-				foreach($get_module_srls as $module_srl)
+				foreach ($get_module_srls as $module_srl)
 				{
 					$extra_vars[$module_srl] = new stdClass;
 				}
 			}
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
-				if(in_array($val->name, array('mid','module')) || $val->value == 'Array') continue;
+				if (in_array($val->name, array('mid', 'module')) || $val->value == 'Array') continue;
 
-				if(!isset($extra_vars[$val->module_srl]))
+				if (!isset($extra_vars[$val->module_srl]))
 				{
 					$extra_vars[$val->module_srl] = new stdClass();
 				}
 				$extra_vars[$val->module_srl]->{$val->name} = $val->value;
 
-				if($oCacheHandler->isSupport())
+				if ($oCacheHandler->isSupport())
 				{
 					$object_key = 'module_extra_vars:'.$val->module_srl;
 					$cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
@@ -1821,27 +1821,27 @@ 
 	{
 		$skin_vars = false;
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$object_key = 'module_skin_vars:'.$module_srl;
 			$cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 			$skin_vars = $oCacheHandler->get($cache_key);
 		}
 
-		if($skin_vars === false)
+		if ($skin_vars === false)
 		{
 			$args = new stdClass();
 			$args->module_srl = $module_srl;
-			$output = executeQueryArray('module.getModuleSkinVars',$args);
-			if(!$output->toBool()) return;
+			$output = executeQueryArray('module.getModuleSkinVars', $args);
+			if (!$output->toBool()) return;
 
 			$skin_vars = array();
-			foreach($output->data as $vars)
+			foreach ($output->data as $vars)
 			{
 				$skin_vars[$vars->name] = $vars;
 			}
 
-			if($oCacheHandler->isSupport()) $oCacheHandler->put($cache_key, $skin_vars);
+			if ($oCacheHandler->isSupport()) $oCacheHandler->put($cache_key, $skin_vars);
 		}
 
 		return $skin_vars;
@@ -1853,32 +1853,32 @@ 
 	function getModuleDefaultSkin($module_name, $skin_type = 'P', $site_srl = 0, $updateCache = true)
 	{
 		$target = ($skin_type == 'M') ? 'mskin' : 'skin';
-		if(!$site_srl) $site_srl = 0;
+		if (!$site_srl) $site_srl = 0;
 
 		$designInfoFile = sprintf(_XE_PATH_.'files/site_design/design_%s.php', $site_srl);
-		if(is_readable($designInfoFile))
+		if (is_readable($designInfoFile))
 		{
 			include($designInfoFile);
 
 			$skinName = $designInfo->module->{$module_name}->{$target};
 		}
-		if(!$skinName)
+		if (!$skinName)
 		{
 			$dir = ($skin_type == 'M') ? 'm.skins/' : 'skins/';
 			$moduleSkinPath = ModuleHandler::getModulePath($module_name).$dir;
 
-			if(is_dir($moduleSkinPath.'default'))
+			if (is_dir($moduleSkinPath.'default'))
 			{
 				$skinName = 'default';
 			}
-			else if(is_dir($moduleSkinPath.'xe_default'))
+			else if (is_dir($moduleSkinPath.'xe_default'))
 			{
 				$skinName = 'xe_default';
 			}
 			else
 			{
 				$skins = FileHandler::readDir($moduleSkinPath);
-				if(count($skins) > 0)
+				if (count($skins) > 0)
 				{
 					$skinName = $skins[0];
 				}
@@ -1888,9 +1888,9 @@ 
 				}
 			}
 
-			if($updateCache && $skinName)
+			if ($updateCache && $skinName)
 			{
-				if(!isset($designInfo->module->{$module_name})) $designInfo->module->{$module_name} = new stdClass();
+				if (!isset($designInfo->module->{$module_name})) $designInfo->module->{$module_name} = new stdClass();
 				$designInfo->module->{$module_name}->{$target} = $skinName;
 
 				$oAdminController = getAdminController('admin');
@@ -1906,10 +1906,10 @@ 
 	 */
 	function syncSkinInfoToModuleInfo(&$module_info)
 	{
-		if(!$module_info->module_srl) return;
+		if (!$module_info->module_srl) return;
 
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if(Mobile::isFromMobilePhone())
+		if (Mobile::isFromMobilePhone())
 		{
 			$skin_vars = $this->getModuleMobileSkinVars($module_info->module_srl);
 		}
@@ -1918,11 +1918,11 @@ 
 			$skin_vars = $this->getModuleSkinVars($module_info->module_srl);
 		}
 
-		if(!$skin_vars) return;
+		if (!$skin_vars) return;
 
-		foreach($skin_vars as $name => $val)
+		foreach ($skin_vars as $name => $val)
 		{
-			if(isset($module_info->{$name})) continue;
+			if (isset($module_info->{$name})) continue;
 			$module_info->{$name} = $val->value;
 		}
 	}
@@ -1936,27 +1936,27 @@ 
 	{
 		$skin_vars = false;
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$object_key = 'module_mobile_skin_vars:'.$module_srl;
 			$cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 			$skin_vars = $oCacheHandler->get($cache_key);
 		}
 
-		if($skin_vars === false)
+		if ($skin_vars === false)
 		{
 			$args = new stdClass();
 			$args->module_srl = $module_srl;
-			$output = executeQueryArray('module.getModuleMobileSkinVars',$args);
-			if(!$output->toBool() || !$output->data) return;
+			$output = executeQueryArray('module.getModuleMobileSkinVars', $args);
+			if (!$output->toBool() || !$output->data) return;
 
 			$skin_vars = array();
-			foreach($output->data as $vars)
+			foreach ($output->data as $vars)
 			{
 				$skin_vars[$vars->name] = $vars;
 			}
 
-			if($oCacheHandler->isSupport()) $oCacheHandler->put($cache_key, $skin_vars);
+			if ($oCacheHandler->isSupport()) $oCacheHandler->put($cache_key, $skin_vars);
 		}
 
 		return $skin_vars;
@@ -1968,32 +1968,32 @@ 
 	 */
 	function syncMobileSkinInfoToModuleInfo(&$module_info)
 	{
-		if(!$module_info->module_srl) return;
+		if (!$module_info->module_srl) return;
 		$skin_vars = false;
 		// cache controll
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$object_key = 'module_mobile_skin_vars:'.$module_info->module_srl;
 			$cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 			$skin_vars = $oCacheHandler->get($cache_key);
 		}
-		if($skin_vars === false)
+		if ($skin_vars === false)
 		{
 			$args = new stdClass;
 			$args->module_srl = $module_info->module_srl;
-			$output = executeQueryArray('module.getModuleMobileSkinVars',$args);
-			if(!$output->toBool()) return;
+			$output = executeQueryArray('module.getModuleMobileSkinVars', $args);
+			if (!$output->toBool()) return;
 			$skin_vars = $output->data;
 
 			//insert in cache
-			if($oCacheHandler->isSupport()) $oCacheHandler->put($cache_key, $skin_vars);
+			if ($oCacheHandler->isSupport()) $oCacheHandler->put($cache_key, $skin_vars);
 		}
-		if(!$skin_vars) return;
+		if (!$skin_vars) return;
 
-		foreach($output->data as $val)
+		foreach ($output->data as $val)
 		{
-			if(isset($module_info->{$val->name})) continue;
+			if (isset($module_info->{$val->name})) continue;
 			$module_info->{$val->name} = $val->value;
 		}
 	}
@@ -2005,7 +2005,7 @@ 
 	{
 		$grant = new stdClass();
 
-		if(!$xml_info)
+		if (!$xml_info)
 		{
 			$module = $module_info->module;
 			$xml_info = $this->getModuleActionXml($module);
@@ -2013,9 +2013,9 @@ 
 		// Set variables to grant group permission
 		$module_srl = $module_info->module_srl;
 		$grant_info = $xml_info->grant;
-		if($member_info->member_srl)
+		if ($member_info->member_srl)
 		{
-			if(is_array($member_info->group_list)) $group_list = array_keys($member_info->group_list);
+			if (is_array($member_info->group_list)) $group_list = array_keys($member_info->group_list);
 			else $group_list = array();
 		}
 		else
@@ -2023,10 +2023,10 @@ 
 			$group_list = array();
 		}
 		// If module_srl doesn't exist(if unable to set permissions)
-		if(!$module_srl)
+		if (!$module_srl)
 		{
 			$grant->access = true;
-			if($this->isSiteAdmin($member_info, $module_info->site_srl))
+			if ($this->isSiteAdmin($member_info, $module_info->site_srl))
 			{
 				$grant->access = $grant->manager = $grant->is_site_admin = true;
 			}
@@ -2037,23 +2037,23 @@ 
 		{
 			// If module_srl exists
 			// Get a type of granted permission
-			$grant->access = $grant->manager = $grant->is_site_admin = ($member_info->is_admin=='Y'||$this->isSiteAdmin($member_info, $module_info->site_srl))?true:false;
+			$grant->access = $grant->manager = $grant->is_site_admin = ($member_info->is_admin == 'Y' || $this->isSiteAdmin($member_info, $module_info->site_srl)) ?true:false;
 			$grant->is_admin = ($member_info->is_admin == 'Y') ? true : false;
 			// If a just logged-in member is, check if the member is a module administrator
-			if(!$grant->manager && $member_info->member_srl)
+			if (!$grant->manager && $member_info->member_srl)
 			{
 				$args = new stdClass();
 				$args->module_srl = $module_srl;
 				$args->member_srl = $member_info->member_srl;
-				$output = executeQuery('module.getModuleAdmin',$args);
-				if($output->data && $output->data->member_srl == $member_info->member_srl) $grant->manager = true;
+				$output = executeQuery('module.getModuleAdmin', $args);
+				if ($output->data && $output->data->member_srl == $member_info->member_srl) $grant->manager = true;
 			}
 			// If not an administrator, get information from the DB and grant manager privilege.
-			if(!$grant->manager)
+			if (!$grant->manager)
 			{
 				$args = new stdClass();
 				// If planet, get permission settings from the planet home
-				if($module_info->module == 'planet')
+				if ($module_info->module == 'planet')
 				{
 					$output = executeQueryArray('module.getPlanetGrants', $args);
 				}
@@ -2066,42 +2066,42 @@ 
 
 				$grant_exists = $granted = array();
 
-				if($output->data)
+				if ($output->data)
 				{
 					// Arrange names and groups who has privileges
-					foreach($output->data as $val)
+					foreach ($output->data as $val)
 					{
 						$grant_exists[$val->name] = true;
-						if($granted[$val->name]) continue;
+						if ($granted[$val->name]) continue;
 						// Log-in member only
-						if($val->group_srl == -1)
+						if ($val->group_srl == -1)
 						{
 							$granted[$val->name] = true;
-							if($member_info->member_srl) $grant->{$val->name} = true;
+							if ($member_info->member_srl) $grant->{$val->name} = true;
 							// Site-joined member only
 						}
-						elseif($val->group_srl == -2)
+						elseif ($val->group_srl == -2)
 						{
 							$granted[$val->name] = true;
 							// Do not grant any permission for non-logged member
-							if(!$member_info->member_srl) $grant->{$val->name} = false;
+							if (!$member_info->member_srl) $grant->{$val->name} = false;
 							// Log-in member
 							else
 							{
 								$site_module_info = Context::get('site_module_info');
 								// Permission granted if no information of the currently connected site exists
-								if(!$site_module_info->site_srl) $grant->{$val->name} = true;
+								if (!$site_module_info->site_srl) $grant->{$val->name} = true;
 								// Permission is not granted if information of the currently connected site exists
-								elseif(count($group_list)) $grant->{$val->name} = true;
+								elseif (count($group_list)) $grant->{$val->name} = true;
 							}
 							// All of non-logged members
 						}
-						elseif($val->group_srl == -3)
+						elseif ($val->group_srl == -3)
 						{
 							$granted[$val->name] = true;
 							$grant->{$val->name} = ($grant->is_admin || $grant->is_site_admin);
 						}
-						elseif($val->group_srl == 0)
+						elseif ($val->group_srl == 0)
 						{
 							$granted[$val->name] = true;
 							$grant->{$val->name} = true;
@@ -2109,7 +2109,7 @@ 
 						}
 						else
 						{
-							if($group_list && count($group_list) && in_array($val->group_srl, $group_list))
+							if ($group_list && count($group_list) && in_array($val->group_srl, $group_list))
 							{
 								$grant->{$val->name} = true;
 								$granted[$val->name] = true;
@@ -2118,29 +2118,29 @@ 
 					}
 				}
 				// Separate processing for the virtual group access
-				if(!$grant_exists['access']) $grant->access = true;
-				if(count($grant_info))
+				if (!$grant_exists['access']) $grant->access = true;
+				if (count($grant_info))
 				{
-					foreach($grant_info as  $grant_name => $grant_item)
+					foreach ($grant_info as  $grant_name => $grant_item)
 					{
-						if($grant_exists[$grant_name]) continue;
-						switch($grant_item->default)
+						if ($grant_exists[$grant_name]) continue;
+						switch ($grant_item->default)
 						{
 							case 'guest' :
 								$grant->{$grant_name} = true;
 								break;
 							case 'member' :
-								if($member_info->member_srl) $grant->{$grant_name} = true;
+								if ($member_info->member_srl) $grant->{$grant_name} = true;
 								else $grant->{$grant_name} = false;
 								break;
 							case 'site' :
 								$site_module_info = Context::get('site_module_info');
-								if($member_info->member_srl && (($site_module_info->site_srl && count($group_list)) || !$site_module_info->site_srl)) $grant->{$grant_name} = true;
+								if ($member_info->member_srl && (($site_module_info->site_srl && count($group_list)) || !$site_module_info->site_srl)) $grant->{$grant_name} = true;
 								else $grant->{$grant_name} = false;
 								break;
 							case 'manager' :
 							case 'root' :
-								if($member_info->is_admin == 'Y') $grant->{$grant_name} = true;
+								if ($member_info->is_admin == 'Y') $grant->{$grant_name} = true;
 								else $grant->{$grant_name} = false;
 								break;
 						}
@@ -2148,12 +2148,12 @@ 
 				}
 			}
 			// Set true to grant all privileges if an administrator is
-			if($grant->manager)
+			if ($grant->manager)
 			{
 				$grant->access = true;
-				if(count($grant_info))
+				if (count($grant_info))
 				{
-					foreach($grant_info as $key => $val)
+					foreach ($grant_info as $key => $val)
 					{
 						$grant->{$key} = true;
 					}
@@ -2185,27 +2185,27 @@ 
 
 	function unserializeAttributes($module_filebox_list)
 	{
-		if(is_array($module_filebox_list->data))
+		if (is_array($module_filebox_list->data))
 		{
-			foreach($module_filebox_list->data as &$item)
+			foreach ($module_filebox_list->data as &$item)
 			{
-				if(empty($item->comment))
+				if (empty($item->comment))
 				{
 					continue;
 				}
 
 				$attributes = explode(';', $item->comment);
-				foreach($attributes as $attribute)
+				foreach ($attributes as $attribute)
 				{
 					$values = explode(':', $attribute);
-					if((count($values) % 2) ==1)
+					if ((count($values) % 2) == 1)
 					{
-						for($i=2;$i<count($values);$i++)
+						for ($i = 2; $i < count($values); $i++)
 						{
-							$values[1].=":".$values[$i];
+							$values[1] .= ":".$values[$i];
 						}
 					}
-					$atts[$values[0]]=$values[1];
+					$atts[$values[0]] = $values[1];
 				}
 				$item->attributes = $atts;
 				unset($atts);
@@ -2217,16 +2217,16 @@ 
 	function getFileBoxListHtml()
 	{
 		$logged_info = Context::get('logged_info');
-		if($logged_info->is_admin !='Y' && !$logged_info->is_site_admin) return new BaseObject(-1, 'msg_not_permitted');
+		if ($logged_info->is_admin != 'Y' && !$logged_info->is_site_admin) return new BaseObject(-1, 'msg_not_permitted');
 		$link = parse_url($_SERVER["HTTP_REFERER"]);
-		$link_params = explode('&',$link['query']);
+		$link_params = explode('&', $link['query']);
 		foreach ($link_params as $param)
 		{
-			$param = explode("=",$param);
-			if($param[0] == 'selected_widget') $selected_widget = $param[1];
+			$param = explode("=", $param);
+			if ($param[0] == 'selected_widget') $selected_widget = $param[1];
 		}
 		$oWidgetModel = getModel('widget');
-		if($selected_widget) $widget_info = $oWidgetModel->getWidgetInfo($selected_widget);
+		if ($selected_widget) $widget_info = $oWidgetModel->getWidgetInfo($selected_widget);
 		Context::set('allow_multiple', $widget_info->extra_var->images->allow_multiple);
 
 		$oModuleModel = getModel('module');
@@ -2242,24 +2242,24 @@ 
 		$security->encodeHTML('filebox_list..comment', 'filebox_list..attributes.');
 
 		$oTemplate = &TemplateHandler::getInstance();
-		$html = $oTemplate->compile(_XE_PATH_ . 'modules/module/tpl/', 'filebox_list_html');
+		$html = $oTemplate->compile(_XE_PATH_.'modules/module/tpl/', 'filebox_list_html');
 
 		$this->add('html', $html);
 	}
 
 	function getModuleFileBoxPath($module_filebox_srl)
 	{
-		return sprintf("./files/attach/filebox/%s",getNumberingPath($module_filebox_srl,3));
+		return sprintf("./files/attach/filebox/%s", getNumberingPath($module_filebox_srl, 3));
 	}
 
 	/**
 	 * @brief Return ruleset cache file path
 	 * @param module, act
 	 */
-	function getValidatorFilePath($module, $ruleset, $mid=null)
+	function getValidatorFilePath($module, $ruleset, $mid = null)
 	{
 		// load dynamic ruleset xml file
-		if(strpos($ruleset, '@') !== false)
+		if (strpos($ruleset, '@') !== false)
 		{
 			$rulsetFile = str_replace('@', '', $ruleset);
 			$xml_file = sprintf('./files/ruleset/%s.xml', $rulsetFile);
@@ -2269,20 +2269,20 @@ 
 		{
 			$rulsetFile = str_replace('#', '', $ruleset).'.'.$mid;
 			$xml_file = sprintf('./files/ruleset/%s.xml', $rulsetFile);
-			if(is_readable($xml_file))
+			if (is_readable($xml_file))
 				return FileHandler::getRealPath($xml_file);
-			else{
+			else {
 				$ruleset = str_replace('#', '', $ruleset);
 			}
 
 		}
 		// Get a path of the requested module. Return if not exists.
 		$class_path = ModuleHandler::getModulePath($module);
-		if(!$class_path) return;
+		if (!$class_path) return;
 
 		// Check if module.xml exists in the path. Return if not exist
 		$xml_file = sprintf("%sruleset/%s.xml", $class_path, $ruleset);
-		if(!file_exists($xml_file)) return;
+		if (!file_exists($xml_file)) return;
 
 		return $xml_file;
 	}
@@ -2294,7 +2294,7 @@ 
 		$requestVars = Context::getRequestVars();
 
 		$args = new stdClass;
-		$args->site_srl = (int)$requestVars->site_srl;
+		$args->site_srl = (int) $requestVars->site_srl;
 		$args->page = 1; // /< Page
 		$args->list_count = 100; // /< the number of posts to display on a single page
 		$args->page_count = 5; // /< the number of pages that appear in the page navigation
@@ -2306,9 +2306,9 @@ 
 
 		$list = array();
 
-		if($output->toBool())
+		if ($output->toBool())
 		{
-			foreach((array)$output->data as $code_info)
+			foreach ((array) $output->data as $code_info)
 			{
 				unset($codeInfo);
 				$codeInfo = array('name'=>'$user_lang->'.$code_info->name, 'value'=>$code_info->value);

Braces +467 added lines, -208 removed lines

@@ -19,26 +19,34 @@ 
 	 */
 	function isIDExists($id, $site_srl = 0)
 	{
-		if(!preg_match('/^[a-z]{1}([a-z0-9_]+)$/i',$id)) return true;
+		if(!preg_match('/^[a-z]{1}([a-z0-9_]+)$/i',$id)) {
+			return true;
+		}
 		// directory and rss/atom/api reserved checking, etc.
 		$dirs = FileHandler::readDir(_XE_PATH_);
 		$dirs[] = 'rss';
 		$dirs[] = 'atom';
 		$dirs[] = 'api';
-		if(in_array($id, $dirs)) return true;
+		if(in_array($id, $dirs)) {
+			return true;
+		}
 		// mid test
 		$args = new stdClass();
 		$args->mid = $id;
 		$args->site_srl = $site_srl;
 		$output = executeQuery('module.isExistsModuleName', $args);
-		if($output->data->count) return true;
+		if($output->data->count) {
+			return true;
+		}
 		// vid test (check mid != vid if site_srl=0, which means it is not a virtual site)
 		if(!$site_srl)
 		{
 			$site_args = new stdClass();
 			$site_args->domain = $id;
 			$output = executeQuery('module.isExistsSiteDomain', $site_args);
-			if($output->data->count) return true;
+			if($output->data->count) {
+				return true;
+			}
 		}
 
 		return false;
@@ -82,10 +90,14 @@ 
 	function getDefaultMid()
 	{
 		$default_url = Context::getDefaultUrl();
-		if($default_url && substr_compare($default_url, '/', -1) === 0) $default_url = substr($default_url, 0, -1);
+		if($default_url && substr_compare($default_url, '/', -1) === 0) {
+			$default_url = substr($default_url, 0, -1);
+		}
 
 		$request_url = Context::getRequestUri();
-		if($request_url && substr_compare($request_url, '/', -1) === 0) $request_url = substr($request_url, 0, -1);
+		if($request_url && substr_compare($request_url, '/', -1) === 0) {
+			$request_url = substr($request_url, 0, -1);
+		}
 
 		$default_url_parse = parse_url($default_url);
 		$request_url_parse = parse_url($request_url);
@@ -100,14 +112,18 @@ 
 			$hostname = $request_url_parse['host'];
 			$path = $request_url_parse['path'];
 			$port = $request_url_parse['port'];
-			if(strlen($path) >= 1 && substr_compare($path, '/', -1) === 0) $path = substr($path, 0, -1);
+			if(strlen($path) >= 1 && substr_compare($path, '/', -1) === 0) {
+				$path = substr($path, 0, -1);
+			}
 
 			$domain = sprintf('%s%s%s', $hostname, $port && ($port != 80 )? ':'.$port  : '', $path);
 		}
 
 		if($domain === '')
 		{
-			if(!$vid) $vid = $mid;
+			if(!$vid) {
+				$vid = $mid;
+			}
 			if($vid)
 			{
 				$domain = $vid;
@@ -133,13 +149,17 @@ 
 				$output = executeQuery('module.getSiteInfoByDomain', $args);
 				$site_info = $output->data;
 
-				if($oCacheHandler->isSupport()) $oCacheHandler->put($domain_cache_key, $site_info);
+				if($oCacheHandler->isSupport()) {
+					$oCacheHandler->put($domain_cache_key, $site_info);
+				}
 			}
 
 			if($site_info && $vid)
 			{
 				Context::set('vid', $site_info->domain, true);
-				if(strtolower($mid)==strtolower($site_info->domain)) Context::set('mid', $site_info->mid,true);
+				if(strtolower($mid)==strtolower($site_info->domain)) {
+					Context::set('mid', $site_info->mid,true);
+				}
 			}
 			if(!$site_info || !$site_info->domain) { $domain = ''; unset($site_info); }
 		}
@@ -165,8 +185,12 @@ 
 				{
 					// Create a table if sites table doesn't exist
 					$oDB = &DB::getInstance();
-					if(!$oDB->isTableExists('sites')) $oDB->createTableByXmlFile(_XE_PATH_.'modules/module/schemas/sites.xml');
-					if(!$oDB->isTableExists('sites')) return;
+					if(!$oDB->isTableExists('sites')) {
+						$oDB->createTableByXmlFile(_XE_PATH_.'modules/module/schemas/sites.xml');
+					}
+					if(!$oDB->isTableExists('sites')) {
+						return;
+					}
 
 					// Get mid, language
 					$mid_output = $oDB->executeQuery('module.getDefaultMidInfo', $args);
@@ -184,21 +208,28 @@ 
 					if($output->data && !$output->data->index_module_srl)
 					{
 						$output = executeQuery('module.updateSite', $site_args);
-					}
-					else
+					} else
 					{
 						$output = executeQuery('module.insertSite', $site_args);
-						if(!$output->toBool()) return $output;
+						if(!$output->toBool()) {
+							return $output;
+						}
 					}
 					$output = executeQuery('module.getSiteInfo', $args);
 				}
 				$site_info = $output->data;
-				if($oCacheHandler->isSupport()) $oCacheHandler->put($default_site_cache_key, $site_info);
+				if($oCacheHandler->isSupport()) {
+					$oCacheHandler->put($default_site_cache_key, $site_info);
+				}
 			}
 		}
 
-		if(!$site_info->module_srl) return $site_info;
-		if(is_array($site_info) && $site_info->data[0]) $site_info = $site_info[0];
+		if(!$site_info->module_srl) {
+			return $site_info;
+		}
+		if(is_array($site_info) && $site_info->data[0]) {
+			$site_info = $site_info[0];
+		}
 		return $this->addModuleExtraVars($site_info);
 	}
 
@@ -248,7 +279,9 @@ 
 		}
 
 		$this->applyDefaultSkin($module_info);
-		if(!$module_info->module_srl && $module_info->data[0]) $module_info = $module_info->data[0];
+		if(!$module_info->module_srl && $module_info->data[0]) {
+			$module_info = $module_info->data[0];
+		}
 		return $this->addModuleExtraVars($module_info);
 	}
 
@@ -340,8 +373,7 @@ 
 				$object_key = 'mid_info:' . $output->data->module_srl;
 				$module_info_cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
 				$oCacheHandler->put($module_info_cache_key, $moduleInfo);
-			}
-			else
+			} else
 			{
 				$mid_info->designSettings = $moduleInfo->designSettings;
 				$moduleInfo = $mid_info;
@@ -382,11 +414,15 @@ 
 			$args = new stdClass();
 			$args->module_srl = $module_srl;
 			$output = executeQuery('module.getMidInfo', $args);
-			if(!$output->toBool()) return;
+			if(!$output->toBool()) {
+				return;
+			}
 
 			$mid_info = $output->data;
 			$this->applyDefaultSkin($mid_info);
-			if($oCacheHandler->isSupport()) $oCacheHandler->put($cache_key, $mid_info);
+			if($oCacheHandler->isSupport()) {
+				$oCacheHandler->put($cache_key, $mid_info);
+			}
 		}
 
 		if($mid_info && count($columnList))
@@ -399,8 +435,9 @@ 
 					$module_info->$key = $item;
 				}
 			}
+		} else {
+			$module_info = $mid_info;
 		}
-		else $module_info = $mid_info;
 
 		$oModuleController = getController('module');
 
@@ -450,11 +487,15 @@ 
 	 */
 	function getModulesInfo($module_srls, $columnList = array())
 	{
-		if(is_array($module_srls)) $module_srls = implode(',',$module_srls);
+		if(is_array($module_srls)) {
+			$module_srls = implode(',',$module_srls);
+		}
 		$args = new stdClass();
 		$args->module_srls = $module_srls;
 		$output = executeQueryArray('module.getModulesInfo', $args, $columnList);
-		if(!$output->toBool()) return;
+		if(!$output->toBool()) {
+			return;
+		}
 		return $this->addModuleExtraVars($output->data);
 	}
 
@@ -464,31 +505,44 @@ 
 	function addModuleExtraVars($module_info)
 	{
 		// Process although one or more module informaion is requested
-		if(!is_array($module_info)) $target_module_info = array($module_info);
-		else $target_module_info = $module_info;
+		if(!is_array($module_info)) {
+			$target_module_info = array($module_info);
+		} else {
+			$target_module_info = $module_info;
+		}
 		// Get module_srl
 		$module_srls = array();
 		foreach($target_module_info as $key => $val)
 		{
 			$module_srl = $val->module_srl;
-			if(!$module_srl) continue;
+			if(!$module_srl) {
+				continue;
+			}
 			$module_srls[] = $val->module_srl;
 		}
 		// Extract extra information of the module and skin
 		$extra_vars = $this->getModuleExtraVars($module_srls);
-		if(!count($module_srls) || !count($extra_vars)) return $module_info;
+		if(!count($module_srls) || !count($extra_vars)) {
+			return $module_info;
+		}
 
 		foreach($target_module_info as $key => $val)
 		{
-			if(!$extra_vars[$val->module_srl] || !count($extra_vars[$val->module_srl])) continue;
+			if(!$extra_vars[$val->module_srl] || !count($extra_vars[$val->module_srl])) {
+				continue;
+			}
 			foreach($extra_vars[$val->module_srl] as $k => $v)
 			{
-				if($target_module_info[$key]->{$k}) continue;
+				if($target_module_info[$key]->{$k}) {
+					continue;
+				}
 				$target_module_info[$key]->{$k} = $v;
 			}
 		}
 
-		if(is_array($module_info)) return $target_module_info;
+		if(is_array($module_info)) {
+			return $target_module_info;
+		}
 		return $target_module_info[0];
 	}
 
@@ -517,7 +571,9 @@ 
 			}
 
 			$output = executeQuery('module.getMidList', $args, $columnList);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 			$list = $output->data;
 
 			if($oCacheHandler->isSupport() && count($args) === 1 && isset($args->site_srl))
@@ -525,9 +581,13 @@ 
 				$oCacheHandler->put($cache_key, $list);
 			}
 		}
-		if(!$list) return;
+		if(!$list) {
+			return;
+		}
 
-		if(!is_array($list)) $list = array($list);
+		if(!is_array($list)) {
+			$list = array($list);
+		}
 
 		foreach($list as $val)
 		{
@@ -543,10 +603,14 @@ 
 	function getModuleSrlList($args = null, $columnList = array())
 	{
 		$output = executeQueryArray('module.getMidList', $args, $columnList);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$list = $output->data;
-		if(!$list) return;
+		if(!$list) {
+			return;
+		}
 
 		return $list;
 	}
@@ -556,20 +620,32 @@ 
 	 */
 	function getModuleSrlByMid($mid)
 	{
-		if($mid && !is_array($mid)) $mid = explode(',',$mid);
-		if(is_array($mid)) $mid = "'".implode("','",$mid)."'";
+		if($mid && !is_array($mid)) {
+			$mid = explode(',',$mid);
+		}
+		if(is_array($mid)) {
+			$mid = "'".implode("','",$mid)."'";
+		}
 
 		$site_module_info = Context::get('site_module_info');
 
 		$args = new stdClass;
 		$args->mid = $mid;
-		if($site_module_info) $args->site_srl = $site_module_info->site_srl;
+		if($site_module_info) {
+			$args->site_srl = $site_module_info->site_srl;
+		}
 		$output = executeQuery('module.getModuleSrlByMid', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$list = $output->data;
-		if(!$list) return;
-		if(!is_array($list)) $list = array($list);
+		if(!$list) {
+			return;
+		}
+		if(!is_array($list)) {
+			$list = array($list);
+		}
 
 		foreach($list as $key => $val)
 		{
@@ -598,8 +674,12 @@ 
 		{
 			$args = new stdClass();
 			$output = executeQueryArray('module.getActionForward',$args);
-			if(!$output->toBool()) return new stdClass;
-			if(!$output->data) $output->data = array();
+			if(!$output->toBool()) {
+				return new stdClass;
+			}
+			if(!$output->data) {
+				$output->data = array();
+			}
 
 			$action_forward = array();
 			foreach($output->data as $item)
@@ -616,8 +696,7 @@ 
 		if($action_forward[$act])
 		{
 			return $action_forward[$act];
-		}
-		else
+		} else
 		{
 			return new stdClass();
 		}
@@ -725,8 +804,7 @@ 
 			if(file_exists($cache_file))
 			{
 				$GLOBALS['__MODULE_EXTEND__'] = include($cache_file);
-			}
-			else
+			} else
 			{
 				$GLOBALS['__MODULE_EXTEND__'] = array();
 			}
@@ -742,16 +820,22 @@ 
 	{
 		// Get a path of the requested module. Return if not exists.
 		$module_path = ModuleHandler::getModulePath($module);
-		if(!$module_path) return;
+		if(!$module_path) {
+			return;
+		}
 		// Read the xml file for module skin information
 		$xml_file = sprintf("%s/conf/info.xml", $module_path);
-		if(!file_exists($xml_file)) return;
+		if(!file_exists($xml_file)) {
+			return;
+		}
 
 		$oXmlParser = new XmlParser();
 		$tmp_xml_obj = $oXmlParser->loadXmlFile($xml_file);
 		$xml_obj = $tmp_xml_obj->module;
 
-		if(!$xml_obj) return;
+		if(!$xml_obj) {
+			return;
+		}
 
 		// Module Information
 		$module_info = new stdClass();
@@ -763,14 +847,19 @@ 
 			$module_info->version = $xml_obj->version->body;
 			$module_info->homepage = $xml_obj->link->body;
 			$module_info->category = $xml_obj->category->body;
-			if(!$module_info->category) $module_info->category = 'service';
+			if(!$module_info->category) {
+				$module_info->category = 'service';
+			}
 			sscanf($xml_obj->date->body, '%d-%d-%d', $date_obj->y, $date_obj->m, $date_obj->d);
 			$module_info->date = sprintf('%04d%02d%02d', $date_obj->y, $date_obj->m, $date_obj->d);
 			$module_info->license = $xml_obj->license->body;
 			$module_info->license_link = $xml_obj->license->attrs->link;
 
-			if(!is_array($xml_obj->author)) $author_list[] = $xml_obj->author;
-			else $author_list = $xml_obj->author;
+			if(!is_array($xml_obj->author)) {
+				$author_list[] = $xml_obj->author;
+			} else {
+				$author_list = $xml_obj->author;
+			}
 
 			foreach($author_list as $author)
 			{
@@ -780,15 +869,16 @@ 
 				$author_obj->homepage = $author->attrs->link;
 				$module_info->author[] = $author_obj;
 			}
-		}
-		else
+		} else
 		{
 			// module format 0.1
 			$module_info->title = $xml_obj->title->body;
 			$module_info->description = $xml_obj->author->description->body;
 			$module_info->version = $xml_obj->attrs->version;
 			$module_info->category = $xml_obj->attrs->category;
-			if(!$module_info->category) $module_info->category = 'service';
+			if(!$module_info->category) {
+				$module_info->category = 'service';
+			}
 			sscanf($xml_obj->author->attrs->date, '%d. %d. %d', $date_obj->y, $date_obj->m, $date_obj->d);
 			$module_info->date = sprintf('%04d%02d%02d', $date_obj->y, $date_obj->m, $date_obj->d);
 			$author_obj = new stdClass();
@@ -817,11 +907,15 @@ 
 	{
 		// Get a path of the requested module. Return if not exists.
 		$class_path = ModuleHandler::getModulePath($module);
-		if(!$class_path) return;
+		if(!$class_path) {
+			return;
+		}
 
 		// Check if module.xml exists in the path. Return if not exist
 		$xml_file = sprintf("%sconf/module.xml", $class_path);
-		if(!file_exists($xml_file)) return;
+		if(!file_exists($xml_file)) {
+			return;
+		}
 
 		// Check if cached file exists
 		$cache_file = sprintf(_XE_PATH_ . "files/cache/module_info/%s.%s.%s.php", $module, Context::getLangType(), __XE_VERSION__);
@@ -840,7 +934,10 @@ 
 
 			$xml_obj = XmlParser::loadXmlFile($xml_file); // /< Read xml file and convert it to xml object
 
-			if(!count($xml_obj->module)) return; // /< Error occurs if module tag doesn't included in the xml
+			if(!count($xml_obj->module)) {
+				return;
+			}
+			// /< Error occurs if module tag doesn't included in the xml
 
 			$grants = $xml_obj->module->grants->grant; // /< Permission information
 			$permissions = $xml_obj->module->permissions->permission; // /<  Acting permission
@@ -852,8 +949,11 @@ 
 			// Arrange permission information
 			if($grants)
 			{
-				if(is_array($grants)) $grant_list = $grants;
-				else $grant_list[] = $grants;
+				if(is_array($grants)) {
+					$grant_list = $grants;
+				} else {
+					$grant_list[] = $grants;
+				}
 
 				$info->grant = new stdClass();
 				$buff[] = '$info->grant = new stdClass;';
@@ -875,8 +975,11 @@ 
 			// Permissions to grant
 			if($permissions)
 			{
-				if(is_array($permissions)) $permission_list = $permissions;
-				else $permission_list[] = $permissions;
+				if(is_array($permissions)) {
+					$permission_list = $permissions;
+				} else {
+					$permission_list[] = $permissions;
+				}
 
 				$buff[] = '$info->permission = new stdClass;';
 
@@ -894,8 +997,11 @@ 
 			// for admin menus
 			if($menus)
 			{
-				if(is_array($menus)) $menu_list = $menus;
-				else $menu_list[] = $menus;
+				if(is_array($menus)) {
+					$menu_list = $menus;
+				} else {
+					$menu_list[] = $menus;
+				}
 
 				$buff[] = '$info->menu = new stdClass;';
 				$info->menu = new stdClass();
@@ -919,8 +1025,11 @@ 
 			// actions
 			if($actions)
 			{
-				if(is_array($actions)) $action_list = $actions;
-				else $action_list[] = $actions;
+				if(is_array($actions)) {
+					$action_list = $actions;
+				} else {
+					$action_list[] = $actions;
+				}
 
 				$buff[] = '$info->action = new stdClass;';
 				$info->action = new stdClass();
@@ -1012,7 +1121,9 @@ 
 			return $info;
 		}
 
-		if(file_exists($cache_file)) return include($cache_file);
+		if(file_exists($cache_file)) {
+			return include($cache_file);
+		}
 	}
 
 	/**
@@ -1050,7 +1161,9 @@ 
 
 		$skin_path = sprintf("%s/%s/", $path, $dir);
 		$list = FileHandler::readDir($skin_path);
-		if(!count($list)) return;
+		if(!count($list)) {
+			return;
+		}
 
 		natcasesort($list);
 
@@ -1104,8 +1217,7 @@ 
 			if($dir == 'skins')
 			{
 				$type = 'P';
-			}
-			else
+			} else
 			{
 				$type = 'M';
 			}
@@ -1133,14 +1245,20 @@ 
 	function loadSkinInfo($path, $skin, $dir = 'skins')
 	{
 		// Read xml file having skin information
-		if(substr($path,-1)!='/') $path .= '/';
+		if(substr($path,-1)!='/') {
+			$path .= '/';
+		}
 		$skin_xml_file = sprintf("%s%s/%s/skin.xml", $path, $dir, $skin);
-		if(!file_exists($skin_xml_file)) return;
+		if(!file_exists($skin_xml_file)) {
+			return;
+		}
 		// Create XmlParser object
 		$oXmlParser = new XmlParser();
 		$_xml_obj = $oXmlParser->loadXmlFile($skin_xml_file);
 		// Return if no skin information is
-		if(!$_xml_obj->skin) return;
+		if(!$_xml_obj->skin) {
+			return;
+		}
 		$xml_obj = $_xml_obj->skin;
 		// Skin Name
 		$skin_info = new stdClass();
@@ -1157,8 +1275,11 @@ 
 			$skin_info->license_link = $xml_obj->license->attrs->link;
 			$skin_info->description = $xml_obj->description->body;
 
-			if(!is_array($xml_obj->author)) $author_list[] = $xml_obj->author;
-			else $author_list = $xml_obj->author;
+			if(!is_array($xml_obj->author)) {
+				$author_list[] = $xml_obj->author;
+			} else {
+				$author_list = $xml_obj->author;
+			}
 
 			foreach($author_list as $author)
 			{
@@ -1172,8 +1293,12 @@ 
 			if($xml_obj->extra_vars)
 			{
 				$extra_var_groups = $xml_obj->extra_vars->group;
-				if(!$extra_var_groups) $extra_var_groups = $xml_obj->extra_vars;
-				if(!is_array($extra_var_groups)) $extra_var_groups = array($extra_var_groups);
+				if(!$extra_var_groups) {
+					$extra_var_groups = $xml_obj->extra_vars;
+				}
+				if(!is_array($extra_var_groups)) {
+					$extra_var_groups = array($extra_var_groups);
+				}
 
 				foreach($extra_var_groups as $group)
 				{
@@ -1182,7 +1307,9 @@ 
 					{
 						continue;
 					}
-					if(!is_array($group->var)) $extra_vars = array($group->var);
+					if(!is_array($group->var)) {
+						$extra_vars = array($group->var);
+					}
 
 					foreach($extra_vars as $key => $val)
 					{
@@ -1209,8 +1336,7 @@ 
 								$obj->options[$i]->title = $val->options[$i]->title->body;
 								$obj->options[$i]->value = $val->options[$i]->attrs->value;
 							}
-						}
-						else
+						} else
 						{
 							$obj->options[0] = new stdClass();
 							$obj->options[0]->title = $val->options->title->body;
@@ -1221,8 +1347,7 @@ 
 					}
 				}
 			}
-		}
-		else
+		} else
 		{
 			// skin format v0.1
 			sscanf($xml_obj->maker->attrs->date, '%d-%d-%d', $date_obj->y, $date_obj->m, $date_obj->d);
@@ -1240,8 +1365,12 @@ 
 			$skin_info->author[0]->homepage = $xml_obj->maker->attrs->link;
 			// Variables used in the skin
 			$extra_var_groups = $xml_obj->extra_vars->group;
-			if(!$extra_var_groups) $extra_var_groups = $xml_obj->extra_vars;
-			if(!is_array($extra_var_groups)) $extra_var_groups = array($extra_var_groups);
+			if(!$extra_var_groups) {
+				$extra_var_groups = $xml_obj->extra_vars;
+			}
+			if(!is_array($extra_var_groups)) {
+				$extra_var_groups = array($extra_var_groups);
+			}
 
 			foreach($extra_var_groups as $group)
 			{
@@ -1249,7 +1378,9 @@ 
 
 				if($extra_vars)
 				{
-					if(!is_array($extra_vars)) $extra_vars = array($extra_vars);
+					if(!is_array($extra_vars)) {
+						$extra_vars = array($extra_vars);
+					}
 
 					foreach($extra_vars as $var)
 					{
@@ -1271,8 +1402,7 @@ 
 								$options[$i]->title = $var->default[$i]->body;
 								$options[$i]->value = $var->default[$i]->body;
 							}
-						}
-						else
+						} else
 						{
 							$options[0]->title = $var->default->body;
 							$options[0]->value = $var->default->body;
@@ -1302,7 +1432,9 @@ 
 		$colorset = $xml_obj->colorset->color;
 		if($colorset)
 		{
-			if(!is_array($colorset)) $colorset = array($colorset);
+			if(!is_array($colorset)) {
+				$colorset = array($colorset);
+			}
 
 			foreach($colorset as $color)
 			{
@@ -1312,9 +1444,12 @@ 
 				if($screenshot)
 				{
 					$screenshot = sprintf("%s%s/%s/%s", $path, $dir, $skin, $screenshot);
-					if(!file_exists($screenshot)) $screenshot = "";
+					if(!file_exists($screenshot)) {
+						$screenshot = "";
+					}
+				} else {
+					$screenshot = "";
 				}
-				else $screenshot = "";
 
 				$obj = new stdClass();
 				$obj->name = $name;
@@ -1327,7 +1462,9 @@ 
 		if($xml_obj->menus->menu)
 		{
 			$menus = $xml_obj->menus->menu;
-			if(!is_array($menus)) $menus = array($menus);
+			if(!is_array($menus)) {
+				$menus = array($menus);
+			}
 
 			$menu_count = count($menus);
 			$skin_info->menu_count = $menu_count;
@@ -1336,7 +1473,9 @@ 
 				unset($obj);
 
 				$obj->name = $menus[$i]->attrs->name;
-				if($menus[$i]->attrs->default == "true") $obj->default = true;
+				if($menus[$i]->attrs->default == "true") {
+					$obj->default = true;
+				}
 				$obj->title = $menus[$i]->title->body;
 				$obj->maxdepth = $menus[$i]->maxdepth->body;
 
@@ -1356,7 +1495,9 @@ 
 	{
 		$args = new stdClass;
 		$args->site_srl = $site_srl;
-		if(!is_null($module)) $args->module = $module;
+		if(!is_null($module)) {
+			$args->module = $module;
+		}
 		$output = executeQuery('module.getModuleCount', $args);
 		return $output->data->count;
 	}
@@ -1385,8 +1526,11 @@ 
 				$args->module = $module;
 				$args->site_srl = $site_srl;
 				$output = executeQuery('module.getModuleConfig', $args);
-				if($output->data->config) $config = unserialize($output->data->config);
-				else $config = null;
+				if($output->data->config) {
+					$config = unserialize($output->data->config);
+				} else {
+					$config = null;
+				}
 
 				//insert in cache
 				if($oCacheHandler->isSupport())
@@ -1425,8 +1569,11 @@ 
 				$args->module = $module;
 				$args->module_srl = $module_srl;
 				$output = executeQuery('module.getModulePartConfig', $args);
-				if($output->data->config) $config = unserialize($output->data->config);
-				else $config = null;
+				if($output->data->config) {
+					$config = unserialize($output->data->config);
+				} else {
+					$config = null;
+				}
 
 				//insert in cache
 				if($oCacheHandler->isSupport())
@@ -1448,9 +1595,13 @@ 
 	{
 		$args = new stdClass();
 		$args->module = $module;
-		if($site_srl) $args->site_srl = $site_srl;
+		if($site_srl) {
+			$args->site_srl = $site_srl;
+		}
 		$output = executeQueryArray('module.getModulePartConfigs', $args);
-		if(!$output->toBool() || !$output->data) return array();
+		if(!$output->toBool() || !$output->data) {
+			return array();
+		}
 
 		foreach($output->data as $key => $val)
 		{
@@ -1468,10 +1619,16 @@ 
 		$args->moduleCategorySrl = $moduleCategorySrl;
 		// Get data from the DB
 		$output = executeQuery('module.getModuleCategories', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 		$list = $output->data;
-		if(!$list) return;
-		if(!is_array($list)) $list = array($list);
+		if(!$list) {
+			return;
+		}
+		if(!is_array($list)) {
+			$list = array($list);
+		}
 
 		foreach($list as $val)
 		{
@@ -1489,7 +1646,9 @@ 
 		$args = new stdClass;
 		$args->module_category_srl = $module_category_srl;
 		$output = executeQuery('module.getModuleCategory', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 		return $output->data;
 	}
 
@@ -1501,7 +1660,9 @@ 
 		// Get a list of downloaded and installed modules
 		$searched_list = FileHandler::readDir('./modules');
 		$searched_count = count($searched_list);
-		if(!$searched_count) return;
+		if(!$searched_count) {
+			return;
+		}
 		sort($searched_list);
 
 		for($i=0;$i<$searched_count;$i++)
@@ -1514,7 +1675,9 @@ 
 			$info = $this->getModuleInfoXml($module_name);
 			unset($obj);
 
-			if(!isset($info)) continue;
+			if(!isset($info)) {
+				continue;
+			}
 			$info->module = $module_name;
 			$info->created_table_count = $created_table_count;
 			$info->table_count = $table_count;
@@ -1540,11 +1703,16 @@ 
 			for($j=0;$j<count($tmp_files);$j++)
 			{
 				list($table_name) = explode(".",$tmp_files[$j]);
-				if($oDB->isTableExists($table_name)) $created_table_count ++;
+				if($oDB->isTableExists($table_name)) {
+					$created_table_count ++;
+				}
 			}
 			// Check if DB is installed
-			if($table_count > $created_table_count) return true;
-			else return false;
+			if($table_count > $created_table_count) {
+				return true;
+			} else {
+				return false;
+			}
 		}
 		return false;
 	}
@@ -1567,15 +1735,23 @@ 
 	 */
 	public function needUpdate($update_id)
 	{
-		if(!is_array($update_id)) $update_id = array($update_id);
+		if(!is_array($update_id)) {
+			$update_id = array($update_id);
+		}
 
 		$args = new stdClass();
 		$args->update_id = implode(',', $update_id);
 		$output = executeQueryArray('module.getModuleUpdateLog', $args);
 
-		if(!!$output->error) return false;
-		if(!$output->data) $output->data = array();
-		if(count($update_id) === count($output->data)) return false;
+		if(!!$output->error) {
+			return false;
+		}
+		if(!$output->data) {
+			$output->data = array();
+		}
+		if(count($update_id) === count($output->data)) {
+			return false;
+		}
 
 		return true;
 	}
@@ -1592,7 +1768,9 @@ 
 		sort($searched_list);
 
 		$searched_count = count($searched_list);
-		if(!$searched_count) return;
+		if(!$searched_count) {
+			return;
+		}
 
 		for($i=0;$i<$searched_count;$i++)
 		{
@@ -1600,7 +1778,9 @@ 
 			$module_name = $searched_list[$i];
 
 			$path = ModuleHandler::getModulePath($module_name);
-			if(!is_dir(FileHandler::getRealPath($path))) continue;
+			if(!is_dir(FileHandler::getRealPath($path))) {
+				continue;
+			}
 
 			// Get the number of xml files to create a table in schemas
 			$tmp_files = FileHandler::readDir($path.'schemas', '/(\.xml)$/');
@@ -1610,13 +1790,17 @@ 
 			for($j=0;$j<$table_count;$j++)
 			{
 				list($table_name) = explode('.',$tmp_files[$j]);
-				if($oDB->isTableExists($table_name)) $created_table_count ++;
+				if($oDB->isTableExists($table_name)) {
+					$created_table_count ++;
+				}
 			}
 			// Get information of the module
 			$info = NULL;
 			$info = $this->getModuleInfoXml($module_name);
 
-			if(!$info) continue;
+			if(!$info) {
+				continue;
+			}
 
 			$info->module = $module_name;
 			$info->category = $info->category;
@@ -1625,16 +1809,18 @@ 
 			$info->path = $path;
 			$info->admin_index_act = $info->admin_index_act;
 			// Check if DB is installed
-			if($table_count > $created_table_count) $info->need_install = true;
-			else $info->need_install = false;
+			if($table_count > $created_table_count) {
+				$info->need_install = true;
+			} else {
+				$info->need_install = false;
+			}
 			// Check if it is upgraded to module.class.php on each module
 			$oDummy = null;
 			$oDummy = getModule($module_name, 'class');
 			if($oDummy && method_exists($oDummy, "checkUpdate"))
 			{
 				$info->need_update = $oDummy->checkUpdate();
-			}
-			else
+			} else
 			{
 				continue;
 			}
@@ -1651,7 +1837,9 @@ 
 	 */
 	function syncModuleToSite(&$data)
 	{
-		if(!$data) return;
+		if(!$data) {
+			return;
+		}
 
 		if(is_array($data))
 		{
@@ -1659,9 +1847,10 @@ 
 			{
 				$module_srls[] = $val->module_srl;
 			}
-			if(!count($module_srls)) return;
-		}
-		else
+			if(!count($module_srls)) {
+				return;
+			}
+		} else
 		{
 			$module_srls[] = $data->module_srl;
 		}
@@ -1669,7 +1858,9 @@ 
 		$args = new stdClass();
 		$args->module_srls = implode(',',$module_srls);
 		$output = executeQueryArray('module.getModuleSites', $args);
-		if(!$output->data) return array();
+		if(!$output->data) {
+			return array();
+		}
 		foreach($output->data as $key => $val)
 		{
 			$modules[$val->module_srl] = $val;
@@ -1681,8 +1872,7 @@ 
 			{
 				$data[$key]->domain = $modules[$val->module_srl]->domain;
 			}
-		}
-		else
+		} else
 		{
 			$data->domain = $modules[$data->module_srl]->domain;
 		}
@@ -1693,24 +1883,31 @@ 
 	 */
 	function isSiteAdmin($member_info, $site_srl = null)
 	{
-		if(!$member_info->member_srl) return false;
-		if($member_info->is_admin == 'Y') return true;
+		if(!$member_info->member_srl) {
+			return false;
+		}
+		if($member_info->is_admin == 'Y') {
+			return true;
+		}
 
 		$args = new stdClass();
 		if(!isset($site_srl))
 		{
 			$site_module_info = Context::get('site_module_info');
-			if(!$site_module_info) return;
+			if(!$site_module_info) {
+				return;
+			}
 			$args->site_srl = $site_module_info->site_srl;
-		}
-		else
+		} else
 		{
 			$args->site_srl = $site_srl;
 		}
 
 		$args->member_srl = $member_info->member_srl;
 		$output = executeQuery('module.isSiteAdmin', $args);
-		if($output->data->member_srl == $args->member_srl) return true;
+		if($output->data->member_srl == $args->member_srl) {
+			return true;
+		}
 		return false;
 	}
 
@@ -1733,7 +1930,9 @@ 
 		$obj = new stdClass();
 		$obj->module_srl = $module_srl;
 		$output = executeQueryArray('module.getAdminID', $obj);
-		if(!$output->toBool() || !$output->data) return;
+		if(!$output->toBool() || !$output->data) {
+			return;
+		}
 
 		return $output->data;
 	}
@@ -1746,7 +1945,9 @@ 
 	{
 		$extra_vars = array();
 		$get_module_srls = array();
-		if(!is_array($list_module_srl)) $list_module_srl = array($list_module_srl);
+		if(!is_array($list_module_srl)) {
+			$list_module_srl = array($list_module_srl);
+		}
 
 		$vars = false;
 		// cache controll
@@ -1762,14 +1963,12 @@ 
 				if($vars)
 				{
 					$extra_vars[$module_srl] = $vars;
-				}
-				else
+				} else
 				{
 					$get_module_srls[] = $module_srl;
 				}
 			}
-		}
-		else
+		} else
 		{
 			$get_module_srls = $list_module_srl;
 		}
@@ -1794,7 +1993,9 @@ 
 			}
 			foreach($output->data as $key => $val)
 			{
-				if(in_array($val->name, array('mid','module')) || $val->value == 'Array') continue;
+				if(in_array($val->name, array('mid','module')) || $val->value == 'Array') {
+					continue;
+				}
 
 				if(!isset($extra_vars[$val->module_srl]))
 				{
@@ -1833,7 +2034,9 @@ 
 			$args = new stdClass();
 			$args->module_srl = $module_srl;
 			$output = executeQueryArray('module.getModuleSkinVars',$args);
-			if(!$output->toBool()) return;
+			if(!$output->toBool()) {
+				return;
+			}
 
 			$skin_vars = array();
 			foreach($output->data as $vars)
@@ -1841,7 +2044,9 @@ 
 				$skin_vars[$vars->name] = $vars;
 			}
 
-			if($oCacheHandler->isSupport()) $oCacheHandler->put($cache_key, $skin_vars);
+			if($oCacheHandler->isSupport()) {
+				$oCacheHandler->put($cache_key, $skin_vars);
+			}
 		}
 
 		return $skin_vars;
@@ -1853,7 +2058,9 @@ 
 	function getModuleDefaultSkin($module_name, $skin_type = 'P', $site_srl = 0, $updateCache = true)
 	{
 		$target = ($skin_type == 'M') ? 'mskin' : 'skin';
-		if(!$site_srl) $site_srl = 0;
+		if(!$site_srl) {
+			$site_srl = 0;
+		}
 
 		$designInfoFile = sprintf(_XE_PATH_.'files/site_design/design_%s.php', $site_srl);
 		if(is_readable($designInfoFile))
@@ -1870,19 +2077,16 @@ 
 			if(is_dir($moduleSkinPath.'default'))
 			{
 				$skinName = 'default';
-			}
-			else if(is_dir($moduleSkinPath.'xe_default'))
+			} else if(is_dir($moduleSkinPath.'xe_default'))
 			{
 				$skinName = 'xe_default';
-			}
-			else
+			} else
 			{
 				$skins = FileHandler::readDir($moduleSkinPath);
 				if(count($skins) > 0)
 				{
 					$skinName = $skins[0];
-				}
-				else
+				} else
 				{
 					$skinName = NULL;
 				}
@@ -1890,7 +2094,9 @@ 
 
 			if($updateCache && $skinName)
 			{
-				if(!isset($designInfo->module->{$module_name})) $designInfo->module->{$module_name} = new stdClass();
+				if(!isset($designInfo->module->{$module_name})) {
+					$designInfo->module->{$module_name} = new stdClass();
+				}
 				$designInfo->module->{$module_name}->{$target} = $skinName;
 
 				$oAdminController = getAdminController('admin');
@@ -1906,23 +2112,28 @@ 
 	 */
 	function syncSkinInfoToModuleInfo(&$module_info)
 	{
-		if(!$module_info->module_srl) return;
+		if(!$module_info->module_srl) {
+			return;
+		}
 
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
 		if(Mobile::isFromMobilePhone())
 		{
 			$skin_vars = $this->getModuleMobileSkinVars($module_info->module_srl);
-		}
-		else
+		} else
 		{
 			$skin_vars = $this->getModuleSkinVars($module_info->module_srl);
 		}
 
-		if(!$skin_vars) return;
+		if(!$skin_vars) {
+			return;
+		}
 
 		foreach($skin_vars as $name => $val)
 		{
-			if(isset($module_info->{$name})) continue;
+			if(isset($module_info->{$name})) {
+				continue;
+			}
 			$module_info->{$name} = $val->value;
 		}
 	}
@@ -1948,7 +2159,9 @@ 
 			$args = new stdClass();
 			$args->module_srl = $module_srl;
 			$output = executeQueryArray('module.getModuleMobileSkinVars',$args);
-			if(!$output->toBool() || !$output->data) return;
+			if(!$output->toBool() || !$output->data) {
+				return;
+			}
 
 			$skin_vars = array();
 			foreach($output->data as $vars)
@@ -1956,7 +2169,9 @@ 
 				$skin_vars[$vars->name] = $vars;
 			}
 
-			if($oCacheHandler->isSupport()) $oCacheHandler->put($cache_key, $skin_vars);
+			if($oCacheHandler->isSupport()) {
+				$oCacheHandler->put($cache_key, $skin_vars);
+			}
 		}
 
 		return $skin_vars;
@@ -1968,7 +2183,9 @@ 
 	 */
 	function syncMobileSkinInfoToModuleInfo(&$module_info)
 	{
-		if(!$module_info->module_srl) return;
+		if(!$module_info->module_srl) {
+			return;
+		}
 		$skin_vars = false;
 		// cache controll
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
@@ -1983,17 +2200,25 @@ 
 			$args = new stdClass;
 			$args->module_srl = $module_info->module_srl;
 			$output = executeQueryArray('module.getModuleMobileSkinVars',$args);
-			if(!$output->toBool()) return;
+			if(!$output->toBool()) {
+				return;
+			}
 			$skin_vars = $output->data;
 
 			//insert in cache
-			if($oCacheHandler->isSupport()) $oCacheHandler->put($cache_key, $skin_vars);
+			if($oCacheHandler->isSupport()) {
+				$oCacheHandler->put($cache_key, $skin_vars);
+			}
+		}
+		if(!$skin_vars) {
+			return;
 		}
-		if(!$skin_vars) return;
 
 		foreach($output->data as $val)
 		{
-			if(isset($module_info->{$val->name})) continue;
+			if(isset($module_info->{$val->name})) {
+				continue;
+			}
 			$module_info->{$val->name} = $val->value;
 		}
 	}
@@ -2015,10 +2240,12 @@ 
 		$grant_info = $xml_info->grant;
 		if($member_info->member_srl)
 		{
-			if(is_array($member_info->group_list)) $group_list = array_keys($member_info->group_list);
-			else $group_list = array();
-		}
-		else
+			if(is_array($member_info->group_list)) {
+				$group_list = array_keys($member_info->group_list);
+			} else {
+				$group_list = array();
+			}
+		} else
 		{
 			$group_list = array();
 		}
@@ -2032,8 +2259,7 @@ 
 			}
 
 			$grant->is_admin = $grant->manager = ($member_info->is_admin == 'Y') ? true : false;
-		}
-		else
+		} else
 		{
 			// If module_srl exists
 			// Get a type of granted permission
@@ -2046,7 +2272,9 @@ 
 				$args->module_srl = $module_srl;
 				$args->member_srl = $member_info->member_srl;
 				$output = executeQuery('module.getModuleAdmin',$args);
-				if($output->data && $output->data->member_srl == $member_info->member_srl) $grant->manager = true;
+				if($output->data && $output->data->member_srl == $member_info->member_srl) {
+					$grant->manager = true;
+				}
 			}
 			// If not an administrator, get information from the DB and grant manager privilege.
 			if(!$grant->manager)
@@ -2056,8 +2284,7 @@ 
 				if($module_info->module == 'planet')
 				{
 					$output = executeQueryArray('module.getPlanetGrants', $args);
-				}
-				else
+				} else
 				{
 					$args = new stdClass;
 					$args->module_srl = $module_srl;
@@ -2072,42 +2299,48 @@ 
 					foreach($output->data as $val)
 					{
 						$grant_exists[$val->name] = true;
-						if($granted[$val->name]) continue;
+						if($granted[$val->name]) {
+							continue;
+						}
 						// Log-in member only
 						if($val->group_srl == -1)
 						{
 							$granted[$val->name] = true;
-							if($member_info->member_srl) $grant->{$val->name} = true;
+							if($member_info->member_srl) {
+								$grant->{$val->name} = true;
+							}
 							// Site-joined member only
-						}
-						elseif($val->group_srl == -2)
+						} elseif($val->group_srl == -2)
 						{
 							$granted[$val->name] = true;
 							// Do not grant any permission for non-logged member
-							if(!$member_info->member_srl) $grant->{$val->name} = false;
+							if(!$member_info->member_srl) {
+								$grant->{$val->name} = false;
+							}
 							// Log-in member
 							else
 							{
 								$site_module_info = Context::get('site_module_info');
 								// Permission granted if no information of the currently connected site exists
-								if(!$site_module_info->site_srl) $grant->{$val->name} = true;
+								if(!$site_module_info->site_srl) {
+									$grant->{$val->name} = true;
+								}
 								// Permission is not granted if information of the currently connected site exists
-								elseif(count($group_list)) $grant->{$val->name} = true;
+								elseif(count($group_list)) {
+									$grant->{$val->name} = true;
+								}
 							}
 							// All of non-logged members
-						}
-						elseif($val->group_srl == -3)
+						} elseif($val->group_srl == -3)
 						{
 							$granted[$val->name] = true;
 							$grant->{$val->name} = ($grant->is_admin || $grant->is_site_admin);
-						}
-						elseif($val->group_srl == 0)
+						} elseif($val->group_srl == 0)
 						{
 							$granted[$val->name] = true;
 							$grant->{$val->name} = true;
 							// If a target is a group
-						}
-						else
+						} else
 						{
 							if($group_list && count($group_list) && in_array($val->group_srl, $group_list))
 							{
@@ -2118,30 +2351,43 @@ 
 					}
 				}
 				// Separate processing for the virtual group access
-				if(!$grant_exists['access']) $grant->access = true;
+				if(!$grant_exists['access']) {
+					$grant->access = true;
+				}
 				if(count($grant_info))
 				{
 					foreach($grant_info as  $grant_name => $grant_item)
 					{
-						if($grant_exists[$grant_name]) continue;
+						if($grant_exists[$grant_name]) {
+							continue;
+						}
 						switch($grant_item->default)
 						{
 							case 'guest' :
 								$grant->{$grant_name} = true;
 								break;
 							case 'member' :
-								if($member_info->member_srl) $grant->{$grant_name} = true;
-								else $grant->{$grant_name} = false;
+								if($member_info->member_srl) {
+									$grant->{$grant_name} = true;
+								} else {
+									$grant->{$grant_name} = false;
+								}
 								break;
 							case 'site' :
 								$site_module_info = Context::get('site_module_info');
-								if($member_info->member_srl && (($site_module_info->site_srl && count($group_list)) || !$site_module_info->site_srl)) $grant->{$grant_name} = true;
-								else $grant->{$grant_name} = false;
+								if($member_info->member_srl && (($site_module_info->site_srl && count($group_list)) || !$site_module_info->site_srl)) {
+									$grant->{$grant_name} = true;
+								} else {
+									$grant->{$grant_name} = false;
+								}
 								break;
 							case 'manager' :
 							case 'root' :
-								if($member_info->is_admin == 'Y') $grant->{$grant_name} = true;
-								else $grant->{$grant_name} = false;
+								if($member_info->is_admin == 'Y') {
+									$grant->{$grant_name} = true;
+								} else {
+									$grant->{$grant_name} = false;
+								}
 								break;
 						}
 					}
@@ -2217,16 +2463,22 @@ 
 	function getFileBoxListHtml()
 	{
 		$logged_info = Context::get('logged_info');
-		if($logged_info->is_admin !='Y' && !$logged_info->is_site_admin) return new BaseObject(-1, 'msg_not_permitted');
+		if($logged_info->is_admin !='Y' && !$logged_info->is_site_admin) {
+			return new BaseObject(-1, 'msg_not_permitted');
+		}
 		$link = parse_url($_SERVER["HTTP_REFERER"]);
 		$link_params = explode('&',$link['query']);
 		foreach ($link_params as $param)
 		{
 			$param = explode("=",$param);
-			if($param[0] == 'selected_widget') $selected_widget = $param[1];
+			if($param[0] == 'selected_widget') {
+				$selected_widget = $param[1];
+			}
 		}
 		$oWidgetModel = getModel('widget');
-		if($selected_widget) $widget_info = $oWidgetModel->getWidgetInfo($selected_widget);
+		if($selected_widget) {
+			$widget_info = $oWidgetModel->getWidgetInfo($selected_widget);
+		}
 		Context::set('allow_multiple', $widget_info->extra_var->images->allow_multiple);
 
 		$oModuleModel = getModel('module');
@@ -2234,7 +2486,9 @@ 
 		Context::set('filebox_list', $output->data);
 
 		$page = Context::get('page');
-		if (!$page) $page = 1;
+		if (!$page) {
+			$page = 1;
+		}
 		Context::set('page', $page);
 		Context::set('page_navigation', $output->page_navigation);
 
@@ -2264,25 +2518,28 @@ 
 			$rulsetFile = str_replace('@', '', $ruleset);
 			$xml_file = sprintf('./files/ruleset/%s.xml', $rulsetFile);
 			return FileHandler::getRealPath($xml_file);
-		}
-		else if (strpos($ruleset, '#') !== false)
+		} else if (strpos($ruleset, '#') !== false)
 		{
 			$rulsetFile = str_replace('#', '', $ruleset).'.'.$mid;
 			$xml_file = sprintf('./files/ruleset/%s.xml', $rulsetFile);
-			if(is_readable($xml_file))
-				return FileHandler::getRealPath($xml_file);
-			else{
+			if(is_readable($xml_file)) {
+							return FileHandler::getRealPath($xml_file);
+			} else{
 				$ruleset = str_replace('#', '', $ruleset);
 			}
 
 		}
 		// Get a path of the requested module. Return if not exists.
 		$class_path = ModuleHandler::getModulePath($module);
-		if(!$class_path) return;
+		if(!$class_path) {
+			return;
+		}
 
 		// Check if module.xml exists in the path. Return if not exist
 		$xml_file = sprintf("%sruleset/%s.xml", $class_path, $ruleset);
-		if(!file_exists($xml_file)) return;
+		if(!file_exists($xml_file)) {
+			return;
+		}
 
 		return $xml_file;
 	}
@@ -2332,7 +2589,9 @@ 
 	function getLangByLangcode()
 	{
 		$langCode = Context::get('langCode');
-		if (!$langCode) return;
+		if (!$langCode) {
+			return;
+		}
 
 		$oModuleController = getController('module');
 		$oModuleController->replaceDefinedLangCode($langCode);