xpressengine / xe-core

modules/member/member.class.php

Spacing +103 added lines, -103 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,76 +184,76 @@ 
 		$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;
 
-			foreach($config->signupForm as $form)
+			foreach ($config->signupForm as $form)
 			{
-				if($form->name === 'email_address' && $form->isPublic !== 'N')
+				if ($form->name === 'email_address' && $form->isPublic !== 'N')
 				{
 					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;
 	}
@@ -271,7 +271,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');
@@ -279,74 +279,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;
@@ -354,7 +354,7 @@ 
 			}
 
 			// check signup form ordering info
-			if(!$config->signupForm || !is_array($config->signupForm))
+			if (!$config->signupForm || !is_array($config->signupForm))
 			{
 				$identifier = 'email_address';
 
@@ -365,9 +365,9 @@ 
 
 			// 회원정보에서 이메일 노출 제거
 			// @see https://github.com/xpressengine/xe-core/issues/2177
-			foreach($config->signupForm as $form)
+			foreach ($config->signupForm as $form)
 			{
-				if($form->name === 'email_address')
+				if ($form->name === 'email_address')
 				{
 					$form->isPublic = 'N';
 					break;
@@ -375,13 +375,13 @@ 
 			}
 			$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');
@@ -391,15 +391,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);
 			}
@@ -408,14 +408,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');
@@ -438,7 +438,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');
@@ -446,18 +446,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;
 			}
@@ -483,7 +483,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');
@@ -491,21 +491,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,36 +213,64 @@ 
 		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;
+			}
 
 			foreach($config->signupForm as $form)
 			{
@@ -227,7 +281,9 @@ 
 			}
 
 			// check agreement field exist
-			if($config->agreement) return true;
+			if($config->agreement) {
+				return true;
+			}
 
 			if($config->skin)
 			{
@@ -235,25 +291,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;
 	}
@@ -391,10 +463,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'))
 			{
@@ -408,12 +482,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'))
 		{
@@ -438,7 +515,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');
@@ -446,7 +525,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'];
@@ -460,16 +541,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;
@@ -483,7 +562,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');
@@ -491,7 +572,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)
@@ -501,8 +584,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.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(in_array($key, array('find_account_question', 'password', 'email_address')))
+			if (in_array($key, array('find_account_question', 'password', 'email_address')))
 			{
 				$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();
 	}

widgets/mcontent/mcontent.class.php

Spacing +145 added lines, -145 removed lines

@@ -18,32 +18,32 @@ 
 	function proc($args)
 	{
 		// Targets to sort
-		if(!in_array($args->order_target, array('list_order','update_order'))) $args->order_target = 'list_order';
+		if (!in_array($args->order_target, array('list_order', 'update_order'))) $args->order_target = 'list_order';
 		// Sort order
-		if(!in_array($args->order_type, array('asc','desc'))) $args->order_type = 'asc';
+		if (!in_array($args->order_type, array('asc', 'desc'))) $args->order_type = 'asc';
 		// The number of displayed lists
-		$args->list_count = (int)$args->list_count;
-		if(!$args->list_count) $args->list_count = 5;
+		$args->list_count = (int) $args->list_count;
+		if (!$args->list_count) $args->list_count = 5;
 		// Cut the length of the title
-		if(!$args->subject_cut_size) $args->subject_cut_size = 0;
+		if (!$args->subject_cut_size) $args->subject_cut_size = 0;
 		// Cut the length of contents
-		if(!$args->content_cut_size) $args->content_cut_size = 100;
+		if (!$args->content_cut_size) $args->content_cut_size = 100;
 		// Viewing options
-		$args->option_view_arr = explode(',',$args->option_view);
+		$args->option_view_arr = explode(',', $args->option_view);
 		// markup options
-		if(!$args->markup_type) $args->markup_type = 'list';
+		if (!$args->markup_type) $args->markup_type = 'list';
 		// Set variables for internal use
 		$oModuleModel = getModel('module');
 		$module_srls = $args->modules_info = $args->module_srls_info = $args->mid_lists = array();
 		$site_module_info = Context::get('site_module_info');
 		// List URLs if a type is RSS
-		if($args->content_type == 'rss')
+		if ($args->content_type == 'rss')
 		{
 			$args->rss_urls = array();
-			$rss_urls = array_unique(array($args->rss_url0,$args->rss_url1,$args->rss_url2,$args->rss_url3,$args->rss_url4));
-			for($i=0,$c=count($rss_urls);$i<$c;$i++)
+			$rss_urls = array_unique(array($args->rss_url0, $args->rss_url1, $args->rss_url2, $args->rss_url3, $args->rss_url4));
+			for ($i = 0, $c = count($rss_urls); $i < $c; $i++)
 			{
-				if($rss_urls[$i]) $args->rss_urls[] = $rss_urls[$i];
+				if ($rss_urls[$i]) $args->rss_urls[] = $rss_urls[$i];
 			}
 			// Get module information after listing module_srls if the module is not RSS
 		}
@@ -51,13 +51,13 @@ 
 		{
 			$obj = new stdClass();
 			// Apply to all modules in the site if a target module is not specified
-			if(!$args->module_srls)
+			if (!$args->module_srls)
 			{
-				$obj->site_srl = (int)$site_module_info->site_srl;
+				$obj->site_srl = (int) $site_module_info->site_srl;
 				$output = executeQueryArray('widgets.content.getMids', $obj);
-				if($output->data)
+				if ($output->data)
 				{
-					foreach($output->data as $key => $val)
+					foreach ($output->data as $key => $val)
 					{
 						$args->modules_info[$val->mid] = $val;
 						$args->module_srls_info[$val->module_srl] = $val;
@@ -73,35 +73,35 @@ 
 			{
 				$obj->module_srls = $args->module_srls;
 				$output = executeQueryArray('widgets.content.getMids', $obj);
-				if($output->data)
+				if ($output->data)
 				{
-					foreach($output->data as $key => $val)
+					foreach ($output->data as $key => $val)
 					{
 						$args->modules_info[$val->mid] = $val;
 						$args->module_srls_info[$val->module_srl] = $val;
 						$module_srls[] = $val->module_srl;
 					}
-					$idx = explode(',',$args->module_srls);
-					for($i=0,$c=count($idx);$i<$c;$i++)
+					$idx = explode(',', $args->module_srls);
+					for ($i = 0, $c = count($idx); $i < $c; $i++)
 					{
 						$srl = $idx[$i];
-						if(!$args->module_srls_info[$srl]) continue;
+						if (!$args->module_srls_info[$srl]) continue;
 						$args->mid_lists[$srl] = $args->module_srls_info[$srl]->mid;
 					}
 				}
 			}
 			// Exit if no module is found
-			if(!count($args->modules_info)) return Context::get('msg_not_founded');
-			$args->module_srl = implode(',',$module_srls);
+			if (!count($args->modules_info)) return Context::get('msg_not_founded');
+			$args->module_srl = implode(',', $module_srls);
 		}
 
 		/**
 		 * Method is separately made because content extraction, articles, comments, trackbacks, RSS and other elements exist
 		 */
 		// tab mode
-		if($args->tab_type == 'none' || $args->tab_type == '')
+		if ($args->tab_type == 'none' || $args->tab_type == '')
 		{
-			switch($args->content_type)
+			switch ($args->content_type)
 			{
 				case 'comment':
 					$content_items = $this->_getCommentItems($args);
@@ -125,17 +125,17 @@ 
 		{
 			$content_items = array();
 
-			switch($args->content_type)
+			switch ($args->content_type)
 			{
 				case 'comment':
-					foreach($args->mid_lists as $module_srl => $mid)
+					foreach ($args->mid_lists as $module_srl => $mid)
 					{
 						$args->module_srl = $module_srl;
 						$content_items[$module_srl] = $this->_getCommentItems($args);
 					}
 					break;
 				case 'image':
-					foreach($args->mid_lists as $module_srl => $mid)
+					foreach ($args->mid_lists as $module_srl => $mid)
 					{
 						$args->module_srl = $module_srl;
 						$content_items[$module_srl] = $this->_getImageItems($args);
@@ -145,13 +145,13 @@ 
 					$content_items = $this->getRssItems($args);
 					break;
 				case 'trackback':
-					foreach($args->mid_lists as $module_srl => $mid){
+					foreach ($args->mid_lists as $module_srl => $mid) {
 						$args->module_srl = $module_srl;
 						$content_items[$module_srl] = $this->_getTrackbackItems($args);
 					}
 					break;
 				default:
-					foreach($args->mid_lists as $module_srl => $mid)
+					foreach ($args->mid_lists as $module_srl => $mid)
 					{
 						$args->module_srl = $module_srl;
 						$content_items[$module_srl] = $this->_getDocumentItems($args);
@@ -160,7 +160,7 @@ 
 			}
 		}
 
-		$output = $this->_compile($args,$content_items);
+		$output = $this->_compile($args, $content_items);
 		return $output;
 	}
 
@@ -180,14 +180,14 @@ 
 
 		$content_items = array();
 
-		if(!count($output)) return;
+		if (!count($output)) return;
 
-		foreach($output as $key => $oComment)
+		foreach ($output as $key => $oComment)
 		{
 			$attribute = $oComment->getObjectVars();
 			$title = $oComment->getSummary($args->content_cut_size);
 			$thumbnail = $oComment->getThumbnail();
-			$url = sprintf("%s#comment_%s",getUrl('', 'mid', $args->mid_lists[$attribute->module_srl], 'document_srl',$oComment->get('document_srl')),$oComment->get('comment_srl'));
+			$url = sprintf("%s#comment_%s", getUrl('', 'mid', $args->mid_lists[$attribute->module_srl], 'document_srl', $oComment->get('document_srl')), $oComment->get('comment_srl'));
 
 			$attribute->mid = $args->mid_lists[$attribute->module_srl];
 			$browser_title = $args->module_srls_info[$attribute->module_srl]->browser_title;
@@ -212,10 +212,10 @@ 
 		// Get categories
 		$obj = new stdClass();
 		$obj->module_srl = $args->module_srl;
-		$output = executeQueryArray('widgets.content.getCategories',$obj);
-		if($output->toBool() && $output->data)
+		$output = executeQueryArray('widgets.content.getCategories', $obj);
+		if ($output->toBool() && $output->data)
 		{
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
 				$category_lists[$val->module_srl][$val->category_srl] = $val;
 			}
@@ -223,17 +223,17 @@ 
 		// Get a list of documents
 		$obj->module_srl = $args->module_srl;
 		$obj->sort_index = $args->order_target;
-		$obj->order_type = $args->order_type=="desc"?"asc":"desc";
+		$obj->order_type = $args->order_type == "desc" ? "asc" : "desc";
 		$obj->list_count = $args->list_count;
 		$obj->statusList = array('PUBLIC');
 		$output = executeQueryArray('widgets.content.getNewestDocuments', $obj);
-		if(!$output->toBool() || !$output->data) return;
+		if (!$output->toBool() || !$output->data) return;
 		// If the result exists, make each document as an object
 		$content_items = array();
 		$first_thumbnail_idx = -1;
-		if(count($output->data))
+		if (count($output->data))
 		{
-			foreach($output->data as $key => $attribute)
+			foreach ($output->data as $key => $attribute)
 			{
 				$oDocument = new documentItem();
 				$oDocument->setAttribute($attribute, false);
@@ -242,23 +242,23 @@ 
 			}
 			$oDocumentModel->setToAllDocumentExtraVars();
 
-			for($i=0,$c=count($document_srls);$i<$c;$i++)
+			for ($i = 0, $c = count($document_srls); $i < $c; $i++)
 			{
 				$oDocument = $GLOBALS['XE_DOCUMENT_LIST'][$document_srls[$i]];
 				$document_srl = $oDocument->document_srl;
 				$module_srl = $oDocument->get('module_srl');
 				$category_srl = $oDocument->get('category_srl');
 				$thumbnail = $oDocument->getThumbnail();
-				$content_item = new mcontentItem( $args->module_srls_info[$module_srl]->browser_title );
+				$content_item = new mcontentItem($args->module_srls_info[$module_srl]->browser_title);
 				$content_item->adds($oDocument->getObjectVars());
 				$content_item->setTitle($oDocument->getTitleText());
-				$content_item->setCategory( $category_lists[$module_srl][$category_srl]->title );
-				$content_item->setDomain( $args->module_srls_info[$module_srl]->domain );
+				$content_item->setCategory($category_lists[$module_srl][$category_srl]->title);
+				$content_item->setDomain($args->module_srls_info[$module_srl]->domain);
 				$content_item->setContent($oDocument->getSummary($args->content_cut_size));
-				$content_item->setLink( getSiteUrl($domain,'', 'mid',$args->mid_lists[$module_srl], 'document_srl',$document_srl) );
+				$content_item->setLink(getSiteUrl($domain, '', 'mid', $args->mid_lists[$module_srl], 'document_srl', $document_srl));
 				$content_item->setThumbnail($thumbnail);
 				$content_item->add('mid', $args->mid_lists[$module_srl]);
-				if($first_thumbnail_idx==-1 && $thumbnail) $first_thumbnail_idx = $i;
+				if ($first_thumbnail_idx == -1 && $thumbnail) $first_thumbnail_idx = $i;
 				$content_items[] = $content_item;
 			}
 
@@ -276,10 +276,10 @@ 
 		$obj->direct_download = 'Y';
 		$obj->isvalid = 'Y';
 		// Get categories
-		$output = executeQueryArray('widgets.content.getCategories',$obj);
-		if($output->toBool() && $output->data)
+		$output = executeQueryArray('widgets.content.getCategories', $obj);
+		if ($output->toBool() && $output->data)
 		{
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
 				$category_lists[$val->module_srl][$val->category_srl] = $val;
 			}
@@ -288,24 +288,24 @@ 
 		$obj->list_count = $args->list_count;
 		$files_output = executeQueryArray("file.getOneFileInDocument", $obj);
 		$files_count = count($files_output->data);
-		if(!$files_count) return;
+		if (!$files_count) return;
 
 		$content_items = array();
 
-		for($i=0;$i<$files_count;$i++) $document_srl_list[] = $files_output->data[$i]->document_srl;
+		for ($i = 0; $i < $files_count; $i++) $document_srl_list[] = $files_output->data[$i]->document_srl;
 
 		$tmp_document_list = $oDocumentModel->getDocuments($document_srl_list);
 
-		if(!count($tmp_document_list)) return;
+		if (!count($tmp_document_list)) return;
 
-		foreach($tmp_document_list as $oDocument)
+		foreach ($tmp_document_list as $oDocument)
 		{
 			$attribute = $oDocument->getObjectVars();
 			$browser_title = $args->module_srls_info[$attribute->module_srl]->browser_title;
 			$domain = $args->module_srls_info[$attribute->module_srl]->domain;
 			$category = $category_lists[$attribute->module_srl]->text;
 			$content = $oDocument->getSummary($args->content_cut_size);
-			$url = sprintf("%s#%s",$oDocument->getPermanentUrl() ,$oDocument->getCommentCount());
+			$url = sprintf("%s#%s", $oDocument->getPermanentUrl(), $oDocument->getCommentCount());
 			$thumbnail = $oDocument->getThumbnail();
 			$content_item = new mcontentItem($browser_title);
 			$content_item->adds($attribute);
@@ -327,11 +327,11 @@ 
 		$content_items = array();
 		$args->mid_lists = array();
 
-		foreach($args->rss_urls as $key => $rss)
+		foreach ($args->rss_urls as $key => $rss)
 		{
 			$args->rss_url = $rss;
 			$content_item = $this->_getRssItems($args);
-			if(count($content_item) > 0)
+			if (count($content_item) > 0)
 			{
 				$browser_title = $content_item[0]->getBrowserTitle();
 				$args->mid_lists[] = $browser_title;
@@ -339,40 +339,40 @@ 
 			}
 		}
 
-		if($args->tab_type == 'none' || $args->tab_type == '')
+		if ($args->tab_type == 'none' || $args->tab_type == '')
 		{
 			$items = array();
-			foreach($content_items as $key => $val)
+			foreach ($content_items as $key => $val)
 			{
-				foreach($val as $k => $v)
+				foreach ($val as $k => $v)
 				{
 					$date = $v->get('regdate');
-					$i=0;
-					while(array_key_exists(sprintf('%s%02d',$date,$i), $items)) $i++;
-					$items[sprintf('%s%02d',$date,$i)] = $v;
+					$i = 0;
+					while (array_key_exists(sprintf('%s%02d', $date, $i), $items)) $i++;
+					$items[sprintf('%s%02d', $date, $i)] = $v;
 				}
 			}
-			if($args->order_type =='asc') ksort($items);
+			if ($args->order_type == 'asc') ksort($items);
 			else krsort($items);
-			$content_items = array_slice(array_values($items),0,$args->list_count);
+			$content_items = array_slice(array_values($items), 0, $args->list_count);
 		} return $content_items;
 	}
 
 	function _getRssBody($value)
 	{
-		if(!$value || is_string($value)) return $value;
-		if(is_object($value)) $value = get_object_vars($value);
+		if (!$value || is_string($value)) return $value;
+		if (is_object($value)) $value = get_object_vars($value);
 		$body = null;
-		if(!count($value)) return;
-		foreach($value as $key => $val)
+		if (!count($value)) return;
+		foreach ($value as $key => $val)
 		{
-			if($key == 'body')
+			if ($key == 'body')
 			{
 				$body = $val;
 				continue;
 			}
-			if(is_object($val)||is_array($val)) $body = $this->_getRssBody($val);
-			if($body !== null) return $body;
+			if (is_object($val) || is_array($val)) $body = $this->_getRssBody($val);
+			if ($body !== null) return $body;
 		}
 		return $body;
 	}
@@ -383,17 +383,17 @@ 
 		// Replace tags such as </p> , </div> , </li> and others to a whitespace
 		$content = str_replace(array('</p>', '</div>', '</li>'), ' ', $content);
 		// Remove Tag
-		$content = preg_replace('!<([^>]*?)>!is','', $content);
+		$content = preg_replace('!<([^>]*?)>!is', '', $content);
 		// Replace tags to < , > , " 
-		$content = str_replace(array('&lt;','&gt;','&quot;','&nbsp;'), array('<','>','"',' '), $content);
+		$content = str_replace(array('&lt;', '&gt;', '&quot;', '&nbsp;'), array('<', '>', '"', ' '), $content);
 		// Delete a series of whitespaces
 		$content = preg_replace('/ ( +)/is', ' ', $content);
 		// Truncate string
 		$content = trim(cut_str($content, $str_size, $tail));
 		// Replace back < , > , "  to theoriginal tags
-		$content = str_replace(array('<','>','"'),array('&lt;','&gt;','&quot;'), $content);
+		$content = str_replace(array('<', '>', '"'), array('&lt;', '&gt;', '&quot;'), $content);
 		// Fixed a newline bug on a set of consecutive English letters 
-		$content = preg_replace('/([a-z0-9\+:\/\.\~,\|\!\@\#\$\%\^\&\*\(\)\_]){20}/is',"$0-",$content);
+		$content = preg_replace('/([a-z0-9\+:\/\.\~,\|\!\@\#\$\%\^\&\*\(\)\_]){20}/is', "$0-", $content);
 		return $content; 
 	}
 
@@ -404,7 +404,7 @@ 
 	 */
 	function requestFeedContents($rss_url)
 	{
-		$rss_url = str_replace('&amp;','&',Context::convertEncodingStr($rss_url));
+		$rss_url = str_replace('&amp;', '&', Context::convertEncodingStr($rss_url));
 		return FileHandler::getRemoteResource($rss_url, null, 3, 'GET', 'application/xml');
 	}
 
@@ -416,52 +416,52 @@ 
 		$buff = $this->requestFeedContents($args->rss_url);
 
 		$encoding = preg_match("/<\?xml.*encoding=\"(.+)\".*\?>/i", $buff, $matches);
-		if($encoding && stripos($matches[1], "UTF-8") === FALSE) $buff = Context::convertEncodingStr($buff);
+		if ($encoding && stripos($matches[1], "UTF-8") === FALSE) $buff = Context::convertEncodingStr($buff);
 
 		$buff = preg_replace("/<\?xml.*\?>/i", "", $buff);
 
 		$oXmlParser = new XmlParser();
 		$xml_doc = $oXmlParser->parse($buff);
 		$rss = new stdClass();
-		if($xml_doc->rss)
+		if ($xml_doc->rss)
 		{
 			$rss->title = $xml_doc->rss->channel->title->body;
 			$rss->link = $xml_doc->rss->channel->link->body;
 
 			$items = $xml_doc->rss->channel->item;
 
-			if(!$items) return;
-			if($items && !is_array($items)) $items = array($items);
+			if (!$items) return;
+			if ($items && !is_array($items)) $items = array($items);
 
 			$content_items = array();
 
 			foreach ($items as $key => $value)
 			{
-				if($key >= $args->list_count) break;
+				if ($key >= $args->list_count) break;
 				unset($item);
 				$item = new stdClass();
 
-				foreach($value as $key2 => $value2)
+				foreach ($value as $key2 => $value2)
 				{
-					if(is_array($value2)) $value2 = array_shift($value2);
+					if (is_array($value2)) $value2 = array_shift($value2);
 					$item->{$key2} = $this->_getRssBody($value2);
 				}
 
 				$content_item = new mcontentItem($rss->title);
 				$content_item->setContentsLink($rss->link);
 				$content_item->setTitle($item->title);
-				$content_item->setNickName(max($item->author,$item->{'dc:creator'}));
+				$content_item->setNickName(max($item->author, $item->{'dc:creator'}));
 				//$content_item->setCategory($item->category);
-				$item->description = preg_replace('!<a href=!is','<a target="_blank" rel="noopener" href=', $item->description);
+				$item->description = preg_replace('!<a href=!is', '<a target="_blank" rel="noopener" href=', $item->description);
 				$content_item->setContent($this->_getSummary($item->description, $args->content_cut_size));
 				$content_item->setLink($item->link);
-				$date = date('YmdHis', strtotime(max($item->pubdate,$item->pubDate,$item->{'dc:date'})));
+				$date = date('YmdHis', strtotime(max($item->pubdate, $item->pubDate, $item->{'dc:date'})));
 				$content_item->setRegdate($date);
 
 				$content_items[] = $content_item;
 			}
 		}
-		else if($xml_doc->{'rdf:rdf'})
+		else if ($xml_doc->{'rdf:rdf'})
 		{
 			// rss1.0 supported (XE's XML is case-insensitive because XML parser converts all to small letters) Fixed by misol
 			$rss->title = $xml_doc->{'rdf:rdf'}->channel->title->body;
@@ -469,102 +469,102 @@ 
 
 			$items = $xml_doc->{'rdf:rdf'}->item;
 
-			if(!$items) return;
-			if($items && !is_array($items)) $items = array($items);
+			if (!$items) return;
+			if ($items && !is_array($items)) $items = array($items);
 
 			$content_items = array();
 
 			foreach ($items as $key => $value)
 			{
-				if($key >= $args->list_count) break;
+				if ($key >= $args->list_count) break;
 				unset($item);
 				$item = new stdClass();
 
-				foreach($value as $key2 => $value2)
+				foreach ($value as $key2 => $value2)
 				{
-					if(is_array($value2)) $value2 = array_shift($value2);
+					if (is_array($value2)) $value2 = array_shift($value2);
 					$item->{$key2} = $this->_getRssBody($value2);
 				}
 
 				$content_item = new mcontentItem($rss->title);
 				$content_item->setContentsLink($rss->link);
 				$content_item->setTitle($item->title);
-				$content_item->setNickName(max($item->author,$item->{'dc:creator'}));
+				$content_item->setNickName(max($item->author, $item->{'dc:creator'}));
 				//$content_item->setCategory($item->category);
-				$item->description = preg_replace('!<a href=!is','<a target="_blank" rel="noopener" href=', $item->description);
+				$item->description = preg_replace('!<a href=!is', '<a target="_blank" rel="noopener" href=', $item->description);
 				$content_item->setContent($this->_getSummary($item->description, $args->content_cut_size));
 				$content_item->setLink($item->link);
-				$date = date('YmdHis', strtotime(max($item->pubdate,$item->pubDate,$item->{'dc:date'})));
+				$date = date('YmdHis', strtotime(max($item->pubdate, $item->pubDate, $item->{'dc:date'})));
 				$content_item->setRegdate($date);
 
 				$content_items[] = $content_item;
 			}
 		}
-		else if($xml_doc->feed && $xml_doc->feed->attrs->xmlns == 'http://www.w3.org/2005/Atom') 
+		else if ($xml_doc->feed && $xml_doc->feed->attrs->xmlns == 'http://www.w3.org/2005/Atom') 
 			{
 			// Atom 1.0 spec supported by misol
 			$rss->title = $xml_doc->feed->title->body;
 			$links = $xml_doc->feed->link;
-			if(is_array($links))
+			if (is_array($links))
 			{
 				foreach ($links as $value)
 				{
-					if($value->attrs->rel == 'alternate')
+					if ($value->attrs->rel == 'alternate')
 					{
 						$rss->link = $value->attrs->href;
 						break;
 					}
 				}
 			}
-			else if($links->attrs->rel == 'alternate') $rss->link = $links->attrs->href;
+			else if ($links->attrs->rel == 'alternate') $rss->link = $links->attrs->href;
 	
 			$items = $xml_doc->feed->entry;
 	
-			if(!$items) return;
-			if($items && !is_array($items)) $items = array($items);
+			if (!$items) return;
+			if ($items && !is_array($items)) $items = array($items);
 	
 			$content_items = array();
 	
 			foreach ($items as $key => $value)
 			{
-				if($key >= $args->list_count) break;
+				if ($key >= $args->list_count) break;
 				unset($item);
 	
-				foreach($value as $key2 => $value2)
+				foreach ($value as $key2 => $value2)
 				{
-					if(is_array($value2)) $value2 = array_shift($value2);
+					if (is_array($value2)) $value2 = array_shift($value2);
 					$item->{$key2} = $this->_getRssBody($value2);
 				}
 	
 				$content_item = new mcontentItem($rss->title);
 				$links = $value->link;
-				if(is_array($links))
+				if (is_array($links))
 				{
 					foreach ($links as $val)
 					{
-						if($val->attrs->rel == 'alternate')
+						if ($val->attrs->rel == 'alternate')
 						{
 							$item->link = $val->attrs->href;
 							break;
 						}
 					}
 				}
-				else if($links->attrs->rel == 'alternate') $item->link = $links->attrs->href;
+				else if ($links->attrs->rel == 'alternate') $item->link = $links->attrs->href;
 	
 				$content_item->setContentsLink($rss->link);
-				if($item->title)
+				if ($item->title)
 				{
-					if(stripos($value->title->attrs->type, "html") === FALSE) $item->title = $value->title->body;
+					if (stripos($value->title->attrs->type, "html") === FALSE) $item->title = $value->title->body;
 				}
 				$content_item->setTitle($item->title);
-				$content_item->setNickName(max($item->author,$item->{'dc:creator'}));
+				$content_item->setNickName(max($item->author, $item->{'dc:creator'}));
 				$content_item->setAuthorSite($value->author->uri->body);
 	
 				//$content_item->setCategory($item->category);
 				$item->description = ($item->content) ? $item->content : $item->description = $item->summary;
-				$item->description = preg_replace('!<a href=!is','<a target="_blank" rel="noopener" href=', $item->description);
+				$item->description = preg_replace('!<a href=!is', '<a target="_blank" rel="noopener" href=', $item->description);
 	
-				if(($item->content && stripos($value->content->attrs->type, "html") === FALSE) || (!$item->content && stripos($value->summary->attrs->type, "html") === FALSE))
+				if (($item->content && stripos($value->content->attrs->type, "html") === FALSE) || (!$item->content && stripos($value->summary->attrs->type, "html") === FALSE))
 				{
 					$item->description = htmlspecialchars($item->description, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 	
@@ -572,7 +572,7 @@ 
 	
 				$content_item->setContent($this->_getSummary($item->description, $args->content_cut_size));
 				$content_item->setLink($item->link);
-				$date = date('YmdHis', strtotime(max($item->published,$item->updated,$item->{'dc:date'})));
+				$date = date('YmdHis', strtotime(max($item->published, $item->updated, $item->{'dc:date'})));
 				$content_item->setRegdate($date);
 	
 				$content_items[] = $content_item;
@@ -581,12 +581,12 @@ 
 		return $content_items;
 	}
 
-	function _getTrackbackItems($args){
+	function _getTrackbackItems($args) {
 		// Get categories
-		$output = executeQueryArray('widgets.content.getCategories',$obj);
-		if($output->toBool() && $output->data)
+		$output = executeQueryArray('widgets.content.getCategories', $obj);
+		if ($output->toBool() && $output->data)
 		{
-			foreach($output->data as $key => $val) {
+			foreach ($output->data as $key => $val) {
 				$category_lists[$val->module_srl][$val->category_srl] = $val;
 			}
 		}
@@ -598,14 +598,14 @@ 
 		$oTrackbackModel = getModel('trackback');
 		$output = $oTrackbackModel->getNewestTrackbackList($obj);
 		// If an error occurs, just ignore it.
-		if(!$output->toBool() || !$output->data) return;
+		if (!$output->toBool() || !$output->data) return;
 		// If the result exists, make each document as an object
 		$content_items = array();
-		foreach($output->data as $key => $item)
+		foreach ($output->data as $key => $item)
 		{
 			$domain = $args->module_srls_info[$item->module_srl]->domain;
 			$category = $category_lists[$item->module_srl]->text;
-			$url = getSiteUrl($domain,'','mid', $args->mid_lists[$item->module_srl],'document_srl',$item->document_srl);
+			$url = getSiteUrl($domain, '', 'mid', $args->mid_lists[$item->module_srl], 'document_srl', $item->document_srl);
 			$browser_title = $args->module_srls_info[$item->module_srl]->browser_title;
 
 			$content_item = new mcontentItem($browser_title);
@@ -613,8 +613,8 @@ 
 			$content_item->setTitle($item->title);
 			$content_item->setCategory($category);
 			$content_item->setNickName($item->blog_name);
-			$content_item->setContent($item->excerpt);  ///<<
-			$content_item->setDomain($domain);  ///<<
+			$content_item->setContent($item->excerpt); ///<<
+			$content_item->setDomain($domain); ///<<
 			$content_item->setLink($url);
 			$content_item->add('mid', $args->mid_lists[$item->module_srl]);
 			$content_item->setRegdate($item->regdate);
@@ -623,7 +623,7 @@ 
 		return $content_items;
 	}
 
-	function _compile($args,$content_items)
+	function _compile($args, $content_items)
 	{
 		$oTemplate = &TemplateHandler::getInstance();
 		// Set variables for widget
@@ -671,7 +671,7 @@ 
 	var $contents_link = null;
 	var $domain = null;
 
-	function mcontentItem($browser_title='')
+	function mcontentItem($browser_title = '')
 	{
 		$this->browser_title = $browser_title;
 	}
@@ -681,58 +681,58 @@ 
 	}
 	function setFirstThumbnailIdx($first_thumbnail_idx)
 	{
-		if(is_null($this->first_thumbnail) && $first_thumbnail_idx>-1) {
+		if (is_null($this->first_thumbnail) && $first_thumbnail_idx > -1) {
 			$this->has_first_thumbnail_idx = true;
-			$this->first_thumbnail_idx= $first_thumbnail_idx;
+			$this->first_thumbnail_idx = $first_thumbnail_idx;
 		}
 	}
 	function setExtraImages($extra_images)
 	{
-		$this->add('extra_images',$extra_images);
+		$this->add('extra_images', $extra_images);
 	}
 	function setDomain($domain)
 	{
 		static $default_domain = null;
-		if(!$domain)
+		if (!$domain)
 		{
-			if(is_null($default_domain)) $default_domain = Context::getDefaultUrl();
+			if (is_null($default_domain)) $default_domain = Context::getDefaultUrl();
 			$domain = $default_domain;
 		}
 		$this->domain = $domain;
 	}
 	function setLink($url)
 	{
-		$this->add('url',$url);
+		$this->add('url', $url);
 	}
 	function setTitle($title)
 	{
-		$this->add('title',$title);
+		$this->add('title', $title);
 	}
 
 	function setThumbnail($thumbnail)
 	{
-		$this->add('thumbnail',$thumbnail);
+		$this->add('thumbnail', $thumbnail);
 	}
 	function setContent($content)
 	{
-		$this->add('content',$content);
+		$this->add('content', $content);
 	}
 	function setRegdate($regdate)
 	{
-		$this->add('regdate',$regdate);
+		$this->add('regdate', $regdate);
 	}
 	function setNickName($nick_name)
 	{
-		$this->add('nick_name',$nick_name);
+		$this->add('nick_name', $nick_name);
 	}
 	// Save author's homepage url by misol
 	function setAuthorSite($site_url)
 	{
-		$this->add('author_site',$site_url);
+		$this->add('author_site', $site_url);
 	}
 	function setCategory($category)
 	{
-		$this->add('category',$category);
+		$this->add('category', $category);
 	}
 	function getBrowserTitle()
 	{
@@ -760,17 +760,17 @@ 
 	{
 		return $this->get('module_srl');
 	}
-	function getTitle($cut_size = 0, $tail='...')
+	function getTitle($cut_size = 0, $tail = '...')
 	{
 		$title = strip_tags($this->get('title'));
 
-		if($cut_size) $title = cut_str($title, $cut_size, $tail);
+		if ($cut_size) $title = cut_str($title, $cut_size, $tail);
 
 		$attrs = array();
-		if($this->get('title_bold') == 'Y') $attrs[] = 'font-weight:bold';
-		if($this->get('title_color') && $this->get('title_color') != 'N') $attrs[] = 'color:#'.$this->get('title_color');
+		if ($this->get('title_bold') == 'Y') $attrs[] = 'font-weight:bold';
+		if ($this->get('title_color') && $this->get('title_color') != 'N') $attrs[] = 'color:#'.$this->get('title_color');
 
-		if(count($attrs)) $title = sprintf("<span style=\"%s\">%s</span>", implode(';', $attrs), htmlspecialchars($title));
+		if (count($attrs)) $title = sprintf("<span style=\"%s\">%s</span>", implode(';', $attrs), htmlspecialchars($title));
 
 		return $title;
 	}
@@ -793,12 +793,12 @@ 
 	function getCommentCount()
 	{
 		$comment_count = $this->get('comment_count');
-		return $comment_count>0 ? $comment_count : '';
+		return $comment_count > 0 ? $comment_count : '';
 	}
 	function getTrackbackCount()
 	{
 		$trackback_count = $this->get('trackback_count');
-		return $trackback_count>0 ? $trackback_count : '';
+		return $trackback_count > 0 ? $trackback_count : '';
 	}
 	function getRegdate($format = 'Y.m.d H:i:s')
 	{

Braces +137 added lines, -55 removed lines

@@ -18,20 +18,32 @@ 
 	function proc($args)
 	{
 		// Targets to sort
-		if(!in_array($args->order_target, array('list_order','update_order'))) $args->order_target = 'list_order';
+		if(!in_array($args->order_target, array('list_order','update_order'))) {
+			$args->order_target = 'list_order';
+		}
 		// Sort order
-		if(!in_array($args->order_type, array('asc','desc'))) $args->order_type = 'asc';
+		if(!in_array($args->order_type, array('asc','desc'))) {
+			$args->order_type = 'asc';
+		}
 		// The number of displayed lists
 		$args->list_count = (int)$args->list_count;
-		if(!$args->list_count) $args->list_count = 5;
+		if(!$args->list_count) {
+			$args->list_count = 5;
+		}
 		// Cut the length of the title
-		if(!$args->subject_cut_size) $args->subject_cut_size = 0;
+		if(!$args->subject_cut_size) {
+			$args->subject_cut_size = 0;
+		}
 		// Cut the length of contents
-		if(!$args->content_cut_size) $args->content_cut_size = 100;
+		if(!$args->content_cut_size) {
+			$args->content_cut_size = 100;
+		}
 		// Viewing options
 		$args->option_view_arr = explode(',',$args->option_view);
 		// markup options
-		if(!$args->markup_type) $args->markup_type = 'list';
+		if(!$args->markup_type) {
+			$args->markup_type = 'list';
+		}
 		// Set variables for internal use
 		$oModuleModel = getModel('module');
 		$module_srls = $args->modules_info = $args->module_srls_info = $args->mid_lists = array();
@@ -43,11 +55,12 @@ 
 			$rss_urls = array_unique(array($args->rss_url0,$args->rss_url1,$args->rss_url2,$args->rss_url3,$args->rss_url4));
 			for($i=0,$c=count($rss_urls);$i<$c;$i++)
 			{
-				if($rss_urls[$i]) $args->rss_urls[] = $rss_urls[$i];
+				if($rss_urls[$i]) {
+					$args->rss_urls[] = $rss_urls[$i];
+				}
 			}
 			// Get module information after listing module_srls if the module is not RSS
-		}
-		else
+		} else
 		{
 			$obj = new stdClass();
 			// Apply to all modules in the site if a target module is not specified
@@ -68,8 +81,7 @@ 
 
 				$args->modules_info = $oModuleModel->getMidList($obj);
 				// Apply to the module only if a target module is specified
-			}
-			else
+			} else
 			{
 				$obj->module_srls = $args->module_srls;
 				$output = executeQueryArray('widgets.content.getMids', $obj);
@@ -85,13 +97,17 @@ 
 					for($i=0,$c=count($idx);$i<$c;$i++)
 					{
 						$srl = $idx[$i];
-						if(!$args->module_srls_info[$srl]) continue;
+						if(!$args->module_srls_info[$srl]) {
+							continue;
+						}
 						$args->mid_lists[$srl] = $args->module_srls_info[$srl]->mid;
 					}
 				}
 			}
 			// Exit if no module is found
-			if(!count($args->modules_info)) return Context::get('msg_not_founded');
+			if(!count($args->modules_info)) {
+				return Context::get('msg_not_founded');
+			}
 			$args->module_srl = implode(',',$module_srls);
 		}
 
@@ -120,8 +136,7 @@ 
 					break;
 			}
 			// If not a tab type
-		}
-		else
+		} else
 		{
 			$content_items = array();
 
@@ -180,7 +195,9 @@ 
 
 		$content_items = array();
 
-		if(!count($output)) return;
+		if(!count($output)) {
+			return;
+		}
 
 		foreach($output as $key => $oComment)
 		{
@@ -227,7 +244,9 @@ 
 		$obj->list_count = $args->list_count;
 		$obj->statusList = array('PUBLIC');
 		$output = executeQueryArray('widgets.content.getNewestDocuments', $obj);
-		if(!$output->toBool() || !$output->data) return;
+		if(!$output->toBool() || !$output->data) {
+			return;
+		}
 		// If the result exists, make each document as an object
 		$content_items = array();
 		$first_thumbnail_idx = -1;
@@ -258,7 +277,9 @@ 
 				$content_item->setLink( getSiteUrl($domain,'', 'mid',$args->mid_lists[$module_srl], 'document_srl',$document_srl) );
 				$content_item->setThumbnail($thumbnail);
 				$content_item->add('mid', $args->mid_lists[$module_srl]);
-				if($first_thumbnail_idx==-1 && $thumbnail) $first_thumbnail_idx = $i;
+				if($first_thumbnail_idx==-1 && $thumbnail) {
+					$first_thumbnail_idx = $i;
+				}
 				$content_items[] = $content_item;
 			}
 
@@ -288,15 +309,21 @@ 
 		$obj->list_count = $args->list_count;
 		$files_output = executeQueryArray("file.getOneFileInDocument", $obj);
 		$files_count = count($files_output->data);
-		if(!$files_count) return;
+		if(!$files_count) {
+			return;
+		}
 
 		$content_items = array();
 
-		for($i=0;$i<$files_count;$i++) $document_srl_list[] = $files_output->data[$i]->document_srl;
+		for($i=0;$i<$files_count;$i++) {
+			$document_srl_list[] = $files_output->data[$i]->document_srl;
+		}
 
 		$tmp_document_list = $oDocumentModel->getDocuments($document_srl_list);
 
-		if(!count($tmp_document_list)) return;
+		if(!count($tmp_document_list)) {
+			return;
+		}
 
 		foreach($tmp_document_list as $oDocument)
 		{
@@ -348,22 +375,33 @@ 
 				{
 					$date = $v->get('regdate');
 					$i=0;
-					while(array_key_exists(sprintf('%s%02d',$date,$i), $items)) $i++;
+					while(array_key_exists(sprintf('%s%02d',$date,$i), $items)) {
+						$i++;
+					}
 					$items[sprintf('%s%02d',$date,$i)] = $v;
 				}
 			}
-			if($args->order_type =='asc') ksort($items);
-			else krsort($items);
+			if($args->order_type =='asc') {
+				ksort($items);
+			} else {
+				krsort($items);
+			}
 			$content_items = array_slice(array_values($items),0,$args->list_count);
 		} return $content_items;
 	}
 
 	function _getRssBody($value)
 	{
-		if(!$value || is_string($value)) return $value;
-		if(is_object($value)) $value = get_object_vars($value);
+		if(!$value || is_string($value)) {
+			return $value;
+		}
+		if(is_object($value)) {
+			$value = get_object_vars($value);
+		}
 		$body = null;
-		if(!count($value)) return;
+		if(!count($value)) {
+			return;
+		}
 		foreach($value as $key => $val)
 		{
 			if($key == 'body')
@@ -371,8 +409,12 @@ 
 				$body = $val;
 				continue;
 			}
-			if(is_object($val)||is_array($val)) $body = $this->_getRssBody($val);
-			if($body !== null) return $body;
+			if(is_object($val)||is_array($val)) {
+				$body = $this->_getRssBody($val);
+			}
+			if($body !== null) {
+				return $body;
+			}
 		}
 		return $body;
 	}
@@ -416,7 +458,9 @@ 
 		$buff = $this->requestFeedContents($args->rss_url);
 
 		$encoding = preg_match("/<\?xml.*encoding=\"(.+)\".*\?>/i", $buff, $matches);
-		if($encoding && stripos($matches[1], "UTF-8") === FALSE) $buff = Context::convertEncodingStr($buff);
+		if($encoding && stripos($matches[1], "UTF-8") === FALSE) {
+			$buff = Context::convertEncodingStr($buff);
+		}
 
 		$buff = preg_replace("/<\?xml.*\?>/i", "", $buff);
 
@@ -430,20 +474,28 @@ 
 
 			$items = $xml_doc->rss->channel->item;
 
-			if(!$items) return;
-			if($items && !is_array($items)) $items = array($items);
+			if(!$items) {
+				return;
+			}
+			if($items && !is_array($items)) {
+				$items = array($items);
+			}
 
 			$content_items = array();
 
 			foreach ($items as $key => $value)
 			{
-				if($key >= $args->list_count) break;
+				if($key >= $args->list_count) {
+					break;
+				}
 				unset($item);
 				$item = new stdClass();
 
 				foreach($value as $key2 => $value2)
 				{
-					if(is_array($value2)) $value2 = array_shift($value2);
+					if(is_array($value2)) {
+						$value2 = array_shift($value2);
+					}
 					$item->{$key2} = $this->_getRssBody($value2);
 				}
 
@@ -460,8 +512,7 @@ 
 
 				$content_items[] = $content_item;
 			}
-		}
-		else if($xml_doc->{'rdf:rdf'})
+		} else if($xml_doc->{'rdf:rdf'})
 		{
 			// rss1.0 supported (XE's XML is case-insensitive because XML parser converts all to small letters) Fixed by misol
 			$rss->title = $xml_doc->{'rdf:rdf'}->channel->title->body;
@@ -469,20 +520,28 @@ 
 
 			$items = $xml_doc->{'rdf:rdf'}->item;
 
-			if(!$items) return;
-			if($items && !is_array($items)) $items = array($items);
+			if(!$items) {
+				return;
+			}
+			if($items && !is_array($items)) {
+				$items = array($items);
+			}
 
 			$content_items = array();
 
 			foreach ($items as $key => $value)
 			{
-				if($key >= $args->list_count) break;
+				if($key >= $args->list_count) {
+					break;
+				}
 				unset($item);
 				$item = new stdClass();
 
 				foreach($value as $key2 => $value2)
 				{
-					if(is_array($value2)) $value2 = array_shift($value2);
+					if(is_array($value2)) {
+						$value2 = array_shift($value2);
+					}
 					$item->{$key2} = $this->_getRssBody($value2);
 				}
 
@@ -499,8 +558,7 @@ 
 
 				$content_items[] = $content_item;
 			}
-		}
-		else if($xml_doc->feed && $xml_doc->feed->attrs->xmlns == 'http://www.w3.org/2005/Atom') 
+		} else if($xml_doc->feed && $xml_doc->feed->attrs->xmlns == 'http://www.w3.org/2005/Atom') 
 			{
 			// Atom 1.0 spec supported by misol
 			$rss->title = $xml_doc->feed->title->body;
@@ -515,24 +573,33 @@ 
 						break;
 					}
 				}
+			} else if($links->attrs->rel == 'alternate') {
+				$rss->link = $links->attrs->href;
 			}
-			else if($links->attrs->rel == 'alternate') $rss->link = $links->attrs->href;
 	
 			$items = $xml_doc->feed->entry;
 	
-			if(!$items) return;
-			if($items && !is_array($items)) $items = array($items);
+			if(!$items) {
+				return;
+			}
+			if($items && !is_array($items)) {
+				$items = array($items);
+			}
 	
 			$content_items = array();
 	
 			foreach ($items as $key => $value)
 			{
-				if($key >= $args->list_count) break;
+				if($key >= $args->list_count) {
+					break;
+				}
 				unset($item);
 	
 				foreach($value as $key2 => $value2)
 				{
-					if(is_array($value2)) $value2 = array_shift($value2);
+					if(is_array($value2)) {
+						$value2 = array_shift($value2);
+					}
 					$item->{$key2} = $this->_getRssBody($value2);
 				}
 	
@@ -548,13 +615,16 @@ 
 							break;
 						}
 					}
+				} else if($links->attrs->rel == 'alternate') {
+					$item->link = $links->attrs->href;
 				}
-				else if($links->attrs->rel == 'alternate') $item->link = $links->attrs->href;
 	
 				$content_item->setContentsLink($rss->link);
 				if($item->title)
 				{
-					if(stripos($value->title->attrs->type, "html") === FALSE) $item->title = $value->title->body;
+					if(stripos($value->title->attrs->type, "html") === FALSE) {
+						$item->title = $value->title->body;
+					}
 				}
 				$content_item->setTitle($item->title);
 				$content_item->setNickName(max($item->author,$item->{'dc:creator'}));
@@ -598,7 +668,9 @@ 
 		$oTrackbackModel = getModel('trackback');
 		$output = $oTrackbackModel->getNewestTrackbackList($obj);
 		// If an error occurs, just ignore it.
-		if(!$output->toBool() || !$output->data) return;
+		if(!$output->toBool() || !$output->data) {
+			return;
+		}
 		// If the result exists, make each document as an object
 		$content_items = array();
 		foreach($output->data as $key => $item)
@@ -695,7 +767,9 @@ 
 		static $default_domain = null;
 		if(!$domain)
 		{
-			if(is_null($default_domain)) $default_domain = Context::getDefaultUrl();
+			if(is_null($default_domain)) {
+				$default_domain = Context::getDefaultUrl();
+			}
 			$domain = $default_domain;
 		}
 		$this->domain = $domain;
@@ -764,13 +838,21 @@ 
 	{
 		$title = strip_tags($this->get('title'));
 
-		if($cut_size) $title = cut_str($title, $cut_size, $tail);
+		if($cut_size) {
+			$title = cut_str($title, $cut_size, $tail);
+		}
 
 		$attrs = array();
-		if($this->get('title_bold') == 'Y') $attrs[] = 'font-weight:bold';
-		if($this->get('title_color') && $this->get('title_color') != 'N') $attrs[] = 'color:#'.$this->get('title_color');
+		if($this->get('title_bold') == 'Y') {
+			$attrs[] = 'font-weight:bold';
+		}
+		if($this->get('title_color') && $this->get('title_color') != 'N') {
+			$attrs[] = 'color:#'.$this->get('title_color');
+		}
 
-		if(count($attrs)) $title = sprintf("<span style=\"%s\">%s</span>", implode(';', $attrs), htmlspecialchars($title));
+		if(count($attrs)) {
+			$title = sprintf("<span style=\"%s\">%s</span>", implode(';', $attrs), htmlspecialchars($title));
+		}
 
 		return $title;
 	}