xpressengine / xe-core

modules/member/member.class.php

Spacing +99 added lines, -99 removed lines

@@ -20,13 +20,13 @@ 
 	 */
 	function member()
 	{
-		if(!Context::isInstalled()) return;
+		if (!Context::isInstalled()) return;
 
 		$oModuleModel = getModel('module');
 		$member_config = $oModuleModel->getModuleConfig('member');
 
 		// Set to use SSL upon actions related member join/information/password and so on. 2013.02.15
-		if(!Context::isExistsSSLAction('dispMemberModifyPassword') && Context::getSslStatus() == 'optional')
+		if (!Context::isExistsSSLAction('dispMemberModifyPassword') && Context::getSslStatus() == 'optional')
 		{
 			$ssl_actions = array('dispMemberModifyPassword', 'dispMemberSignUpForm', 'dispMemberModifyInfo', 'dispMemberModifyEmailAddress', 'dispMemberGetTempPassword', 'dispMemberResendAuthMail', 'dispMemberLoginForm', 'dispMemberFindAccount', 'dispMemberLeave', 'procMemberLogin', 'procMemberModifyPassword', 'procMemberInsert', 'procMemberModifyInfo', 'procMemberFindAccount', 'procMemberModifyEmailAddress', 'procMemberResendAuthMail', 'procMemberLeave'/*, 'getMemberMenu'*/, 'procMemberFindAccountByQuestion');
 			Context::addSSLActions($ssl_actions);
@@ -44,12 +44,12 @@ 
 		$oModuleController = getController('module');
 
 		$oDB = &DB::getInstance();
-		$oDB->addIndex("member_group","idx_site_title", array("site_srl","title"),true);
+		$oDB->addIndex("member_group", "idx_site_title", array("site_srl", "title"), true);
 
 		$oModuleModel = getModel('module');
 		$config = $oModuleModel->getModuleConfig('member');
 
-		if(empty($config))
+		if (empty($config))
 		{
 			$isNotInstall = true;
 			$config = new stdClass;
@@ -58,29 +58,29 @@ 
 		// Set the basic information
 		$config->enable_join = 'Y';
 		$config->enable_openid = 'N';
-		if(!$config->enable_auth_mail) $config->enable_auth_mail = 'N';
-		if(!$config->image_name) $config->image_name = 'Y';
-		if(!$config->image_mark) $config->image_mark = 'Y';
-		if(!$config->profile_image) $config->profile_image = 'Y';
-		if(!$config->image_name_max_width) $config->image_name_max_width = '90';
-		if(!$config->image_name_max_height) $config->image_name_max_height = '20';
-		if(!$config->image_mark_max_width) $config->image_mark_max_width = '20';
-		if(!$config->image_mark_max_height) $config->image_mark_max_height = '20';
-		if(!$config->profile_image_max_width) $config->profile_image_max_width = '90';
-		if(!$config->profile_image_max_height) $config->profile_image_max_height = '90';
-		if($config->group_image_mark!='Y') $config->group_image_mark = 'N';
-		if(!$config->password_strength) $config->password_strength = 'normal';
+		if (!$config->enable_auth_mail) $config->enable_auth_mail = 'N';
+		if (!$config->image_name) $config->image_name = 'Y';
+		if (!$config->image_mark) $config->image_mark = 'Y';
+		if (!$config->profile_image) $config->profile_image = 'Y';
+		if (!$config->image_name_max_width) $config->image_name_max_width = '90';
+		if (!$config->image_name_max_height) $config->image_name_max_height = '20';
+		if (!$config->image_mark_max_width) $config->image_mark_max_width = '20';
+		if (!$config->image_mark_max_height) $config->image_mark_max_height = '20';
+		if (!$config->profile_image_max_width) $config->profile_image_max_width = '90';
+		if (!$config->profile_image_max_height) $config->profile_image_max_height = '90';
+		if ($config->group_image_mark != 'Y') $config->group_image_mark = 'N';
+		if (!$config->password_strength) $config->password_strength = 'normal';
 		
-		if(!$config->password_hashing_algorithm)
+		if (!$config->password_hashing_algorithm)
 		{
 			$oPassword = new Password();
 			$config->password_hashing_algorithm = $oPassword->getBestAlgorithm();
 		}
-		if(!$config->password_hashing_work_factor)
+		if (!$config->password_hashing_work_factor)
 		{
 			$config->password_hashing_work_factor = 8;
 		}
-		if(!$config->password_hashing_auto_upgrade)
+		if (!$config->password_hashing_auto_upgrade)
 		{
 			$config->password_hashing_auto_upgrade = 'Y';
 		}
@@ -91,7 +91,7 @@ 
 		$oMemberController = getController('member');
 		$oMemberAdminController = getAdminController('member');
 
-		if(!$config->signupForm || !is_array($config->signupForm))
+		if (!$config->signupForm || !is_array($config->signupForm))
 		{
 			$identifier = $isNotInstall ? 'email_address' : 'user_id';
 
@@ -106,10 +106,10 @@ 
 			$oMemberAdminController->_createFindAccountByQuestion($config->identifier);
 		}
 		
-		$oModuleController->insertModuleConfig('member',$config);
+		$oModuleController->insertModuleConfig('member', $config);
 
 		$groups = $oMemberModel->getGroups();
-		if(!count($groups))
+		if (!count($groups))
 		{
 			// Set an administrator, regular member(group1), and associate member(group2)
 			$group_args = new stdClass;
@@ -135,10 +135,10 @@ 
 		$admin_args = new stdClass;
 		$admin_args->is_admin = 'Y';
 		$output = executeQuery('member.getMemberList', $admin_args);
-		if(!$output->data)
+		if (!$output->data)
 		{
-			$admin_info = Context::gets('password','nick_name','email_address', 'user_id');
-			if($admin_info->email_address)
+			$admin_info = Context::gets('password', 'nick_name', 'email_address', 'user_id');
+			if ($admin_info->email_address)
 			{
 				$admin_info->user_name = 'admin';
 				// Insert admin information
@@ -150,16 +150,16 @@ 
 		// Register denied ID(default + module name)
 		$oModuleModel = getModel('module');
 		$module_list = $oModuleModel->getModuleList();
-		foreach($module_list as $key => $val)
+		foreach ($module_list as $key => $val)
 		{
-			$oMemberAdminController->insertDeniedID($val->module,'');
+			$oMemberAdminController->insertDeniedID($val->module, '');
 		}
-		$oMemberAdminController->insertDeniedID('www','');
-		$oMemberAdminController->insertDeniedID('root','');
-		$oMemberAdminController->insertDeniedID('administrator','');
-		$oMemberAdminController->insertDeniedID('telnet','');
-		$oMemberAdminController->insertDeniedID('ftp','');
-		$oMemberAdminController->insertDeniedID('http','');
+		$oMemberAdminController->insertDeniedID('www', '');
+		$oMemberAdminController->insertDeniedID('root', '');
+		$oMemberAdminController->insertDeniedID('administrator', '');
+		$oMemberAdminController->insertDeniedID('telnet', '');
+		$oMemberAdminController->insertDeniedID('ftp', '');
+		$oMemberAdminController->insertDeniedID('http', '');
 		// Create cache directory to use in the member module
 		FileHandler::makeDir('./files/member_extra_info/image_name');
 		FileHandler::makeDir('./files/member_extra_info/image_mark');
@@ -184,68 +184,68 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
 			// check member directory (11/08/2007 added)
-			if(!is_dir("./files/member_extra_info")) return true;
+			if (!is_dir("./files/member_extra_info")) return true;
 			// check member directory (22/10/2007 added)
-			if(!is_dir("./files/member_extra_info/profile_image")) return true;
+			if (!is_dir("./files/member_extra_info/profile_image")) return true;
 			// Add a column(is_register) to "member_auth_mail" table (22/04/2008)
 			$act = $oDB->isColumnExists("member_auth_mail", "is_register");
-			if(!$act) return true;
+			if (!$act) return true;
 			// Add a column(site_srl) to "member_group_member" table (11/15/2008)
-			if(!$oDB->isColumnExists("member_group_member", "site_srl")) return true;
-			if(!$oDB->isColumnExists("member_group", "site_srl")) return true;
-			if($oDB->isIndexExists("member_group","uni_member_group_title")) return true;
+			if (!$oDB->isColumnExists("member_group_member", "site_srl")) return true;
+			if (!$oDB->isColumnExists("member_group", "site_srl")) return true;
+			if ($oDB->isIndexExists("member_group", "uni_member_group_title")) return true;
 
 			// Add a column for list_order (05/18/2011)
-			if(!$oDB->isColumnExists("member_group", "list_order")) return true;
+			if (!$oDB->isColumnExists("member_group", "list_order")) return true;
 
 			// image_mark 추가 (2009. 02. 14)
-			if(!$oDB->isColumnExists("member_group", "image_mark")) return true;
+			if (!$oDB->isColumnExists("member_group", "image_mark")) return true;
 			// Add c column for password expiration date
-			if(!$oDB->isColumnExists("member", "change_password_date")) return true;
+			if (!$oDB->isColumnExists("member", "change_password_date")) return true;
 
 			// Add columns of question and answer to verify a password
-			if(!$oDB->isColumnExists("member", "find_account_question")) return true;
-			if(!$oDB->isColumnExists("member", "find_account_answer")) return true;
+			if (!$oDB->isColumnExists("member", "find_account_question")) return true;
+			if (!$oDB->isColumnExists("member", "find_account_answer")) return true;
 
-			if(!$oDB->isColumnExists("member", "list_order")) return true;
-			if(!$oDB->isIndexExists("member","idx_list_order")) return true;
+			if (!$oDB->isColumnExists("member", "list_order")) return true;
+			if (!$oDB->isIndexExists("member", "idx_list_order")) return true;
 
 			$oModuleModel = getModel('module');
 			$config = $oModuleModel->getModuleConfig('member');
 			// check signup form ordering info
-			if(!$config->signupForm) return true;
+			if (!$config->signupForm) return true;
 
 			// check agreement field exist
-			if($config->agreement) return true;
+			if ($config->agreement) return true;
 
-			if($config->skin)
+			if ($config->skin)
 			{
 				$config_parse = explode('.', $config->skin);
-				if(count($config_parse) > 1)
+				if (count($config_parse) > 1)
 				{
 					$template_path = sprintf('./themes/%s/modules/member/', $config_parse[0]);
-					if(is_dir($template_path)) return true;
+					if (is_dir($template_path)) return true;
 				}
 			}
 
 			// supprot multilanguage agreement.
-			if(is_readable('./files/member_extra_info/agreement.txt')) return true;
+			if (is_readable('./files/member_extra_info/agreement.txt')) return true;
 
 			// 2013. 11. 22 add menu when popup document menu called
-			if(!$oModuleModel->getTrigger('document.getDocumentMenu', 'member', 'controller', 'triggerGetDocumentMenu', 'after')) return true;
-			if(!$oModuleModel->getTrigger('comment.getCommentMenu', 'member', 'controller', 'triggerGetCommentMenu', 'after')) return true;
+			if (!$oModuleModel->getTrigger('document.getDocumentMenu', 'member', 'controller', 'triggerGetDocumentMenu', 'after')) return true;
+			if (!$oModuleModel->getTrigger('comment.getCommentMenu', 'member', 'controller', 'triggerGetCommentMenu', 'after')) return true;
 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
 
-		if(!is_readable('./files/ruleset/insertMember.xml')) return true;
-		if(!is_readable('./files/ruleset/login.xml')) return true;
-		if(!is_readable('./files/ruleset/find_member_account_by_question.xml')) return true;
+		if (!is_readable('./files/ruleset/insertMember.xml')) return true;
+		if (!is_readable('./files/ruleset/login.xml')) return true;
+		if (!is_readable('./files/ruleset/find_member_account_by_question.xml')) return true;
 
-		if($oModuleModel->needUpdate('member.1.8.43.recreate_signup_ruleset')) return true;
+		if ($oModuleModel->needUpdate('member.1.8.43.recreate_signup_ruleset')) return true;
 
 		return false;
 	}
@@ -263,7 +263,7 @@ 
 		$oMemberAdminController = getAdminController('member');
 		$config = $oModuleModel->getModuleConfig('member');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
 			// Check member directory
 			FileHandler::makeDir('./files/member_extra_info/image_name');
@@ -271,74 +271,74 @@ 
 			FileHandler::makeDir('./files/member_extra_info/signature');
 			FileHandler::makeDir('./files/member_extra_info/profile_image');
 			// Add a column
-			if(!$oDB->isColumnExists("member_auth_mail", "is_register"))
+			if (!$oDB->isColumnExists("member_auth_mail", "is_register"))
 			{
 				$oDB->addColumn("member_auth_mail", "is_register", "char", 1, "N", true);
 			}
 			// Add a column(site_srl) to "member_group_member" table (11/15/2008)
-			if(!$oDB->isColumnExists("member_group_member", "site_srl"))
+			if (!$oDB->isColumnExists("member_group_member", "site_srl"))
 			{
 				$oDB->addColumn("member_group_member", "site_srl", "number", 11, 0, true);
 				$oDB->addIndex("member_group_member", "idx_site_srl", "site_srl", false);
 			}
-			if(!$oDB->isColumnExists("member_group", "site_srl"))
+			if (!$oDB->isColumnExists("member_group", "site_srl"))
 			{
 				$oDB->addColumn("member_group", "site_srl", "number", 11, 0, true);
-				$oDB->addIndex("member_group","idx_site_title", array("site_srl","title"),true);
+				$oDB->addIndex("member_group", "idx_site_title", array("site_srl", "title"), true);
 			}
-			if($oDB->isIndexExists("member_group","uni_member_group_title"))
+			if ($oDB->isIndexExists("member_group", "uni_member_group_title"))
 			{
-				$oDB->dropIndex("member_group","uni_member_group_title",true);
+				$oDB->dropIndex("member_group", "uni_member_group_title", true);
 			}
 
 			// Add a column(list_order) to "member_group" table (05/18/2011)
-			if(!$oDB->isColumnExists("member_group", "list_order"))
+			if (!$oDB->isColumnExists("member_group", "list_order"))
 			{
 				$oDB->addColumn("member_group", "list_order", "number", 11, '', true);
-				$oDB->addIndex("member_group","idx_list_order", "list_order",false);
+				$oDB->addIndex("member_group", "idx_list_order", "list_order", false);
 				$output = executeQuery('member.updateAllMemberGroupListOrder');
 			}
 			// Add a column for image_mark (02/14/2009)
-			if(!$oDB->isColumnExists("member_group", "image_mark"))
+			if (!$oDB->isColumnExists("member_group", "image_mark"))
 			{
 				$oDB->addColumn("member_group", "image_mark", "text");
 			}
 			// Add a column for password expiration date
-			if(!$oDB->isColumnExists("member", "change_password_date"))
+			if (!$oDB->isColumnExists("member", "change_password_date"))
 			{
 				$oDB->addColumn("member", "change_password_date", "date");
 				executeQuery('member.updateAllChangePasswordDate');
 			}
 
 			// Add columns of question and answer to verify a password
-			if(!$oDB->isColumnExists("member", "find_account_question"))
+			if (!$oDB->isColumnExists("member", "find_account_question"))
 			{
 				$oDB->addColumn("member", "find_account_question", "number", 11);
 			}
-			if(!$oDB->isColumnExists("member", "find_account_answer"))
+			if (!$oDB->isColumnExists("member", "find_account_answer"))
 			{
 				$oDB->addColumn("member", "find_account_answer", "varchar", 250);
 			}
 
-			if(!$oDB->isColumnExists("member", "list_order"))
+			if (!$oDB->isColumnExists("member", "list_order"))
 			{
 				$oDB->addColumn("member", "list_order", "number", 11);
 				@set_time_limit(0);
 				$args->list_order = 'member_srl';
-				executeQuery('member.updateMemberListOrderAll',$args);
+				executeQuery('member.updateMemberListOrderAll', $args);
 				executeQuery('member.updateMemberListOrderAll');
 			}
-			if(!$oDB->isIndexExists("member","idx_list_order"))
+			if (!$oDB->isIndexExists("member", "idx_list_order"))
 			{
-				$oDB->addIndex("member","idx_list_order", array("list_order"));
+				$oDB->addIndex("member", "idx_list_order", array("list_order"));
 			}
 
 			$config = $oModuleModel->getModuleConfig('member');
 
 			// check agreement value exist
-			if($config->agreement)
+			if ($config->agreement)
 			{
-				$agreement_file = _XE_PATH_.'files/member_extra_info/agreement_' . Context::get('lang_type') . '.txt';
+				$agreement_file = _XE_PATH_.'files/member_extra_info/agreement_'.Context::get('lang_type').'.txt';
 				$output = FileHandler::writeFile($agreement_file, $config->agreement);
 
 				$config->agreement = NULL;
@@ -346,7 +346,7 @@ 
 			}
 
 			// check signup form ordering info
-			if(!$config->signupForm || !is_array($config->signupForm))
+			if (!$config->signupForm || !is_array($config->signupForm))
 			{
 				$identifier = 'user_id';
 
@@ -356,13 +356,13 @@ 
 				$output = $oModuleController->updateModuleConfig('member', $config);
 			}
 
-			if($config->skin)
+			if ($config->skin)
 			{
 				$config_parse = explode('.', $config->skin);
 				if (count($config_parse) > 1)
 				{
 					$template_path = sprintf('./themes/%s/modules/member/', $config_parse[0]);
-					if(is_dir($template_path))
+					if (is_dir($template_path))
 					{
 						$config->skin = implode('|@|', $config_parse);
 						$oModuleController = getController('module');
@@ -372,15 +372,15 @@ 
 			}
 			
 			// 2013. 11. 22 add menu when popup document menu called
-			if(!$oModuleModel->getTrigger('document.getDocumentMenu', 'member', 'controller', 'triggerGetDocumentMenu', 'after'))
+			if (!$oModuleModel->getTrigger('document.getDocumentMenu', 'member', 'controller', 'triggerGetDocumentMenu', 'after'))
 				$oModuleController->insertTrigger('document.getDocumentMenu', 'member', 'controller', 'triggerGetDocumentMenu', 'after');
-			if(!$oModuleModel->getTrigger('comment.getCommentMenu', 'member', 'controller', 'triggerGetCommentMenu', 'after'))
+			if (!$oModuleModel->getTrigger('comment.getCommentMenu', 'member', 'controller', 'triggerGetCommentMenu', 'after'))
 				$oModuleController->insertTrigger('comment.getCommentMenu', 'member', 'controller', 'triggerGetCommentMenu', 'after');
 
-			if(is_readable('./files/member_extra_info/agreement.txt'))
+			if (is_readable('./files/member_extra_info/agreement.txt'))
 			{
 				$source_file = _XE_PATH_.'files/member_extra_info/agreement.txt';
-				$target_file = _XE_PATH_.'files/member_extra_info/agreement_' . Context::get('lang_type') . '.txt';
+				$target_file = _XE_PATH_.'files/member_extra_info/agreement_'.Context::get('lang_type').'.txt';
 
 				FileHandler::rename($source_file, $target_file);
 			}
@@ -389,14 +389,14 @@ 
 		}
 
 		FileHandler::makeDir('./files/ruleset');
-		if(!is_readable('./files/ruleset/insertMember.xml'))
+		if (!is_readable('./files/ruleset/insertMember.xml'))
 			$oMemberAdminController->_createSignupRuleset($config->signupForm);
-		if(!is_readable('./files/ruleset/login.xml'))
+		if (!is_readable('./files/ruleset/login.xml'))
 			$oMemberAdminController->_createLoginRuleset($config->identifier);
-		if(!is_readable('./files/ruleset/find_member_account_by_question.xml'))
+		if (!is_readable('./files/ruleset/find_member_account_by_question.xml'))
 			$oMemberAdminController->_createFindAccountByQuestion($config->identifier);
 
-		if($oModuleModel->needUpdate('member.1.8.43.recreate_signup_ruleset'))
+		if ($oModuleModel->needUpdate('member.1.8.43.recreate_signup_ruleset'))
 		{
 			$oMemberAdminController->_createSignupRuleset($config->signupForm);
 			$oModuleController->insertUpdatedLog('member.1.8.43.recreate_signup_ruleset');
@@ -419,7 +419,7 @@ 
 	 */
 	function recordLoginError($error = 0, $message = 'success')
 	{
-		if($error == 0) return new Object($error, $message);
+		if ($error == 0) return new Object($error, $message);
 
 		// Create a member model object
 		$oMemberModel = getModel('member');
@@ -427,18 +427,18 @@ 
 
 		// Check if there is recoding table.
 		$oDB = &DB::getInstance();
-		if(!$oDB->isTableExists('member_login_count') || $config->enable_login_fail_report == 'N') return new Object($error, $message);
+		if (!$oDB->isTableExists('member_login_count') || $config->enable_login_fail_report == 'N') return new Object($error, $message);
 
 		$args = new stdClass();
 		$args->ipaddress = $_SERVER['REMOTE_ADDR'];
 
 		$output = executeQuery('member.getLoginCountByIp', $args);
-		if($output->data && $output->data->count)
+		if ($output->data && $output->data->count)
 		{
 			$last_update = strtotime($output->data->last_update);
-			$term = intval($_SERVER['REQUEST_TIME']-$last_update);
+			$term = intval($_SERVER['REQUEST_TIME'] - $last_update);
 			//update, if IP address access in a short time, update count. If not, make count 1.
-			if($term < $config->max_error_count_time)
+			if ($term < $config->max_error_count_time)
 			{
 				$args->count = $output->data->count + 1;
 			}
@@ -464,7 +464,7 @@ 
 	 */
 	function recordMemberLoginError($error = 0, $message = 'success', $args = NULL)
 	{
-		if($error == 0 || !$args->member_srl) return new Object($error, $message);
+		if ($error == 0 || !$args->member_srl) return new Object($error, $message);
 
 		// Create a member model object
 		$oMemberModel = getModel('member');
@@ -472,21 +472,21 @@ 
 
 		// Check if there is recoding table.
 		$oDB = &DB::getInstance();
-		if(!$oDB->isTableExists('member_count_history') || $config->enable_login_fail_report == 'N') return new Object($error, $message);
+		if (!$oDB->isTableExists('member_count_history') || $config->enable_login_fail_report == 'N') return new Object($error, $message);
 
 		$output = executeQuery('member.getLoginCountHistoryByMemberSrl', $args);
-		if($output->data && $output->data->content)
+		if ($output->data && $output->data->content)
 		{
 			//update
 			$content = unserialize($output->data->content);
-			$content[] = array($_SERVER['REMOTE_ADDR'],Context::getLang($message),$_SERVER['REQUEST_TIME']);
+			$content[] = array($_SERVER['REMOTE_ADDR'], Context::getLang($message), $_SERVER['REQUEST_TIME']);
 			$args->content = serialize($content);
 			$output = executeQuery('member.updateLoginCountHistoryByMemberSrl', $args);
 		}
 		else
 		{
 			//insert
-			$content[0] = array($_SERVER['REMOTE_ADDR'],Context::getLang($message),$_SERVER['REQUEST_TIME']);
+			$content[0] = array($_SERVER['REMOTE_ADDR'], Context::getLang($message), $_SERVER['REQUEST_TIME']);
 			$args->content = serialize($content);
 			$output = executeQuery('member.insertLoginCountHistoryByMemberSrl', $args);
 		}

Braces +138 added lines, -56 removed lines

@@ -20,7 +20,9 @@ 
 	 */
 	function member()
 	{
-		if(!Context::isInstalled()) return;
+		if(!Context::isInstalled()) {
+			return;
+		}
 
 		$oModuleModel = getModel('module');
 		$member_config = $oModuleModel->getModuleConfig('member');
@@ -58,18 +60,42 @@ 
 		// Set the basic information
 		$config->enable_join = 'Y';
 		$config->enable_openid = 'N';
-		if(!$config->enable_auth_mail) $config->enable_auth_mail = 'N';
-		if(!$config->image_name) $config->image_name = 'Y';
-		if(!$config->image_mark) $config->image_mark = 'Y';
-		if(!$config->profile_image) $config->profile_image = 'Y';
-		if(!$config->image_name_max_width) $config->image_name_max_width = '90';
-		if(!$config->image_name_max_height) $config->image_name_max_height = '20';
-		if(!$config->image_mark_max_width) $config->image_mark_max_width = '20';
-		if(!$config->image_mark_max_height) $config->image_mark_max_height = '20';
-		if(!$config->profile_image_max_width) $config->profile_image_max_width = '90';
-		if(!$config->profile_image_max_height) $config->profile_image_max_height = '90';
-		if($config->group_image_mark!='Y') $config->group_image_mark = 'N';
-		if(!$config->password_strength) $config->password_strength = 'normal';
+		if(!$config->enable_auth_mail) {
+			$config->enable_auth_mail = 'N';
+		}
+		if(!$config->image_name) {
+			$config->image_name = 'Y';
+		}
+		if(!$config->image_mark) {
+			$config->image_mark = 'Y';
+		}
+		if(!$config->profile_image) {
+			$config->profile_image = 'Y';
+		}
+		if(!$config->image_name_max_width) {
+			$config->image_name_max_width = '90';
+		}
+		if(!$config->image_name_max_height) {
+			$config->image_name_max_height = '20';
+		}
+		if(!$config->image_mark_max_width) {
+			$config->image_mark_max_width = '20';
+		}
+		if(!$config->image_mark_max_height) {
+			$config->image_mark_max_height = '20';
+		}
+		if(!$config->profile_image_max_width) {
+			$config->profile_image_max_width = '90';
+		}
+		if(!$config->profile_image_max_height) {
+			$config->profile_image_max_height = '90';
+		}
+		if($config->group_image_mark!='Y') {
+			$config->group_image_mark = 'N';
+		}
+		if(!$config->password_strength) {
+			$config->password_strength = 'normal';
+		}
 		
 		if(!$config->password_hashing_algorithm)
 		{
@@ -187,39 +213,69 @@ 
 		if($oModuleModel->needUpdate($version_update_id))
 		{
 			// check member directory (11/08/2007 added)
-			if(!is_dir("./files/member_extra_info")) return true;
+			if(!is_dir("./files/member_extra_info")) {
+				return true;
+			}
 			// check member directory (22/10/2007 added)
-			if(!is_dir("./files/member_extra_info/profile_image")) return true;
+			if(!is_dir("./files/member_extra_info/profile_image")) {
+				return true;
+			}
 			// Add a column(is_register) to "member_auth_mail" table (22/04/2008)
 			$act = $oDB->isColumnExists("member_auth_mail", "is_register");
-			if(!$act) return true;
+			if(!$act) {
+				return true;
+			}
 			// Add a column(site_srl) to "member_group_member" table (11/15/2008)
-			if(!$oDB->isColumnExists("member_group_member", "site_srl")) return true;
-			if(!$oDB->isColumnExists("member_group", "site_srl")) return true;
-			if($oDB->isIndexExists("member_group","uni_member_group_title")) return true;
+			if(!$oDB->isColumnExists("member_group_member", "site_srl")) {
+				return true;
+			}
+			if(!$oDB->isColumnExists("member_group", "site_srl")) {
+				return true;
+			}
+			if($oDB->isIndexExists("member_group","uni_member_group_title")) {
+				return true;
+			}
 
 			// Add a column for list_order (05/18/2011)
-			if(!$oDB->isColumnExists("member_group", "list_order")) return true;
+			if(!$oDB->isColumnExists("member_group", "list_order")) {
+				return true;
+			}
 
 			// image_mark 추가 (2009. 02. 14)
-			if(!$oDB->isColumnExists("member_group", "image_mark")) return true;
+			if(!$oDB->isColumnExists("member_group", "image_mark")) {
+				return true;
+			}
 			// Add c column for password expiration date
-			if(!$oDB->isColumnExists("member", "change_password_date")) return true;
+			if(!$oDB->isColumnExists("member", "change_password_date")) {
+				return true;
+			}
 
 			// Add columns of question and answer to verify a password
-			if(!$oDB->isColumnExists("member", "find_account_question")) return true;
-			if(!$oDB->isColumnExists("member", "find_account_answer")) return true;
+			if(!$oDB->isColumnExists("member", "find_account_question")) {
+				return true;
+			}
+			if(!$oDB->isColumnExists("member", "find_account_answer")) {
+				return true;
+			}
 
-			if(!$oDB->isColumnExists("member", "list_order")) return true;
-			if(!$oDB->isIndexExists("member","idx_list_order")) return true;
+			if(!$oDB->isColumnExists("member", "list_order")) {
+				return true;
+			}
+			if(!$oDB->isIndexExists("member","idx_list_order")) {
+				return true;
+			}
 
 			$oModuleModel = getModel('module');
 			$config = $oModuleModel->getModuleConfig('member');
 			// check signup form ordering info
-			if(!$config->signupForm) return true;
+			if(!$config->signupForm) {
+				return true;
+			}
 
 			// check agreement field exist
-			if($config->agreement) return true;
+			if($config->agreement) {
+				return true;
+			}
 
 			if($config->skin)
 			{
@@ -227,25 +283,41 @@ 
 				if(count($config_parse) > 1)
 				{
 					$template_path = sprintf('./themes/%s/modules/member/', $config_parse[0]);
-					if(is_dir($template_path)) return true;
+					if(is_dir($template_path)) {
+						return true;
+					}
 				}
 			}
 
 			// supprot multilanguage agreement.
-			if(is_readable('./files/member_extra_info/agreement.txt')) return true;
+			if(is_readable('./files/member_extra_info/agreement.txt')) {
+				return true;
+			}
 
 			// 2013. 11. 22 add menu when popup document menu called
-			if(!$oModuleModel->getTrigger('document.getDocumentMenu', 'member', 'controller', 'triggerGetDocumentMenu', 'after')) return true;
-			if(!$oModuleModel->getTrigger('comment.getCommentMenu', 'member', 'controller', 'triggerGetCommentMenu', 'after')) return true;
+			if(!$oModuleModel->getTrigger('document.getDocumentMenu', 'member', 'controller', 'triggerGetDocumentMenu', 'after')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('comment.getCommentMenu', 'member', 'controller', 'triggerGetCommentMenu', 'after')) {
+				return true;
+			}
 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
 
-		if(!is_readable('./files/ruleset/insertMember.xml')) return true;
-		if(!is_readable('./files/ruleset/login.xml')) return true;
-		if(!is_readable('./files/ruleset/find_member_account_by_question.xml')) return true;
+		if(!is_readable('./files/ruleset/insertMember.xml')) {
+			return true;
+		}
+		if(!is_readable('./files/ruleset/login.xml')) {
+			return true;
+		}
+		if(!is_readable('./files/ruleset/find_member_account_by_question.xml')) {
+			return true;
+		}
 
-		if($oModuleModel->needUpdate('member.1.8.43.recreate_signup_ruleset')) return true;
+		if($oModuleModel->needUpdate('member.1.8.43.recreate_signup_ruleset')) {
+			return true;
+		}
 
 		return false;
 	}
@@ -372,10 +444,12 @@ 
 			}
 			
 			// 2013. 11. 22 add menu when popup document menu called
-			if(!$oModuleModel->getTrigger('document.getDocumentMenu', 'member', 'controller', 'triggerGetDocumentMenu', 'after'))
-				$oModuleController->insertTrigger('document.getDocumentMenu', 'member', 'controller', 'triggerGetDocumentMenu', 'after');
-			if(!$oModuleModel->getTrigger('comment.getCommentMenu', 'member', 'controller', 'triggerGetCommentMenu', 'after'))
-				$oModuleController->insertTrigger('comment.getCommentMenu', 'member', 'controller', 'triggerGetCommentMenu', 'after');
+			if(!$oModuleModel->getTrigger('document.getDocumentMenu', 'member', 'controller', 'triggerGetDocumentMenu', 'after')) {
+							$oModuleController->insertTrigger('document.getDocumentMenu', 'member', 'controller', 'triggerGetDocumentMenu', 'after');
+			}
+			if(!$oModuleModel->getTrigger('comment.getCommentMenu', 'member', 'controller', 'triggerGetCommentMenu', 'after')) {
+							$oModuleController->insertTrigger('comment.getCommentMenu', 'member', 'controller', 'triggerGetCommentMenu', 'after');
+			}
 
 			if(is_readable('./files/member_extra_info/agreement.txt'))
 			{
@@ -389,12 +463,15 @@ 
 		}
 
 		FileHandler::makeDir('./files/ruleset');
-		if(!is_readable('./files/ruleset/insertMember.xml'))
-			$oMemberAdminController->_createSignupRuleset($config->signupForm);
-		if(!is_readable('./files/ruleset/login.xml'))
-			$oMemberAdminController->_createLoginRuleset($config->identifier);
-		if(!is_readable('./files/ruleset/find_member_account_by_question.xml'))
-			$oMemberAdminController->_createFindAccountByQuestion($config->identifier);
+		if(!is_readable('./files/ruleset/insertMember.xml')) {
+					$oMemberAdminController->_createSignupRuleset($config->signupForm);
+		}
+		if(!is_readable('./files/ruleset/login.xml')) {
+					$oMemberAdminController->_createLoginRuleset($config->identifier);
+		}
+		if(!is_readable('./files/ruleset/find_member_account_by_question.xml')) {
+					$oMemberAdminController->_createFindAccountByQuestion($config->identifier);
+		}
 
 		if($oModuleModel->needUpdate('member.1.8.43.recreate_signup_ruleset'))
 		{
@@ -419,7 +496,9 @@ 
 	 */
 	function recordLoginError($error = 0, $message = 'success')
 	{
-		if($error == 0) return new Object($error, $message);
+		if($error == 0) {
+			return new Object($error, $message);
+		}
 
 		// Create a member model object
 		$oMemberModel = getModel('member');
@@ -427,7 +506,9 @@ 
 
 		// Check if there is recoding table.
 		$oDB = &DB::getInstance();
-		if(!$oDB->isTableExists('member_login_count') || $config->enable_login_fail_report == 'N') return new Object($error, $message);
+		if(!$oDB->isTableExists('member_login_count') || $config->enable_login_fail_report == 'N') {
+			return new Object($error, $message);
+		}
 
 		$args = new stdClass();
 		$args->ipaddress = $_SERVER['REMOTE_ADDR'];
@@ -441,16 +522,14 @@ 
 			if($term < $config->max_error_count_time)
 			{
 				$args->count = $output->data->count + 1;
-			}
-			else
+			} else
 			{
 				$args->count = 1;
 			}
 			unset($oMemberModel);
 			unset($config);
 			$output = executeQuery('member.updateLoginCountByIp', $args);
-		}
-		else
+		} else
 		{
 			//insert
 			$args->count = 1;
@@ -464,7 +543,9 @@ 
 	 */
 	function recordMemberLoginError($error = 0, $message = 'success', $args = NULL)
 	{
-		if($error == 0 || !$args->member_srl) return new Object($error, $message);
+		if($error == 0 || !$args->member_srl) {
+			return new Object($error, $message);
+		}
 
 		// Create a member model object
 		$oMemberModel = getModel('member');
@@ -472,7 +553,9 @@ 
 
 		// Check if there is recoding table.
 		$oDB = &DB::getInstance();
-		if(!$oDB->isTableExists('member_count_history') || $config->enable_login_fail_report == 'N') return new Object($error, $message);
+		if(!$oDB->isTableExists('member_count_history') || $config->enable_login_fail_report == 'N') {
+			return new Object($error, $message);
+		}
 
 		$output = executeQuery('member.getLoginCountHistoryByMemberSrl', $args);
 		if($output->data && $output->data->content)
@@ -482,8 +565,7 @@ 
 			$content[] = array($_SERVER['REMOTE_ADDR'],Context::getLang($message),$_SERVER['REQUEST_TIME']);
 			$args->content = serialize($content);
 			$output = executeQuery('member.updateLoginCountHistoryByMemberSrl', $args);
-		}
-		else
+		} else
 		{
 			//insert
 			$content[0] = array($_SERVER['REMOTE_ADDR'],Context::getLang($message),$_SERVER['REQUEST_TIME']);

modules/member/member.admin.view.php

Spacing +72 added lines, -72 removed lines

@@ -43,16 +43,16 @@ 
 
 		// if member_srl exists, set memberInfo
 		$member_srl = Context::get('member_srl');
-		if($member_srl) 
+		if ($member_srl) 
 		{
 			$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);
 			}
 		}
 
@@ -79,30 +79,30 @@ 
 
 		$filter = Context::get('filter_type');
 		global $lang;
-		switch($filter)
+		switch ($filter)
 		{
-			case 'super_admin' : Context::set('filter_type_title', $lang->cmd_show_super_admin_member);break;
-			case 'site_admin' : Context::set('filter_type_title', $lang->cmd_show_site_admin_member);break;
-			default : Context::set('filter_type_title', $lang->cmd_show_all_member);break;
+			case 'super_admin' : Context::set('filter_type_title', $lang->cmd_show_super_admin_member); break;
+			case 'site_admin' : Context::set('filter_type_title', $lang->cmd_show_site_admin_member); break;
+			default : Context::set('filter_type_title', $lang->cmd_show_all_member); break;
 		}
 		// retrieve list of groups for each member
-		if($output->data)
+		if ($output->data)
 		{
-			foreach($output->data as $key => $member)
+			foreach ($output->data as $key => $member)
 			{
-				$output->data[$key]->group_list = $oMemberModel->getMemberGroups($member->member_srl,0);
+				$output->data[$key]->group_list = $oMemberModel->getMemberGroups($member->member_srl, 0);
 			}
 		}
 		$config = $this->memberConfig;
 		$memberIdentifiers = array('user_id'=>'user_id', 'user_name'=>'user_name', 'nick_name'=>'nick_name');
 		$usedIdentifiers = array();	
 
-		if(is_array($config->signupForm))
+		if (is_array($config->signupForm))
 		{
-			foreach($config->signupForm as $signupItem)
+			foreach ($config->signupForm as $signupItem)
 			{
-				if(!count($memberIdentifiers)) break;
-				if(in_array($signupItem->name, $memberIdentifiers) && ($signupItem->required || $signupItem->isUse))
+				if (!count($memberIdentifiers)) break;
+				if (in_array($signupItem->name, $memberIdentifiers) && ($signupItem->required || $signupItem->isUse))
 				{
 					unset($memberIdentifiers[$signupItem->name]);
 					$usedIdentifiers[$signupItem->name] = $lang->{$signupItem->name};
@@ -140,14 +140,14 @@ 
 	{
 		$config = $this->memberConfig;
 
-		if($config->redirect_url)
+		if ($config->redirect_url)
 		{
 			$mid = str_ireplace(Context::getDefaultUrl(), '', $config->redirect_url);
 
 			$siteModuleInfo = Context::get('site_module_info');
 
 			$oModuleModel = getModel('module');
-			$moduleInfo = $oModuleModel->getModuleInfoByMid($mid, (int)$siteModuleInfo->site_srl);
+			$moduleInfo = $oModuleModel->getModuleInfoByMid($mid, (int) $siteModuleInfo->site_srl);
 
 			$config->redirect_url = $moduleInfo->module_srl;
 			Context::set('config', $config);
@@ -173,9 +173,9 @@ 
 		Context::set('editor', $editor);
 
 		$signupForm = $config->signupForm;
-		foreach($signupForm as $val)
+		foreach ($signupForm as $val)
 		{
-			if($val->name == 'user_id')
+			if ($val->name == 'user_id')
 			{
 				$userIdInfo = $val;
 				break;
@@ -183,7 +183,7 @@ 
 		}
 
 		$oSecurity = new Security();
-		if($userIdInfo->isUse)
+		if ($userIdInfo->isUse)
 		{
 			// get denied ID list
 			Context::set('useUserID', 1);
@@ -274,16 +274,16 @@ 
 		Context::set('editor', $editor);
 
 		$signupForm = $config->signupForm;
-		foreach($signupForm as $val)
+		foreach ($signupForm as $val)
 		{
-			if($val->name == 'user_id')
+			if ($val->name == 'user_id')
 			{
 				$userIdInfo = $val;
 				break;
 			}
 		}
 
-		if($userIdInfo->isUse)
+		if ($userIdInfo->isUse)
 		{
 			// get denied ID list
 			Context::set('useUserID', 1);
@@ -344,14 +344,14 @@ 
 		$oMemberModel = getModel('member');
 
 		$memberInfo = Context::get('member_info');
-		if(isset($memberInfo))
+		if (isset($memberInfo))
 		{
 			$memberInfo->signature = $oMemberModel->getSignature($this->memberInfo->member_srl);
 		}
 		Context::set('member_info', $memberInfo);
 
 		// get an editor for the signature
-		if($memberInfo->member_srl)
+		if ($memberInfo->member_srl)
 		{
 			$oEditorModel = getModel('editor');
 			$option = new stdClass();
@@ -405,7 +405,7 @@ 
 		}
 
 		$member_config = $this->memberConfig;
-		if(!$this->memberConfig)
+		if (!$this->memberConfig)
 		{
 			$member_config = $this->memberConfig = $oMemberModel->getMemberConfig();
 		}
@@ -416,25 +416,25 @@ 
 		$formTags = array();
 		global $lang;
 
-		foreach($member_config->signupForm as $no=>$formInfo)
+		foreach ($member_config->signupForm as $no=>$formInfo)
 		{
-			if(!$formInfo->isUse)continue;
+			if (!$formInfo->isUse)continue;
 
 			// 회원 본인이 아닌 경우 입력 폼 제거
-			if($formInfo->name == 'find_account_question' && $memberInfo['member_srl'] !== $logged_info->member_srl)
+			if ($formInfo->name == 'find_account_question' && $memberInfo['member_srl'] !== $logged_info->member_srl)
 			{
 				unset($member_config->signupForm[$no]);
 				continue;
 			}
 
-			if($formInfo->name == $member_config->identifier || $formInfo->name == 'password') continue;
+			if ($formInfo->name == $member_config->identifier || $formInfo->name == 'password') continue;
 
 			$formTag = new stdClass();
 			$inputTag = '';
 			$formTag->title = ($formInfo->isDefaultForm) ? $lang->{$formInfo->name} : $formInfo->title;
-			if($isAdmin)
+			if ($isAdmin)
 			{
-				if($formInfo->mustRequired) $formTag->title = '<em style="color:red">*</em> '.$formTag->title;
+				if ($formInfo->mustRequired) $formTag->title = '<em style="color:red">*</em> '.$formTag->title;
 			}
 			else
 			{
@@ -442,28 +442,28 @@ 
 			}
 			$formTag->name = $formInfo->name;
 
-			if($formInfo->isDefaultForm)
+			if ($formInfo->isDefaultForm)
 			{
-				if($formInfo->imageType)
+				if ($formInfo->imageType)
 				{
 					$formTag->type = 'image';
-					if($formInfo->name == 'profile_image')
+					if ($formInfo->name == 'profile_image')
 					{
 						$target = $memberInfo['profile_image'];
 						$functionName = 'doDeleteProfileImage';
 					}
-					else if($formInfo->name == 'image_name')
+					else if ($formInfo->name == 'image_name')
 					{
 						$target = $memberInfo['image_name'];
 						$functionName = 'doDeleteImageName';
 					}
-					else if($formInfo->name == 'image_mark')
+					else if ($formInfo->name == 'image_mark')
 					{
 						$target = $memberInfo['image_mark'];
 						$functionName = 'doDeleteImageMark';
 					}
 
-					if($target->src)
+					if ($target->src)
 					{
 						$inputTag = sprintf('<input type="hidden" name="__%s_exist" value="true" /><span id="%s"><img src="%s" alt="%s" /> <button type="button" onclick="%s(%d);return false;">%s</button></span>',
 							$formInfo->name,
@@ -486,7 +486,7 @@ 
 						$lang->{$formInfo->name.'_max_height'},
 						$member_config->{$formInfo->name.'_max_height'});
 					}//end imageType
-					else if($formInfo->name == 'birthday')
+					else if ($formInfo->name == 'birthday')
 					{
 						$formTag->type = 'date';
 						$inputTag = sprintf('<input type="hidden" name="birthday" id="date_birthday" value="%s" /><input type="text" placeholder="YYYY-MM-DD" name="birthday_ui" class="inputDate" id="birthday" value="%s" readonly="readonly" /> <input type="button" value="%s" class="btn dateRemover" />',
@@ -494,16 +494,16 @@ 
 							zdate($memberInfo['birthday'], 'Y-m-d', false),
 							$lang->cmd_delete);
 					}
-					else if($formInfo->name == 'find_account_question')
+					else if ($formInfo->name == 'find_account_question')
 					{
 						$disabled = (!!$memberInfo['member_srl']) ? 'disabled="disabled"' : '';
 
 						$formTag->type = 'select';
 						$inputTag = '<select name="find_account_question" id="find_account_question" style="display:block;margin:0 0 8px 0" %s>%s</select>';
 						$optionTag = array();
-						foreach($lang->find_account_question_items as $key=>$val)
+						foreach ($lang->find_account_question_items as $key=>$val)
 						{
-							if($key == $memberInfo['find_account_question']) $selected = 'selected="selected"';
+							if ($key == $memberInfo['find_account_question']) $selected = 'selected="selected"';
 							else $selected = '';
 							$optionTag[] = sprintf('<option value="%s" %s >%s</option>',
 								$key,
@@ -511,24 +511,24 @@ 
 								$val);
 						}
 						$inputTag = sprintf($inputTag, $disabled, implode('', $optionTag));
-						$inputTag .= '<input type="text" name="find_account_answer" id="find_account_answer" title="'.Context::getLang('find_account_answer').'" value="" ' . $disabled . ' />';
+						$inputTag .= '<input type="text" name="find_account_answer" id="find_account_answer" title="'.Context::getLang('find_account_answer').'" value="" '.$disabled.' />';
 
-						if($disabled) {
-							$inputTag .= ' <label><input type="checkbox" name="modify_find_account_answer" value="Y" /> ' . Context::getLang('cmd_modify') . '</label>';
+						if ($disabled) {
+							$inputTag .= ' <label><input type="checkbox" name="modify_find_account_answer" value="Y" /> '.Context::getLang('cmd_modify').'</label>';
 							$inputTag .= '<script>(function($) {$(function() {$(\'[name=modify_find_account_answer]\').change(function() {var $this = $(this); if($this.prop(\'checked\')) {$(\'[name=find_account_question],[name=find_account_answer]\').attr(\'disabled\', false); } else {$(\'[name=find_account_question]\').attr(\'disabled\', true); $(\'[name=find_account_answer]\').attr(\'disabled\', true).val(\'\'); } }); }); })(jQuery);</script>';
 						}
 					}
-					else if($formInfo->name == 'email_address')
+					else if ($formInfo->name == 'email_address')
 					{
 						$formTag->type = 'email';
 						$inputTag = '<input type="email" name="email_address" id="email_address" value="'.$memberInfo['email_address'].'" />';
 					}
-					else if($formInfo->name == 'homepage')
+					else if ($formInfo->name == 'homepage')
 					{
 						$formTag->type = 'url';
 						$inputTag = '<input type="url" name="homepage" id="homepage" value="'.$memberInfo['homepage'].'" />';
 					}
-					else if($formInfo->name == 'blog')
+					else if ($formInfo->name == 'blog')
 					{
 						$formTag->type = 'url';
 						$inputTag = '<input type="url" name="blog" id="blog" value="'.$memberInfo['blog'].'" />';
@@ -549,92 +549,92 @@ 
 					$extentionReplace = array();
 
 					$formTag->type = $extendForm->column_type;
-					if($extendForm->column_type == 'text')
+					if ($extendForm->column_type == 'text')
 					{
 						$template = '<input type="text" name="%column_name%" id="%column_name%" value="%value%" />';
 					}
-					else if($extendForm->column_type == 'homepage')
+					else if ($extendForm->column_type == 'homepage')
 					{
 						$template = '<input type="url" name="%column_name%" id="%column_name%" value="%value%" />';
 					}
-					else if($extendForm->column_type == 'email_address')
+					else if ($extendForm->column_type == 'email_address')
 					{
 						$template = '<input type="email" name="%column_name%" id="%column_name%" value="%value%" />';
 					}
-					else if($extendForm->column_type == 'tel')
+					else if ($extendForm->column_type == 'tel')
 					{
 						$extentionReplace = array('tel_0' => $extendForm->value[0],
 							'tel_1' => $extendForm->value[1],
 							'tel_2' => $extendForm->value[2]);
 						$template = '<input type="tel" name="%column_name%[]" id="%column_name%" value="%tel_0%" size="4" maxlength="4" style="width:30px" title="First Number" /> - <input type="tel" name="%column_name%[]" value="%tel_1%" size="4" maxlength="4" style="width:35px" title="Second Number" /> - <input type="tel" name="%column_name%[]" value="%tel_2%" size="4" maxlength="4" style="width:35px" title="Third Number" />';
 					}
-					else if($extendForm->column_type == 'textarea')
+					else if ($extendForm->column_type == 'textarea')
 					{
 						$template = '<textarea name="%column_name%" id="%column_name%" rows="4" cols="42">%value%</textarea>';
 					}
-					else if($extendForm->column_type == 'checkbox')
+					else if ($extendForm->column_type == 'checkbox')
 					{
 						$template = '';
-						if($extendForm->default_value)
+						if ($extendForm->default_value)
 						{
 							$template = '<div style="padding-top:5px">%s</div>';
 							$__i = 0;
 							$optionTag = array();
-							foreach($extendForm->default_value as $v)
+							foreach ($extendForm->default_value as $v)
 							{
 								$checked = '';
-								if(is_array($extendForm->value) && in_array($v, $extendForm->value))$checked = 'checked="checked"';
+								if (is_array($extendForm->value) && in_array($v, $extendForm->value))$checked = 'checked="checked"';
 								$optionTag[] = '<label for="%column_name%'.$__i.'"><input type="checkbox" id="%column_name%'.$__i.'" name="%column_name%[]" value="'.$v.'" '.$checked.' /> '.$v.'</label>';
 								$__i++;
 							}
 							$template = sprintf($template, implode('', $optionTag));
 						}
 					}
-					else if($extendForm->column_type == 'radio')
+					else if ($extendForm->column_type == 'radio')
 					{
 						$template = '';
-						if($extendForm->default_value)
+						if ($extendForm->default_value)
 						{
 							$template = '<div style="padding-top:5px">%s</div>';
 							$optionTag = array();
-							foreach($extendForm->default_value as $v)
+							foreach ($extendForm->default_value as $v)
 							{
-								if($extendForm->value == $v)$checked = 'checked="checked"';
+								if ($extendForm->value == $v)$checked = 'checked="checked"';
 								else $checked = '';
 								$optionTag[] = '<label><input type="radio" name="%column_name%" value="'.$v.'" '.$checked.' /> '.$v.'</label>';
 							}
 							$template = sprintf($template, implode('', $optionTag));
 						}
 					}
-					else if($extendForm->column_type == 'select')
+					else if ($extendForm->column_type == 'select')
 					{
 						$template = '<select name="'.$formInfo->name.'" id="'.$formInfo->name.'">%s</select>';
 						$optionTag = array();
 						$optionTag[] = sprintf('<option value="">%s</option>', $lang->cmd_select);
-						if($extendForm->default_value)
+						if ($extendForm->default_value)
 						{
-							foreach($extendForm->default_value as $v)
+							foreach ($extendForm->default_value as $v)
 							{
-								if($v == $extendForm->value) $selected = 'selected="selected"';
+								if ($v == $extendForm->value) $selected = 'selected="selected"';
 								else $selected = '';
 								$optionTag[] = sprintf('<option value="%s" %s >%s</option>', $v, $selected, $v);
 							}
 						}
 						$template = sprintf($template, implode('', $optionTag));
 					}
-					else if($extendForm->column_type == 'kr_zip')
+					else if ($extendForm->column_type == 'kr_zip')
 					{
 						$krzipModel = getModel('krzip');
-						if($krzipModel && method_exists($krzipModel , 'getKrzipCodeSearchHtml' ))
+						if ($krzipModel && method_exists($krzipModel, 'getKrzipCodeSearchHtml'))
 						{
 							$template = $krzipModel->getKrzipCodeSearchHtml($extendForm->column_name, $extendForm->value);
 						}
 					}
-					else if($extendForm->column_type == 'jp_zip')
+					else if ($extendForm->column_type == 'jp_zip')
 					{
 						$template = '<input type="text" name="%column_name%" id="%column_name%" value="%value%" />';
 					}
-					else if($extendForm->column_type == 'date')
+					else if ($extendForm->column_type == 'date')
 					{
 						$extentionReplace = array('date' => zdate($extendForm->value, 'Y-m-d'), 'cmd_delete' => $lang->cmd_delete);
 						$template = '<input type="hidden" name="%column_name%" id="date_%column_name%" value="%value%" /><input type="text" placeholder="YYYY-MM-DD" class="inputDate" value="%date%" readonly="readonly" /> <input type="button" value="%cmd_delete%" class="btn dateRemover" />';
@@ -643,7 +643,7 @@ 
 					$replace = array_merge($extentionReplace, $replace);
 					$inputTag = preg_replace_callback('@%(\w+)%@', function($n) use($replace) { return $replace[$n[1]]; }, $template);
 
-					if($extendForm->description)
+					if ($extendForm->description)
 						$inputTag .= '<p class="help-block">'.$extendForm->description.'</p>';
 				}
 				$formTag->inputTag = $inputTag;
@@ -674,12 +674,12 @@ 
 	function dispMemberAdminInsertJoinForm() {
 		// Get the value of join_form
 		$member_join_form_srl = Context::get('member_join_form_srl');
-		if($member_join_form_srl)
+		if ($member_join_form_srl)
 		{
 			$oMemberModel = getModel('member');
 			$join_form = $oMemberModel->getJoinForm($member_join_form_srl);
 
-			if(!$join_form) Context::set('member_join_form_srl','',true);
+			if (!$join_form) Context::set('member_join_form_srl', '', true);
 			else
 			{
 				Context::set('join_form', $join_form);

Braces +62 added lines, -57 removed lines

@@ -49,8 +49,7 @@ 
 			if(!$this->memberInfo)
 			{
 				Context::set('member_srl','');
-			}
-			else
+			} else
 			{
 				Context::set('member_info',$this->memberInfo);
 			}
@@ -101,7 +100,9 @@ 
 		{
 			foreach($config->signupForm as $signupItem)
 			{
-				if(!count($memberIdentifiers)) break;
+				if(!count($memberIdentifiers)) {
+					break;
+				}
 				if(in_array($signupItem->name, $memberIdentifiers) && ($signupItem->required || $signupItem->isUse))
 				{
 					unset($memberIdentifiers[$signupItem->name]);
@@ -316,7 +317,9 @@ 
 		$extendForm = $oMemberModel->getCombineJoinForm($this->memberInfo);
 		Context::set('extend_form_list', $extendForm);
 		$memberInfo = get_object_vars(Context::get('member_info'));
-		if (!is_array($memberInfo['group_list'])) $memberInfo['group_list'] = array();
+		if (!is_array($memberInfo['group_list'])) {
+			$memberInfo['group_list'] = array();
+		}
 		Context::set('memberInfo', $memberInfo);
 
 		$disableColumns = array('password', 'find_account_question', 'find_account_answer');
@@ -418,7 +421,9 @@ 
 
 		foreach($member_config->signupForm as $no=>$formInfo)
 		{
-			if(!$formInfo->isUse)continue;
+			if(!$formInfo->isUse) {
+				continue;
+			}
 
 			// 회원 본인이 아닌 경우 입력 폼 제거
 			if($formInfo->name == 'find_account_question' && $memberInfo['member_srl'] !== $logged_info->member_srl)
@@ -427,18 +432,23 @@ 
 				continue;
 			}
 
-			if($formInfo->name == $member_config->identifier || $formInfo->name == 'password') continue;
+			if($formInfo->name == $member_config->identifier || $formInfo->name == 'password') {
+				continue;
+			}
 
 			$formTag = new stdClass();
 			$inputTag = '';
 			$formTag->title = ($formInfo->isDefaultForm) ? $lang->{$formInfo->name} : $formInfo->title;
 			if($isAdmin)
 			{
-				if($formInfo->mustRequired) $formTag->title = '<em style="color:red">*</em> '.$formTag->title;
-			}
-			else
+				if($formInfo->mustRequired) {
+					$formTag->title = '<em style="color:red">*</em> '.$formTag->title;
+				}
+			} else
 			{
-				if ($formInfo->required && $formInfo->name != 'password') $formTag->title = '<em style="color:red">*</em> '.$formTag->title;
+				if ($formInfo->required && $formInfo->name != 'password') {
+					$formTag->title = '<em style="color:red">*</em> '.$formTag->title;
+				}
 			}
 			$formTag->name = $formInfo->name;
 
@@ -451,13 +461,11 @@ 
 					{
 						$target = $memberInfo['profile_image'];
 						$functionName = 'doDeleteProfileImage';
-					}
-					else if($formInfo->name == 'image_name')
+					} else if($formInfo->name == 'image_name')
 					{
 						$target = $memberInfo['image_name'];
 						$functionName = 'doDeleteImageName';
-					}
-					else if($formInfo->name == 'image_mark')
+					} else if($formInfo->name == 'image_mark')
 					{
 						$target = $memberInfo['image_mark'];
 						$functionName = 'doDeleteImageMark';
@@ -473,8 +481,7 @@ 
 							$functionName,
 							$memberInfo['member_srl'],
 							$lang->cmd_delete);
-					}
-					else
+					} else
 					{
 						$inputTag = sprintf('<input type="hidden" name="__%s_exist" value="false" />', $formInfo->name);
 					}
@@ -493,8 +500,7 @@ 
 							$memberInfo['birthday'],
 							zdate($memberInfo['birthday'], 'Y-m-d', false),
 							$lang->cmd_delete);
-					}
-					else if($formInfo->name == 'find_account_question')
+					} else if($formInfo->name == 'find_account_question')
 					{
 						$disabled = (!!$memberInfo['member_srl']) ? 'disabled="disabled"' : '';
 
@@ -503,8 +509,11 @@ 
 						$optionTag = array();
 						foreach($lang->find_account_question_items as $key=>$val)
 						{
-							if($key == $memberInfo['find_account_question']) $selected = 'selected="selected"';
-							else $selected = '';
+							if($key == $memberInfo['find_account_question']) {
+								$selected = 'selected="selected"';
+							} else {
+								$selected = '';
+							}
 							$optionTag[] = sprintf('<option value="%s" %s >%s</option>',
 								$key,
 								$selected,
@@ -517,23 +526,19 @@ 
 							$inputTag .= ' <label><input type="checkbox" name="modify_find_account_answer" value="Y" /> ' . Context::getLang('cmd_modify') . '</label>';
 							$inputTag .= '<script>(function($) {$(function() {$(\'[name=modify_find_account_answer]\').change(function() {var $this = $(this); if($this.prop(\'checked\')) {$(\'[name=find_account_question],[name=find_account_answer]\').attr(\'disabled\', false); } else {$(\'[name=find_account_question]\').attr(\'disabled\', true); $(\'[name=find_account_answer]\').attr(\'disabled\', true).val(\'\'); } }); }); })(jQuery);</script>';
 						}
-					}
-					else if($formInfo->name == 'email_address')
+					} else if($formInfo->name == 'email_address')
 					{
 						$formTag->type = 'email';
 						$inputTag = '<input type="email" name="email_address" id="email_address" value="'.$memberInfo['email_address'].'" />';
-					}
-					else if($formInfo->name == 'homepage')
+					} else if($formInfo->name == 'homepage')
 					{
 						$formTag->type = 'url';
 						$inputTag = '<input type="url" name="homepage" id="homepage" value="'.$memberInfo['homepage'].'" />';
-					}
-					else if($formInfo->name == 'blog')
+					} else if($formInfo->name == 'blog')
 					{
 						$formTag->type = 'url';
 						$inputTag = '<input type="url" name="blog" id="blog" value="'.$memberInfo['blog'].'" />';
-					}
-					else
+					} else
 					{
 						$formTag->type = 'text';
 						$inputTag = sprintf('<input type="text" name="%s" id="%s" value="%s" />',
@@ -552,27 +557,22 @@ 
 					if($extendForm->column_type == 'text')
 					{
 						$template = '<input type="text" name="%column_name%" id="%column_name%" value="%value%" />';
-					}
-					else if($extendForm->column_type == 'homepage')
+					} else if($extendForm->column_type == 'homepage')
 					{
 						$template = '<input type="url" name="%column_name%" id="%column_name%" value="%value%" />';
-					}
-					else if($extendForm->column_type == 'email_address')
+					} else if($extendForm->column_type == 'email_address')
 					{
 						$template = '<input type="email" name="%column_name%" id="%column_name%" value="%value%" />';
-					}
-					else if($extendForm->column_type == 'tel')
+					} else if($extendForm->column_type == 'tel')
 					{
 						$extentionReplace = array('tel_0' => $extendForm->value[0],
 							'tel_1' => $extendForm->value[1],
 							'tel_2' => $extendForm->value[2]);
 						$template = '<input type="tel" name="%column_name%[]" id="%column_name%" value="%tel_0%" size="4" maxlength="4" style="width:30px" title="First Number" /> - <input type="tel" name="%column_name%[]" value="%tel_1%" size="4" maxlength="4" style="width:35px" title="Second Number" /> - <input type="tel" name="%column_name%[]" value="%tel_2%" size="4" maxlength="4" style="width:35px" title="Third Number" />';
-					}
-					else if($extendForm->column_type == 'textarea')
+					} else if($extendForm->column_type == 'textarea')
 					{
 						$template = '<textarea name="%column_name%" id="%column_name%" rows="4" cols="42">%value%</textarea>';
-					}
-					else if($extendForm->column_type == 'checkbox')
+					} else if($extendForm->column_type == 'checkbox')
 					{
 						$template = '';
 						if($extendForm->default_value)
@@ -583,14 +583,15 @@ 
 							foreach($extendForm->default_value as $v)
 							{
 								$checked = '';
-								if(is_array($extendForm->value) && in_array($v, $extendForm->value))$checked = 'checked="checked"';
+								if(is_array($extendForm->value) && in_array($v, $extendForm->value)) {
+									$checked = 'checked="checked"';
+								}
 								$optionTag[] = '<label for="%column_name%'.$__i.'"><input type="checkbox" id="%column_name%'.$__i.'" name="%column_name%[]" value="'.$v.'" '.$checked.' /> '.$v.'</label>';
 								$__i++;
 							}
 							$template = sprintf($template, implode('', $optionTag));
 						}
-					}
-					else if($extendForm->column_type == 'radio')
+					} else if($extendForm->column_type == 'radio')
 					{
 						$template = '';
 						if($extendForm->default_value)
@@ -599,14 +600,16 @@ 
 							$optionTag = array();
 							foreach($extendForm->default_value as $v)
 							{
-								if($extendForm->value == $v)$checked = 'checked="checked"';
-								else $checked = '';
+								if($extendForm->value == $v) {
+									$checked = 'checked="checked"';
+								} else {
+									$checked = '';
+								}
 								$optionTag[] = '<label><input type="radio" name="%column_name%" value="'.$v.'" '.$checked.' /> '.$v.'</label>';
 							}
 							$template = sprintf($template, implode('', $optionTag));
 						}
-					}
-					else if($extendForm->column_type == 'select')
+					} else if($extendForm->column_type == 'select')
 					{
 						$template = '<select name="'.$formInfo->name.'" id="'.$formInfo->name.'">%s</select>';
 						$optionTag = array();
@@ -615,26 +618,26 @@ 
 						{
 							foreach($extendForm->default_value as $v)
 							{
-								if($v == $extendForm->value) $selected = 'selected="selected"';
-								else $selected = '';
+								if($v == $extendForm->value) {
+									$selected = 'selected="selected"';
+								} else {
+									$selected = '';
+								}
 								$optionTag[] = sprintf('<option value="%s" %s >%s</option>', $v, $selected, $v);
 							}
 						}
 						$template = sprintf($template, implode('', $optionTag));
-					}
-					else if($extendForm->column_type == 'kr_zip')
+					} else if($extendForm->column_type == 'kr_zip')
 					{
 						$krzipModel = getModel('krzip');
 						if($krzipModel && method_exists($krzipModel , 'getKrzipCodeSearchHtml' ))
 						{
 							$template = $krzipModel->getKrzipCodeSearchHtml($extendForm->column_name, $extendForm->value);
 						}
-					}
-					else if($extendForm->column_type == 'jp_zip')
+					} else if($extendForm->column_type == 'jp_zip')
 					{
 						$template = '<input type="text" name="%column_name%" id="%column_name%" value="%value%" />';
-					}
-					else if($extendForm->column_type == 'date')
+					} else if($extendForm->column_type == 'date')
 					{
 						$extentionReplace = array('date' => zdate($extendForm->value, 'Y-m-d'), 'cmd_delete' => $lang->cmd_delete);
 						$template = '<input type="hidden" name="%column_name%" id="date_%column_name%" value="%value%" /><input type="text" placeholder="YYYY-MM-DD" class="inputDate" value="%date%" readonly="readonly" /> <input type="button" value="%cmd_delete%" class="btn dateRemover" />';
@@ -643,8 +646,9 @@ 
 					$replace = array_merge($extentionReplace, $replace);
 					$inputTag = preg_replace_callback('@%(\w+)%@', function($n) use($replace) { return $replace[$n[1]]; }, $template);
 
-					if($extendForm->description)
-						$inputTag .= '<p class="help-block">'.$extendForm->description.'</p>';
+					if($extendForm->description) {
+											$inputTag .= '<p class="help-block">'.$extendForm->description.'</p>';
+					}
 				}
 				$formTag->inputTag = $inputTag;
 				$formTags[] = $formTag;
@@ -679,8 +683,9 @@ 
 			$oMemberModel = getModel('member');
 			$join_form = $oMemberModel->getJoinForm($member_join_form_srl);
 
-			if(!$join_form) Context::set('member_join_form_srl','',true);
-			else
+			if(!$join_form) {
+				Context::set('member_join_form_srl','',true);
+			} else
 			{
 				Context::set('join_form', $join_form);
 				$security = new Security();

modules/member/member.admin.controller.php

Spacing +175 added lines, -175 removed lines

@@ -24,31 +24,31 @@ 
 		// if(Context::getRequestMethod() == "GET") return new Object(-1, "msg_invalid_request");
 		// Extract the necessary information in advance
 		$logged_info = Context::get('logged_info');
-		if($logged_info->is_admin != 'Y' || !checkCSRF())
+		if ($logged_info->is_admin != 'Y' || !checkCSRF())
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
 
-		$args = Context::gets('member_srl','email_address','find_account_answer', 'allow_mailing','allow_message','denied','is_admin','description','group_srl_list','limit_date');
-		$oMemberModel = &getModel ('member');
-		$config = $oMemberModel->getMemberConfig ();
+		$args = Context::gets('member_srl', 'email_address', 'find_account_answer', 'allow_mailing', 'allow_message', 'denied', 'is_admin', 'description', 'group_srl_list', 'limit_date');
+		$oMemberModel = &getModel('member');
+		$config = $oMemberModel->getMemberConfig();
 		$getVars = array();
-		if($config->signupForm)
+		if ($config->signupForm)
 		{
-			foreach($config->signupForm as $formInfo)
+			foreach ($config->signupForm as $formInfo)
 			{
-				if($formInfo->isDefaultForm && ($formInfo->isUse || $formInfo->required || $formInfo->mustRequired))
+				if ($formInfo->isDefaultForm && ($formInfo->isUse || $formInfo->required || $formInfo->mustRequired))
 				{
 					$getVars[] = $formInfo->name;
 				}
 			}
 		}
-		foreach($getVars as $val)
+		foreach ($getVars as $val)
 		{
 			$args->{$val} = Context::get($val);
 		}
 		$args->member_srl = Context::get('member_srl');
-		if(Context::get('reset_password'))
+		if (Context::get('reset_password'))
 			$args->password = Context::get('reset_password');
 		else unset($args->password);
 
@@ -60,7 +60,7 @@ 
 		unset($all_args->error_return_url);
 		unset($all_args->success_return_url);
 		unset($all_args->ruleset);
-		if(!isset($args->limit_date)) $args->limit_date = "";
+		if (!isset($args->limit_date)) $args->limit_date = "";
 		unset($all_args->password);
 		unset($all_args->password2);
 		unset($all_args->reset_password);
@@ -68,7 +68,7 @@ 
 		$extra_vars = delObjectVars($all_args, $args);
 		$args->extra_vars = serialize($extra_vars);
 		// Check if an original member exists having the member_srl
-		if($args->member_srl)
+		if ($args->member_srl)
 		{
 			// Create a member model object
 			$oMemberModel = getModel('member');
@@ -76,14 +76,14 @@ 
 			$columnList = array('member_srl');
 			$member_info = $oMemberModel->getMemberInfoByMemberSrl($args->member_srl, 0, $columnList);
 			// If no original member exists, make a new one
-			if($member_info->member_srl != $args->member_srl) unset($args->member_srl);
+			if ($member_info->member_srl != $args->member_srl) unset($args->member_srl);
 		}
 
 		// remove whitespace
 		$checkInfos = array('user_id', 'user_name', 'nick_name', 'email_address');
-		foreach($checkInfos as $val)
+		foreach ($checkInfos as $val)
 		{
-			if(isset($args->{$val}))
+			if (isset($args->{$val}))
 			{
 				$args->{$val} = preg_replace('/[\pZ\pC]+/u', '', $args->{$val});
 			}
@@ -91,7 +91,7 @@ 
 
 		$oMemberController = getController('member');
 		// Execute insert or update depending on the value of member_srl
-		if(!$args->member_srl)
+		if (!$args->member_srl)
 		{
 			$args->password = Context::get('password');
 			$output = $oMemberController->insertMember($args);
@@ -103,7 +103,7 @@ 
 			$msg_code = 'success_updated';
 		}
 
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 		// Save Signature
 		$signature = Context::get('signature');
 		$oMemberController->putSignature($args->member_srl, $signature);
@@ -112,13 +112,13 @@ 
 		$this->setMessage($msg_code);
 
 		$profile_image = $_FILES['profile_image'];
-		if(is_uploaded_file($profile_image['tmp_name']))
+		if (is_uploaded_file($profile_image['tmp_name']))
 		{
 			$oMemberController->insertProfileImage($args->member_srl, $profile_image['tmp_name']);
 		}
 
 		$image_mark = $_FILES['image_mark'];
-		if(is_uploaded_file($image_mark['tmp_name']))
+		if (is_uploaded_file($image_mark['tmp_name']))
 		{
 			$oMemberController->insertImageMark($args->member_srl, $image_mark['tmp_name']);
 		}
@@ -144,9 +144,9 @@ 
 
 		$oMemberController = getController('member');
 		$output = $oMemberController->deleteMember($member_srl);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		$this->add('page',Context::get('page'));
+		$this->add('page', Context::get('page'));
 		$this->setMessage("success_deleted");
 	}
 
@@ -165,26 +165,26 @@ 
 		);
 		
 		$oPassword = new Password();
-		if(!array_key_exists($args->password_hashing_algorithm, $oPassword->getSupportedAlgorithms()))
+		if (!array_key_exists($args->password_hashing_algorithm, $oPassword->getSupportedAlgorithms()))
 		{
 			$args->password_hashing_algorithm = 'md5';
 		}
 		
 		$args->password_hashing_work_factor = intval($args->password_hashing_work_factor, 10);
-		if($args->password_hashing_work_factor < 4)
+		if ($args->password_hashing_work_factor < 4)
 		{
 			$args->password_hashing_work_factor = 4;
 		}
-		if($args->password_hashing_work_factor > 16)
+		if ($args->password_hashing_work_factor > 16)
 		{
 			$args->password_hashing_work_factor = 16;
 		}
-		if($args->password_hashing_auto_upgrade != 'Y')
+		if ($args->password_hashing_auto_upgrade != 'Y')
 		{
 			$args->password_hashing_auto_upgrade = 'N';
 		}
 
-		if((!$args->webmaster_name || !$args->webmaster_email) && $args->enable_confirm == 'Y')
+		if ((!$args->webmaster_name || !$args->webmaster_email) && $args->enable_confirm == 'Y')
 		{
 			return new Object(-1, 'msg_mail_authorization');
 		}
@@ -219,20 +219,20 @@ 
 		$usable_list = Context::get('usable_list');
 		$all_args = Context::getRequestVars();
 
-		$args->limit_day = (int)$args->limit_day;
-		if(!trim(strip_tags($args->agreement)))
+		$args->limit_day = (int) $args->limit_day;
+		if (!trim(strip_tags($args->agreement)))
 		{
-			$agreement_file = _XE_PATH_.'files/member_extra_info/agreement_' . Context::get('lang_type') . '.txt';
+			$agreement_file = _XE_PATH_.'files/member_extra_info/agreement_'.Context::get('lang_type').'.txt';
 			FileHandler::removeFile($agreement_file);
 			$args->agreement = NULL;
 		}
 
-		if($args->redirect_url)
+		if ($args->redirect_url)
 		{
 			$oModuleModel = getModel('module');
 			$redirectModuleInfo = $oModuleModel->getModuleInfoByModuleSrl($args->redirect_url, array('mid'));
 
-			if(!$redirectModuleInfo)
+			if (!$redirectModuleInfo)
 			{
 				return new Object('-1', 'msg_exist_selected_module');
 			}
@@ -256,28 +256,28 @@ 
 		$items = array('user_id', 'password', 'user_name', 'nick_name', 'email_address', 'find_account_question', 'homepage', 'blog', 'birthday', 'signature', 'profile_image', 'image_name', 'image_mark', 'profile_image_max_width', 'profile_image_max_height', 'image_name_max_width', 'image_name_max_height', 'image_mark_max_width', 'image_mark_max_height');
 		$mustRequireds = array('email_address', 'nick_name', 'password', 'find_account_question');
 		$extendItems = $oMemberModel->getJoinFormList();
-		foreach($list_order as $key)
+		foreach ($list_order as $key)
 		{
 			$signupItem = new stdClass();
 			$signupItem->isIdentifier = ($key == $all_args->identifier);
 			$signupItem->isDefaultForm = in_array($key, $items);
 
 			$signupItem->name = $key;
-			if(!in_array($key, $items)) $signupItem->title = $key;
+			if (!in_array($key, $items)) $signupItem->title = $key;
 			else $signupItem->title = $lang->{$key};
 			$signupItem->mustRequired = in_array($key, $mustRequireds);
 			$signupItem->imageType = (strpos($key, 'image') !== false);
 			$signupItem->required = ($all_args->{$key} == 'required') || $signupItem->mustRequired || $signupItem->isIdentifier;
 			$signupItem->isUse = in_array($key, $usable_list) || $signupItem->required;
 			$signupItem->isPublic = ($all_args->{'is_'.$key.'_public'} == 'Y' && $signupItem->isUse) ? 'Y' : 'N';
-			if($signupItem->imageType)
+			if ($signupItem->imageType)
 			{
 				$signupItem->max_width = $all_args->{$key.'_max_width'};
 				$signupItem->max_height = $all_args->{$key.'_max_height'};
 			}
 
 			// set extends form
-			if(!$signupItem->isDefaultForm)
+			if (!$signupItem->isDefaultForm)
 			{
 				$extendItem = $extendItems[$all_args->{$key.'_member_join_form_srl'}];
 				$signupItem->type = $extendItem->column_type;
@@ -286,13 +286,13 @@ 
 				$signupItem->description = $extendItem->description;
 
 				// check usable value change, required/option
-				if($signupItem->isUse != ($extendItem->is_active == 'Y') || $signupItem->required != ($extendItem->required == 'Y'))
+				if ($signupItem->isUse != ($extendItem->is_active == 'Y') || $signupItem->required != ($extendItem->required == 'Y'))
 				{
 					unset($update_args);
 					$update_args = new stdClass;
 					$update_args->member_join_form_srl = $extendItem->member_join_form_srl;
-					$update_args->is_active = $signupItem->isUse?'Y':'N';
-					$update_args->required = $signupItem->required?'Y':'N';
+					$update_args->is_active = $signupItem->isUse ? 'Y' : 'N';
+					$update_args->required = $signupItem->required ? 'Y' : 'N';
 
 					$update_output = executeQuery('member.updateJoinForm', $update_args);
 				}
@@ -309,9 +309,9 @@ 
 		$this->_createFindAccountByQuestion($args->identifier);
 
 		// check agreement value exist
-		if($args->agreement)
+		if ($args->agreement)
 		{
-			$agreement_file = _XE_PATH_.'files/member_extra_info/agreement_' . Context::get('lang_type') . '.txt';
+			$agreement_file = _XE_PATH_.'files/member_extra_info/agreement_'.Context::get('lang_type').'.txt';
 			$output = FileHandler::writeFile($agreement_file, $args->agreement);
 
 			unset($args->agreement);
@@ -339,16 +339,16 @@ 
 			'after_logout_url'
 		);
 
-		if(!$args->change_password_date)
+		if (!$args->change_password_date)
 		{
 			$args->change_password_date = 0;
 		}
 
-		if(!trim(strip_tags($args->after_login_url)))
+		if (!trim(strip_tags($args->after_login_url)))
 		{
 			$args->after_login_url = NULL;
 		}
-		if(!trim(strip_tags($args->after_logout_url)))
+		if (!trim(strip_tags($args->after_logout_url)))
 		{
 			$args->after_logout_url = NULL;
 		}
@@ -375,17 +375,17 @@ 
 		);
 
 		$args->layout_srl = $args->layout_srl ? $args->layout_srl : NULL;
-		if(!$args->skin)
+		if (!$args->skin)
 		{
 			$args->skin = 'default';
 		}
-		if(!$args->colorset)
+		if (!$args->colorset)
 		{
 			$args->colorset = 'white';
 		}
 
 		$args->mlayout_srl = $args->mlayout_srl ? $args->mlayout_srl : NULL;
-		if(!$args->mskin)
+		if (!$args->mskin)
 		{
 			$args->mskin = 'default';
 		}
@@ -408,12 +408,12 @@ 
 		$extendItems = $oMemberModel->getJoinFormList();
 
 		$items = array('user_id', 'password', 'user_name', 'nick_name', 'email_address', 'find_account_question', 'homepage', 'blog', 'birthday', 'signature', 'profile_image', 'image_name', 'image_mark');
-		$mustRequireds = array('email_address', 'nick_name','password', 'find_account_question');
+		$mustRequireds = array('email_address', 'nick_name', 'password', 'find_account_question');
 		$orgRequireds = array('email_address', 'password', 'find_account_question', 'user_id', 'nick_name', 'user_name');
 		$orgUse = array('email_address', 'password', 'find_account_question', 'user_id', 'nick_name', 'user_name', 'homepage', 'blog', 'birthday');
 		$list_order = array();
 
-		foreach($items as $key)
+		foreach ($items as $key)
 		{
 			unset($signupItem);
 			$signupItem = new stdClass;
@@ -425,23 +425,23 @@ 
 			$signupItem->required = in_array($key, $orgRequireds);
 			$signupItem->isUse = ($config->{$key} == 'Y') || in_array($key, $orgUse);
 			$signupItem->isPublic = ($signupItem->isUse) ? 'Y' : 'N';
-			if($key == 'find_account_question' || $key == 'password')
+			if ($key == 'find_account_question' || $key == 'password')
 			{
 				$signupItem->isPublic = 'N';
 			}
 			$signupItem->isIdentifier = ($key == $identifier);
-			if ($signupItem->imageType){
+			if ($signupItem->imageType) {
 				$signupItem->max_width = $config->{$key.'_max_width'};
 				$signupItem->max_height = $config->{$key.'_max_height'};
 			}
-			if($signupItem->isIdentifier)
+			if ($signupItem->isIdentifier)
 				array_unshift($list_order, $signupItem);
 			else
 				$list_order[] = $signupItem;
 		}
-		if(is_array($extendItems))
+		if (is_array($extendItems))
 		{
-			foreach($extendItems as $form_srl=>$item_info)
+			foreach ($extendItems as $form_srl=>$item_info)
 			{
 				unset($signupItem);
 				$signupItem = new stdClass;
@@ -454,7 +454,7 @@ 
 				$signupItem->isUse = ($item_info->is_active == 'Y');
 				$signupItem->isPublic = ($signupItem->isUse) ? 'Y' : 'N';
 				$signupItem->description = $item_info->description;
-				if($signupItem->imageType)
+				if ($signupItem->imageType)
 				{
 					$signupItem->max_width = $config->{$key.'_max_width'};
 					$signupItem->max_height = $config->{$key.'_max_height'};
@@ -472,13 +472,13 @@ 
 	 * @param string $agreement
 	 * @return void
 	 */
-	function _createSignupRuleset($signupForm, $agreement = null){
+	function _createSignupRuleset($signupForm, $agreement = null) {
 		$xml_file = './files/ruleset/insertMember.xml';
-		$buff = '<?xml version="1.0" encoding="utf-8"?>' . PHP_EOL.
-			'<ruleset version="1.5.0">' . PHP_EOL.
-			'<customrules>' . PHP_EOL.
-			'</customrules>' . PHP_EOL.
-			'<fields>' . PHP_EOL . '%s' . PHP_EOL . '</fields>' . PHP_EOL.
+		$buff = '<?xml version="1.0" encoding="utf-8"?>'.PHP_EOL.
+			'<ruleset version="1.5.0">'.PHP_EOL.
+			'<customrules>'.PHP_EOL.
+			'</customrules>'.PHP_EOL.
+			'<fields>'.PHP_EOL.'%s'.PHP_EOL.'</fields>'.PHP_EOL.
 			'</ruleset>';
 
 		$fields = array();
@@ -487,41 +487,41 @@ 
 		{
 			$fields[] = '<field name="accept_agreement"><if test="$act == \'procMemberInsert\'" attr="required" value="true" /></field>';
 		}
-		foreach($signupForm as $formInfo)
+		foreach ($signupForm as $formInfo)
 		{
-			if($formInfo->required || $formInfo->mustRequired)
+			if ($formInfo->required || $formInfo->mustRequired)
 			{
-				if($formInfo->type == 'tel' || $formInfo->type == 'kr_zip')
+				if ($formInfo->type == 'tel' || $formInfo->type == 'kr_zip')
 				{
 					$fields[] = sprintf('<field name="%s[]" required="true" />', $formInfo->name);
 				}
-				else if($formInfo->name == 'password')
+				else if ($formInfo->name == 'password')
 				{
 					$fields[] = '<field name="password"><if test="$act == \'procMemberInsert\'" attr="required" value="true" /><if test="$act == \'procMemberInsert\'" attr="length" value="4:60" /></field>';
 					$fields[] = '<field name="password2"><if test="$act == \'procMemberInsert\'" attr="required" value="true" /><if test="$act == \'procMemberInsert\'" attr="equalto" value="password" /></field>';
 				}
-				else if($formInfo->name == 'find_account_question')
+				else if ($formInfo->name == 'find_account_question')
 				{
 					$fields[] = '<field name="find_account_question"><if test="$modify_find_account_answer" attr="required" value="true" /></field>';
 					$fields[] = '<field name="find_account_answer" length=":250"><if test="$modify_find_account_answer" attr="required" value="true" /></field>';
 				}
-				else if($formInfo->name == 'email_address')
+				else if ($formInfo->name == 'email_address')
 				{
 					$fields[] = sprintf('<field name="%s" required="true" rule="email"/>', $formInfo->name);
 				}
-				else if($formInfo->name == 'user_id')
+				else if ($formInfo->name == 'user_id')
 				{
 					$fields[] = sprintf('<field name="%s" required="true" rule="userid" length="3:20" />', $formInfo->name);
 				}
-				else if($formInfo->name == 'nick_name')
+				else if ($formInfo->name == 'nick_name')
 				{
 					$fields[] = sprintf('<field name="%s" required="true" length="2:20" />', $formInfo->name);
 				}
-				else if(strpos($formInfo->name, 'image') !== false)
+				else if (strpos($formInfo->name, 'image') !== false)
 				{
 					$fields[] = sprintf('<field name="%s"><if test="$act != \'procMemberAdminInsert\' &amp;&amp; $__%s_exist != \'true\'" attr="required" value="true" /></field>', $formInfo->name, $formInfo->name);
 				}
-				else if($formInfo->name == 'signature')
+				else if ($formInfo->name == 'signature')
 				{
 					$fields[] = '<field name="signature"><if test="$member_srl" attr="required" value="true" /></field>';
 				}
@@ -536,7 +536,7 @@ 
 		FileHandler::writeFile($xml_file, $xml_buff);
 		unset($xml_buff);
 
-		$validator   = new Validator($xml_file);
+		$validator = new Validator($xml_file);
 		$validator->setCacheDir('files/cache');
 		$validator->getJsPath();
 	}
@@ -564,7 +564,7 @@ 
 		$xml_buff = sprintf($buff, implode('', $fields));
 		Filehandler::writeFile($xml_file, $xml_buff);
 
-		$validator   = new Validator($xml_file);
+		$validator = new Validator($xml_file);
 		$validator->setCacheDir('files/cache');
 		$validator->getJsPath();
 	}
@@ -585,7 +585,7 @@ 
 			'</ruleset>';
 
 		$fields = array();
-		if($identifier == 'user_id')
+		if ($identifier == 'user_id')
 			$fields[] = '<field name="user_id" required="true" rule="userid" />';
 
 		$fields[] = '<field name="email_address" required="true" rule="email" />';
@@ -595,7 +595,7 @@ 
 		$xml_buff = sprintf($buff, implode('', $fields));
 		Filehandler::writeFile($xml_file, $xml_buff);
 
-		$validator   = new Validator($xml_file);
+		$validator = new Validator($xml_file);
 		$validator->setCacheDir('files/cache');
 		$validator->getJsPath();
 	}
@@ -606,12 +606,12 @@ 
 	 */
 	function procMemberAdminInsertGroup()
 	{
-		$args = Context::gets('title','description','is_default','image_mark');
+		$args = Context::gets('title', 'description', 'is_default', 'image_mark');
 		$output = $this->insertGroup($args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		$this->add('group_srl','');
-		$this->add('page',Context::get('page'));
+		$this->add('group_srl', '');
+		$this->add('page', Context::get('page'));
 		$this->setMessage('success_registed');
 
 		$returnUrl = Context::get('success_return_url') ? Context::get('success_return_url') : getNotEncodedUrl('', 'module', 'admin', 'act', 'dispMemberAdminGroupList');
@@ -626,13 +626,13 @@ 
 	{
 		$group_srl = Context::get('group_srl');
 
-		$args = Context::gets('group_srl','title','description','is_default','image_mark');
+		$args = Context::gets('group_srl', 'title', 'description', 'is_default', 'image_mark');
 		$args->site_srl = 0;
 		$output = $this->updateGroup($args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		$this->add('group_srl','');
-		$this->add('page',Context::get('page'));
+		$this->add('group_srl', '');
+		$this->add('page', Context::get('page'));
 		$this->setMessage('success_updated');
 
 		$returnUrl = Context::get('success_return_url') ? Context::get('success_return_url') : getNotEncodedUrl('', 'module', 'admin', 'act', 'dispMemberAdminGroupList');
@@ -648,10 +648,10 @@ 
 		$group_srl = Context::get('group_srl');
 
 		$output = $this->deleteGroup($group_srl);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		$this->add('group_srl','');
-		$this->add('page',Context::get('page'));
+		$this->add('group_srl', '');
+		$this->add('page', Context::get('page'));
 		$this->setMessage('success_deleted');
 
 		$returnUrl = Context::get('success_return_url') ? Context::get('success_return_url') : getNotEncodedUrl('', 'module', 'admin', 'act', 'dispMemberAdminGroupList');
@@ -673,10 +673,10 @@ 
 		$args->default_value = explode("\n", str_replace("\r", '', Context::get('default_value')));
 		$args->required = Context::get('required');
 		$args->is_active = (isset($args->required));
-		if(!in_array(strtoupper($args->required), array('Y','N')))$args->required = 'N';
+		if (!in_array(strtoupper($args->required), array('Y', 'N')))$args->required = 'N';
 		$args->description = Context::get('description') ? Context::get('description') : '';
 		// Default values
-		if(in_array($args->column_type, array('checkbox','select','radio')) && count($args->default_value))
+		if (in_array($args->column_type, array('checkbox', 'select', 'radio')) && count($args->default_value))
 		{
 			$args->default_value = serialize($args->default_value);
 		}
@@ -688,17 +688,17 @@ 
 		// Check ID duplicated
 		$oMemberModel = getModel('member');
 		$config = $oMemberModel->getMemberConfig();
-		foreach($config->signupForm as $item)
+		foreach ($config->signupForm as $item)
 		{
-			if($item->name == $args->column_name)
+			if ($item->name == $args->column_name)
 			{
-				if($args->member_join_form_srl && $args->member_join_form_srl == $item->member_join_form_srl) continue;
-				return new Object(-1,'msg_exists_user_id');
+				if ($args->member_join_form_srl && $args->member_join_form_srl == $item->member_join_form_srl) continue;
+				return new Object(-1, 'msg_exists_user_id');
 			}
 		}
 		// Fix if member_join_form_srl exists. Add if not exists.
 		$isInsert;
-		if(!$args->member_join_form_srl)
+		if (!$args->member_join_form_srl)
 		{
 			$isInsert = true;
 			$args->list_order = $args->member_join_form_srl = getNextSequence();
@@ -709,7 +709,7 @@ 
 			$output = executeQuery('member.updateJoinForm', $args);
 		}
 
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		// memberConfig update
 		$signupItem = new stdClass();
@@ -726,15 +726,15 @@ 
 		$config = $oMemberModel->getMemberConfig();
 		unset($config->agreement);
 
-		if($isInsert)
+		if ($isInsert)
 		{
 			$config->signupForm[] = $signupItem;
 		}
 		else
 		{
-			foreach($config->signupForm as $key=>$val)
+			foreach ($config->signupForm as $key=>$val)
 			{
-				if($val->member_join_form_srl == $signupItem->member_join_form_srl)
+				if ($val->member_join_form_srl == $signupItem->member_join_form_srl)
 				{
 					$config->signupForm[$key] = $signupItem;
 				}
@@ -762,9 +762,9 @@ 
 		$config = $oMemberModel->getMemberConfig();
 		unset($config->agreement);
 
-		foreach($config->signupForm as $key=>$val)
+		foreach ($config->signupForm as $key=>$val)
 		{
-			if($val->member_join_form_srl == $member_join_form_srl)
+			if ($val->member_join_form_srl == $member_join_form_srl)
 			{
 				unset($config->signupForm[$key]);
 				break;
@@ -784,7 +784,7 @@ 
 		$member_join_form_srl = Context::get('member_join_form_srl');
 		$mode = Context::get('mode');
 
-		switch($mode)
+		switch ($mode)
 		{
 			case 'up' :
 				$output = $this->moveJoinFormUp($member_join_form_srl);
@@ -801,7 +801,7 @@ 
 			case 'update' :
 				break;
 		}
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$this->setMessage($msg_code);
 	}
@@ -820,40 +820,40 @@ 
 		$oDB->begin();
 
 		$oMemberController = getController('member');
-		foreach($members as $key=>$member_srl)
+		foreach ($members as $key=>$member_srl)
 		{
 			$args = new stdClass();
 			$args->member_srl = $member_srl;
-			switch($var->type)
+			switch ($var->type)
 			{
 				case 'modify':
 					{
-						if(count($groups) > 0)
+						if (count($groups) > 0)
 						{
 							$args->site_srl = 0;
 							// One of its members to delete all the group
 							$output = executeQuery('member.deleteMemberGroupMember', $args);
-							if(!$output->toBool())
+							if (!$output->toBool())
 							{
 								$oDB->rollback();
 								return $output;
 							}
 							// Enter one of the loop a
-							foreach($groups as $group_srl)
+							foreach ($groups as $group_srl)
 							{
-								$output = $oMemberController->addMemberToGroup($args->member_srl,$group_srl);
-								if(!$output->toBool())
+								$output = $oMemberController->addMemberToGroup($args->member_srl, $group_srl);
+								if (!$output->toBool())
 								{
 									$oDB->rollback();
 									return $output;
 								}
 							}
 						}
-						if($var->denied)
+						if ($var->denied)
 						{
 							$args->denied = $var->denied;
 							$output = executeQuery('member.updateMemberDeniedInfo', $args);
-							if(!$output->toBool())
+							if (!$output->toBool())
 							{
 								$oDB->rollback();
 								return $output;
@@ -866,7 +866,7 @@ 
 					{
 						$oMemberController->memberInfo = null;
 						$output = $oMemberController->deleteMember($member_srl);
-						if(!$output->toBool())
+						if (!$output->toBool())
 						{
 							$oDB->rollback();
 							return $output;
@@ -879,15 +879,15 @@ 
 
 		$message = $var->message;
 		// Send a message
-		if($message)
+		if ($message)
 		{
 			$oCommunicationController = getController('communication');
 
 			$logged_info = Context::get('logged_info');
-			$title = cut_str($message,10,'...');
+			$title = cut_str($message, 10, '...');
 			$sender_member_srl = $logged_info->member_srl;
 
-			foreach($members as $member_srl)
+			foreach ($members as $member_srl)
 			{
 				$oCommunicationController->sendMessage($sender_member_srl, $member_srl, $title, $message, false);
 			}
@@ -904,14 +904,14 @@ 
 	function procMemberAdminDeleteMembers()
 	{
 		$target_member_srls = Context::get('target_member_srls');
-		if(!$target_member_srls) return new Object(-1, 'msg_invalid_request');
+		if (!$target_member_srls) return new Object(-1, 'msg_invalid_request');
 		$member_srls = explode(',', $target_member_srls);
 		$oMemberController = getController('member');
 
-		foreach($member_srls as $member)
+		foreach ($member_srls as $member)
 		{
 			$output = $oMemberController->deleteMember($member);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				$this->setMessage('failed_deleted');
 				return $output;
@@ -928,11 +928,11 @@ 
 	function procMemberAdminUpdateMembersGroup()
 	{
 		$member_srl = Context::get('member_srl');
-		if(!$member_srl) return new Object(-1,'msg_invalid_request');
-		$member_srls = explode(',',$member_srl);
+		if (!$member_srl) return new Object(-1, 'msg_invalid_request');
+		$member_srls = explode(',', $member_srl);
 
 		$group_srl = Context::get('group_srls');
-		if(!is_array($group_srl)) $group_srls = explode('|@|', $group_srl);
+		if (!is_array($group_srl)) $group_srls = explode('|@|', $group_srl);
 		else $group_srls = $group_srl;
 
 		$oDB = &DB::getInstance();
@@ -941,7 +941,7 @@ 
 		$args = new stdClass;
 		$args->member_srl = $member_srl;
 		$output = executeQuery('member.deleteMembersGroup', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
@@ -949,21 +949,21 @@ 
 		// Add to a selected group
 		$group_count = count($group_srls);
 		$member_count = count($member_srls);
-		for($j=0;$j<$group_count;$j++)
+		for ($j = 0; $j < $group_count; $j++)
 		{
-			$group_srl = (int)trim($group_srls[$j]);
-			if(!$group_srl) continue;
-			for($i=0;$i<$member_count;$i++)
+			$group_srl = (int) trim($group_srls[$j]);
+			if (!$group_srl) continue;
+			for ($i = 0; $i < $member_count; $i++)
 			{
-				$member_srl = (int)trim($member_srls[$i]);
-				if(!$member_srl) continue;
+				$member_srl = (int) trim($member_srls[$i]);
+				if (!$member_srl) continue;
 
 				$args = new stdClass;
 				$args->member_srl = $member_srl;
 				$args->group_srl = $group_srl;
 
 				$output = executeQuery('member.addMemberToGroup', $args);
-				if(!$output->toBool())
+				if (!$output->toBool())
 				{
 					$oDB->rollback();
 					return $output;
@@ -976,7 +976,7 @@ 
 
 		$this->setMessage('success_updated');
 
-		if(!in_array(Context::getRequestMethod(),array('XMLRPC','JSON')))
+		if (!in_array(Context::getRequestMethod(), array('XMLRPC', 'JSON')))
 		{
 			global $lang;
 			htmlHeader();
@@ -997,19 +997,19 @@ 
 	{
 		$user_ids = Context::get('user_id');
 
-		$user_ids = explode(',',$user_ids);
+		$user_ids = explode(',', $user_ids);
 		$success_ids = array();
 
-		foreach($user_ids as $val)
+		foreach ($user_ids as $val)
 		{
 			$val = trim($val);
-			if(!$val) continue;
+			if (!$val) continue;
 
 			$output = $this->insertDeniedID($val, '');
-			if($output->toBool()) $success_ids[] = $val;
+			if ($output->toBool()) $success_ids[] = $val;
 		}
 
-		$this->add('user_ids', implode(',',$success_ids));
+		$this->add('user_ids', implode(',', $success_ids));
 
 		$returnUrl = Context::get('success_return_url') ? Context::get('success_return_url') : getNotEncodedUrl('', 'module', 'admin', 'act', 'dispMemberAdminDeniedIDList');
 		$this->setRedirectUrl($returnUrl);
@@ -1026,10 +1026,10 @@ 
 		$mode = Context::get('mode');
 		$mode = $mode ? $mode : 'insert';
 
-		if($mode == 'delete')
+		if ($mode == 'delete')
 		{
 			$output = $this->deleteDeniedNickName($nick_name);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				return $output;
 			}
@@ -1038,19 +1038,19 @@ 
 		}
 		else
 		{
-			$nick_names = explode(',',$nick_name);
+			$nick_names = explode(',', $nick_name);
 			$success_nick_names = array();
 
-			foreach($nick_names as $val)
+			foreach ($nick_names as $val)
 			{
 				$val = trim($val);
-				if(!$val) continue;
+				if (!$val) continue;
 
 				$output = $this->insertDeniedNickName($val, '');
-				if($output->toBool()) $success_nick_names[] = $val;
+				if ($output->toBool()) $success_nick_names[] = $val;
 			}
 
-			$this->add('nick_names', implode(',',$success_nick_names));
+			$this->add('nick_names', implode(',', $success_nick_names));
 		}
 	}
 
@@ -1063,16 +1063,16 @@ 
 		$user_id = Context::get('user_id');
 		$mode = Context::get('mode');
 
-		switch($mode)
+		switch ($mode)
 		{
 			case 'delete' :
 				$output = $this->deleteDeniedID($user_id);
-				if(!$output->toBool()) return $output;
+				if (!$output->toBool()) return $output;
 				$msg_code = 'success_deleted';
 				break;
 		}
 
-		$this->add('page',Context::get('page'));
+		$this->add('page', Context::get('page'));
 		$this->setMessage($msg_code);
 	}
 
@@ -1119,24 +1119,24 @@ 
 	 */
 	function insertGroup($args)
 	{
-		if(!$args->site_srl) $args->site_srl = 0;
+		if (!$args->site_srl) $args->site_srl = 0;
 		// Check the value of is_default.
-		if($args->is_default != 'Y')
+		if ($args->is_default != 'Y')
 		{
 			$args->is_default = 'N';
 		}
 		else
 		{
 			$output = executeQuery('member.updateGroupDefaultClear', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 		}
 
-		if(!isset($args->list_order) || $args->list_order=='')
+		if (!isset($args->list_order) || $args->list_order == '')
 		{
 			$args->list_order = $args->group_srl;
 		}
 
-		if(!$args->group_srl) $args->group_srl = getNextSequence();
+		if (!$args->group_srl) $args->group_srl = getNextSequence();
 		$args->list_order = $args->group_srl;
 		$output = executeQuery('member.insertGroup', $args);
 		$this->_deleteMemberGroupCache($args->site_srl);
@@ -1151,17 +1151,17 @@ 
 	 */
 	function updateGroup($args)
 	{
-		if(!$args->site_srl) $args->site_srl = 0;
+		if (!$args->site_srl) $args->site_srl = 0;
 		// Check the value of is_default.
-		if(!$args->group_srl) return new Object(-1, 'lang->msg_not_founded');
-		if($args->is_default!='Y')
+		if (!$args->group_srl) return new Object(-1, 'lang->msg_not_founded');
+		if ($args->is_default != 'Y')
 		{
 			$args->is_default = 'N';
 		}
 		else
 		{
 			$output = executeQuery('member.updateGroupDefaultClear', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 		}
 
 		$output = executeQuery('member.updateGroup', $args);
@@ -1184,8 +1184,8 @@ 
 		$columnList = array('group_srl', 'is_default');
 		$group_info = $oMemberModel->getGroup($group_srl, $columnList);
 
-		if(!$group_info) return new Object(-1, 'lang->msg_not_founded');
-		if($group_info->is_default == 'Y') return new Object(-1, 'msg_not_delete_default');
+		if (!$group_info) return new Object(-1, 'lang->msg_not_founded');
+		if ($group_info->is_default == 'Y') return new Object(-1, 'msg_not_delete_default');
 
 		// Get groups where is_default == 'Y'
 		$columnList = array('site_srl', 'group_srl');
@@ -1222,7 +1222,7 @@ 
 		$defaultGroup = $oMemberModel->getDefaultGroup(0);
 		$defaultGroupSrl = $defaultGroup->group_srl;
 		$group_srls = $vars->group_srls;
-		foreach($group_srls as $order=>$group_srl)
+		foreach ($group_srls as $order=>$group_srl)
 		{
 			$isInsert = false;
 			$update_args = new stdClass();
@@ -1231,9 +1231,9 @@ 
 			$update_args->image_mark = $vars->image_marks[$order];
 			$update_args->list_order = $order + 1;
 
-			if(!$update_args->title) continue;
+			if (!$update_args->title) continue;
 
-			if(is_numeric($group_srl)) {
+			if (is_numeric($group_srl)) {
 				$update_args->group_srl = $group_srl;
 				$output = $this->updateGroup($update_args);
 			}
@@ -1242,7 +1242,7 @@ 
 				$output = $this->insertGroup($update_args);
 			}
 
-			if($vars->defaultGroup == $group_srl) {
+			if ($vars->defaultGroup == $group_srl) {
 				$defaultGroupSrl = $update_args->group_srl;
 			}
 		}
@@ -1268,7 +1268,7 @@ 
 	{
 		$vars = Context::getRequestVars();
 
-		foreach($vars->group_srls as $key => $val)
+		foreach ($vars->group_srls as $key => $val)
 		{
 			$args = new stdClass;
 			$args->group_srl = $val;
@@ -1289,7 +1289,7 @@ 
 	{
 		//remove from cache
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$oCacheHandler->invalidateGroupKey('member');
 		}
@@ -1306,7 +1306,7 @@ 
 		$args = new stdClass();
 		$args->user_id = $user_id;
 		$args->description = $description;
-		$args->list_order = -1*getNextSequence();
+		$args->list_order = -1 * getNextSequence();
 
 		return executeQuery('member.insertDeniedID', $args);
 	}
@@ -1327,7 +1327,7 @@ 
 	 */
 	function deleteDeniedID($user_id)
 	{
-		if(!$user_id) unset($user_id);
+		if (!$user_id) unset($user_id);
 
 		$args = new stdClass;
 		$args->user_id = $user_id;
@@ -1341,7 +1341,7 @@ 
 	 */
 	function deleteDeniedNickName($nick_name)
 	{
-		if(!$nick_name) unset($nick_name);
+		if (!$nick_name) unset($nick_name);
 
 		$args = new stdClass;
 		$args->nick_name = $nick_name;
@@ -1380,16 +1380,16 @@ 
 		// Get a list of all join forms
 		$join_form_list = $oMemberModel->getJoinFormList();
 		$join_form_srl_list = array_keys($join_form_list);
-		if(count($join_form_srl_list)<2) return new Object();
+		if (count($join_form_srl_list) < 2) return new Object();
 
 		$prev_member_join_form = NULL;
-		foreach($join_form_list as $key => $val)
+		foreach ($join_form_list as $key => $val)
 		{
-			if($val->member_join_form_srl == $member_join_form_srl) break;
+			if ($val->member_join_form_srl == $member_join_form_srl) break;
 			$prev_member_join_form = $val;
 		}
 		// Return if no previous join form exists
-		if(!$prev_member_join_form) return new Object();
+		if (!$prev_member_join_form) return new Object();
 		// Information of the join form
 		$cur_args = new stdClass;
 		$cur_args->member_join_form_srl = $member_join_form_srl;
@@ -1400,10 +1400,10 @@ 
 		$prev_args->list_order = $list_order;
 		// Execute Query
 		$output = executeQuery('member.updateMemberJoinFormListorder', $cur_args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		executeQuery('member.updateMemberJoinFormListorder', $prev_args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		return new Object();
 	}
@@ -1427,16 +1427,16 @@ 
 		// Get information of all join forms
 		$join_form_list = $oMemberModel->getJoinFormList();
 		$join_form_srl_list = array_keys($join_form_list);
-		if(count($join_form_srl_list)<2) return new Object();
+		if (count($join_form_srl_list) < 2) return new Object();
 
-		for($i=0;$i<count($join_form_srl_list);$i++)
+		for ($i = 0; $i < count($join_form_srl_list); $i++)
 		{
-			if($join_form_srl_list[$i]==$member_join_form_srl) break;
+			if ($join_form_srl_list[$i] == $member_join_form_srl) break;
 		}
 
-		$next_member_join_form_srl = $join_form_srl_list[$i+1];
+		$next_member_join_form_srl = $join_form_srl_list[$i + 1];
 		// Return if no previous join form exists
-		if(!$next_member_join_form_srl) return new Object();
+		if (!$next_member_join_form_srl) return new Object();
 		$next_member_join_form = $join_form_list[$next_member_join_form_srl];
 		// Information of the join form
 		$cur_args = new stdClass;
@@ -1448,10 +1448,10 @@ 
 		$next_args->list_order = $list_order;
 		// Execute Query
 		$output = executeQuery('member.updateMemberJoinFormListorder', $cur_args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$output = executeQuery('member.updateMemberJoinFormListorder', $next_args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		return new Object();
 	}

Braces +162 added lines, -86 removed lines

@@ -48,9 +48,11 @@ 
 			$args->{$val} = Context::get($val);
 		}
 		$args->member_srl = Context::get('member_srl');
-		if(Context::get('reset_password'))
-			$args->password = Context::get('reset_password');
-		else unset($args->password);
+		if(Context::get('reset_password')) {
+					$args->password = Context::get('reset_password');
+		} else {
+			unset($args->password);
+		}
 
 		// Remove some unnecessary variables from all the vars
 		$all_args = Context::getRequestVars();
@@ -60,7 +62,9 @@ 
 		unset($all_args->error_return_url);
 		unset($all_args->success_return_url);
 		unset($all_args->ruleset);
-		if(!isset($args->limit_date)) $args->limit_date = "";
+		if(!isset($args->limit_date)) {
+			$args->limit_date = "";
+		}
 		unset($all_args->password);
 		unset($all_args->password2);
 		unset($all_args->reset_password);
@@ -76,7 +80,9 @@ 
 			$columnList = array('member_srl');
 			$member_info = $oMemberModel->getMemberInfoByMemberSrl($args->member_srl, 0, $columnList);
 			// If no original member exists, make a new one
-			if($member_info->member_srl != $args->member_srl) unset($args->member_srl);
+			if($member_info->member_srl != $args->member_srl) {
+				unset($args->member_srl);
+			}
 		}
 
 		// remove whitespace
@@ -96,14 +102,15 @@ 
 			$args->password = Context::get('password');
 			$output = $oMemberController->insertMember($args);
 			$msg_code = 'success_registed';
-		}
-		else
+		} else
 		{
 			$output = $oMemberController->updateMember($args);
 			$msg_code = 'success_updated';
 		}
 
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 		// Save Signature
 		$signature = Context::get('signature');
 		$oMemberController->putSignature($args->member_srl, $signature);
@@ -144,7 +151,9 @@ 
 
 		$oMemberController = getController('member');
 		$output = $oMemberController->deleteMember($member_srl);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->add('page',Context::get('page'));
 		$this->setMessage("success_deleted");
@@ -263,8 +272,11 @@ 
 			$signupItem->isDefaultForm = in_array($key, $items);
 
 			$signupItem->name = $key;
-			if(!in_array($key, $items)) $signupItem->title = $key;
-			else $signupItem->title = $lang->{$key};
+			if(!in_array($key, $items)) {
+				$signupItem->title = $key;
+			} else {
+				$signupItem->title = $lang->{$key};
+			}
 			$signupItem->mustRequired = in_array($key, $mustRequireds);
 			$signupItem->imageType = (strpos($key, 'image') !== false);
 			$signupItem->required = ($all_args->{$key} == 'required') || $signupItem->mustRequired || $signupItem->isIdentifier;
@@ -434,10 +446,11 @@ 
 				$signupItem->max_width = $config->{$key.'_max_width'};
 				$signupItem->max_height = $config->{$key.'_max_height'};
 			}
-			if($signupItem->isIdentifier)
-				array_unshift($list_order, $signupItem);
-			else
-				$list_order[] = $signupItem;
+			if($signupItem->isIdentifier) {
+							array_unshift($list_order, $signupItem);
+			} else {
+							$list_order[] = $signupItem;
+			}
 		}
 		if(is_array($extendItems))
 		{
@@ -494,38 +507,30 @@ 
 				if($formInfo->type == 'tel' || $formInfo->type == 'kr_zip')
 				{
 					$fields[] = sprintf('<field name="%s[]" required="true" />', $formInfo->name);
-				}
-				else if($formInfo->name == 'password')
+				} else if($formInfo->name == 'password')
 				{
 					$fields[] = '<field name="password"><if test="$act == \'procMemberInsert\'" attr="required" value="true" /><if test="$act == \'procMemberInsert\'" attr="length" value="4:60" /></field>';
 					$fields[] = '<field name="password2"><if test="$act == \'procMemberInsert\'" attr="required" value="true" /><if test="$act == \'procMemberInsert\'" attr="equalto" value="password" /></field>';
-				}
-				else if($formInfo->name == 'find_account_question')
+				} else if($formInfo->name == 'find_account_question')
 				{
 					$fields[] = '<field name="find_account_question"><if test="$modify_find_account_answer" attr="required" value="true" /></field>';
 					$fields[] = '<field name="find_account_answer" length=":250"><if test="$modify_find_account_answer" attr="required" value="true" /></field>';
-				}
-				else if($formInfo->name == 'email_address')
+				} else if($formInfo->name == 'email_address')
 				{
 					$fields[] = sprintf('<field name="%s" required="true" rule="email"/>', $formInfo->name);
-				}
-				else if($formInfo->name == 'user_id')
+				} else if($formInfo->name == 'user_id')
 				{
 					$fields[] = sprintf('<field name="%s" required="true" rule="userid" length="3:20" />', $formInfo->name);
-				}
-				else if($formInfo->name == 'nick_name')
+				} else if($formInfo->name == 'nick_name')
 				{
 					$fields[] = sprintf('<field name="%s" required="true" length="2:20" />', $formInfo->name);
-				}
-				else if(strpos($formInfo->name, 'image') !== false)
+				} else if(strpos($formInfo->name, 'image') !== false)
 				{
 					$fields[] = sprintf('<field name="%s"><if test="$act != \'procMemberAdminInsert\' &amp;&amp; $__%s_exist != \'true\'" attr="required" value="true" /></field>', $formInfo->name, $formInfo->name);
-				}
-				else if($formInfo->name == 'signature')
+				} else if($formInfo->name == 'signature')
 				{
 					$fields[] = '<field name="signature"><if test="$member_srl" attr="required" value="true" /></field>';
-				}
-				else
+				} else
 				{
 					$fields[] = sprintf('<field name="%s" required="true" />', $formInfo->name);
 				}
@@ -585,8 +590,9 @@ 
 			'</ruleset>';
 
 		$fields = array();
-		if($identifier == 'user_id')
-			$fields[] = '<field name="user_id" required="true" rule="userid" />';
+		if($identifier == 'user_id') {
+					$fields[] = '<field name="user_id" required="true" rule="userid" />';
+		}
 
 		$fields[] = '<field name="email_address" required="true" rule="email" />';
 		$fields[] = '<field name="find_account_question" required="true" />';
@@ -608,7 +614,9 @@ 
 	{
 		$args = Context::gets('title','description','is_default','image_mark');
 		$output = $this->insertGroup($args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->add('group_srl','');
 		$this->add('page',Context::get('page'));
@@ -629,7 +637,9 @@ 
 		$args = Context::gets('group_srl','title','description','is_default','image_mark');
 		$args->site_srl = 0;
 		$output = $this->updateGroup($args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->add('group_srl','');
 		$this->add('page',Context::get('page'));
@@ -648,7 +658,9 @@ 
 		$group_srl = Context::get('group_srl');
 
 		$output = $this->deleteGroup($group_srl);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->add('group_srl','');
 		$this->add('page',Context::get('page'));
@@ -673,14 +685,15 @@ 
 		$args->default_value = explode("\n", str_replace("\r", '', Context::get('default_value')));
 		$args->required = Context::get('required');
 		$args->is_active = (isset($args->required));
-		if(!in_array(strtoupper($args->required), array('Y','N')))$args->required = 'N';
+		if(!in_array(strtoupper($args->required), array('Y','N'))) {
+			$args->required = 'N';
+		}
 		$args->description = Context::get('description') ? Context::get('description') : '';
 		// Default values
 		if(in_array($args->column_type, array('checkbox','select','radio')) && count($args->default_value))
 		{
 			$args->default_value = serialize($args->default_value);
-		}
-		else
+		} else
 		{
 			$args->default_value = '';
 		}
@@ -692,7 +705,9 @@ 
 		{
 			if($item->name == $args->column_name)
 			{
-				if($args->member_join_form_srl && $args->member_join_form_srl == $item->member_join_form_srl) continue;
+				if($args->member_join_form_srl && $args->member_join_form_srl == $item->member_join_form_srl) {
+					continue;
+				}
 				return new Object(-1,'msg_exists_user_id');
 			}
 		}
@@ -703,13 +718,14 @@ 
 			$isInsert = true;
 			$args->list_order = $args->member_join_form_srl = getNextSequence();
 			$output = executeQuery('member.insertJoinForm', $args);
-		}
-		else
+		} else
 		{
 			$output = executeQuery('member.updateJoinForm', $args);
 		}
 
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		// memberConfig update
 		$signupItem = new stdClass();
@@ -729,8 +745,7 @@ 
 		if($isInsert)
 		{
 			$config->signupForm[] = $signupItem;
-		}
-		else
+		} else
 		{
 			foreach($config->signupForm as $key=>$val)
 			{
@@ -801,7 +816,9 @@ 
 			case 'update' :
 				break;
 		}
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->setMessage($msg_code);
 	}
@@ -904,7 +921,9 @@ 
 	function procMemberAdminDeleteMembers()
 	{
 		$target_member_srls = Context::get('target_member_srls');
-		if(!$target_member_srls) return new Object(-1, 'msg_invalid_request');
+		if(!$target_member_srls) {
+			return new Object(-1, 'msg_invalid_request');
+		}
 		$member_srls = explode(',', $target_member_srls);
 		$oMemberController = getController('member');
 
@@ -928,12 +947,17 @@ 
 	function procMemberAdminUpdateMembersGroup()
 	{
 		$member_srl = Context::get('member_srl');
-		if(!$member_srl) return new Object(-1,'msg_invalid_request');
+		if(!$member_srl) {
+			return new Object(-1,'msg_invalid_request');
+		}
 		$member_srls = explode(',',$member_srl);
 
 		$group_srl = Context::get('group_srls');
-		if(!is_array($group_srl)) $group_srls = explode('|@|', $group_srl);
-		else $group_srls = $group_srl;
+		if(!is_array($group_srl)) {
+			$group_srls = explode('|@|', $group_srl);
+		} else {
+			$group_srls = $group_srl;
+		}
 
 		$oDB = &DB::getInstance();
 		$oDB->begin();
@@ -952,11 +976,15 @@ 
 		for($j=0;$j<$group_count;$j++)
 		{
 			$group_srl = (int)trim($group_srls[$j]);
-			if(!$group_srl) continue;
+			if(!$group_srl) {
+				continue;
+			}
 			for($i=0;$i<$member_count;$i++)
 			{
 				$member_srl = (int)trim($member_srls[$i]);
-				if(!$member_srl) continue;
+				if(!$member_srl) {
+					continue;
+				}
 
 				$args = new stdClass;
 				$args->member_srl = $member_srl;
@@ -1003,10 +1031,14 @@ 
 		foreach($user_ids as $val)
 		{
 			$val = trim($val);
-			if(!$val) continue;
+			if(!$val) {
+				continue;
+			}
 
 			$output = $this->insertDeniedID($val, '');
-			if($output->toBool()) $success_ids[] = $val;
+			if($output->toBool()) {
+				$success_ids[] = $val;
+			}
 		}
 
 		$this->add('user_ids', implode(',',$success_ids));
@@ -1035,8 +1067,7 @@ 
 			}
 			$msg_code = 'success_deleted';
 			$this->setMessage($msg_code);
-		}
-		else
+		} else
 		{
 			$nick_names = explode(',',$nick_name);
 			$success_nick_names = array();
@@ -1044,10 +1075,14 @@ 
 			foreach($nick_names as $val)
 			{
 				$val = trim($val);
-				if(!$val) continue;
+				if(!$val) {
+					continue;
+				}
 
 				$output = $this->insertDeniedNickName($val, '');
-				if($output->toBool()) $success_nick_names[] = $val;
+				if($output->toBool()) {
+					$success_nick_names[] = $val;
+				}
 			}
 
 			$this->add('nick_names', implode(',',$success_nick_names));
@@ -1067,7 +1102,9 @@ 
 		{
 			case 'delete' :
 				$output = $this->deleteDeniedID($user_id);
-				if(!$output->toBool()) return $output;
+				if(!$output->toBool()) {
+					return $output;
+				}
 				$msg_code = 'success_deleted';
 				break;
 		}
@@ -1119,16 +1156,19 @@ 
 	 */
 	function insertGroup($args)
 	{
-		if(!$args->site_srl) $args->site_srl = 0;
+		if(!$args->site_srl) {
+			$args->site_srl = 0;
+		}
 		// Check the value of is_default.
 		if($args->is_default != 'Y')
 		{
 			$args->is_default = 'N';
-		}
-		else
+		} else
 		{
 			$output = executeQuery('member.updateGroupDefaultClear', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 		}
 
 		if(!isset($args->list_order) || $args->list_order=='')
@@ -1136,7 +1176,9 @@ 
 			$args->list_order = $args->group_srl;
 		}
 
-		if(!$args->group_srl) $args->group_srl = getNextSequence();
+		if(!$args->group_srl) {
+			$args->group_srl = getNextSequence();
+		}
 		$args->list_order = $args->group_srl;
 		$output = executeQuery('member.insertGroup', $args);
 		$this->_deleteMemberGroupCache($args->site_srl);
@@ -1151,17 +1193,22 @@ 
 	 */
 	function updateGroup($args)
 	{
-		if(!$args->site_srl) $args->site_srl = 0;
+		if(!$args->site_srl) {
+			$args->site_srl = 0;
+		}
 		// Check the value of is_default.
-		if(!$args->group_srl) return new Object(-1, 'lang->msg_not_founded');
+		if(!$args->group_srl) {
+			return new Object(-1, 'lang->msg_not_founded');
+		}
 		if($args->is_default!='Y')
 		{
 			$args->is_default = 'N';
-		}
-		else
+		} else
 		{
 			$output = executeQuery('member.updateGroupDefaultClear', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 		}
 
 		$output = executeQuery('member.updateGroup', $args);
@@ -1184,8 +1231,12 @@ 
 		$columnList = array('group_srl', 'is_default');
 		$group_info = $oMemberModel->getGroup($group_srl, $columnList);
 
-		if(!$group_info) return new Object(-1, 'lang->msg_not_founded');
-		if($group_info->is_default == 'Y') return new Object(-1, 'msg_not_delete_default');
+		if(!$group_info) {
+			return new Object(-1, 'lang->msg_not_founded');
+		}
+		if($group_info->is_default == 'Y') {
+			return new Object(-1, 'msg_not_delete_default');
+		}
 
 		// Get groups where is_default == 'Y'
 		$columnList = array('site_srl', 'group_srl');
@@ -1231,13 +1282,14 @@ 
 			$update_args->image_mark = $vars->image_marks[$order];
 			$update_args->list_order = $order + 1;
 
-			if(!$update_args->title) continue;
+			if(!$update_args->title) {
+				continue;
+			}
 
 			if(is_numeric($group_srl)) {
 				$update_args->group_srl = $group_srl;
 				$output = $this->updateGroup($update_args);
-			}
-			else {
+			} else {
 				$update_args->group_srl = getNextSequence();
 				$output = $this->insertGroup($update_args);
 			}
@@ -1327,7 +1379,9 @@ 
 	 */
 	function deleteDeniedID($user_id)
 	{
-		if(!$user_id) unset($user_id);
+		if(!$user_id) {
+			unset($user_id);
+		}
 
 		$args = new stdClass;
 		$args->user_id = $user_id;
@@ -1341,7 +1395,9 @@ 
 	 */
 	function deleteDeniedNickName($nick_name)
 	{
-		if(!$nick_name) unset($nick_name);
+		if(!$nick_name) {
+			unset($nick_name);
+		}
 
 		$args = new stdClass;
 		$args->nick_name = $nick_name;
@@ -1380,16 +1436,22 @@ 
 		// Get a list of all join forms
 		$join_form_list = $oMemberModel->getJoinFormList();
 		$join_form_srl_list = array_keys($join_form_list);
-		if(count($join_form_srl_list)<2) return new Object();
+		if(count($join_form_srl_list)<2) {
+			return new Object();
+		}
 
 		$prev_member_join_form = NULL;
 		foreach($join_form_list as $key => $val)
 		{
-			if($val->member_join_form_srl == $member_join_form_srl) break;
+			if($val->member_join_form_srl == $member_join_form_srl) {
+				break;
+			}
 			$prev_member_join_form = $val;
 		}
 		// Return if no previous join form exists
-		if(!$prev_member_join_form) return new Object();
+		if(!$prev_member_join_form) {
+			return new Object();
+		}
 		// Information of the join form
 		$cur_args = new stdClass;
 		$cur_args->member_join_form_srl = $member_join_form_srl;
@@ -1400,10 +1462,14 @@ 
 		$prev_args->list_order = $list_order;
 		// Execute Query
 		$output = executeQuery('member.updateMemberJoinFormListorder', $cur_args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		executeQuery('member.updateMemberJoinFormListorder', $prev_args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		return new Object();
 	}
@@ -1427,16 +1493,22 @@ 
 		// Get information of all join forms
 		$join_form_list = $oMemberModel->getJoinFormList();
 		$join_form_srl_list = array_keys($join_form_list);
-		if(count($join_form_srl_list)<2) return new Object();
+		if(count($join_form_srl_list)<2) {
+			return new Object();
+		}
 
 		for($i=0;$i<count($join_form_srl_list);$i++)
 		{
-			if($join_form_srl_list[$i]==$member_join_form_srl) break;
+			if($join_form_srl_list[$i]==$member_join_form_srl) {
+				break;
+			}
 		}
 
 		$next_member_join_form_srl = $join_form_srl_list[$i+1];
 		// Return if no previous join form exists
-		if(!$next_member_join_form_srl) return new Object();
+		if(!$next_member_join_form_srl) {
+			return new Object();
+		}
 		$next_member_join_form = $join_form_list[$next_member_join_form_srl];
 		// Information of the join form
 		$cur_args = new stdClass;
@@ -1448,10 +1520,14 @@ 
 		$next_args->list_order = $list_order;
 		// Execute Query
 		$output = executeQuery('member.updateMemberJoinFormListorder', $cur_args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$output = executeQuery('member.updateMemberJoinFormListorder', $next_args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		return new Object();
 	}

modules/communication/communication.class.php

Spacing +11 added lines, -11 removed lines

@@ -29,17 +29,17 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
 			$config = $oModuleModel->getModuleConfig('message');
 
-			if($config->skin)
+			if ($config->skin)
 			{
 				$config_parse = explode('.', $config->skin);
-				if(count($config_parse) > 1)
+				if (count($config_parse) > 1)
 				{
 					$template_path = sprintf('./themes/%s/modules/communication/', $config_parse[0]);
-					if(is_dir($template_path))
+					if (is_dir($template_path))
 					{
 						return TRUE;
 					}
@@ -49,7 +49,7 @@ 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
 
-		if(!is_dir("./files/member_extra_info/new_message_flags"))
+		if (!is_dir("./files/member_extra_info/new_message_flags"))
 		{
 			return TRUE;
 		}
@@ -66,21 +66,21 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
 			$config = $oModuleModel->getModuleConfig('message');
-			if(!is_object($config))
+			if (!is_object($config))
 			{
 				$config = new stdClass();
 			}
 
-			if($config->skin)
+			if ($config->skin)
 			{
 				$config_parse = explode('.', $config->skin);
-				if(count($config_parse) > 1)
+				if (count($config_parse) > 1)
 				{
 					$template_path = sprintf('./themes/%s/modules/communication/', $config_parse[0]);
-					if(is_dir($template_path))
+					if (is_dir($template_path))
 					{
 						$config->skin = implode('|@|', $config_parse);
 						$oModuleController = getController('module');
@@ -92,7 +92,7 @@ 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
 
-		if(!is_dir("./files/member_extra_info/new_message_flags"))
+		if (!is_dir("./files/member_extra_info/new_message_flags"))
 		{
 			FileHandler::makeDir('./files/member_extra_info/new_message_flags');
 		}

modules/document/document.class.php

Spacing +93 added lines, -93 removed lines

@@ -17,7 +17,7 @@ 
 	 * Search option to use in admin page
 	 * @var array
 	 */
-	var $search_option = array('title','content','title_content','user_name',); // /< Search options
+	var $search_option = array('title', 'content', 'title_content', 'user_name',); // /< Search options
 	/**
 	 * Status list
 	 * @var array
@@ -34,22 +34,22 @@ 
 		$oModuleController = getController('module');
 
 		$oDB = &DB::getInstance();
-		$oDB->addIndex("documents","idx_module_list_order", array("module_srl","list_order"));
-		$oDB->addIndex("documents","idx_module_update_order", array("module_srl","update_order"));
-		$oDB->addIndex("documents","idx_module_readed_count", array("module_srl","readed_count"));
-		$oDB->addIndex("documents","idx_module_voted_count", array("module_srl","voted_count"));
-		$oDB->addIndex("documents","idx_module_notice", array("module_srl","is_notice"));
-		$oDB->addIndex("documents","idx_module_document_srl", array("module_srl","document_srl"));
-		$oDB->addIndex("documents","idx_module_blamed_count", array("module_srl","blamed_count"));
-		$oDB->addIndex("document_aliases", "idx_module_title", array("module_srl","alias_title"), true);
-		$oDB->addIndex("document_extra_vars", "unique_extra_vars", array("module_srl","document_srl","var_idx","lang_code"), true);
+		$oDB->addIndex("documents", "idx_module_list_order", array("module_srl", "list_order"));
+		$oDB->addIndex("documents", "idx_module_update_order", array("module_srl", "update_order"));
+		$oDB->addIndex("documents", "idx_module_readed_count", array("module_srl", "readed_count"));
+		$oDB->addIndex("documents", "idx_module_voted_count", array("module_srl", "voted_count"));
+		$oDB->addIndex("documents", "idx_module_notice", array("module_srl", "is_notice"));
+		$oDB->addIndex("documents", "idx_module_document_srl", array("module_srl", "document_srl"));
+		$oDB->addIndex("documents", "idx_module_blamed_count", array("module_srl", "blamed_count"));
+		$oDB->addIndex("document_aliases", "idx_module_title", array("module_srl", "alias_title"), true);
+		$oDB->addIndex("document_extra_vars", "unique_extra_vars", array("module_srl", "document_srl", "var_idx", "lang_code"), true);
 		// 2007. 10. 17 Add a trigger to delete all posts together when the module is deleted
 		$oModuleController->insertTrigger('module.deleteModule', 'document', 'controller', 'triggerDeleteModuleDocuments', 'after');
 
 		// 2009. 01. 29 Added a trigger for additional setup
 		$oModuleController->insertTrigger('module.dispAdditionSetup', 'document', 'view', 'triggerDispDocumentAdditionSetup', 'before');
 
-		if(!is_dir('./files/cache/tmp'))
+		if (!is_dir('./files/cache/tmp'))
 		{
 			FileHandler::makeDir('./files/cache/tmp');
 		}
@@ -67,71 +67,71 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
 			// 2007. 7. 25: Add a column(notify_message) for notification
-			if(!$oDB->isColumnExists("documents","notify_message")) return true;
+			if (!$oDB->isColumnExists("documents", "notify_message")) return true;
 
 			// 2007. 8. 23: create a clustered index in the document table
-			if(!$oDB->isIndexExists("documents","idx_module_list_order")) return true;
-			if(!$oDB->isIndexExists("documents","idx_module_update_order")) return true;
-			if(!$oDB->isIndexExists("documents","idx_module_readed_count")) return true;
-			if(!$oDB->isIndexExists("documents","idx_module_voted_count")) return true;
+			if (!$oDB->isIndexExists("documents", "idx_module_list_order")) return true;
+			if (!$oDB->isIndexExists("documents", "idx_module_update_order")) return true;
+			if (!$oDB->isIndexExists("documents", "idx_module_readed_count")) return true;
+			if (!$oDB->isIndexExists("documents", "idx_module_voted_count")) return true;
 			// 2007. 10. 17 Add a trigger to delete all posts together when the module is deleted
-			if(!$oModuleModel->getTrigger('module.deleteModule', 'document', 'controller', 'triggerDeleteModuleDocuments', 'after')) return true;
+			if (!$oModuleModel->getTrigger('module.deleteModule', 'document', 'controller', 'triggerDeleteModuleDocuments', 'after')) return true;
 			// 2007. 10. 25 add parent_srl, expand to the document category
-			if(!$oDB->isColumnExists("document_categories","parent_srl")) return true;
-			if(!$oDB->isColumnExists("document_categories","expand")) return true;
-			if(!$oDB->isColumnExists("document_categories","group_srls")) return true;
+			if (!$oDB->isColumnExists("document_categories", "parent_srl")) return true;
+			if (!$oDB->isColumnExists("document_categories", "expand")) return true;
+			if (!$oDB->isColumnExists("document_categories", "group_srls")) return true;
 			// 2007. 11. 20 create a composite index on the columns(module_srl + is_notice)
-			if(!$oDB->isIndexExists("documents","idx_module_notice")) return true;
+			if (!$oDB->isIndexExists("documents", "idx_module_notice")) return true;
 			// 2008. 02. 18 create a composite index on the columns(module_srl + document_srl) (checked by Manian))
-			if(!$oDB->isIndexExists("documents","idx_module_document_srl")) return true;
+			if (!$oDB->isIndexExists("documents", "idx_module_document_srl")) return true;
 
 			// 2007. 12. 03: Add if the colume(extra_vars) doesn't exist
-			if(!$oDB->isColumnExists("documents","extra_vars")) return true;
+			if (!$oDB->isColumnExists("documents", "extra_vars")) return true;
 			// 2008. 04. 23 Add a column(blamed_count)
-			if(!$oDB->isColumnExists("documents", "blamed_count")) return true;
-			if(!$oDB->isIndexExists("documents","idx_module_blamed_count")) return true;
-			if(!$oDB->isColumnExists("document_voted_log", "point")) return true;
+			if (!$oDB->isColumnExists("documents", "blamed_count")) return true;
+			if (!$oDB->isIndexExists("documents", "idx_module_blamed_count")) return true;
+			if (!$oDB->isColumnExists("document_voted_log", "point")) return true;
 			// 2008-12-15 Add a column(color)
-			if(!$oDB->isColumnExists("document_categories", "color")) return true;
+			if (!$oDB->isColumnExists("document_categories", "color")) return true;
 
 			/**
 			 * 2009. 01. 29: Add a column(lang_code) if not exist in the document_extra_vars table
 			 */
-			if(!$oDB->isColumnExists("document_extra_vars","lang_code")) return true;
+			if (!$oDB->isColumnExists("document_extra_vars", "lang_code")) return true;
 
-			if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'document', 'view', 'triggerDispDocumentAdditionSetup', 'before')) return true;
+			if (!$oModuleModel->getTrigger('module.dispAdditionSetup', 'document', 'view', 'triggerDispDocumentAdditionSetup', 'before')) return true;
 			// 2009. 03. 09 Add a column(lang_code) to the documnets table
-			if(!$oDB->isColumnExists("documents","lang_code")) return true;
+			if (!$oDB->isColumnExists("documents", "lang_code")) return true;
 			// 2009. 03. 11 check the index in the document_extra_vars table
-			if(!$oDB->isIndexExists("document_extra_vars", "unique_extra_vars")) return true;
+			if (!$oDB->isIndexExists("document_extra_vars", "unique_extra_vars")) return true;
 
 			// 2009. 03. 19: Add a column(eid) if not exist in the table
-			if(!$oDB->isColumnExists("document_extra_keys","eid")) return true;
-			if(!$oDB->isColumnExists("document_extra_vars","eid")) return true;
+			if (!$oDB->isColumnExists("document_extra_keys", "eid")) return true;
+			if (!$oDB->isColumnExists("document_extra_vars", "eid")) return true;
 
 			// 2011. 03. 30 Cubrid index Check the index in the document_extra_vars table
-			if(!$oDB->isIndexExists("document_extra_vars", "idx_document_list_order")) return true;
+			if (!$oDB->isIndexExists("document_extra_vars", "idx_document_list_order")) return true;
 
 			//2011. 04. 07 adding description column to document categories
-			if(!$oDB->isColumnExists("document_categories","description")) return true;
+			if (!$oDB->isColumnExists("document_categories", "description")) return true;
 
 			//2011. 05. 23 adding status column to document
-			if(!$oDB->isColumnExists('documents', 'status')) return true;
+			if (!$oDB->isColumnExists('documents', 'status')) return true;
 
 			//2011. 06. 07 check comment status update
-			if($oDB->isColumnExists('documents', 'allow_comment') || $oDB->isColumnExists('documents', 'lock_comment')) return true;
+			if ($oDB->isColumnExists('documents', 'allow_comment') || $oDB->isColumnExists('documents', 'lock_comment')) return true;
 
 			// 2011. 10. 25 status index check
-			if(!$oDB->isIndexExists("documents", "idx_module_status")) return true;
+			if (!$oDB->isIndexExists("documents", "idx_module_status")) return true;
 
 			// 2012. 02. 27 Add a trigger to copy extra keys when the module is copied 
-			if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModuleExtraKeys', 'after')) return true;
+			if (!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModuleExtraKeys', 'after')) return true;
 
 			// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-			if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModule', 'after')) return true;
+			if (!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModule', 'after')) return true;
 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
@@ -149,101 +149,101 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
 			// 2007. 7. 25: Add a column(notify_message) for notification
-			if(!$oDB->isColumnExists("documents","notify_message"))
+			if (!$oDB->isColumnExists("documents", "notify_message"))
 			{
-				$oDB->addColumn('documents',"notify_message","char",1);
+				$oDB->addColumn('documents', "notify_message", "char", 1);
 			}
 
 			// 2007. 8. 23: create a clustered index in the document table
-			if(!$oDB->isIndexExists("documents","idx_module_list_order"))
+			if (!$oDB->isIndexExists("documents", "idx_module_list_order"))
 			{
-				$oDB->addIndex("documents","idx_module_list_order", array("module_srl","list_order"));
+				$oDB->addIndex("documents", "idx_module_list_order", array("module_srl", "list_order"));
 			}
 
-			if(!$oDB->isIndexExists("documents","idx_module_update_order"))
+			if (!$oDB->isIndexExists("documents", "idx_module_update_order"))
 			{
-				$oDB->addIndex("documents","idx_module_update_order", array("module_srl","update_order"));
+				$oDB->addIndex("documents", "idx_module_update_order", array("module_srl", "update_order"));
 			}
 
-			if(!$oDB->isIndexExists("documents","idx_module_readed_count"))
+			if (!$oDB->isIndexExists("documents", "idx_module_readed_count"))
 			{
-				$oDB->addIndex("documents","idx_module_readed_count", array("module_srl","readed_count"));
+				$oDB->addIndex("documents", "idx_module_readed_count", array("module_srl", "readed_count"));
 			}
 
-			if(!$oDB->isIndexExists("documents","idx_module_voted_count"))
+			if (!$oDB->isIndexExists("documents", "idx_module_voted_count"))
 			{
-				$oDB->addIndex("documents","idx_module_voted_count", array("module_srl","voted_count"));
+				$oDB->addIndex("documents", "idx_module_voted_count", array("module_srl", "voted_count"));
 			}
 			// 2007. 10. 17 Add a trigger to delete all posts together when the module is deleted
-			if(!$oModuleModel->getTrigger('module.deleteModule', 'document', 'controller', 'triggerDeleteModuleDocuments', 'after'))
+			if (!$oModuleModel->getTrigger('module.deleteModule', 'document', 'controller', 'triggerDeleteModuleDocuments', 'after'))
 				$oModuleController->insertTrigger('module.deleteModule', 'document', 'controller', 'triggerDeleteModuleDocuments', 'after');
 			// 2007. 10. 25 add columns(parent_srl, expand) 
-			if(!$oDB->isColumnExists("document_categories","parent_srl")) $oDB->addColumn('document_categories',"parent_srl","number",12,0);
-			if(!$oDB->isColumnExists("document_categories","expand")) $oDB->addColumn('document_categories',"expand","char",1,"N");
-			if(!$oDB->isColumnExists("document_categories","group_srls")) $oDB->addColumn('document_categories',"group_srls","text");
+			if (!$oDB->isColumnExists("document_categories", "parent_srl")) $oDB->addColumn('document_categories', "parent_srl", "number", 12, 0);
+			if (!$oDB->isColumnExists("document_categories", "expand")) $oDB->addColumn('document_categories', "expand", "char", 1, "N");
+			if (!$oDB->isColumnExists("document_categories", "group_srls")) $oDB->addColumn('document_categories', "group_srls", "text");
 			// 2007. 11. 20 create a composite index on the columns(module_srl + is_notice)
-			if(!$oDB->isIndexExists("documents","idx_module_notice")) $oDB->addIndex("documents","idx_module_notice", array("module_srl","is_notice"));
+			if (!$oDB->isIndexExists("documents", "idx_module_notice")) $oDB->addIndex("documents", "idx_module_notice", array("module_srl", "is_notice"));
 
 			// 2007. 12. 03: Add if the colume(extra_vars) doesn't exist
-			if(!$oDB->isColumnExists("documents","extra_vars")) $oDB->addColumn('documents','extra_vars','text');
+			if (!$oDB->isColumnExists("documents", "extra_vars")) $oDB->addColumn('documents', 'extra_vars', 'text');
 
 			// 2008. 02. 18 create a composite index on the columns(module_srl + document_srl) (checked by Manian))
-			if(!$oDB->isIndexExists("documents","idx_module_document_srl")) $oDB->addIndex("documents","idx_module_document_srl", array("module_srl","document_srl"));
+			if (!$oDB->isIndexExists("documents", "idx_module_document_srl")) $oDB->addIndex("documents", "idx_module_document_srl", array("module_srl", "document_srl"));
 			// 2008. 04. 23 Add a column(blamed count)
-			if(!$oDB->isColumnExists("documents", "blamed_count"))
+			if (!$oDB->isColumnExists("documents", "blamed_count"))
 			{
 				$oDB->addColumn('documents', 'blamed_count', 'number', 11, 0, true);
 				$oDB->addIndex('documents', 'idx_blamed_count', array('blamed_count'));
 			}
 
-			if(!$oDB->isIndexExists("documents","idx_module_blamed_count"))
+			if (!$oDB->isIndexExists("documents", "idx_module_blamed_count"))
 			{
 				$oDB->addIndex('documents', 'idx_module_blamed_count', array('module_srl', 'blamed_count'));
 			}
 
-			if(!$oDB->isColumnExists("document_voted_log", "point"))
+			if (!$oDB->isColumnExists("document_voted_log", "point"))
 				$oDB->addColumn('document_voted_log', 'point', 'number', 11, 0, true);
 
 
-			if(!$oDB->isColumnExists("document_categories","color")) $oDB->addColumn('document_categories',"color","char",7);
+			if (!$oDB->isColumnExists("document_categories", "color")) $oDB->addColumn('document_categories', "color", "char", 7);
 
 			// 2009. 01. 29: Add a column(lang_code) if not exist in the document_extra_vars table
-			if(!$oDB->isColumnExists("document_extra_vars","lang_code")) $oDB->addColumn('document_extra_vars',"lang_code","varchar",10);
+			if (!$oDB->isColumnExists("document_extra_vars", "lang_code")) $oDB->addColumn('document_extra_vars', "lang_code", "varchar", 10);
 
 			// 2009. 01. 29 Added a trigger for additional setup
-			if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'document', 'view', 'triggerDispDocumentAdditionSetup', 'before'))
+			if (!$oModuleModel->getTrigger('module.dispAdditionSetup', 'document', 'view', 'triggerDispDocumentAdditionSetup', 'before'))
 				$oModuleController->insertTrigger('module.dispAdditionSetup', 'document', 'view', 'triggerDispDocumentAdditionSetup', 'before');
 			// 2009. 03. 09 Add a column(lang_code) to the documnets table
-			if(!$oDB->isColumnExists("documents","lang_code"))
+			if (!$oDB->isColumnExists("documents", "lang_code"))
 			{
 				$db_info = Context::getDBInfo();
-				$oDB->addColumn('documents',"lang_code","varchar",10, $db_info->lang_code);
+				$oDB->addColumn('documents', "lang_code", "varchar", 10, $db_info->lang_code);
 				$obj->lang_code = $db_info->lang_type;
 				executeQuery('document.updateDocumentsLangCode', $obj);
 			}
 			// 2009. 03. 11 Check the index in the document_extra_vars table
-			if(!$oDB->isIndexExists("document_extra_vars", "unique_extra_vars"))
+			if (!$oDB->isIndexExists("document_extra_vars", "unique_extra_vars"))
 			{
-				$oDB->addIndex("document_extra_vars", "unique_extra_vars", array("module_srl","document_srl","var_idx","lang_code"), true);
+				$oDB->addIndex("document_extra_vars", "unique_extra_vars", array("module_srl", "document_srl", "var_idx", "lang_code"), true);
 			}
 
-			if($oDB->isIndexExists("document_extra_vars", "unique_module_vars"))
+			if ($oDB->isIndexExists("document_extra_vars", "unique_module_vars"))
 			{
 				$oDB->dropIndex("document_extra_vars", "unique_module_vars", true);
 			}
 
 			// 2009. 03. 19: Add a column(eid)
 			// 2009. 04. 12: Fixed the issue(#17922959) that changes another column values when adding eid column
-			if(!$oDB->isColumnExists("document_extra_keys","eid"))
+			if (!$oDB->isColumnExists("document_extra_keys", "eid"))
 			{
-				$oDB->addColumn("document_extra_keys","eid","varchar",40);
+				$oDB->addColumn("document_extra_keys", "eid", "varchar", 40);
 
 				$output = executeQuery('document.getGroupsExtraKeys', $obj);
-				if($output->toBool() && $output->data && count($output->data)) {
-					foreach($output->data as $extra_keys) {
+				if ($output->toBool() && $output->data && count($output->data)) {
+					foreach ($output->data as $extra_keys) {
 						$args->module_srl = $extra_keys->module_srl;
 						$args->var_idx = $extra_keys->idx;
 						$args->new_eid = "extra_vars".$extra_keys->idx;
@@ -252,14 +252,14 @@ 
 				}
 			}
 
-			if(!$oDB->isColumnExists("document_extra_vars","eid"))
+			if (!$oDB->isColumnExists("document_extra_vars", "eid"))
 			{
-				$oDB->addColumn("document_extra_vars","eid","varchar",40);
+				$oDB->addColumn("document_extra_vars", "eid", "varchar", 40);
 				$obj->var_idx = '-1,-2';
 				$output = executeQuery('document.getGroupsExtraVars', $obj);
-				if($output->toBool() && $output->data && count($output->data))
+				if ($output->toBool() && $output->data && count($output->data))
 				{
-					foreach($output->data as $extra_vars)
+					foreach ($output->data as $extra_vars)
 					{
 						$args->module_srl = $extra_vars->module_srl;
 						$args->var_idx = $extra_vars->idx;
@@ -270,16 +270,16 @@ 
 			}
 
 			// 2011. 03. 30 Cubrid index Check the index in the document_extra_vars table
-			if(!$oDB->isIndexExists("document_extra_vars", "idx_document_list_order"))
+			if (!$oDB->isIndexExists("document_extra_vars", "idx_document_list_order"))
 			{
-				$oDB->addIndex("document_extra_vars", "idx_document_list_order", array("document_srl","module_srl","var_idx"), false);
+				$oDB->addIndex("document_extra_vars", "idx_document_list_order", array("document_srl", "module_srl", "var_idx"), false);
 			}
 
 			//2011. 04. 07 adding description column to document categories
-			if(!$oDB->isColumnExists("document_categories","description")) $oDB->addColumn('document_categories',"description","varchar",200,0);
+			if (!$oDB->isColumnExists("document_categories", "description")) $oDB->addColumn('document_categories', "description", "varchar", 200, 0);
 
 			//2011. 05. 23 adding status column to document
-			if(!$oDB->isColumnExists('documents', 'status'))
+			if (!$oDB->isColumnExists('documents', 'status'))
 			{
 				$oDB->addColumn('documents', 'status', 'varchar', 20, 'PUBLIC');
 				$args->is_secret = 'Y';
@@ -287,11 +287,11 @@ 
 			}
 
 			// 2011. 09. 08 drop column document is_secret
-			if($oDB->isColumnExists('documents', 'status') && $oDB->isColumnExists('documents', 'is_secret'))
+			if ($oDB->isColumnExists('documents', 'status') && $oDB->isColumnExists('documents', 'is_secret'))
 				$oDB->dropColumn('documents', 'is_secret');
 
 			//2011. 06. 07 merge column, allow_comment and lock_comment
-			if($oDB->isColumnExists('documents', 'allow_comment') || $oDB->isColumnExists('documents', 'lock_comment'))
+			if ($oDB->isColumnExists('documents', 'allow_comment') || $oDB->isColumnExists('documents', 'lock_comment'))
 			{
 				$oDB->addColumn('documents', 'comment_status', 'varchar', 20, 'ALLOW');
 
@@ -313,23 +313,23 @@ 
 				$output = executeQuery('document.updateDocumentCommentStatus', $args);
 			}
 
-			if($oDB->isColumnExists('documents', 'allow_comment') && $oDB->isColumnExists('documents', 'comment_status'))
+			if ($oDB->isColumnExists('documents', 'allow_comment') && $oDB->isColumnExists('documents', 'comment_status'))
 				$oDB->dropColumn('documents', 'allow_comment');
 
-			if($oDB->isColumnExists('documents', 'lock_comment') && $oDB->isColumnExists('documents', 'comment_status'))
+			if ($oDB->isColumnExists('documents', 'lock_comment') && $oDB->isColumnExists('documents', 'comment_status'))
 				$oDB->dropColumn('documents', 'lock_comment');
 
-			if(!$oDB->isIndexExists("documents", "idx_module_status"))
-				$oDB->addIndex("documents", "idx_module_status", array("module_srl","status"));
+			if (!$oDB->isIndexExists("documents", "idx_module_status"))
+				$oDB->addIndex("documents", "idx_module_status", array("module_srl", "status"));
 
 			// 2012. 02. 27 Add a trigger to copy extra keys when the module is copied 
-			if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModuleExtraKeys', 'after'))
+			if (!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModuleExtraKeys', 'after'))
 			{
 				$oModuleController->insertTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModuleExtraKeys', 'after');
 			}
 
 			// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-			if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModule', 'after'))
+			if (!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModule', 'after'))
 			{
 				$oModuleController->insertTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModule', 'after');
 			}
@@ -337,7 +337,7 @@ 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
 
-		return new Object(0,'success_updated');
+		return new Object(0, 'success_updated');
 	}
 
 	/**
@@ -346,7 +346,7 @@ 
 	 */
 	function recompileCache()
 	{
-		if(!is_dir('./files/cache/tmp'))
+		if (!is_dir('./files/cache/tmp'))
 		{
 			FileHandler::makeDir('./files/cache/tmp');
 		}
@@ -376,7 +376,7 @@ 
 	 */
 	function getConfigStatus($key)
 	{
-		if(array_key_exists(strtolower($key), $this->statusList)) return $this->statusList[$key];
+		if (array_key_exists(strtolower($key), $this->statusList)) return $this->statusList[$key];
 		else $this->getDefaultStatus();
 	}
 }

Braces +140 added lines, -54 removed lines

@@ -70,68 +70,126 @@ 
 		if($oModuleModel->needUpdate($version_update_id))
 		{
 			// 2007. 7. 25: Add a column(notify_message) for notification
-			if(!$oDB->isColumnExists("documents","notify_message")) return true;
+			if(!$oDB->isColumnExists("documents","notify_message")) {
+				return true;
+			}
 
 			// 2007. 8. 23: create a clustered index in the document table
-			if(!$oDB->isIndexExists("documents","idx_module_list_order")) return true;
-			if(!$oDB->isIndexExists("documents","idx_module_update_order")) return true;
-			if(!$oDB->isIndexExists("documents","idx_module_readed_count")) return true;
-			if(!$oDB->isIndexExists("documents","idx_module_voted_count")) return true;
+			if(!$oDB->isIndexExists("documents","idx_module_list_order")) {
+				return true;
+			}
+			if(!$oDB->isIndexExists("documents","idx_module_update_order")) {
+				return true;
+			}
+			if(!$oDB->isIndexExists("documents","idx_module_readed_count")) {
+				return true;
+			}
+			if(!$oDB->isIndexExists("documents","idx_module_voted_count")) {
+				return true;
+			}
 			// 2007. 10. 17 Add a trigger to delete all posts together when the module is deleted
-			if(!$oModuleModel->getTrigger('module.deleteModule', 'document', 'controller', 'triggerDeleteModuleDocuments', 'after')) return true;
+			if(!$oModuleModel->getTrigger('module.deleteModule', 'document', 'controller', 'triggerDeleteModuleDocuments', 'after')) {
+				return true;
+			}
 			// 2007. 10. 25 add parent_srl, expand to the document category
-			if(!$oDB->isColumnExists("document_categories","parent_srl")) return true;
-			if(!$oDB->isColumnExists("document_categories","expand")) return true;
-			if(!$oDB->isColumnExists("document_categories","group_srls")) return true;
+			if(!$oDB->isColumnExists("document_categories","parent_srl")) {
+				return true;
+			}
+			if(!$oDB->isColumnExists("document_categories","expand")) {
+				return true;
+			}
+			if(!$oDB->isColumnExists("document_categories","group_srls")) {
+				return true;
+			}
 			// 2007. 11. 20 create a composite index on the columns(module_srl + is_notice)
-			if(!$oDB->isIndexExists("documents","idx_module_notice")) return true;
+			if(!$oDB->isIndexExists("documents","idx_module_notice")) {
+				return true;
+			}
 			// 2008. 02. 18 create a composite index on the columns(module_srl + document_srl) (checked by Manian))
-			if(!$oDB->isIndexExists("documents","idx_module_document_srl")) return true;
+			if(!$oDB->isIndexExists("documents","idx_module_document_srl")) {
+				return true;
+			}
 
 			// 2007. 12. 03: Add if the colume(extra_vars) doesn't exist
-			if(!$oDB->isColumnExists("documents","extra_vars")) return true;
+			if(!$oDB->isColumnExists("documents","extra_vars")) {
+				return true;
+			}
 			// 2008. 04. 23 Add a column(blamed_count)
-			if(!$oDB->isColumnExists("documents", "blamed_count")) return true;
-			if(!$oDB->isIndexExists("documents","idx_module_blamed_count")) return true;
-			if(!$oDB->isColumnExists("document_voted_log", "point")) return true;
+			if(!$oDB->isColumnExists("documents", "blamed_count")) {
+				return true;
+			}
+			if(!$oDB->isIndexExists("documents","idx_module_blamed_count")) {
+				return true;
+			}
+			if(!$oDB->isColumnExists("document_voted_log", "point")) {
+				return true;
+			}
 			// 2008-12-15 Add a column(color)
-			if(!$oDB->isColumnExists("document_categories", "color")) return true;
+			if(!$oDB->isColumnExists("document_categories", "color")) {
+				return true;
+			}
 
 			/**
 			 * 2009. 01. 29: Add a column(lang_code) if not exist in the document_extra_vars table
 			 */
-			if(!$oDB->isColumnExists("document_extra_vars","lang_code")) return true;
+			if(!$oDB->isColumnExists("document_extra_vars","lang_code")) {
+				return true;
+			}
 
-			if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'document', 'view', 'triggerDispDocumentAdditionSetup', 'before')) return true;
+			if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'document', 'view', 'triggerDispDocumentAdditionSetup', 'before')) {
+				return true;
+			}
 			// 2009. 03. 09 Add a column(lang_code) to the documnets table
-			if(!$oDB->isColumnExists("documents","lang_code")) return true;
+			if(!$oDB->isColumnExists("documents","lang_code")) {
+				return true;
+			}
 			// 2009. 03. 11 check the index in the document_extra_vars table
-			if(!$oDB->isIndexExists("document_extra_vars", "unique_extra_vars")) return true;
+			if(!$oDB->isIndexExists("document_extra_vars", "unique_extra_vars")) {
+				return true;
+			}
 
 			// 2009. 03. 19: Add a column(eid) if not exist in the table
-			if(!$oDB->isColumnExists("document_extra_keys","eid")) return true;
-			if(!$oDB->isColumnExists("document_extra_vars","eid")) return true;
+			if(!$oDB->isColumnExists("document_extra_keys","eid")) {
+				return true;
+			}
+			if(!$oDB->isColumnExists("document_extra_vars","eid")) {
+				return true;
+			}
 
 			// 2011. 03. 30 Cubrid index Check the index in the document_extra_vars table
-			if(!$oDB->isIndexExists("document_extra_vars", "idx_document_list_order")) return true;
+			if(!$oDB->isIndexExists("document_extra_vars", "idx_document_list_order")) {
+				return true;
+			}
 
 			//2011. 04. 07 adding description column to document categories
-			if(!$oDB->isColumnExists("document_categories","description")) return true;
+			if(!$oDB->isColumnExists("document_categories","description")) {
+				return true;
+			}
 
 			//2011. 05. 23 adding status column to document
-			if(!$oDB->isColumnExists('documents', 'status')) return true;
+			if(!$oDB->isColumnExists('documents', 'status')) {
+				return true;
+			}
 
 			//2011. 06. 07 check comment status update
-			if($oDB->isColumnExists('documents', 'allow_comment') || $oDB->isColumnExists('documents', 'lock_comment')) return true;
+			if($oDB->isColumnExists('documents', 'allow_comment') || $oDB->isColumnExists('documents', 'lock_comment')) {
+				return true;
+			}
 
 			// 2011. 10. 25 status index check
-			if(!$oDB->isIndexExists("documents", "idx_module_status")) return true;
+			if(!$oDB->isIndexExists("documents", "idx_module_status")) {
+				return true;
+			}
 
 			// 2012. 02. 27 Add a trigger to copy extra keys when the module is copied 
-			if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModuleExtraKeys', 'after')) return true;
+			if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModuleExtraKeys', 'after')) {
+				return true;
+			}
 
 			// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-			if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModule', 'after')) return true;
+			if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModule', 'after')) {
+				return true;
+			}
 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
@@ -178,20 +236,33 @@ 
 				$oDB->addIndex("documents","idx_module_voted_count", array("module_srl","voted_count"));
 			}
 			// 2007. 10. 17 Add a trigger to delete all posts together when the module is deleted
-			if(!$oModuleModel->getTrigger('module.deleteModule', 'document', 'controller', 'triggerDeleteModuleDocuments', 'after'))
-				$oModuleController->insertTrigger('module.deleteModule', 'document', 'controller', 'triggerDeleteModuleDocuments', 'after');
+			if(!$oModuleModel->getTrigger('module.deleteModule', 'document', 'controller', 'triggerDeleteModuleDocuments', 'after')) {
+							$oModuleController->insertTrigger('module.deleteModule', 'document', 'controller', 'triggerDeleteModuleDocuments', 'after');
+			}
 			// 2007. 10. 25 add columns(parent_srl, expand) 
-			if(!$oDB->isColumnExists("document_categories","parent_srl")) $oDB->addColumn('document_categories',"parent_srl","number",12,0);
-			if(!$oDB->isColumnExists("document_categories","expand")) $oDB->addColumn('document_categories',"expand","char",1,"N");
-			if(!$oDB->isColumnExists("document_categories","group_srls")) $oDB->addColumn('document_categories',"group_srls","text");
+			if(!$oDB->isColumnExists("document_categories","parent_srl")) {
+				$oDB->addColumn('document_categories',"parent_srl","number",12,0);
+			}
+			if(!$oDB->isColumnExists("document_categories","expand")) {
+				$oDB->addColumn('document_categories',"expand","char",1,"N");
+			}
+			if(!$oDB->isColumnExists("document_categories","group_srls")) {
+				$oDB->addColumn('document_categories',"group_srls","text");
+			}
 			// 2007. 11. 20 create a composite index on the columns(module_srl + is_notice)
-			if(!$oDB->isIndexExists("documents","idx_module_notice")) $oDB->addIndex("documents","idx_module_notice", array("module_srl","is_notice"));
+			if(!$oDB->isIndexExists("documents","idx_module_notice")) {
+				$oDB->addIndex("documents","idx_module_notice", array("module_srl","is_notice"));
+			}
 
 			// 2007. 12. 03: Add if the colume(extra_vars) doesn't exist
-			if(!$oDB->isColumnExists("documents","extra_vars")) $oDB->addColumn('documents','extra_vars','text');
+			if(!$oDB->isColumnExists("documents","extra_vars")) {
+				$oDB->addColumn('documents','extra_vars','text');
+			}
 
 			// 2008. 02. 18 create a composite index on the columns(module_srl + document_srl) (checked by Manian))
-			if(!$oDB->isIndexExists("documents","idx_module_document_srl")) $oDB->addIndex("documents","idx_module_document_srl", array("module_srl","document_srl"));
+			if(!$oDB->isIndexExists("documents","idx_module_document_srl")) {
+				$oDB->addIndex("documents","idx_module_document_srl", array("module_srl","document_srl"));
+			}
 			// 2008. 04. 23 Add a column(blamed count)
 			if(!$oDB->isColumnExists("documents", "blamed_count"))
 			{
@@ -204,18 +275,24 @@ 
 				$oDB->addIndex('documents', 'idx_module_blamed_count', array('module_srl', 'blamed_count'));
 			}
 
-			if(!$oDB->isColumnExists("document_voted_log", "point"))
-				$oDB->addColumn('document_voted_log', 'point', 'number', 11, 0, true);
+			if(!$oDB->isColumnExists("document_voted_log", "point")) {
+							$oDB->addColumn('document_voted_log', 'point', 'number', 11, 0, true);
+			}
 
 
-			if(!$oDB->isColumnExists("document_categories","color")) $oDB->addColumn('document_categories',"color","char",7);
+			if(!$oDB->isColumnExists("document_categories","color")) {
+				$oDB->addColumn('document_categories',"color","char",7);
+			}
 
 			// 2009. 01. 29: Add a column(lang_code) if not exist in the document_extra_vars table
-			if(!$oDB->isColumnExists("document_extra_vars","lang_code")) $oDB->addColumn('document_extra_vars',"lang_code","varchar",10);
+			if(!$oDB->isColumnExists("document_extra_vars","lang_code")) {
+				$oDB->addColumn('document_extra_vars',"lang_code","varchar",10);
+			}
 
 			// 2009. 01. 29 Added a trigger for additional setup
-			if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'document', 'view', 'triggerDispDocumentAdditionSetup', 'before'))
-				$oModuleController->insertTrigger('module.dispAdditionSetup', 'document', 'view', 'triggerDispDocumentAdditionSetup', 'before');
+			if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'document', 'view', 'triggerDispDocumentAdditionSetup', 'before')) {
+							$oModuleController->insertTrigger('module.dispAdditionSetup', 'document', 'view', 'triggerDispDocumentAdditionSetup', 'before');
+			}
 			// 2009. 03. 09 Add a column(lang_code) to the documnets table
 			if(!$oDB->isColumnExists("documents","lang_code"))
 			{
@@ -276,7 +353,9 @@ 
 			}
 
 			//2011. 04. 07 adding description column to document categories
-			if(!$oDB->isColumnExists("document_categories","description")) $oDB->addColumn('document_categories',"description","varchar",200,0);
+			if(!$oDB->isColumnExists("document_categories","description")) {
+				$oDB->addColumn('document_categories',"description","varchar",200,0);
+			}
 
 			//2011. 05. 23 adding status column to document
 			if(!$oDB->isColumnExists('documents', 'status'))
@@ -287,8 +366,9 @@ 
 			}
 
 			// 2011. 09. 08 drop column document is_secret
-			if($oDB->isColumnExists('documents', 'status') && $oDB->isColumnExists('documents', 'is_secret'))
-				$oDB->dropColumn('documents', 'is_secret');
+			if($oDB->isColumnExists('documents', 'status') && $oDB->isColumnExists('documents', 'is_secret')) {
+							$oDB->dropColumn('documents', 'is_secret');
+			}
 
 			//2011. 06. 07 merge column, allow_comment and lock_comment
 			if($oDB->isColumnExists('documents', 'allow_comment') || $oDB->isColumnExists('documents', 'lock_comment'))
@@ -313,14 +393,17 @@ 
 				$output = executeQuery('document.updateDocumentCommentStatus', $args);
 			}
 
-			if($oDB->isColumnExists('documents', 'allow_comment') && $oDB->isColumnExists('documents', 'comment_status'))
-				$oDB->dropColumn('documents', 'allow_comment');
+			if($oDB->isColumnExists('documents', 'allow_comment') && $oDB->isColumnExists('documents', 'comment_status')) {
+							$oDB->dropColumn('documents', 'allow_comment');
+			}
 
-			if($oDB->isColumnExists('documents', 'lock_comment') && $oDB->isColumnExists('documents', 'comment_status'))
-				$oDB->dropColumn('documents', 'lock_comment');
+			if($oDB->isColumnExists('documents', 'lock_comment') && $oDB->isColumnExists('documents', 'comment_status')) {
+							$oDB->dropColumn('documents', 'lock_comment');
+			}
 
-			if(!$oDB->isIndexExists("documents", "idx_module_status"))
-				$oDB->addIndex("documents", "idx_module_status", array("module_srl","status"));
+			if(!$oDB->isIndexExists("documents", "idx_module_status")) {
+							$oDB->addIndex("documents", "idx_module_status", array("module_srl","status"));
+			}
 
 			// 2012. 02. 27 Add a trigger to copy extra keys when the module is copied 
 			if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'document', 'controller', 'triggerCopyModuleExtraKeys', 'after'))
@@ -376,8 +459,11 @@ 
 	 */
 	function getConfigStatus($key)
 	{
-		if(array_key_exists(strtolower($key), $this->statusList)) return $this->statusList[$key];
-		else $this->getDefaultStatus();
+		if(array_key_exists(strtolower($key), $this->statusList)) {
+			return $this->statusList[$key];
+		} else {
+			$this->getDefaultStatus();
+		}
 	}
 }
 /* End of file document.class.php */

modules/board/board.class.php

Spacing +14 added lines, -14 removed lines

@@ -9,7 +9,7 @@ 
 
 class board extends ModuleObject
 {
-	var $search_option = array('title_content','title','content','comment','user_name','nick_name','user_id','tag'); ///< 검색 옵션
+	var $search_option = array('title_content', 'title', 'content', 'comment', 'user_name', 'nick_name', 'user_id', 'tag'); ///< 검색 옵션
 
 	var $order_target = array('list_order', 'update_order', 'regdate', 'voted_count', 'blamed_count', 'readed_count', 'comment_count', 'title', 'nick_name', 'user_name', 'user_id'); // 정렬 옵션
 
@@ -25,14 +25,14 @@ 
 	 */
 	function board()
 	{
-		if(!Context::isInstalled()) return;
+		if (!Context::isInstalled()) return;
 
-		if(!Context::isExistsSSLAction('dispBoardWrite') && Context::getSslStatus() == 'optional')
+		if (!Context::isExistsSSLAction('dispBoardWrite') && Context::getSslStatus() == 'optional')
 		{
 			$ssl_actions = array('dispBoardWrite', 'dispBoardWriteComment', 'dispBoardReplyComment', 'dispBoardModifyComment', 'dispBoardDelete', 'dispBoardDeleteComment', 'procBoardInsertDocument', 'procBoardDeleteDocument', 'procBoardInsertComment', 'procBoardDeleteComment', 'procBoardVerificationPassword');
 			Context::addSSLActions($ssl_actions);
 		}
-		if(!Context::isExistsSSLAction('dispTempSavedList') && Context::getSslStatus() == 'optional')
+		if (!Context::isExistsSSLAction('dispTempSavedList') && Context::getSslStatus() == 'optional')
 		{
 			Context::addSSLAction('dispTempSavedList');
 		}
@@ -54,7 +54,7 @@ 
 		$args = new stdClass;
 		$args->site_srl = 0;
 		$output = executeQuery('module.getSite', $args);
-		if(!$output->data->index_module_srl)
+		if (!$output->data->index_module_srl)
 		{
 			$args->mid = 'board';
 			$args->module = 'board';
@@ -63,7 +63,7 @@ 
 			$args->site_srl = 0;
 			$output = $oModuleController->insertModule($args);
 
-			if($output->toBool())
+			if ($output->toBool())
 			{
 				$module_srl = $output->get('module_srl');
 
@@ -87,13 +87,13 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
 			// 2007. 10. 17 get the member menu trigger
-			if(!$oModuleModel->getTrigger('member.getMemberMenu', 'board', 'controller', 'triggerMemberMenu', 'after')) return true;
+			if (!$oModuleModel->getTrigger('member.getMemberMenu', 'board', 'controller', 'triggerMemberMenu', 'after')) return true;
 
 			// 2011. 09. 20 when add new menu in sitemap, custom menu add
-			if(!$oModuleModel->getTrigger('menu.getModuleListInSitemap', 'board', 'model', 'triggerModuleListInSitemap', 'after')) return true;
+			if (!$oModuleModel->getTrigger('menu.getModuleListInSitemap', 'board', 'model', 'triggerModuleListInSitemap', 'after')) return true;
 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
@@ -109,16 +109,16 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
 			// 2007. 10. 17  check the member menu trigger, if it is not existed then insert
-			if(!$oModuleModel->getTrigger('member.getMemberMenu', 'board', 'controller', 'triggerMemberMenu', 'after'))
+			if (!$oModuleModel->getTrigger('member.getMemberMenu', 'board', 'controller', 'triggerMemberMenu', 'after'))
 			{
 				$oModuleController->insertTrigger('member.getMemberMenu', 'board', 'controller', 'triggerMemberMenu', 'after');
 			}
 
 			// 2011. 09. 20 when add new menu in sitemap, custom menu add
-			if(!$oModuleModel->getTrigger('menu.getModuleListInSitemap', 'board', 'model', 'triggerModuleListInSitemap', 'after'))
+			if (!$oModuleModel->getTrigger('menu.getModuleListInSitemap', 'board', 'model', 'triggerModuleListInSitemap', 'after'))
 			{
 				$oModuleController->insertTrigger('menu.getModuleListInSitemap', 'board', 'model', 'triggerModuleListInSitemap', 'after');
 			}
@@ -131,12 +131,12 @@ 
 	function moduleUninstall()
 	{
 		$output = executeQueryArray("board.getAllBoard");
-		if(!$output->data) return new Object();
+		if (!$output->data) return new Object();
 		@set_time_limit(0);
 
 		$oModuleController = getController('module');
 
-		foreach($output->data as $board)
+		foreach ($output->data as $board)
 		{
 			$oModuleController->deleteModule($board->module_srl);
 		}

Braces +12 added lines, -4 removed lines

@@ -25,7 +25,9 @@ 
 	 */
 	function board()
 	{
-		if(!Context::isInstalled()) return;
+		if(!Context::isInstalled()) {
+			return;
+		}
 
 		if(!Context::isExistsSSLAction('dispBoardWrite') && Context::getSslStatus() == 'optional')
 		{
@@ -90,10 +92,14 @@ 
 		if($oModuleModel->needUpdate($version_update_id))
 		{
 			// 2007. 10. 17 get the member menu trigger
-			if(!$oModuleModel->getTrigger('member.getMemberMenu', 'board', 'controller', 'triggerMemberMenu', 'after')) return true;
+			if(!$oModuleModel->getTrigger('member.getMemberMenu', 'board', 'controller', 'triggerMemberMenu', 'after')) {
+				return true;
+			}
 
 			// 2011. 09. 20 when add new menu in sitemap, custom menu add
-			if(!$oModuleModel->getTrigger('menu.getModuleListInSitemap', 'board', 'model', 'triggerModuleListInSitemap', 'after')) return true;
+			if(!$oModuleModel->getTrigger('menu.getModuleListInSitemap', 'board', 'model', 'triggerModuleListInSitemap', 'after')) {
+				return true;
+			}
 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
@@ -131,7 +137,9 @@ 
 	function moduleUninstall()
 	{
 		$output = executeQueryArray("board.getAllBoard");
-		if(!$output->data) return new Object();
+		if(!$output->data) {
+			return new Object();
+		}
 		@set_time_limit(0);
 
 		$oModuleController = getController('module');

modules/counter/counter.class.php

Spacing +6 added lines, -6 removed lines

@@ -38,14 +38,14 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
-			if(!$oDB->isColumnExists('counter_log', 'site_srl'))
+			if (!$oDB->isColumnExists('counter_log', 'site_srl'))
 			{
 				return TRUE;
 			}
 
-			if(!$oDB->isIndexExists('counter_log', 'idx_site_counter_log'))
+			if (!$oDB->isIndexExists('counter_log', 'idx_site_counter_log'))
 			{
 				return TRUE;
 			}
@@ -69,14 +69,14 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
-			if(!$oDB->isColumnExists('counter_log', 'site_srl'))
+			if (!$oDB->isColumnExists('counter_log', 'site_srl'))
 			{
 				$oDB->addColumn('counter_log', 'site_srl', 'number', 11, 0, TRUE);
 			}
 
-			if(!$oDB->isIndexExists('counter_log', 'idx_site_counter_log'))
+			if (!$oDB->isIndexExists('counter_log', 'idx_site_counter_log'))
 			{
 				$oDB->addIndex('counter_log', 'idx_site_counter_log', array('site_srl', 'ipaddress'), FALSE);
 			}

modules/page/page.class.php

Spacing +14 added lines, -14 removed lines

@@ -26,13 +26,13 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
 			$output = executeQuery('page.pageTypeOpageCheck');
-			if($output->toBool() && $output->data) return true;
+			if ($output->toBool() && $output->data) return true;
 
 			$output = executeQuery('page.pageTypeNullCheck');
-			if($output->toBool() && $output->data) return true;
+			if ($output->toBool() && $output->data) return true;
 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
@@ -48,44 +48,44 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
 			$args = new stdClass;
 			// opage module instance update
 			$output = executeQueryArray('page.pageTypeOpageCheck');
-			if($output->toBool() && count($output->data) > 0)
+			if ($output->toBool() && count($output->data) > 0)
 			{
-				foreach($output->data as $val)
+				foreach ($output->data as $val)
 				{
 					$args->module_srl = $val->module_srl;
 					$args->name = 'page_type';
-					$args->value= 'OUTSIDE';
+					$args->value = 'OUTSIDE';
 					$in_out = executeQuery('page.insertPageType', $args);
 				}
 				$output = executeQuery('page.updateAllOpage');
-				if(!$output->toBool()) return $output;
+				if (!$output->toBool()) return $output;
 			}
 
 			// old page module instance update
 			$output = executeQueryArray('page.pageTypeNullCheck');
 			$skin_update_srls = array();
-			if($output->toBool() && $output->data)
+			if ($output->toBool() && $output->data)
 			{
-				foreach($output->data as $val)
+				foreach ($output->data as $val)
 				{
 					$args->module_srl = $val->module_srl;
 					$args->name = 'page_type';
-					$args->value= 'WIDGET';
+					$args->value = 'WIDGET';
 					$in_out = executeQuery('page.insertPageType', $args);
 
 					$skin_update_srls[] = $val->module_srl;
 				}
 			}
 
-			if(count($skin_update_srls)>0)
+			if (count($skin_update_srls) > 0)
 			{
 				$skin_args = new stdClass;
-				$skin_args->module_srls = implode(',',$skin_update_srls);
+				$skin_args->module_srls = implode(',', $skin_update_srls);
 				$skin_args->is_skin_fix = "Y";
 				$ouput = executeQuery('page.updateSkinFix', $skin_args);
 			}
@@ -93,7 +93,7 @@ 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
 
-		return new Object(0,'success_updated');
+		return new Object(0, 'success_updated');
 	}
 
 	/**

Braces +9 added lines, -3 removed lines

@@ -29,10 +29,14 @@ 
 		if($oModuleModel->needUpdate($version_update_id))
 		{
 			$output = executeQuery('page.pageTypeOpageCheck');
-			if($output->toBool() && $output->data) return true;
+			if($output->toBool() && $output->data) {
+				return true;
+			}
 
 			$output = executeQuery('page.pageTypeNullCheck');
-			if($output->toBool() && $output->data) return true;
+			if($output->toBool() && $output->data) {
+				return true;
+			}
 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
@@ -63,7 +67,9 @@ 
 					$in_out = executeQuery('page.insertPageType', $args);
 				}
 				$output = executeQuery('page.updateAllOpage');
-				if(!$output->toBool()) return $output;
+				if(!$output->toBool()) {
+					return $output;
+				}
 			}
 
 			// old page module instance update

modules/module/module.class.php

Spacing +111 added lines, -111 removed lines

@@ -16,8 +16,8 @@ 
 		$oModuleController = getController('module');
 
 		$oDB = &DB::getInstance();
-		$oDB->addIndex("modules","idx_site_mid", array("site_srl","mid"), true);
-		$oDB->addIndex('sites','unique_domain',array('domain'),true);
+		$oDB->addIndex("modules", "idx_site_mid", array("site_srl", "mid"), true);
+		$oDB->addIndex('sites', 'unique_domain', array('domain'), true);
 		// Create a directory to use in the module module
 		FileHandler::makeDir('./files/cache/module_info');
 		FileHandler::makeDir('./files/cache/triggers');
@@ -27,21 +27,21 @@ 
 		$args = new stdClass;
 		$args->site_srl = 0;
 		$output = $oDB->executeQuery('module.getSite', $args);
-		if(!$output->data || !$output->data->index_module_srl)
+		if (!$output->data || !$output->data->index_module_srl)
 		{
 			$db_info = Context::getDBInfo();
 			$domain = Context::getDefaultUrl();
 			$url_info = parse_url($domain);
-			$domain = $url_info['host'].( (!empty($url_info['port'])&&$url_info['port']!=80)?':'.$url_info['port']:'').$url_info['path'];
+			$domain = $url_info['host'].((!empty($url_info['port']) && $url_info['port'] != 80) ? ':'.$url_info['port'] : '').$url_info['path'];
 
 			$site_args = new stdClass;
 			$site_args->site_srl = 0;
-			$site_args->index_module_srl  = 0;
+			$site_args->index_module_srl = 0;
 			$site_args->domain = $domain;
 			$site_args->default_language = $db_info->lang_type;
 
 			$output = executeQuery('module.insertSite', $site_args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 		}
 
 		return new Object();
@@ -56,66 +56,66 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
 			// 2008. 10. 27 Add multi-index in the table, the module_part_config
-			if(!$oDB->isIndexExists("module_part_config","idx_module_part_config")) return true;
+			if (!$oDB->isIndexExists("module_part_config", "idx_module_part_config")) return true;
 			// 2008. 11. 13 Delete unique constraint on mid in modules. Add site_srl and then create unique index on site_srl and mid
-			if(!$oDB->isIndexExists('modules',"idx_site_mid")) return true;
+			if (!$oDB->isIndexExists('modules', "idx_site_mid")) return true;
 			// Move permissions/skin information of all modules to the table, grants.
-			if($oDB->isColumnExists('modules', 'grants')) return true;
+			if ($oDB->isColumnExists('modules', 'grants')) return true;
 			// Move permissions/skin information of all modules to the table, grants.
-			if(!$oDB->isColumnExists('sites', 'default_language')) return true;
+			if (!$oDB->isColumnExists('sites', 'default_language')) return true;
 			// Delete extra_vars* column
-			for($i=1;$i<=20;$i++)
+			for ($i = 1; $i <= 20; $i++)
 			{
-				if($oDB->isColumnExists("documents","extra_vars".$i)) return true;
+				if ($oDB->isColumnExists("documents", "extra_vars".$i)) return true;
 			}
 			// Insert site information to the table, sites
 			$args = new stdClass();
 			$args->site_srl = 0;
 			$output = $oDB->executeQuery('module.getSite', $args);
-			if(!$output->data) return true;
+			if (!$output->data) return true;
 
 			// If domain index is defined on the table, sites
-			if($oDB->isIndexExists('sites', 'idx_domain')) return true;
-			if(!$oDB->isIndexExists('sites','unique_domain')) return true;
+			if ($oDB->isIndexExists('sites', 'idx_domain')) return true;
+			if (!$oDB->isIndexExists('sites', 'unique_domain')) return true;
 
-			if(!$oDB->isColumnExists("modules", "use_mobile")) return true;
-			if(!$oDB->isColumnExists("modules", "mlayout_srl")) return true;
-			if(!$oDB->isColumnExists("modules", "mcontent")) return true;
-			if(!$oDB->isColumnExists("modules", "mskin")) return true;
+			if (!$oDB->isColumnExists("modules", "use_mobile")) return true;
+			if (!$oDB->isColumnExists("modules", "mlayout_srl")) return true;
+			if (!$oDB->isColumnExists("modules", "mcontent")) return true;
+			if (!$oDB->isColumnExists("modules", "mskin")) return true;
 
 			// check fix skin
-			if(!$oDB->isColumnExists("modules", "is_skin_fix")) return true;
+			if (!$oDB->isColumnExists("modules", "is_skin_fix")) return true;
 
-			if(!$oDB->isColumnExists("module_config", "site_srl")) return true;
+			if (!$oDB->isColumnExists("module_config", "site_srl")) return true;
 
 			$args->skin = '.';
 			$output = executeQueryArray('module.getModuleSkinDotList', $args);
-			if($output->data && count($output->data) > 0)
+			if ($output->data && count($output->data) > 0)
 			{
-				foreach($output->data as $item)
+				foreach ($output->data as $item)
 				{
 					$skin_path = explode('.', $item->skin);
-					if(count($skin_path) != 2) continue;
-					if(is_dir(sprintf(_XE_PATH_ . 'themes/%s/modules/%s', $skin_path[0], $skin_path[1]))) return true;
+					if (count($skin_path) != 2) continue;
+					if (is_dir(sprintf(_XE_PATH_.'themes/%s/modules/%s', $skin_path[0], $skin_path[1]))) return true;
 				}
 			}
 
 			// XE 1.7
 
 			// check fix mskin
-			if(!$oDB->isColumnExists("modules", "is_mskin_fix")) return true;
+			if (!$oDB->isColumnExists("modules", "is_mskin_fix")) return true;
 
 			$oModuleModel = getModel('module');
 			$moduleConfig = $oModuleModel->getModuleConfig('module');
-			if(!$moduleConfig->isUpdateFixedValue) return true;
+			if (!$moduleConfig->isUpdateFixedValue) return true;
 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
 
-		if(!is_dir('./files/ruleset')) return true;
+		if (!is_dir('./files/ruleset')) return true;
 
 		return false;
 	}
@@ -129,22 +129,22 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		$version_update_id = implode('.', array(__CLASS__, __XE_VERSION__, 'updated'));
-		if($oModuleModel->needUpdate($version_update_id))
+		if ($oModuleModel->needUpdate($version_update_id))
 		{
 			// 2008. 10. 27 module_part_config Add a multi-index to the table and check all information of module_configg
-			if(!$oDB->isIndexExists("module_part_config","idx_module_part_config"))
+			if (!$oDB->isIndexExists("module_part_config", "idx_module_part_config"))
 			{
 				$oModuleModel = getModel('module');
 				$oModuleController = getController('module');
 				$modules = $oModuleModel->getModuleList();
-				foreach($modules as $key => $module_info)
+				foreach ($modules as $key => $module_info)
 				{
 					$module = $module_info->module;
-					if(!in_array($module, array('point','trackback','layout','rss','file','comment','editor'))) continue;
+					if (!in_array($module, array('point', 'trackback', 'layout', 'rss', 'file', 'comment', 'editor'))) continue;
 					$config = $oModuleModel->getModuleConfig($module);
 
 					$module_config = null;
-					switch($module)
+					switch ($module)
 					{
 						case 'point' :
 							$module_config = $config->module_point;
@@ -157,21 +157,21 @@ 
 						case 'editor' :
 							$module_config = $config->module_config;
 							unset($config->module_config);
-							if(is_array($module_config) && count($module_config))
+							if (is_array($module_config) && count($module_config))
 							{
-								foreach($module_config as $key => $val)
+								foreach ($module_config as $key => $val)
 								{
-									if(isset($module_config[$key]->module_srl)) unset($module_config[$key]->module_srl);
+									if (isset($module_config[$key]->module_srl)) unset($module_config[$key]->module_srl);
 								}
 							}
 							break;
 						case 'layout' :
 							$tmp = $config->header_script;
-							if(is_array($tmp) && count($tmp))
+							if (is_array($tmp) && count($tmp))
 							{
-								foreach($tmp as $k => $v)
+								foreach ($tmp as $k => $v)
 								{
-									if(!$v && !trim($v)) continue;
+									if (!$v && !trim($v)) continue;
 									$module_config[$k]->header_script = $v;
 								}
 							}
@@ -181,29 +181,29 @@ 
 
 					$oModuleController->insertModuleConfig($module, $config);
 
-					if(is_array($module_config) && count($module_config))
+					if (is_array($module_config) && count($module_config))
 					{
-						foreach($module_config as $module_srl => $module_part_config)
+						foreach ($module_config as $module_srl => $module_part_config)
 						{
-							$oModuleController->insertModulePartConfig($module,$module_srl,$module_part_config);
+							$oModuleController->insertModulePartConfig($module, $module_srl, $module_part_config);
 						}
 					}
 				}
-				$oDB->addIndex("module_part_config","idx_module_part_config", array("module","module_srl"));
+				$oDB->addIndex("module_part_config", "idx_module_part_config", array("module", "module_srl"));
 			}
 			// 2008. 11. 13 drop index(unique_mid). Add a column and index on site_srl and mid columns
-			if(!$oDB->isIndexExists('modules',"idx_site_mid"))
+			if (!$oDB->isIndexExists('modules', "idx_site_mid"))
 			{
-				$oDB->dropIndex("modules","unique_mid",true);
-				$oDB->addColumn('modules','site_srl','number',11,0,true);
-				$oDB->addIndex("modules","idx_site_mid", array("site_srl","mid"),true);
+				$oDB->dropIndex("modules", "unique_mid", true);
+				$oDB->addColumn('modules', 'site_srl', 'number', 11, 0, true);
+				$oDB->addIndex("modules", "idx_site_mid", array("site_srl", "mid"), true);
 			}
 			// document extra vars
-			if(!$oDB->isTableExists('document_extra_vars')) $oDB->createTableByXmlFile('./modules/document/schemas/document_extra_vars.xml');
+			if (!$oDB->isTableExists('document_extra_vars')) $oDB->createTableByXmlFile('./modules/document/schemas/document_extra_vars.xml');
 
-			if(!$oDB->isTableExists('document_extra_keys')) $oDB->createTableByXmlFile('./modules/document/schemas/document_extra_keys.xml');
+			if (!$oDB->isTableExists('document_extra_keys')) $oDB->createTableByXmlFile('./modules/document/schemas/document_extra_keys.xml');
 			// Move permission, skin info, extection info, admin ID of all modules to the table, grants
-			if($oDB->isColumnExists('modules', 'grants'))
+			if ($oDB->isColumnExists('modules', 'grants'))
 			{
 				$oModuleController = getController('module');
 				$oDocumentController = getController('document');
@@ -211,26 +211,26 @@ 
 				$lang_code = Context::getLangType();
 				// Get module_info of all modules
 				$output = executeQueryArray('module.getModuleInfos');
-				if(count($output->data))
+				if (count($output->data))
 				{
-					foreach($output->data as $module_info)
+					foreach ($output->data as $module_info)
 					{
 						// Separate information about permission granted to the module, extra vars, skin vars, super-admin's authority
 						$module_srl = trim($module_info->module_srl);
 						// grant an authority
 						$grants = unserialize($module_info->grants);
-						if($grants) $oModuleController->insertModuleGrants($module_srl, $grants);
+						if ($grants) $oModuleController->insertModuleGrants($module_srl, $grants);
 						// Insert skin vars
 						$skin_vars = unserialize($module_info->skin_vars);
-						if($skin_vars) $oModuleController->insertModuleSkinVars($module_srl, $skin_vars);
+						if ($skin_vars) $oModuleController->insertModuleSkinVars($module_srl, $skin_vars);
 						// Insert super admin's ID
 						$admin_id = trim($module_info->admin_id);
-						if($admin_id && $admin_id != 'Array')
+						if ($admin_id && $admin_id != 'Array')
 						{
-							$admin_ids = explode(',',$admin_id);
-							if(count($admin_id))
+							$admin_ids = explode(',', $admin_id);
+							if (count($admin_id))
 							{
-								foreach($admin_ids as $admin_id)
+								foreach ($admin_ids as $admin_id)
 								{
 									$oModuleController->insertAdminId($module_srl, $admin_id);
 								}
@@ -239,20 +239,20 @@ 
 						// Save extra configurations for each module(column data which doesn't exist in the defaut modules)
 						$extra_vars = unserialize($module_info->extra_vars);
 						$document_extra_keys = null;
-						if($extra_vars->extra_vars && count($extra_vars->extra_vars))
+						if ($extra_vars->extra_vars && count($extra_vars->extra_vars))
 						{
 							$document_extra_keys = $extra_vars->extra_vars;
 							unset($extra_vars->extra_vars);
 						}
-						if($extra_vars) $oModuleController->insertModuleExtraVars($module_srl, $extra_vars);
+						if ($extra_vars) $oModuleController->insertModuleExtraVars($module_srl, $extra_vars);
 
 						/**
 						 * Move document extra vars(it should have conducted in the documents module however extra vars in modules table should be listed up in this module)
 						 */
 						// Insert extra vars if planet module is
-						if($module_info->module == 'planet')
+						if ($module_info->module == 'planet')
 						{
-							if(!$document_extra_keys || !is_array($document_extra_keys)) $document_extra_keys = array();
+							if (!$document_extra_keys || !is_array($document_extra_keys)) $document_extra_keys = array();
 							$planet_extra_keys->name = 'postscript';
 							$planet_extra_keys->type = 'text';
 							$planet_extra_keys->is_required = 'N';
@@ -262,9 +262,9 @@ 
 							$document_extra_keys[20] = $planet_extra_keys;
 						}
 						// Register keys for document extra vars
-						if(count($document_extra_keys))
+						if (count($document_extra_keys))
 						{
-							foreach($document_extra_keys as $var_idx => $val)
+							foreach ($document_extra_keys as $var_idx => $val)
 							{
 								$oDocumentController->insertDocumentExtraKey($module_srl, $var_idx, $val->name, $val->type, $val->is_required, $val->search, $val->default, $val->desc, 'extra_vars'.$var_idx);
 							}
@@ -272,7 +272,7 @@ 
 							$oDocumentModel = getModel('document');
 							$total_count = $oDocumentModel->getDocumentCount($module_srl);
 
-							if($total_count > 0)
+							if ($total_count > 0)
 							{
 								$per_page = 100;
 								$total_pages = (int) (($total_count - 1) / $per_page) + 1;
@@ -283,19 +283,19 @@ 
 								$doc_args->sort_index = 'list_order';
 								$doc_args->order_type = 'asc';
 
-								for($doc_args->page = 1; $doc_args->page <= $total_pages; $doc_args->page++)
+								for ($doc_args->page = 1; $doc_args->page <= $total_pages; $doc_args->page++)
 								{
 									$output = executeQueryArray('document.getDocumentList', $doc_args);
 
-									if($output->toBool() && $output->data && count($output->data))
+									if ($output->toBool() && $output->data && count($output->data))
 									{
 										foreach ($output->data as $document)
 										{
-											if(!$document) continue;
+											if (!$document) continue;
 											foreach ($document as $key => $var)
 											{
 												if (strpos($key, 'extra_vars') !== 0 || !trim($var) || $var == 'N;') continue;
-												$var_idx = str_replace('extra_vars','',$key);
+												$var_idx = str_replace('extra_vars', '', $key);
 												$oDocumentController->insertDocumentExtraVar($module_srl, $document->document_srl, $var_idx, $var, 'extra_vars'.$var_idx, $lang_code);
 											}
 										}
@@ -311,96 +311,96 @@ 
 						executeQuery('module.updateModule', $module_info);
 
 						$oCacheHandler = CacheHandler::getInstance('object', null, true);
-						if($oCacheHandler->isSupport())
+						if ($oCacheHandler->isSupport())
 						{
 							$oCacheHandler->invalidateGroupKey('site_and_module');
 						}
 					}
 				}
 				// Various column drop
-				$oDB->dropColumn('modules','grants');
-				$oDB->dropColumn('modules','admin_id');
-				$oDB->dropColumn('modules','skin_vars');
-				$oDB->dropColumn('modules','extra_vars');
+				$oDB->dropColumn('modules', 'grants');
+				$oDB->dropColumn('modules', 'admin_id');
+				$oDB->dropColumn('modules', 'skin_vars');
+				$oDB->dropColumn('modules', 'extra_vars');
 			}
 			// Rights of all modules/skins transferring the information into a table Update grants
-			if(!$oDB->isColumnExists('sites', 'default_language'))
+			if (!$oDB->isColumnExists('sites', 'default_language'))
 			{
-				$oDB->addColumn('sites','default_language','varchar',255,0,false);
+				$oDB->addColumn('sites', 'default_language', 'varchar', 255, 0, false);
 			}
 			// extra_vars * Remove Column
-			for($i=1;$i<=20;$i++)
+			for ($i = 1; $i <= 20; $i++)
 			{
-				if(!$oDB->isColumnExists("documents","extra_vars".$i)) continue;
-				$oDB->dropColumn('documents','extra_vars'.$i);
+				if (!$oDB->isColumnExists("documents", "extra_vars".$i)) continue;
+				$oDB->dropColumn('documents', 'extra_vars'.$i);
 			}
 
 			// Enter the main site information sites on the table
 			$args = new stdClass;
 			$args->site_srl = 0;
 			$output = $oDB->executeQuery('module.getSite', $args);
-			if(!$output->data)
+			if (!$output->data)
 			{
 				// Basic mid, language Wanted
 				$mid_output = $oDB->executeQuery('module.getDefaultMidInfo', $args);
 				$db_info = Context::getDBInfo();
 				$domain = Context::getDefaultUrl();
 				$url_info = parse_url($domain);
-				$domain = $url_info['host'].( (!empty($url_info['port'])&&$url_info['port']!=80)?':'.$url_info['port']:'').$url_info['path'];
+				$domain = $url_info['host'].((!empty($url_info['port']) && $url_info['port'] != 80) ? ':'.$url_info['port'] : '').$url_info['path'];
 				$site_args->site_srl = 0;
-				$site_args->index_module_srl  = $mid_output->data->module_srl;
+				$site_args->index_module_srl = $mid_output->data->module_srl;
 				$site_args->domain = $domain;
 				$site_args->default_language = $db_info->lang_type;
 
 				$output = executeQuery('module.insertSite', $site_args);
-				if(!$output->toBool()) return $output;
+				if (!$output->toBool()) return $output;
 			}
 
-			if($oDB->isIndexExists('sites','idx_domain'))
+			if ($oDB->isIndexExists('sites', 'idx_domain'))
 			{
-				$oDB->dropIndex('sites','idx_domain');
+				$oDB->dropIndex('sites', 'idx_domain');
 			}
-			if(!$oDB->isIndexExists('sites','unique_domain'))
+			if (!$oDB->isIndexExists('sites', 'unique_domain'))
 			{
 				$this->updateForUniqueSiteDomain();
-				$oDB->addIndex('sites','unique_domain',array('domain'),true);
+				$oDB->addIndex('sites', 'unique_domain', array('domain'), true);
 			}
 
-			if(!$oDB->isColumnExists("modules", "use_mobile"))
+			if (!$oDB->isColumnExists("modules", "use_mobile"))
 			{
-				$oDB->addColumn('modules','use_mobile','char',1,'N');
+				$oDB->addColumn('modules', 'use_mobile', 'char', 1, 'N');
 			}
-			if(!$oDB->isColumnExists("modules", "mlayout_srl"))
+			if (!$oDB->isColumnExists("modules", "mlayout_srl"))
 			{
-				$oDB->addColumn('modules','mlayout_srl','number',11, 0);
+				$oDB->addColumn('modules', 'mlayout_srl', 'number', 11, 0);
 			}
-			if(!$oDB->isColumnExists("modules", "mcontent"))
+			if (!$oDB->isColumnExists("modules", "mcontent"))
 			{
-				$oDB->addColumn('modules','mcontent','bigtext');
+				$oDB->addColumn('modules', 'mcontent', 'bigtext');
 			}
-			if(!$oDB->isColumnExists("modules", "mskin"))
+			if (!$oDB->isColumnExists("modules", "mskin"))
 			{
-				$oDB->addColumn('modules','mskin','varchar',250);
+				$oDB->addColumn('modules', 'mskin', 'varchar', 250);
 			}
-			if(!$oDB->isColumnExists("modules", "is_skin_fix"))
+			if (!$oDB->isColumnExists("modules", "is_skin_fix"))
 			{
 				$oDB->addColumn('modules', 'is_skin_fix', 'char', 1, 'N');
 				$output = executeQuery('module.updateSkinFixModules');
 			}
-			if(!$oDB->isColumnExists("module_config", "site_srl"))
+			if (!$oDB->isColumnExists("module_config", "site_srl"))
 			{
 				$oDB->addColumn('module_config', 'site_srl', 'number', 11, 0, true);
 			}
 
 			$args->skin = '.';
 			$output = executeQueryArray('module.getModuleSkinDotList', $args);
-			if($output->data && count($output->data) > 0)
+			if ($output->data && count($output->data) > 0)
 			{
-				foreach($output->data as $item)
+				foreach ($output->data as $item)
 				{
 					$skin_path = explode('.', $item->skin);
-					if(count($skin_path) != 2) continue;
-					if(is_dir(sprintf(_XE_PATH_ . 'themes/%s/modules/%s', $skin_path[0], $skin_path[1])))
+					if (count($skin_path) != 2) continue;
+					if (is_dir(sprintf(_XE_PATH_.'themes/%s/modules/%s', $skin_path[0], $skin_path[1])))
 					{
 						unset($args);
 						$args->skin = $item->skin;
@@ -411,7 +411,7 @@ 
 			}
 
 			// XE 1.7
-			if(!$oDB->isColumnExists("modules", "is_mskin_fix"))
+			if (!$oDB->isColumnExists("modules", "is_mskin_fix"))
 			{
 				$oDB->addColumn('modules', 'is_mskin_fix', 'char', 1, 'N');
 				$output = executeQuery('module.updateMobileSkinFixModules');
@@ -419,13 +419,13 @@ 
 
 			$oModuleModel = getModel('module');
 			$moduleConfig = $oModuleModel->getModuleConfig('module');
-			if(!$moduleConfig->isUpdateFixedValue)
+			if (!$moduleConfig->isUpdateFixedValue)
 			{
 				$output = executeQuery('module.updateSkinFixModules');
 				$output = executeQuery('module.updateMobileSkinFixModules');
 
 				$oModuleController = getController('module');
-				if(!$moduleConfig) $moduleConfig = new stdClass;
+				if (!$moduleConfig) $moduleConfig = new stdClass;
 				$moduleConfig->isUpdateFixedValue = TRUE;
 				$output = $oModuleController->updateModuleConfig('module', $moduleConfig);
 			}
@@ -433,7 +433,7 @@ 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
 
-		if(!is_dir('./files/ruleset')) FileHandler::makeDir('./files/ruleset');
+		if (!is_dir('./files/ruleset')) FileHandler::makeDir('./files/ruleset');
 
 		return new Object(0, 'success_updated');
 	}
@@ -441,18 +441,18 @@ 
 	function updateForUniqueSiteDomain()
 	{
 		$output = executeQueryArray("module.getNonuniqueDomains");
-		if(!$output->data) return;
-		foreach($output->data as $data)
+		if (!$output->data) return;
+		foreach ($output->data as $data)
 		{
-			if($data->count == 1) continue;
+			if ($data->count == 1) continue;
 			$domain = $data->domain;
 			$args = new stdClass;
 			$args->domain = $domain;
 			$output2 = executeQueryArray("module.getSiteByDomain", $args);
 			$bFirst = true;
-			foreach($output2->data as $site)
+			foreach ($output2->data as $site)
 			{
-				if($bFirst)
+				if ($bFirst)
 				{
 					$bFirst = false;
 					continue;

Braces +114 added lines, -38 removed lines

@@ -41,7 +41,9 @@ 
 			$site_args->default_language = $db_info->lang_type;
 
 			$output = executeQuery('module.insertSite', $site_args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 		}
 
 		return new Object();
@@ -59,37 +61,65 @@ 
 		if($oModuleModel->needUpdate($version_update_id))
 		{
 			// 2008. 10. 27 Add multi-index in the table, the module_part_config
-			if(!$oDB->isIndexExists("module_part_config","idx_module_part_config")) return true;
+			if(!$oDB->isIndexExists("module_part_config","idx_module_part_config")) {
+				return true;
+			}
 			// 2008. 11. 13 Delete unique constraint on mid in modules. Add site_srl and then create unique index on site_srl and mid
-			if(!$oDB->isIndexExists('modules',"idx_site_mid")) return true;
+			if(!$oDB->isIndexExists('modules',"idx_site_mid")) {
+				return true;
+			}
 			// Move permissions/skin information of all modules to the table, grants.
-			if($oDB->isColumnExists('modules', 'grants')) return true;
+			if($oDB->isColumnExists('modules', 'grants')) {
+				return true;
+			}
 			// Move permissions/skin information of all modules to the table, grants.
-			if(!$oDB->isColumnExists('sites', 'default_language')) return true;
+			if(!$oDB->isColumnExists('sites', 'default_language')) {
+				return true;
+			}
 			// Delete extra_vars* column
 			for($i=1;$i<=20;$i++)
 			{
-				if($oDB->isColumnExists("documents","extra_vars".$i)) return true;
+				if($oDB->isColumnExists("documents","extra_vars".$i)) {
+					return true;
+				}
 			}
 			// Insert site information to the table, sites
 			$args = new stdClass();
 			$args->site_srl = 0;
 			$output = $oDB->executeQuery('module.getSite', $args);
-			if(!$output->data) return true;
+			if(!$output->data) {
+				return true;
+			}
 
 			// If domain index is defined on the table, sites
-			if($oDB->isIndexExists('sites', 'idx_domain')) return true;
-			if(!$oDB->isIndexExists('sites','unique_domain')) return true;
+			if($oDB->isIndexExists('sites', 'idx_domain')) {
+				return true;
+			}
+			if(!$oDB->isIndexExists('sites','unique_domain')) {
+				return true;
+			}
 
-			if(!$oDB->isColumnExists("modules", "use_mobile")) return true;
-			if(!$oDB->isColumnExists("modules", "mlayout_srl")) return true;
-			if(!$oDB->isColumnExists("modules", "mcontent")) return true;
-			if(!$oDB->isColumnExists("modules", "mskin")) return true;
+			if(!$oDB->isColumnExists("modules", "use_mobile")) {
+				return true;
+			}
+			if(!$oDB->isColumnExists("modules", "mlayout_srl")) {
+				return true;
+			}
+			if(!$oDB->isColumnExists("modules", "mcontent")) {
+				return true;
+			}
+			if(!$oDB->isColumnExists("modules", "mskin")) {
+				return true;
+			}
 
 			// check fix skin
-			if(!$oDB->isColumnExists("modules", "is_skin_fix")) return true;
+			if(!$oDB->isColumnExists("modules", "is_skin_fix")) {
+				return true;
+			}
 
-			if(!$oDB->isColumnExists("module_config", "site_srl")) return true;
+			if(!$oDB->isColumnExists("module_config", "site_srl")) {
+				return true;
+			}
 
 			$args->skin = '.';
 			$output = executeQueryArray('module.getModuleSkinDotList', $args);
@@ -98,24 +128,34 @@ 
 				foreach($output->data as $item)
 				{
 					$skin_path = explode('.', $item->skin);
-					if(count($skin_path) != 2) continue;
-					if(is_dir(sprintf(_XE_PATH_ . 'themes/%s/modules/%s', $skin_path[0], $skin_path[1]))) return true;
+					if(count($skin_path) != 2) {
+						continue;
+					}
+					if(is_dir(sprintf(_XE_PATH_ . 'themes/%s/modules/%s', $skin_path[0], $skin_path[1]))) {
+						return true;
+					}
 				}
 			}
 
 			// XE 1.7
 
 			// check fix mskin
-			if(!$oDB->isColumnExists("modules", "is_mskin_fix")) return true;
+			if(!$oDB->isColumnExists("modules", "is_mskin_fix")) {
+				return true;
+			}
 
 			$oModuleModel = getModel('module');
 			$moduleConfig = $oModuleModel->getModuleConfig('module');
-			if(!$moduleConfig->isUpdateFixedValue) return true;
+			if(!$moduleConfig->isUpdateFixedValue) {
+				return true;
+			}
 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
 
-		if(!is_dir('./files/ruleset')) return true;
+		if(!is_dir('./files/ruleset')) {
+			return true;
+		}
 
 		return false;
 	}
@@ -140,7 +180,9 @@ 
 				foreach($modules as $key => $module_info)
 				{
 					$module = $module_info->module;
-					if(!in_array($module, array('point','trackback','layout','rss','file','comment','editor'))) continue;
+					if(!in_array($module, array('point','trackback','layout','rss','file','comment','editor'))) {
+						continue;
+					}
 					$config = $oModuleModel->getModuleConfig($module);
 
 					$module_config = null;
@@ -161,7 +203,9 @@ 
 							{
 								foreach($module_config as $key => $val)
 								{
-									if(isset($module_config[$key]->module_srl)) unset($module_config[$key]->module_srl);
+									if(isset($module_config[$key]->module_srl)) {
+										unset($module_config[$key]->module_srl);
+									}
 								}
 							}
 							break;
@@ -171,7 +215,9 @@ 
 							{
 								foreach($tmp as $k => $v)
 								{
-									if(!$v && !trim($v)) continue;
+									if(!$v && !trim($v)) {
+										continue;
+									}
 									$module_config[$k]->header_script = $v;
 								}
 							}
@@ -199,9 +245,13 @@ 
 				$oDB->addIndex("modules","idx_site_mid", array("site_srl","mid"),true);
 			}
 			// document extra vars
-			if(!$oDB->isTableExists('document_extra_vars')) $oDB->createTableByXmlFile('./modules/document/schemas/document_extra_vars.xml');
+			if(!$oDB->isTableExists('document_extra_vars')) {
+				$oDB->createTableByXmlFile('./modules/document/schemas/document_extra_vars.xml');
+			}
 
-			if(!$oDB->isTableExists('document_extra_keys')) $oDB->createTableByXmlFile('./modules/document/schemas/document_extra_keys.xml');
+			if(!$oDB->isTableExists('document_extra_keys')) {
+				$oDB->createTableByXmlFile('./modules/document/schemas/document_extra_keys.xml');
+			}
 			// Move permission, skin info, extection info, admin ID of all modules to the table, grants
 			if($oDB->isColumnExists('modules', 'grants'))
 			{
@@ -219,10 +269,14 @@ 
 						$module_srl = trim($module_info->module_srl);
 						// grant an authority
 						$grants = unserialize($module_info->grants);
-						if($grants) $oModuleController->insertModuleGrants($module_srl, $grants);
+						if($grants) {
+							$oModuleController->insertModuleGrants($module_srl, $grants);
+						}
 						// Insert skin vars
 						$skin_vars = unserialize($module_info->skin_vars);
-						if($skin_vars) $oModuleController->insertModuleSkinVars($module_srl, $skin_vars);
+						if($skin_vars) {
+							$oModuleController->insertModuleSkinVars($module_srl, $skin_vars);
+						}
 						// Insert super admin's ID
 						$admin_id = trim($module_info->admin_id);
 						if($admin_id && $admin_id != 'Array')
@@ -244,7 +298,9 @@ 
 							$document_extra_keys = $extra_vars->extra_vars;
 							unset($extra_vars->extra_vars);
 						}
-						if($extra_vars) $oModuleController->insertModuleExtraVars($module_srl, $extra_vars);
+						if($extra_vars) {
+							$oModuleController->insertModuleExtraVars($module_srl, $extra_vars);
+						}
 
 						/**
 						 * Move document extra vars(it should have conducted in the documents module however extra vars in modules table should be listed up in this module)
@@ -252,7 +308,9 @@ 
 						// Insert extra vars if planet module is
 						if($module_info->module == 'planet')
 						{
-							if(!$document_extra_keys || !is_array($document_extra_keys)) $document_extra_keys = array();
+							if(!$document_extra_keys || !is_array($document_extra_keys)) {
+								$document_extra_keys = array();
+							}
 							$planet_extra_keys->name = 'postscript';
 							$planet_extra_keys->type = 'text';
 							$planet_extra_keys->is_required = 'N';
@@ -291,10 +349,14 @@ 
 									{
 										foreach ($output->data as $document)
 										{
-											if(!$document) continue;
+											if(!$document) {
+												continue;
+											}
 											foreach ($document as $key => $var)
 											{
-												if (strpos($key, 'extra_vars') !== 0 || !trim($var) || $var == 'N;') continue;
+												if (strpos($key, 'extra_vars') !== 0 || !trim($var) || $var == 'N;') {
+													continue;
+												}
 												$var_idx = str_replace('extra_vars','',$key);
 												$oDocumentController->insertDocumentExtraVar($module_srl, $document->document_srl, $var_idx, $var, 'extra_vars'.$var_idx, $lang_code);
 											}
@@ -331,7 +393,9 @@ 
 			// extra_vars * Remove Column
 			for($i=1;$i<=20;$i++)
 			{
-				if(!$oDB->isColumnExists("documents","extra_vars".$i)) continue;
+				if(!$oDB->isColumnExists("documents","extra_vars".$i)) {
+					continue;
+				}
 				$oDB->dropColumn('documents','extra_vars'.$i);
 			}
 
@@ -353,7 +417,9 @@ 
 				$site_args->default_language = $db_info->lang_type;
 
 				$output = executeQuery('module.insertSite', $site_args);
-				if(!$output->toBool()) return $output;
+				if(!$output->toBool()) {
+					return $output;
+				}
 			}
 
 			if($oDB->isIndexExists('sites','idx_domain'))
@@ -399,7 +465,9 @@ 
 				foreach($output->data as $item)
 				{
 					$skin_path = explode('.', $item->skin);
-					if(count($skin_path) != 2) continue;
+					if(count($skin_path) != 2) {
+						continue;
+					}
 					if(is_dir(sprintf(_XE_PATH_ . 'themes/%s/modules/%s', $skin_path[0], $skin_path[1])))
 					{
 						unset($args);
@@ -425,7 +493,9 @@ 
 				$output = executeQuery('module.updateMobileSkinFixModules');
 
 				$oModuleController = getController('module');
-				if(!$moduleConfig) $moduleConfig = new stdClass;
+				if(!$moduleConfig) {
+					$moduleConfig = new stdClass;
+				}
 				$moduleConfig->isUpdateFixedValue = TRUE;
 				$output = $oModuleController->updateModuleConfig('module', $moduleConfig);
 			}
@@ -433,7 +503,9 @@ 
 			$oModuleController->insertUpdatedLog($version_update_id);
 		}
 
-		if(!is_dir('./files/ruleset')) FileHandler::makeDir('./files/ruleset');
+		if(!is_dir('./files/ruleset')) {
+			FileHandler::makeDir('./files/ruleset');
+		}
 
 		return new Object(0, 'success_updated');
 	}
@@ -441,10 +513,14 @@ 
 	function updateForUniqueSiteDomain()
 	{
 		$output = executeQueryArray("module.getNonuniqueDomains");
-		if(!$output->data) return;
+		if(!$output->data) {
+			return;
+		}
 		foreach($output->data as $data)
 		{
-			if($data->count == 1) continue;
+			if($data->count == 1) {
+				continue;
+			}
 			$domain = $data->domain;
 			$args = new stdClass;
 			$args->domain = $domain;