xpressengine / xe-core

addons/captcha/captcha.addon.php

Doc Comments +3 added lines

@@ -275,6 +275,9 @@
 			exit();
 		}
 
+		/**
+		 * @param string $string
+		 */
 		function createCaptchaAudio($string)
 		{
 			$data = '';

Braces +8 added lines, -6 removed lines

@@ -1,7 +1,9 @@ 
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 
-if(!defined("__XE__")) exit();
+if(!defined("__XE__")) {
+	exit();
+}
 
 /**
  * @file captcha.addon.php
@@ -74,8 +76,7 @@ 
 						$_SESSION['XE_VALIDATOR_RETURN_URL'] = Context::get('error_return_url');
 						$ModuleHandler->_setInputValueToSession();
 					}
-				}
-				else
+				} else
 				{
 					Context::addHtmlHeader('<script>
 						if(!captchaTargetAct) {var captchaTargetAct = [];}
@@ -205,8 +206,7 @@ 
 
 					$background_color = imagecolorallocate($im[$i + 1], 255, 255, 255);
 					imagecolortransparent($im[$i + 1], $background_color);
-				}
-				else
+				} else
 				{
 					imagestring($im[$i + 1], (array_pop($ran) % 3) + 3, 2, (array_pop($ran) % 4), $str, $text_color);
 				}
@@ -304,7 +304,9 @@ 
 
 		function compareCaptcha()
 		{
-			if(!in_array(Context::get('act'), $this->target_acts)) return true;
+			if(!in_array(Context::get('act'), $this->target_acts)) {
+				return true;
+			}
 
 			if($_SESSION['captcha_authed'])
 			{

Spacing +43 added lines, -43 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 
-if(!defined("__XE__")) exit();
+if (!defined("__XE__")) exit();
 
 /**
  * @file captcha.addon.php
@@ -9,7 +9,7 @@ 
  * @brief Captcha for a particular action
  * English alphabets and voice verification added
  * */
-if(!class_exists('AddonCaptcha', false))
+if (!class_exists('AddonCaptcha', false))
 {
 	class AddonCaptcha
 	{
@@ -24,7 +24,7 @@ 
 
 		function before_module_proc()
 		{
-			if($this->addon_info->act_type == 'everytime' && $_SESSION['captcha_authed'])
+			if ($this->addon_info->act_type == 'everytime' && $_SESSION['captcha_authed'])
 			{
 				unset($_SESSION['captcha_authed']);
 			}
@@ -33,19 +33,19 @@ 
 		function before_module_init(&$ModuleHandler)
 		{
 			$logged_info = Context::get('logged_info');
-			if($logged_info->is_admin == 'Y' || $logged_info->is_site_admin)
+			if ($logged_info->is_admin == 'Y' || $logged_info->is_site_admin)
 			{
 				return false;
 			}
-			if($this->addon_info->target != 'all' && Context::get('is_logged'))
+			if ($this->addon_info->target != 'all' && Context::get('is_logged'))
 			{
 				return false;
 			}
-			if($_SESSION['XE_VALIDATOR_ERROR'] == -1)
+			if ($_SESSION['XE_VALIDATOR_ERROR'] == -1)
 			{
 				$_SESSION['captcha_authed'] = false;
 			}
-			if($_SESSION['captcha_authed'])
+			if ($_SESSION['captcha_authed'])
 			{
 				return false;
 			}
@@ -54,13 +54,13 @@ 
 
 			$this->target_acts = array('procBoardInsertDocument', 'procBoardInsertComment', 'procIssuetrackerInsertIssue', 'procIssuetrackerInsertHistory', 'procTextyleInsertComment');
 
-			if(Context::getRequestMethod() != 'XMLRPC' && Context::getRequestMethod() !== 'JSON')
+			if (Context::getRequestMethod() != 'XMLRPC' && Context::getRequestMethod() !== 'JSON')
 			{
-				if($type == 'inline')
+				if ($type == 'inline')
 				{
-					if(!$this->compareCaptcha())
+					if (!$this->compareCaptcha())
 					{
-						Context::loadLang(_XE_PATH_ . 'addons/captcha/lang');
+						Context::loadLang(_XE_PATH_.'addons/captcha/lang');
 						$_SESSION['XE_VALIDATOR_ERROR'] = -1;
 						$_SESSION['XE_VALIDATOR_MESSAGE'] = Context::getLang('captcha_denied');
 						$_SESSION['XE_VALIDATOR_MESSAGE_TYPE'] = 'error';
@@ -72,16 +72,16 @@ 
 				{
 					Context::addHtmlHeader('<script>
 						if(!captchaTargetAct) {var captchaTargetAct = [];}
-						captchaTargetAct.push("' . implode('","', $this->target_acts) . '");
+						captchaTargetAct.push("' . implode('","', $this->target_acts).'");
 						</script>');
 					Context::loadFile(array('./addons/captcha/captcha.min.js', 'body', '', null), true);
 				}
 			}
 
 			// compare session when calling actions such as writing a post or a comment on the board/issue tracker module
-			if(!$_SESSION['captcha_authed'] && in_array(Context::get('act'), $this->target_acts))
+			if (!$_SESSION['captcha_authed'] && in_array(Context::get('act'), $this->target_acts))
 			{
-				Context::loadLang(_XE_PATH_ . 'addons/captcha/lang');
+				Context::loadLang(_XE_PATH_.'addons/captcha/lang');
 				$ModuleHandler->error = "captcha_denied";
 			}
 
@@ -91,7 +91,7 @@ 
 		function createKeyword()
 		{
 			$type = Context::get('captchaType');
-			if($type == 'inline' && $_SESSION['captcha_keyword'])
+			if ($type == 'inline' && $_SESSION['captcha_keyword'])
 			{
 				return;
 			}
@@ -104,19 +104,19 @@ 
 
 		function before_module_init_setCaptchaSession()
 		{
-			if($_SESSION['captcha_authed'])
+			if ($_SESSION['captcha_authed'])
 			{
 				return false;
 			}
 			// Load language files
-			Context::loadLang(_XE_PATH_ . 'addons/captcha/lang');
+			Context::loadLang(_XE_PATH_.'addons/captcha/lang');
 			// Generate keywords
 			$this->createKeyword();
 
 			$target = Context::getLang('target_captcha');
 			header("Content-Type: text/xml; charset=UTF-8");
 			header("Expires: Mon, 26 Jul 1997 05:00:00 GMT");
-			header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
+			header("Last-Modified: ".gmdate("D, d M Y H:i:s")." GMT");
 			header("Cache-Control: no-store, no-cache, must-revalidate");
 			header("Cache-Control: post-check=0, pre-check=0", false);
 			header("Pragma: no-cache");
@@ -133,11 +133,11 @@ 
 
 		function before_module_init_captchaImage()
 		{
-			if($_SESSION['captcha_authed'])
+			if ($_SESSION['captcha_authed'])
 			{
 				return false;
 			}
-			if(Context::get('renew'))
+			if (Context::get('renew'))
 			{
 				$this->createKeyword();
 			}
@@ -159,7 +159,7 @@ 
 		function createCaptchaImage($string)
 		{
 			$arr = array();
-			for($i = 0, $c = strlen($string); $i < $c; $i++)
+			for ($i = 0, $c = strlen($string); $i < $c; $i++)
 			{
 				$arr[] = $string{$i};
 			}
@@ -181,7 +181,7 @@ 
 			shuffle($deg);
 
 			// Create an image for each letter
-			foreach($arr as $i => $str)
+			foreach ($arr as $i => $str)
 			{
 				$im[$i + 1] = @imagecreate($w, $h);
 				$background_color = imagecolorallocate($im[$i + 1], 255, 255, 255);
@@ -191,7 +191,7 @@ 
 				$ran = range(1, 20);
 				shuffle($ran);
 
-				if(function_exists('imagerotate'))
+				if (function_exists('imagerotate'))
 				{
 					imagestring($im[$i + 1], (array_pop($ran) % 3) + 3, 2, (array_pop($ran) % 8), $str, $text_color);
 					$im[$i + 1] = imagerotate($im[$i + 1], array_pop($deg), 0);
@@ -206,7 +206,7 @@ 
 			}
 
 			// Combine images of each character
-			for($i = 1, $c = count($im); $i<$c; $i++)
+			for ($i = 1, $c = count($im); $i < $c; $i++)
 			{
 				imagecopy($im[0], $im[$i], (($w + 2) * ($i - 1)), 0, 0, 0, $w, $h);
 				imagedestroy($im[$i]);
@@ -218,7 +218,7 @@ 
 			imagecopyresized($big, $im[0], 0, 0, 0, 0, ($w + 2) * $big_count * $c, $h * $big_count, ($w + 2) * $c, $h);
 			imagedestroy($im[0]);
 
-			if(function_exists('imageantialias'))
+			if (function_exists('imageantialias'))
 			{
 				imageantialias($big, true);
 			}
@@ -230,7 +230,7 @@ 
 			$h = $h * $big_count;
 			$d = array_pop($deg);
 
-			for($i = -abs($d); $i < $h + abs($d); $i = $i + 7)
+			for ($i = -abs($d); $i < $h + abs($d); $i = $i + 7)
 			{
 				imageline($big, 0, $i + $d, $w, $i, $line_color);
 			}
@@ -238,7 +238,7 @@ 
 			$x = range(0, ($w - 10));
 			shuffle($x);
 
-			for($i = 0; $i < 200; $i++)
+			for ($i = 0; $i < 200; $i++)
 			{
 				imagesetpixel($big, $x[$i] % $w, $x[$i + 1] % $h, $line_color);
 			}
@@ -248,7 +248,7 @@ 
 
 		function before_module_init_captchaAudio()
 		{
-			if($_SESSION['captcha_authed'])
+			if ($_SESSION['captcha_authed'])
 			{
 				return false;
 			}
@@ -260,8 +260,8 @@ 
 			header("Content-Disposition: attachment; filename=\"captcha_audio.mp3\"");
 			header('Cache-Control: no-store, no-cache, must-revalidate');
 			header('Expires: Sun, 1 Jan 2000 12:00:00 GMT');
-			header('Last-Modified: ' . gmdate('D, d M Y H:i:s') . 'GMT');
-			header('Content-Length: ' . strlen($data));
+			header('Last-Modified: '.gmdate('D, d M Y H:i:s').'GMT');
+			header('Content-Length: '.strlen($data));
 
 			echo $data;
 			Context::close();
@@ -272,17 +272,17 @@ 
 		{
 			$data = '';
 			$_audio = './addons/captcha/audio/F_%s.mp3';
-			for($i = 0, $c = strlen($string); $i < $c; $i++)
+			for ($i = 0, $c = strlen($string); $i < $c; $i++)
 			{
 				$_data = FileHandler::readFile(sprintf($_audio, $string{$i}));
 
 				$start = rand(5, 68); // Random start in 4-byte header and 64 byte data
 				$datalen = strlen($_data) - $start - 256; // Last unchanged 256 bytes
 
-				for($j = $start; $j < $datalen; $j+=64)
+				for ($j = $start; $j < $datalen; $j += 64)
 				{
 					$ch = ord($_data{$j});
-					if($ch < 9 || $ch > 119)
+					if ($ch < 9 || $ch > 119)
 					{
 						continue;
 					}
@@ -297,14 +297,14 @@ 
 
 		function compareCaptcha()
 		{
-			if(!in_array(Context::get('act'), $this->target_acts)) return true;
+			if (!in_array(Context::get('act'), $this->target_acts)) return true;
 
-			if($_SESSION['captcha_authed'])
+			if ($_SESSION['captcha_authed'])
 			{
 				return true;
 			}
 
-			if(strtoupper($_SESSION['captcha_keyword']) == strtoupper(Context::get('secret_text')))
+			if (strtoupper($_SESSION['captcha_keyword']) == strtoupper(Context::get('secret_text')))
 			{
 				$_SESSION['captcha_authed'] = true;
 				return true;
@@ -317,14 +317,14 @@ 
 
 		function before_module_init_captchaCompare()
 		{
-			if(!$this->compareCaptcha())
+			if (!$this->compareCaptcha())
 			{
 				return false;
 			}
 
 			header("Content-Type: text/xml; charset=UTF-8");
 			header("Expires: Mon, 26 Jul 1997 05:00:00 GMT");
-			header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
+			header("Last-Modified: ".gmdate("D, d M Y H:i:s")." GMT");
 			header("Cache-Control: no-store, no-cache, must-revalidate");
 			header("Cache-Control: post-check=0, pre-check=0", false);
 			header("Pragma: no-cache");
@@ -339,7 +339,7 @@ 
 			unset($_SESSION['captcha_authed']);
 			$this->createKeyword();
 
-			$swfURL = getUrl() . 'addons/captcha/swf/play.swf';
+			$swfURL = getUrl().'addons/captcha/swf/play.swf';
 			Context::unloadFile('./addons/captcha/captcha.min.js');
 			Context::loadFile(array('./addons/captcha/inline_captcha.js', 'body'));
 
@@ -377,18 +377,18 @@ 
 
 $oAddonCaptcha = &$GLOBALS['__AddonCaptcha__'];
 
-if(method_exists($oAddonCaptcha, $called_position))
+if (method_exists($oAddonCaptcha, $called_position))
 {
-	if(!call_user_func_array(array(&$oAddonCaptcha, $called_position), array(&$this)))
+	if (!call_user_func_array(array(&$oAddonCaptcha, $called_position), array(&$this)))
 	{
 		return false;
 	}
 }
 
 $addon_act = Context::get('captcha_action');
-if($addon_act && method_exists($oAddonCaptcha, $called_position . '_' . $addon_act))
+if ($addon_act && method_exists($oAddonCaptcha, $called_position.'_'.$addon_act))
 {
-	if(!call_user_func_array(array(&$oAddonCaptcha, $called_position . '_' . $addon_act), array(&$this)))
+	if (!call_user_func_array(array(&$oAddonCaptcha, $called_position.'_'.$addon_act), array(&$this)))
 	{
 		return false;
 	}

addons/mobile/classes/mobile.class.php

Doc Comments +13 added lines

@@ -279,6 +279,7 @@ 
 
 	/**
 	 * @brief Specify title
+	 * @param string $title
 	 */
 	function setTitle($title)
 	{
@@ -298,6 +299,7 @@ 
 	/**
 	 * @brief Content Cleanup
 	 * In HTML content, the ability to extract text and links
+	 * @param string $content
 	 */
 	function setContent($content)
 	{
@@ -371,6 +373,9 @@ 
 
 	/**
 	 * @brief cutting the number of byte functions
+	 * @param string $string
+	 * @param integer $cut_size
+	 * @return string
 	 */
 	function cutStr($string, $cut_size)
 	{
@@ -387,6 +392,8 @@ 
 
 	/**
 	 * @brief Specifies the home url
+	 * @param string $url
+	 * @param string $text
 	 */
 	function setHomeUrl($url, $text)
 	{
@@ -397,6 +404,8 @@ 
 
 	/**
 	 * @brief Specify upper url
+	 * @param string $url
+	 * @param string $text
 	 */
 	function setUpperUrl($url, $text)
 	{
@@ -407,6 +416,8 @@ 
 
 	/**
 	 * @brief Specify prev url
+	 * @param string $url
+	 * @param string $text
 	 */
 	function setPrevUrl($url, $text)
 	{
@@ -417,6 +428,8 @@ 
 
 	/**
 	 * @brief Specify next url
+	 * @param string $url
+	 * @param string $text
 	 */
 	function setNextUrl($url, $text)
 	{

Braces +87 added lines, -37 removed lines

@@ -49,7 +49,9 @@ 
 		{
 
 			$browserType = mobileXE::getBrowserType();
-			if(!$browserType) return;
+			if(!$browserType) {
+				return;
+			}
 
 			$class_file = sprintf('%saddons/mobile/classes/%s.class.php', _XE_PATH_, $browserType);
 			require_once($class_file);
@@ -59,14 +61,18 @@ 
 			{
 				$lang_supported = Context::get('lang_supported');
 				$this->lang = str_replace(array('<?php /**','**/ ?>'),array('',''),$this->lang);
-				if(isset($lang_supported[$this->lang])) Context::setLangType($this->lang);
+				if(isset($lang_supported[$this->lang])) {
+					Context::setLangType($this->lang);
+				}
 			}
 			Context::loadLang(_XE_PATH_.'addons/mobile/lang');
 
 			$instance = new wap();
 
 			$mobilePage = (int)Context::get('mpage');
-			if(!$mobilePage) $mobilePage = 1;
+			if(!$mobilePage) {
+				$mobilePage = 1;
+			}
 
 			$instance->setMobilePage($mobilePage);
 		}
@@ -107,8 +113,11 @@ 
 	 */
 	function isLangChange()
 	{
-		if($this->languageMode) return true;
-		else return false;
+		if($this->languageMode) {
+			return true;
+		} else {
+			return false;
+		}
 	}
 
 	/**
@@ -122,7 +131,9 @@ 
 		if($this->lang && isset($lang_supported[$this->lang]))
 		{
 			$langbuff = FileHandler::readFile('./files/cache/addons/mobile/setLangType/personal_settings/'.md5(trim($_SERVER['HTTP_USER_AGENT']).trim($_SERVER['HTTP_PHONE_NUMBER']).trim($_SERVER['HTTP_HTTP_PHONE_NUMBER'])).'.php');
-			if($langbuff) FileHandler::removeFile('./files/cache/addons/mobile/setLangType/personal_settings/'.md5(trim($_SERVER['HTTP_USER_AGENT']).trim($_SERVER['HTTP_PHONE_NUMBER']).trim($_SERVER['HTTP_HTTP_PHONE_NUMBER'])).'.php');
+			if($langbuff) {
+				FileHandler::removeFile('./files/cache/addons/mobile/setLangType/personal_settings/'.md5(trim($_SERVER['HTTP_USER_AGENT']).trim($_SERVER['HTTP_PHONE_NUMBER']).trim($_SERVER['HTTP_HTTP_PHONE_NUMBER'])).'.php');
+			}
 			$langbuff = '<?php /**'.$this->lang.'**/ ?>';
 			FileHandler::writeFile('./files/cache/addons/mobile/setLangType/personal_settings/'.md5(trim($_SERVER['HTTP_USER_AGENT']).trim($_SERVER['HTTP_PHONE_NUMBER']).trim($_SERVER['HTTP_HTTP_PHONE_NUMBER'])).'.php',$langbuff);
 		}
@@ -133,7 +144,9 @@ 
 	 */
 	function setModuleInfo(&$module_info)
 	{
-		if($this->module_info) return; 
+		if($this->module_info) {
+			return;
+		}
 		$this->module_info = $module_info;
 	}
 
@@ -142,12 +155,16 @@ 
 	 */
 	function setModuleInstance(&$oModule)
 	{
-		if($this->oModule) return;
+		if($this->oModule) {
+			return;
+		}
 		// Save instance
 		$this->oModule = $oModule;
 		// Of the current module if there is a menu by menu
 		$menu_cache_file = sprintf(_XE_PATH_.'files/cache/menu/%d.php', $this->module_info->menu_srl);
-		if(!file_exists($menu_cache_file)) return;
+		if(!file_exists($menu_cache_file)) {
+			return;
+		}
 
 		include $menu_cache_file;
 		// One-dimensional arrangement of menu changes
@@ -168,10 +185,11 @@ 
 			if($parent_srl && $listed_items[$parent_srl])
 			{
 				$parent_item = $listed_items[$parent_srl];
-				if($parent_item) $this->setUpperUrl(getUrl('','mid',$parent_item['mid']), Context::getLang('cmd_go_upper'));
+				if($parent_item) {
+					$this->setUpperUrl(getUrl('','mid',$parent_item['mid']), Context::getLang('cmd_go_upper'));
+				}
 			}
-		}
-		elseif (!$this->isNavigationMode())
+		} elseif (!$this->isNavigationMode())
 		{
 			$this->setUpperUrl(getUrl('','mid',$this->index_mid,'nm','1','cmid',0), Context::getLang('cmd_view_sitemap'));
 		}
@@ -183,7 +201,9 @@ 
 	 */
 	function getBrowserType()
 	{
-		if(Context::get('smartphone')) return null;
+		if(Context::get('smartphone')) {
+			return null;
+		}
 		// Determine the type of browser
 		$browserAccept = $_SERVER['HTTP_ACCEPT'];
 		$userAgent = $_SERVER['HTTP_USER_AGENT'];
@@ -193,9 +213,11 @@ 
 		{
 			Context::set('mobile_skt',1);
 			return "wml";
+		} elseif(stripos($browserAccept, "hdml") !== FALSE) {
+			return "hdml";
+		} elseif(stripos($userAgent, "cellphone") !== FALSE) {
+			return  "mhtml";
 		}
-		elseif(stripos($browserAccept, "hdml") !== FALSE) return "hdml";
-		elseif(stripos($userAgent, "cellphone") !== FALSE) return  "mhtml";
 		return null;
 	}
 
@@ -204,9 +226,13 @@ 
 	 */
 	function setCharSet($charset = 'UTF-8')
 	{
-		if(!$charset) $charset = 'UTF-8';
+		if(!$charset) {
+			$charset = 'UTF-8';
+		}
 		// SKT supports the euc-kr
-		if(Context::get('mobile_skt')==1) $charset = 'euc-kr';
+		if(Context::get('mobile_skt')==1) {
+			$charset = 'euc-kr';
+		}
 
 		$this->charset = $charset;
 	}
@@ -216,7 +242,9 @@ 
 	 */
 	function setMobilePage($page=1)
 	{
-		if(!$page) $page = 1;
+		if(!$page) {
+			$page = 1;
+		}
 		$this->mobilePage = $page;
 	}
 
@@ -329,7 +357,9 @@ 
 
 			foreach($allow_tag_array as $tag)
 			{
-				if($tag == '<br>') continue;
+				if($tag == '<br>') {
+					continue;
+				}
 				$tag_open_pos = strpos($content, str_replace('>','',$tag));
 				$tag_close_pos = strpos($content, str_replace('<','</',$tag));
 				if($tag_open_pos!==false && $tag_close_pos || $tag_close_pos < $tag_open_pos)
@@ -390,7 +420,9 @@ 
 	 */
 	function setHomeUrl($url, $text)
 	{
-		if(!$url) $url = '#';
+		if(!$url) {
+			$url = '#';
+		}
 		$this->homeUrl->url = $url;
 		$this->homeUrl->text = $text;
 	}
@@ -400,7 +432,9 @@ 
 	 */
 	function setUpperUrl($url, $text)
 	{
-		if(!$url) $url = '#';
+		if(!$url) {
+			$url = '#';
+		}
 		$this->upperUrl->url = $url;
 		$this->upperUrl->text = $text;
 	}
@@ -410,7 +444,9 @@ 
 	 */
 	function setPrevUrl($url, $text)
 	{
-		if(!$url) $url = '#';
+		if(!$url) {
+			$url = '#';
+		}
 		$this->prevUrl->url = $url;
 		$this->prevUrl->text = $text;
 	}
@@ -420,7 +456,9 @@ 
 	 */
 	function setNextUrl($url, $text)
 	{
-		if(!$url) $url = '#';
+		if(!$url) {
+			$url = '#';
+		}
 		$this->nextUrl->url = $url;
 		$this->nextUrl->text = $text;
 	}
@@ -430,7 +468,9 @@ 
 	 */
 	function setEtcBtn($url, $text)
 	{
-		if(!$url) $url = '#';
+		if(!$url) {
+			$url = '#';
+		}
 		$etc['url'] = $url;
 		$etc['text'] = htmlspecialchars($text);
 		$this->etcBtn[] = $etc;
@@ -444,7 +484,9 @@ 
 		// Home button assignments
 		$this->setHomeUrl(getUrl(), Context::getLang('cmd_go_home'));
 		// Specify the title
-		if(!$this->title) $this->setTitle(Context::getBrowserTitle());
+		if(!$this->title) {
+			$this->setTitle(Context::getBrowserTitle());
+		}
 
 		ob_start();
 		// Output header
@@ -460,8 +502,11 @@ 
 
 		$content = ob_get_clean();
 		// After conversion output
-		if(strtolower($this->charset) == 'utf-8') print $content;
-		else print iconv('UTF-8',$this->charset."//TRANSLIT//IGNORE", $content);
+		if(strtolower($this->charset) == 'utf-8') {
+			print $content;
+		} else {
+			print iconv('UTF-8',$this->charset."//TRANSLIT//IGNORE", $content);
+		}
 
 		exit();
 	}
@@ -490,15 +535,16 @@ 
 	 */
 	function getListedItems($menu, &$listed_items, &$node_list)
 	{
-		if(!count($menu)) return;
+		if(!count($menu)) {
+			return;
+		}
 		foreach($menu as $node_srl => $item)
 		{
 			if(preg_match('/^([a-zA-Z0-9\_\-]+)$/', $item['url']))
 			{
 				$mid = $item['mid'] = $item['url'];
 				$node_list[$mid] = $node_srl;
-			}
-			else
+			} else
 			{
 				$mid = $item['mid'] = null;
 			}
@@ -529,8 +575,7 @@ 
 				$childs[] = $obj;
 			}
 
-		}
-		else
+		} else
 		{
 			$list = $this->menu;
 			$upper_srl = 0;
@@ -540,13 +585,14 @@ 
 		{
 			foreach($list as $key => $val)
 			{
-				if(!$val['text']) continue;
+				if(!$val['text']) {
+					continue;
+				}
 				$obj = array();
 				if(!count($val['list']))
 				{
 					$obj['href'] = getUrl('','mid',$val['url']);
-				}
-				else
+				} else
 				{
 					$obj['href'] = getUrl('cmid',$val['node_srl']);
 				}
@@ -602,10 +648,14 @@ 
 	{
 		// Create WAP class objects of the selected module
 		$oModule = &getWap($this->module_info->module);
-		if(!$oModule || !method_exists($oModule, 'procWAP') ) return;
+		if(!$oModule || !method_exists($oModule, 'procWAP') ) {
+			return;
+		}
 
 		$vars = get_object_vars($this->oModule);
-		if(count($vars)) foreach($vars as $key => $val) $oModule->{$key}  = $val;
+		if(count($vars)) {
+			foreach($vars as $key => $val) $oModule->{$key}  = $val;
+		}
 		// Run
 		$oModule->procWAP($this);
 		// Output

Spacing +98 added lines, -98 removed lines

@@ -45,28 +45,28 @@ 
 	{
 		static $instance = null;
 
-		if(!$instance)
+		if (!$instance)
 		{
 
 			$browserType = mobileXE::getBrowserType();
-			if(!$browserType) return;
+			if (!$browserType) return;
 
 			$class_file = sprintf('%saddons/mobile/classes/%s.class.php', _XE_PATH_, $browserType);
 			require_once($class_file);
 			// Download mobile language settings (cookies, not willing to come up when you click create cache file ...- is initialized ..)
 			$this->lang = FileHandler::readFile('./files/cache/addons/mobile/setLangType/personal_settings/'.md5(trim($_SERVER['HTTP_USER_AGENT']).trim($_SERVER['HTTP_PHONE_NUMBER']).trim($_SERVER['HTTP_HTTP_PHONE_NUMBER'])).'.php');
-			if($this->lang)
+			if ($this->lang)
 			{
 				$lang_supported = Context::get('lang_supported');
-				$this->lang = str_replace(array('<?php /**','**/ ?>'),array('',''),$this->lang);
-				if(isset($lang_supported[$this->lang])) Context::setLangType($this->lang);
+				$this->lang = str_replace(array('<?php /**', '**/ ?>'), array('', ''), $this->lang);
+				if (isset($lang_supported[$this->lang])) Context::setLangType($this->lang);
 			}
 			Context::loadLang(_XE_PATH_.'addons/mobile/lang');
 
 			$instance = new wap();
 
-			$mobilePage = (int)Context::get('mpage');
-			if(!$mobilePage) $mobilePage = 1;
+			$mobilePage = (int) Context::get('mpage');
+			if (!$mobilePage) $mobilePage = 1;
 
 			$instance->setMobilePage($mobilePage);
 		}
@@ -79,13 +79,13 @@ 
 	function mobileXE()
 	{
 		// Check navigation mode
-		if(Context::get('nm'))
+		if (Context::get('nm'))
 		{
 			$this->navigationMode = 1;
-			$this->cmid = (int)Context::get('cmid');
+			$this->cmid = (int) Context::get('cmid');
 		}
 
-		if(Context::get('lcm'))
+		if (Context::get('lcm'))
 		{
 			$this->languageMode = 1;
 			$this->lang = Context::get('sel_lang');
@@ -98,7 +98,7 @@ 
 	 */
 	function isNavigationMode()
 	{
-		return ($this->navigationMode && $this->module_info->menu_srl)?true:false;
+		return ($this->navigationMode && $this->module_info->menu_srl) ?true:false;
 	}
 
 	/**
@@ -107,7 +107,7 @@ 
 	 */
 	function isLangChange()
 	{
-		if($this->languageMode) return true;
+		if ($this->languageMode) return true;
 		else return false;
 	}
 
@@ -119,12 +119,12 @@ 
 	{
 		$lang_supported = Context::get('lang_supported');
 		// Make sure that the language variables and parameters are valid
-		if($this->lang && isset($lang_supported[$this->lang]))
+		if ($this->lang && isset($lang_supported[$this->lang]))
 		{
 			$langbuff = FileHandler::readFile('./files/cache/addons/mobile/setLangType/personal_settings/'.md5(trim($_SERVER['HTTP_USER_AGENT']).trim($_SERVER['HTTP_PHONE_NUMBER']).trim($_SERVER['HTTP_HTTP_PHONE_NUMBER'])).'.php');
-			if($langbuff) FileHandler::removeFile('./files/cache/addons/mobile/setLangType/personal_settings/'.md5(trim($_SERVER['HTTP_USER_AGENT']).trim($_SERVER['HTTP_PHONE_NUMBER']).trim($_SERVER['HTTP_HTTP_PHONE_NUMBER'])).'.php');
+			if ($langbuff) FileHandler::removeFile('./files/cache/addons/mobile/setLangType/personal_settings/'.md5(trim($_SERVER['HTTP_USER_AGENT']).trim($_SERVER['HTTP_PHONE_NUMBER']).trim($_SERVER['HTTP_HTTP_PHONE_NUMBER'])).'.php');
 			$langbuff = '<?php /**'.$this->lang.'**/ ?>';
-			FileHandler::writeFile('./files/cache/addons/mobile/setLangType/personal_settings/'.md5(trim($_SERVER['HTTP_USER_AGENT']).trim($_SERVER['HTTP_PHONE_NUMBER']).trim($_SERVER['HTTP_HTTP_PHONE_NUMBER'])).'.php',$langbuff);
+			FileHandler::writeFile('./files/cache/addons/mobile/setLangType/personal_settings/'.md5(trim($_SERVER['HTTP_USER_AGENT']).trim($_SERVER['HTTP_PHONE_NUMBER']).trim($_SERVER['HTTP_HTTP_PHONE_NUMBER'])).'.php', $langbuff);
 		}
 	}
 
@@ -133,7 +133,7 @@ 
 	 */
 	function setModuleInfo(&$module_info)
 	{
-		if($this->module_info) return; 
+		if ($this->module_info) return; 
 		$this->module_info = $module_info;
 	}
 
@@ -142,12 +142,12 @@ 
 	 */
 	function setModuleInstance(&$oModule)
 	{
-		if($this->oModule) return;
+		if ($this->oModule) return;
 		// Save instance
 		$this->oModule = $oModule;
 		// Of the current module if there is a menu by menu
 		$menu_cache_file = sprintf(_XE_PATH_.'files/cache/menu/%d.php', $this->module_info->menu_srl);
-		if(!file_exists($menu_cache_file)) return;
+		if (!file_exists($menu_cache_file)) return;
 
 		include $menu_cache_file;
 		// One-dimensional arrangement of menu changes
@@ -162,18 +162,18 @@ 
 		$this->index_mid = $k[0];
 		// The depth of the current menu, the top button to specify if one or more
 		$cur_menu_item = $listed_items[$node_list[$this->module_info->mid]];
-		if($cur_menu_item['parent_srl'])
+		if ($cur_menu_item['parent_srl'])
 		{
 			$parent_srl = $cur_menu_item['parent_srl'];
-			if($parent_srl && $listed_items[$parent_srl])
+			if ($parent_srl && $listed_items[$parent_srl])
 			{
 				$parent_item = $listed_items[$parent_srl];
-				if($parent_item) $this->setUpperUrl(getUrl('','mid',$parent_item['mid']), Context::getLang('cmd_go_upper'));
+				if ($parent_item) $this->setUpperUrl(getUrl('', 'mid', $parent_item['mid']), Context::getLang('cmd_go_upper'));
 			}
 		}
 		elseif (!$this->isNavigationMode())
 		{
-			$this->setUpperUrl(getUrl('','mid',$this->index_mid,'nm','1','cmid',0), Context::getLang('cmd_view_sitemap'));
+			$this->setUpperUrl(getUrl('', 'mid', $this->index_mid, 'nm', '1', 'cmid', 0), Context::getLang('cmd_view_sitemap'));
 		}
 	}
 
@@ -183,19 +183,19 @@ 
 	 */
 	function getBrowserType()
 	{
-		if(Context::get('smartphone')) return null;
+		if (Context::get('smartphone')) return null;
 		// Determine the type of browser
 		$browserAccept = $_SERVER['HTTP_ACCEPT'];
 		$userAgent = $_SERVER['HTTP_USER_AGENT'];
 		$wap_sid = $_SERVER['HTTP_X_UP_SUBNO'];
 
-		if(stripos($userAgent, "SKT11") !== FALSE || stripos($browserAccept, "skt") !== FALSE)
+		if (stripos($userAgent, "SKT11") !== FALSE || stripos($browserAccept, "skt") !== FALSE)
 		{
-			Context::set('mobile_skt',1);
+			Context::set('mobile_skt', 1);
 			return "wml";
 		}
-		elseif(stripos($browserAccept, "hdml") !== FALSE) return "hdml";
-		elseif(stripos($userAgent, "cellphone") !== FALSE) return  "mhtml";
+		elseif (stripos($browserAccept, "hdml") !== FALSE) return "hdml";
+		elseif (stripos($userAgent, "cellphone") !== FALSE) return  "mhtml";
 		return null;
 	}
 
@@ -204,9 +204,9 @@ 
 	 */
 	function setCharSet($charset = 'UTF-8')
 	{
-		if(!$charset) $charset = 'UTF-8';
+		if (!$charset) $charset = 'UTF-8';
 		// SKT supports the euc-kr
-		if(Context::get('mobile_skt')==1) $charset = 'euc-kr';
+		if (Context::get('mobile_skt') == 1) $charset = 'euc-kr';
 
 		$this->charset = $charset;
 	}
@@ -214,9 +214,9 @@ 
 	/**
 	 * @brief Limited capacity of mobile devices, specifying a different virtual page
 	 */
-	function setMobilePage($page=1)
+	function setMobilePage($page = 1)
 	{
-		if(!$page) $page = 1;
+		if (!$page) $page = 1;
 		$this->mobilePage = $page;
 	}
 
@@ -227,34 +227,34 @@ 
 	{
 		// If more than nine the number of menu paging processing itself
 		$menu_count = count($childs);
-		if($menu_count>9)
+		if ($menu_count > 9)
 		{
-			$startNum = ($this->mobilePage-1)*9;
+			$startNum = ($this->mobilePage - 1) * 9;
 			$idx = 0;
 			$new_childs = array();
-			foreach($childs as $k => $v)
+			foreach ($childs as $k => $v)
 			{
-				if($idx >= $startNum && $idx < $startNum+9)
+				if ($idx >= $startNum && $idx < $startNum + 9)
 				{
 					$new_childs[$k] = $v;
 				}
-				$idx ++;
+				$idx++;
 			}
 			$childs = $new_childs;
 
-			$this->totalPage = (int)(($menu_count-1)/9)+1;
+			$this->totalPage = (int) (($menu_count - 1) / 9) + 1;
 			// next/prevUrl specify
-			if($this->mobilePage>1)
+			if ($this->mobilePage > 1)
 			{
-				$url = getUrl('mid',$_GET['mid'],'mpage',$this->mobilePage-1);
-				$text = sprintf('%s (%d/%d)', Context::getLang('cmd_prev'), $this->mobilePage-1, $this->totalPage);
+				$url = getUrl('mid', $_GET['mid'], 'mpage', $this->mobilePage - 1);
+				$text = sprintf('%s (%d/%d)', Context::getLang('cmd_prev'), $this->mobilePage - 1, $this->totalPage);
 				$this->setPrevUrl($url, $text);
 			}
 
-			if($this->mobilePage<$this->totalPage)
+			if ($this->mobilePage < $this->totalPage)
 			{
-				$url = getUrl('mid',$_GET['mid'],'mpage',$this->mobilePage+1);
-				$text = sprintf('%s (%d/%d)', Context::getLang('cmd_next'), $this->mobilePage+1, $this->totalPage);
+				$url = getUrl('mid', $_GET['mid'], 'mpage', $this->mobilePage + 1);
+				$text = sprintf('%s (%d/%d)', Context::getLang('cmd_next'), $this->mobilePage + 1, $this->totalPage);
 				$this->setNextUrl($url, $text);
 			}
 		} 
@@ -266,7 +266,7 @@ 
 	 */
 	function hasChilds()
 	{
-		return count($this->childs)?true:0;
+		return count($this->childs) ?true:0;
 	}
 
 	/**
@@ -302,7 +302,7 @@ 
 	function setContent($content)
 	{
 		$oModuleController = getController('module');
-		$allow_tag_array = array('<a>','<br>','<p>','<b>','<i>','<u>','<em>','<small>','<strong>','<big>','<table>','<tr>','<td>');
+		$allow_tag_array = array('<a>', '<br>', '<p>', '<b>', '<i>', '<u>', '<em>', '<small>', '<strong>', '<big>', '<table>', '<tr>', '<td>');
 		// Links/wrap, remove all tags except gangjoman
 		$content = strip_tags($content, implode($allow_tag_array));
 		// Margins tab removed
@@ -310,16 +310,16 @@ 
 		// Repeat two more times the space and remove julnanumeul
 		$content = preg_replace('/( ){2,}/s', '', $content);
 		$content = preg_replace("/([\r\n]+)/s", "\r\n", $content);
-		$content = preg_replace(array("/<a/i","/<\/a/i","/<b/i","/<\/b/i","/<br/i"),array('<a','</a','<b','</b','<br'),$content);
-		$content = str_replace(array("<br>","<br />"), array("<br/>","<br/>"), $content);
+		$content = preg_replace(array("/<a/i", "/<\/a/i", "/<b/i", "/<\/b/i", "/<br/i"), array('<a', '</a', '<b', '</b', '<br'), $content);
+		$content = str_replace(array("<br>", "<br />"), array("<br/>", "<br/>"), $content);
 
-		while(strpos($content, '<br/><br/>'))
+		while (strpos($content, '<br/><br/>'))
 		{
-			$content = str_replace('<br/><br/>','<br/>',$content);
+			$content = str_replace('<br/><br/>', '<br/>', $content);
 		}
 		// If the required size of a deck of mobile content to write down all the dividing pages
 		$contents = array();
-		while($content)
+		while ($content)
 		{
 			$tmp = $this->cutStr($content, $this->deckSize, '');
 			$contents[] = $tmp;
@@ -327,46 +327,46 @@ 
 
 			//$content = str_replace(array('&','<','>','"','&amp;nbsp;'), array('&amp;','&lt;','&gt;','&quot;',' '), $content);
 
-			foreach($allow_tag_array as $tag)
+			foreach ($allow_tag_array as $tag)
 			{
-				if($tag == '<br>') continue;
-				$tag_open_pos = strpos($content, str_replace('>','',$tag));
-				$tag_close_pos = strpos($content, str_replace('<','</',$tag));
-				if($tag_open_pos!==false && $tag_close_pos || $tag_close_pos < $tag_open_pos)
+				if ($tag == '<br>') continue;
+				$tag_open_pos = strpos($content, str_replace('>', '', $tag));
+				$tag_close_pos = strpos($content, str_replace('<', '</', $tag));
+				if ($tag_open_pos !== false && $tag_close_pos || $tag_close_pos < $tag_open_pos)
 				{
-					$contents[count($contents)-1] .= substr($content, 0, $tag_close_pos + strlen($tag) + 1);
+					$contents[count($contents) - 1] .= substr($content, 0, $tag_close_pos + strlen($tag) + 1);
 					$content = substr($content, $tag_close_pos + strlen($tag) + 1);
 				}
 			}
 
 			$tag_open_pos = strpos($content, '&');
 			$tag_close_pos = strpos($content, ';');
-			if($tag_open_pos!==false && $tag_close_pos || $tag_close_pos < $tag_open_pos)
+			if ($tag_open_pos !== false && $tag_close_pos || $tag_close_pos < $tag_open_pos)
 			{
-				$contents[count($contents)-1] .= substr($content, 0, $tag_close_pos + 1);
+				$contents[count($contents) - 1] .= substr($content, 0, $tag_close_pos + 1);
 				$content = substr($content, $tag_close_pos + 1);
 			}
 		}
 
 		$this->totalPage = count($contents);
 		// next/prevUrl specify
-		if($this->mobilePage>1)
+		if ($this->mobilePage > 1)
 		{
-			$url = getUrl('mid',$_GET['mid'],'mpage',$this->mobilePage-1);
-			$text = sprintf('%s (%d/%d)', Context::getLang('cmd_prev'), $this->mobilePage-1, $this->totalPage);
+			$url = getUrl('mid', $_GET['mid'], 'mpage', $this->mobilePage - 1);
+			$text = sprintf('%s (%d/%d)', Context::getLang('cmd_prev'), $this->mobilePage - 1, $this->totalPage);
 			$this->setPrevUrl($url, $text);
 		}
 
-		if($this->mobilePage<$this->totalPage)
+		if ($this->mobilePage < $this->totalPage)
 		{
-			$url = getUrl('mid',$_GET['mid'],'mpage',$this->mobilePage+1);
-			$text = sprintf('%s (%d/%d)', Context::getLang('cmd_next'), $this->mobilePage+1, $this->totalPage);
+			$url = getUrl('mid', $_GET['mid'], 'mpage', $this->mobilePage + 1);
+			$text = sprintf('%s (%d/%d)', Context::getLang('cmd_next'), $this->mobilePage + 1, $this->totalPage);
 			$this->setNextUrl($url, $text);
 		}
 
-		$this->content = $contents[$this->mobilePage-1];
+		$this->content = $contents[$this->mobilePage - 1];
 		$oModuleController->replaceDefinedLangCode($this->content);
-		$content = str_replace(array('$','\''), array('$$','&apos;'), $content);
+		$content = str_replace(array('$', '\''), array('$$', '&apos;'), $content);
 	}
 
 	/**
@@ -390,7 +390,7 @@ 
 	 */
 	function setHomeUrl($url, $text)
 	{
-		if(!$url) $url = '#';
+		if (!$url) $url = '#';
 		$this->homeUrl->url = $url;
 		$this->homeUrl->text = $text;
 	}
@@ -400,7 +400,7 @@ 
 	 */
 	function setUpperUrl($url, $text)
 	{
-		if(!$url) $url = '#';
+		if (!$url) $url = '#';
 		$this->upperUrl->url = $url;
 		$this->upperUrl->text = $text;
 	}
@@ -410,7 +410,7 @@ 
 	 */
 	function setPrevUrl($url, $text)
 	{
-		if(!$url) $url = '#';
+		if (!$url) $url = '#';
 		$this->prevUrl->url = $url;
 		$this->prevUrl->text = $text;
 	}
@@ -420,7 +420,7 @@ 
 	 */
 	function setNextUrl($url, $text)
 	{
-		if(!$url) $url = '#';
+		if (!$url) $url = '#';
 		$this->nextUrl->url = $url;
 		$this->nextUrl->text = $text;
 	}
@@ -430,7 +430,7 @@ 
 	 */
 	function setEtcBtn($url, $text)
 	{
-		if(!$url) $url = '#';
+		if (!$url) $url = '#';
 		$etc['url'] = $url;
 		$etc['text'] = htmlspecialchars($text);
 		$this->etcBtn[] = $etc;
@@ -444,7 +444,7 @@ 
 		// Home button assignments
 		$this->setHomeUrl(getUrl(), Context::getLang('cmd_go_home'));
 		// Specify the title
-		if(!$this->title) $this->setTitle(Context::getBrowserTitle());
+		if (!$this->title) $this->setTitle(Context::getBrowserTitle());
 
 		ob_start();
 		// Output header
@@ -460,8 +460,8 @@ 
 
 		$content = ob_get_clean();
 		// After conversion output
-		if(strtolower($this->charset) == 'utf-8') print $content;
-		else print iconv('UTF-8',$this->charset."//TRANSLIT//IGNORE", $content);
+		if (strtolower($this->charset) == 'utf-8') print $content;
+		else print iconv('UTF-8', $this->charset."//TRANSLIT//IGNORE", $content);
 
 		exit();
 	}
@@ -490,10 +490,10 @@ 
 	 */
 	function getListedItems($menu, &$listed_items, &$node_list)
 	{
-		if(!count($menu)) return;
-		foreach($menu as $node_srl => $item)
+		if (!count($menu)) return;
+		foreach ($menu as $node_srl => $item)
 		{
-			if(preg_match('/^([a-zA-Z0-9\_\-]+)$/', $item['url']))
+			if (preg_match('/^([a-zA-Z0-9\_\-]+)$/', $item['url']))
 			{
 				$mid = $item['mid'] = $item['url'];
 				$node_list[$mid] = $node_srl;
@@ -515,16 +515,16 @@ 
 	{
 		$childs = array();
 
-		if($this->cmid)
+		if ($this->cmid)
 		{
 			$cur_item = $this->listed_items[$this->cmid];
-			$upper_srl = $cur_item['parent_srl'];;
-			$list = $cur_item['list'];;
-			$this->setUpperUrl(getUrl('cmid',$upper_srl), Context::getLang('cmd_go_upper'));
-			if(preg_match('/^([a-zA-Z0-9\_\-]+)$/', $cur_item['url']))
+			$upper_srl = $cur_item['parent_srl']; ;
+			$list = $cur_item['list']; ;
+			$this->setUpperUrl(getUrl('cmid', $upper_srl), Context::getLang('cmd_go_upper'));
+			if (preg_match('/^([a-zA-Z0-9\_\-]+)$/', $cur_item['url']))
 			{
 				$obj = array();
-				$obj['href'] = getUrl('','mid',$cur_item['url']);
+				$obj['href'] = getUrl('', 'mid', $cur_item['url']);
 				$obj['link'] = $obj['text'] = '['.$cur_item['text'].']';
 				$childs[] = $obj;
 			}
@@ -536,19 +536,19 @@ 
 			$upper_srl = 0;
 		}
 
-		if(count($list))
+		if (count($list))
 		{
-			foreach($list as $key => $val)
+			foreach ($list as $key => $val)
 			{
-				if(!$val['text']) continue;
+				if (!$val['text']) continue;
 				$obj = array();
-				if(!count($val['list']))
+				if (!count($val['list']))
 				{
-					$obj['href'] = getUrl('','mid',$val['url']);
+					$obj['href'] = getUrl('', 'mid', $val['url']);
 				}
 				else
 				{
-					$obj['href'] = getUrl('cmid',$val['node_srl']);
+					$obj['href'] = getUrl('cmid', $val['node_srl']);
 				}
 				$obj['link'] = $obj['text'] = $val['text'];
 				$childs[] = $obj;
@@ -567,25 +567,25 @@ 
 		$childs = array();
 
 		$this->lang = FileHandler::readFile('./files/cache/addons/mobile/setLangType/personal_settings/'.md5(trim($_SERVER['HTTP_USER_AGENT']).trim($_SERVER['HTTP_PHONE_NUMBER']).trim($_SERVER['HTTP_HTTP_PHONE_NUMBER'])).'.php');
-		if($this->lang)
+		if ($this->lang)
 		{
-			$this->lang = str_replace(array('<?php /**','**/ ?>'),array('',''),$this->lang);
+			$this->lang = str_replace(array('<?php /**', '**/ ?>'), array('', ''), $this->lang);
 			Context::setLangType($this->lang);
 		}
 		$lang_supported = Context::get('lang_supported');
 		$lang_type = Context::getLangType();
 		$obj = array();
 		$obj['link'] = $obj['text'] = Context::getLang('president_lang').' : '.$lang_supported[$lang_type];
-		$obj['href'] = getUrl('sel_lang',$lang_type);
+		$obj['href'] = getUrl('sel_lang', $lang_type);
 		$childs[] = $obj;
 
-		if(is_array($lang_supported))
+		if (is_array($lang_supported))
 		{
-			foreach($lang_supported as $key => $val)
+			foreach ($lang_supported as $key => $val)
 			{
 				$obj = array();
 				$obj['link'] = $obj['text'] = $val;
-				$obj['href'] = getUrl('sel_lang',$key);
+				$obj['href'] = getUrl('sel_lang', $key);
 				$childs[] = $obj;
 			}
 		}
@@ -602,10 +602,10 @@ 
 	{
 		// Create WAP class objects of the selected module
 		$oModule = &getWap($this->module_info->module);
-		if(!$oModule || !method_exists($oModule, 'procWAP') ) return;
+		if (!$oModule || !method_exists($oModule, 'procWAP')) return;
 
 		$vars = get_object_vars($this->oModule);
-		if(count($vars)) foreach($vars as $key => $val) $oModule->{$key}  = $val;
+		if (count($vars)) foreach ($vars as $key => $val) $oModule->{$key} = $val;
 		// Run
 		$oModule->procWAP($this);
 		// Output
@@ -617,7 +617,7 @@ 
 	 */
 	function displayContent()
 	{
-		Context::set('layout','none');
+		Context::set('layout', 'none');
 		// Compile a template
 		$oTemplate = new TemplateHandler();
 		$oContext = &Context::getInstance();

classes/cache/CacheFile.class.php

Doc Comments +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
 	/**

 	 * Return whether support or not support cache

 	 *

-	 * @return true

+	 * @return boolean

 	 */

 	function isSupport()

 	{

@@ -164,7 +164,7 @@ 
 	/**

 	 * Truncate all existing variables at the cache

 	 *

-	 * @return bool Returns true on success or false on failure.

+	 * @return boolean|null Returns true on success or false on failure.

 	 */

 	function truncate()

 	{

Spacing +10 added lines, -10 removed lines

@@ -23,7 +23,7 @@ 
 	 */

 	function getInstance()

 	{

-		if(!$GLOBALS['__CacheFile__'])

+		if (!$GLOBALS['__CacheFile__'])

 		{

 			$GLOBALS['__CacheFile__'] = new CacheFile();

 		}

@@ -37,7 +37,7 @@ 
 	 */

 	function CacheFile()

 	{

-		$this->cache_dir = _XE_PATH_ . $this->cache_dir;

+		$this->cache_dir = _XE_PATH_.$this->cache_dir;

 		FileHandler::makeDir($this->cache_dir);

 	}

 

@@ -49,7 +49,7 @@ 
 	 */

 	function getCacheFileName($key)

 	{

-		return $this->cache_dir . str_replace(':', DIRECTORY_SEPARATOR, $key) . '.php';

+		return $this->cache_dir.str_replace(':', DIRECTORY_SEPARATOR, $key).'.php';

 	}

 

 	/**

@@ -76,9 +76,9 @@ 
 		$content = array();

 		$content[] = '<?php';

 		$content[] = 'if(!defined(\'__XE__\')) { exit(); }';

-		$content[] = 'return \'' . addslashes(serialize($obj)) . '\';';

+		$content[] = 'return \''.addslashes(serialize($obj)).'\';';

 		FileHandler::writeFile($cache_file, implode(PHP_EOL, $content));

-		if(function_exists('opcache_invalidate'))

+		if (function_exists('opcache_invalidate'))

 		{

 			@opcache_invalidate($cache_file, true);

 		}

@@ -95,9 +95,9 @@ 
 	{

 		$cache_file = $this->getCacheFileName($key);

 

-		if(file_exists($cache_file))

+		if (file_exists($cache_file))

 		{

-			if($modified_time > 0 && filemtime($cache_file) < $modified_time)

+			if ($modified_time > 0 && filemtime($cache_file) < $modified_time)

 			{

 				FileHandler::removeFile($cache_file);

 				return false;

@@ -118,12 +118,12 @@ 
 	 */

 	function get($key, $modified_time = 0)

 	{

-		if(!$cache_file = FileHandler::exists($this->getCacheFileName($key)))

+		if (!$cache_file = FileHandler::exists($this->getCacheFileName($key)))

 		{

 			return false;

 		}

 

-		if($modified_time > 0 && filemtime($cache_file) < $modified_time)

+		if ($modified_time > 0 && filemtime($cache_file) < $modified_time)

 		{

 			FileHandler::removeFile($cache_file);

 			return false;

@@ -143,7 +143,7 @@ 
 	function _delete($_key)

 	{

 		$cache_file = $this->getCacheFileName($_key);

-		if(function_exists('opcache_invalidate'))

+		if (function_exists('opcache_invalidate'))

 		{

 			@opcache_invalidate($cache_file, true);

 		}

classes/db/DBMysql_innodb.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
 	 * opt : notnull, default, size\n
 	 * index : primary key, index, unique\n
 	 * @param string $xml_doc xml schema contents
-	 * @return void|object
+	 * @return null|false
 	 */
 	function _createTable($xml_doc)
 	{

Spacing +24 added lines, -24 removed lines

@@ -23,7 +23,7 @@ 
 	function DBMysql_innodb($auto_connect = TRUE)
 	{
 		$this->_setDBInfo();
-		if($auto_connect) $this->_connect();
+		if ($auto_connect) $this->_connect();
 	}
 
 	/**
@@ -55,13 +55,13 @@ 
 	{
 		$connection = $this->_getConnection('master');
 
-		if(!$transactionLevel)
+		if (!$transactionLevel)
 		{
 			$this->_query("START TRANSACTION", $connection);
 		}
 		else
 		{
-			$this->_query("SAVEPOINT SP" . $transactionLevel, $connection);
+			$this->_query("SAVEPOINT SP".$transactionLevel, $connection);
 		}
 		return true;
 	}
@@ -77,9 +77,9 @@ 
 
 		$point = $transactionLevel - 1;
 
-		if($point)
+		if ($point)
 		{
-			$this->_query("ROLLBACK TO SP" . $point, $connection);
+			$this->_query("ROLLBACK TO SP".$point, $connection);
 		}
 		else
 		{
@@ -109,14 +109,14 @@ 
 	 */
 	function __query($query, $connection)
 	{
-		if(!$connection)
+		if (!$connection)
 		{
 			exit('XE cannot handle DB connection.');
 		}
 		// Run the query statement
 		$result = @mysql_query($query, $connection);
 		// Error Check
-		if(mysql_error($connection))
+		if (mysql_error($connection))
 		{
 			$this->setError(mysql_errno($connection), mysql_error($connection));
 		}
@@ -140,13 +140,13 @@ 
 		$xml_obj = $oXml->parse($xml_doc);
 		// Create a table schema
 		$table_name = $xml_obj->table->attrs->name;
-		if($this->isTableExists($table_name))
+		if ($this->isTableExists($table_name))
 		{
 			return;
 		}
-		$table_name = $this->prefix . $table_name;
+		$table_name = $this->prefix.$table_name;
 
-		if(!is_array($xml_obj->table->column))
+		if (!is_array($xml_obj->table->column))
 		{
 			$columns[] = $xml_obj->table->column;
 		}
@@ -155,7 +155,7 @@ 
 			$columns = $xml_obj->table->column;
 		}
 
-		foreach($columns as $column)
+		foreach ($columns as $column)
 		{
 			$name = $column->attrs->name;
 			$type = $column->attrs->type;
@@ -167,47 +167,47 @@ 
 			$default = $column->attrs->default;
 			$auto_increment = $column->attrs->auto_increment;
 
-			$column_schema[] = sprintf('`%s` %s%s %s %s %s', $name, $this->column_type[$type], $size ? '(' . $size . ')' : '', isset($default) ? "default '" . $default . "'" : '', $notnull ? 'not null' : '', $auto_increment ? 'auto_increment' : '');
+			$column_schema[] = sprintf('`%s` %s%s %s %s %s', $name, $this->column_type[$type], $size ? '('.$size.')' : '', isset($default) ? "default '".$default."'" : '', $notnull ? 'not null' : '', $auto_increment ? 'auto_increment' : '');
 
-			if($primary_key)
+			if ($primary_key)
 			{
 				$primary_list[] = $name;
 			}
-			else if($unique)
+			else if ($unique)
 			{
 				$unique_list[$unique][] = $name;
 			}
-			else if($index)
+			else if ($index)
 			{
 				$index_list[$index][] = $name;
 			}
 		}
 
-		if(count($primary_list))
+		if (count($primary_list))
 		{
-			$column_schema[] = sprintf("primary key (%s)", '`' . implode($primary_list, '`,`') . '`');
+			$column_schema[] = sprintf("primary key (%s)", '`'.implode($primary_list, '`,`').'`');
 		}
 
-		if(count($unique_list))
+		if (count($unique_list))
 		{
-			foreach($unique_list as $key => $val)
+			foreach ($unique_list as $key => $val)
 			{
-				$column_schema[] = sprintf("unique %s (%s)", $key, '`' . implode($val, '`,`') . '`');
+				$column_schema[] = sprintf("unique %s (%s)", $key, '`'.implode($val, '`,`').'`');
 			}
 		}
 
-		if(count($index_list))
+		if (count($index_list))
 		{
-			foreach($index_list as $key => $val)
+			foreach ($index_list as $key => $val)
 			{
-				$column_schema[] = sprintf("index %s (%s)", $key, '`' . implode($val, '`,`') . '`');
+				$column_schema[] = sprintf("index %s (%s)", $key, '`'.implode($val, '`,`').'`');
 			}
 		}
 
 		$schema = sprintf('create table `%s` (%s%s) %s;', $this->addQuotes($table_name), "\n", implode($column_schema, ",\n"), "ENGINE = INNODB CHARACTER SET utf8 COLLATE utf8_general_ci");
 
 		$output = $this->_query($schema);
-		if(!$output)
+		if (!$output)
 		{
 			return false;
 		}

Braces +8 added lines, -11 removed lines

@@ -23,7 +23,9 @@ 
 	function DBMysql_innodb($auto_connect = TRUE)
 	{
 		$this->_setDBInfo();
-		if($auto_connect) $this->_connect();
+		if($auto_connect) {
+			$this->_connect();
+		}
 	}
 
 	/**
@@ -58,8 +60,7 @@ 
 		if(!$transactionLevel)
 		{
 			$this->_query("START TRANSACTION", $connection);
-		}
-		else
+		} else
 		{
 			$this->_query("SAVEPOINT SP" . $transactionLevel, $connection);
 		}
@@ -80,8 +81,7 @@ 
 		if($point)
 		{
 			$this->_query("ROLLBACK TO SP" . $point, $connection);
-		}
-		else
+		} else
 		{
 			$this->_query("ROLLBACK", $connection);
 		}
@@ -149,8 +149,7 @@ 
 		if(!is_array($xml_obj->table->column))
 		{
 			$columns[] = $xml_obj->table->column;
-		}
-		else
+		} else
 		{
 			$columns = $xml_obj->table->column;
 		}
@@ -172,12 +171,10 @@ 
 			if($primary_key)
 			{
 				$primary_list[] = $name;
-			}
-			else if($unique)
+			} else if($unique)
 			{
 				$unique_list[$unique][] = $name;
-			}
-			else if($index)
+			} else if($index)
 			{
 				$index_list[$index][] = $name;
 			}

classes/db/queryparts/condition/Condition.class.php

Doc Comments +3 added lines

@@ -97,6 +97,9 @@
 		return $this->pipe . ' ' . $this->getConditionPart($this->_value);

 	}

 

+	/**
+	 * @param string $pipe
+	 */
 	function setPipe($pipe)

 	{

 		$this->pipe = $pipe;

Braces +8 added lines, -10 removed lines

@@ -66,12 +66,10 @@ 
 			if(!$this->show())

 			{

 				$this->_value_to_string = '';

-			}

-			else if($withValue)

+			} else if($withValue)

 			{

 				$this->_value_to_string = $this->toStringWithValue();

-			}

-			else

+			} else

 			{

 				$this->_value_to_string = $this->toStringWithoutValue();

 			}

@@ -112,8 +110,7 @@ 
 			if(is_array($this->_value) && count($this->_value) === 1 && $this->_value[0] === '')

 			{

 				$this->_show = false;

-			}

-			else

+			} else

 			{

 				$this->_show = true;

 				switch($this->operation)

@@ -209,10 +206,11 @@ 
 			case 'like_prefix' :

 			case 'like' :

 				if(defined('__CUBRID_VERSION__')

-						&& __CUBRID_VERSION__ >= '8.4.1')

-					return $name . ' rlike ' . $value;

-				else

-					return $name . ' like ' . $value;

+						&& __CUBRID_VERSION__ >= '8.4.1') {
+									return $name . ' rlike ' . $value;
+				} else {
+									return $name . ' like ' . $value;
+				}
 				break;

 			case 'notlike_tail' :

 			case 'notlike_prefix' :

Spacing +33 added lines, -33 removed lines

@@ -61,13 +61,13 @@ 
 	 */

 	function toString($withValue = true)

 	{

-		if(!isset($this->_value_to_string))

+		if (!isset($this->_value_to_string))

 		{

-			if(!$this->show())

+			if (!$this->show())

 			{

 				$this->_value_to_string = '';

 			}

-			else if($withValue)

+			else if ($withValue)

 			{

 				$this->_value_to_string = $this->toStringWithValue();

 			}

@@ -85,7 +85,7 @@ 
 	 */

 	function toStringWithoutValue()

 	{

-		return $this->pipe . ' ' . $this->getConditionPart($this->_value);

+		return $this->pipe.' '.$this->getConditionPart($this->_value);

 	}

 

 	/**

@@ -94,7 +94,7 @@ 
 	 */

 	function toStringWithValue()

 	{

-		return $this->pipe . ' ' . $this->getConditionPart($this->_value);

+		return $this->pipe.' '.$this->getConditionPart($this->_value);

 	}

 

 	function setPipe($pipe)

@@ -107,16 +107,16 @@ 
 	 */

 	function show()

 	{

-		if(!isset($this->_show))

+		if (!isset($this->_show))

 		{

-			if(is_array($this->_value) && count($this->_value) === 1 && $this->_value[0] === '')

+			if (is_array($this->_value) && count($this->_value) === 1 && $this->_value[0] === '')

 			{

 				$this->_show = false;

 			}

 			else

 			{

 				$this->_show = true;

-				switch($this->operation)

+				switch ($this->operation)

 				{

 					case 'equal' :

 					case 'more' :

@@ -138,30 +138,30 @@ 
 					case 'not':

 					case 'notequal' :

 						// if variable is not set or is not string or number, return

-						if(!isset($this->_value))

+						if (!isset($this->_value))

 						{

 							$this->_show = false;

 							break;

 						}

-						if($this->_value === '')

+						if ($this->_value === '')

 						{

 							$this->_show = false;

 							break;

 						}

 						$tmpArray = array('string' => 1, 'integer' => 1);

-						if(!isset($tmpArray[gettype($this->_value)]))

+						if (!isset($tmpArray[gettype($this->_value)]))

 						{

 							$this->_show = false;

 							break;

 						}

 						break;

 					case 'between' :

-						if(!is_array($this->_value))

+						if (!is_array($this->_value))

 						{

 							$this->_show = false;

 							break;

 						}

-						if(count($this->_value) != 2)

+						if (count($this->_value) != 2)

 						{

 							$this->_show = false;

 							break;

@@ -188,67 +188,67 @@ 
 		$name = $this->column_name;

 		$operation = $this->operation;

 

-		switch($operation)

+		switch ($operation)

 		{

 			case 'equal' :

-				return $name . ' = ' . $value;

+				return $name.' = '.$value;

 				break;

 			case 'more' :

-				return $name . ' >= ' . $value;

+				return $name.' >= '.$value;

 				break;

 			case 'excess' :

-				return $name . ' > ' . $value;

+				return $name.' > '.$value;

 				break;

 			case 'less' :

-				return $name . ' <= ' . $value;

+				return $name.' <= '.$value;

 				break;

 			case 'below' :

-				return $name . ' < ' . $value;

+				return $name.' < '.$value;

 				break;

 			case 'like_tail' :

 			case 'like_prefix' :

 			case 'like' :

-				if(defined('__CUBRID_VERSION__')

+				if (defined('__CUBRID_VERSION__')

 						&& __CUBRID_VERSION__ >= '8.4.1')

-					return $name . ' rlike ' . $value;

+					return $name.' rlike '.$value;

 				else

-					return $name . ' like ' . $value;

+					return $name.' like '.$value;

 				break;

 			case 'notlike_tail' :

 			case 'notlike_prefix' :

 			case 'notlike' :

-				return $name . ' not like ' . $value;

+				return $name.' not like '.$value;

 				break;

 			case 'in' :

-				return $name . ' in ' . $value;

+				return $name.' in '.$value;

 				break;

 			case 'notin' :

 			case 'not_in' :

-				return $name . ' not in ' . $value;

+				return $name.' not in '.$value;

 				break;

 			case 'notequal' :

-				return $name . ' <> ' . $value;

+				return $name.' <> '.$value;

 				break;

 			case 'notnull' :

-				return $name . ' is not null';

+				return $name.' is not null';

 				break;

 			case 'null' :

-				return $name . ' is null';

+				return $name.' is null';

 				break;

 			case 'and' :

-				return $name . ' & ' . $value;

+				return $name.' & '.$value;

 				break;

 			case 'or' :

-				return $name . ' | ' . $value;

+				return $name.' | '.$value;

 				break;

 			case 'xor' :

-				return $name . ' ^ ' . $value;

+				return $name.' ^ '.$value;

 				break;

 			case 'not' :

-				return $name . ' ~ ' . $value;

+				return $name.' ~ '.$value;

 				break;

 			case 'between' :

-				return $name . ' between ' . $value[0] . ' and ' . $value[1];

+				return $name.' between '.$value[0].' and '.$value[1];

 				break;

 		}

 	}

classes/db/queryparts/condition/ConditionWithoutArgument.class.php

Doc Comments +2 added lines, -2 removed lines

@@ -12,10 +12,10 @@
 	/**

 	 * constructor

 	 * @param string $column_name

-	 * @param mixed $argument

+	 * @param integer $argument

 	 * @param string $operation

 	 * @param string $pipe

-	 * @return void

+	 * @return boolean

 	 */

 	function ConditionWithoutArgument($column_name, $argument, $operation, $pipe = "")

 	{

Braces +1 added lines, -2 removed lines

@@ -28,8 +28,7 @@
 				$argument = implode($argument, ',');

 			}

 			$this->_value = '(' . $argument . ')';

-		}

-		else

+		} else

 		{

 			$this->_value = $argument;

 		}

Spacing +3 added lines, -3 removed lines

@@ -21,13 +21,13 @@
 	{

 		parent::Condition($column_name, $argument, $operation, $pipe);

 		$tmpArray = array('in' => 1, 'notin' => 1, 'not_in' => 1);

-		if(isset($tmpArray[$operation]))

+		if (isset($tmpArray[$operation]))

 		{

-			if(is_array($argument))

+			if (is_array($argument))

 			{

 				$argument = implode($argument, ',');

 			}

-			$this->_value = '(' . $argument . ')';

+			$this->_value = '('.$argument.')';

 		}

 		else

 		{

classes/db/queryparts/expression/UpdateExpression.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 	/**

 	 * constructor

 	 * @param string $column_name

-	 * @param object $argument

+	 * @param Argument $argument

 	 * @return void

 	 */

 	function UpdateExpression($column_name, $argument)

Spacing +8 added lines, -8 removed lines

@@ -35,7 +35,7 @@ 
 	 */

 	function getExpression($with_value = true)

 	{

-		if($with_value)

+		if ($with_value)

 		{

 			return $this->getExpressionWithValue();

 		}

@@ -50,7 +50,7 @@ 
 	{

 		$value = $this->argument->getValue();

 		$operation = $this->argument->getColumnOperation();

-		if(isset($operation))

+		if (isset($operation))

 		{

 			return "$this->column_name = $this->column_name $operation $value";

 		}

@@ -65,7 +65,7 @@ 
 	function getExpressionWithoutValue()

 	{

 		$operation = $this->argument->getColumnOperation();

-		if(isset($operation))

+		if (isset($operation))

 		{

 			return "$this->column_name = $this->column_name $operation ?";

 		}

@@ -76,21 +76,21 @@ 
 	{

 		// TODO Escape value according to column type instead of variable type

 		$value = $this->argument->getValue();

-		if(!is_numeric($value))

+		if (!is_numeric($value))

 		{

-			return "'" . $value . "'";

+			return "'".$value."'";

 		}

 		return $value;

 	}

 

 	function show()

 	{

-		if(!$this->argument)

+		if (!$this->argument)

 		{

 			return false;

 		}

 		$value = $this->argument->getValue();

-		if(!isset($value))

+		if (!isset($value))

 		{

 			return false;

 		}

@@ -104,7 +104,7 @@ 
 

 	function getArguments()

 	{

-		if($this->argument)

+		if ($this->argument)

 		{

 			return array($this->argument);

 		}

Braces +1 added lines, -2 removed lines

@@ -62,8 +62,7 @@
 		if($this->argument)

 		{

 			return array($this->argument);

-		}

-		else

+		} else

 		{

 			return array();

 		}

classes/db/queryparts/Query.class.php

Doc Comments +3 added lines

@@ -251,6 +251,9 @@
 		return $this->orderby;
 	}
 
+	/**
+	 * @param integer $limit
+	 */
 	function setLimit($limit = NULL)
 	{
 		if(!isset($limit))

Braces +5 added lines, -5 removed lines

@@ -378,8 +378,7 @@ 
 				if($column->isSubquery())
 				{
 					$select[] = $column->toString($with_values) . ' as ' . $column->getAlias();
-				}
-				else
+				} else
 				{
 					$select[] = $column->getExpression($with_values);
 				}
@@ -403,7 +402,9 @@ 
 			}
 		}
 
-		if(!$update) return;
+		if(!$update) {
+			return;
+		}
 		return trim(implode($update, ', '));
 	}
 
@@ -465,8 +466,7 @@ 
 			if($table->isJoinTable() || !$simple_table_count)
 			{
 				$from .= $table->toString($with_values) . ' ';
-			}
-			else
+			} else
 			{
 				$from .= ', ' . $table->toString($with_values) . ' ';
 			}

Spacing +70 added lines, -70 removed lines

@@ -108,7 +108,7 @@ 
 		$this->action = $action;
 		$this->priority = $priority;
 
-		if(!isset($tables))
+		if (!isset($tables))
 		{
 			return;
 		}
@@ -144,12 +144,12 @@ 
 	function setColumnList($columnList)
 	{
 		$this->columnList = $columnList;
-		if(count($this->columnList) > 0)
+		if (count($this->columnList) > 0)
 		{
 			$selectColumns = array();
 			$dbParser = DB::getParser();
 
-			foreach($this->columnList as $columnName)
+			foreach ($this->columnList as $columnName)
 			{
 				$columnName = $dbParser->escapeColumn($columnName);
 				$selectColumns[] = new SelectExpression($columnName);
@@ -161,13 +161,13 @@ 
 
 	function setColumns($columns)
 	{
-		if(!isset($columns) || count($columns) === 0)
+		if (!isset($columns) || count($columns) === 0)
 		{
 			$this->columns = array(new StarExpression());
 			return;
 		}
 
-		if(!is_array($columns))
+		if (!is_array($columns))
 		{
 			$columns = array($columns);
 		}
@@ -177,14 +177,14 @@ 
 
 	function setTables($tables)
 	{
-		if(!isset($tables) || count($tables) === 0)
+		if (!isset($tables) || count($tables) === 0)
 		{
 			$this->setError(TRUE);
 			$this->setMessage("You must provide at least one table for the query.");
 			return;
 		}
 
-		if(!is_array($tables))
+		if (!is_array($tables))
 		{
 			$tables = array($tables);
 		}
@@ -200,18 +200,18 @@ 
 	function setConditions($conditions)
 	{
 		$this->conditions = array();
-		if(!isset($conditions) || count($conditions) === 0)
+		if (!isset($conditions) || count($conditions) === 0)
 		{
 			return;
 		}
-		if(!is_array($conditions))
+		if (!is_array($conditions))
 		{
 			$conditions = array($conditions);
 		}
 
-		foreach($conditions as $conditionGroup)
+		foreach ($conditions as $conditionGroup)
 		{
-			if($conditionGroup->show())
+			if ($conditionGroup->show())
 			{
 				$this->conditions[] = $conditionGroup;
 			}
@@ -220,11 +220,11 @@ 
 
 	function setGroups($groups)
 	{
-		if(!isset($groups) || count($groups) === 0)
+		if (!isset($groups) || count($groups) === 0)
 		{
 			return;
 		}
-		if(!is_array($groups))
+		if (!is_array($groups))
 		{
 			$groups = array($groups);
 		}
@@ -234,11 +234,11 @@ 
 
 	function setOrder($order)
 	{
-		if(!isset($order) || count($order) === 0)
+		if (!isset($order) || count($order) === 0)
 		{
 			return;
 		}
-		if(!is_array($order))
+		if (!is_array($order))
 		{
 			$order = array($order);
 		}
@@ -253,7 +253,7 @@ 
 
 	function setLimit($limit = NULL)
 	{
-		if(!isset($limit))
+		if (!isset($limit))
 		{
 			return;
 		}
@@ -354,9 +354,9 @@ 
 	function getClickCountColumns()
 	{
 		$click_count_columns = array();
-		foreach($this->columns as $column)
+		foreach ($this->columns as $column)
 		{
-			if($column->show() && is_a($column, 'ClickCountExpression'))
+			if ($column->show() && is_a($column, 'ClickCountExpression'))
 			{
 				$click_count_columns[] = $column;
 			}
@@ -371,13 +371,13 @@ 
 	 */
 	function getSelectString($with_values = TRUE)
 	{
-		foreach($this->columns as $column)
+		foreach ($this->columns as $column)
 		{
-			if($column->show())
+			if ($column->show())
 			{
-				if($column->isSubquery())
+				if ($column->isSubquery())
 				{
-					$select[] = $column->toString($with_values) . ' as ' . $column->getAlias();
+					$select[] = $column->toString($with_values).' as '.$column->getAlias();
 				}
 				else
 				{
@@ -395,15 +395,15 @@ 
 	 */
 	function getUpdateString($with_values = TRUE)
 	{
-		foreach($this->columns as $column)
+		foreach ($this->columns as $column)
 		{
-			if($column->show())
+			if ($column->show())
 			{
 				$update[] = $column->getExpression($with_values);
 			}
 		}
 
-		if(!$update) return;
+		if (!$update) return;
 		return trim(implode($update, ', '));
 	}
 
@@ -416,11 +416,11 @@ 
 	{
 		$columnsList = '';
 		// means we have insert-select
-		if($this->subquery)
+		if ($this->subquery)
 		{
-			foreach($this->columns as $column)
+			foreach ($this->columns as $column)
 			{
-				$columnsList .= $column->getColumnName() . ', ';
+				$columnsList .= $column->getColumnName().', ';
 			}
 			$columnsList = substr($columnsList, 0, -2);
 			$selectStatement = $this->subquery->toString($with_values);
@@ -429,12 +429,12 @@ 
 		}
 
 		$valuesList = '';
-		foreach($this->columns as $column)
+		foreach ($this->columns as $column)
 		{
-			if($column->show())
+			if ($column->show())
 			{
-				$columnsList .= $column->getColumnName() . ', ';
-				$valuesList .= $column->getValue($with_values) . ', ';
+				$columnsList .= $column->getColumnName().', ';
+				$valuesList .= $column->getValue($with_values).', ';
 			}
 		}
 		$columnsList = substr($columnsList, 0, -2);
@@ -460,25 +460,25 @@ 
 	{
 		$from = '';
 		$simple_table_count = 0;
-		foreach($this->tables as $table)
+		foreach ($this->tables as $table)
 		{
-			if($table->isJoinTable() || !$simple_table_count)
+			if ($table->isJoinTable() || !$simple_table_count)
 			{
-				$from .= $table->toString($with_values) . ' ';
+				$from .= $table->toString($with_values).' ';
 			}
 			else
 			{
-				$from .= ', ' . $table->toString($with_values) . ' ';
+				$from .= ', '.$table->toString($with_values).' ';
 			}
 
-			if(is_a($table, 'Subquery'))
+			if (is_a($table, 'Subquery'))
 			{
-				$from .= $table->getAlias() ? ' as ' . $table->getAlias() . ' ' : ' ';
+				$from .= $table->getAlias() ? ' as '.$table->getAlias().' ' : ' ';
 			}
 
 			$simple_table_count++;
 		}
-		if(trim($from) == '')
+		if (trim($from) == '')
 		{
 			return '';
 		}
@@ -496,9 +496,9 @@ 
 		$where = '';
 		$condition_count = 0;
 
-		foreach($this->conditions as $conditionGroup)
+		foreach ($this->conditions as $conditionGroup)
 		{
-			if($condition_count === 0)
+			if ($condition_count === 0)
 			{
 				$conditionGroup->setPipe("");
 			}
@@ -507,26 +507,26 @@ 
 			$condition_count++;
 		}
 
-		if($with_optimization &&
+		if ($with_optimization &&
 				(strstr($this->getOrderByString(), 'list_order') || strstr($this->getOrderByString(), 'update_order')))
 		{
 
-			if($condition_count !== 0)
+			if ($condition_count !== 0)
 			{
-				$where = '(' . $where . ') ';
+				$where = '('.$where.') ';
 			}
 
-			foreach($this->orderby as $order)
+			foreach ($this->orderby as $order)
 			{
 				$colName = $order->getColumnName();
-				if(strstr($colName, 'list_order') || strstr($colName, 'update_order'))
+				if (strstr($colName, 'list_order') || strstr($colName, 'update_order'))
 				{
 					$opt_condition = new ConditionWithoutArgument($colName, 2100000000, 'less', 'and');
-					if($condition_count === 0)
+					if ($condition_count === 0)
 					{
 						$opt_condition->setPipe("");
 					}
-					$where .= $opt_condition->toString($with_values) . ' ';
+					$where .= $opt_condition->toString($with_values).' ';
 					$condition_count++;
 				}
 			}
@@ -542,9 +542,9 @@ 
 	function getGroupByString()
 	{
 		$groupBy = '';
-		if($this->groups)
+		if ($this->groups)
 		{
-			if($this->groups[0] !== "")
+			if ($this->groups[0] !== "")
 			{
 				$groupBy = implode(', ', $this->groups);
 			}
@@ -558,16 +558,16 @@ 
 	 */
 	function getOrderByString()
 	{
-		if(!$this->_orderByString)
+		if (!$this->_orderByString)
 		{
-			if(count($this->orderby) === 0)
+			if (count($this->orderby) === 0)
 			{
 				return '';
 			}
 			$orderBy = '';
-			foreach($this->orderby as $order)
+			foreach ($this->orderby as $order)
 			{
-				$orderBy .= $order->toString() . ', ';
+				$orderBy .= $order->toString().', ';
 			}
 			$orderBy = substr($orderBy, 0, -2);
 			$this->_orderByString = $orderBy;
@@ -587,7 +587,7 @@ 
 	function getLimitString()
 	{
 		$limit = '';
-		if(count($this->limit) > 0)
+		if (count($this->limit) > 0)
 		{
 			$limit = '';
 			$limit .= $this->limit->toString();
@@ -606,19 +606,19 @@ 
 	 */
 	function getArguments()
 	{
-		if(!isset($this->arguments))
+		if (!isset($this->arguments))
 		{
 			$this->arguments = array();
 
 			// Join table arguments
-			if(count($this->tables) > 0)
+			if (count($this->tables) > 0)
 			{
-				foreach($this->tables as $table)
+				foreach ($this->tables as $table)
 				{
-					if($table->isJoinTable() || is_a($table, 'Subquery'))
+					if ($table->isJoinTable() || is_a($table, 'Subquery'))
 					{
 						$args = $table->getArguments();
-						if($args)
+						if ($args)
 						{
 							$this->arguments = array_merge($this->arguments, $args);
 						}
@@ -628,14 +628,14 @@ 
 
 			// Column arguments
 			// The if is for delete statements, all others must have columns
-			if(count($this->columns) > 0)
+			if (count($this->columns) > 0)
 			{
-				foreach($this->columns as $column)
+				foreach ($this->columns as $column)
 				{
-					if($column->show())
+					if ($column->show())
 					{
 						$args = $column->getArguments();
-						if($args)
+						if ($args)
 						{
 							$this->arguments = array_merge($this->arguments, $args);
 						}
@@ -644,12 +644,12 @@ 
 			}
 
 			// Condition arguments
-			if(count($this->conditions) > 0)
+			if (count($this->conditions) > 0)
 			{
-				foreach($this->conditions as $conditionGroup)
+				foreach ($this->conditions as $conditionGroup)
 				{
 					$args = $conditionGroup->getArguments();
-					if(count($args) > 0)
+					if (count($args) > 0)
 					{
 						$this->arguments = array_merge($this->arguments, $args);
 					}
@@ -657,12 +657,12 @@ 
 			}
 
 			// Navigation arguments
-			if(count($this->orderby) > 0)
+			if (count($this->orderby) > 0)
 			{
-				foreach($this->orderby as $order)
+				foreach ($this->orderby as $order)
 				{
 					$args = $order->getArguments();
-					if(count($args) > 0)
+					if (count($args) > 0)
 					{
 						$this->arguments = array_merge($this->arguments, $args);
 					}

classes/display/DisplayHandler.class.php

Doc Comments +2 added lines, -1 removed lines

@@ -133,7 +133,7 @@ 
 	 * Print debugging message to designated output source depending on the value set to __DEBUG_OUTPUT_. \n
 	 * This method only functions when __DEBUG__ variable is set to 1.
 	 * __DEBUG_OUTPUT__ == 0, messages are written in ./files/_debug_message.php
-	 * @return void
+	 * @return null|string
 	 */
 	function _debugOutput()
 	{
@@ -358,6 +358,7 @@ 
 
 	/**
 	 * print a HTTP HEADER for HTML, which is encoded in UTF-8
+	 * @param integer $code
 	 * @return void
 	 */
 	function _printHttpStatusCode($code)

Spacing +43 added lines, -43 removed lines

@@ -25,7 +25,7 @@ 
 	function printContent(&$oModule)
 	{
 		// Check if the gzip encoding supported
-		if(
+		if (
 				(defined('__OB_GZHANDLER_ENABLE__') && __OB_GZHANDLER_ENABLE__ == 1) &&
 				strpos($_SERVER['HTTP_ACCEPT_ENCODING'], 'gzip') !== FALSE &&
 				extension_loaded('zlib') &&
@@ -36,23 +36,23 @@ 
 		}
 
 		// Extract contents to display by the request method
-		if(Context::get('xeVirtualRequestMethod') == 'xml')
+		if (Context::get('xeVirtualRequestMethod') == 'xml')
 		{
 			$handler = new VirtualXMLDisplayHandler();
 		}
-		else if(Context::getRequestMethod() == 'XMLRPC')
+		else if (Context::getRequestMethod() == 'XMLRPC')
 		{
 			$handler = new XMLDisplayHandler();
-			if(strpos($_SERVER['HTTP_USER_AGENT'], 'MSIE') !== FALSE)
+			if (strpos($_SERVER['HTTP_USER_AGENT'], 'MSIE') !== FALSE)
 			{
 				$this->gz_enabled = FALSE;
 			}
 		}
-		else if(Context::getRequestMethod() == 'JSON')
+		else if (Context::getRequestMethod() == 'JSON')
 		{
 			$handler = new JSONDisplayHandler();
 		}
-		else if(Context::getRequestMethod() == 'JS_CALLBACK')
+		else if (Context::getRequestMethod() == 'JS_CALLBACK')
 		{
 			$handler = new JSCallbackDisplayHandler();
 		}
@@ -70,9 +70,9 @@ 
 		$called_position = 'before_display_content';
 		$oAddonController = getController('addon');
 		$addon_file = $oAddonController->getCacheFilePath(Mobile::isFromMobilePhone() ? "mobile" : "pc");
-		if(file_exists($addon_file)) include($addon_file);
+		if (file_exists($addon_file)) include($addon_file);
 
-		if(method_exists($handler, "prepareToPrint"))
+		if (method_exists($handler, "prepareToPrint"))
 		{
 			$handler->prepareToPrint($output);
 		}
@@ -80,17 +80,17 @@ 
 		// header output
 
 		$httpStatusCode = $oModule->getHttpStatusCode();
-		if($httpStatusCode && $httpStatusCode != 200)
+		if ($httpStatusCode && $httpStatusCode != 200)
 		{
 			$this->_printHttpStatusCode($httpStatusCode);
 		}
 		else
 		{
-			if(Context::getResponseMethod() == 'JSON' || Context::getResponseMethod() == 'JS_CALLBACK')
+			if (Context::getResponseMethod() == 'JSON' || Context::getResponseMethod() == 'JS_CALLBACK')
 			{
 				$this->_printJSONHeader();
 			}
-			else if(Context::getResponseMethod() != 'HTML')
+			else if (Context::getResponseMethod() != 'HTML')
 			{
 				$this->_printXMLHeader();
 			}
@@ -106,13 +106,13 @@ 
 
 		// disable gzip if output already exists
 		ob_flush();
-		if(headers_sent())
+		if (headers_sent())
 		{
 			$this->gz_enabled = FALSE;
 		}
 
 		// enable gzip using zlib extension
-		if($this->gz_enabled)
+		if ($this->gz_enabled)
 		{
 			ini_set('zlib.output_compression', true);
 		}
@@ -133,7 +133,7 @@ 
 	 */
 	function _debugOutput()
 	{
-		if(!__DEBUG__)
+		if (!__DEBUG__)
 		{
 			return;
 		}
@@ -141,32 +141,32 @@ 
 		$end = getMicroTime();
 
 		// Firebug console output
-		if(__DEBUG_OUTPUT__ == 2 && version_compare(PHP_VERSION, '6.0.0') === -1)
+		if (__DEBUG_OUTPUT__ == 2 && version_compare(PHP_VERSION, '6.0.0') === -1)
 		{
 			static $firephp;
-			if(!isset($firephp))
+			if (!isset($firephp))
 			{
 				$firephp = FirePHP::getInstance(true);
 			}
 
-			if(__DEBUG_PROTECT__ == 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
+			if (__DEBUG_PROTECT__ == 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
 			{
 				$firephp->fb('Change the value of __DEBUG_PROTECT_IP__ into your IP address in config/config.user.inc.php or config/config.inc.php', 'The IP address is not allowed.');
 				return;
 			}
 			// display total execution time and Request/Response info
-			if(__DEBUG__ & 2)
+			if (__DEBUG__ & 2)
 			{
 				$firephp->fb(
 						array(
-							'Request / Response info >>> ' . $_SERVER['REQUEST_METHOD'] . ' / ' . Context::getResponseMethod(),
+							'Request / Response info >>> '.$_SERVER['REQUEST_METHOD'].' / '.Context::getResponseMethod(),
 							array(
 								array('Request URI', 'Request method', 'Response method', 'Response contents size', 'Memory peak usage'),
 								array(
 									sprintf("%s:%s%s%s%s", $_SERVER['SERVER_NAME'], $_SERVER['SERVER_PORT'], $_SERVER['PHP_SELF'], $_SERVER['QUERY_STRING'] ? '?' : '', $_SERVER['QUERY_STRING']),
 									$_SERVER['REQUEST_METHOD'],
 									Context::getResponseMethod(),
-									$this->content_size . ' byte',
+									$this->content_size.' byte',
 									FileHandler::filesize(memory_get_peak_usage())
 								)
 							)
@@ -175,7 +175,7 @@ 
 				);
 				$firephp->fb(
 						array(
-							'Elapsed time >>> Total : ' . sprintf('%0.5f sec', $end - __StartTime__),
+							'Elapsed time >>> Total : '.sprintf('%0.5f sec', $end - __StartTime__),
 							array(array('DB queries', 'class file load', 'Template compile', 'XmlParse compile', 'PHP', 'Widgets', 'Trans Content'),
 								array(
 									sprintf('%0.5f sec', $GLOBALS['__db_elapsed_time__']),
@@ -193,16 +193,16 @@ 
 			}
 
 			// display DB query history
-			if((__DEBUG__ & 4) && $GLOBALS['__db_queries__'])
+			if ((__DEBUG__ & 4) && $GLOBALS['__db_queries__'])
 			{
 				$queries_output = array(array('Result/'.PHP_EOL.'Elapsed time', 'Query ID', 'Query'));
-				foreach($GLOBALS['__db_queries__'] as $query)
+				foreach ($GLOBALS['__db_queries__'] as $query)
 				{
-					$queries_output[] = array($query['result'] . PHP_EOL . sprintf('%0.5f', $query['elapsed_time']), str_replace(_XE_PATH_, '', $query['called_file']) . PHP_EOL . $query['called_method'] . '()' . PHP_EOL . $query['query_id'], $query['query']);
+					$queries_output[] = array($query['result'].PHP_EOL.sprintf('%0.5f', $query['elapsed_time']), str_replace(_XE_PATH_, '', $query['called_file']).PHP_EOL.$query['called_method'].'()'.PHP_EOL.$query['query_id'], $query['query']);
 				}
 				$firephp->fb(
 						array(
-							'DB Queries >>> ' . count($GLOBALS['__db_queries__']) . ' Queries, ' . sprintf('%0.5f sec', $GLOBALS['__db_elapsed_time__']),
+							'DB Queries >>> '.count($GLOBALS['__db_queries__']).' Queries, '.sprintf('%0.5f sec', $GLOBALS['__db_elapsed_time__']),
 							$queries_output
 						),
 						'TABLE'
@@ -215,9 +215,9 @@ 
 
 			$buff = array();
 			// display total execution time and Request/Response info
-			if(__DEBUG__ & 2)
+			if (__DEBUG__ & 2)
 			{
-				if(__DEBUG_PROTECT__ == 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
+				if (__DEBUG_PROTECT__ == 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
 				{
 					return;
 				}
@@ -248,21 +248,21 @@ 
 				$buff[] = sprintf("\tTrans Content \t\t\t\t\t: %0.5f sec", $GLOBALS['__trans_content_elapsed__']);
 			}
 			// DB Logging
-			if(__DEBUG__ & 4)
+			if (__DEBUG__ & 4)
 			{
-				if(__DEBUG_PROTECT__ == 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
+				if (__DEBUG_PROTECT__ == 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
 				{
 					return;
 				}
 
-				if($GLOBALS['__db_queries__'])
+				if ($GLOBALS['__db_queries__'])
 				{
 					$buff[] = sprintf("\n- DB Queries : %d Queries. %0.5f sec", count($GLOBALS['__db_queries__']), $GLOBALS['__db_elapsed_time__']);
 					$num = 0;
 
-					foreach($GLOBALS['__db_queries__'] as $query)
+					foreach ($GLOBALS['__db_queries__'] as $query)
 					{
-						if($query['result'] == 'Success')
+						if ($query['result'] == 'Success')
 						{
 							$query_result = "Query Success";
 						}
@@ -279,30 +279,30 @@ 
 			}
 
 			// Output in HTML comments
-			if($buff && __DEBUG_OUTPUT__ == 1 && Context::getResponseMethod() == 'HTML')
+			if ($buff && __DEBUG_OUTPUT__ == 1 && Context::getResponseMethod() == 'HTML')
 			{
 				$buff = implode("\r\n", $buff);
 				$buff = sprintf("[%s %s:%d]\r\n%s", date('Y-m-d H:i:s'), $file_name, $line_num, print_r($buff, true));
 
-				if(__DEBUG_PROTECT__ == 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
+				if (__DEBUG_PROTECT__ == 1 && __DEBUG_PROTECT_IP__ != $_SERVER['REMOTE_ADDR'])
 				{
 					$buff = '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';
 				}
 
-				return "<!--\r\n" . $buff . "\r\n-->";
+				return "<!--\r\n".$buff."\r\n-->";
 			}
 
 			// Output to a file
-			if($buff && __DEBUG_OUTPUT__ == 0)
+			if ($buff && __DEBUG_OUTPUT__ == 0)
 			{
-				$debug_file = _XE_PATH_ . 'files/_debug_message.php';
+				$debug_file = _XE_PATH_.'files/_debug_message.php';
 				$buff = implode(PHP_EOL, $buff);
 				$buff = sprintf("[%s]\n%s", date('Y-m-d H:i:s'), print_r($buff, true));
 
-				$buff = str_repeat('=', 80) . "\n" . $buff . "\n" . str_repeat('-', 80);
-				$buff = "\n<?php\n/*" . $buff . "*/\n?>\n";
+				$buff = str_repeat('=', 80)."\n".$buff."\n".str_repeat('-', 80);
+				$buff = "\n<?php\n/*".$buff."*/\n?>\n";
 
-				if (!@file_put_contents($debug_file, $buff, FILE_APPEND|LOCK_EX))
+				if (!@file_put_contents($debug_file, $buff, FILE_APPEND | LOCK_EX))
 				{
 					return;
 				}
@@ -318,7 +318,7 @@ 
 	{
 		header("Content-Type: text/xml; charset=UTF-8");
 		header("Expires: Mon, 26 Jul 1997 05:00:00 GMT");
-		header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
+		header("Last-Modified: ".gmdate("D, d M Y H:i:s")." GMT");
 		header("Cache-Control: no-store, no-cache, must-revalidate");
 		header("Cache-Control: post-check=0, pre-check=0", false);
 		header("Pragma: no-cache");
@@ -332,7 +332,7 @@ 
 	{
 		header("Content-Type: text/html; charset=UTF-8");
 		header("Expires: Mon, 26 Jul 1997 05:00:00 GMT");
-		header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
+		header("Last-Modified: ".gmdate("D, d M Y H:i:s")." GMT");
 		header("Cache-Control: no-store, no-cache, must-revalidate");
 		header("Cache-Control: post-check=0, pre-check=0", false);
 		header("Pragma: no-cache");
@@ -346,7 +346,7 @@ 
 	{
 		header("Content-Type: text/html; charset=UTF-8");
 		header("Expires: Mon, 26 Jul 1997 05:00:00 GMT");
-		header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
+		header("Last-Modified: ".gmdate("D, d M Y H:i:s")." GMT");
 		header("Cache-Control: no-store, no-cache, must-revalidate");
 		header("Cache-Control: post-check=0, pre-check=0", false);
 		header("Pragma: no-cache");

Braces +12 added lines, -19 removed lines

@@ -39,24 +39,20 @@ 
 		if(Context::get('xeVirtualRequestMethod') == 'xml')
 		{
 			$handler = new VirtualXMLDisplayHandler();
-		}
-		else if(Context::getRequestMethod() == 'XMLRPC')
+		} else if(Context::getRequestMethod() == 'XMLRPC')
 		{
 			$handler = new XMLDisplayHandler();
 			if(strpos($_SERVER['HTTP_USER_AGENT'], 'MSIE') !== FALSE)
 			{
 				$this->gz_enabled = FALSE;
 			}
-		}
-		else if(Context::getRequestMethod() == 'JSON')
+		} else if(Context::getRequestMethod() == 'JSON')
 		{
 			$handler = new JSONDisplayHandler();
-		}
-		else if(Context::getRequestMethod() == 'JS_CALLBACK')
+		} else if(Context::getRequestMethod() == 'JS_CALLBACK')
 		{
 			$handler = new JSCallbackDisplayHandler();
-		}
-		else
+		} else
 		{
 			$handler = new HTMLDisplayHandler();
 		}
@@ -70,7 +66,9 @@ 
 		$called_position = 'before_display_content';
 		$oAddonController = getController('addon');
 		$addon_file = $oAddonController->getCacheFilePath(Mobile::isFromMobilePhone() ? "mobile" : "pc");
-		if(file_exists($addon_file)) include($addon_file);
+		if(file_exists($addon_file)) {
+			include($addon_file);
+		}
 
 		if(method_exists($handler, "prepareToPrint"))
 		{
@@ -83,18 +81,15 @@ 
 		if($httpStatusCode && $httpStatusCode != 200)
 		{
 			$this->_printHttpStatusCode($httpStatusCode);
-		}
-		else
+		} else
 		{
 			if(Context::getResponseMethod() == 'JSON' || Context::getResponseMethod() == 'JS_CALLBACK')
 			{
 				$this->_printJSONHeader();
-			}
-			else if(Context::getResponseMethod() != 'HTML')
+			} else if(Context::getResponseMethod() != 'HTML')
 			{
 				$this->_printXMLHeader();
-			}
-			else
+			} else
 			{
 				$this->_printHTMLHeader();
 			}
@@ -209,8 +204,7 @@ 
 				);
 			}
 			// dislpay the file and HTML comments
-		}
-		else
+		} else
 		{
 
 			$buff = array();
@@ -265,8 +259,7 @@ 
 						if($query['result'] == 'Success')
 						{
 							$query_result = "Query Success";
-						}
-						else
+						} else
 						{
 							$query_result = sprintf("Query $s : %d\n\t\t\t   %s", $query['result'], $query['errno'], $query['errstr']);
 						}