xpressengine / xe-core

classes/xml/xmlquery/queryargument/DefaultValue.class.php

Spacing +7 added lines, -7 removed lines

@@ -70,7 +70,7 @@ 
 	{

 		return $this->_is_string;

 		$str_pos = strpos($this->value, '(');

-		if($str_pos === false)

+		if ($str_pos === false)

 		{

 			return TRUE;

 		}

@@ -99,31 +99,31 @@ 
 

 	function _setValue()

 	{

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

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

 		{

 			return;

 		}

 

 		// If value contains comma separated values and does not contain paranthesis

 		//  -> default value is an array

-		if(strpos($this->value, ',') !== FALSE && strpos($this->value, '(') === FALSE)

+		if (strpos($this->value, ',') !== FALSE && strpos($this->value, '(') === FALSE)

 		{

 			return sprintf('array(%s)', $this->value);

 		}

 

 		$str_pos = strpos($this->value, '(');

 		// // TODO Replace this with parseExpression

-		if($str_pos === FALSE)

+		if ($str_pos === FALSE)

 		{

 			$this->_is_string = TRUE;

-			return '\'' . $this->value . '\'';

+			return '\''.$this->value.'\'';

 		}

 		//if($str_pos===false) return $this->value;

 

 		$func_name = substr($this->value, 0, $str_pos);

 		$args = substr($this->value, $str_pos + 1, -1);

 

-		switch($func_name)

+		switch ($func_name)

 		{

 			case 'ipaddress' :

 				$val = '$_SERVER[\'REMOTE_ADDR\']';

@@ -159,7 +159,7 @@ 
 				$val = sprintf('%d', $args);

 				break;

 			default :

-				$val = '\'' . $this->value . '\'';

+				$val = '\''.$this->value.'\'';

 			//$val = $this->value;

 		}

 

addons/point_level_icon/point_level_icon.lib.php

Spacing +13 added lines, -13 removed lines

@@ -7,24 +7,24 @@ 
 function pointLevelIconTrans($matches)
 {
 	$member_srl = $matches[3];
-	if($member_srl < 1)
+	if ($member_srl < 1)
 	{
 		return $matches[0];
 	}
 
-	$orig_text = preg_replace('/' . preg_quote($matches[5], '/') . '<\/' . $matches[6] . '>$/', '', $matches[0]);
+	$orig_text = preg_replace('/'.preg_quote($matches[5], '/').'<\/'.$matches[6].'>$/', '', $matches[0]);
 
 	// Check Group Image Mark
 	$oMemberModel = getModel('member');
-	if($oMemberModel->getGroupImageMark($member_srl))
+	if ($oMemberModel->getGroupImageMark($member_srl))
 	{
-		return $orig_text . $matches[5] . '</' . $matches[6] . '>';
+		return $orig_text.$matches[5].'</'.$matches[6].'>';
 	}
 
-	if(!isset($GLOBALS['_pointLevelIcon'][$member_srl]))
+	if (!isset($GLOBALS['_pointLevelIcon'][$member_srl]))
 	{
 		// Get point configuration
-		if(!$GLOBALS['_pointConfig'])
+		if (!$GLOBALS['_pointConfig'])
 		{
 			$oModuleModel = getModel('module');
 			$GLOBALS['_pointConfig'] = $oModuleModel->getModuleConfig('point');
@@ -32,14 +32,14 @@ 
 		$config = $GLOBALS['_pointConfig'];
 
 		// Get point model
-		if(!$GLOBALS['_pointModel'])
+		if (!$GLOBALS['_pointModel'])
 		{
 			$GLOBALS['_pointModel'] = getModel('point');
 		}
 		$oPointModel = &$GLOBALS['_pointModel'];
 
 		// Get points
-		if(!$oPointModel->isExistsPoint($member_srl))
+		if (!$oPointModel->isExistsPoint($member_srl))
 		{
 			return $matches[0];
 		}
@@ -55,25 +55,25 @@ 
 
 		// Get per to go to the next level if not a top level
 		$per = NULL;
-		if($level < $config->max_level)
+		if ($level < $config->max_level)
 		{
 			$next_point = $config->level_step[$level + 1];
 			$present_point = $config->level_step[$level];
-			if($next_point > 0)
+			if ($next_point > 0)
 			{
 				$per = (int) (($point - $present_point) / ($next_point - $present_point) * 100);
-				$per = $per . '%';
+				$per = $per.'%';
 			}
 		}
 
-		$title = sprintf('%s:%s%s%s, %s:%s/%s', Context::getLang('point'), $point, $config->point_name, $per ? ' (' . $per . ')' : '', Context::getLang('level'), $level, $config->max_level);
+		$title = sprintf('%s:%s%s%s, %s:%s/%s', Context::getLang('point'), $point, $config->point_name, $per ? ' ('.$per.')' : '', Context::getLang('level'), $level, $config->max_level);
 		$alt = sprintf('[%s:%s]', Context::getLang('level'), $level);
 
 		$GLOBALS['_pointLevelIcon'][$member_srl] = sprintf('<img src="%s" alt="%s" title="%s" class="xe_point_level_icon" style="vertical-align:middle;margin-right:3px;" />', $level_icon, $alt, $title);
 	}
 	$text = $GLOBALS['_pointLevelIcon'][$member_srl];
 
-	return $orig_text . $text . $matches[5] . '</' . $matches[6] . '>';
+	return $orig_text.$text.$matches[5].'</'.$matches[6].'>';
 }
 
 /* End of file point_level_icon.lib.php */

modules/point/point.admin.controller.php

Spacing +83 added lines, -83 removed lines

@@ -26,10 +26,10 @@ 
 		$args = Context::getRequestVars();
 
 		//if module IO config is off
-		if($args->able_module == 'Y')
+		if ($args->able_module == 'Y')
 		{
 			// Re-install triggers, if it was disabled.
-			if($config->able_module == 'N')
+			if ($config->able_module == 'N')
 			{
 				$this->moduleUpdate();
 			}
@@ -39,51 +39,51 @@ 
 
 			// Check the point name
 			$config->point_name = $args->point_name;
-			if(!$config->point_name) $config->point_name = 'point';
+			if (!$config->point_name) $config->point_name = 'point';
 			// Specify the default points
-			$config->signup_point = (int)$args->signup_point;
-			$config->login_point = (int)$args->login_point;
-			$config->insert_document = (int)$args->insert_document;
-			$config->read_document = (int)$args->read_document;
-			$config->insert_comment = (int)$args->insert_comment;
-			$config->upload_file = (int)$args->upload_file;
-			$config->download_file = (int)$args->download_file;
-			$config->voted = (int)$args->voted;
-			$config->blamed = (int)$args->blamed;
+			$config->signup_point = (int) $args->signup_point;
+			$config->login_point = (int) $args->login_point;
+			$config->insert_document = (int) $args->insert_document;
+			$config->read_document = (int) $args->read_document;
+			$config->insert_comment = (int) $args->insert_comment;
+			$config->upload_file = (int) $args->upload_file;
+			$config->download_file = (int) $args->download_file;
+			$config->voted = (int) $args->voted;
+			$config->blamed = (int) $args->blamed;
 			// The highest level
 			$config->max_level = $args->max_level;
-			if($config->max_level>1000) $config->max_level = 1000;
-			if($config->max_level<1) $config->max_level = 1;
+			if ($config->max_level > 1000) $config->max_level = 1000;
+			if ($config->max_level < 1) $config->max_level = 1;
 			// Set the level icon
 			$config->level_icon = $args->level_icon;
 			// Check if downloads are not allowed
-			if($args->disable_download == 'Y') $config->disable_download = 'Y';
+			if ($args->disable_download == 'Y') $config->disable_download = 'Y';
 			else $config->disable_download = 'N';
 			// Check if reading a document is not allowed
-			if($args->disable_read_document == 'Y') $config->disable_read_document = 'Y';
+			if ($args->disable_read_document == 'Y') $config->disable_read_document = 'Y';
 			else $config->disable_read_document = 'N';
 
 			$oMemberModel = getModel('member');
 			$group_list = $oMemberModel->getGroups();
 
 			// Per-level group configurations
-			foreach($group_list as $group)
+			foreach ($group_list as $group)
 			{
 				// Admin group should not be connected to point.
-				if($group->is_admin == 'Y' || $group->is_default == 'Y') continue;
+				if ($group->is_admin == 'Y' || $group->is_default == 'Y') continue;
 
 				$group_srl = $group->group_srl;
 
-				if(isset($args->{'point_group_'.$group_srl}))
+				if (isset($args->{'point_group_'.$group_srl}))
 				{
 					//if group level is higher than max level, change to max level
-					if($args->{'point_group_'.$group_srl} > $args->max_level)
+					if ($args->{'point_group_'.$group_srl} > $args->max_level)
 					{
 						$args->{'point_group_'.$group_srl} = $args->max_level;
 					}
 
 					//if group level is lower than 1, change to 1
-					if($args->{'point_group_'.$group_srl} < 1)
+					if ($args->{'point_group_'.$group_srl} < 1)
 					{
 						$args->{'point_group_'.$group_srl} = 1;
 					}
@@ -98,10 +98,10 @@ 
 			$config->group_reset = $args->group_reset;
 			// Per-level point configurations
 			unset($config->level_step);
-			for($i=1;$i<=$config->max_level;$i++)
+			for ($i = 1; $i <= $config->max_level; $i++)
 			{
 				$key = "level_step_".$i;
-				$config->level_step[$i] = (int)$args->{$key};
+				$config->level_step[$i] = (int) $args->{$key};
 			}
 			// A function to calculate per-level points
 			$config->expression = $args->expression;
@@ -133,11 +133,11 @@ 
 		$args = Context::getRequestVars();
 
 		$configTypeList = array('insert_document', 'insert_comment', 'upload_file', 'download_file', 'read_document', 'voted', 'blamed');
-		foreach($configTypeList AS $config)
+		foreach ($configTypeList AS $config)
 		{
-			if(is_array($args->{$config}))
+			if (is_array($args->{$config}))
 			{
-				foreach($args->{$config} AS $key=>$value)
+				foreach ($args->{$config} AS $key=>$value)
 				{
 					$module_config[$key][$config] = $value;
 				}
@@ -145,16 +145,16 @@ 
 		}
 
 		$oModuleController = getController('module');
-		if(count($module_config))
+		if (count($module_config))
 		{
-			foreach($module_config as $module_srl => $config)
+			foreach ($module_config as $module_srl => $config)
 			{
-				$oModuleController->insertModulePartConfig('point',$module_srl,$config);
+				$oModuleController->insertModulePartConfig('point', $module_srl, $config);
 			}
 		}
 
 		$this->setMessage('success_updated');
-		if(!in_array(Context::getRequestMethod(),array('XMLRPC','JSON')))
+		if (!in_array(Context::getRequestMethod(), array('XMLRPC', 'JSON')))
 		{
 			$returnUrl = Context::get('success_return_url') ? Context::get('success_return_url') : getNotEncodedUrl('', 'module', 'admin', 'act', 'dispPointAdminModuleConfig');
 			header('location:'.$returnUrl);
@@ -168,24 +168,24 @@ 
 	function procPointAdminInsertPointModuleConfig()
 	{
 		$module_srl = Context::get('target_module_srl');
-		if(!$module_srl) return new Object(-1, 'msg_invalid_request');
+		if (!$module_srl) return new Object(-1, 'msg_invalid_request');
 		// In case of batch configuration of several modules
-		if(preg_match('/^([0-9,]+)$/',$module_srl)) $module_srl = explode(',',$module_srl);
+		if (preg_match('/^([0-9,]+)$/', $module_srl)) $module_srl = explode(',', $module_srl);
 		else $module_srl = array($module_srl);
 		// Save configurations
 		$oModuleController = getController('module');
-		for($i=0;$i<count($module_srl);$i++)
+		for ($i = 0; $i < count($module_srl); $i++)
 		{
 			$srl = trim($module_srl[$i]);
-			if(!$srl) continue;
+			if (!$srl) continue;
 			unset($config);
-			$config['insert_document'] = (int)Context::get('insert_document');
-			$config['insert_comment'] = (int)Context::get('insert_comment');
-			$config['upload_file'] = (int)Context::get('upload_file');
-			$config['download_file'] = (int)Context::get('download_file');
-			$config['read_document'] = (int)Context::get('read_document');
-			$config['voted'] = (int)Context::get('voted');
-			$config['blamed'] = (int)Context::get('blamed');
+			$config['insert_document'] = (int) Context::get('insert_document');
+			$config['insert_comment'] = (int) Context::get('insert_comment');
+			$config['upload_file'] = (int) Context::get('upload_file');
+			$config['download_file'] = (int) Context::get('download_file');
+			$config['read_document'] = (int) Context::get('read_document');
+			$config['voted'] = (int) Context::get('voted');
+			$config['blamed'] = (int) Context::get('blamed');
 			$oModuleController->insertModulePartConfig('point', $srl, $config);
 		}
 
@@ -207,7 +207,7 @@ 
 		preg_match('/^(\+|-)?([1-9][0-9]*)$/', $point, $m);
 
 		$action = '';
-		switch($m[1])
+		switch ($m[1])
 		{
 			case '+':
 				$action = 'add';
@@ -222,7 +222,7 @@ 
 		$point = $m[2];
 
 		$oPointController = getController('point');
-		$output = $oPointController->setPoint($member_srl, (int)$point, $action);
+		$output = $oPointController->setPoint($member_srl, (int) $point, $action);
 
 		$this->setError(-1);
 		$this->setMessage('success_updated', 'info');
@@ -247,29 +247,29 @@ 
 
 		// Get member infomation
 		$output = executeQueryArray('point.getMemberCount');
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		if($output->data)
+		if ($output->data)
 		{
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
-				if(!$val->member_srl) continue;
+				if (!$val->member_srl) continue;
 				$member[$val->member_srl] = 0;
 			}
 		}
 
 		// Get post information
 		$output = executeQueryArray('point.getDocumentPoint');
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		if($output->data)
+		if ($output->data)
 		{
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
-				if($module_config[$val->module_srl]['insert_document']) $insert_point = $module_config[$val->module_srl]['insert_document'];
+				if ($module_config[$val->module_srl]['insert_document']) $insert_point = $module_config[$val->module_srl]['insert_document'];
 				else $insert_point = $config->insert_document;
 
-				if(!$val->member_srl) continue;
+				if (!$val->member_srl) continue;
 				$point = $insert_point * $val->count;
 				$member[$val->member_srl] += $point;
 			}
@@ -278,16 +278,16 @@ 
 		$output = null;
 		// Get comments information
 		$output = executeQueryArray('point.getCommentPoint');
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		if($output->data)
+		if ($output->data)
 		{
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
-				if($module_config[$val->module_srl]['insert_comment']) $insert_point = $module_config[$val->module_srl]['insert_comment'];
+				if ($module_config[$val->module_srl]['insert_comment']) $insert_point = $module_config[$val->module_srl]['insert_comment'];
 				else $insert_point = $config->insert_comment;
 
-				if(!$val->member_srl) continue;
+				if (!$val->member_srl) continue;
 				$point = $insert_point * $val->count;
 				$member[$val->member_srl] += $point;
 			}
@@ -295,16 +295,16 @@ 
 		$output = null;
 		// Get the attached files' information
 		$output = executeQueryArray('point.getFilePoint');
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		if($output->data)
+		if ($output->data)
 		{
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
-				if($module_config[$val->module_srl]['upload_file']) $insert_point = $module_config[$val->module_srl]['upload_file'];
+				if ($module_config[$val->module_srl]['upload_file']) $insert_point = $module_config[$val->module_srl]['upload_file'];
 				else $insert_point = $config->upload_file;
 
-				if(!$val->member_srl) continue;
+				if (!$val->member_srl) continue;
 				$point = $insert_point * $val->count;
 				$member[$val->member_srl] += $point;
 			}
@@ -312,13 +312,13 @@ 
 		$output = null;
 		// Set all members' points to 0
 		$output = executeQuery("point.initMemberPoint");
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 		// Save the file temporarily
 		
 		$str = '';
-		foreach($member as $key => $val)
+		foreach ($member as $key => $val)
 		{
-			$val += (int)$config->signup_point;
+			$val += (int) $config->signup_point;
 			$str .= $key.','.$val."\r\n";
 		}
 
@@ -326,7 +326,7 @@ 
 
 		$this->add('total', count($member));
 		$this->add('position', 0);
-		$this->setMessage( sprintf(Context::getLang('point_recal_message'), 0, $this->get('total')) );
+		$this->setMessage(sprintf(Context::getLang('point_recal_message'), 0, $this->get('total')));
 	}
 
 	/**
@@ -334,35 +334,35 @@ 
 	 */
 	function procPointAdminApplyPoint()
 	{
-		$position = (int)Context::get('position');
-		$total = (int)Context::get('total');
+		$position = (int) Context::get('position');
+		$total = (int) Context::get('total');
 
-		if(!file_exists('./files/cache/pointRecal.txt')) return new Object(-1, 'msg_invalid_request');
+		if (!file_exists('./files/cache/pointRecal.txt')) return new Object(-1, 'msg_invalid_request');
 
 		$idx = 0;
-		$f = fopen("./files/cache/pointRecal.txt","r");
-		while(!feof($f))
+		$f = fopen("./files/cache/pointRecal.txt", "r");
+		while (!feof($f))
 		{
 			$str = trim(fgets($f, 1024));
-			$idx ++;
-			if($idx > $position)
+			$idx++;
+			if ($idx > $position)
 			{
-				list($member_srl, $point) = explode(',',$str);
+				list($member_srl, $point) = explode(',', $str);
 
 				$args = new stdClass();
 				$args->member_srl = $member_srl;
 				$args->point = $point;
-				$output = executeQuery('point.insertPoint',$args);
-				if($idx%5000==0) break;
+				$output = executeQuery('point.insertPoint', $args);
+				if ($idx % 5000 == 0) break;
 			}
 		}
 
-		if(feof($f))
+		if (feof($f))
 		{
 			FileHandler::removeFile('./files/cache/pointRecal.txt');
 			$idx = $total;
 
-			FileHandler::rename('./files/member_extra_info/point','./files/member_extra_info/point.old');
+			FileHandler::rename('./files/member_extra_info/point', './files/member_extra_info/point.old');
 
 			FileHandler::removeDir('./files/member_extra_info/point.old');
 		}
@@ -380,16 +380,16 @@ 
 	function procPointAdminReset()
 	{
 		$module_srl = Context::get('module_srls');
-		if(!$module_srl) return new Object(-1, 'msg_invalid_request');
+		if (!$module_srl) return new Object(-1, 'msg_invalid_request');
 		// In case of batch configuration of several modules
-		if(preg_match('/^([0-9,]+)$/',$module_srl)) $module_srl = explode(',',$module_srl);
+		if (preg_match('/^([0-9,]+)$/', $module_srl)) $module_srl = explode(',', $module_srl);
 		else $module_srl = array($module_srl);
 		// Save configurations
 		$oModuleController = getController('module');
-		for($i=0;$i<count($module_srl);$i++)
+		for ($i = 0; $i < count($module_srl); $i++)
 		{
 			$srl = trim($module_srl[$i]);
-			if(!$srl) continue;
+			if (!$srl) continue;
 			$args = new stdClass();
 			$args->module = 'point';
 			$args->module_srl = $srl;
@@ -397,7 +397,7 @@ 
 		}
 
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$oCacheHandler->invalidateGroupKey('site_and_module');
 		}

Braces +93 added lines, -35 removed lines

@@ -39,7 +39,9 @@ 
 
 			// Check the point name
 			$config->point_name = $args->point_name;
-			if(!$config->point_name) $config->point_name = 'point';
+			if(!$config->point_name) {
+				$config->point_name = 'point';
+			}
 			// Specify the default points
 			$config->signup_point = (int)$args->signup_point;
 			$config->login_point = (int)$args->login_point;
@@ -52,16 +54,26 @@ 
 			$config->blamed = (int)$args->blamed;
 			// The highest level
 			$config->max_level = $args->max_level;
-			if($config->max_level>1000) $config->max_level = 1000;
-			if($config->max_level<1) $config->max_level = 1;
+			if($config->max_level>1000) {
+				$config->max_level = 1000;
+			}
+			if($config->max_level<1) {
+				$config->max_level = 1;
+			}
 			// Set the level icon
 			$config->level_icon = $args->level_icon;
 			// Check if downloads are not allowed
-			if($args->disable_download == 'Y') $config->disable_download = 'Y';
-			else $config->disable_download = 'N';
+			if($args->disable_download == 'Y') {
+				$config->disable_download = 'Y';
+			} else {
+				$config->disable_download = 'N';
+			}
 			// Check if reading a document is not allowed
-			if($args->disable_read_document == 'Y') $config->disable_read_document = 'Y';
-			else $config->disable_read_document = 'N';
+			if($args->disable_read_document == 'Y') {
+				$config->disable_read_document = 'Y';
+			} else {
+				$config->disable_read_document = 'N';
+			}
 
 			$oMemberModel = getModel('member');
 			$group_list = $oMemberModel->getGroups();
@@ -71,7 +83,9 @@ 
 			foreach($group_list as $group)
 			{
 				// Admin group should not be connected to point.
-				if($group->is_admin == 'Y' || $group->is_default == 'Y') continue;
+				if($group->is_admin == 'Y' || $group->is_default == 'Y') {
+					continue;
+				}
 
 				$group_srl = $group->group_srl;
 
@@ -102,8 +116,7 @@ 
 			}
 			// A function to calculate per-level points
 			$config->expression = $args->expression;
-		}
-		else
+		} else
 		{
 			//module IO config is OFF, Other settings will not be modified.
 			$config->able_module = 'N';
@@ -165,16 +178,23 @@ 
 	function procPointAdminInsertPointModuleConfig()
 	{
 		$module_srl = Context::get('target_module_srl');
-		if(!$module_srl) return new Object(-1, 'msg_invalid_request');
+		if(!$module_srl) {
+			return new Object(-1, 'msg_invalid_request');
+		}
 		// In case of batch configuration of several modules
-		if(preg_match('/^([0-9,]+)$/',$module_srl)) $module_srl = explode(',',$module_srl);
-		else $module_srl = array($module_srl);
+		if(preg_match('/^([0-9,]+)$/',$module_srl)) {
+			$module_srl = explode(',',$module_srl);
+		} else {
+			$module_srl = array($module_srl);
+		}
 		// Save configurations
 		$oModuleController = getController('module');
 		for($i=0;$i<count($module_srl);$i++)
 		{
 			$srl = trim($module_srl[$i]);
-			if(!$srl) continue;
+			if(!$srl) {
+				continue;
+			}
 			unset($config);
 			$config['insert_document'] = (int)Context::get('insert_document');
 			$config['insert_comment'] = (int)Context::get('insert_comment');
@@ -244,29 +264,40 @@ 
 
 		// Get member infomation
 		$output = executeQueryArray('point.getMemberCount');
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		if($output->data)
 		{
 			foreach($output->data as $key => $val)
 			{
-				if(!$val->member_srl) continue;
+				if(!$val->member_srl) {
+					continue;
+				}
 				$member[$val->member_srl] = 0;
 			}
 		}
 
 		// Get post information
 		$output = executeQueryArray('point.getDocumentPoint');
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		if($output->data)
 		{
 			foreach($output->data as $key => $val)
 			{
-				if($module_config[$val->module_srl]['insert_document']) $insert_point = $module_config[$val->module_srl]['insert_document'];
-				else $insert_point = $config->insert_document;
+				if($module_config[$val->module_srl]['insert_document']) {
+					$insert_point = $module_config[$val->module_srl]['insert_document'];
+				} else {
+					$insert_point = $config->insert_document;
+				}
 
-				if(!$val->member_srl) continue;
+				if(!$val->member_srl) {
+					continue;
+				}
 				$point = $insert_point * $val->count;
 				$member[$val->member_srl] += $point;
 			}
@@ -275,16 +306,23 @@ 
 		$output = null;
 		// Get comments information
 		$output = executeQueryArray('point.getCommentPoint');
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		if($output->data)
 		{
 			foreach($output->data as $key => $val)
 			{
-				if($module_config[$val->module_srl]['insert_comment']) $insert_point = $module_config[$val->module_srl]['insert_comment'];
-				else $insert_point = $config->insert_comment;
+				if($module_config[$val->module_srl]['insert_comment']) {
+					$insert_point = $module_config[$val->module_srl]['insert_comment'];
+				} else {
+					$insert_point = $config->insert_comment;
+				}
 
-				if(!$val->member_srl) continue;
+				if(!$val->member_srl) {
+					continue;
+				}
 				$point = $insert_point * $val->count;
 				$member[$val->member_srl] += $point;
 			}
@@ -292,16 +330,23 @@ 
 		$output = null;
 		// Get the attached files' information
 		$output = executeQueryArray('point.getFilePoint');
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		if($output->data)
 		{
 			foreach($output->data as $key => $val)
 			{
-				if($module_config[$val->module_srl]['upload_file']) $insert_point = $module_config[$val->module_srl]['upload_file'];
-				else $insert_point = $config->upload_file;
+				if($module_config[$val->module_srl]['upload_file']) {
+					$insert_point = $module_config[$val->module_srl]['upload_file'];
+				} else {
+					$insert_point = $config->upload_file;
+				}
 
-				if(!$val->member_srl) continue;
+				if(!$val->member_srl) {
+					continue;
+				}
 				$point = $insert_point * $val->count;
 				$member[$val->member_srl] += $point;
 			}
@@ -309,7 +354,9 @@ 
 		$output = null;
 		// Set all members' points to 0
 		$output = executeQuery("point.initMemberPoint");
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 		// Save the file temporarily
 		
 		$str = '';
@@ -334,7 +381,9 @@ 
 		$position = (int)Context::get('position');
 		$total = (int)Context::get('total');
 
-		if(!file_exists('./files/cache/pointRecal.txt')) return new Object(-1, 'msg_invalid_request');
+		if(!file_exists('./files/cache/pointRecal.txt')) {
+			return new Object(-1, 'msg_invalid_request');
+		}
 
 		$idx = 0;
 		$f = fopen("./files/cache/pointRecal.txt","r");
@@ -350,7 +399,9 @@ 
 				$args->member_srl = $member_srl;
 				$args->point = $point;
 				$output = executeQuery('point.insertPoint',$args);
-				if($idx%5000==0) break;
+				if($idx%5000==0) {
+					break;
+				}
 			}
 		}
 
@@ -377,16 +428,23 @@ 
 	function procPointAdminReset()
 	{
 		$module_srl = Context::get('module_srls');
-		if(!$module_srl) return new Object(-1, 'msg_invalid_request');
+		if(!$module_srl) {
+			return new Object(-1, 'msg_invalid_request');
+		}
 		// In case of batch configuration of several modules
-		if(preg_match('/^([0-9,]+)$/',$module_srl)) $module_srl = explode(',',$module_srl);
-		else $module_srl = array($module_srl);
+		if(preg_match('/^([0-9,]+)$/',$module_srl)) {
+			$module_srl = explode(',',$module_srl);
+		} else {
+			$module_srl = array($module_srl);
+		}
 		// Save configurations
 		$oModuleController = getController('module');
 		for($i=0;$i<count($module_srl);$i++)
 		{
 			$srl = trim($module_srl[$i]);
-			if(!$srl) continue;
+			if(!$srl) {
+				continue;
+			}
 			$args = new stdClass();
 			$args->module = 'point';
 			$args->module_srl = $srl;

modules/point/point.admin.view.php

Spacing +6 added lines, -6 removed lines

@@ -20,7 +20,7 @@ 
 
 		//Security
 		$security = new Security();
-		$security->encodeHTML('config.point_name','config.level_icon');
+		$security->encodeHTML('config.point_name', 'config.level_icon');
 		$security->encodeHTML('module_info..');
 
 		// Set the template path
@@ -39,9 +39,9 @@ 
 		$oMemberModel = getModel('member');
 		$group_list = $oMemberModel->getGroups();
 		$selected_group_list = array();
-		if(count($group_list))
+		if (count($group_list))
 		{
-			foreach($group_list as $key => $val)
+			foreach ($group_list as $key => $val)
 			{
 				$selected_group_list[$key] = $val;
 			}
@@ -49,7 +49,7 @@ 
 		Context::set('group_list', $selected_group_list);
 		//Security
 		$security = new Security();
-		$security->encodeHTML('group_list..title','group_list..description');
+		$security->encodeHTML('group_list..title', 'group_list..description');
 
 		// Set the template
 		$this->setTemplateFile('config');
@@ -69,7 +69,7 @@ 
 		Context::set('module_config', $oModuleModel->getModulePartConfigs('point'));
 		//Security
 		$security = new Security();			
-		$security->encodeHTML('mid_list..browser_title','mid_list..mid');
+		$security->encodeHTML('mid_list..browser_title', 'mid_list..mid');
 
 		// Set the template
 		$this->setTemplateFile('module_config');
@@ -115,7 +115,7 @@ 
 		Context::set('group_list', $this->group_list);
 		//Security
 		$security = new Security();			
-		$security->encodeHTML('group_list..title','group_list..description');
+		$security->encodeHTML('group_list..title', 'group_list..description');
 		$security->encodeHTML('member_list..');
 
 		// Set the template

modules/communication/communication.admin.view.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
 		Context::set('group_list', $group_list);
 
 		// specify a template
-		$this->setTemplatePath($this->module_path . 'tpl');
+		$this->setTemplatePath($this->module_path.'tpl');
 		$this->setTemplateFile('index');
 	}
 

modules/addon/addon.controller.php

Spacing +22 added lines, -22 removed lines

@@ -26,7 +26,7 @@ 
 	function getCacheFilePath($type = "pc")
 	{
 		static $addon_file;
-		if(isset($addon_file))
+		if (isset($addon_file))
 		{
 			return $addon_file;
 		}
@@ -34,10 +34,10 @@ 
 		$site_module_info = Context::get('site_module_info');
 		$site_srl = $site_module_info->site_srl;
 
-		$addon_path = _XE_PATH_ . 'files/cache/addons/';
-		$addon_file = $addon_path . $site_srl . $type . '.acivated_addons.cache.php';
+		$addon_path = _XE_PATH_.'files/cache/addons/';
+		$addon_file = $addon_path.$site_srl.$type.'.acivated_addons.cache.php';
 
-		if($this->addon_file_called)
+		if ($this->addon_file_called)
 		{
 			return $addon_file;
 		}
@@ -46,7 +46,7 @@ 
 
 		FileHandler::makeDir($addon_path);
 
-		if(!file_exists($addon_file))
+		if (!file_exists($addon_file))
 		{
 			$this->makeCacheFile($site_srl, $type);
 		}
@@ -82,30 +82,30 @@ 
 		$buff = array('<?php if(!defined("__XE__")) exit();', '$_m = Context::get(\'mid\');');
 		$oAddonModel = getAdminModel('addon');
 		$addon_list = $oAddonModel->getInsertedAddons($site_srl, $gtype);
-		foreach($addon_list as $addon => $val)
+		foreach ($addon_list as $addon => $val)
 		{
-			if($val->addon == "smartphone"
+			if ($val->addon == "smartphone"
 				|| ($type == "pc" && $val->is_used != 'Y') 
 				|| ($type == "mobile" && $val->is_used_m != 'Y') 
 				|| ($gtype == 'global' && $val->is_fixed != 'Y')
-				|| !is_dir(_XE_PATH_ . 'addons/' . $addon))
+				|| !is_dir(_XE_PATH_.'addons/'.$addon))
 			{
 				continue;
 			}
 
 			$extra_vars = unserialize($val->extra_vars);
 			$mid_list = $extra_vars->mid_list;
-			if(!is_array($mid_list) || count($mid_list) < 1)
+			if (!is_array($mid_list) || count($mid_list) < 1)
 			{
 				$mid_list = NULL;
 			}
 
 			$buff[] = '$before_time = microtime(true);';
-			$buff[] = '$rm = \'' . $extra_vars->xe_run_method . "';";
+			$buff[] = '$rm = \''.$extra_vars->xe_run_method."';";
 			$buff[] = '$ml = array(';
-			if($mid_list)
+			if ($mid_list)
 			{
-				foreach($mid_list as $mid)
+				foreach ($mid_list as $mid)
 				{
 					$buff[] = "'$mid' => 1,";
 				}
@@ -113,7 +113,7 @@ 
 			$buff[] = ');';
 			$buff[] = sprintf('$addon_file = \'./addons/%s/%s.addon.php\';', $addon, $addon);
 
-			if($val->extra_vars)
+			if ($val->extra_vars)
 			{
 				unset($extra_vars);
 				$extra_vars = base64_encode($val->extra_vars);
@@ -131,13 +131,13 @@ 
 			$buff[] = '$after_time = microtime(true);';
 			$buff[] = '$addon_time_log = new stdClass();';
 			$buff[] = '$addon_time_log->caller = $called_position;';
-			$buff[] = '$addon_time_log->called = "' . $addon . '";';
-			$buff[] = '$addon_time_log->called_extension = "' . $addon . '";';
+			$buff[] = '$addon_time_log->called = "'.$addon.'";';
+			$buff[] = '$addon_time_log->called_extension = "'.$addon.'";';
 			$buff[] = 'writeSlowlog("addon",$after_time-$before_time,$addon_time_log);';
 		}
-		$addon_path = _XE_PATH_ . 'files/cache/addons/';
+		$addon_path = _XE_PATH_.'files/cache/addons/';
 		FileHandler::makeDir($addon_path);
-		$addon_file = $addon_path . ($gtype == 'site' ? $site_srl : '') . $type . '.acivated_addons.cache.php';
+		$addon_file = $addon_path.($gtype == 'site' ? $site_srl : '').$type.'.acivated_addons.cache.php';
 		FileHandler::writeFile($addon_file, join(PHP_EOL, $buff));
 	}
 
@@ -152,7 +152,7 @@ 
 	 */
 	function doSetup($addon, $extra_vars, $site_srl = 0, $gtype = 'site')
 	{
-		if(!is_array($extra_vars->mid_list))
+		if (!is_array($extra_vars->mid_list))
 		{
 			unset($extra_vars->mid_list);
 		}
@@ -160,7 +160,7 @@ 
 		$args = new stdClass();
 		$args->addon = $addon;
 		$args->extra_vars = serialize($extra_vars);
-		if($gtype == 'global')
+		if ($gtype == 'global')
 		{
 			return executeQuery('addon.updateAddon', $args);
 		}
@@ -176,9 +176,9 @@ 
 	 */
 	function removeAddonConfig($site_srl)
 	{
-		$addon_path = _XE_PATH_ . 'files/cache/addons/';
-		$addon_file = $addon_path . $site_srl . '.acivated_addons.cache.php';
-		if(file_exists($addon_file))
+		$addon_path = _XE_PATH_.'files/cache/addons/';
+		$addon_file = $addon_path.$site_srl.'.acivated_addons.cache.php';
+		if (file_exists($addon_file))
 		{
 			FileHandler::removeFile($addon_file);
 		}

modules/importer/importer.admin.view.php

Spacing +2 added lines, -2 removed lines

@@ -28,7 +28,7 @@ 
 		$this->setTemplatePath($this->module_path.'tpl');
 
 		$source_type = Context::get('source_type');
-		switch($source_type)
+		switch ($source_type)
 		{
 			case 'member' : 
 				$template_filename = "member";
@@ -67,7 +67,7 @@ 
 	 */
 	function dispImporterAdminImportForm()
 	{
-		$oDocumentModel = getModel('document');	//for document lang use in this page
+		$oDocumentModel = getModel('document'); //for document lang use in this page
 
 		$this->setTemplatePath($this->module_path.'tpl');
 		$this->setTemplateFile('index');

modules/trash/trash.class.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 		//$oDB = &DB::getInstance();
 		//$oModuleModel = getModel('module');
 
-		return new Object(0,'success_updated');
+		return new Object(0, 'success_updated');
 	}
 }
 /* End of file trash.class.php */

modules/message/message.admin.view.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 
 		// Get configurations (using module model object)
 		$config = $oModuleModel->getModuleConfig('message');
-		Context::set('config',$config);
+		Context::set('config', $config);
 
 		// Set a template file
 		$this->setTemplatePath($this->module_path.'tpl');