xpressengine / xe-core

modules/member/member.admin.model.php

Braces +45 added lines, -18 removed lines

@@ -63,20 +63,28 @@ 
 			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;
 					$args->html_nick_name = htmlspecialchars($search_keyword, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 					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' :
@@ -129,8 +137,7 @@ 
 		{
 			$query_id = 'member.getMemberListWithinGroup';
 			$args->sort_index = "member.".$sort_index;
-		}
-		else
+		} else
 		{
 			$query_id = 'member.getMemberList';
 			$args->sort_index = $sort_index; 
@@ -174,7 +181,9 @@ 
 	function getSiteAdminMemberSrls()
 	{
 		$output = executeQueryArray('member.getSiteAdminMemberSrls');
-		if(!$output->toBool() || !$output->data) return array();
+		if(!$output->toBool() || !$output->data) {
+			return array();
+		}
 
 		$member_srls = array();
 		foreach($output->data as $member_info)
@@ -193,8 +202,9 @@ 
 	function getMemberAdminColorset()
 	{
 		$skin = Context::get('skin');
-		if(!$skin) $tpl = "";
-		else
+		if(!$skin) {
+			$tpl = "";
+		} else
 		{
 			$oModuleModel = getModel('module');
 			$skin_info = $oModuleModel->loadSkinInfo($this->module_path, $skin);
@@ -202,7 +212,9 @@ 
 
 			$oModuleModel = getModel('module');
 			$config = $oModuleModel->getModuleConfig('member');
-			if(!$config->colorset) $config->colorset = "white";
+			if(!$config->colorset) {
+				$config->colorset = "white";
+			}
 			Context::set('config', $config);
 
 			$oTemplate = &TemplateHandler::getInstance();
@@ -223,10 +235,14 @@ 
 	public function getMemberCountByDate($date = '')
 	{
 		$args = new stdClass();
-		if($date) $args->regDate = date('Ymd', strtotime($date));
+		if($date) {
+			$args->regDate = date('Ymd', strtotime($date));
+		}
 
 		$output = executeQuery('member.getMemberCountByDate', $args);
-		if(!$output->toBool()) return 0;
+		if(!$output->toBool()) {
+			return 0;
+		}
 
 		return $output->data->count;
 	}
@@ -240,10 +256,14 @@ 
 	 */
 	function getMemberGroupMemberCountByDate($date = '')
 	{
-		if($date) $args->regDate = date('Ymd', strtotime($date));
+		if($date) {
+			$args->regDate = date('Ymd', strtotime($date));
+		}
 
 		$output = executeQuery('member.getMemberGroupMemberCountByDate', $args);
-		if(!$output->toBool()) return 0;
+		if(!$output->toBool()) {
+			return 0;
+		}
 
 		return count($output->data);
 	}
@@ -298,10 +318,17 @@ 
 	{
 		$db_info = Context::getDBInfo();
 		$admin_ip_list = $db_info->admin_ip_list;
-		if(!$admin_ip_list) return true;
-		if(!is_array($admin_ip_list)) $admin_ip_list = explode(',',$admin_ip_list);
-		if(!count($admin_ip_list) || IpFilter::filter($admin_ip_list)) return true;
-		else return false;
+		if(!$admin_ip_list) {
+			return true;
+		}
+		if(!is_array($admin_ip_list)) {
+			$admin_ip_list = explode(',',$admin_ip_list);
+		}
+		if(!count($admin_ip_list) || IpFilter::filter($admin_ip_list)) {
+			return true;
+		} else {
+			return false;
+		}
 	}
 }
 /* End of file member.admin.model.php */

Spacing +41 added lines, -41 removed lines

@@ -42,64 +42,64 @@ 
 	{
 		// Search option
 		$args = new stdClass();
-		$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');
 
 		$filter = Context::get('filter_type');
-		switch($filter)
+		switch ($filter)
 		{
-			case 'super_admin' : $args->is_admin = 'Y';break;
-			case 'site_admin' : $args->member_srls = $this->getSiteAdminMemberSrls();break;
-			case 'enable' : $args->is_denied = 'N';break;
-			case 'disable' : $args->is_denied = 'Y';break;
+			case 'super_admin' : $args->is_admin = 'Y'; break;
+			case 'site_admin' : $args->member_srls = $this->getSiteAdminMemberSrls(); break;
+			case 'enable' : $args->is_denied = 'N'; break;
+			case 'disable' : $args->is_denied = 'Y'; break;
 		}
 
 		$search_target = trim(Context::get('search_target'));
 		$search_keyword = trim(Context::get('search_keyword'));
 
-		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;
 					$args->html_nick_name = htmlspecialchars($search_keyword, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 					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' :
-					$args->s_regdate = preg_replace("/[^0-9]/","",$search_keyword);
+					$args->s_regdate = preg_replace("/[^0-9]/", "", $search_keyword);
 					break;
 				case 'regdate_more' :
-					$args->s_regdate_more = substr(preg_replace("/[^0-9]/","",$search_keyword) . '00000000000000',0,14);
+					$args->s_regdate_more = substr(preg_replace("/[^0-9]/", "", $search_keyword).'00000000000000', 0, 14);
 					break;
 				case 'regdate_less' :
-					$args->s_regdate_less = substr(preg_replace("/[^0-9]/","",$search_keyword) . '00000000000000',0,14);
+					$args->s_regdate_less = substr(preg_replace("/[^0-9]/", "", $search_keyword).'00000000000000', 0, 14);
 					break;
 				case 'last_login' :
-					$args->s_last_login = preg_replace("/[^0-9]/","",$search_keyword);
+					$args->s_last_login = preg_replace("/[^0-9]/", "", $search_keyword);
 					//$args->s_last_login = $search_keyword;
 					break;
 				case 'last_login_more' :
-					$args->s_last_login_more = substr(preg_replace("/[^0-9]/","",$search_keyword) . '00000000000000',0,14);
+					$args->s_last_login_more = substr(preg_replace("/[^0-9]/", "", $search_keyword).'00000000000000', 0, 14);
 					break;
 				case 'last_login_less' :
-					$args->s_last_login_less = substr(preg_replace("/[^0-9]/","",$search_keyword) . '00000000000000',0,14);
+					$args->s_last_login_less = substr(preg_replace("/[^0-9]/", "", $search_keyword).'00000000000000', 0, 14);
 					break;
 				case 'birthday' :
-					$args->s_birthday = preg_replace("/[^0-9]/","",$search_keyword);
+					$args->s_birthday = preg_replace("/[^0-9]/", "", $search_keyword);
 					break;
 				case 'extra_vars' :
 					$args->s_extra_vars = $search_keyword;
@@ -110,22 +110,22 @@ 
 		// Change the query id if selected_group_srl exists (for table join)
 		$sort_order = Context::get('sort_order');
 		$sort_index = Context::get('sort_index');
-		if(!$sort_index)
+		if (!$sort_index)
 		{
 			$sort_index = "list_order";
 		}
 
-		if(!$sort_order)
+		if (!$sort_order)
 		{
 			$sort_order = 'asc';
 		}
 
-		if($sort_order != 'asc')
+		if ($sort_order != 'asc')
 		{
 			$sort_order = 'desc';
 		}
 
-		if($args->selected_group_srl)
+		if ($args->selected_group_srl)
 		{
 			$query_id = 'member.getMemberListWithinGroup';
 			$args->sort_index = "member.".$sort_index;
@@ -174,10 +174,10 @@ 
 	function getSiteAdminMemberSrls()
 	{
 		$output = executeQueryArray('member.getSiteAdminMemberSrls');
-		if(!$output->toBool() || !$output->data) return array();
+		if (!$output->toBool() || !$output->data) return array();
 
 		$member_srls = array();
-		foreach($output->data as $member_info)
+		foreach ($output->data as $member_info)
 		{
 			$member_srls[] = $member_info->member_srl;
 		}
@@ -193,7 +193,7 @@ 
 	function getMemberAdminColorset()
 	{
 		$skin = Context::get('skin');
-		if(!$skin) $tpl = "";
+		if (!$skin) $tpl = "";
 		else
 		{
 			$oModuleModel = getModel('module');
@@ -202,7 +202,7 @@ 
 
 			$oModuleModel = getModel('module');
 			$config = $oModuleModel->getModuleConfig('member');
-			if(!$config->colorset) $config->colorset = "white";
+			if (!$config->colorset) $config->colorset = "white";
 			Context::set('config', $config);
 
 			$oTemplate = &TemplateHandler::getInstance();
@@ -223,10 +223,10 @@ 
 	public function getMemberCountByDate($date = '')
 	{
 		$args = new stdClass();
-		if($date) $args->regDate = date('Ymd', strtotime($date));
+		if ($date) $args->regDate = date('Ymd', strtotime($date));
 
 		$output = executeQuery('member.getMemberCountByDate', $args);
-		if(!$output->toBool()) return 0;
+		if (!$output->toBool()) return 0;
 
 		return $output->data->count;
 	}
@@ -240,10 +240,10 @@ 
 	 */
 	function getMemberGroupMemberCountByDate($date = '')
 	{
-		if($date) $args->regDate = date('Ymd', strtotime($date));
+		if ($date) $args->regDate = date('Ymd', strtotime($date));
 
 		$output = executeQuery('member.getMemberGroupMemberCountByDate', $args);
-		if(!$output->toBool()) return 0;
+		if (!$output->toBool()) return 0;
 
 		return count($output->data);
 	}
@@ -261,11 +261,11 @@ 
 		$args->member_join_form_srl = $member_join_form_srl;
 		$output = executeQuery('member.getJoinForm', $args);
 
-		if($output->toBool() && $output->data)
+		if ($output->toBool() && $output->data)
 		{
 			$formInfo = $output->data;
 			$default_value = $formInfo->default_value;
-			if($default_value)
+			if ($default_value)
 			{
 				$default_value = unserialize($default_value);
 				Context::set('default_value', $default_value);
@@ -275,18 +275,18 @@ 
 
 		$oMemberModel = getModel('member');
 		$config = $oMemberModel->getMemberConfig();
-		foreach($config->signupForm as $item) 
+		foreach ($config->signupForm as $item) 
 		{
 			$list[] = $item->name;
 		}
 
-		$id_list = implode(',',$list);
-		Context::set('id_list',$id_list);
+		$id_list = implode(',', $list);
+		Context::set('id_list', $id_list);
 
 		$oTemplate = &TemplateHandler::getInstance();
 		$tpl = $oTemplate->compile($this->module_path.'tpl', 'insert_join_form');
 
-		$this->add('tpl', str_replace("\n"," ",$tpl));
+		$this->add('tpl', str_replace("\n", " ", $tpl));
 	}
 
 	/**
@@ -298,9 +298,9 @@ 
 	{
 		$db_info = Context::getDBInfo();
 		$admin_ip_list = $db_info->admin_ip_list;
-		if(!$admin_ip_list) return true;
-		if(!is_array($admin_ip_list)) $admin_ip_list = explode(',',$admin_ip_list);
-		if(!count($admin_ip_list) || IpFilter::filter($admin_ip_list)) return true;
+		if (!$admin_ip_list) return true;
+		if (!is_array($admin_ip_list)) $admin_ip_list = explode(',', $admin_ip_list);
+		if (!count($admin_ip_list) || IpFilter::filter($admin_ip_list)) return true;
 		else return false;
 	}
 }

modules/member/member.mobile.php

Braces +2 added lines, -4 removed lines

@@ -25,8 +25,7 @@ 
 		{
 			$mskin = 'default';
 			$template_path = sprintf('%sm.skins/%s', $this->module_path, $mskin);
-		}
-		else
+		} else
 		{
 			$template_path = sprintf('%sm.skins/%s', $this->module_path, $mskin);
 		}
@@ -40,8 +39,7 @@ 
 			if(!$this->memberInfo)
 			{
 				Context::set('member_srl','');
-			}
-			else
+			} else
 			{
 				Context::set('member_info',$this->memberInfo);
 			}

Spacing +7 added lines, -7 removed lines

@@ -21,7 +21,7 @@ 
 
 		$mskin = $this->member_config->mskin;
 		// Set the template path
-		if(!$mskin)
+		if (!$mskin)
 		{
 			$mskin = 'default';
 			$template_path = sprintf('%sm.skins/%s', $this->module_path, $mskin);
@@ -33,17 +33,17 @@ 
 
 		// if member_srl exists, set memberInfo
 		$member_srl = Context::get('member_srl');
-		if($member_srl)
+		if ($member_srl)
 		{
 			$oMemberModel = getModel('member');
 			$this->memberInfo = $oMemberModel->getMemberInfoByMemberSrl($member_srl);
-			if(!$this->memberInfo)
+			if (!$this->memberInfo)
 			{
-				Context::set('member_srl','');
+				Context::set('member_srl', '');
 			}
 			else
 			{
-				Context::set('member_info',$this->memberInfo);
+				Context::set('member_info', $this->memberInfo);
 			}
 		}
 
@@ -51,7 +51,7 @@ 
 
 		$oLayoutModel = getModel('layout');
 		$layout_info = $oLayoutModel->getLayout($this->member_config->mlayout_srl);
-		if($layout_info)
+		if ($layout_info)
 		{
 			$this->module_info->mlayout_srl = $this->member_config->mlayout_srl;
 			$this->setLayoutPath($layout_info->path);
@@ -62,7 +62,7 @@ 
 	{
 		parent::dispMemberModifyInfo();
 
-		if($this->member_info)
+		if ($this->member_info)
 		{
 			Context::set('oMemberInfo', get_object_vars($this->member_info));
 		}

modules/menu/menu.admin.view.php

Braces +15 added lines, -8 removed lines

@@ -36,8 +36,11 @@ 
 
 		if(!$site_srl)
 		{
-			if($logged_info->is_admin == 'Y' && !$site_keyword) $site_srl = 0;
-			else $site_srl = (int)$site_module_info->site_srl;
+			if($logged_info->is_admin == 'Y' && !$site_keyword) {
+				$site_srl = 0;
+			} else {
+				$site_srl = (int)$site_module_info->site_srl;
+			}
 		}
 
 		// process for unlinked modules
@@ -51,7 +54,9 @@ 
 
 		$oMenuAdminModel = getAdminModel('menu');
 		$menuListFromDB = $oMenuAdminModel->getMenus();
-		if(is_array($menuListFromDB)) $output = array_reverse($menuListFromDB);
+		if(is_array($menuListFromDB)) {
+			$output = array_reverse($menuListFromDB);
+		}
 
 		$menuList = array();
 		if(is_array($output))
@@ -59,13 +64,16 @@ 
 			$menuItems = array();
 			foreach($output as $key=>$value)
 			{
-				if($value->title == $oAdmin->getAdminMenuName()) unset($output[$key]);
-				else
+				if($value->title == $oAdmin->getAdminMenuName()) {
+					unset($output[$key]);
+				} else
 				{
 					unset($menu, $menuItems);
 					//$value->xml_file = sprintf('./files/cache/menu/%s.xml.php',$value->menu_srl);
 					$value->php_file = sprintf(_XE_PATH_ . 'files/cache/menu/%s.php',$value->menu_srl);
-					if(file_exists($value->php_file)) include($value->php_file);
+					if(file_exists($value->php_file)) {
+						include($value->php_file);
+					}
 
 					if(count($menu->list)>0)
 					{
@@ -102,8 +110,7 @@ 
 		{
 			include($theme_file);
 			Context::set('current_layout', $theme_info->layout);
-		}
-		else
+		} else
 		{
 			$oModuleModel = getModel('module');
 			$default_mid = $oModuleModel->getDefaultMid();

Spacing +23 added lines, -23 removed lines

@@ -34,14 +34,14 @@ 
 		$site_srl = Context::get('site_srl');
 		$site_module_info = Context::get('site_module_info');
 
-		if(!$site_srl)
+		if (!$site_srl)
 		{
-			if($logged_info->is_admin == 'Y' && !$site_keyword) $site_srl = 0;
-			else $site_srl = (int)$site_module_info->site_srl;
+			if ($logged_info->is_admin == 'Y' && !$site_keyword) $site_srl = 0;
+			else $site_srl = (int) $site_module_info->site_srl;
 		}
 
 		// process for unlinked modules
-		if($site_srl == 0)
+		if ($site_srl == 0)
 		{
 			$oMenuController = getAdminController('menu');
 			$oMenuController->linkAllModuleInstancesToSitemap();
@@ -51,25 +51,25 @@ 
 
 		$oMenuAdminModel = getAdminModel('menu');
 		$menuListFromDB = $oMenuAdminModel->getMenus();
-		if(is_array($menuListFromDB)) $output = array_reverse($menuListFromDB);
+		if (is_array($menuListFromDB)) $output = array_reverse($menuListFromDB);
 
 		$menuList = array();
-		if(is_array($output))
+		if (is_array($output))
 		{
 			$menuItems = array();
-			foreach($output as $key=>$value)
+			foreach ($output as $key=>$value)
 			{
-				if($value->title == $oAdmin->getAdminMenuName()) unset($output[$key]);
+				if ($value->title == $oAdmin->getAdminMenuName()) unset($output[$key]);
 				else
 				{
 					unset($menu, $menuItems);
 					//$value->xml_file = sprintf('./files/cache/menu/%s.xml.php',$value->menu_srl);
-					$value->php_file = sprintf(_XE_PATH_ . 'files/cache/menu/%s.php',$value->menu_srl);
-					if(file_exists($value->php_file)) include($value->php_file);
+					$value->php_file = sprintf(_XE_PATH_.'files/cache/menu/%s.php', $value->menu_srl);
+					if (file_exists($value->php_file)) include($value->php_file);
 
-					if(count($menu->list)>0)
+					if (count($menu->list) > 0)
 					{
-						foreach($menu->list as $key2=>$value2)
+						foreach ($menu->list as $key2=>$value2)
 						{
 							$this->_menuInfoSetting($menu->list[$key2]);
 						}
@@ -98,7 +98,7 @@ 
 
 		// choice theme file
 		$theme_file = _XE_PATH_.'files/theme/theme_info.php';
-		if(is_readable($theme_file))
+		if (is_readable($theme_file))
 		{
 			include($theme_file);
 			Context::set('current_layout', $theme_info->layout);
@@ -113,10 +113,10 @@ 
 		// get default group list
 		$oMemberModel = getModel('member');
 		$output = $oMemberModel->getGroups();
-		if(is_array($output))
+		if (is_array($output))
 		{
 			$groupList = array();
-			foreach($output as $value)
+			foreach ($output as $value)
 			{
 				$groupList[$value->group_srl] = new stdClass();
 				$groupList[$value->group_srl]->group_srl = $value->group_srl;
@@ -145,13 +145,13 @@ 
 	function _menuInfoSetting(&$menu)
 	{
 		$oModuleModel = getModel('module');
-		if($menu['url'] && strncasecmp('http', $menu['url'], 4) !== 0)
+		if ($menu['url'] && strncasecmp('http', $menu['url'], 4) !== 0)
 		{
 			unset($midInfo);
 			unset($moduleInfo);
 			$midInfo = $oModuleModel->getModuleInfoByMid($menu['url'], 0);
 			$moduleInfo = $oModuleModel->getModuleInfoXml($midInfo->module);
-			if($moduleInfo->setup_index_act)
+			if ($moduleInfo->setup_index_act)
 			{
 				$menu['module_srl'] = $midInfo->module_srl;
 				$menu['setup_index_act'] = $moduleInfo->setup_index_act;
@@ -159,9 +159,9 @@ 
 			// setting layout srl for layout management
 			$menu['layout_srl'] = $midInfo->layout_srl;
 		}
-		if(count($menu['list']) > 0)
+		if (count($menu['list']) > 0)
 		{
-			foreach($menu['list'] as $key=>$value)
+			foreach ($menu['list'] as $key=>$value)
 			{
 				$this->_menuInfoSetting($menu['list'][$key]);
 			}
@@ -175,17 +175,17 @@ 
 	 */
 	function _arrangeMenuItem($menuItems)
 	{
-		if(is_array($menuItems))
+		if (is_array($menuItems))
 		{
 			$arrangedMenuItemList = array();
-			foreach($menuItems as $value)
+			foreach ($menuItems as $value)
 			{
-				if($value->parent_srl == 0)
+				if ($value->parent_srl == 0)
 				{
 					$arrangedMenuItemList[$value->menu_item_srl] = array('name'=>$value->name, 'subMenu'=>array());
 				}
 
-				if($value->parent_srl > 0 && isset($arrangedMenuItemList[$value->parent_srl]))
+				if ($value->parent_srl > 0 && isset($arrangedMenuItemList[$value->parent_srl]))
 				{
 					$arrangedMenuItemList[$value->parent_srl]['subMenu'][$value->menu_item_srl] = $value;
 				}

modules/message/message.admin.controller.php

Spacing +2 added lines, -2 removed lines

@@ -23,8 +23,8 @@
 		$args = Context::gets('skin', 'mskin', 'colorset', 'mcolorset');
 		// Create a module Controller object
 		$oModuleController = getController('module');
-		$output = $oModuleController->insertModuleConfig('message',$args);
-		if(!$output->toBool()) return $output;
+		$output = $oModuleController->insertModuleConfig('message', $args);
+		if (!$output->toBool()) return $output;
 
 		$this->setMessage('success_updated');
 

Braces +3 added lines, -1 removed lines

@@ -24,7 +24,9 @@
 		// Create a module Controller object
 		$oModuleController = getController('module');
 		$output = $oModuleController->insertModuleConfig('message',$args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->setMessage('success_updated');
 

modules/message/message.admin.model.php

Braces +1 added lines, -2 removed lines

@@ -11,8 +11,7 @@
 		if(!$skin)
 		{
 			$tpl = '';
-		}
-		else
+		} else
 		{
 			$oModuleModel = getModel('module'); /* @var $oModuleModel moduleModel */
 			$skin_info = $oModuleModel->loadSkinInfo($this->module_path, $skin, $dir);

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
-class messageAdminModel extends message{
+class messageAdminModel extends message {
 	public function getMessageAdminColorset()
 	{
 		$skin = Context::get('skin');
@@ -8,7 +8,7 @@ 
 		Context::set('type', $type);
 		$dir = $type == 'P' ? 'skins' : 'm.skins';
 
-		if(!$skin)
+		if (!$skin)
 		{
 			$tpl = '';
 		}

modules/message/message.mobile.php

Braces +9 added lines, -3 removed lines

@@ -18,8 +18,12 @@ 
 		// Get configurations (using module model object)
 		$oModuleModel = getModel('module');
 		$config = $oModuleModel->getModuleConfig('message');
-		if(!is_object($config)) $config = new stdClass;
-		if(!$config->mskin) $config->mskin = 'default';
+		if(!is_object($config)) {
+			$config = new stdClass;
+		}
+		if(!$config->mskin) {
+			$config->mskin = 'default';
+		}
 		// Set the template path
 		$template_path = sprintf('%sm.skins/%s', $this->module_path, $config->mskin);
 		// Get the member configuration
@@ -30,7 +34,9 @@ 
 		$ssl_mode = false;
 		if($member_config->enable_ssl == 'Y')
 		{
-			if(strncasecmp('https://', Context::getRequestUri(), 8) === 0) $ssl_mode = true;
+			if(strncasecmp('https://', Context::getRequestUri(), 8) === 0) {
+				$ssl_mode = true;
+			}
 		}
 		Context::set('ssl_mode',$ssl_mode);
 

Spacing +5 added lines, -5 removed lines

@@ -18,8 +18,8 @@ 
 		// Get configurations (using module model object)
 		$oModuleModel = getModel('module');
 		$config = $oModuleModel->getModuleConfig('message');
-		if(!is_object($config)) $config = new stdClass;
-		if(!$config->mskin) $config->mskin = 'default';
+		if (!is_object($config)) $config = new stdClass;
+		if (!$config->mskin) $config->mskin = 'default';
 		// Set the template path
 		$template_path = sprintf('%sm.skins/%s', $this->module_path, $config->mskin);
 		// Get the member configuration
@@ -28,11 +28,11 @@ 
 		Context::set('member_config', $member_config);
 		// Set a flag to check if the https connection is made when using SSL and create https url 
 		$ssl_mode = false;
-		if($member_config->enable_ssl == 'Y')
+		if ($member_config->enable_ssl == 'Y')
 		{
-			if(strncasecmp('https://', Context::getRequestUri(), 8) === 0) $ssl_mode = true;
+			if (strncasecmp('https://', Context::getRequestUri(), 8) === 0) $ssl_mode = true;
 		}
-		Context::set('ssl_mode',$ssl_mode);
+		Context::set('ssl_mode', $ssl_mode);
 
 		Context::set('system_message', nl2br($this->getMessage()));
 

modules/message/message.view.php

Spacing +5 added lines, -5 removed lines

@@ -23,12 +23,12 @@ 
 		$oModuleModel = getModel('module');
 		$this->module_config = $config = $oModuleModel->getModuleConfig('message', $this->module_info->site_srl);
 
-		if(!$config)
+		if (!$config)
 		{
 			$config = new stdClass();
 		}
 
-		if(!$config->skin)
+		if (!$config->skin)
 		{
 			$config->skin = 'xedition';
 			$template_path = sprintf('%sskins/%s', $this->module_path, $config->skin);
@@ -54,11 +54,11 @@ 
 		Context::set('member_config', $member_config);
 		// Set a flag to check if the https connection is made when using SSL and create https url
 		$ssl_mode = false;
-		if($member_config->enable_ssl == 'Y')
+		if ($member_config->enable_ssl == 'Y')
 		{
-			if(strncasecmp('https://', Context::getRequestUri(), 8) === 0) $ssl_mode = true;
+			if (strncasecmp('https://', Context::getRequestUri(), 8) === 0) $ssl_mode = true;
 		}
-		Context::set('ssl_mode',$ssl_mode);
+		Context::set('ssl_mode', $ssl_mode);
 
 		Context::set('system_message', nl2br($this->getMessage()));
 

Braces +5 added lines, -5 removed lines

@@ -32,16 +32,14 @@ 
 		{
 			$config->skin = 'xedition';
 			$template_path = sprintf('%sskins/%s', $this->module_path, $config->skin);
-		}
-		else
+		} else
 		{
 			//check theme
 			$config_parse = explode('|@|', $config->skin);
 			if (count($config_parse) > 1)
 			{
 				$template_path = sprintf('./themes/%s/modules/message/', $config_parse[0]);
-			}
-			else
+			} else
 			{
 				$template_path = sprintf('%sskins/%s', $this->module_path, $config->skin);
 			}
@@ -56,7 +54,9 @@ 
 		$ssl_mode = false;
 		if($member_config->enable_ssl == 'Y')
 		{
-			if(strncasecmp('https://', Context::getRequestUri(), 8) === 0) $ssl_mode = true;
+			if(strncasecmp('https://', Context::getRequestUri(), 8) === 0) {
+				$ssl_mode = true;
+			}
 		}
 		Context::set('ssl_mode',$ssl_mode);
 

modules/module/module.admin.view.php

Braces +16 added lines, -7 removed lines

@@ -121,8 +121,7 @@ 
 			// Set a template file
 			$this->setTemplateFile('category_update_form');
 			// If not selected, display a list of categories
-		}
-		else
+		} else
 		{
 			$category_list = $oModuleModel->getModuleCategories();
 			Context::set('category_list', $category_list);
@@ -165,7 +164,9 @@ 
 		$module_srls = Context::get('module_srls');
 
 		$modules = explode(',',$module_srls);
-		if(!count($modules)) if(!$module_srls) return new Object(-1,'msg_invalid_request');
+		if(!count($modules)) {
+			if(!$module_srls) return new Object(-1,'msg_invalid_request');
+		}
 
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'module');
@@ -201,7 +202,9 @@ 
 		$module_srls = Context::get('module_srls');
 
 		$modules = explode(',',$module_srls);
-		if(!count($modules)) if(!$module_srls) return new Object(-1,'msg_invalid_request');
+		if(!count($modules)) {
+			if(!$module_srls) return new Object(-1,'msg_invalid_request');
+		}
 		// pre-define variables because you can get contents from other module (call by reference)
 		$content = '';
 		// Call a trigger for additional settings
@@ -224,7 +227,9 @@ 
 		$module_srls = Context::get('module_srls');
 
 		$modules = explode(',',$module_srls);
-		if(!count($modules)) if(!$module_srls) return new Object(-1,'msg_invalid_request');
+		if(!count($modules)) {
+			if(!$module_srls) return new Object(-1,'msg_invalid_request');
+		}
 
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'module', 'site_srl');
@@ -238,8 +243,12 @@ 
 		{
 			foreach($source_grant_list as $key => $val)
 			{
-				if(!$val->default) $val->default = 'guest';
-				if($val->default == 'root') $val->default = 'manager';
+				if(!$val->default) {
+					$val->default = 'guest';
+				}
+				if($val->default == 'root') {
+					$val->default = 'manager';
+				}
 				$grant_list->{$key} = $val;
 			}
 		}

Spacing +23 added lines, -23 removed lines

@@ -35,9 +35,9 @@ 
 		$oAutoinstallModel = getModel('autoinstall');
 
 		$module_list = $oModuleModel->getModuleList();
-		if(is_array($module_list))
+		if (is_array($module_list))
 		{
-			foreach($module_list as $key => $val)
+			foreach ($module_list as $key => $val)
 			{
 				$module_list[$key]->delete_url = $oAutoinstallModel->getRemoveUrlByPath($val->path);
 
@@ -47,7 +47,7 @@ 
 				$module_list[$key]->need_autoinstall_update = $package[$packageSrl]->need_update;
 
 				// get easyinstall update url
-				if($module_list[$key]->need_autoinstall_update == 'Y')
+				if ($module_list[$key]->need_autoinstall_update == 'Y')
 				{
 					$module_list[$key]->update_url = $oAutoinstallModel->getUpdateUrlByPackageSrl($packageSrl);
 				}
@@ -58,9 +58,9 @@ 
 
 		$favoriteList = $output->get('favoriteList');
 		$favoriteModuleList = array();
-		if($favoriteList)
+		if ($favoriteList)
 		{
-			foreach($favoriteList as $favorite => $favorite_info)
+			foreach ($favoriteList as $favorite => $favorite_info)
 			{
 				$favoriteModuleList[] = $favorite_info->module;
 			}
@@ -110,9 +110,9 @@ 
 		//Security
 		$security = new Security();
 
-		if($module_category_srl)
+		if ($module_category_srl)
 		{
-			$selected_category  = $oModuleModel->getModuleCategory($module_category_srl);
+			$selected_category = $oModuleModel->getModuleCategory($module_category_srl);
 			Context::set('selected_category', $selected_category);
 
 			//Security
@@ -164,15 +164,15 @@ 
 	{
 		$module_srls = Context::get('module_srls');
 
-		$modules = explode(',',$module_srls);
-		if(!count($modules)) if(!$module_srls) return new Object(-1,'msg_invalid_request');
+		$modules = explode(',', $module_srls);
+		if (!count($modules)) if (!$module_srls) return new Object(-1, 'msg_invalid_request');
 
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($modules[0], $columnList);
 		// Get a skin list of the module
-		$skin_list = $oModuleModel->getSkins(_XE_PATH_ . 'modules/'.$module_info->module);
-		Context::set('skin_list',$skin_list);
+		$skin_list = $oModuleModel->getSkins(_XE_PATH_.'modules/'.$module_info->module);
+		Context::set('skin_list', $skin_list);
 		// Get a layout list
 		$oLayoutModel = getModel('layout');
 		$layout_list = $oLayoutModel->getLayoutList();
@@ -182,7 +182,7 @@ 
 		Context::set('module_category', $module_category);
 
 		$security = new Security();
-		$security->encodeHTML('layout_list..title','layout_list..layout');
+		$security->encodeHTML('layout_list..title', 'layout_list..layout');
 		$security->encodeHTML('skin_list....');
 		$security->encodeHTML('module_category...');
 
@@ -200,8 +200,8 @@ 
 	{
 		$module_srls = Context::get('module_srls');
 
-		$modules = explode(',',$module_srls);
-		if(!count($modules)) if(!$module_srls) return new Object(-1,'msg_invalid_request');
+		$modules = explode(',', $module_srls);
+		if (!count($modules)) if (!$module_srls) return new Object(-1, 'msg_invalid_request');
 		// pre-define variables because you can get contents from other module (call by reference)
 		$content = '';
 		// Call a trigger for additional settings
@@ -223,8 +223,8 @@ 
 	{
 		$module_srls = Context::get('module_srls');
 
-		$modules = explode(',',$module_srls);
-		if(!count($modules)) if(!$module_srls) return new Object(-1,'msg_invalid_request');
+		$modules = explode(',', $module_srls);
+		if (!count($modules)) if (!$module_srls) return new Object(-1, 'msg_invalid_request');
 
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'module', 'site_srl');
@@ -234,12 +234,12 @@ 
 		// Grant virtual permissions for access and manager
 		$grant_list->access->title = Context::getLang('grant_access');
 		$grant_list->access->default = 'guest';
-		if(count($source_grant_list))
+		if (count($source_grant_list))
 		{
-			foreach($source_grant_list as $key => $val)
+			foreach ($source_grant_list as $key => $val)
 			{
-				if(!$val->default) $val->default = 'guest';
-				if($val->default == 'root') $val->default = 'manager';
+				if (!$val->default) $val->default = 'guest';
+				if ($val->default == 'root') $val->default = 'manager';
 				$grant_list->{$key} = $val;
 			}
 		}
@@ -268,7 +268,7 @@ 
 		// Get the language file of the current site
 		$site_module_info = Context::get('site_module_info');
 		$args = new stdClass();
-		$args->site_srl = (int)$site_module_info->site_srl;
+		$args->site_srl = (int) $site_module_info->site_srl;
 		$args->langCode = Context::get('lang_type');
 		$args->page = Context::get('page'); // /< Page
 		$args->list_count = 30; // /< the number of posts to display on a single page
@@ -287,7 +287,7 @@ 
 		Context::set('lang_code_list', $output->data);
 		Context::set('page_navigation', $output->page_navigation);
 
-		if(Context::get('module') != 'admin')
+		if (Context::get('module') != 'admin')
 		{
 			$this->setLayoutPath('./common/tpl');
 			$this->setLayoutFile('popup_layout');
@@ -301,7 +301,7 @@ 
 		$oModuleModel = getModel('module');
 		$output = $oModuleModel->getModuleFileBoxList();
 		$page = Context::get('page');
-		$page = $page?$page:1;
+		$page = $page ? $page : 1;
 		Context::set('filebox_list', $output->data);
 		Context::set('page_navigation', $output->page_navigation);
 		Context::set('page', $page);

modules/module/module.mobile.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 {
 	function dispModuleChangeLang()
 	{
-		$this->setTemplatePath(sprintf("%stpl/",$this->module_path));
+		$this->setTemplatePath(sprintf("%stpl/", $this->module_path));
 		$this->setTemplateFile('lang.html');
 	}
 }