xpressengine / xe-core

modules/counter/counter.admin.view.php

Spacing +4 added lines, -4 removed lines

@@ -17,7 +17,7 @@ 
 	function init()
 	{
 		// set the template path
-		$this->setTemplatePath($this->module_path . 'tpl');
+		$this->setTemplatePath($this->module_path.'tpl');
 	}
 
 	/**
@@ -28,9 +28,9 @@ 
 	function dispCounterAdminIndex()
 	{
 		// set today's if no date is given
-		$selected_date = (int)Context::get('selected_date');
+		$selected_date = (int) Context::get('selected_date');
 
-		if(!$selected_date)
+		if (!$selected_date)
 		{
 			$selected_date = date("Ymd");
 		}
@@ -50,7 +50,7 @@ 
 		// get data by time, day, month, and year
 		$type = Context::get('type');
 
-		if(!$type)
+		if (!$type)
 		{
 			$type = 'day';
 			Context::set('type', $type);

addons/captcha_member/captcha_member.addon.php

Doc Comments +3 added lines

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

Spacing +44 added lines, -44 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 /* Copyright (C) XEHub <https://www.xehub.io> */
 
-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('AddonMemberCaptcha', false))
+if (!class_exists('AddonMemberCaptcha', false))
 {
 	class AddonMemberCaptcha
 	{
@@ -32,7 +32,7 @@ 
 		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;
 			}
@@ -40,11 +40,11 @@ 
 			// {
 			// 	return false;
 			// }
-			if($_SESSION['XE_VALIDATOR_ERROR'] == -1)
+			if ($_SESSION['XE_VALIDATOR_ERROR'] == -1)
 			{
 				$_SESSION['member_captcha_authed'] = false;
 			}
-			if($_SESSION['member_captcha_authed'])
+			if ($_SESSION['member_captcha_authed'])
 			{
 				return false;
 			}
@@ -52,26 +52,26 @@ 
 			$type = Context::get('captchaType');
 
 			$this->target_acts = array();
-			if($this->addon_info->apply_find_account == 'apply')
+			if ($this->addon_info->apply_find_account == 'apply')
 			{
 				$this->target_acts[] = 'procMemberFindAccount';
 			}
-			if($this->addon_info->apply_resend_auth_mail == 'apply')
+			if ($this->addon_info->apply_resend_auth_mail == 'apply')
 			{
 				$this->target_acts[] = 'procMemberResendAuthMail';
 			}
-			if($this->addon_info->apply_signup == 'apply')
+			if ($this->addon_info->apply_signup == 'apply')
 			{
 				$this->target_acts[] = 'procMemberInsert';
 			}
 
-			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_member/lang');
+						Context::loadLang(_XE_PATH_.'addons/captcha_member/lang');
 						$_SESSION['XE_VALIDATOR_ERROR'] = -1;
 						$_SESSION['XE_VALIDATOR_MESSAGE'] = Context::getLang('captcha_denied');
 						$_SESSION['XE_VALIDATOR_MESSAGE_TYPE'] = 'error';
@@ -83,16 +83,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['member_captcha_authed'] && in_array(Context::get('act'), $this->target_acts))
+			if (!$_SESSION['member_captcha_authed'] && in_array(Context::get('act'), $this->target_acts))
 			{
-				Context::loadLang(_XE_PATH_ . 'addons/captcha_member/lang');
+				Context::loadLang(_XE_PATH_.'addons/captcha_member/lang');
 				$ModuleHandler->error = "captcha_denied";
 			}
 
@@ -102,7 +102,7 @@ 
 		function createKeyword()
 		{
 			$type = Context::get('captchaType');
-			if($type == 'inline' && $_SESSION['captcha_keyword'])
+			if ($type == 'inline' && $_SESSION['captcha_keyword'])
 			{
 				return;
 			}
@@ -115,19 +115,19 @@ 
 
 		function before_module_init_setCaptchaSession()
 		{
-			if($_SESSION['member_captcha_authed'])
+			if ($_SESSION['member_captcha_authed'])
 			{
 				return false;
 			}
 			// Load language files
-			Context::loadLang(_XE_PATH_ . 'addons/captcha_member/lang');
+			Context::loadLang(_XE_PATH_.'addons/captcha_member/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");
@@ -144,11 +144,11 @@ 
 
 		function before_module_init_captchaImage()
 		{
-			if($_SESSION['member_captcha_authed'])
+			if ($_SESSION['member_captcha_authed'])
 			{
 				return false;
 			}
-			if(Context::get('renew'))
+			if (Context::get('renew'))
 			{
 				$this->createKeyword();
 			}
@@ -170,7 +170,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};
 			}
@@ -192,7 +192,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);
@@ -202,7 +202,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);
@@ -217,7 +217,7 @@ 
 			}
 
 			// Combine images of each character
-			for($i = 1; $i < count($im); $i++)
+			for ($i = 1; $i < count($im); $i++)
 			{
 				imagecopy($im[0], $im[$i], (($w + 2) * ($i - 1)), 0, 0, 0, $w, $h);
 				imagedestroy($im[$i]);
@@ -229,7 +229,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);
 			}
@@ -241,7 +241,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);
 			}
@@ -249,7 +249,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);
 			}
@@ -259,7 +259,7 @@ 
 
 		function before_module_init_captchaAudio()
 		{
-			if($_SESSION['member_captcha_authed'])
+			if ($_SESSION['member_captcha_authed'])
 			{
 				return false;
 			}
@@ -271,8 +271,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();
@@ -283,17 +283,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;
 					}
@@ -308,14 +308,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['member_captcha_authed'])
+			if ($_SESSION['member_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['member_captcha_authed'] = true;
 				return true;
@@ -328,14 +328,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");
@@ -350,7 +350,7 @@ 
 			unset($_SESSION['member_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'));
 
@@ -388,18 +388,18 @@ 
 
 $oAddonMemberCaptcha = &$GLOBALS['__AddonMemberCaptcha__'];
 
-if(method_exists($oAddonMemberCaptcha, $called_position))
+if (method_exists($oAddonMemberCaptcha, $called_position))
 {
-	if(!call_user_func_array(array(&$oAddonMemberCaptcha, $called_position), array(&$this)))
+	if (!call_user_func_array(array(&$oAddonMemberCaptcha, $called_position), array(&$this)))
 	{
 		return false;
 	}
 }
 
 $addon_act = Context::get('captcha_action');
-if($addon_act && method_exists($oAddonMemberCaptcha, $called_position . '_' . $addon_act))
+if ($addon_act && method_exists($oAddonMemberCaptcha, $called_position.'_'.$addon_act))
 {
-	if(!call_user_func_array(array(&$oAddonMemberCaptcha, $called_position . '_' . $addon_act), array(&$this)))
+	if (!call_user_func_array(array(&$oAddonMemberCaptcha, $called_position.'_'.$addon_act), array(&$this)))
 	{
 		return false;
 	}

Braces +8 added lines, -6 removed lines

@@ -1,7 +1,9 @@ 
 <?php
 /* Copyright (C) XEHub <https://www.xehub.io> */
 
-if(!defined("__XE__")) exit();
+if(!defined("__XE__")) {
+	exit();
+}
 
 /**
  * @file captcha.addon.php
@@ -78,8 +80,7 @@ 
 						$_SESSION['XE_VALIDATOR_RETURN_URL'] = Context::get('error_return_url');
 						$ModuleHandler->_setInputValueToSession();
 					}
-				}
-				else
+				} else
 				{
 					Context::addHtmlHeader('<script>
 						if(!captchaTargetAct) {var captchaTargetAct = [];}
@@ -209,8 +210,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);
 				}
@@ -308,7 +308,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['member_captcha_authed'])
 			{

modules/comment/comment.admin.view.php

Spacing +9 added lines, -9 removed lines

@@ -52,7 +52,7 @@ 
 		// get a list by using comment->getCommentList. 
 		$oCommentModel = getModel('comment');
 		$secretNameList = $oCommentModel->getSecretNameList();
-		$columnList = array('comment_srl', 'document_srl','module_srl','is_secret', 'status', 'content', 'comments.member_srl', 'comments.nick_name', 'comments.regdate', 'ipaddress', 'voted_count', 'blamed_count');
+		$columnList = array('comment_srl', 'document_srl', 'module_srl', 'is_secret', 'status', 'content', 'comments.member_srl', 'comments.nick_name', 'comments.regdate', 'ipaddress', 'voted_count', 'blamed_count');
 		$output = $oCommentModel->getTotalCommentList($args, $columnList);
 
 		// $modules = $oCommentModel->getDistinctModules();
@@ -70,20 +70,20 @@ 
 		$oModuleModel = getModel('module');
 		$module_list = array();
 		$mod_srls = array();
-		foreach($output->data as $val)
+		foreach ($output->data as $val)
 		{
 			$mod_srls[] = $val->module_srl;
 		}
 		$mod_srls = array_unique($mod_srls);
 		// Module List
 		$mod_srls_count = count($mod_srls);
-		if($mod_srls_count)
+		if ($mod_srls_count)
 		{
 			$columnList = array('module_srl', 'mid', 'browser_title');
 			$module_output = $oModuleModel->getModulesInfo($mod_srls, $columnList);
-			if($module_output && is_array($module_output))
+			if ($module_output && is_array($module_output))
 			{
-				foreach($module_output as $module)
+				foreach ($module_output as $module)
 				{
 					$module_list[$module->module_srl] = $module;
 				}
@@ -95,7 +95,7 @@ 
 		$security->encodeHTML('search_target', 'search_keyword');
 
 		// set the template 
-		$this->setTemplatePath($this->module_path . 'tpl');
+		$this->setTemplatePath($this->module_path.'tpl');
 		$this->setTemplateFile('comment_list');
 	}
 
@@ -117,11 +117,11 @@ 
 		$declared_output = executeQuery('comment.getDeclaredList', $args);
 		$oCommentModel = getModel('comment');
 
-		if($declared_output->data && count($declared_output->data))
+		if ($declared_output->data && count($declared_output->data))
 		{
 			$comment_list = array();
 
-			foreach($declared_output->data as $key => $comment)
+			foreach ($declared_output->data as $key => $comment)
 			{
 				$comment_list[$key] = new commentItem();
 				$comment_list[$key]->setAttribute($comment);
@@ -139,7 +139,7 @@ 
 		Context::set('page_navigation', $declared_output->page_navigation);
 		Context::set('secret_name_list', $secretNameList);
 		// set the template
-		$this->setTemplatePath($this->module_path . 'tpl');
+		$this->setTemplatePath($this->module_path.'tpl');
 		$this->setTemplateFile('declared_list');
 	}
 

libs/PEAR.1.9/HTTP/Request.php

Spacing +4 added lines, -4 removed lines

@@ -18,28 +18,28 @@
 	}
 
 	public function getResponseCode() {
-		if($this->response)
+		if ($this->response)
 		{
 			return $this->response->getStatus();
 		}
 	}
 
 	public function getResponseHeader() {
-		if($this->response)
+		if ($this->response)
 		{
 			return $this->response->getHeader();	
 		}
 	}
 
 	public function getResponseBody() {
-		if($this->response)
+		if ($this->response)
 		{
 			return $this->response->getBody();
 		}
 	}
 
 	public function getResponseCookies() {
-		if($this->response)
+		if ($this->response)
 		{
 			return $this->response->getCookies();
 		}

modules/session/session.model.php

Spacing +17 added lines, -17 removed lines

@@ -21,14 +21,14 @@ 
 
 	function read($session_key)
 	{
-		if(!$session_key || !$this->session_started) return;
+		if (!$session_key || !$this->session_started) return;
 
 		$args = new stdClass();
 		$args->session_key = $session_key;
 		$columnList = array('session_key', 'cur_mid', 'val');
 		$output = executeQuery('session.getSession', $args, $columnList);
 
-		if(!$output->data)
+		if (!$output->data)
 		{
 			return '';
 		}
@@ -46,47 +46,47 @@ 
 	 */
 	function getLoggedMembers($args)
 	{
-		if(!$args->site_srl)
+		if (!$args->site_srl)
 		{
 			$site_module_info = Context::get('site_module_info');
-			$args->site_srl = (int)$site_module_info->site_srl;
+			$args->site_srl = (int) $site_module_info->site_srl;
 		}
-		if(!$args->list_count) $args->list_count = 20;
-		if(!$args->page) $args->page = 1;
-		if(!$args->period_time) $args->period_time = 3;
-		$args->last_update = date("YmdHis", $_SERVER['REQUEST_TIME'] - $args->period_time*60);
+		if (!$args->list_count) $args->list_count = 20;
+		if (!$args->page) $args->page = 1;
+		if (!$args->period_time) $args->period_time = 3;
+		$args->last_update = date("YmdHis", $_SERVER['REQUEST_TIME'] - $args->period_time * 60);
 
 		$output = executeQueryArray('session.getLoggedMembers', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$member_srls = array();
 		$member_keys = array();
-		if(count($output->data))
+		if (count($output->data))
 		{
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
 				$member_srls[$key] = $val->member_srl;
 				$member_keys[$val->member_srl] = $key;
 			}
 		}
 
-		if(Context::get('is_logged'))
+		if (Context::get('is_logged'))
 		{
 			$logged_info = Context::get('logged_info');
-			if(!in_array($logged_info->member_srl, $member_srls))
+			if (!in_array($logged_info->member_srl, $member_srls))
 			{
 				$member_srls[0] = $logged_info->member_srl;
 				$member_keys[$logged_info->member_srl] = 0;
 			}
 		}
 
-		if(!count($member_srls)) return $output;
+		if (!count($member_srls)) return $output;
 
-		$member_args->member_srl = implode(',',$member_srls);
+		$member_args->member_srl = implode(',', $member_srls);
 		$member_output = executeQueryArray('member.getMembers', $member_args);
-		if($member_output->data)
+		if ($member_output->data)
 		{
-			foreach($member_output->data as $key => $val)
+			foreach ($member_output->data as $key => $val)
 			{
 				$output->data[$member_keys[$val->member_srl]] = $val;
 			}

Braces +18 added lines, -6 removed lines

@@ -21,7 +21,9 @@ 
 
 	function read($session_key)
 	{
-		if(!$session_key || !$this->session_started) return;
+		if(!$session_key || !$this->session_started) {
+			return;
+		}
 
 		$args = new stdClass();
 		$args->session_key = $session_key;
@@ -51,13 +53,21 @@ 
 			$site_module_info = Context::get('site_module_info');
 			$args->site_srl = (int)$site_module_info->site_srl;
 		}
-		if(!$args->list_count) $args->list_count = 20;
-		if(!$args->page) $args->page = 1;
-		if(!$args->period_time) $args->period_time = 3;
+		if(!$args->list_count) {
+			$args->list_count = 20;
+		}
+		if(!$args->page) {
+			$args->page = 1;
+		}
+		if(!$args->period_time) {
+			$args->period_time = 3;
+		}
 		$args->last_update = date("YmdHis", $_SERVER['REQUEST_TIME'] - $args->period_time*60);
 
 		$output = executeQueryArray('session.getLoggedMembers', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$member_srls = array();
 		$member_keys = array();
@@ -80,7 +90,9 @@ 
 			}
 		}
 
-		if(!count($member_srls)) return $output;
+		if(!count($member_srls)) {
+			return $output;
+		}
 
 		$member_args->member_srl = implode(',',$member_srls);
 		$member_output = executeQueryArray('member.getMembers', $member_args);

modules/point/point.model.php

Spacing +36 added lines, -36 removed lines

@@ -24,28 +24,28 @@ 
 		$member_srl = abs($member_srl);
 
 		// Get from instance memory
-		if($this->pointList[$member_srl]) return true;
+		if ($this->pointList[$member_srl]) return true;
 
 		// Get from file cache
-		$path = sprintf(_XE_PATH_ . 'files/member_extra_info/point/%s',getNumberingPath($member_srl));
+		$path = sprintf(_XE_PATH_.'files/member_extra_info/point/%s', getNumberingPath($member_srl));
 		$cache_filename = sprintf('%s%d.cache.txt', $path, $member_srl);
-		if(file_exists($cache_filename))
+		if (file_exists($cache_filename))
 		{
-			if(!$this->pointList[$member_srl])
+			if (!$this->pointList[$member_srl])
 				$this->pointList[$member_srl] = trim(FileHandler::readFile($cache_filename));
 			return true;
 		}
 
-		$args =new stdClass();
+		$args = new stdClass();
 		$args->member_srl = $member_srl;
 		$output = executeQuery('point.getPoint', $args);
-		if($output->data->member_srl == $member_srl)
+		if ($output->data->member_srl == $member_srl)
 		{
-			if(!$this->pointList[$member_srl])
+			if (!$this->pointList[$member_srl])
 			{
-				$this->pointList[$member_srl] = (int)$output->data->point;
+				$this->pointList[$member_srl] = (int) $output->data->point;
 				FileHandler::makeDir($path);
-				FileHandler::writeFile($cache_filename, (int)$output->data->point);
+				FileHandler::writeFile($cache_filename, (int) $output->data->point);
 			}
 			return true;
 		}
@@ -60,25 +60,25 @@ 
 		$member_srl = abs($member_srl);
 
 		// Get from instance memory
-		if(!$from_db && $this->pointList[$member_srl]) return $this->pointList[$member_srl];
+		if (!$from_db && $this->pointList[$member_srl]) return $this->pointList[$member_srl];
 
 		// Get from file cache
-		$path = sprintf(_XE_PATH_ . 'files/member_extra_info/point/%s',getNumberingPath($member_srl));
+		$path = sprintf(_XE_PATH_.'files/member_extra_info/point/%s', getNumberingPath($member_srl));
 		$cache_filename = sprintf('%s%d.cache.txt', $path, $member_srl);
 
-		if(!$from_db && file_exists($cache_filename))
+		if (!$from_db && file_exists($cache_filename))
 			return $this->pointList[$member_srl] = trim(FileHandler::readFile($cache_filename));
 
 		// Get from the DB
-		$args =new stdClass();
+		$args = new stdClass();
 		$args->member_srl = $member_srl;
 		$output = executeQuery('point.getPoint', $args);
 
-		if(isset($output->data->member_srl))
+		if (isset($output->data->member_srl))
 		{
-			$point = (int)$output->data->point;
+			$point = (int) $output->data->point;
 			$this->pointList[$member_srl] = $point;
-			if(!is_dir($path)) FileHandler::makeDir($path);
+			if (!is_dir($path)) FileHandler::makeDir($path);
 			FileHandler::writeFile($cache_filename, $point);
 			return $point;
 		}
@@ -91,8 +91,8 @@ 
 	function getLevel($point, $level_step)
 	{
 		$level_count = count($level_step);
-		for($level=0;$level<=$level_count;$level++) if($point < $level_step[$level]) break;
-		$level --;
+		for ($level = 0; $level <= $level_count; $level++) if ($point < $level_step[$level]) break;
+		$level--;
 		return $level;
 	}
 
@@ -103,20 +103,20 @@ 
 	{
 		$member_srls = Context::get('member_srls');
 		$member_srls = array_unique(explode(',', $member_srls));
-		if(!count($member_srls))
+		if (!count($member_srls))
 		{
 			return;
 		}
 
 		$logged_info = Context::get('logged_info');
-		if(!$logged_info->member_srl)
+		if (!$logged_info->member_srl)
 		{
 			return;
 		}
 
-		if(!getModel('module')->isSiteAdmin($logged_info))
+		if (!getModel('module')->isSiteAdmin($logged_info))
 		{
-			if(in_array($logged_info->member_srl, $member_srls))
+			if (in_array($logged_info->member_srl, $member_srls))
 			{
 				$member_srls = array($logged_info->member_srl);
 			}
@@ -130,7 +130,7 @@ 
 		$config = $oModuleModel->getModuleConfig('point');
 
 		$info = array();
-		foreach($member_srls as $v)
+		foreach ($member_srls as $v)
 		{
 			$obj = new stdClass;
 			$obj->point = $this->getPoint($v);
@@ -139,7 +139,7 @@ 
 			$info[] = $obj;
 		}
 
-		$this->add('point_info',$info);
+		$this->add('point_info', $info);
 	}
 
 
@@ -149,37 +149,37 @@ 
 	function getMemberList($args = null, $columnList = array())
 	{
 		// Arrange the search options
-		$args->is_admin = Context::get('is_admin')=='Y'?'Y':'';
-		$args->is_denied = Context::get('is_denied')=='Y'?'Y':'';
+		$args->is_admin = Context::get('is_admin') == 'Y' ? 'Y' : '';
+		$args->is_denied = Context::get('is_denied') == 'Y' ? 'Y' : '';
 		$args->selected_group_srl = Context::get('selected_group_srl');
 
 		$search_target = trim(Context::get('search_target'));
 		$search_keyword = trim(Context::get('search_keyword'));
 
 		// if search keyword is emtpy, show all list
-		if(!$search_keyword)
+		if (!$search_keyword)
 		{
 			unset($args->is_admin, $args->is_denied, $args->selected_group_srl, $search_target);
 		}
 
-		if($search_target && $search_keyword)
+		if ($search_target && $search_keyword)
 		{
-			switch($search_target)
+			switch ($search_target)
 			{
 				case 'user_id' :
-					if($search_keyword) $search_keyword = str_replace(' ','%',$search_keyword);
+					if ($search_keyword) $search_keyword = str_replace(' ', '%', $search_keyword);
 					$args->s_user_id = $search_keyword;
 					break;
 				case 'user_name' :
-					if($search_keyword) $search_keyword = str_replace(' ','%',$search_keyword);
+					if ($search_keyword) $search_keyword = str_replace(' ', '%', $search_keyword);
 					$args->s_user_name = $search_keyword;
 					break;
 				case 'nick_name' :
-					if($search_keyword) $search_keyword = str_replace(' ','%',$search_keyword);
+					if ($search_keyword) $search_keyword = str_replace(' ', '%', $search_keyword);
 					$args->s_nick_name = $search_keyword;
 					break;
 				case 'email_address' :
-					if($search_keyword) $search_keyword = str_replace(' ','%',$search_keyword);
+					if ($search_keyword) $search_keyword = str_replace(' ', '%', $search_keyword);
 					$args->s_email_address = $search_keyword;
 					break;
 				case 'regdate' :
@@ -194,7 +194,7 @@ 
 			}
 		}
 		// If there is a selected_group_srl, change the "query id" (for table join)
-		if($args->selected_group_srl)
+		if ($args->selected_group_srl)
 		{
 			$query_id = 'point.getMemberListWithinGroup';
 		}
@@ -205,12 +205,12 @@ 
 
 		$output = executeQuery($query_id, $args, $columnList);
 
-		if($output->total_count)
+		if ($output->total_count)
 		{
 			$oModuleModel = getModel('module');
 			$config = $oModuleModel->getModuleConfig('point');
 
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
 				$output->data[$key]->level = $this->getLevel($val->point, $config->level_step);
 			}

Braces +32 added lines, -16 removed lines

@@ -24,15 +24,18 @@ 
 		$member_srl = abs($member_srl);
 
 		// Get from instance memory
-		if($this->pointList[$member_srl]) return true;
+		if($this->pointList[$member_srl]) {
+			return true;
+		}
 
 		// Get from file cache
 		$path = sprintf(_XE_PATH_ . 'files/member_extra_info/point/%s',getNumberingPath($member_srl));
 		$cache_filename = sprintf('%s%d.cache.txt', $path, $member_srl);
 		if(file_exists($cache_filename))
 		{
-			if(!$this->pointList[$member_srl])
-				$this->pointList[$member_srl] = trim(FileHandler::readFile($cache_filename));
+			if(!$this->pointList[$member_srl]) {
+							$this->pointList[$member_srl] = trim(FileHandler::readFile($cache_filename));
+			}
 			return true;
 		}
 
@@ -60,14 +63,17 @@ 
 		$member_srl = abs($member_srl);
 
 		// Get from instance memory
-		if(!$from_db && $this->pointList[$member_srl]) return $this->pointList[$member_srl];
+		if(!$from_db && $this->pointList[$member_srl]) {
+			return $this->pointList[$member_srl];
+		}
 
 		// Get from file cache
 		$path = sprintf(_XE_PATH_ . 'files/member_extra_info/point/%s',getNumberingPath($member_srl));
 		$cache_filename = sprintf('%s%d.cache.txt', $path, $member_srl);
 
-		if(!$from_db && file_exists($cache_filename))
-			return $this->pointList[$member_srl] = trim(FileHandler::readFile($cache_filename));
+		if(!$from_db && file_exists($cache_filename)) {
+					return $this->pointList[$member_srl] = trim(FileHandler::readFile($cache_filename));
+		}
 
 		// Get from the DB
 		$args =new stdClass();
@@ -78,7 +84,9 @@ 
 		{
 			$point = (int)$output->data->point;
 			$this->pointList[$member_srl] = $point;
-			if(!is_dir($path)) FileHandler::makeDir($path);
+			if(!is_dir($path)) {
+				FileHandler::makeDir($path);
+			}
 			FileHandler::writeFile($cache_filename, $point);
 			return $point;
 		}
@@ -91,7 +99,9 @@ 
 	function getLevel($point, $level_step)
 	{
 		$level_count = count($level_step);
-		for($level=0;$level<=$level_count;$level++) if($point < $level_step[$level]) break;
+		for($level=0;$level<=$level_count;$level++) {
+			if($point < $level_step[$level]) break;
+		}
 		$level --;
 		return $level;
 	}
@@ -119,8 +129,7 @@ 
 			if(in_array($logged_info->member_srl, $member_srls))
 			{
 				$member_srls = array($logged_info->member_srl);
-			}
-			else
+			} else
 			{
 				return;
 			}
@@ -167,19 +176,27 @@ 
 			switch($search_target)
 			{
 				case 'user_id' :
-					if($search_keyword) $search_keyword = str_replace(' ','%',$search_keyword);
+					if($search_keyword) {
+						$search_keyword = str_replace(' ','%',$search_keyword);
+					}
 					$args->s_user_id = $search_keyword;
 					break;
 				case 'user_name' :
-					if($search_keyword) $search_keyword = str_replace(' ','%',$search_keyword);
+					if($search_keyword) {
+						$search_keyword = str_replace(' ','%',$search_keyword);
+					}
 					$args->s_user_name = $search_keyword;
 					break;
 				case 'nick_name' :
-					if($search_keyword) $search_keyword = str_replace(' ','%',$search_keyword);
+					if($search_keyword) {
+						$search_keyword = str_replace(' ','%',$search_keyword);
+					}
 					$args->s_nick_name = $search_keyword;
 					break;
 				case 'email_address' :
-					if($search_keyword) $search_keyword = str_replace(' ','%',$search_keyword);
+					if($search_keyword) {
+						$search_keyword = str_replace(' ','%',$search_keyword);
+					}
 					$args->s_email_address = $search_keyword;
 					break;
 				case 'regdate' :
@@ -197,8 +214,7 @@ 
 		if($args->selected_group_srl)
 		{
 			$query_id = 'point.getMemberListWithinGroup';
-		}
-		else
+		} else
 		{
 			$query_id = 'point.getMemberList';
 		}

classes/display/XMLDisplayHandler.php

Spacing +5 added lines, -5 removed lines

@@ -31,25 +31,25 @@
 	 */
 	function _makeXmlDoc($obj)
 	{
-		if(!count($obj))
+		if (!count($obj))
 		{
 			return;
 		}
 
 		$xmlDoc = '';
 
-		foreach($obj as $key => $val)
+		foreach ($obj as $key => $val)
 		{
-			if(is_numeric($key))
+			if (is_numeric($key))
 			{
 				$key = 'item';
 			}
 
-			if(is_string($val))
+			if (is_string($val))
 			{
 				$xmlDoc .= sprintf('<%s>%s</%s>%s', $key, htmlspecialchars($val, ENT_COMPAT, 'UTF-8', true), $key, "\n");
 			}
-			else if(!is_array($val) && !is_object($val))
+			else if (!is_array($val) && !is_object($val))
 			{
 				$xmlDoc .= sprintf('<%s>%s</%s>%s', $key, $val, $key, "\n");
 			}

Braces +2 added lines, -4 removed lines

@@ -48,12 +48,10 @@
 			if(is_string($val))
 			{
 				$xmlDoc .= sprintf('<%s>%s</%s>%s', $key, htmlspecialchars($val, ENT_COMPAT, 'UTF-8', true), $key, "\n");
-			}
-			else if(!is_array($val) && !is_object($val))
+			} else if(!is_array($val) && !is_object($val))
 			{
 				$xmlDoc .= sprintf('<%s>%s</%s>%s', $key, $val, $key, "\n");
-			}
-			else
+			} else
 			{
 				$xmlDoc .= sprintf('<%s>%s%s</%s>%s', $key, "\n", $this->_makeXmlDoc($val), $key, "\n");
 			}

classes/display/VirtualXMLDisplayHandler.php

Spacing +8 added lines, -8 removed lines

@@ -17,35 +17,35 @@
 		$request_url = Context::getRequestUri();
 		$output = new stdClass();
 
-		if(substr_compare($request_url, '/', -1) !== 0)
+		if (substr_compare($request_url, '/', -1) !== 0)
 		{
 			$request_url .= '/';
 		}
 
-		if($error === 0)
+		if ($error === 0)
 		{
-			if($message != 'success') $output->message = $message;
+			if ($message != 'success') $output->message = $message;
 
 			$output->url = ($redirect_url) ? $redirect_url : $request_uri;
 		}
 		else
 		{
-			if($message != 'fail') $output->message = $message;
+			if ($message != 'fail') $output->message = $message;
 		}
 
 		$html = array();
 		$html[] = '<!DOCTYPE html><html><head><title>Moved...</title><meta charset="utf-8" /><script>';
 
-		if($output->message)
+		if ($output->message)
 		{
-			$html[] = 'alert(' . json_encode($output->message, JSON_UNESCAPED_SLASHES) . ');';
+			$html[] = 'alert('.json_encode($output->message, JSON_UNESCAPED_SLASHES).');';
 		}
 
-		if($output->url)
+		if ($output->url)
 		{
 			$url = json_encode(preg_replace('/#(.+)$/i', '', $output->url), JSON_UNESCAPED_SLASHES);
 			$html[] = 'var win = (window.opener) ? window.opener : window.parent;';
-			$html[] = 'win.location.href = ' . $url;
+			$html[] = 'win.location.href = '.$url;
 			$html[] = 'if(window.opener) self.close();';
 		}
 

Braces +7 added lines, -4 removed lines

@@ -24,13 +24,16 @@
 
 		if($error === 0)
 		{
-			if($message != 'success') $output->message = $message;
+			if($message != 'success') {
+				$output->message = $message;
+			}
 
 			$output->url = ($redirect_url) ? $redirect_url : $request_uri;
-		}
-		else
+		} else
 		{
-			if($message != 'fail') $output->message = $message;
+			if($message != 'fail') {
+				$output->message = $message;
+			}
 		}
 
 		$html = array();

classes/mobile/Mobile.class.php

Spacing +28 added lines, -28 removed lines

@@ -23,7 +23,7 @@ 
 	function &getInstance()
 	{
 		static $theInstance;
-		if(!isset($theInstance))
+		if (!isset($theInstance))
 		{
 			$theInstance = new Mobile();
 		}
@@ -48,11 +48,11 @@ 
 	 */
 	function _isFromMobilePhone()
 	{
-		if($this->ismobile !== NULL)
+		if ($this->ismobile !== NULL)
 		{
 			return $this->ismobile;
 		}
-		if(Mobile::isMobileEnabled() === false || Context::get('full_browse') || $_COOKIE["FullBrowse"])
+		if (Mobile::isMobileEnabled() === false || Context::get('full_browse') || $_COOKIE["FullBrowse"])
 		{
 			return ($this->ismobile = false);
 		}
@@ -63,22 +63,22 @@ 
 		$this->ismobile = FALSE;
 
 		$m = Context::get('m');
-		if(strlen($m) == 1)
+		if (strlen($m) == 1)
 		{
-			if($m == "1")
+			if ($m == "1")
 			{
 				$this->ismobile = TRUE;
 			}
-			elseif($m == "0")
+			elseif ($m == "0")
 			{
 				$this->ismobile = FALSE;
 			}
 		}
-		elseif(isset($_COOKIE['mobile']))
+		elseif (isset($_COOKIE['mobile']))
 		{
-			if($_COOKIE['user-agent'] == md5($_SERVER['HTTP_USER_AGENT']))
+			if ($_COOKIE['user-agent'] == md5($_SERVER['HTTP_USER_AGENT']))
 			{
-				if($_COOKIE['mobile'] == 'true')
+				if ($_COOKIE['mobile'] == 'true')
 				{
 					$this->ismobile = TRUE;
 				}
@@ -92,7 +92,7 @@ 
 				$this->ismobile = FALSE;
 				setcookie("mobile", FALSE);
 				setcookie("user-agent", FALSE);
-				if(!$this->isMobilePadCheckByAgent() && $this->isMobileCheckByAgent())
+				if (!$this->isMobilePadCheckByAgent() && $this->isMobileCheckByAgent())
 				{
 					$this->ismobile = TRUE;
 				}
@@ -100,36 +100,36 @@ 
 		}
 		else
 		{
-			if($this->isMobilePadCheckByAgent())
+			if ($this->isMobilePadCheckByAgent())
 			{
 				$this->ismobile = FALSE;
 			}
 			else
 			{
-				if($this->isMobileCheckByAgent())
+				if ($this->isMobileCheckByAgent())
 				{
 					$this->ismobile = TRUE;
 				}
 			}
 		}
 
-		if($this->ismobile !== NULL)
+		if ($this->ismobile !== NULL)
 		{
-			if($this->ismobile == TRUE)
+			if ($this->ismobile == TRUE)
 			{
-				if($_COOKIE['mobile'] != 'true')
+				if ($_COOKIE['mobile'] != 'true')
 				{
 					$_COOKIE['mobile'] = 'true';
 					setcookie("mobile", 'true');
 				}
 			}
-			elseif($_COOKIE['mobile'] != 'false')
+			elseif ($_COOKIE['mobile'] != 'false')
 			{
 				$_COOKIE['mobile'] = 'false';
 				setcookie("mobile", 'false');
 			}
 
-			if($_COOKIE['user-agent'] != md5($_SERVER['HTTP_USER_AGENT']))
+			if ($_COOKIE['user-agent'] != md5($_SERVER['HTTP_USER_AGENT']))
 			{
 				setcookie("user-agent", md5($_SERVER['HTTP_USER_AGENT']));
 			}
@@ -146,7 +146,7 @@ 
 	function isMobileCheckByAgent()
 	{
 		static $UACheck;
-		if(isset($UACheck))
+		if (isset($UACheck))
 		{
 			return $UACheck;
 		}
@@ -154,15 +154,15 @@ 
 		$oMobile = Mobile::getInstance();
 		$mobileAgent = array('iPod', 'iPhone', 'Android', 'BlackBerry', 'SymbianOS', 'Bada', 'Tizen', 'Kindle', 'Wii', 'SCH-', 'SPH-', 'CANU-', 'Windows Phone', 'Windows CE', 'POLARIS', 'Palm', 'Dorothy Browser', 'Mobile', 'Opera Mobi', 'Opera Mini', 'Minimo', 'AvantGo', 'NetFront', 'Nokia', 'LGPlayer', 'SonyEricsson', 'HTC');
 
-		if($oMobile->isMobilePadCheckByAgent())
+		if ($oMobile->isMobilePadCheckByAgent())
 		{
 			$UACheck = TRUE;
 			return TRUE;
 		}
 
-		foreach($mobileAgent as $agent)
+		foreach ($mobileAgent as $agent)
 		{
-			if(stripos($_SERVER['HTTP_USER_AGENT'], $agent) !== FALSE)
+			if (stripos($_SERVER['HTTP_USER_AGENT'], $agent) !== FALSE)
 			{
 				$UACheck = TRUE;
 				return TRUE;
@@ -180,7 +180,7 @@ 
 	function isMobilePadCheckByAgent()
 	{
 		static $UACheck;
-		if(isset($UACheck))
+		if (isset($UACheck))
 		{
 			return $UACheck;
 		}
@@ -190,21 +190,21 @@ 
 		// $exceptionAgent[0] contains exception agents for all exceptions.
 		$exceptionAgent = array(0 => array('Opera Mini', 'Opera Mobi'), 'Android' => 'Mobile');
 
-		foreach($padAgent as $agent)
+		foreach ($padAgent as $agent)
 		{
-			if(strpos($_SERVER['HTTP_USER_AGENT'], $agent) !== FALSE)
+			if (strpos($_SERVER['HTTP_USER_AGENT'], $agent) !== FALSE)
 			{
-				if(!isset($exceptionAgent[$agent]))
+				if (!isset($exceptionAgent[$agent]))
 				{
 					$UACheck = TRUE;
 					return TRUE;
 				}
-				elseif(strpos($_SERVER['HTTP_USER_AGENT'], $exceptionAgent[$agent]) === FALSE)
+				elseif (strpos($_SERVER['HTTP_USER_AGENT'], $exceptionAgent[$agent]) === FALSE)
 				{
 					// If the agent is the Android, that can be either tablet and mobile phone.
-					foreach($exceptionAgent[0] as $val)
+					foreach ($exceptionAgent[0] as $val)
 					{
-						if(strpos($_SERVER['HTTP_USER_AGENT'], $val) !== FALSE)
+						if (strpos($_SERVER['HTTP_USER_AGENT'], $val) !== FALSE)
 						{
 							$UACheck = FALSE;
 							return FALSE;

Braces +8 added lines, -16 removed lines

@@ -68,26 +68,22 @@ 
 			if($m == "1")
 			{
 				$this->ismobile = TRUE;
-			}
-			elseif($m == "0")
+			} elseif($m == "0")
 			{
 				$this->ismobile = FALSE;
 			}
-		}
-		elseif(isset($_COOKIE['mobile']))
+		} elseif(isset($_COOKIE['mobile']))
 		{
 			if($_COOKIE['user-agent'] == md5($_SERVER['HTTP_USER_AGENT']))
 			{
 				if($_COOKIE['mobile'] == 'true')
 				{
 					$this->ismobile = TRUE;
-				}
-				else
+				} else
 				{
 					$this->ismobile = FALSE;
 				}
-			}
-			else
+			} else
 			{
 				$this->ismobile = FALSE;
 				setcookie("mobile", FALSE);
@@ -97,14 +93,12 @@ 
 					$this->ismobile = TRUE;
 				}
 			}
-		}
-		else
+		} else
 		{
 			if($this->isMobilePadCheckByAgent())
 			{
 				$this->ismobile = FALSE;
-			}
-			else
+			} else
 			{
 				if($this->isMobileCheckByAgent())
 				{
@@ -122,8 +116,7 @@ 
 					$_COOKIE['mobile'] = 'true';
 					setcookie("mobile", 'true');
 				}
-			}
-			elseif($_COOKIE['mobile'] != 'false')
+			} elseif($_COOKIE['mobile'] != 'false')
 			{
 				$_COOKIE['mobile'] = 'false';
 				setcookie("mobile", 'false');
@@ -198,8 +191,7 @@ 
 				{
 					$UACheck = TRUE;
 					return TRUE;
-				}
-				elseif(strpos($_SERVER['HTTP_USER_AGENT'], $exceptionAgent[$agent]) === FALSE)
+				} elseif(strpos($_SERVER['HTTP_USER_AGENT'], $exceptionAgent[$agent]) === FALSE)
 				{
 					// If the agent is the Android, that can be either tablet and mobile phone.
 					foreach($exceptionAgent[0] as $val)