xpressengine / xe-core

modules/document/document.controller.php

Spacing +367 added lines, -367 removed lines

@@ -24,19 +24,19 @@ 
 	 */
 	function procDocumentVoteUp()
 	{
-		if(!Context::get('is_logged')) return new BaseObject(-1, 'msg_invalid_request');
+		if (!Context::get('is_logged')) return new BaseObject(-1, 'msg_invalid_request');
 
 		$document_srl = Context::get('target_srl');
-		if(!$document_srl) return new BaseObject(-1, 'msg_invalid_request');
+		if (!$document_srl) return new BaseObject(-1, 'msg_invalid_request');
 
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument($document_srl, false, false);
 		$module_srl = $oDocument->get('module_srl');
-		if(!$module_srl) return new BaseObject(-1, 'msg_invalid_request');
+		if (!$module_srl) return new BaseObject(-1, 'msg_invalid_request');
 
 		$oModuleModel = getModel('module');
-		$document_config = $oModuleModel->getModulePartConfig('document',$module_srl);
-		if($document_config->use_vote_up=='N') return new BaseObject(-1, 'msg_invalid_request');
+		$document_config = $oModuleModel->getModulePartConfig('document', $module_srl);
+		if ($document_config->use_vote_up == 'N') return new BaseObject(-1, 'msg_invalid_request');
 
 		$point = 1;
 		$output = $this->updateVotedCount($document_srl, $point);
@@ -69,19 +69,19 @@ 
 	 */
 	function procDocumentVoteDown()
 	{
-		if(!Context::get('is_logged')) return new BaseObject(-1, 'msg_invalid_request');
+		if (!Context::get('is_logged')) return new BaseObject(-1, 'msg_invalid_request');
 
 		$document_srl = Context::get('target_srl');
-		if(!$document_srl) return new BaseObject(-1, 'msg_invalid_request');
+		if (!$document_srl) return new BaseObject(-1, 'msg_invalid_request');
 
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument($document_srl, false, false);
 		$module_srl = $oDocument->get('module_srl');
-		if(!$module_srl) return new BaseObject(-1, 'msg_invalid_request');
+		if (!$module_srl) return new BaseObject(-1, 'msg_invalid_request');
 
 		$oModuleModel = getModel('module');
-		$document_config = $oModuleModel->getModulePartConfig('document',$module_srl);
-		if($document_config->use_vote_down=='N') return new BaseObject(-1, 'msg_invalid_request');
+		$document_config = $oModuleModel->getModulePartConfig('document', $module_srl);
+		if ($document_config->use_vote_down == 'N') return new BaseObject(-1, 'msg_invalid_request');
 
 		$point = -1;
 		$output = $this->updateVotedCount($document_srl, $point);
@@ -95,10 +95,10 @@ 
 	 */
 	function procDocumentDeclare()
 	{
-		if(!Context::get('is_logged')) return new BaseObject(-1, 'msg_invalid_request');
+		if (!Context::get('is_logged')) return new BaseObject(-1, 'msg_invalid_request');
 
 		$document_srl = Context::get('target_srl');
-		if(!$document_srl) return new BaseObject(-1, 'msg_invalid_request');
+		if (!$document_srl) return new BaseObject(-1, 'msg_invalid_request');
 
 		return $this->declaredDocument($document_srl);
 	}
@@ -140,7 +140,7 @@ 
 		$args->history_srl = $history_srl;
 		$args->module_srl = $module_srl;
 		$args->document_srl = $document_srl;
-		if(!$args->history_srl && !$args->module_srl && !$args->document_srl) return;
+		if (!$args->history_srl && !$args->module_srl && !$args->document_srl) return;
 		executeQuery("document.deleteHistory", $args);
 	}
 
@@ -152,15 +152,15 @@ 
 	function triggerDeleteModuleDocuments(&$obj)
 	{
 		$module_srl = $obj->module_srl;
-		if(!$module_srl) return new BaseObject();
+		if (!$module_srl) return new BaseObject();
 		// Delete the document
 		$oDocumentAdminController = getAdminController('document');
 		$output = $oDocumentAdminController->deleteModuleDocument($module_srl);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 		// Delete the category
 		$oDocumentController = getController('document');
 		$output = $oDocumentController->deleteModuleCategory($module_srl);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 		// Delete extra key and variable, because module deleted
 		$this->deleteDocumentExtraKeys($module_srl);
 
@@ -193,7 +193,7 @@ 
 	 */
 	function insertDocument($obj, $manual_inserted = false, $isRestore = false, $isLatest = true)
 	{
-		if(!$manual_inserted && !checkCSRF())
+		if (!$manual_inserted && !checkCSRF())
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
@@ -202,32 +202,32 @@ 
 		$oDB = &DB::getInstance();
 		$oDB->begin();
 		// List variables
-		if($obj->comment_status) $obj->commentStatus = $obj->comment_status;
-		if(!$obj->commentStatus) $obj->commentStatus = 'DENY';
-		if($obj->commentStatus == 'DENY') $this->_checkCommentStatusForOldVersion($obj);
-		if($obj->allow_trackback!='Y') $obj->allow_trackback = 'N';
-		if($obj->homepage) 
+		if ($obj->comment_status) $obj->commentStatus = $obj->comment_status;
+		if (!$obj->commentStatus) $obj->commentStatus = 'DENY';
+		if ($obj->commentStatus == 'DENY') $this->_checkCommentStatusForOldVersion($obj);
+		if ($obj->allow_trackback != 'Y') $obj->allow_trackback = 'N';
+		if ($obj->homepage) 
 		{
 			$obj->homepage = removeHackTag($obj->homepage);
-			if(!preg_match('/^[a-z]+:\/\//i',$obj->homepage))
+			if (!preg_match('/^[a-z]+:\/\//i', $obj->homepage))
 			{
 				$obj->homepage = 'http://'.$obj->homepage;
 			}
 		}
 		
-		if($obj->notify_message != 'Y') $obj->notify_message = 'N';
-		if(!$obj->email_address) $obj->email_address = '';
-		if(!$isRestore) $obj->ipaddress = $_SERVER['REMOTE_ADDR'];
+		if ($obj->notify_message != 'Y') $obj->notify_message = 'N';
+		if (!$obj->email_address) $obj->email_address = '';
+		if (!$isRestore) $obj->ipaddress = $_SERVER['REMOTE_ADDR'];
 
                 // can modify regdate only manager
                 $grant = Context::get('grant');
-		if(!$grant->manager)
+		if (!$grant->manager)
 		{
 			unset($obj->regdate);
 		}
 		
 		// Serialize the $extra_vars, check the extra_vars type, because duplicate serialized avoid
-		if(!is_string($obj->extra_vars)) $obj->extra_vars = serialize($obj->extra_vars);
+		if (!is_string($obj->extra_vars)) $obj->extra_vars = serialize($obj->extra_vars);
 		// Remove the columns for automatic saving
 		unset($obj->_saved_doc_srl);
 		unset($obj->_saved_doc_title);
@@ -235,34 +235,34 @@ 
 		unset($obj->_saved_doc_message);
 		// Call a trigger (before)
 		$output = ModuleHandler::triggerCall('document.insertDocument', 'before', $obj);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 		// Register it if no given document_srl exists
-		if(!$obj->document_srl) $obj->document_srl = getNextSequence();
-		elseif(!$manual_inserted && !$isRestore && !checkUserSequence($obj->document_srl)) return new BaseObject(-1, 'msg_not_permitted');
+		if (!$obj->document_srl) $obj->document_srl = getNextSequence();
+		elseif (!$manual_inserted && !$isRestore && !checkUserSequence($obj->document_srl)) return new BaseObject(-1, 'msg_not_permitted');
 
 		$oDocumentModel = getModel('document');
 		// Set to 0 if the category_srl doesn't exist
-		if($obj->category_srl)
+		if ($obj->category_srl)
 		{
 			$category_list = $oDocumentModel->getCategoryList($obj->module_srl);
-			if(count($category_list) > 0 && !$category_list[$obj->category_srl]->grant)
+			if (count($category_list) > 0 && !$category_list[$obj->category_srl]->grant)
 			{
 				return new BaseObject(-1, 'msg_not_permitted');
 			}
-			if(count($category_list) > 0 && !$category_list[$obj->category_srl]) $obj->category_srl = 0;
+			if (count($category_list) > 0 && !$category_list[$obj->category_srl]) $obj->category_srl = 0;
 		}
 		// Set the read counts and update order.
-		if(!$obj->readed_count) $obj->readed_count = 0;
-		if($isLatest) $obj->update_order = $obj->list_order = $obj->document_srl * -1;
+		if (!$obj->readed_count) $obj->readed_count = 0;
+		if ($isLatest) $obj->update_order = $obj->list_order = $obj->document_srl * -1;
 		else $obj->update_order = $obj->list_order;
 		// Check the status of password hash for manually inserting. Apply hashing for otherwise.
-		if($obj->password && !$obj->password_is_hashed)
+		if ($obj->password && !$obj->password_is_hashed)
 		{
 			$obj->password = getModel('member')->hashPassword($obj->password);
 		}
 		// Insert member's information only if the member is logged-in and not manually registered.
 		$logged_info = Context::get('logged_info');
-		if(Context::get('is_logged') && !$manual_inserted && !$isRestore)
+		if (Context::get('is_logged') && !$manual_inserted && !$isRestore)
 		{
 			$obj->member_srl = $logged_info->member_srl;
 
@@ -276,61 +276,61 @@ 
 		// If the tile is empty, extract string from the contents.
 		$obj->title = htmlspecialchars($obj->title, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 		settype($obj->title, "string");
-		if($obj->title == '') $obj->title = cut_str(trim(strip_tags(nl2br($obj->content))),20,'...');
+		if ($obj->title == '') $obj->title = cut_str(trim(strip_tags(nl2br($obj->content))), 20, '...');
 		// If no tile extracted from the contents, leave it untitled.
-		if($obj->title == '') $obj->title = 'Untitled';
+		if ($obj->title == '') $obj->title = 'Untitled';
 		// Remove XE's own tags from the contents.
 		$obj->content = preg_replace('!<\!--(Before|After)(Document|Comment)\(([0-9]+),([0-9]+)\)-->!is', '', $obj->content);
-		if(Mobile::isFromMobilePhone() && $obj->use_editor != 'Y')
+		if (Mobile::isFromMobilePhone() && $obj->use_editor != 'Y')
 		{
-			if($obj->use_html != 'Y')
+			if ($obj->use_html != 'Y')
 			{
 				$obj->content = htmlspecialchars($obj->content, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 			}
 			$obj->content = nl2br($obj->content);
 		}
 		// Remove iframe and script if not a top adminisrator in the session.
-		if($logged_info->is_admin != 'Y') $obj->content = removeHackTag($obj->content);
+		if ($logged_info->is_admin != 'Y') $obj->content = removeHackTag($obj->content);
 		// An error appears if both log-in info and user name don't exist.
-		if(!$logged_info->member_srl && !$obj->nick_name) return new BaseObject(-1,'msg_invalid_request');
+		if (!$logged_info->member_srl && !$obj->nick_name) return new BaseObject(-1, 'msg_invalid_request');
 
 		$obj->lang_code = Context::getLangType();
 		// Insert data into the DB
-		if(!$obj->status) $this->_checkDocumentStatusForOldVersion($obj);
+		if (!$obj->status) $this->_checkDocumentStatusForOldVersion($obj);
 		$output = executeQuery('document.insertDocument', $obj);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 		// Insert extra variables if the document successfully inserted.
 		$extra_keys = $oDocumentModel->getExtraKeys($obj->module_srl);
-		if(count($extra_keys))
+		if (count($extra_keys))
 		{
-			foreach($extra_keys as $idx => $extra_item)
+			foreach ($extra_keys as $idx => $extra_item)
 			{
 				$value = NULL;
-				if(isset($obj->{'extra_vars'.$idx}))
+				if (isset($obj->{'extra_vars'.$idx}))
 				{
 					$tmp = $obj->{'extra_vars'.$idx};
-					if(is_array($tmp))
+					if (is_array($tmp))
 						$value = implode('|@|', $tmp);
 					else
 						$value = trim($tmp);
 				}
-				else if(isset($obj->{$extra_item->name})) $value = trim($obj->{$extra_item->name});
-				if($value == NULL) continue;
+				else if (isset($obj->{$extra_item->name})) $value = trim($obj->{$extra_item->name});
+				if ($value == NULL) continue;
 
 				$this->insertDocumentExtraVar($obj->module_srl, $obj->document_srl, $idx, $value, $extra_item->eid);
 			}
 		}
 		// Update the category if the category_srl exists.
-		if($obj->category_srl) $this->updateCategoryCount($obj->module_srl, $obj->category_srl);
+		if ($obj->category_srl) $this->updateCategoryCount($obj->module_srl, $obj->category_srl);
 		// Call a trigger (after)
-		if($output->toBool())
+		if ($output->toBool())
 		{
 			$trigger_output = ModuleHandler::triggerCall('document.insertDocument', 'after', $obj);
-			if(!$trigger_output->toBool())
+			if (!$trigger_output->toBool())
 			{
 				$oDB->rollback();
 				return $trigger_output;
@@ -341,12 +341,12 @@ 
 		$oDB->commit();
 
 		// return
-		if(!$manual_inserted)
+		if (!$manual_inserted)
 		{
 			$this->addGrant($obj->document_srl);
 		}
-		$output->add('document_srl',$obj->document_srl);
-		$output->add('category_srl',$obj->category_srl);
+		$output->add('document_srl', $obj->document_srl);
+		$output->add('category_srl', $obj->category_srl);
 
 		return $output;
 	}
@@ -362,41 +362,41 @@ 
 	{
 		$logged_info = Context::get('logged_info');
 
-		if(!$manual_updated && !checkCSRF())
+		if (!$manual_updated && !checkCSRF())
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
 
-		if(!$source_obj->document_srl || !$obj->document_srl) return new BaseObject(-1,'msg_invalied_request');
-		if(!$obj->status && $obj->is_secret == 'Y') $obj->status = 'SECRET';
-		if(!$obj->status) $obj->status = 'PUBLIC';
+		if (!$source_obj->document_srl || !$obj->document_srl) return new BaseObject(-1, 'msg_invalied_request');
+		if (!$obj->status && $obj->is_secret == 'Y') $obj->status = 'SECRET';
+		if (!$obj->status) $obj->status = 'PUBLIC';
 
 		// Call a trigger (before)
 		$output = ModuleHandler::triggerCall('document.updateDocument', 'before', $obj);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		// begin transaction
 		$oDB = &DB::getInstance();
 		$oDB->begin();
 
 		$oModuleModel = getModel('module');
-		if(!$obj->module_srl) $obj->module_srl = $source_obj->get('module_srl');
+		if (!$obj->module_srl) $obj->module_srl = $source_obj->get('module_srl');
 		$module_srl = $obj->module_srl;
 		$document_config = $oModuleModel->getModulePartConfig('document', $module_srl);
-		if(!$document_config)
+		if (!$document_config)
 		{
 			$document_config = new stdClass();
 		}
-		if(!isset($document_config->use_history)) $document_config->use_history = 'N';
+		if (!isset($document_config->use_history)) $document_config->use_history = 'N';
 		$bUseHistory = $document_config->use_history == 'Y' || $document_config->use_history == 'Trace';
 
-		if($bUseHistory)
+		if ($bUseHistory)
 		{
 			$args = new stdClass;
 			$args->history_srl = getNextSequence();
 			$args->document_srl = $obj->document_srl;
 			$args->module_srl = $module_srl;
-			if($document_config->use_history == 'Y') $args->content = $source_obj->get('content');
+			if ($document_config->use_history == 'Y') $args->content = $source_obj->get('content');
 			$args->nick_name = $source_obj->get('nick_name');
 			$args->member_srl = $source_obj->get('member_srl');
 			$args->regdate = $source_obj->get('last_update');
@@ -408,30 +408,30 @@ 
 			$obj->ipaddress = $source_obj->get('ipaddress');
 		}
 		// List variables
-		if($obj->comment_status) $obj->commentStatus = $obj->comment_status;
-		if(!$obj->commentStatus) $obj->commentStatus = 'DENY';
-		if($obj->commentStatus == 'DENY') $this->_checkCommentStatusForOldVersion($obj);
-		if($obj->allow_trackback!='Y') $obj->allow_trackback = 'N';
-		if($obj->homepage)
+		if ($obj->comment_status) $obj->commentStatus = $obj->comment_status;
+		if (!$obj->commentStatus) $obj->commentStatus = 'DENY';
+		if ($obj->commentStatus == 'DENY') $this->_checkCommentStatusForOldVersion($obj);
+		if ($obj->allow_trackback != 'Y') $obj->allow_trackback = 'N';
+		if ($obj->homepage)
 		{
 			$obj->homepage = removeHackTag($obj->homepage);
-			if(!preg_match('/^[a-z]+:\/\//i',$obj->homepage))
+			if (!preg_match('/^[a-z]+:\/\//i', $obj->homepage))
 			{
 				$obj->homepage = 'http://'.$obj->homepage;
 			}
 		}
 		
-		if($obj->notify_message != 'Y') $obj->notify_message = 'N';
+		if ($obj->notify_message != 'Y') $obj->notify_message = 'N';
 		
 		// can modify regdate only manager
                 $grant = Context::get('grant');
-		if(!$grant->manager)
+		if (!$grant->manager)
 		{
 			unset($obj->regdate);
 		}
 		
 		// Serialize the $extra_vars
-		if(!is_string($obj->extra_vars)) $obj->extra_vars = serialize($obj->extra_vars);
+		if (!is_string($obj->extra_vars)) $obj->extra_vars = serialize($obj->extra_vars);
 		// Remove the columns for automatic saving
 		unset($obj->_saved_doc_srl);
 		unset($obj->_saved_doc_title);
@@ -440,23 +440,23 @@ 
 
 		$oDocumentModel = getModel('document');
 		// Set the category_srl to 0 if the changed category is not exsiting.
-		if($source_obj->get('category_srl')!=$obj->category_srl)
+		if ($source_obj->get('category_srl') != $obj->category_srl)
 		{
 			$category_list = $oDocumentModel->getCategoryList($obj->module_srl);
-			if(!$category_list[$obj->category_srl]) $obj->category_srl = 0;
+			if (!$category_list[$obj->category_srl]) $obj->category_srl = 0;
 		}
 		// Change the update order
 		$obj->update_order = getNextSequence() * -1;
 		// Hash the password if it exists
-		if($obj->password)
+		if ($obj->password)
 		{
 			$obj->password = getModel('member')->hashPassword($obj->password);
 		}
 
 		// If an author is identical to the modifier or history is used, use the logged-in user's information.
-		if(Context::get('is_logged') && !$manual_updated)
+		if (Context::get('is_logged') && !$manual_updated)
 		{
-			if($source_obj->get('member_srl')==$logged_info->member_srl)
+			if ($source_obj->get('member_srl') == $logged_info->member_srl)
 			{
 				$obj->member_srl = $logged_info->member_srl;
 				$obj->user_name = htmlspecialchars_decode($logged_info->user_name);
@@ -467,7 +467,7 @@ 
 		}
 
 		// For the document written by logged-in user however no nick_name exists
-		if($source_obj->get('member_srl')&& !$obj->nick_name)
+		if ($source_obj->get('member_srl') && !$obj->nick_name)
 		{
 			$obj->member_srl = $source_obj->get('member_srl');
 			$obj->user_name = $source_obj->get('user_name');
@@ -478,24 +478,24 @@ 
 		// If the tile is empty, extract string from the contents.
 		$obj->title = htmlspecialchars($obj->title, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 		settype($obj->title, "string");
-		if($obj->title == '') $obj->title = cut_str(strip_tags($obj->content),20,'...');
+		if ($obj->title == '') $obj->title = cut_str(strip_tags($obj->content), 20, '...');
 		// If no tile extracted from the contents, leave it untitled.
-		if($obj->title == '') $obj->title = 'Untitled';
+		if ($obj->title == '') $obj->title = 'Untitled';
 		// Remove XE's own tags from the contents.
 		$obj->content = preg_replace('!<\!--(Before|After)(Document|Comment)\(([0-9]+),([0-9]+)\)-->!is', '', $obj->content);
-		if(Mobile::isFromMobilePhone() && $obj->use_editor != 'Y')
+		if (Mobile::isFromMobilePhone() && $obj->use_editor != 'Y')
 		{
-			if($obj->use_html != 'Y')
+			if ($obj->use_html != 'Y')
 			{
 				$obj->content = htmlspecialchars($obj->content, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 			}
 			$obj->content = nl2br($obj->content);
 		}
 		// Change not extra vars but language code of the original document if document's lang_code is different from author's setting.
-		if($source_obj->get('lang_code') != Context::getLangType())
+		if ($source_obj->get('lang_code') != Context::getLangType())
 		{
 			// Change not extra vars but language code of the original document if document's lang_code doesn't exist.
-			if(!$source_obj->get('lang_code'))
+			if (!$source_obj->get('lang_code'))
 			{
 				$lang_code_args->document_srl = $source_obj->get('document_srl');
 				$lang_code_args->lang_code = Context::getLangType();
@@ -515,59 +515,59 @@ 
 			}
 		}
 		// Remove iframe and script if not a top adminisrator in the session.
-		if($logged_info->is_admin != 'Y')
+		if ($logged_info->is_admin != 'Y')
 		{
 			$obj->content = removeHackTag($obj->content);
 		}
 		// if temporary document, regdate is now setting
-		if($source_obj->get('status') == $this->getConfigStatus('temp')) $obj->regdate = date('YmdHis');
+		if ($source_obj->get('status') == $this->getConfigStatus('temp')) $obj->regdate = date('YmdHis');
 
 		// Insert data into the DB
 		$output = executeQuery('document.updateDocument', $obj);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 		// Remove all extra variables
-		if(Context::get('act')!='procFileDelete')
+		if (Context::get('act') != 'procFileDelete')
 		{
 			$this->deleteDocumentExtraVars($source_obj->get('module_srl'), $obj->document_srl, null, Context::getLangType());
 			// Insert extra variables if the document successfully inserted.
 			$extra_keys = $oDocumentModel->getExtraKeys($obj->module_srl);
-			if(count($extra_keys))
+			if (count($extra_keys))
 			{
-				foreach($extra_keys as $idx => $extra_item)
+				foreach ($extra_keys as $idx => $extra_item)
 				{
 					$value = NULL;
-					if(isset($obj->{'extra_vars'.$idx}))
+					if (isset($obj->{'extra_vars'.$idx}))
 					{
 						$tmp = $obj->{'extra_vars'.$idx};
-						if(is_array($tmp))
+						if (is_array($tmp))
 							$value = implode('|@|', $tmp);
 						else
 							$value = trim($tmp);
 					}
-					else if(isset($obj->{$extra_item->name})) $value = trim($obj->{$extra_item->name});
-					if($value == NULL) continue;
+					else if (isset($obj->{$extra_item->name})) $value = trim($obj->{$extra_item->name});
+					if ($value == NULL) continue;
 					$this->insertDocumentExtraVar($obj->module_srl, $obj->document_srl, $idx, $value, $extra_item->eid);
 				}
 			}
 			// Inert extra vars for multi-language support of title and contents.
-			if($extra_content->title) $this->insertDocumentExtraVar($obj->module_srl, $obj->document_srl, -1, $extra_content->title, 'title_'.Context::getLangType());
-			if($extra_content->content) $this->insertDocumentExtraVar($obj->module_srl, $obj->document_srl, -2, $extra_content->content, 'content_'.Context::getLangType());
+			if ($extra_content->title) $this->insertDocumentExtraVar($obj->module_srl, $obj->document_srl, -1, $extra_content->title, 'title_'.Context::getLangType());
+			if ($extra_content->content) $this->insertDocumentExtraVar($obj->module_srl, $obj->document_srl, -2, $extra_content->content, 'content_'.Context::getLangType());
 		}
 		// Update the category if the category_srl exists.
-		if($source_obj->get('category_srl') != $obj->category_srl || $source_obj->get('module_srl') == $logged_info->member_srl)
+		if ($source_obj->get('category_srl') != $obj->category_srl || $source_obj->get('module_srl') == $logged_info->member_srl)
 		{
-			if($source_obj->get('category_srl') != $obj->category_srl) $this->updateCategoryCount($obj->module_srl, $source_obj->get('category_srl'));
-			if($obj->category_srl) $this->updateCategoryCount($obj->module_srl, $obj->category_srl);
+			if ($source_obj->get('category_srl') != $obj->category_srl) $this->updateCategoryCount($obj->module_srl, $source_obj->get('category_srl'));
+			if ($obj->category_srl) $this->updateCategoryCount($obj->module_srl, $obj->category_srl);
 		}
 		// Call a trigger (after)
-		if($output->toBool())
+		if ($output->toBool())
 		{
 			$trigger_output = ModuleHandler::triggerCall('document.updateDocument', 'after', $obj);
-			if(!$trigger_output->toBool())
+			if (!$trigger_output->toBool())
 			{
 				$oDB->rollback();
 				return $trigger_output;
@@ -577,15 +577,15 @@ 
 		// commit
 		$oDB->commit();
 		// Remove the thumbnail file
-		FileHandler::removeDir(sprintf('files/thumbnails/%s',getNumberingPath($obj->document_srl, 3)));
+		FileHandler::removeDir(sprintf('files/thumbnails/%s', getNumberingPath($obj->document_srl, 3)));
 
-		$output->add('document_srl',$obj->document_srl);
+		$output->add('document_srl', $obj->document_srl);
 		//remove from cache
 		$oCacheHandler = CacheHandler::getInstance('object');
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			//remove document item from cache
-			$cache_key = 'document_item:'. getNumberingPath($obj->document_srl) . $obj->document_srl;
+			$cache_key = 'document_item:'.getNumberingPath($obj->document_srl).$obj->document_srl;
 			$oCacheHandler->delete($cache_key);
 		}
 
@@ -606,33 +606,33 @@ 
 		$trigger_obj = new stdClass();
 		$trigger_obj->document_srl = $document_srl;
 		$output = ModuleHandler::triggerCall('document.deleteDocument', 'before', $trigger_obj);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		// begin transaction
 		$oDB = &DB::getInstance();
 		$oDB->begin();
 
-		if(!$isEmptyTrash)
+		if (!$isEmptyTrash)
 		{
 			// get model object of the document
 			$oDocumentModel = getModel('document');
 			// Check if the documnet exists
 			$oDocument = $oDocumentModel->getDocument($document_srl, $is_admin);
 		}
-		else if($isEmptyTrash && $oDocument == null) return new BaseObject(-1, 'document is not exists');
+		else if ($isEmptyTrash && $oDocument == null) return new BaseObject(-1, 'document is not exists');
 
-		if(!$oDocument->isExists() || $oDocument->document_srl != $document_srl) return new BaseObject(-1, 'msg_invalid_document');
+		if (!$oDocument->isExists() || $oDocument->document_srl != $document_srl) return new BaseObject(-1, 'msg_invalid_document');
 		// Check if a permossion is granted
-		if(!$oDocument->isGranted()) return new BaseObject(-1, 'msg_not_permitted');
+		if (!$oDocument->isGranted()) return new BaseObject(-1, 'msg_not_permitted');
 
 		//if empty trash, document already deleted, therefore document not delete
 		$args = new stdClass();
 		$args->document_srl = $document_srl;
-		if(!$isEmptyTrash)
+		if (!$isEmptyTrash)
 		{
 			// Delete the document
 			$output = executeQuery('document.deleteDocument', $args);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				$oDB->rollback();
 				return $output;
@@ -643,7 +643,7 @@ 
 
 		$this->deleteDocumentHistory(null, $document_srl, null);
 		// Update category information if the category_srl exists.
-		if($oDocument->get('category_srl')) $this->updateCategoryCount($oDocument->get('module_srl'),$oDocument->get('category_srl'));
+		if ($oDocument->get('category_srl')) $this->updateCategoryCount($oDocument->get('module_srl'), $oDocument->get('category_srl'));
 		// Delete a declared list
 		executeQuery('document.deleteDeclared', $args);
 		// Delete extra variable
@@ -651,11 +651,11 @@ 
 
 		//this
 		// Call a trigger (after)
-		if($output->toBool())
+		if ($output->toBool())
 		{
 			$trigger_obj = $oDocument->getObjectVars();
 			$trigger_output = ModuleHandler::triggerCall('document.deleteDocument', 'after', $trigger_obj);
-			if(!$trigger_output->toBool())
+			if (!$trigger_output->toBool())
 			{
 				$oDB->rollback();
 				return $trigger_output;
@@ -667,16 +667,16 @@ 
 		$this->_deleteDocumentVotedLog($args);
 
 		// Remove the thumbnail file
-		FileHandler::removeDir(sprintf('files/thumbnails/%s',getNumberingPath($document_srl, 3)));
+		FileHandler::removeDir(sprintf('files/thumbnails/%s', getNumberingPath($document_srl, 3)));
 
 		// commit
 		$oDB->commit();
 
 		//remove from cache
 		$oCacheHandler = CacheHandler::getInstance('object');
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
-			$cache_key = 'document_item:'. getNumberingPath($document_srl) . $document_srl;
+			$cache_key = 'document_item:'.getNumberingPath($document_srl).$document_srl;
 			$oCacheHandler->delete($cache_key);
 		}
 
@@ -723,7 +723,7 @@ 
 	{
 		$trash_args = new stdClass();
 		// Get trash_srl if a given trash_srl doesn't exist
-		if(!$obj->trash_srl) $trash_args->trash_srl = getNextSequence();
+		if (!$obj->trash_srl) $trash_args->trash_srl = getNextSequence();
 		else $trash_args->trash_srl = $obj->trash_srl;
 		// Get its module_srl which the document belongs to
 		$oDocumentModel = getModel('document');
@@ -732,12 +732,12 @@ 
 		$trash_args->module_srl = $oDocument->get('module_srl');
 		$obj->module_srl = $oDocument->get('module_srl');
 		// Cannot throw data from the trash to the trash
-		if($trash_args->module_srl == 0) return false;
+		if ($trash_args->module_srl == 0) return false;
 		// Data setting
 		$trash_args->document_srl = $obj->document_srl;
 		$trash_args->description = $obj->description;
 		// Insert member's information only if the member is logged-in and not manually registered.
-		if(Context::get('is_logged')&&!$manual_inserted)
+		if (Context::get('is_logged') && !$manual_inserted)
 		{
 			$logged_info = Context::get('logged_info');
 			$trash_args->member_srl = $logged_info->member_srl;
@@ -773,14 +773,14 @@ 
 
 		$oTrashAdminController = getAdminController('trash');
 		$output = $oTrashAdminController->insertTrash($oTrashVO);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 
 		$output = executeQuery('document.deleteDocument', $trash_args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
@@ -793,12 +793,12 @@ 
 		  }*/
 
 		// update category
-		if($oDocument->get('category_srl')) $this->updateCategoryCount($oDocument->get('module_srl'),$oDocument->get('category_srl'));
+		if ($oDocument->get('category_srl')) $this->updateCategoryCount($oDocument->get('module_srl'), $oDocument->get('category_srl'));
 
 		// remove thumbnails
-		FileHandler::removeDir(sprintf('files/thumbnails/%s',getNumberingPath($obj->document_srl, 3)));
+		FileHandler::removeDir(sprintf('files/thumbnails/%s', getNumberingPath($obj->document_srl, 3)));
 		// Set the attachment to be invalid state
-		if($oDocument->hasUploadedFiles())
+		if ($oDocument->hasUploadedFiles())
 		{
 			$args = new stdClass();
 			$args->upload_target_srl = $oDocument->document_srl;
@@ -806,10 +806,10 @@ 
 			executeQuery('file.updateFileValid', $args);
 		}
 		// Call a trigger (after)
-		if($output->toBool())
+		if ($output->toBool())
 		{
 			$trigger_output = ModuleHandler::triggerCall('document.moveDocumentToTrash', 'after', $obj);
-			if(!$trigger_output->toBool())
+			if (!$trigger_output->toBool())
 			{
 				$oDB->rollback();
 				return $trigger_output;
@@ -821,9 +821,9 @@ 
 
 		// Clear cache
 		$oCacheHandler = CacheHandler::getInstance('object');
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
-			$cache_key = 'document_item:'. getNumberingPath($oDocument->document_srl) . $oDocument->document_srl;
+			$cache_key = 'document_item:'.getNumberingPath($oDocument->document_srl).$oDocument->document_srl;
 			$oCacheHandler->delete($cache_key);
 		}
 
@@ -838,7 +838,7 @@ 
 	function updateReadedCount(&$oDocument)
 	{
 		// Pass if Crawler access
-		if(isCrawler()) return false;
+		if (isCrawler()) return false;
 		
 		$document_srl = $oDocument->document_srl;
 		$member_srl = $oDocument->get('member_srl');
@@ -846,19 +846,19 @@ 
 
 		// Call a trigger when the read count is updated (before)
 		$trigger_output = ModuleHandler::triggerCall('document.updateReadedCount', 'before', $oDocument);
-		if(!$trigger_output->toBool()) return $trigger_output;
+		if (!$trigger_output->toBool()) return $trigger_output;
 
 		// Pass if read count is increaded on the session information
-		if($_SESSION['readed_document'][$document_srl]) return false;
+		if ($_SESSION['readed_document'][$document_srl]) return false;
 
 		// Pass if the author's IP address is as same as visitor's.
-		if($oDocument->get('ipaddress') == $_SERVER['REMOTE_ADDR'])
+		if ($oDocument->get('ipaddress') == $_SERVER['REMOTE_ADDR'])
 		{
 			$_SESSION['readed_document'][$document_srl] = true;
 			return false;
 		}
 		// Pass ater registering sesscion if the author is a member and has same information as the currently logged-in user.
-		if($member_srl && $logged_info->member_srl == $member_srl)
+		if ($member_srl && $logged_info->member_srl == $member_srl)
 		{
 			$_SESSION['readed_document'][$document_srl] = true;
 			return false;
@@ -874,7 +874,7 @@ 
 
 		// Call a trigger when the read count is updated (after)
 		$trigger_output = ModuleHandler::triggerCall('document.updateReadedCount', 'after', $oDocument);
-		if(!$trigger_output->toBool())
+		if (!$trigger_output->toBool())
 		{
 			$oDB->rollback();
 			return $trigger_output;
@@ -883,15 +883,15 @@ 
 		$oDB->commit();
 
 		$oCacheHandler = CacheHandler::getInstance('object');
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			//remove document item from cache
-			$cache_key = 'document_item:'. getNumberingPath($document_srl) . $document_srl;
+			$cache_key = 'document_item:'.getNumberingPath($document_srl).$document_srl;
 			$oCacheHandler->delete($cache_key);
 		}
 
 		// Register session
-		if(!$_SESSION['banned_document'][$document_srl]) 
+		if (!$_SESSION['banned_document'][$document_srl]) 
 		{
 			$_SESSION['readed_document'][$document_srl] = true;
 		}
@@ -914,21 +914,21 @@ 
 	 */
 	function insertDocumentExtraKey($module_srl, $var_idx, $var_name, $var_type, $var_is_required = 'N', $var_search = 'N', $var_default = '', $var_desc = '', $eid)
 	{
-		if(!$module_srl || !$var_idx || !$var_name || !$var_type || !$eid) return new BaseObject(-1,'msg_invalid_request');
+		if (!$module_srl || !$var_idx || !$var_name || !$var_type || !$eid) return new BaseObject(-1, 'msg_invalid_request');
 
 		$obj = new stdClass();
 		$obj->module_srl = $module_srl;
 		$obj->var_idx = $var_idx;
 		$obj->var_name = $var_name;
 		$obj->var_type = $var_type;
-		$obj->var_is_required = $var_is_required=='Y'?'Y':'N';
-		$obj->var_search = $var_search=='Y'?'Y':'N';
+		$obj->var_is_required = $var_is_required == 'Y' ? 'Y' : 'N';
+		$obj->var_search = $var_search == 'Y' ? 'Y' : 'N';
 		$obj->var_default = $var_default;
 		$obj->var_desc = $var_desc;
 		$obj->eid = $eid;
 
 		$output = executeQuery('document.getDocumentExtraKeys', $obj);
-		if(!$output->data)
+		if (!$output->data)
 		{
 			$output = executeQuery('document.insertDocumentExtraKey', $obj);
 		}
@@ -940,7 +940,7 @@ 
 		}
 
 		$oCacheHandler = CacheHandler::getInstance('object', NULL, TRUE);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$object_key = 'module_document_extra_keys:'.$module_srl;
 			$cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
@@ -958,42 +958,42 @@ 
 	 */
 	function deleteDocumentExtraKeys($module_srl, $var_idx = null)
 	{
-		if(!$module_srl) return new BaseObject(-1,'msg_invalid_request');
+		if (!$module_srl) return new BaseObject(-1, 'msg_invalid_request');
 		$obj = new stdClass();
 		$obj->module_srl = $module_srl;
-		if(!is_null($var_idx)) $obj->var_idx = $var_idx;
+		if (!is_null($var_idx)) $obj->var_idx = $var_idx;
 
 		$oDB = DB::getInstance();
 		$oDB->begin();
 
 		$output = $oDB->executeQuery('document.deleteDocumentExtraKeys', $obj);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 
-		if($var_idx != NULL)
+		if ($var_idx != NULL)
 		{
 			$output = $oDB->executeQuery('document.updateDocumentExtraKeyIdxOrder', $obj);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				$oDB->rollback();
 				return $output;
 			}
 		}
 
-		$output =  executeQuery('document.deleteDocumentExtraVars', $obj);
-		if(!$output->toBool())
+		$output = executeQuery('document.deleteDocumentExtraVars', $obj);
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 
-		if($var_idx != NULL)
+		if ($var_idx != NULL)
 		{
 			$output = $oDB->executeQuery('document.updateDocumentExtraVarIdxOrder', $obj);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				$oDB->rollback();
 				return $output;
@@ -1003,7 +1003,7 @@ 
 		$oDB->commit();
 
 		$oCacheHandler = CacheHandler::getInstance('object', NULL, TRUE);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$object_key = 'module_document_extra_keys:'.$module_srl;
 			$cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
@@ -1025,8 +1025,8 @@ 
 	 */
 	function insertDocumentExtraVar($module_srl, $document_srl, $var_idx, $value, $eid = null, $lang_code = '')
 	{
-		if(!$module_srl || !$document_srl || !$var_idx || !isset($value)) return new BaseObject(-1,'msg_invalid_request');
-		if(!$lang_code) $lang_code = Context::getLangType();
+		if (!$module_srl || !$document_srl || !$var_idx || !isset($value)) return new BaseObject(-1, 'msg_invalid_request');
+		if (!$lang_code) $lang_code = Context::getLangType();
 
 		$obj = new stdClass;
 		$obj->module_srl = $module_srl;
@@ -1052,10 +1052,10 @@ 
 	{
 		$obj = new stdClass();
 		$obj->module_srl = $module_srl;
-		if(!is_null($document_srl)) $obj->document_srl = $document_srl;
-		if(!is_null($var_idx)) $obj->var_idx = $var_idx;
-		if(!is_null($lang_code)) $obj->lang_code = $lang_code;
-		if(!is_null($eid)) $obj->eid = $eid;
+		if (!is_null($document_srl)) $obj->document_srl = $document_srl;
+		if (!is_null($var_idx)) $obj->var_idx = $var_idx;
+		if (!is_null($lang_code)) $obj->lang_code = $lang_code;
+		if (!is_null($eid)) $obj->eid = $eid;
 		$output = executeQuery('document.deleteDocumentExtraVars', $obj);
 		return $output;
 	}
@@ -1069,10 +1069,10 @@ 
 	 */
 	function updateVotedCount($document_srl, $point = 1)
 	{
-		if($point > 0) $failed_voted = 'failed_voted';
+		if ($point > 0) $failed_voted = 'failed_voted';
 		else $failed_voted = 'failed_blamed';
 		// Return fail if session already has information about votes
-		if($_SESSION['voted_document'][$document_srl])
+		if ($_SESSION['voted_document'][$document_srl])
 		{
 			return new BaseObject(-1, $failed_voted);
 		}
@@ -1080,7 +1080,7 @@ 
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument($document_srl, false, false);
 		// Pass if the author's IP address is as same as visitor's.
-		if($oDocument->get('ipaddress') == $_SERVER['REMOTE_ADDR'])
+		if ($oDocument->get('ipaddress') == $_SERVER['REMOTE_ADDR'])
 		{
 			$_SESSION['voted_document'][$document_srl] = true;
 			return new BaseObject(-1, $failed_voted);
@@ -1091,10 +1091,10 @@ 
 		$member_srl = $oMemberModel->getLoggedMemberSrl();
 
 		// Check if document's author is a member.
-		if($oDocument->get('member_srl'))
+		if ($oDocument->get('member_srl'))
 		{
 			// Pass after registering a session if author's information is same as the currently logged-in user's.
-			if($member_srl && $member_srl == abs($oDocument->get('member_srl')))
+			if ($member_srl && $member_srl == abs($oDocument->get('member_srl')))
 			{
 				$_SESSION['voted_document'][$document_srl] = true;
 				return new BaseObject(-1, $failed_voted);
@@ -1103,7 +1103,7 @@ 
 
 		// Use member_srl for logged-in members and IP address for non-members.
 		$args = new stdClass;
-		if($member_srl)
+		if ($member_srl)
 		{
 			$args->member_srl = $member_srl;
 		}
@@ -1114,7 +1114,7 @@ 
 		$args->document_srl = $document_srl;
 		$output = executeQuery('document.getDocumentVotedLogInfo', $args);
 		// Pass after registering a session if log information has vote-up logs
-		if($output->data->count)
+		if ($output->data->count)
 		{
 			$_SESSION['voted_document'][$document_srl] = true;
 			return new BaseObject(-1, $failed_voted);
@@ -1125,7 +1125,7 @@ 
 		$oDB->begin();
 
 		// Update the voted count
-		if($point < 0)
+		if ($point < 0)
 		{
 			$args->blamed_count = $oDocument->get('blamed_count') + $point;
 			$output = executeQuery('document.updateBlamedCount', $args);
@@ -1135,11 +1135,11 @@ 
 			$args->voted_count = $oDocument->get('voted_count') + $point;
 			$output = executeQuery('document.updateVotedCount', $args);
 		}
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 		// Leave logs
 		$args->point = $point;
 		$output = executeQuery('document.insertDocumentVotedLog', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$obj = new stdClass;
 		$obj->member_srl = $oDocument->get('member_srl');
@@ -1150,7 +1150,7 @@ 
 		$obj->before_point = ($point < 0) ? $oDocument->get('blamed_count') : $oDocument->get('voted_count');
 		$obj->after_point = ($point < 0) ? $args->blamed_count : $args->voted_count;
 		$trigger_output = ModuleHandler::triggerCall('document.updateVotedCount', 'after', $obj);
-		if(!$trigger_output->toBool())
+		if (!$trigger_output->toBool())
 		{
 			$oDB->rollback();
 			return $trigger_output;
@@ -1159,10 +1159,10 @@ 
 		$oDB->commit();
 
 		$oCacheHandler = CacheHandler::getInstance('object');
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			//remove document item from cache
-			$cache_key = 'document_item:'. getNumberingPath($document_srl) . $document_srl;
+			$cache_key = 'document_item:'.getNumberingPath($document_srl).$document_srl;
 			$oCacheHandler->delete($cache_key);
 		}
 
@@ -1171,7 +1171,7 @@ 
 
 		// Return result
 		$output = new BaseObject();
-		if($point > 0)
+		if ($point > 0)
 		{
 			$output->setMessage('success_voted');
 			$output->add('voted_count', $obj->after_point);
@@ -1193,13 +1193,13 @@ 
 	function declaredDocument($document_srl)
 	{
 		// Fail if session information already has a reported document
-		if($_SESSION['declared_document'][$document_srl]) return new BaseObject(-1, 'failed_declared');
+		if ($_SESSION['declared_document'][$document_srl]) return new BaseObject(-1, 'failed_declared');
 
 		// Check if previously reported
 		$args = new stdClass();
 		$args->document_srl = $document_srl;
 		$output = executeQuery('document.getDeclaredDocument', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$declared_count = ($output->data->declared_count) ? $output->data->declared_count : 0;
 
@@ -1209,7 +1209,7 @@ 
 
 		// Call a trigger (before)
 		$trigger_output = ModuleHandler::triggerCall('document.declaredDocument', 'before', $trigger_obj);
-		if(!$trigger_output->toBool())
+		if (!$trigger_output->toBool())
 		{
 			return $trigger_output;
 		}
@@ -1219,19 +1219,19 @@ 
 		$oDocument = $oDocumentModel->getDocument($document_srl, false, false);
 
 		// Pass if the author's IP address is as same as visitor's.
-		if($oDocument->get('ipaddress') == $_SERVER['REMOTE_ADDR']) {
+		if ($oDocument->get('ipaddress') == $_SERVER['REMOTE_ADDR']) {
 			$_SESSION['declared_document'][$document_srl] = true;
 			return new BaseObject(-1, 'failed_declared');
 		}
 
 		// Check if document's author is a member.
-		if($oDocument->get('member_srl'))
+		if ($oDocument->get('member_srl'))
 		{
 			// Create a member model object
 			$oMemberModel = getModel('member');
 			$member_srl = $oMemberModel->getLoggedMemberSrl();
 			// Pass after registering a session if author's information is same as the currently logged-in user's.
-			if($member_srl && $member_srl == abs($oDocument->get('member_srl')))
+			if ($member_srl && $member_srl == abs($oDocument->get('member_srl')))
 			{
 				$_SESSION['declared_document'][$document_srl] = true;
 				return new BaseObject(-1, 'failed_declared');
@@ -1240,7 +1240,7 @@ 
 
 		// Use member_srl for logged-in members and IP address for non-members.
 		$args = new stdClass;
-		if($member_srl)
+		if ($member_srl)
 		{
 			$args->member_srl = $member_srl;
 		}
@@ -1253,7 +1253,7 @@ 
 		$output = executeQuery('document.getDocumentDeclaredLogInfo', $args);
 
 		// Pass after registering a sesson if reported/declared documents are in the logs.
-		if($output->data->count)
+		if ($output->data->count)
 		{
 			$_SESSION['declared_document'][$document_srl] = true;
 			return new BaseObject(-1, 'failed_declared');
@@ -1264,23 +1264,23 @@ 
 		$oDB->begin();
 
 		// Add the declared document
-		if($declared_count > 0) $output = executeQuery('document.updateDeclaredDocument', $args);
+		if ($declared_count > 0) $output = executeQuery('document.updateDeclaredDocument', $args);
 		else $output = executeQuery('document.insertDeclaredDocument', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 		// Leave logs
 		$output = executeQuery('document.insertDocumentDeclaredLog', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 
-		$this->add('declared_count', $declared_count+1);
+		$this->add('declared_count', $declared_count + 1);
 
 		// Call a trigger (after)
 		$trigger_obj->declared_count = $declared_count + 1;
 		$trigger_output = ModuleHandler::triggerCall('document.declaredDocument', 'after', $trigger_obj);
-		if(!$trigger_output->toBool())
+		if (!$trigger_output->toBool())
 		{
 			$oDB->rollback();
 			return $trigger_output;
@@ -1310,16 +1310,16 @@ 
 		$args->document_srl = $document_srl;
 		$args->comment_count = $comment_count;
 
-		if($comment_inserted)
+		if ($comment_inserted)
 		{
-			$args->update_order = -1*getNextSequence();
+			$args->update_order = -1 * getNextSequence();
 			$args->last_updater = $last_updater;
 
 			$oCacheHandler = CacheHandler::getInstance('object');
-			if($oCacheHandler->isSupport())
+			if ($oCacheHandler->isSupport())
 			{
 				//remove document item from cache
-				$cache_key = 'document_item:'. getNumberingPath($document_srl) . $document_srl;
+				$cache_key = 'document_item:'.getNumberingPath($document_srl).$document_srl;
 				$oCacheHandler->delete($cache_key);
 			}
 		}
@@ -1340,10 +1340,10 @@ 
 		$args->trackback_count = $trackback_count;
 
 		$oCacheHandler = CacheHandler::getInstance('object');
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			//remove document item from cache
-			$cache_key = 'document_item:'. getNumberingPath($document_srl) . $document_srl;
+			$cache_key = 'document_item:'.getNumberingPath($document_srl).$document_srl;
 			$oCacheHandler->delete($cache_key);
 		}
 
@@ -1358,14 +1358,14 @@ 
 	function insertCategory($obj)
 	{
 		// Sort the order to display if a child category is added
-		if($obj->parent_srl)
+		if ($obj->parent_srl)
 		{
 			// Get its parent category
 			$oDocumentModel = getModel('document');
 			$parent_category = $oDocumentModel->getCategory($obj->parent_srl);
 			$obj->list_order = $parent_category->list_order;
-			$this->updateCategoryListOrder($parent_category->module_srl, $parent_category->list_order+1);
-			if(!$obj->category_srl) $obj->category_srl = getNextSequence();
+			$this->updateCategoryListOrder($parent_category->module_srl, $parent_category->list_order + 1);
+			if (!$obj->category_srl) $obj->category_srl = getNextSequence();
 		}
 		else
 		{
@@ -1373,7 +1373,7 @@ 
 		}
 
 		$output = executeQuery('document.insertCategory', $obj);
-		if($output->toBool())
+		if ($output->toBool())
 		{
 			$output->add('category_srl', $obj->category_srl);
 			$this->makeCategoryFile($obj->module_srl);
@@ -1407,13 +1407,13 @@ 
 	{
 		// Create a document model object
 		$oDocumentModel = getModel('document');
-		if(!$document_count) $document_count = $oDocumentModel->getCategoryDocumentCount($module_srl,$category_srl);
+		if (!$document_count) $document_count = $oDocumentModel->getCategoryDocumentCount($module_srl, $category_srl);
 
 		$args = new stdClass;
 		$args->category_srl = $category_srl;
 		$args->document_count = $document_count;
 		$output = executeQuery('document.updateCategoryCount', $args);
-		if($output->toBool()) $this->makeCategoryFile($module_srl);
+		if ($output->toBool()) $this->makeCategoryFile($module_srl);
 
 		return $output;
 	}
@@ -1426,7 +1426,7 @@ 
 	function updateCategory($obj)
 	{
 		$output = executeQuery('document.updateCategory', $obj);
-		if($output->toBool()) $this->makeCategoryFile($obj->module_srl);
+		if ($output->toBool()) $this->makeCategoryFile($obj->module_srl);
 		return $output;
 	}
 
@@ -1443,32 +1443,32 @@ 
 		$category_info = $oDocumentModel->getCategory($category_srl);
 		// Display an error that the category cannot be deleted if it has a child
 		$output = executeQuery('document.getChildCategoryCount', $args);
-		if(!$output->toBool()) return $output;
-		if($output->data->count>0) return new BaseObject(-1, 'msg_cannot_delete_for_child');
+		if (!$output->toBool()) return $output;
+		if ($output->data->count > 0) return new BaseObject(-1, 'msg_cannot_delete_for_child');
 		// Delete a category information
 		$output = executeQuery('document.deleteCategory', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$this->makeCategoryFile($category_info->module_srl);
 		// remvove cache
 		$oCacheHandler = CacheHandler::getInstance('object');
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$page = 0;
-			while(true) {
+			while (true) {
 				$args = new stdClass();
 				$args->category_srl = $category_srl;
 				$args->list_count = 100;
 				$args->page = ++$page;
 				$output = executeQuery('document.getDocumentList', $args, array('document_srl'));
 
-				if($output->data == array())
+				if ($output->data == array())
 					break;
 
-				foreach($output->data as $val)
+				foreach ($output->data as $val)
 				{
 					//remove document item from cache
-					$cache_key = 'document_item:'. getNumberingPath($val->document_srl) . $val->document_srl;
+					$cache_key = 'document_item:'.getNumberingPath($val->document_srl).$val->document_srl;
 					$oCacheHandler->delete($cache_key);
 				}
 			}
@@ -1515,18 +1515,18 @@ 
 		// Seek a full list of categories
 		$category_list = $oDocumentModel->getCategoryList($module_srl);
 		$category_srl_list = array_keys($category_list);
-		if(count($category_srl_list)<2) return new BaseObject();
+		if (count($category_srl_list) < 2) return new BaseObject();
 
 		$prev_category = NULL;
-		foreach($category_list as $key => $val)
+		foreach ($category_list as $key => $val)
 		{
-			if($key==$category_srl) break;
+			if ($key == $category_srl) break;
 			$prev_category = $val;
 		}
 		// Return if the previous category doesn't exist
-		if(!$prev_category) return new BaseObject(-1,Context::getLang('msg_category_not_moved'));
+		if (!$prev_category) return new BaseObject(-1, Context::getLang('msg_category_not_moved'));
 		// Return if the selected category is the top level
-		if($category_srl_list[0]==$category_srl) return new BaseObject(-1,Context::getLang('msg_category_not_moved'));
+		if ($category_srl_list[0] == $category_srl) return new BaseObject(-1, Context::getLang('msg_category_not_moved'));
 		// Information of the selected category
 		$cur_args = new stdClass;
 		$cur_args->category_srl = $category_srl;
@@ -1562,15 +1562,15 @@ 
 		// Seek a full list of categories
 		$category_list = $oDocumentModel->getCategoryList($module_srl);
 		$category_srl_list = array_keys($category_list);
-		if(count($category_srl_list)<2) return new BaseObject();
+		if (count($category_srl_list) < 2) return new BaseObject();
 
-		for($i=0;$i<count($category_srl_list);$i++)
+		for ($i = 0; $i < count($category_srl_list); $i++)
 		{
-			if($category_srl_list[$i]==$category_srl) break;
+			if ($category_srl_list[$i] == $category_srl) break;
 		}
 
-		$next_category_srl = $category_srl_list[$i+1];
-		if(!$category_list[$next_category_srl]) return new BaseObject(-1,Context::getLang('msg_category_not_moved'));
+		$next_category_srl = $category_srl_list[$i + 1];
+		if (!$category_list[$next_category_srl]) return new BaseObject(-1, Context::getLang('msg_category_not_moved'));
 		$next_category = $category_list[$next_category_srl];
 		// Information of the selected category
 		$cur_args = new stdClass;
@@ -1597,7 +1597,7 @@ 
 	{
 		$oDocumentModel = getModel('document');
 		$extra_keys = $oDocumentModel->getExtraKeys($module_srl);
-		if(!count($extra_keys)) return;
+		if (!count($extra_keys)) return;
 
 		$js_code = array();
 		$js_code[] = '<script>//<![CDATA[';
@@ -1607,16 +1607,16 @@ 
 
 		$logged_info = Context::get('logged_info');
 
-		foreach($extra_keys as $idx => $val)
+		foreach ($extra_keys as $idx => $val)
 		{
 			$idx = $val->idx;
-			if($val->type == 'kr_zip')
+			if ($val->type == 'kr_zip')
 			{
 				$idx .= '[]';
 			}
 			$name = str_ireplace(array('<script', '</script'), array('<scr" + "ipt', '</scr" + "ipt'), $val->name);
 			$js_code[] = sprintf('validator.cast("ADD_MESSAGE", ["extra_vars%s","%s"]);', $idx, $name);
-			if($val->is_required == 'Y') $js_code[] = sprintf('validator.cast("ADD_EXTRA_FIELD", ["extra_vars%s", { required:true }]);', $idx);
+			if ($val->is_required == 'Y') $js_code[] = sprintf('validator.cast("ADD_EXTRA_FIELD", ["extra_vars%s", { required:true }]);', $idx);
 		}
 
 		$js_code[] = '})(jQuery);';
@@ -1634,12 +1634,12 @@ 
 	function procDocumentInsertCategory($args = null)
 	{
 		// List variables
-		if(!$args) $args = Context::gets('module_srl','category_srl','parent_srl','category_title','category_description','expand','group_srls','category_color','mid');
+		if (!$args) $args = Context::gets('module_srl', 'category_srl', 'parent_srl', 'category_title', 'category_description', 'expand', 'group_srls', 'category_color', 'mid');
 		$args->title = $args->category_title;
 		$args->description = $args->category_description;
 		$args->color = $args->category_color;
 
-		if(!$args->module_srl && $args->mid)
+		if (!$args->module_srl && $args->mid)
 		{
 			$mid = $args->mid;
 			unset($args->mid);
@@ -1650,28 +1650,28 @@ 
 		$columnList = array('module_srl', 'module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($args->module_srl, $columnList);
 		$grant = $oModuleModel->getGrant($module_info, Context::get('logged_info'));
-		if(!$grant->manager) return new BaseObject(-1,'msg_not_permitted');
+		if (!$grant->manager) return new BaseObject(-1, 'msg_not_permitted');
 
-		if($args->expand !="Y") $args->expand = "N";
-		if(!is_array($args->group_srls)) $args->group_srls = str_replace('|@|',',',$args->group_srls);
+		if ($args->expand != "Y") $args->expand = "N";
+		if (!is_array($args->group_srls)) $args->group_srls = str_replace('|@|', ',', $args->group_srls);
 		else $args->group_srls = implode(',', $args->group_srls);
-		$args->parent_srl = (int)$args->parent_srl;
+		$args->parent_srl = (int) $args->parent_srl;
 
 		$oDocumentModel = getModel('document');
 
 		$oDB = &DB::getInstance();
 		$oDB->begin();
 		// Check if already exists
-		if($args->category_srl)
+		if ($args->category_srl)
 		{
 			$category_info = $oDocumentModel->getCategory($args->category_srl);
-			if($category_info->category_srl != $args->category_srl) $args->category_srl = null;
+			if ($category_info->category_srl != $args->category_srl) $args->category_srl = null;
 		}
 		// Update if exists
-		if($args->category_srl)
+		if ($args->category_srl)
 		{
 			$output = $this->updateCategory($args);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				$oDB->rollback();
 				return $output;
@@ -1681,7 +1681,7 @@ 
 		else
 		{
 			$output = $this->insertCategory($args);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				$oDB->rollback();
 				return $output;
@@ -1720,11 +1720,11 @@ 
 		$columnList = array('module_srl', 'module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($source_category->module_srl, $columnList);
 		$grant = $oModuleModel->getGrant($module_info, Context::get('logged_info'));
-		if(!$grant->manager) return new BaseObject(-1,'msg_not_permitted');
+		if (!$grant->manager) return new BaseObject(-1, 'msg_not_permitted');
 
 		// First child of the parent_category_srl
 		$source_args = new stdClass;
-		if($parent_category_srl > 0 || ($parent_category_srl == 0 && $target_category_srl == 0))
+		if ($parent_category_srl > 0 || ($parent_category_srl == 0 && $target_category_srl == 0))
 		{
 			$parent_category = $oDocumentModel->getCategory($parent_category_srl);
 
@@ -1733,30 +1733,30 @@ 
 			$args->parent_srl = $parent_category_srl;
 			$output = executeQuery('document.getChildCategoryMinListOrder', $args);
 
-			if(!$output->toBool()) return $output;
-			$args->list_order = (int)$output->data->list_order;
-			if(!$args->list_order) $args->list_order = 0;
+			if (!$output->toBool()) return $output;
+			$args->list_order = (int) $output->data->list_order;
+			if (!$args->list_order) $args->list_order = 0;
 			$args->list_order--;
 
 			$source_args->category_srl = $source_category_srl;
 			$source_args->parent_srl = $parent_category_srl;
 			$source_args->list_order = $args->list_order;
 			$output = $this->updateCategory($source_args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 			// Sibling of the $target_category_srl
 		}
-		else if($target_category_srl > 0)
+		else if ($target_category_srl > 0)
 		{
 			$target_category = $oDocumentModel->getCategory($target_category_srl);
 			// Move all siblings of the $target_category down
-			$output = $this->updateCategoryListOrder($target_category->module_srl, $target_category->list_order+1);
-			if(!$output->toBool()) return $output;
+			$output = $this->updateCategoryListOrder($target_category->module_srl, $target_category->list_order + 1);
+			if (!$output->toBool()) return $output;
 
 			$source_args->category_srl = $source_category_srl;
 			$source_args->parent_srl = $target_category->parent_srl;
-			$source_args->list_order = $target_category->list_order+1;
+			$source_args->list_order = $target_category->list_order + 1;
 			$output = $this->updateCategory($source_args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 		}
 		// Re-generate the xml file
 		$xml_file = $this->makeCategoryFile($source_category->module_srl);
@@ -1772,7 +1772,7 @@ 
 	function procDocumentDeleteCategory()
 	{
 		// List variables
-		$args = Context::gets('module_srl','category_srl');
+		$args = Context::gets('module_srl', 'category_srl');
 
 		$oDB = &DB::getInstance();
 		$oDB->begin();
@@ -1781,17 +1781,17 @@ 
 		$columnList = array('module_srl', 'module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($args->module_srl, $columnList);
 		$grant = $oModuleModel->getGrant($module_info, Context::get('logged_info'));
-		if(!$grant->manager) return new BaseObject(-1,'msg_not_permitted');
+		if (!$grant->manager) return new BaseObject(-1, 'msg_not_permitted');
 
 		$oDocumentModel = getModel('document');
 		// Get original information
 		$category_info = $oDocumentModel->getCategory($args->category_srl);
-		if($category_info->parent_srl) $parent_srl = $category_info->parent_srl;
+		if ($category_info->parent_srl) $parent_srl = $category_info->parent_srl;
 		// Display an error that the category cannot be deleted if it has a child node
-		if($oDocumentModel->getCategoryChlidCount($args->category_srl)) return new BaseObject(-1, 'msg_cannot_delete_for_child');
+		if ($oDocumentModel->getCategoryChlidCount($args->category_srl)) return new BaseObject(-1, 'msg_cannot_delete_for_child');
 		// Remove from the DB
 		$output = $this->deleteCategory($args->category_srl);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
@@ -1822,11 +1822,11 @@ 
 		$columnList = array('module_srl', 'module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl, $columnList);
 		$grant = $oModuleModel->getGrant($module_info, Context::get('logged_info'));
-		if(!$grant->manager) return new BaseObject(-1,'msg_not_permitted');
+		if (!$grant->manager) return new BaseObject(-1, 'msg_not_permitted');
 
 		$xml_file = $this->makeCategoryFile($module_srl);
 		// Set return value
-		$this->add('xml_file',$xml_file);
+		$this->add('xml_file', $xml_file);
 	}
 
 	/**
@@ -1837,14 +1837,14 @@ 
 	function makeCategoryFile($module_srl)
 	{
 		// Return if there is no information you need for creating a cache file
-		if(!$module_srl) return false;
+		if (!$module_srl) return false;
 		// Get module information (to obtain mid)
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'mid', 'site_srl');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl, $columnList);
 		$mid = $module_info->mid;
 
-		if(!is_dir('./files/cache/document_category')) FileHandler::makeDir('./files/cache/document_category');
+		if (!is_dir('./files/cache/document_category')) FileHandler::makeDir('./files/cache/document_category');
 		// Cache file's name
 		$xml_file = sprintf("./files/cache/document_category/%s.xml.php", $module_srl);
 		$php_file = sprintf("./files/cache/document_category/%s.php", $module_srl);
@@ -1856,17 +1856,17 @@ 
 
 		$category_list = $output->data;
 
-		if(!is_array($category_list)) $category_list = array($category_list);
+		if (!is_array($category_list)) $category_list = array($category_list);
 
 		$category_count = count($category_list);
-		for($i=0;$i<$category_count;$i++)
+		for ($i = 0; $i < $category_count; $i++)
 		{
 			$category_srl = $category_list[$i]->category_srl;
-			if(!preg_match('/^[0-9,]+$/', $category_list[$i]->group_srls)) $category_list[$i]->group_srls = '';
+			if (!preg_match('/^[0-9,]+$/', $category_list[$i]->group_srls)) $category_list[$i]->group_srls = '';
 			$list[$category_srl] = $category_list[$i];
 		}
 		// Create the xml file without node data if no data is obtained
-		if(!$list)
+		if (!$list)
 		{
 			$xml_buff = "<root />";
 			FileHandler::writeFile($xml_file, $xml_buff);
@@ -1874,12 +1874,12 @@ 
 			return $xml_file;
 		}
 		// Change to an array if only a single data is obtained
-		if(!is_array($list)) $list = array($list);
+		if (!is_array($list)) $list = array($list);
 		// Create a tree for loop
-		foreach($list as $category_srl => $node)
+		foreach ($list as $category_srl => $node)
 		{
 			$node->mid = $mid;
-			$parent_srl = (int)$node->parent_srl;
+			$parent_srl = (int) $node->parent_srl;
 			$tree[$parent_srl][$category_srl] = $node;
 		}
 		// A common header to set permissions and groups of the cache file
@@ -1953,13 +1953,13 @@ 
 	 */
 	function getXmlTree($source_node, $tree, $site_srl, &$xml_header_buff)
 	{
-		if(!$source_node) return;
+		if (!$source_node) return;
 
-		foreach($source_node as $category_srl => $node)
+		foreach ($source_node as $category_srl => $node)
 		{
 			$child_buff = "";
 			// Get data of the child nodes
-			if($category_srl && $tree[$category_srl]) $child_buff = $this->getXmlTree($tree[$category_srl], $tree, $site_srl, $xml_header_buff);
+			if ($category_srl && $tree[$category_srl]) $child_buff = $this->getXmlTree($tree[$category_srl], $tree, $site_srl, $xml_header_buff);
 			// List variables
 			$expand = ($node->expand) ? $node->expand : 'N';
 			$group_srls = ($node->group_srls) ? $node->group_srls : '';
@@ -1969,27 +1969,27 @@ 
 			$color = ($node->color) ? $node->color : '';
 			$description = ($node->description) ? $node->description : '';
 			// If node->group_srls value exists
-			if($group_srls) $group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s)))))',$group_srls);
+			if ($group_srls) $group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s)))))', $group_srls);
 			else $group_check_code = "true";
 
 			$title = $node->title;
 			$oModuleAdminModel = getAdminModel('module');
 
 			$langs = $oModuleAdminModel->getLangCode($site_srl, $title);
-			if(count($langs))
+			if (count($langs))
 			{
-				foreach($langs as $key => $val)
+				foreach ($langs as $key => $val)
 				{
-					$xml_header_buff .= sprintf('$_titles[%d]["%s"] = %s; ', $category_srl, $key, var_export(str_replace('"','\\"',htmlspecialchars($val, ENT_COMPAT | ENT_HTML401, 'UTF-8', false)), true));
+					$xml_header_buff .= sprintf('$_titles[%d]["%s"] = %s; ', $category_srl, $key, var_export(str_replace('"', '\\"', htmlspecialchars($val, ENT_COMPAT | ENT_HTML401, 'UTF-8', false)), true));
 				}
 			}
 
 			$langx = $oModuleAdminModel->getLangCode($site_srl, $description);
-			if(count($langx))
+			if (count($langx))
 			{
-				foreach($langx as $key => $val)
+				foreach ($langx as $key => $val)
 				{
-					$xml_header_buff .= sprintf('$_descriptions[%d]["%s"] = %s; ', $category_srl, $key, var_export(str_replace('"','\\"',htmlspecialchars($val, ENT_COMPAT | ENT_HTML401, 'UTF-8', false)), true));
+					$xml_header_buff .= sprintf('$_descriptions[%d]["%s"] = %s; ', $category_srl, $key, var_export(str_replace('"', '\\"', htmlspecialchars($val, ENT_COMPAT | ENT_HTML401, 'UTF-8', false)), true));
 				}
 			}
 
@@ -2002,7 +2002,7 @@ 
 				$category_srl,
 				$group_check_code,
 				$category_srl,
-				var_export(getUrl('','mid',$node->mid,'category',$category_srl), true),
+				var_export(getUrl('', 'mid', $node->mid, 'category', $category_srl), true),
 				var_export($expand, true),
 				var_export($color, true),
 				$group_check_code,
@@ -2010,8 +2010,8 @@ 
 				$node->document_count
 			);
 
-			if($child_buff) $buff .= sprintf('<node %s>%s</node>', $attribute, $child_buff);
-			else $buff .=  sprintf('<node %s />', $attribute);
+			if ($child_buff) $buff .= sprintf('<node %s>%s</node>', $attribute, $child_buff);
+			else $buff .= sprintf('<node %s />', $attribute);
 		}
 		return $buff;
 	}
@@ -2030,13 +2030,13 @@ 
 	function getPhpCacheCode($source_node, $tree, $site_srl, &$php_header_buff)
 	{
 		$output = array("buff"=>"", "category_srl_list"=>array());
-		if(!$source_node) return $output;
+		if (!$source_node) return $output;
 
 		// Set to an arraty for looping and then generate php script codes to be included
-		foreach($source_node as $category_srl => $node)
+		foreach ($source_node as $category_srl => $node)
 		{
 			// Get data from child nodes first if exist.
-			if($category_srl && $tree[$category_srl]){
+			if ($category_srl && $tree[$category_srl]) {
 				$child_output = $this->getPhpCacheCode($tree[$category_srl], $tree, $site_srl, $php_header_buff);
 			} else {
 				$child_output = array("buff"=>"", "category_srl_list"=>array());
@@ -2047,14 +2047,14 @@ 
 			$output['category_srl_list'] = array_merge($output['category_srl_list'], $child_output['category_srl_list']);
 
 			// If node->group_srls value exists
-			if($node->group_srls) {
-				$group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s)))))',$node->group_srls);
+			if ($node->group_srls) {
+				$group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s)))))', $node->group_srls);
 			} else {
 				$group_check_code = "true";
 			}
 
 			// List variables
-			$selected = '"' . implode('","', $child_output['category_srl_list']) . '"';
+			$selected = '"'.implode('","', $child_output['category_srl_list']).'"';
 			$child_buff = $child_output['buff'];
 			$expand = $node->expand;
 
@@ -2063,32 +2063,32 @@ 
 			$oModuleAdminModel = getAdminModel('module');
 			$langs = $oModuleAdminModel->getLangCode($site_srl, $title);
 
-			if(count($langs))
+			if (count($langs))
 			{
-				foreach($langs as $key => $val)
+				foreach ($langs as $key => $val)
 				{
 					$val = htmlspecialchars($val, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 					$php_header_buff .= sprintf(
 						'$_titles[%d]["%s"] = %s; ',
 						$category_srl,
 						$key,
-						var_export(str_replace('"','\\"', $val), true)
+						var_export(str_replace('"', '\\"', $val), true)
 					);
 				}
 			}
 
 			$langx = $oModuleAdminModel->getLangCode($site_srl, $description);
 
-			if(count($langx))
+			if (count($langx))
 			{
-				foreach($langx as $key => $val)
+				foreach ($langx as $key => $val)
 				{
 					$val = htmlspecialchars($val, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 					$php_header_buff .= sprintf(
 						'$_descriptions[%d]["%s"] = %s; ',
 						$category_srl,
 						$key,
-						var_export(str_replace('"','\\"', $val), true)
+						var_export(str_replace('"', '\\"', $val), true)
 					);
 				}
 			}
@@ -2112,7 +2112,7 @@ 
 			);
 
 			// Generate buff data
-			$output['buff'] .=  sprintf('%s=>array(%s),', $node->category_srl, $attribute);
+			$output['buff'] .= sprintf('%s=>array(%s),', $node->category_srl, $attribute);
 		}
 
 		return $output;
@@ -2129,7 +2129,7 @@ 
 	function addDocumentPopupMenu($url, $str, $icon = '', $target = 'self')
 	{
 		$document_popup_menu_list = Context::get('document_popup_menu_list');
-		if(!is_array($document_popup_menu_list)) $document_popup_menu_list = array();
+		if (!is_array($document_popup_menu_list)) $document_popup_menu_list = array();
 
 		$obj = new stdClass();
 		$obj->url = $url;
@@ -2147,68 +2147,68 @@ 
 	 */
 	function procDocumentAddCart()
 	{
-		if(!Context::get('is_logged')) return new BaseObject(-1, 'msg_not_permitted');
+		if (!Context::get('is_logged')) return new BaseObject(-1, 'msg_not_permitted');
 
 		// Get document_srl
-		$srls = explode(',',Context::get('srls'));
-		for($i = 0; $i < count($srls); $i++)
+		$srls = explode(',', Context::get('srls'));
+		for ($i = 0; $i < count($srls); $i++)
 		{
 			$srl = trim($srls[$i]);
 
-			if(!$srl) continue;
+			if (!$srl) continue;
 
 			$document_srls[] = $srl;
 		}
-		if(!count($document_srls)) return;
+		if (!count($document_srls)) return;
 
 		// Get module_srl of the documents
 		$args = new stdClass;
 		$args->list_count = count($document_srls);
-		$args->document_srls = implode(',',$document_srls);
+		$args->document_srls = implode(',', $document_srls);
 		$args->order_type = 'asc';
 		$output = executeQueryArray('document.getDocuments', $args);
-		if(!$output->data) return new BaseObject();
+		if (!$output->data) return new BaseObject();
 
 		unset($document_srls);
-		foreach($output->data as $key => $val)
+		foreach ($output->data as $key => $val)
 		{
 			$document_srls[$val->module_srl][] = $val->document_srl;
 		}
-		if(!$document_srls || !count($document_srls)) return new BaseObject();
+		if (!$document_srls || !count($document_srls)) return new BaseObject();
 
 		// Check if each of module administrators exists. Top-level administator will have a permission to modify every document of all modules.(Even to modify temporarily saved or trashed documents)
 		$oModuleModel = getModel('module');
 		$module_srls = array_keys($document_srls);
-		for($i=0;$i<count($module_srls);$i++)
+		for ($i = 0; $i < count($module_srls); $i++)
 		{
 			$module_srl = $module_srls[$i];
 			$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl);
 			$logged_info = Context::get('logged_info');
-			if($logged_info->is_admin != 'Y')
+			if ($logged_info->is_admin != 'Y')
 			{
-				if(!$module_info)
+				if (!$module_info)
 				{
 					unset($document_srls[$module_srl]);
 					continue;
 				}
 				$grant = $oModuleModel->getGrant($module_info, $logged_info);
-				if(!$grant->manager)
+				if (!$grant->manager)
 				{
 					unset($document_srls[$module_srl]);
 					continue;
 				}
 			}
 		}
-		if(!count($document_srls)) return new BaseObject();
+		if (!count($document_srls)) return new BaseObject();
 
-		foreach($document_srls as $module_srl => $documents)
+		foreach ($document_srls as $module_srl => $documents)
 		{
 			$cnt = count($documents);
-			for($i=0;$i<$cnt;$i++)
+			for ($i = 0; $i < $cnt; $i++)
 			{
-				$document_srl = (int)trim($documents[$i]);
-				if(!$document_srls) continue;
-				if($_SESSION['document_management'][$document_srl]) unset($_SESSION['document_management'][$document_srl]);
+				$document_srl = (int) trim($documents[$i]);
+				if (!$document_srls) continue;
+				if ($_SESSION['document_management'][$document_srl]) unset($_SESSION['document_management'][$document_srl]);
 				else $_SESSION['document_management'][$document_srl] = true;
 			}
 		}
@@ -2221,9 +2221,9 @@ 
 	function procDocumentManageCheckedDocument()
 	{
 		@set_time_limit(0);
-		if(!Context::get('is_logged')) return new BaseObject(-1,'msg_not_permitted');
+		if (!Context::get('is_logged')) return new BaseObject(-1, 'msg_not_permitted');
 
-		if(!checkCSRF())
+		if (!checkCSRF())
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
@@ -2231,44 +2231,44 @@ 
 		$type = Context::get('type');
 		$target_module = Context::get('target_module');
 		$module_srl = Context::get('module_srl');
-		if($target_module && !$module_srl) $module_srl = $target_module;
+		if ($target_module && !$module_srl) $module_srl = $target_module;
 		$category_srl = Context::get('target_category');
 		$message_content = Context::get('message_content');
-		if($message_content) $message_content = nl2br($message_content);
+		if ($message_content) $message_content = nl2br($message_content);
 
 		$cart = Context::get('cart');
-		if(!is_array($cart)) $document_srl_list = explode('|@|', $cart);
+		if (!is_array($cart)) $document_srl_list = explode('|@|', $cart);
 		else $document_srl_list = $cart;
 
 		$document_srl_count = count($document_srl_list);
 
 		$oDocumentModel = getModel('document');
 		$document_items = array();
-		foreach($document_srl_list as $document_srl)
+		foreach ($document_srl_list as $document_srl)
 		{
 			$oDocument = $oDocumentModel->getDocument($document_srl);
 			$document_items[] = $oDocument;
-			if(!$oDocument->isGranted()) return $this->stop('msg_not_permitted');
+			if (!$oDocument->isGranted()) return $this->stop('msg_not_permitted');
 		}
 
 		// Send a message
-		if($message_content)
+		if ($message_content)
 		{
 
 			$oCommunicationController = getController('communication');
 
 			$logged_info = Context::get('logged_info');
 
-			$title = cut_str($message_content,10,'...');
+			$title = cut_str($message_content, 10, '...');
 			$sender_member_srl = $logged_info->member_srl;
 
-			foreach($document_items as $oDocument)
+			foreach ($document_items as $oDocument)
 			{
-				if(!$oDocument->get('member_srl') || $oDocument->get('member_srl')==$sender_member_srl) continue;
+				if (!$oDocument->get('member_srl') || $oDocument->get('member_srl') == $sender_member_srl) continue;
 
-				if($type=='move') $purl = sprintf("<a href=\"%s\" target=\"_blank\">%s</a>", $oDocument->getPermanentUrl(), $oDocument->getPermanentUrl());
+				if ($type == 'move') $purl = sprintf("<a href=\"%s\" target=\"_blank\">%s</a>", $oDocument->getPermanentUrl(), $oDocument->getPermanentUrl());
 				else $purl = "";
-				$content = sprintf("<div>%s</div><hr />%s<div style=\"font-weight:bold\">%s</div>%s",$message_content, $purl, $oDocument->getTitleText(), $oDocument->getContent(false, false, false));
+				$content = sprintf("<div>%s</div><hr />%s<div style=\"font-weight:bold\">%s</div>%s", $message_content, $purl, $oDocument->getTitleText(), $oDocument->getContent(false, false, false));
 
 				$oCommunicationController->sendMessage($sender_member_srl, $oDocument->get('member_srl'), $title, $content, false);
 			}
@@ -2278,54 +2278,54 @@ 
 		$oSpamController->setAvoidLog();
 
 		$oDocumentAdminController = getAdminController('document');
-		if($type == 'move')
+		if ($type == 'move')
 		{
-			if(!$module_srl) return new BaseObject(-1, 'fail_to_move');
+			if (!$module_srl) return new BaseObject(-1, 'fail_to_move');
 
 			$output = $oDocumentAdminController->moveDocumentModule($document_srl_list, $module_srl, $category_srl);
-			if(!$output->toBool()) return new BaseObject(-1, 'fail_to_move');
+			if (!$output->toBool()) return new BaseObject(-1, 'fail_to_move');
 
 			$msg_code = 'success_moved';
 
 		}
-		else if($type == 'copy')
+		else if ($type == 'copy')
 		{
-			if(!$module_srl) return new BaseObject(-1, 'fail_to_move');
+			if (!$module_srl) return new BaseObject(-1, 'fail_to_move');
 
 			$output = $oDocumentAdminController->copyDocumentModule($document_srl_list, $module_srl, $category_srl);
-			if(!$output->toBool()) return new BaseObject(-1, 'fail_to_move');
+			if (!$output->toBool()) return new BaseObject(-1, 'fail_to_move');
 
 			$msg_code = 'success_copied';
 		}
-		else if($type =='delete')
+		else if ($type == 'delete')
 		{
 			$oDB = &DB::getInstance();
 			$oDB->begin();
-			for($i=0;$i<$document_srl_count;$i++)
+			for ($i = 0; $i < $document_srl_count; $i++)
 			{
 				$document_srl = $document_srl_list[$i];
 				$output = $this->deleteDocument($document_srl, true);
-				if(!$output->toBool()) return new BaseObject(-1, 'fail_to_delete');
+				if (!$output->toBool()) return new BaseObject(-1, 'fail_to_delete');
 			}
 			$oDB->commit();
 			$msg_code = 'success_deleted';
 		}
-		else if($type == 'trash')
+		else if ($type == 'trash')
 		{
 			$args = new stdClass();
 			$args->description = $message_content;
 
 			$oDB = &DB::getInstance();
 			$oDB->begin();
-			for($i=0;$i<$document_srl_count;$i++) {
+			for ($i = 0; $i < $document_srl_count; $i++) {
 				$args->document_srl = $document_srl_list[$i];
 				$output = $this->moveDocumentToTrash($args);
-				if(!$output || !$output->toBool()) return new BaseObject(-1, 'fail_to_trash');
+				if (!$output || !$output->toBool()) return new BaseObject(-1, 'fail_to_trash');
 			}
 			$oDB->commit();
 			$msg_code = 'success_trashed';
 		}
-		else if($type == 'cancelDeclare')
+		else if ($type == 'cancelDeclare')
 		{
 			$args->document_srl = $document_srl_list;
 			$output = executeQuery('document.deleteDeclaredDocuments', $args);
@@ -2347,27 +2347,27 @@ 
 	function procDocumentInsertModuleConfig()
 	{
 		$module_srl = Context::get('target_module_srl');
-		if(preg_match('/^([0-9,]+)$/',$module_srl)) $module_srl = explode(',',$module_srl);
+		if (preg_match('/^([0-9,]+)$/', $module_srl)) $module_srl = explode(',', $module_srl);
 		else $module_srl = array($module_srl);
 
 		$document_config = new stdClass();
 		$document_config->use_history = Context::get('use_history');
-		if(!$document_config->use_history) $document_config->use_history = 'N';
+		if (!$document_config->use_history) $document_config->use_history = 'N';
 
 		$document_config->use_vote_up = Context::get('use_vote_up');
-		if(!$document_config->use_vote_up) $document_config->use_vote_up = 'Y';
+		if (!$document_config->use_vote_up) $document_config->use_vote_up = 'Y';
 
 		$document_config->use_vote_down = Context::get('use_vote_down');
-		if(!$document_config->use_vote_down) $document_config->use_vote_down = 'Y';
+		if (!$document_config->use_vote_down) $document_config->use_vote_down = 'Y';
 
 		$document_config->use_status = Context::get('use_status');
 
 		$oModuleController = getController('module');
-		for($i=0;$i<count($module_srl);$i++)
+		for ($i = 0; $i < count($module_srl); $i++)
 		{
 			$srl = trim($module_srl[$i]);
-			if(!$srl) continue;
-			$output = $oModuleController->insertModulePartConfig('document',$srl,$document_config);
+			if (!$srl) continue;
+			$output = $oModuleController->insertModulePartConfig('document', $srl, $document_config);
 		}
 		$this->setError(-1);
 		$this->setMessage('success_updated', 'info');
@@ -2383,7 +2383,7 @@ 
 	function procDocumentTempSave()
 	{
 		// Check login information
-		if(!Context::get('is_logged')) return new BaseObject(-1, 'msg_not_logged');
+		if (!Context::get('is_logged')) return new BaseObject(-1, 'msg_not_logged');
 		$module_info = Context::get('module_info');
 		$logged_info = Context::get('logged_info');
 
@@ -2395,7 +2395,7 @@ 
 		unset($obj->is_notice);
 
 		// Extract from beginning part of contents in the guestbook
-		if(!$obj->title)
+		if (!$obj->title)
 		{
 			$obj->title = cut_str(strip_tags($obj->content), 20, '...');
 		}
@@ -2406,13 +2406,13 @@ 
 		$oDocument = $oDocumentModel->getDocument($obj->document_srl, $this->grant->manager);
 
 		// Update if already exists
-		if($oDocument->isExists() && $oDocument->document_srl == $obj->document_srl)
+		if ($oDocument->isExists() && $oDocument->document_srl == $obj->document_srl)
 		{
-			if($oDocument->get('module_srl') != $obj->module_srl)
+			if ($oDocument->get('module_srl') != $obj->module_srl)
 			{
 				return new BaseObject(-1, 'msg_invalid_request');
 			}
-			if(!$oDocument->isGranted())
+			if (!$oDocument->isGranted())
 			{
 				return new BaseObject(-1, 'msg_invalid_request');
 			}
@@ -2430,7 +2430,7 @@ 
 			$oDocument = $oDocumentModel->getDocument($obj->document_srl, $this->grant->manager);
 		}
 		// Set the attachment to be invalid state
-		if($oDocument->hasUploadedFiles())
+		if ($oDocument->hasUploadedFiles())
 		{
 			$args = new stdClass;
 			$args->upload_target_srl = $oDocument->document_srl;
@@ -2448,11 +2448,11 @@ 
 	 */
 	function procDocumentGetList()
 	{
-		if(!Context::get('is_logged')) return new BaseObject(-1,'msg_not_permitted');
+		if (!Context::get('is_logged')) return new BaseObject(-1, 'msg_not_permitted');
 		$documentSrls = Context::get('document_srls');
-		if($documentSrls) $documentSrlList = explode(',', $documentSrls);
+		if ($documentSrls) $documentSrlList = explode(',', $documentSrls);
 
-		if(count($documentSrlList) > 0)
+		if (count($documentSrlList) > 0)
 		{
 			$oDocumentModel = getModel('document');
 			$columnList = array('document_srl', 'title', 'nick_name', 'status');
@@ -2476,10 +2476,10 @@ 
 	 */
 	function _checkCommentStatusForOldVersion(&$obj)
 	{
-		if(!isset($obj->allow_comment)) $obj->allow_comment = 'N';
-		if(!isset($obj->lock_comment)) $obj->lock_comment = 'N';
+		if (!isset($obj->allow_comment)) $obj->allow_comment = 'N';
+		if (!isset($obj->lock_comment)) $obj->lock_comment = 'N';
 
-		if($obj->allow_comment == 'Y' && $obj->lock_comment == 'N') $obj->commentStatus = 'ALLOW';
+		if ($obj->allow_comment == 'Y' && $obj->lock_comment == 'N') $obj->commentStatus = 'ALLOW';
 		else $obj->commentStatus = 'DENY';
 	}
 
@@ -2490,8 +2490,8 @@ 
 	 */
 	function _checkDocumentStatusForOldVersion(&$obj)
 	{
-		if(!$obj->status && $obj->is_secret == 'Y') $obj->status = $this->getConfigStatus('secret');
-		if(!$obj->status && $obj->is_secret != 'Y') $obj->status = $this->getConfigStatus('public');
+		if (!$obj->status && $obj->is_secret == 'Y') $obj->status = $this->getConfigStatus('secret');
+		if (!$obj->status && $obj->is_secret != 'Y') $obj->status = $this->getConfigStatus('public');
 	}
 
 	public function updateUploaedCount($documentSrlList)
@@ -2499,10 +2499,10 @@ 
 		$oDocumentModel = getModel('document');
 		$oFileModel = getModel('file');
 
-		if(is_array($documentSrlList))
+		if (is_array($documentSrlList))
 		{
 			$documentSrlList = array_unique($documentSrlList);
-			foreach($documentSrlList AS $key => $documentSrl)
+			foreach ($documentSrlList AS $key => $documentSrl)
 			{
 				$fileCount = $oFileModel->getFilesCount($documentSrl);
 				$args = new stdClass();
@@ -2523,14 +2523,14 @@ 
 		$oDocumentModel = getModel('document');
 		$documentExtraKeys = $oDocumentModel->getExtraKeys($obj->originModuleSrl);
 
-		if(is_array($documentExtraKeys) && is_array($obj->moduleSrlList))
+		if (is_array($documentExtraKeys) && is_array($obj->moduleSrlList))
 		{
-			$oDocumentController=getController('document');
-			foreach($obj->moduleSrlList AS $key=>$value)
+			$oDocumentController = getController('document');
+			foreach ($obj->moduleSrlList AS $key=>$value)
 			{
-				foreach($documentExtraKeys AS $extraItem)
+				foreach ($documentExtraKeys AS $extraItem)
 				{
-					$oDocumentController->insertDocumentExtraKey($value, $extraItem->idx, $extraItem->name, $extraItem->type, $extraItem->is_required , $extraItem->search , $extraItem->default , $extraItem->desc, $extraItem->eid) ;
+					$oDocumentController->insertDocumentExtraKey($value, $extraItem->idx, $extraItem->name, $extraItem->type, $extraItem->is_required, $extraItem->search, $extraItem->default, $extraItem->desc, $extraItem->eid);
 				}
 			}
 		}
@@ -2542,9 +2542,9 @@ 
 		$documentConfig = $oModuleModel->getModulePartConfig('document', $obj->originModuleSrl);
 
 		$oModuleController = getController('module');
-		if(is_array($obj->moduleSrlList))
+		if (is_array($obj->moduleSrlList))
 		{
-			foreach($obj->moduleSrlList AS $key=>$moduleSrl)
+			foreach ($obj->moduleSrlList AS $key=>$moduleSrl)
 			{
 				$oModuleController->insertModulePartConfig('document', $moduleSrl, $documentConfig);
 			}

Braces +535 added lines, -216 removed lines

@@ -24,19 +24,27 @@ 
 	 */
 	function procDocumentVoteUp()
 	{
-		if(!Context::get('is_logged')) return new BaseObject(-1, 'msg_invalid_request');
+		if(!Context::get('is_logged')) {
+			return new BaseObject(-1, 'msg_invalid_request');
+		}
 
 		$document_srl = Context::get('target_srl');
-		if(!$document_srl) return new BaseObject(-1, 'msg_invalid_request');
+		if(!$document_srl) {
+			return new BaseObject(-1, 'msg_invalid_request');
+		}
 
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument($document_srl, false, false);
 		$module_srl = $oDocument->get('module_srl');
-		if(!$module_srl) return new BaseObject(-1, 'msg_invalid_request');
+		if(!$module_srl) {
+			return new BaseObject(-1, 'msg_invalid_request');
+		}
 
 		$oModuleModel = getModel('module');
 		$document_config = $oModuleModel->getModulePartConfig('document',$module_srl);
-		if($document_config->use_vote_up=='N') return new BaseObject(-1, 'msg_invalid_request');
+		if($document_config->use_vote_up=='N') {
+			return new BaseObject(-1, 'msg_invalid_request');
+		}
 
 		$point = 1;
 		$output = $this->updateVotedCount($document_srl, $point);
@@ -69,19 +77,27 @@ 
 	 */
 	function procDocumentVoteDown()
 	{
-		if(!Context::get('is_logged')) return new BaseObject(-1, 'msg_invalid_request');
+		if(!Context::get('is_logged')) {
+			return new BaseObject(-1, 'msg_invalid_request');
+		}
 
 		$document_srl = Context::get('target_srl');
-		if(!$document_srl) return new BaseObject(-1, 'msg_invalid_request');
+		if(!$document_srl) {
+			return new BaseObject(-1, 'msg_invalid_request');
+		}
 
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument($document_srl, false, false);
 		$module_srl = $oDocument->get('module_srl');
-		if(!$module_srl) return new BaseObject(-1, 'msg_invalid_request');
+		if(!$module_srl) {
+			return new BaseObject(-1, 'msg_invalid_request');
+		}
 
 		$oModuleModel = getModel('module');
 		$document_config = $oModuleModel->getModulePartConfig('document',$module_srl);
-		if($document_config->use_vote_down=='N') return new BaseObject(-1, 'msg_invalid_request');
+		if($document_config->use_vote_down=='N') {
+			return new BaseObject(-1, 'msg_invalid_request');
+		}
 
 		$point = -1;
 		$output = $this->updateVotedCount($document_srl, $point);
@@ -95,10 +111,14 @@ 
 	 */
 	function procDocumentDeclare()
 	{
-		if(!Context::get('is_logged')) return new BaseObject(-1, 'msg_invalid_request');
+		if(!Context::get('is_logged')) {
+			return new BaseObject(-1, 'msg_invalid_request');
+		}
 
 		$document_srl = Context::get('target_srl');
-		if(!$document_srl) return new BaseObject(-1, 'msg_invalid_request');
+		if(!$document_srl) {
+			return new BaseObject(-1, 'msg_invalid_request');
+		}
 
 		return $this->declaredDocument($document_srl);
 	}
@@ -140,7 +160,9 @@ 
 		$args->history_srl = $history_srl;
 		$args->module_srl = $module_srl;
 		$args->document_srl = $document_srl;
-		if(!$args->history_srl && !$args->module_srl && !$args->document_srl) return;
+		if(!$args->history_srl && !$args->module_srl && !$args->document_srl) {
+			return;
+		}
 		executeQuery("document.deleteHistory", $args);
 	}
 
@@ -152,15 +174,21 @@ 
 	function triggerDeleteModuleDocuments(&$obj)
 	{
 		$module_srl = $obj->module_srl;
-		if(!$module_srl) return new BaseObject();
+		if(!$module_srl) {
+			return new BaseObject();
+		}
 		// Delete the document
 		$oDocumentAdminController = getAdminController('document');
 		$output = $oDocumentAdminController->deleteModuleDocument($module_srl);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 		// Delete the category
 		$oDocumentController = getController('document');
 		$output = $oDocumentController->deleteModuleCategory($module_srl);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 		// Delete extra key and variable, because module deleted
 		$this->deleteDocumentExtraKeys($module_srl);
 
@@ -202,10 +230,18 @@ 
 		$oDB = &DB::getInstance();
 		$oDB->begin();
 		// List variables
-		if($obj->comment_status) $obj->commentStatus = $obj->comment_status;
-		if(!$obj->commentStatus) $obj->commentStatus = 'DENY';
-		if($obj->commentStatus == 'DENY') $this->_checkCommentStatusForOldVersion($obj);
-		if($obj->allow_trackback!='Y') $obj->allow_trackback = 'N';
+		if($obj->comment_status) {
+			$obj->commentStatus = $obj->comment_status;
+		}
+		if(!$obj->commentStatus) {
+			$obj->commentStatus = 'DENY';
+		}
+		if($obj->commentStatus == 'DENY') {
+			$this->_checkCommentStatusForOldVersion($obj);
+		}
+		if($obj->allow_trackback!='Y') {
+			$obj->allow_trackback = 'N';
+		}
 		if($obj->homepage) 
 		{
 			$obj->homepage = removeHackTag($obj->homepage);
@@ -215,9 +251,15 @@ 
 			}
 		}
 		
-		if($obj->notify_message != 'Y') $obj->notify_message = 'N';
-		if(!$obj->email_address) $obj->email_address = '';
-		if(!$isRestore) $obj->ipaddress = $_SERVER['REMOTE_ADDR'];
+		if($obj->notify_message != 'Y') {
+			$obj->notify_message = 'N';
+		}
+		if(!$obj->email_address) {
+			$obj->email_address = '';
+		}
+		if(!$isRestore) {
+			$obj->ipaddress = $_SERVER['REMOTE_ADDR'];
+		}
 
                 // can modify regdate only manager
                 $grant = Context::get('grant');
@@ -227,7 +269,9 @@ 
 		}
 		
 		// Serialize the $extra_vars, check the extra_vars type, because duplicate serialized avoid
-		if(!is_string($obj->extra_vars)) $obj->extra_vars = serialize($obj->extra_vars);
+		if(!is_string($obj->extra_vars)) {
+			$obj->extra_vars = serialize($obj->extra_vars);
+		}
 		// Remove the columns for automatic saving
 		unset($obj->_saved_doc_srl);
 		unset($obj->_saved_doc_title);
@@ -235,10 +279,15 @@ 
 		unset($obj->_saved_doc_message);
 		// Call a trigger (before)
 		$output = ModuleHandler::triggerCall('document.insertDocument', 'before', $obj);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 		// Register it if no given document_srl exists
-		if(!$obj->document_srl) $obj->document_srl = getNextSequence();
-		elseif(!$manual_inserted && !$isRestore && !checkUserSequence($obj->document_srl)) return new BaseObject(-1, 'msg_not_permitted');
+		if(!$obj->document_srl) {
+			$obj->document_srl = getNextSequence();
+		} elseif(!$manual_inserted && !$isRestore && !checkUserSequence($obj->document_srl)) {
+			return new BaseObject(-1, 'msg_not_permitted');
+		}
 
 		$oDocumentModel = getModel('document');
 		// Set to 0 if the category_srl doesn't exist
@@ -249,12 +298,19 @@ 
 			{
 				return new BaseObject(-1, 'msg_not_permitted');
 			}
-			if(count($category_list) > 0 && !$category_list[$obj->category_srl]) $obj->category_srl = 0;
+			if(count($category_list) > 0 && !$category_list[$obj->category_srl]) {
+				$obj->category_srl = 0;
+			}
 		}
 		// Set the read counts and update order.
-		if(!$obj->readed_count) $obj->readed_count = 0;
-		if($isLatest) $obj->update_order = $obj->list_order = $obj->document_srl * -1;
-		else $obj->update_order = $obj->list_order;
+		if(!$obj->readed_count) {
+			$obj->readed_count = 0;
+		}
+		if($isLatest) {
+			$obj->update_order = $obj->list_order = $obj->document_srl * -1;
+		} else {
+			$obj->update_order = $obj->list_order;
+		}
 		// Check the status of password hash for manually inserting. Apply hashing for otherwise.
 		if($obj->password && !$obj->password_is_hashed)
 		{
@@ -276,9 +332,13 @@ 
 		// If the tile is empty, extract string from the contents.
 		$obj->title = htmlspecialchars($obj->title, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 		settype($obj->title, "string");
-		if($obj->title == '') $obj->title = cut_str(trim(strip_tags(nl2br($obj->content))),20,'...');
+		if($obj->title == '') {
+			$obj->title = cut_str(trim(strip_tags(nl2br($obj->content))),20,'...');
+		}
 		// If no tile extracted from the contents, leave it untitled.
-		if($obj->title == '') $obj->title = 'Untitled';
+		if($obj->title == '') {
+			$obj->title = 'Untitled';
+		}
 		// Remove XE's own tags from the contents.
 		$obj->content = preg_replace('!<\!--(Before|After)(Document|Comment)\(([0-9]+),([0-9]+)\)-->!is', '', $obj->content);
 		if(Mobile::isFromMobilePhone() && $obj->use_editor != 'Y')
@@ -290,13 +350,19 @@ 
 			$obj->content = nl2br($obj->content);
 		}
 		// Remove iframe and script if not a top adminisrator in the session.
-		if($logged_info->is_admin != 'Y') $obj->content = removeHackTag($obj->content);
+		if($logged_info->is_admin != 'Y') {
+			$obj->content = removeHackTag($obj->content);
+		}
 		// An error appears if both log-in info and user name don't exist.
-		if(!$logged_info->member_srl && !$obj->nick_name) return new BaseObject(-1,'msg_invalid_request');
+		if(!$logged_info->member_srl && !$obj->nick_name) {
+			return new BaseObject(-1,'msg_invalid_request');
+		}
 
 		$obj->lang_code = Context::getLangType();
 		// Insert data into the DB
-		if(!$obj->status) $this->_checkDocumentStatusForOldVersion($obj);
+		if(!$obj->status) {
+			$this->_checkDocumentStatusForOldVersion($obj);
+		}
 		$output = executeQuery('document.insertDocument', $obj);
 		if(!$output->toBool())
 		{
@@ -313,19 +379,25 @@ 
 				if(isset($obj->{'extra_vars'.$idx}))
 				{
 					$tmp = $obj->{'extra_vars'.$idx};
-					if(is_array($tmp))
-						$value = implode('|@|', $tmp);
-					else
-						$value = trim($tmp);
+					if(is_array($tmp)) {
+											$value = implode('|@|', $tmp);
+					} else {
+											$value = trim($tmp);
+					}
+				} else if(isset($obj->{$extra_item->name})) {
+					$value = trim($obj->{$extra_item->name});
+				}
+				if($value == NULL) {
+					continue;
 				}
-				else if(isset($obj->{$extra_item->name})) $value = trim($obj->{$extra_item->name});
-				if($value == NULL) continue;
 
 				$this->insertDocumentExtraVar($obj->module_srl, $obj->document_srl, $idx, $value, $extra_item->eid);
 			}
 		}
 		// Update the category if the category_srl exists.
-		if($obj->category_srl) $this->updateCategoryCount($obj->module_srl, $obj->category_srl);
+		if($obj->category_srl) {
+			$this->updateCategoryCount($obj->module_srl, $obj->category_srl);
+		}
 		// Call a trigger (after)
 		if($output->toBool())
 		{
@@ -367,27 +439,39 @@ 
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
 
-		if(!$source_obj->document_srl || !$obj->document_srl) return new BaseObject(-1,'msg_invalied_request');
-		if(!$obj->status && $obj->is_secret == 'Y') $obj->status = 'SECRET';
-		if(!$obj->status) $obj->status = 'PUBLIC';
+		if(!$source_obj->document_srl || !$obj->document_srl) {
+			return new BaseObject(-1,'msg_invalied_request');
+		}
+		if(!$obj->status && $obj->is_secret == 'Y') {
+			$obj->status = 'SECRET';
+		}
+		if(!$obj->status) {
+			$obj->status = 'PUBLIC';
+		}
 
 		// Call a trigger (before)
 		$output = ModuleHandler::triggerCall('document.updateDocument', 'before', $obj);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		// begin transaction
 		$oDB = &DB::getInstance();
 		$oDB->begin();
 
 		$oModuleModel = getModel('module');
-		if(!$obj->module_srl) $obj->module_srl = $source_obj->get('module_srl');
+		if(!$obj->module_srl) {
+			$obj->module_srl = $source_obj->get('module_srl');
+		}
 		$module_srl = $obj->module_srl;
 		$document_config = $oModuleModel->getModulePartConfig('document', $module_srl);
 		if(!$document_config)
 		{
 			$document_config = new stdClass();
 		}
-		if(!isset($document_config->use_history)) $document_config->use_history = 'N';
+		if(!isset($document_config->use_history)) {
+			$document_config->use_history = 'N';
+		}
 		$bUseHistory = $document_config->use_history == 'Y' || $document_config->use_history == 'Trace';
 
 		if($bUseHistory)
@@ -396,22 +480,31 @@ 
 			$args->history_srl = getNextSequence();
 			$args->document_srl = $obj->document_srl;
 			$args->module_srl = $module_srl;
-			if($document_config->use_history == 'Y') $args->content = $source_obj->get('content');
+			if($document_config->use_history == 'Y') {
+				$args->content = $source_obj->get('content');
+			}
 			$args->nick_name = $source_obj->get('nick_name');
 			$args->member_srl = $source_obj->get('member_srl');
 			$args->regdate = $source_obj->get('last_update');
 			$args->ipaddress = $source_obj->get('ipaddress');
 			$output = executeQuery("document.insertHistory", $args);
-		}
-		else
+		} else
 		{
 			$obj->ipaddress = $source_obj->get('ipaddress');
 		}
 		// List variables
-		if($obj->comment_status) $obj->commentStatus = $obj->comment_status;
-		if(!$obj->commentStatus) $obj->commentStatus = 'DENY';
-		if($obj->commentStatus == 'DENY') $this->_checkCommentStatusForOldVersion($obj);
-		if($obj->allow_trackback!='Y') $obj->allow_trackback = 'N';
+		if($obj->comment_status) {
+			$obj->commentStatus = $obj->comment_status;
+		}
+		if(!$obj->commentStatus) {
+			$obj->commentStatus = 'DENY';
+		}
+		if($obj->commentStatus == 'DENY') {
+			$this->_checkCommentStatusForOldVersion($obj);
+		}
+		if($obj->allow_trackback!='Y') {
+			$obj->allow_trackback = 'N';
+		}
 		if($obj->homepage)
 		{
 			$obj->homepage = removeHackTag($obj->homepage);
@@ -421,7 +514,9 @@ 
 			}
 		}
 		
-		if($obj->notify_message != 'Y') $obj->notify_message = 'N';
+		if($obj->notify_message != 'Y') {
+			$obj->notify_message = 'N';
+		}
 		
 		// can modify regdate only manager
                 $grant = Context::get('grant');
@@ -431,7 +526,9 @@ 
 		}
 		
 		// Serialize the $extra_vars
-		if(!is_string($obj->extra_vars)) $obj->extra_vars = serialize($obj->extra_vars);
+		if(!is_string($obj->extra_vars)) {
+			$obj->extra_vars = serialize($obj->extra_vars);
+		}
 		// Remove the columns for automatic saving
 		unset($obj->_saved_doc_srl);
 		unset($obj->_saved_doc_title);
@@ -443,7 +540,9 @@ 
 		if($source_obj->get('category_srl')!=$obj->category_srl)
 		{
 			$category_list = $oDocumentModel->getCategoryList($obj->module_srl);
-			if(!$category_list[$obj->category_srl]) $obj->category_srl = 0;
+			if(!$category_list[$obj->category_srl]) {
+				$obj->category_srl = 0;
+			}
 		}
 		// Change the update order
 		$obj->update_order = getNextSequence() * -1;
@@ -478,9 +577,13 @@ 
 		// If the tile is empty, extract string from the contents.
 		$obj->title = htmlspecialchars($obj->title, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 		settype($obj->title, "string");
-		if($obj->title == '') $obj->title = cut_str(strip_tags($obj->content),20,'...');
+		if($obj->title == '') {
+			$obj->title = cut_str(strip_tags($obj->content),20,'...');
+		}
 		// If no tile extracted from the contents, leave it untitled.
-		if($obj->title == '') $obj->title = 'Untitled';
+		if($obj->title == '') {
+			$obj->title = 'Untitled';
+		}
 		// Remove XE's own tags from the contents.
 		$obj->content = preg_replace('!<\!--(Before|After)(Document|Comment)\(([0-9]+),([0-9]+)\)-->!is', '', $obj->content);
 		if(Mobile::isFromMobilePhone() && $obj->use_editor != 'Y')
@@ -500,8 +603,7 @@ 
 				$lang_code_args->document_srl = $source_obj->get('document_srl');
 				$lang_code_args->lang_code = Context::getLangType();
 				$output = executeQuery('document.updateDocumentsLangCode', $lang_code_args);
-			}
-			else
+			} else
 			{
 				$extra_content = new stdClass;
 				$extra_content->title = $obj->title;
@@ -520,7 +622,9 @@ 
 			$obj->content = removeHackTag($obj->content);
 		}
 		// if temporary document, regdate is now setting
-		if($source_obj->get('status') == $this->getConfigStatus('temp')) $obj->regdate = date('YmdHis');
+		if($source_obj->get('status') == $this->getConfigStatus('temp')) {
+			$obj->regdate = date('YmdHis');
+		}
 
 		// Insert data into the DB
 		$output = executeQuery('document.updateDocument', $obj);
@@ -543,25 +647,37 @@ 
 					if(isset($obj->{'extra_vars'.$idx}))
 					{
 						$tmp = $obj->{'extra_vars'.$idx};
-						if(is_array($tmp))
-							$value = implode('|@|', $tmp);
-						else
-							$value = trim($tmp);
+						if(is_array($tmp)) {
+													$value = implode('|@|', $tmp);
+						} else {
+													$value = trim($tmp);
+						}
+					} else if(isset($obj->{$extra_item->name})) {
+						$value = trim($obj->{$extra_item->name});
+					}
+					if($value == NULL) {
+						continue;
 					}
-					else if(isset($obj->{$extra_item->name})) $value = trim($obj->{$extra_item->name});
-					if($value == NULL) continue;
 					$this->insertDocumentExtraVar($obj->module_srl, $obj->document_srl, $idx, $value, $extra_item->eid);
 				}
 			}
 			// Inert extra vars for multi-language support of title and contents.
-			if($extra_content->title) $this->insertDocumentExtraVar($obj->module_srl, $obj->document_srl, -1, $extra_content->title, 'title_'.Context::getLangType());
-			if($extra_content->content) $this->insertDocumentExtraVar($obj->module_srl, $obj->document_srl, -2, $extra_content->content, 'content_'.Context::getLangType());
+			if($extra_content->title) {
+				$this->insertDocumentExtraVar($obj->module_srl, $obj->document_srl, -1, $extra_content->title, 'title_'.Context::getLangType());
+			}
+			if($extra_content->content) {
+				$this->insertDocumentExtraVar($obj->module_srl, $obj->document_srl, -2, $extra_content->content, 'content_'.Context::getLangType());
+			}
 		}
 		// Update the category if the category_srl exists.
 		if($source_obj->get('category_srl') != $obj->category_srl || $source_obj->get('module_srl') == $logged_info->member_srl)
 		{
-			if($source_obj->get('category_srl') != $obj->category_srl) $this->updateCategoryCount($obj->module_srl, $source_obj->get('category_srl'));
-			if($obj->category_srl) $this->updateCategoryCount($obj->module_srl, $obj->category_srl);
+			if($source_obj->get('category_srl') != $obj->category_srl) {
+				$this->updateCategoryCount($obj->module_srl, $source_obj->get('category_srl'));
+			}
+			if($obj->category_srl) {
+				$this->updateCategoryCount($obj->module_srl, $obj->category_srl);
+			}
 		}
 		// Call a trigger (after)
 		if($output->toBool())
@@ -606,7 +722,9 @@ 
 		$trigger_obj = new stdClass();
 		$trigger_obj->document_srl = $document_srl;
 		$output = ModuleHandler::triggerCall('document.deleteDocument', 'before', $trigger_obj);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		// begin transaction
 		$oDB = &DB::getInstance();
@@ -618,12 +736,17 @@ 
 			$oDocumentModel = getModel('document');
 			// Check if the documnet exists
 			$oDocument = $oDocumentModel->getDocument($document_srl, $is_admin);
+		} else if($isEmptyTrash && $oDocument == null) {
+			return new BaseObject(-1, 'document is not exists');
 		}
-		else if($isEmptyTrash && $oDocument == null) return new BaseObject(-1, 'document is not exists');
 
-		if(!$oDocument->isExists() || $oDocument->document_srl != $document_srl) return new BaseObject(-1, 'msg_invalid_document');
+		if(!$oDocument->isExists() || $oDocument->document_srl != $document_srl) {
+			return new BaseObject(-1, 'msg_invalid_document');
+		}
 		// Check if a permossion is granted
-		if(!$oDocument->isGranted()) return new BaseObject(-1, 'msg_not_permitted');
+		if(!$oDocument->isGranted()) {
+			return new BaseObject(-1, 'msg_not_permitted');
+		}
 
 		//if empty trash, document already deleted, therefore document not delete
 		$args = new stdClass();
@@ -643,7 +766,9 @@ 
 
 		$this->deleteDocumentHistory(null, $document_srl, null);
 		// Update category information if the category_srl exists.
-		if($oDocument->get('category_srl')) $this->updateCategoryCount($oDocument->get('module_srl'),$oDocument->get('category_srl'));
+		if($oDocument->get('category_srl')) {
+			$this->updateCategoryCount($oDocument->get('module_srl'),$oDocument->get('category_srl'));
+		}
 		// Delete a declared list
 		executeQuery('document.deleteDeclared', $args);
 		// Delete extra variable
@@ -723,8 +848,11 @@ 
 	{
 		$trash_args = new stdClass();
 		// Get trash_srl if a given trash_srl doesn't exist
-		if(!$obj->trash_srl) $trash_args->trash_srl = getNextSequence();
-		else $trash_args->trash_srl = $obj->trash_srl;
+		if(!$obj->trash_srl) {
+			$trash_args->trash_srl = getNextSequence();
+		} else {
+			$trash_args->trash_srl = $obj->trash_srl;
+		}
 		// Get its module_srl which the document belongs to
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument($obj->document_srl);
@@ -732,7 +860,9 @@ 
 		$trash_args->module_srl = $oDocument->get('module_srl');
 		$obj->module_srl = $oDocument->get('module_srl');
 		// Cannot throw data from the trash to the trash
-		if($trash_args->module_srl == 0) return false;
+		if($trash_args->module_srl == 0) {
+			return false;
+		}
 		// Data setting
 		$trash_args->document_srl = $obj->document_srl;
 		$trash_args->description = $obj->description;
@@ -793,7 +923,9 @@ 
 		  }*/
 
 		// update category
-		if($oDocument->get('category_srl')) $this->updateCategoryCount($oDocument->get('module_srl'),$oDocument->get('category_srl'));
+		if($oDocument->get('category_srl')) {
+			$this->updateCategoryCount($oDocument->get('module_srl'),$oDocument->get('category_srl'));
+		}
 
 		// remove thumbnails
 		FileHandler::removeDir(sprintf('files/thumbnails/%s',getNumberingPath($obj->document_srl, 3)));
@@ -838,7 +970,9 @@ 
 	function updateReadedCount(&$oDocument)
 	{
 		// Pass if Crawler access
-		if(isCrawler()) return false;
+		if(isCrawler()) {
+			return false;
+		}
 		
 		$document_srl = $oDocument->document_srl;
 		$member_srl = $oDocument->get('member_srl');
@@ -846,10 +980,14 @@ 
 
 		// Call a trigger when the read count is updated (before)
 		$trigger_output = ModuleHandler::triggerCall('document.updateReadedCount', 'before', $oDocument);
-		if(!$trigger_output->toBool()) return $trigger_output;
+		if(!$trigger_output->toBool()) {
+			return $trigger_output;
+		}
 
 		// Pass if read count is increaded on the session information
-		if($_SESSION['readed_document'][$document_srl]) return false;
+		if($_SESSION['readed_document'][$document_srl]) {
+			return false;
+		}
 
 		// Pass if the author's IP address is as same as visitor's.
 		if($oDocument->get('ipaddress') == $_SERVER['REMOTE_ADDR'])
@@ -914,7 +1052,9 @@ 
 	 */
 	function insertDocumentExtraKey($module_srl, $var_idx, $var_name, $var_type, $var_is_required = 'N', $var_search = 'N', $var_default = '', $var_desc = '', $eid)
 	{
-		if(!$module_srl || !$var_idx || !$var_name || !$var_type || !$eid) return new BaseObject(-1,'msg_invalid_request');
+		if(!$module_srl || !$var_idx || !$var_name || !$var_type || !$eid) {
+			return new BaseObject(-1,'msg_invalid_request');
+		}
 
 		$obj = new stdClass();
 		$obj->module_srl = $module_srl;
@@ -931,8 +1071,7 @@ 
 		if(!$output->data)
 		{
 			$output = executeQuery('document.insertDocumentExtraKey', $obj);
-		}
-		else
+		} else
 		{
 			$output = executeQuery('document.updateDocumentExtraKey', $obj);
 			// Update the extra var(eid)
@@ -958,10 +1097,14 @@ 
 	 */
 	function deleteDocumentExtraKeys($module_srl, $var_idx = null)
 	{
-		if(!$module_srl) return new BaseObject(-1,'msg_invalid_request');
+		if(!$module_srl) {
+			return new BaseObject(-1,'msg_invalid_request');
+		}
 		$obj = new stdClass();
 		$obj->module_srl = $module_srl;
-		if(!is_null($var_idx)) $obj->var_idx = $var_idx;
+		if(!is_null($var_idx)) {
+			$obj->var_idx = $var_idx;
+		}
 
 		$oDB = DB::getInstance();
 		$oDB->begin();
@@ -1025,8 +1168,12 @@ 
 	 */
 	function insertDocumentExtraVar($module_srl, $document_srl, $var_idx, $value, $eid = null, $lang_code = '')
 	{
-		if(!$module_srl || !$document_srl || !$var_idx || !isset($value)) return new BaseObject(-1,'msg_invalid_request');
-		if(!$lang_code) $lang_code = Context::getLangType();
+		if(!$module_srl || !$document_srl || !$var_idx || !isset($value)) {
+			return new BaseObject(-1,'msg_invalid_request');
+		}
+		if(!$lang_code) {
+			$lang_code = Context::getLangType();
+		}
 
 		$obj = new stdClass;
 		$obj->module_srl = $module_srl;
@@ -1052,10 +1199,18 @@ 
 	{
 		$obj = new stdClass();
 		$obj->module_srl = $module_srl;
-		if(!is_null($document_srl)) $obj->document_srl = $document_srl;
-		if(!is_null($var_idx)) $obj->var_idx = $var_idx;
-		if(!is_null($lang_code)) $obj->lang_code = $lang_code;
-		if(!is_null($eid)) $obj->eid = $eid;
+		if(!is_null($document_srl)) {
+			$obj->document_srl = $document_srl;
+		}
+		if(!is_null($var_idx)) {
+			$obj->var_idx = $var_idx;
+		}
+		if(!is_null($lang_code)) {
+			$obj->lang_code = $lang_code;
+		}
+		if(!is_null($eid)) {
+			$obj->eid = $eid;
+		}
 		$output = executeQuery('document.deleteDocumentExtraVars', $obj);
 		return $output;
 	}
@@ -1069,8 +1224,11 @@ 
 	 */
 	function updateVotedCount($document_srl, $point = 1)
 	{
-		if($point > 0) $failed_voted = 'failed_voted';
-		else $failed_voted = 'failed_blamed';
+		if($point > 0) {
+			$failed_voted = 'failed_voted';
+		} else {
+			$failed_voted = 'failed_blamed';
+		}
 		// Return fail if session already has information about votes
 		if($_SESSION['voted_document'][$document_srl])
 		{
@@ -1106,8 +1264,7 @@ 
 		if($member_srl)
 		{
 			$args->member_srl = $member_srl;
-		}
-		else
+		} else
 		{
 			$args->ipaddress = $_SERVER['REMOTE_ADDR'];
 		}
@@ -1129,17 +1286,20 @@ 
 		{
 			$args->blamed_count = $oDocument->get('blamed_count') + $point;
 			$output = executeQuery('document.updateBlamedCount', $args);
-		}
-		else
+		} else
 		{
 			$args->voted_count = $oDocument->get('voted_count') + $point;
 			$output = executeQuery('document.updateVotedCount', $args);
 		}
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 		// Leave logs
 		$args->point = $point;
 		$output = executeQuery('document.insertDocumentVotedLog', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$obj = new stdClass;
 		$obj->member_srl = $oDocument->get('member_srl');
@@ -1175,8 +1335,7 @@ 
 		{
 			$output->setMessage('success_voted');
 			$output->add('voted_count', $obj->after_point);
-		}
-		else
+		} else
 		{
 			$output->setMessage('success_blamed');
 			$output->add('blamed_count', $obj->after_point);
@@ -1193,13 +1352,17 @@ 
 	function declaredDocument($document_srl)
 	{
 		// Fail if session information already has a reported document
-		if($_SESSION['declared_document'][$document_srl]) return new BaseObject(-1, 'failed_declared');
+		if($_SESSION['declared_document'][$document_srl]) {
+			return new BaseObject(-1, 'failed_declared');
+		}
 
 		// Check if previously reported
 		$args = new stdClass();
 		$args->document_srl = $document_srl;
 		$output = executeQuery('document.getDeclaredDocument', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$declared_count = ($output->data->declared_count) ? $output->data->declared_count : 0;
 
@@ -1243,8 +1406,7 @@ 
 		if($member_srl)
 		{
 			$args->member_srl = $member_srl;
-		}
-		else
+		} else
 		{
 			$args->ipaddress = $_SERVER['REMOTE_ADDR'];
 		}
@@ -1264,9 +1426,14 @@ 
 		$oDB->begin();
 
 		// Add the declared document
-		if($declared_count > 0) $output = executeQuery('document.updateDeclaredDocument', $args);
-		else $output = executeQuery('document.insertDeclaredDocument', $args);
-		if(!$output->toBool()) return $output;
+		if($declared_count > 0) {
+			$output = executeQuery('document.updateDeclaredDocument', $args);
+		} else {
+			$output = executeQuery('document.insertDeclaredDocument', $args);
+		}
+		if(!$output->toBool()) {
+			return $output;
+		}
 		// Leave logs
 		$output = executeQuery('document.insertDocumentDeclaredLog', $args);
 		if(!$output->toBool())
@@ -1365,9 +1532,10 @@ 
 			$parent_category = $oDocumentModel->getCategory($obj->parent_srl);
 			$obj->list_order = $parent_category->list_order;
 			$this->updateCategoryListOrder($parent_category->module_srl, $parent_category->list_order+1);
-			if(!$obj->category_srl) $obj->category_srl = getNextSequence();
-		}
-		else
+			if(!$obj->category_srl) {
+				$obj->category_srl = getNextSequence();
+			}
+		} else
 		{
 			$obj->list_order = $obj->category_srl = getNextSequence();
 		}
@@ -1407,13 +1575,17 @@ 
 	{
 		// Create a document model object
 		$oDocumentModel = getModel('document');
-		if(!$document_count) $document_count = $oDocumentModel->getCategoryDocumentCount($module_srl,$category_srl);
+		if(!$document_count) {
+			$document_count = $oDocumentModel->getCategoryDocumentCount($module_srl,$category_srl);
+		}
 
 		$args = new stdClass;
 		$args->category_srl = $category_srl;
 		$args->document_count = $document_count;
 		$output = executeQuery('document.updateCategoryCount', $args);
-		if($output->toBool()) $this->makeCategoryFile($module_srl);
+		if($output->toBool()) {
+			$this->makeCategoryFile($module_srl);
+		}
 
 		return $output;
 	}
@@ -1426,7 +1598,9 @@ 
 	function updateCategory($obj)
 	{
 		$output = executeQuery('document.updateCategory', $obj);
-		if($output->toBool()) $this->makeCategoryFile($obj->module_srl);
+		if($output->toBool()) {
+			$this->makeCategoryFile($obj->module_srl);
+		}
 		return $output;
 	}
 
@@ -1443,11 +1617,17 @@ 
 		$category_info = $oDocumentModel->getCategory($category_srl);
 		// Display an error that the category cannot be deleted if it has a child
 		$output = executeQuery('document.getChildCategoryCount', $args);
-		if(!$output->toBool()) return $output;
-		if($output->data->count>0) return new BaseObject(-1, 'msg_cannot_delete_for_child');
+		if(!$output->toBool()) {
+			return $output;
+		}
+		if($output->data->count>0) {
+			return new BaseObject(-1, 'msg_cannot_delete_for_child');
+		}
 		// Delete a category information
 		$output = executeQuery('document.deleteCategory', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->makeCategoryFile($category_info->module_srl);
 		// remvove cache
@@ -1462,8 +1642,9 @@ 
 				$args->page = ++$page;
 				$output = executeQuery('document.getDocumentList', $args, array('document_srl'));
 
-				if($output->data == array())
-					break;
+				if($output->data == array()) {
+									break;
+				}
 
 				foreach($output->data as $val)
 				{
@@ -1515,18 +1696,26 @@ 
 		// Seek a full list of categories
 		$category_list = $oDocumentModel->getCategoryList($module_srl);
 		$category_srl_list = array_keys($category_list);
-		if(count($category_srl_list)<2) return new BaseObject();
+		if(count($category_srl_list)<2) {
+			return new BaseObject();
+		}
 
 		$prev_category = NULL;
 		foreach($category_list as $key => $val)
 		{
-			if($key==$category_srl) break;
+			if($key==$category_srl) {
+				break;
+			}
 			$prev_category = $val;
 		}
 		// Return if the previous category doesn't exist
-		if(!$prev_category) return new BaseObject(-1,Context::getLang('msg_category_not_moved'));
+		if(!$prev_category) {
+			return new BaseObject(-1,Context::getLang('msg_category_not_moved'));
+		}
 		// Return if the selected category is the top level
-		if($category_srl_list[0]==$category_srl) return new BaseObject(-1,Context::getLang('msg_category_not_moved'));
+		if($category_srl_list[0]==$category_srl) {
+			return new BaseObject(-1,Context::getLang('msg_category_not_moved'));
+		}
 		// Information of the selected category
 		$cur_args = new stdClass;
 		$cur_args->category_srl = $category_srl;
@@ -1562,15 +1751,21 @@ 
 		// Seek a full list of categories
 		$category_list = $oDocumentModel->getCategoryList($module_srl);
 		$category_srl_list = array_keys($category_list);
-		if(count($category_srl_list)<2) return new BaseObject();
+		if(count($category_srl_list)<2) {
+			return new BaseObject();
+		}
 
 		for($i=0;$i<count($category_srl_list);$i++)
 		{
-			if($category_srl_list[$i]==$category_srl) break;
+			if($category_srl_list[$i]==$category_srl) {
+				break;
+			}
 		}
 
 		$next_category_srl = $category_srl_list[$i+1];
-		if(!$category_list[$next_category_srl]) return new BaseObject(-1,Context::getLang('msg_category_not_moved'));
+		if(!$category_list[$next_category_srl]) {
+			return new BaseObject(-1,Context::getLang('msg_category_not_moved'));
+		}
 		$next_category = $category_list[$next_category_srl];
 		// Information of the selected category
 		$cur_args = new stdClass;
@@ -1597,7 +1792,9 @@ 
 	{
 		$oDocumentModel = getModel('document');
 		$extra_keys = $oDocumentModel->getExtraKeys($module_srl);
-		if(!count($extra_keys)) return;
+		if(!count($extra_keys)) {
+			return;
+		}
 
 		$js_code = array();
 		$js_code[] = '<script>//<![CDATA[';
@@ -1616,7 +1813,9 @@ 
 			}
 			$name = str_ireplace(array('<script', '</script'), array('<scr" + "ipt', '</scr" + "ipt'), $val->name);
 			$js_code[] = sprintf('validator.cast("ADD_MESSAGE", ["extra_vars%s","%s"]);', $idx, $name);
-			if($val->is_required == 'Y') $js_code[] = sprintf('validator.cast("ADD_EXTRA_FIELD", ["extra_vars%s", { required:true }]);', $idx);
+			if($val->is_required == 'Y') {
+				$js_code[] = sprintf('validator.cast("ADD_EXTRA_FIELD", ["extra_vars%s", { required:true }]);', $idx);
+			}
 		}
 
 		$js_code[] = '})(jQuery);';
@@ -1634,7 +1833,9 @@ 
 	function procDocumentInsertCategory($args = null)
 	{
 		// List variables
-		if(!$args) $args = Context::gets('module_srl','category_srl','parent_srl','category_title','category_description','expand','group_srls','category_color','mid');
+		if(!$args) {
+			$args = Context::gets('module_srl','category_srl','parent_srl','category_title','category_description','expand','group_srls','category_color','mid');
+		}
 		$args->title = $args->category_title;
 		$args->description = $args->category_description;
 		$args->color = $args->category_color;
@@ -1650,11 +1851,18 @@ 
 		$columnList = array('module_srl', 'module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($args->module_srl, $columnList);
 		$grant = $oModuleModel->getGrant($module_info, Context::get('logged_info'));
-		if(!$grant->manager) return new BaseObject(-1,'msg_not_permitted');
+		if(!$grant->manager) {
+			return new BaseObject(-1,'msg_not_permitted');
+		}
 
-		if($args->expand !="Y") $args->expand = "N";
-		if(!is_array($args->group_srls)) $args->group_srls = str_replace('|@|',',',$args->group_srls);
-		else $args->group_srls = implode(',', $args->group_srls);
+		if($args->expand !="Y") {
+			$args->expand = "N";
+		}
+		if(!is_array($args->group_srls)) {
+			$args->group_srls = str_replace('|@|',',',$args->group_srls);
+		} else {
+			$args->group_srls = implode(',', $args->group_srls);
+		}
 		$args->parent_srl = (int)$args->parent_srl;
 
 		$oDocumentModel = getModel('document');
@@ -1665,7 +1873,9 @@ 
 		if($args->category_srl)
 		{
 			$category_info = $oDocumentModel->getCategory($args->category_srl);
-			if($category_info->category_srl != $args->category_srl) $args->category_srl = null;
+			if($category_info->category_srl != $args->category_srl) {
+				$args->category_srl = null;
+			}
 		}
 		// Update if exists
 		if($args->category_srl)
@@ -1677,8 +1887,7 @@ 
 				return $output;
 			}
 			// Insert if not exist
-		}
-		else
+		} else
 		{
 			$output = $this->insertCategory($args);
 			if(!$output->toBool())
@@ -1720,7 +1929,9 @@ 
 		$columnList = array('module_srl', 'module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($source_category->module_srl, $columnList);
 		$grant = $oModuleModel->getGrant($module_info, Context::get('logged_info'));
-		if(!$grant->manager) return new BaseObject(-1,'msg_not_permitted');
+		if(!$grant->manager) {
+			return new BaseObject(-1,'msg_not_permitted');
+		}
 
 		// First child of the parent_category_srl
 		$source_args = new stdClass;
@@ -1733,30 +1944,39 @@ 
 			$args->parent_srl = $parent_category_srl;
 			$output = executeQuery('document.getChildCategoryMinListOrder', $args);
 
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 			$args->list_order = (int)$output->data->list_order;
-			if(!$args->list_order) $args->list_order = 0;
+			if(!$args->list_order) {
+				$args->list_order = 0;
+			}
 			$args->list_order--;
 
 			$source_args->category_srl = $source_category_srl;
 			$source_args->parent_srl = $parent_category_srl;
 			$source_args->list_order = $args->list_order;
 			$output = $this->updateCategory($source_args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 			// Sibling of the $target_category_srl
-		}
-		else if($target_category_srl > 0)
+		} else if($target_category_srl > 0)
 		{
 			$target_category = $oDocumentModel->getCategory($target_category_srl);
 			// Move all siblings of the $target_category down
 			$output = $this->updateCategoryListOrder($target_category->module_srl, $target_category->list_order+1);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 
 			$source_args->category_srl = $source_category_srl;
 			$source_args->parent_srl = $target_category->parent_srl;
 			$source_args->list_order = $target_category->list_order+1;
 			$output = $this->updateCategory($source_args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 		}
 		// Re-generate the xml file
 		$xml_file = $this->makeCategoryFile($source_category->module_srl);
@@ -1781,14 +2001,20 @@ 
 		$columnList = array('module_srl', 'module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($args->module_srl, $columnList);
 		$grant = $oModuleModel->getGrant($module_info, Context::get('logged_info'));
-		if(!$grant->manager) return new BaseObject(-1,'msg_not_permitted');
+		if(!$grant->manager) {
+			return new BaseObject(-1,'msg_not_permitted');
+		}
 
 		$oDocumentModel = getModel('document');
 		// Get original information
 		$category_info = $oDocumentModel->getCategory($args->category_srl);
-		if($category_info->parent_srl) $parent_srl = $category_info->parent_srl;
+		if($category_info->parent_srl) {
+			$parent_srl = $category_info->parent_srl;
+		}
 		// Display an error that the category cannot be deleted if it has a child node
-		if($oDocumentModel->getCategoryChlidCount($args->category_srl)) return new BaseObject(-1, 'msg_cannot_delete_for_child');
+		if($oDocumentModel->getCategoryChlidCount($args->category_srl)) {
+			return new BaseObject(-1, 'msg_cannot_delete_for_child');
+		}
 		// Remove from the DB
 		$output = $this->deleteCategory($args->category_srl);
 		if(!$output->toBool())
@@ -1822,7 +2048,9 @@ 
 		$columnList = array('module_srl', 'module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl, $columnList);
 		$grant = $oModuleModel->getGrant($module_info, Context::get('logged_info'));
-		if(!$grant->manager) return new BaseObject(-1,'msg_not_permitted');
+		if(!$grant->manager) {
+			return new BaseObject(-1,'msg_not_permitted');
+		}
 
 		$xml_file = $this->makeCategoryFile($module_srl);
 		// Set return value
@@ -1837,14 +2065,18 @@ 
 	function makeCategoryFile($module_srl)
 	{
 		// Return if there is no information you need for creating a cache file
-		if(!$module_srl) return false;
+		if(!$module_srl) {
+			return false;
+		}
 		// Get module information (to obtain mid)
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'mid', 'site_srl');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl, $columnList);
 		$mid = $module_info->mid;
 
-		if(!is_dir('./files/cache/document_category')) FileHandler::makeDir('./files/cache/document_category');
+		if(!is_dir('./files/cache/document_category')) {
+			FileHandler::makeDir('./files/cache/document_category');
+		}
 		// Cache file's name
 		$xml_file = sprintf("./files/cache/document_category/%s.xml.php", $module_srl);
 		$php_file = sprintf("./files/cache/document_category/%s.php", $module_srl);
@@ -1856,13 +2088,17 @@ 
 
 		$category_list = $output->data;
 
-		if(!is_array($category_list)) $category_list = array($category_list);
+		if(!is_array($category_list)) {
+			$category_list = array($category_list);
+		}
 
 		$category_count = count($category_list);
 		for($i=0;$i<$category_count;$i++)
 		{
 			$category_srl = $category_list[$i]->category_srl;
-			if(!preg_match('/^[0-9,]+$/', $category_list[$i]->group_srls)) $category_list[$i]->group_srls = '';
+			if(!preg_match('/^[0-9,]+$/', $category_list[$i]->group_srls)) {
+				$category_list[$i]->group_srls = '';
+			}
 			$list[$category_srl] = $category_list[$i];
 		}
 		// Create the xml file without node data if no data is obtained
@@ -1874,7 +2110,9 @@ 
 			return $xml_file;
 		}
 		// Change to an array if only a single data is obtained
-		if(!is_array($list)) $list = array($list);
+		if(!is_array($list)) {
+			$list = array($list);
+		}
 		// Create a tree for loop
 		foreach($list as $category_srl => $node)
 		{
@@ -1953,13 +2191,17 @@ 
 	 */
 	function getXmlTree($source_node, $tree, $site_srl, &$xml_header_buff)
 	{
-		if(!$source_node) return;
+		if(!$source_node) {
+			return;
+		}
 
 		foreach($source_node as $category_srl => $node)
 		{
 			$child_buff = "";
 			// Get data of the child nodes
-			if($category_srl && $tree[$category_srl]) $child_buff = $this->getXmlTree($tree[$category_srl], $tree, $site_srl, $xml_header_buff);
+			if($category_srl && $tree[$category_srl]) {
+				$child_buff = $this->getXmlTree($tree[$category_srl], $tree, $site_srl, $xml_header_buff);
+			}
 			// List variables
 			$expand = ($node->expand) ? $node->expand : 'N';
 			$group_srls = ($node->group_srls) ? $node->group_srls : '';
@@ -1969,8 +2211,11 @@ 
 			$color = ($node->color) ? $node->color : '';
 			$description = ($node->description) ? $node->description : '';
 			// If node->group_srls value exists
-			if($group_srls) $group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s)))))',$group_srls);
-			else $group_check_code = "true";
+			if($group_srls) {
+				$group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s)))))',$group_srls);
+			} else {
+				$group_check_code = "true";
+			}
 
 			$title = $node->title;
 			$oModuleAdminModel = getAdminModel('module');
@@ -2010,8 +2255,11 @@ 
 				$node->document_count
 			);
 
-			if($child_buff) $buff .= sprintf('<node %s>%s</node>', $attribute, $child_buff);
-			else $buff .=  sprintf('<node %s />', $attribute);
+			if($child_buff) {
+				$buff .= sprintf('<node %s>%s</node>', $attribute, $child_buff);
+			} else {
+				$buff .=  sprintf('<node %s />', $attribute);
+			}
 		}
 		return $buff;
 	}
@@ -2030,7 +2278,9 @@ 
 	function getPhpCacheCode($source_node, $tree, $site_srl, &$php_header_buff)
 	{
 		$output = array("buff"=>"", "category_srl_list"=>array());
-		if(!$source_node) return $output;
+		if(!$source_node) {
+			return $output;
+		}
 
 		// Set to an arraty for looping and then generate php script codes to be included
 		foreach($source_node as $category_srl => $node)
@@ -2129,7 +2379,9 @@ 
 	function addDocumentPopupMenu($url, $str, $icon = '', $target = 'self')
 	{
 		$document_popup_menu_list = Context::get('document_popup_menu_list');
-		if(!is_array($document_popup_menu_list)) $document_popup_menu_list = array();
+		if(!is_array($document_popup_menu_list)) {
+			$document_popup_menu_list = array();
+		}
 
 		$obj = new stdClass();
 		$obj->url = $url;
@@ -2147,7 +2399,9 @@ 
 	 */
 	function procDocumentAddCart()
 	{
-		if(!Context::get('is_logged')) return new BaseObject(-1, 'msg_not_permitted');
+		if(!Context::get('is_logged')) {
+			return new BaseObject(-1, 'msg_not_permitted');
+		}
 
 		// Get document_srl
 		$srls = explode(',',Context::get('srls'));
@@ -2155,11 +2409,15 @@ 
 		{
 			$srl = trim($srls[$i]);
 
-			if(!$srl) continue;
+			if(!$srl) {
+				continue;
+			}
 
 			$document_srls[] = $srl;
 		}
-		if(!count($document_srls)) return;
+		if(!count($document_srls)) {
+			return;
+		}
 
 		// Get module_srl of the documents
 		$args = new stdClass;
@@ -2167,14 +2425,18 @@ 
 		$args->document_srls = implode(',',$document_srls);
 		$args->order_type = 'asc';
 		$output = executeQueryArray('document.getDocuments', $args);
-		if(!$output->data) return new BaseObject();
+		if(!$output->data) {
+			return new BaseObject();
+		}
 
 		unset($document_srls);
 		foreach($output->data as $key => $val)
 		{
 			$document_srls[$val->module_srl][] = $val->document_srl;
 		}
-		if(!$document_srls || !count($document_srls)) return new BaseObject();
+		if(!$document_srls || !count($document_srls)) {
+			return new BaseObject();
+		}
 
 		// Check if each of module administrators exists. Top-level administator will have a permission to modify every document of all modules.(Even to modify temporarily saved or trashed documents)
 		$oModuleModel = getModel('module');
@@ -2199,7 +2461,9 @@ 
 				}
 			}
 		}
-		if(!count($document_srls)) return new BaseObject();
+		if(!count($document_srls)) {
+			return new BaseObject();
+		}
 
 		foreach($document_srls as $module_srl => $documents)
 		{
@@ -2207,9 +2471,14 @@ 
 			for($i=0;$i<$cnt;$i++)
 			{
 				$document_srl = (int)trim($documents[$i]);
-				if(!$document_srls) continue;
-				if($_SESSION['document_management'][$document_srl]) unset($_SESSION['document_management'][$document_srl]);
-				else $_SESSION['document_management'][$document_srl] = true;
+				if(!$document_srls) {
+					continue;
+				}
+				if($_SESSION['document_management'][$document_srl]) {
+					unset($_SESSION['document_management'][$document_srl]);
+				} else {
+					$_SESSION['document_management'][$document_srl] = true;
+				}
 			}
 		}
 	}
@@ -2221,7 +2490,9 @@ 
 	function procDocumentManageCheckedDocument()
 	{
 		@set_time_limit(0);
-		if(!Context::get('is_logged')) return new BaseObject(-1,'msg_not_permitted');
+		if(!Context::get('is_logged')) {
+			return new BaseObject(-1,'msg_not_permitted');
+		}
 
 		if(!checkCSRF())
 		{
@@ -2231,14 +2502,21 @@ 
 		$type = Context::get('type');
 		$target_module = Context::get('target_module');
 		$module_srl = Context::get('module_srl');
-		if($target_module && !$module_srl) $module_srl = $target_module;
+		if($target_module && !$module_srl) {
+			$module_srl = $target_module;
+		}
 		$category_srl = Context::get('target_category');
 		$message_content = Context::get('message_content');
-		if($message_content) $message_content = nl2br($message_content);
+		if($message_content) {
+			$message_content = nl2br($message_content);
+		}
 
 		$cart = Context::get('cart');
-		if(!is_array($cart)) $document_srl_list = explode('|@|', $cart);
-		else $document_srl_list = $cart;
+		if(!is_array($cart)) {
+			$document_srl_list = explode('|@|', $cart);
+		} else {
+			$document_srl_list = $cart;
+		}
 
 		$document_srl_count = count($document_srl_list);
 
@@ -2248,7 +2526,9 @@ 
 		{
 			$oDocument = $oDocumentModel->getDocument($document_srl);
 			$document_items[] = $oDocument;
-			if(!$oDocument->isGranted()) return $this->stop('msg_not_permitted');
+			if(!$oDocument->isGranted()) {
+				return $this->stop('msg_not_permitted');
+			}
 		}
 
 		// Send a message
@@ -2264,10 +2544,15 @@ 
 
 			foreach($document_items as $oDocument)
 			{
-				if(!$oDocument->get('member_srl') || $oDocument->get('member_srl')==$sender_member_srl) continue;
+				if(!$oDocument->get('member_srl') || $oDocument->get('member_srl')==$sender_member_srl) {
+					continue;
+				}
 
-				if($type=='move') $purl = sprintf("<a href=\"%s\" target=\"_blank\">%s</a>", $oDocument->getPermanentUrl(), $oDocument->getPermanentUrl());
-				else $purl = "";
+				if($type=='move') {
+					$purl = sprintf("<a href=\"%s\" target=\"_blank\">%s</a>", $oDocument->getPermanentUrl(), $oDocument->getPermanentUrl());
+				} else {
+					$purl = "";
+				}
 				$content = sprintf("<div>%s</div><hr />%s<div style=\"font-weight:bold\">%s</div>%s",$message_content, $purl, $oDocument->getTitleText(), $oDocument->getContent(false, false, false));
 
 				$oCommunicationController->sendMessage($sender_member_srl, $oDocument->get('member_srl'), $title, $content, false);
@@ -2280,24 +2565,30 @@ 
 		$oDocumentAdminController = getAdminController('document');
 		if($type == 'move')
 		{
-			if(!$module_srl) return new BaseObject(-1, 'fail_to_move');
+			if(!$module_srl) {
+				return new BaseObject(-1, 'fail_to_move');
+			}
 
 			$output = $oDocumentAdminController->moveDocumentModule($document_srl_list, $module_srl, $category_srl);
-			if(!$output->toBool()) return new BaseObject(-1, 'fail_to_move');
+			if(!$output->toBool()) {
+				return new BaseObject(-1, 'fail_to_move');
+			}
 
 			$msg_code = 'success_moved';
 
-		}
-		else if($type == 'copy')
+		} else if($type == 'copy')
 		{
-			if(!$module_srl) return new BaseObject(-1, 'fail_to_move');
+			if(!$module_srl) {
+				return new BaseObject(-1, 'fail_to_move');
+			}
 
 			$output = $oDocumentAdminController->copyDocumentModule($document_srl_list, $module_srl, $category_srl);
-			if(!$output->toBool()) return new BaseObject(-1, 'fail_to_move');
+			if(!$output->toBool()) {
+				return new BaseObject(-1, 'fail_to_move');
+			}
 
 			$msg_code = 'success_copied';
-		}
-		else if($type =='delete')
+		} else if($type =='delete')
 		{
 			$oDB = &DB::getInstance();
 			$oDB->begin();
@@ -2305,12 +2596,13 @@ 
 			{
 				$document_srl = $document_srl_list[$i];
 				$output = $this->deleteDocument($document_srl, true);
-				if(!$output->toBool()) return new BaseObject(-1, 'fail_to_delete');
+				if(!$output->toBool()) {
+					return new BaseObject(-1, 'fail_to_delete');
+				}
 			}
 			$oDB->commit();
 			$msg_code = 'success_deleted';
-		}
-		else if($type == 'trash')
+		} else if($type == 'trash')
 		{
 			$args = new stdClass();
 			$args->description = $message_content;
@@ -2320,12 +2612,13 @@ 
 			for($i=0;$i<$document_srl_count;$i++) {
 				$args->document_srl = $document_srl_list[$i];
 				$output = $this->moveDocumentToTrash($args);
-				if(!$output || !$output->toBool()) return new BaseObject(-1, 'fail_to_trash');
+				if(!$output || !$output->toBool()) {
+					return new BaseObject(-1, 'fail_to_trash');
+				}
 			}
 			$oDB->commit();
 			$msg_code = 'success_trashed';
-		}
-		else if($type == 'cancelDeclare')
+		} else if($type == 'cancelDeclare')
 		{
 			$args->document_srl = $document_srl_list;
 			$output = executeQuery('document.deleteDeclaredDocuments', $args);
@@ -2347,18 +2640,27 @@ 
 	function procDocumentInsertModuleConfig()
 	{
 		$module_srl = Context::get('target_module_srl');
-		if(preg_match('/^([0-9,]+)$/',$module_srl)) $module_srl = explode(',',$module_srl);
-		else $module_srl = array($module_srl);
+		if(preg_match('/^([0-9,]+)$/',$module_srl)) {
+			$module_srl = explode(',',$module_srl);
+		} else {
+			$module_srl = array($module_srl);
+		}
 
 		$document_config = new stdClass();
 		$document_config->use_history = Context::get('use_history');
-		if(!$document_config->use_history) $document_config->use_history = 'N';
+		if(!$document_config->use_history) {
+			$document_config->use_history = 'N';
+		}
 
 		$document_config->use_vote_up = Context::get('use_vote_up');
-		if(!$document_config->use_vote_up) $document_config->use_vote_up = 'Y';
+		if(!$document_config->use_vote_up) {
+			$document_config->use_vote_up = 'Y';
+		}
 
 		$document_config->use_vote_down = Context::get('use_vote_down');
-		if(!$document_config->use_vote_down) $document_config->use_vote_down = 'Y';
+		if(!$document_config->use_vote_down) {
+			$document_config->use_vote_down = 'Y';
+		}
 
 		$document_config->use_status = Context::get('use_status');
 
@@ -2366,7 +2668,9 @@ 
 		for($i=0;$i<count($module_srl);$i++)
 		{
 			$srl = trim($module_srl[$i]);
-			if(!$srl) continue;
+			if(!$srl) {
+				continue;
+			}
 			$output = $oModuleController->insertModulePartConfig('document',$srl,$document_config);
 		}
 		$this->setError(-1);
@@ -2383,7 +2687,9 @@ 
 	function procDocumentTempSave()
 	{
 		// Check login information
-		if(!Context::get('is_logged')) return new BaseObject(-1, 'msg_not_logged');
+		if(!Context::get('is_logged')) {
+			return new BaseObject(-1, 'msg_not_logged');
+		}
 		$module_info = Context::get('module_info');
 		$logged_info = Context::get('logged_info');
 
@@ -2421,8 +2727,7 @@ 
 			$output = $oDocumentController->updateDocument($oDocument, $obj);
 			$msg_code = 'success_updated';
 			// Otherwise, get a new
-		}
-		else
+		} else
 		{
 			$output = $oDocumentController->insertDocument($obj);
 			$msg_code = 'success_registed';
@@ -2448,17 +2753,20 @@ 
 	 */
 	function procDocumentGetList()
 	{
-		if(!Context::get('is_logged')) return new BaseObject(-1,'msg_not_permitted');
+		if(!Context::get('is_logged')) {
+			return new BaseObject(-1,'msg_not_permitted');
+		}
 		$documentSrls = Context::get('document_srls');
-		if($documentSrls) $documentSrlList = explode(',', $documentSrls);
+		if($documentSrls) {
+			$documentSrlList = explode(',', $documentSrls);
+		}
 
 		if(count($documentSrlList) > 0)
 		{
 			$oDocumentModel = getModel('document');
 			$columnList = array('document_srl', 'title', 'nick_name', 'status');
 			$documentList = $oDocumentModel->getDocuments($documentSrlList, $this->grant->is_admin, false, $columnList);
-		}
-		else
+		} else
 		{
 			global $lang;
 			$documentList = array();
@@ -2476,11 +2784,18 @@ 
 	 */
 	function _checkCommentStatusForOldVersion(&$obj)
 	{
-		if(!isset($obj->allow_comment)) $obj->allow_comment = 'N';
-		if(!isset($obj->lock_comment)) $obj->lock_comment = 'N';
+		if(!isset($obj->allow_comment)) {
+			$obj->allow_comment = 'N';
+		}
+		if(!isset($obj->lock_comment)) {
+			$obj->lock_comment = 'N';
+		}
 
-		if($obj->allow_comment == 'Y' && $obj->lock_comment == 'N') $obj->commentStatus = 'ALLOW';
-		else $obj->commentStatus = 'DENY';
+		if($obj->allow_comment == 'Y' && $obj->lock_comment == 'N') {
+			$obj->commentStatus = 'ALLOW';
+		} else {
+			$obj->commentStatus = 'DENY';
+		}
 	}
 
 	/**
@@ -2490,8 +2805,12 @@ 
 	 */
 	function _checkDocumentStatusForOldVersion(&$obj)
 	{
-		if(!$obj->status && $obj->is_secret == 'Y') $obj->status = $this->getConfigStatus('secret');
-		if(!$obj->status && $obj->is_secret != 'Y') $obj->status = $this->getConfigStatus('public');
+		if(!$obj->status && $obj->is_secret == 'Y') {
+			$obj->status = $this->getConfigStatus('secret');
+		}
+		if(!$obj->status && $obj->is_secret != 'Y') {
+			$obj->status = $this->getConfigStatus('public');
+		}
 	}
 
 	public function updateUploaedCount($documentSrlList)

modules/menu/menu.admin.controller.php

Spacing +304 added lines, -304 removed lines

@@ -56,7 +56,7 @@ 
 	}
 
 	function __construct() {
-		$this->homeMenuCacheFile = _XE_PATH_ . $this->homeMenuCacheFile;
+		$this->homeMenuCacheFile = _XE_PATH_.$this->homeMenuCacheFile;
 	}
 
 	/**
@@ -68,8 +68,8 @@ 
 		// List variables
 		$site_module_info = Context::get('site_module_info');
 
-		$output = $this->addMenu(Context::get('title'), (int)$site_module_info->site_srl);
-		if(!$output->toBool()) return $output;
+		$output = $this->addMenu(Context::get('title'), (int) $site_module_info->site_srl);
+		if (!$output->toBool()) return $output;
 
 		$this->add('menu_srl', $output->get('menuSrl'));
 		$this->setMessage('success_registed');
@@ -95,7 +95,7 @@ 
 		$args->listorder = $args->menu_srl * -1;
 
 		$output = executeQuery('menu.insertMenu', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -109,13 +109,13 @@ 
 		$unlinked_modules = false;
 		$args = new stdClass;
 		$args->site_srl = 0;
-		$output = executeQueryArray('module.getNotLinkedModuleBySiteSrl',$args);
-		if($output->toBool() && $output->data && count($output->data) > 0)
+		$output = executeQueryArray('module.getNotLinkedModuleBySiteSrl', $args);
+		if ($output->toBool() && $output->data && count($output->data) > 0)
 		{
 			$unlinked_modules = $output->data;
 		}
 
-		if($unlinked_modules)
+		if ($unlinked_modules)
 		{
 			$unlinked_menu_srl = $this->getUnlinkedMenu();
 			$output = $this->updateLinkModule($unlinked_modules, $unlinked_menu_srl);
@@ -129,21 +129,21 @@ 
 		$oModuleModel = getModel('module');
 		$moduleConfig = $oModuleModel->getModuleConfig('menu');
 
-		if($moduleConfig->unlinked_menu_srl)
+		if ($moduleConfig->unlinked_menu_srl)
 		{
 			$menuArgs = new stdClass;
 			$menuArgs->menu_srl = $moduleConfig->unlinked_menu_srl;
 			$menuOutput = executeQuery('menu.getMenu', $menuArgs);
-			if(!$menuOutput->data)
+			if (!$menuOutput->data)
 			{
 				unset($moduleConfig->unlinked_menu_srl);
 			}
 		}
 
-		if(!$moduleConfig->unlinked_menu_srl)
+		if (!$moduleConfig->unlinked_menu_srl)
 		{
 			$output = $this->addMenu('unlinked', 0);
-			if($output->toBool())
+			if ($output->toBool())
 			{
 				$moduleConfig->unlinked_menu_srl = $output->get('menuSrl');
 				$oModuleController = getController('module');
@@ -168,12 +168,12 @@ 
 	 */
 	function updateLinkModule($moduleInfos, $menuSrl)
 	{
-		if(!$moduleInfos || !is_array($moduleInfos) || count($moduleInfos) == 0 || $menuSrl == 0)
+		if (!$moduleInfos || !is_array($moduleInfos) || count($moduleInfos) == 0 || $menuSrl == 0)
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
 
-		foreach($moduleInfos as $moduleInfo)
+		foreach ($moduleInfos as $moduleInfo)
 		{
 			// search menu.
 			$args = new stdClass;
@@ -183,7 +183,7 @@ 
 
 			$output = executeQuery('menu.getMenuItemByUrl', $args);
 
-			if($output->toBool() && $output->data)
+			if ($output->toBool() && $output->data)
 			{
 				$moduleInfo->menu_srl = $output->data->menu_srl;
 			}
@@ -194,10 +194,10 @@ 
 				$item_args->url = $moduleInfo->mid;
 				$item_args->name = $moduleInfo->mid;
 				$item_args->menu_item_srl = getNextSequence();
-				$item_args->listorder = -1*$item_args->menu_item_srl;
+				$item_args->listorder = -1 * $item_args->menu_item_srl;
 
 				$output = executeQuery('menu.insertMenuItem', $item_args);
-				if(!$output->toBool())
+				if (!$output->toBool())
 				{
 					return $output;
 				}
@@ -210,7 +210,7 @@ 
 		}
 
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$oCacheHandler->invalidateGroupKey('site_and_module');
 		}
@@ -235,7 +235,7 @@ 
 		$args->menu_srl = Context::get('menu_srl');
 
 		$output = executeQuery('menu.updateMenu', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$this->setMessage('success_registed');
 
@@ -255,14 +255,14 @@ 
 		$menuInfo = $oMenuAdminModel->getMenu($menu_srl);
 
 		$oAdmin = getClass('admin');
-		if($menuInfo->title == $oAdmin->getAdminMenuName())
+		if ($menuInfo->title == $oAdmin->getAdminMenuName())
 			return new BaseObject(-1, 'msg_adminmenu_cannot_delete');
 
 		// get menu properies with child menu
 		$phpFile = sprintf("./files/cache/menu/%s.php", $menu_srl);
 		$originMenu = NULL;
 
-		if(is_readable(FileHandler::getRealPath($phpFile)))
+		if (is_readable(FileHandler::getRealPath($phpFile)))
 		{
 			include(FileHandler::getRealPath($phpFile));
 		}
@@ -273,25 +273,25 @@ 
 
 		$isStartmenuInclude = false;
 
-		if(is_array($menu->list))
+		if (is_array($menu->list))
 		{
-			foreach($menu->list AS $key=>$value)
+			foreach ($menu->list AS $key=>$value)
 			{
 				$originMenu = $value;
 				$this->_checkHomeMenuInOriginMenu($originMenu, $siteInfo->mid, $isStartmenuInclude);
 
-				if($isStartmenuInclude)
+				if ($isStartmenuInclude)
 					break;
 			}
 		}
 
-		if($isStartmenuInclude)
+		if ($isStartmenuInclude)
 		{
 			return new BaseObject(-1, 'msg_cannot_delete_homemenu');
 		}
 
 		$output = $this->deleteMenu($menu_srl);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return new BaseObject(-1, $output->message);
 		}
@@ -319,7 +319,7 @@ 
 
 		// Delete modules
 		$output = executeQueryArray('menu.getMenuItems', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -327,15 +327,15 @@ 
 		$oModuleController = getController('module');
 		$oModuleModel = getModel('module');
 
-		foreach($output->data as $itemInfo)
+		foreach ($output->data as $itemInfo)
 		{
-			if($itemInfo->is_shortcut != 'Y' && strncasecmp('http', $itemInfo->url, 4) !== 0)
+			if ($itemInfo->is_shortcut != 'Y' && strncasecmp('http', $itemInfo->url, 4) !== 0)
 			{
 				$moduleInfo = $oModuleModel->getModuleInfoByMid($itemInfo->url, $menuInfo->site_srl);
-				if($moduleInfo->module_srl)
+				if ($moduleInfo->module_srl)
 				{
 					$output = $oModuleController->onlyDeleteModule($moduleInfo->module_srl);
-					if(!$output->toBool())
+					if (!$output->toBool())
 					{
 						$oDB->rollback();
 						return $output;
@@ -346,27 +346,27 @@ 
 
 		// Delete menu items
 		$output = executeQuery("menu.deleteMenuItems", $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 		// Delete the menu
 		$output = executeQuery("menu.deleteMenu", $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 
 		// Delete cache files
-		$cache_list = FileHandler::readDir("./files/cache/menu","",false,true);
-		if(count($cache_list))
+		$cache_list = FileHandler::readDir("./files/cache/menu", "", false, true);
+		if (count($cache_list))
 		{
-			foreach($cache_list as $cache_file)
+			foreach ($cache_list as $cache_file)
 			{
 				$pos = strpos($cache_file, $menu_srl.'.');
-				if($pos>0)FileHandler::removeFile($cache_file);
+				if ($pos > 0)FileHandler::removeFile($cache_file);
 			}
 		}
 		// Delete images of menu buttons
@@ -375,7 +375,7 @@ 
 
 		$oDB->commit();
 
-		return new BaseObject(0,'success_deleted');
+		return new BaseObject(0, 'success_deleted');
 	}
 
 	/**
@@ -385,24 +385,24 @@ 
 	public function procMenuAdminInsertItem($request = NULL)
 	{
 		$isProc = false;
-		if(!$request)
+		if (!$request)
 		{
 			$isProc = true;
 			$request = Context::getRequestVars();
 		}
 
-		if(!$request->parent_srl || !$request->menu_name)
+		if (!$request->parent_srl || !$request->menu_name)
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
 
 		$this->_setMenuSrl($request->parent_srl, $request->menu_srl);
-		if(!$request->menu_srl)
+		if (!$request->menu_srl)
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
 
-		if($request->is_shortcut == 'Y')
+		if ($request->is_shortcut == 'Y')
 		{
 			$result = $this->_insertShortcut($request);
 		}
@@ -411,7 +411,7 @@ 
 			$result = $this->_insertMenu($request, $isProc);
 		}
 
-		if($result->error < 0)
+		if ($result->error < 0)
 		{
 			return new BaseObject($result->error, $result->message);
 		}
@@ -419,7 +419,7 @@ 
 		// recreate menu cache file
 		$this->makeXmlFile($request->menu_srl);
 
-		if(!$isProc)
+		if (!$isProc)
 		{
 			return $this->get('menu_item_srl');
 		}
@@ -431,7 +431,7 @@ 
 		$oMenuAdminModel = getAdminModel('menu');
 		$itemInfo = $oMenuAdminModel->getMenuItemInfo($parent_srl);
 		// parent_srl is parent menu item's srl
-		if($itemInfo->menu_srl)
+		if ($itemInfo->menu_srl)
 		{
 			$menu_srl = $itemInfo->menu_srl;
 		}
@@ -439,7 +439,7 @@ 
 		else
 		{
 			$output = $oMenuAdminModel->getMenu($parent_srl);
-			if($output->menu_srl == $parent_srl)
+			if ($output->menu_srl == $parent_srl)
 			{
 				$menu_srl = $output->menu_srl;
 				$parent_srl = 0;
@@ -453,7 +453,7 @@ 
 		$oDB->begin();
 
 		// type is url
-		if(strncasecmp('http', $request->shortcut_target, 4) === 0 || preg_match('/^(\.\/|\.\.\/|\/).*$/', $request->shortcut_target))
+		if (strncasecmp('http', $request->shortcut_target, 4) === 0 || preg_match('/^(\.\/|\.\.\/|\/).*$/', $request->shortcut_target))
 		{
 			// set menu variable
 			$args = new stdClass();
@@ -464,27 +464,27 @@ 
 			$args->is_shortcut = $request->is_shortcut;
 			$args->url = $request->shortcut_target;
 
-			if(!$args->open_window) $args->open_window = 'N';
-			if(!$args->expand) $args->expand = 'N';
-			if(!$args->is_shortcut) $args->is_shortcut = 'Y';
+			if (!$args->open_window) $args->open_window = 'N';
+			if (!$args->expand) $args->expand = 'N';
+			if (!$args->is_shortcut) $args->is_shortcut = 'Y';
 
-			if($request->menu_name_key) $args->name = $request->menu_name_key;
+			if ($request->menu_name_key) $args->name = $request->menu_name_key;
 			else $args->name = $request->menu_name;
 		}
 		// type is module short cut
-		else if(is_numeric($request->shortcut_target))
+		else if (is_numeric($request->shortcut_target))
 		{
 			// Get original information
 			$oMenuAdminModel = getAdminModel('menu');
 			$itemInfo = $oMenuAdminModel->getMenuItemInfo($request->shortcut_target);
-			if(!$itemInfo->menu_item_srl)
+			if (!$itemInfo->menu_item_srl)
 			{
 				return new BaseObject(-1, 'msg_invalid_request');
 			}
 			unset($itemInfo->normal_btn, $itemInfo->hover_btn, $itemInfo->active_btn);
 
 			$args = $itemInfo;
-			if(count($args->group_srls) == 0)
+			if (count($args->group_srls) == 0)
 			{
 				unset($args->group_srls);
 			}
@@ -504,13 +504,13 @@ 
 			$args->url = '#';
 		}
 
-		if($request->menu_desc) $args->desc = $request->menu_desc;
+		if ($request->menu_desc) $args->desc = $request->menu_desc;
 		else $args->desc = '';
 
 		$args->menu_item_srl = getNextSequence();
-		$args->listorder = -1*$args->menu_item_srl;
+		$args->listorder = -1 * $args->menu_item_srl;
 		$output = executeQuery('menu.insertMenuItem', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$oDB->commit();
 
@@ -532,46 +532,46 @@ 
 		$args->expand = $request->menu_expand;
 		$args->is_shortcut = $request->is_shortcut;
 
-		if(!$args->open_window) $args->open_window = 'N';
-		if(!$args->expand) $args->expand = 'N';
-		if(!$args->is_shortcut) $args->is_shortcut = 'N';
+		if (!$args->open_window) $args->open_window = 'N';
+		if (!$args->expand) $args->expand = 'N';
+		if (!$args->is_shortcut) $args->is_shortcut = 'N';
 
-		if($request->menu_name_key) $args->name = $request->menu_name_key;
+		if ($request->menu_name_key) $args->name = $request->menu_name_key;
 		else $args->name = $request->menu_name;
 
-		if($request->menu_desc) $args->desc = $request->menu_desc;
+		if ($request->menu_desc) $args->desc = $request->menu_desc;
 		else $args->desc = '';
 
-		if($request->module_id && strncasecmp('http', $request->module_id, 4) === 0)
+		if ($request->module_id && strncasecmp('http', $request->module_id, 4) === 0)
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
 
 		// when menu copy, module already copied
-		if($isProc)
+		if ($isProc)
 		{
 			$result = $this->_insertModule($request, $args);
-			if(!$result->toBool())
+			if (!$result->toBool())
 			{
 				return new BaseObject(-1, $result->message);
 			}
 		}
 
 		// if setting button variables, set argument button variables for db insert. but not upload in this method
-		if($request->normal_btn) $args->normal_btn = $request->normal_btn;
-		if($request->hover_btn) $args->hover_btn = $request->hover_btn;
-		if($request->active_btn) $args->active_btn = $request->active_btn;
+		if ($request->normal_btn) $args->normal_btn = $request->normal_btn;
+		if ($request->hover_btn) $args->hover_btn = $request->hover_btn;
+		if ($request->active_btn) $args->active_btn = $request->active_btn;
 
-		if(!$request->module_id)
+		if (!$request->module_id)
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
 
 		$args->url = $request->module_id;
 		$args->menu_item_srl = getNextSequence();
-		$args->listorder = -1*$args->menu_item_srl;
+		$args->listorder = -1 * $args->menu_item_srl;
 		$output = executeQuery('menu.insertMenuItem', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$oDB->commit();
 
@@ -603,7 +603,7 @@ 
 
 		//module create
 		$site_module_info = Context::get('site_module_info');
-		$cmArgs->site_srl = (int)$site_module_info->site_srl;
+		$cmArgs->site_srl = (int) $site_module_info->site_srl;
 		$cmArgs->browser_title = $args->name;
 		$cmArgs->menu_srl = $request->menu_srl;
 		$cmArgs->layout_srl = -1;
@@ -611,13 +611,13 @@ 
 		$cmArgs->is_skin_fix = 'N';
 		$cmArgs->is_mskin_fix = 'N';
 
-		if(Mobile::isMobileEnabled() === true)
+		if (Mobile::isMobileEnabled() === true)
 		{
 			$cmArgs->use_mobile = 'Y';
 		}
 
 		// if mid is empty, auto create mid
-		if(!$request->module_id)
+		if (!$request->module_id)
 		{
 			$randomMid = $this->_makeRandomMid();
 			$request->module_id = $cmArgs->module.'_'.$randomMid;
@@ -627,7 +627,7 @@ 
 		// check already created module instance
 		$oModuleModel = getModel('module');
 		$output = $oModuleModel->getModuleInfoByMid($request->module_id);
-		if($output->module_srl)
+		if ($output->module_srl)
 		{
 			return new BaseObject(-1, 'msg_module_name_exists');
 		}
@@ -646,14 +646,14 @@ 
 	{
 		$request = Context::getRequestVars();
 
-		if(!$request->menu_item_srl || !$request->menu_name)
+		if (!$request->menu_item_srl || !$request->menu_name)
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
 
 		// variables set
-		if($request->menu_open_window != "Y") $request->menu_open_window = "N";
-		if($request->menu_expand != "Y") $request->menu_expand = "N";
+		if ($request->menu_open_window != "Y") $request->menu_open_window = "N";
+		if ($request->menu_expand != "Y") $request->menu_expand = "N";
 
 		// Get original information
 		$oMenuAdminModel = getAdminModel('menu');
@@ -661,19 +661,19 @@ 
 		$args = $itemInfo;
 
 		// if menu type is module, check exists module and update
-		if($itemInfo->is_shortcut == 'Y')
+		if ($itemInfo->is_shortcut == 'Y')
 		{
 			// type is url
-			if(strncasecmp('http', $request->shortcut_target, 4) === 0 || preg_match('/^(\.\/|\.\.\/|\/).*$/', $request->shortcut_target))
+			if (strncasecmp('http', $request->shortcut_target, 4) === 0 || preg_match('/^(\.\/|\.\.\/|\/).*$/', $request->shortcut_target))
 			{
 				$args->url = $request->shortcut_target;
 			}
 			// type is module short cut
-			else if(is_numeric($request->shortcut_target))
+			else if (is_numeric($request->shortcut_target))
 			{
 				// Get new original information
 				$newItemInfo = $oMenuAdminModel->getMenuItemInfo($request->shortcut_target);
-				if(!$newItemInfo->menu_item_srl)
+				if (!$newItemInfo->menu_item_srl)
 				{
 					return new BaseObject(-1, 'msg_invalid_request');
 				}
@@ -690,10 +690,10 @@ 
 		{
 			// check already created module instance
 			$oModuleModel = getModel('module');
-			if($request->module_id != $itemInfo->url)
+			if ($request->module_id != $itemInfo->url)
 			{
 				$output = $oModuleModel->getModuleInfoByMid($request->module_id);
-				if($output->module_srl)
+				if ($output->module_srl)
 				{
 					return new BaseObject(-1, 'msg_module_name_exists');
 				}
@@ -701,13 +701,13 @@ 
 
 			// if not exist module, return error
 			$moduleInfo = $oModuleModel->getModuleInfoByMid($itemInfo->url);
-			if(!$moduleInfo)
+			if (!$moduleInfo)
 			{
 				return new BaseObject(-1, 'msg_invalid_request');
 			}
 
 			$moduleInfo->mid = $request->module_id;
-			if($request->browser_title)
+			if ($request->browser_title)
 			{
 				$moduleInfo->browser_title = $request->browser_title;
 			}
@@ -716,7 +716,7 @@ 
 			$args->url = $request->module_id;
 		}
 
-		if($request->menu_name_key)
+		if ($request->menu_name_key)
 		{
 			$args->name = $request->menu_name_key;
 		}
@@ -725,7 +725,7 @@ 
 			$args->name = $request->menu_name;
 		}
 
-		if($request->menu_desc) $args->desc = $request->menu_desc;
+		if ($request->menu_desc) $args->desc = $request->menu_desc;
 		else $args->desc = '';
 
 		unset($args->group_srls);
@@ -753,38 +753,38 @@ 
 
 		$btnOutput = $this->_uploadButton($args);
 
-		if($btnOutput['normal_btn'])
+		if ($btnOutput['normal_btn'])
 		{
 			$this->add('normal_btn', $btnOutput['normal_btn']);
 			$item_info->normal_btn = $btnOutput['normal_btn'];
 		}
-		if($btnOutput['hover_btn'])
+		if ($btnOutput['hover_btn'])
 		{
 			$this->add('hover_btn', $btnOutput['hover_btn']);
 			$item_info->hover_btn = $btnOutput['hover_btn'];
 		}
-		if($btnOutput['active_btn'])
+		if ($btnOutput['active_btn'])
 		{
 			$this->add('active_btn', $btnOutput['active_btn']);
 			$item_info->active_btn = $btnOutput['active_btn'];
 		}
 
 		// group_srls check
-		if(count($item_info->group_srls) == 0)
+		if (count($item_info->group_srls) == 0)
 		{
 			unset($item_info->group_srls);
 		}
 
 		// Button delete check
-		if(!$btnOutput['normal_btn'] && $args->isNormalDelete == 'Y')
+		if (!$btnOutput['normal_btn'] && $args->isNormalDelete == 'Y')
 		{
 			$item_info->normal_btn = '';
 		}
-		if(!$btnOutput['hover_btn'] && $args->isHoverDelete == 'Y')
+		if (!$btnOutput['hover_btn'] && $args->isHoverDelete == 'Y')
 		{
 			$item_info->hover_btn = '';
 		}
-		if(!$btnOutput['active_btn'] && $args->isActiveDelete == 'Y')
+		if (!$btnOutput['active_btn'] && $args->isActiveDelete == 'Y')
 		{
 			$item_info->active_btn = '';
 		}
@@ -824,7 +824,7 @@ 
 		$args->is_force = Context::get('is_force');
 
 		$returnObj = $this->deleteItem($args);
-		if(is_object($returnObj))
+		if (is_object($returnObj))
 		{
 			$this->setError($returnObj->error);
 			$this->setMessage($returnObj->message);
@@ -853,11 +853,11 @@ 
 		$args->menu_srl = $itemInfo->menu_srl;
 
 		// Display an error that the category cannot be deleted if it has a child node	603
-		if($args->is_force != 'Y')
+		if ($args->is_force != 'Y')
 		{
 			$output = executeQuery('menu.getChildMenuCount', $args);
-			if(!$output->toBool()) return $output;
-			if($output->data->count > 0)
+			if (!$output->toBool()) return $output;
+			if ($output->data->count > 0)
 			{
 				return new BaseObject(-1001, 'msg_cannot_delete_for_child');
 			}
@@ -869,22 +869,22 @@ 
 
 		// check admin menu delete
 		$oAdmin = getClass('admin');
-		if($menu_title == $oAdmin->getAdminMenuName() && $itemInfo->parent_srl == 0)
+		if ($menu_title == $oAdmin->getAdminMenuName() && $itemInfo->parent_srl == 0)
 		{
 			return $this->stop('msg_cannot_delete_for_admin_topmenu');
 		}
 
-		if($itemInfo->parent_srl) $parent_srl = $itemInfo->parent_srl;
+		if ($itemInfo->parent_srl) $parent_srl = $itemInfo->parent_srl;
 
 		// get menu properies with child menu
 		$phpFile = sprintf("./files/cache/menu/%s.php", $args->menu_srl);
 		$originMenu = NULL;
 
-		if(is_readable(FileHandler::getRealPath($phpFile)))
+		if (is_readable(FileHandler::getRealPath($phpFile)))
 		{
 			include(FileHandler::getRealPath($phpFile));
 
-			if(is_array($menu->list))
+			if (is_array($menu->list))
 			{
 				$this->_searchMenu($menu->list, $args->menu_item_srl, $originMenu);
 			}
@@ -894,7 +894,7 @@ 
 		$siteInfo = $oModuleModel->getSiteInfo($menuInfo->site_srl);
 		$isStartmenuInclude = false;
 		$this->_checkHomeMenuInOriginMenu($originMenu, $siteInfo->mid, $isStartmenuInclude);
-		if($isStartmenuInclude)
+		if ($isStartmenuInclude)
 		{
 			return new BaseObject(-1, 'msg_cannot_delete_homemenu');
 		}
@@ -918,14 +918,14 @@ 
 
 	private function _checkHomeMenuInOriginMenu($originMenu, $startMid, &$isStartmenuInclude)
 	{
-		if($originMenu['is_shortcut'] != 'Y' && $originMenu['url'] == $startMid)
+		if ($originMenu['is_shortcut'] != 'Y' && $originMenu['url'] == $startMid)
 		{
 			$isStartmenuInclude = true;
 		}
 
-		if(!$isStartmenuInclude && is_array($originMenu['list']))
+		if (!$isStartmenuInclude && is_array($originMenu['list']))
 		{
-			foreach($originMenu['list'] AS $key=>$value)
+			foreach ($originMenu['list'] AS $key=>$value)
 			{
 				$this->_checkHomeMenuInOriginMenu($value, $startMid, $isStartmenuInclude);
 			}
@@ -939,7 +939,7 @@ 
 		$args->menu_srl = $menuSrl;
 		$args->menu_item_srl = $node['node_srl'];
 		$output = executeQuery("menu.deleteMenuItem", $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
@@ -948,12 +948,12 @@ 
 		// Update the xml file and get its location
 		$xml_file = $this->makeXmlFile($args->menu_srl);
 		// Delete all of image buttons
-		if($node['normal_btn']) FileHandler::removeFile($node['normal_btn']);
-		if($node['hover_btn']) FileHandler::removeFile($node['hover_btn']);
-		if($node['active_btn']) FileHandler::removeFile($node['active_btn']);
+		if ($node['normal_btn']) FileHandler::removeFile($node['normal_btn']);
+		if ($node['hover_btn']) FileHandler::removeFile($node['hover_btn']);
+		if ($node['active_btn']) FileHandler::removeFile($node['active_btn']);
 
 		// Delete module
-		if($node['is_shortcut'] != 'Y' && strncasecmp('http', $node['url'], 4) !== 0)
+		if ($node['is_shortcut'] != 'Y' && strncasecmp('http', $node['url'], 4) !== 0)
 		{
 			$oModuleController = getController('module');
 			$oModuleModel = getModel('module');
@@ -963,12 +963,12 @@ 
 			$args->site_srl = $menuInfo->site_srl;
 			$args->is_shortcut = 'Y';
 			$output = executeQuery('menu.getMenuItemByUrl', $args);
-			if($output->data->menu_item_srl)
+			if ($output->data->menu_item_srl)
 			{
 				$output->data->url = '';
 				$referenceItem = $output->data;
 				$output = $this->_updateMenuItem($referenceItem);
-				if(!$output->toBool())
+				if (!$output->toBool())
 				{
 					$oDB->rollback();
 					return $output;
@@ -976,10 +976,10 @@ 
 			}
 
 			$moduleInfo = $oModuleModel->getModuleInfoByMid($node['url'], $menuInfo->site_srl);
-			if($moduleInfo->module_srl)
+			if ($moduleInfo->module_srl)
 			{
 				$output = $oModuleController->onlyDeleteModule($moduleInfo->module_srl);
-				if(!$output->toBool())
+				if (!$output->toBool())
 				{
 					$oDB->rollback();
 					return $output;
@@ -992,14 +992,14 @@ 
 	private function _recursiveDeleteMenuItem(&$oDB, &$menuInfo, $node)
 	{
 		$output = $this->_deleteMenuItem($oDB, $menuInfo, $node);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return new BaseObject(-1, $output->message);
 		}
 
-		if(is_array($node['list']))
+		if (is_array($node['list']))
 		{
-			foreach($node['list'] AS $key=>$value)
+			foreach ($node['list'] AS $key=>$value)
 			{
 				$this->_recursiveDeleteMenuItem($oDB, $menuInfo, $value);
 			}
@@ -1012,31 +1012,31 @@ 
 	 */
 	function procMenuAdminMoveItem()
 	{
-		$mode = Context::get('mode');	//move
-		$parent_srl = Context::get('parent_srl');	// Parent menu item serial number
-		$source_srl = Context::get('source_srl');	// Same hierarchy's menu item serial number
-		$target_srl = Context::get('target_srl');	// Self menu item serial number
+		$mode = Context::get('mode'); //move
+		$parent_srl = Context::get('parent_srl'); // Parent menu item serial number
+		$source_srl = Context::get('source_srl'); // Same hierarchy's menu item serial number
+		$target_srl = Context::get('target_srl'); // Self menu item serial number
 
-		if(!$mode || !$parent_srl || !$target_srl) return new BaseObject(-1,'msg_invalid_request');
+		if (!$mode || !$parent_srl || !$target_srl) return new BaseObject(-1, 'msg_invalid_request');
 
 		$oMenuAdminModel = getAdminModel('menu');
 
 		// get original menu item info for cache file recreate
 		$originalItemInfo = $oMenuAdminModel->getMenuItemInfo($target_srl);
-		if(!$originalItemInfo->menu_item_srl)
+		if (!$originalItemInfo->menu_item_srl)
 		{
 			return new BaseObject(-1, 'msg_empty_menu_item');
 		}
 
 		// get menu properies with child menu
-		$phpFile = sprintf(_XE_PATH_ . "files/cache/menu/%s.php", $originalItemInfo->menu_srl);
+		$phpFile = sprintf(_XE_PATH_."files/cache/menu/%s.php", $originalItemInfo->menu_srl);
 		$originMenu = NULL;
 
-		if(is_readable(FileHandler::getRealPath($phpFile)))
+		if (is_readable(FileHandler::getRealPath($phpFile)))
 		{
 			include(FileHandler::getRealPath($phpFile));
 
-			if(is_array($menu->list))
+			if (is_array($menu->list))
 			{
 				$this->_searchMenu($menu->list, $originalItemInfo->menu_item_srl, $originMenu);
 			}
@@ -1045,7 +1045,7 @@ 
 		// get target menu info for move
 		$targetMenuItemInfo = $oMenuAdminModel->getMenuItemInfo($parent_srl);
 		// if move in same sitemap
-		if($targetMenuItemInfo->menu_item_srl)
+		if ($targetMenuItemInfo->menu_item_srl)
 		{
 			$menu_srl = $targetMenuItemInfo->menu_srl;
 		}
@@ -1057,20 +1057,20 @@ 
 			$parent_srl = 0;
 		}
 
-		if(!$this->homeModuleMid)
+		if (!$this->homeModuleMid)
 		{
 			$oModuleModel = getModel('module');
 			$oMenuAdminController = getAdminController('menu');
 			$columnList = array('modules.mid',);
 			$output = $oModuleModel->getSiteInfo(0, $columnList);
-			if($output->mid)
+			if ($output->mid)
 			{
 				$this->homeModuleMid = $output->mid;
 			}
 		}
 
 		$this->moveMenuItem($menu_srl, $parent_srl, $source_srl, $target_srl, $mode, $originMenu['is_shortcut'], $originMenu['url']);
-		if(count($originMenu['list']) > 0)
+		if (count($originMenu['list']) > 0)
 		{
 			$this->_recursiveUpdateMenuItem($originMenu['list'], $menu_srl);
 		}
@@ -1084,9 +1084,9 @@ 
 
 	private function _recursiveUpdateMenuItem($node, $menu_srl)
 	{
-		if(is_array($node))
+		if (is_array($node))
 		{
-			foreach($node AS $key=>$node)
+			foreach ($node AS $key=>$node)
 			{
 				$args = new stdClass();
 				$args->menu_srl = $menu_srl;
@@ -1094,11 +1094,11 @@ 
 				$output = $this->_updateMenuItem($args);
 
 				//module's menu_srl move also
-				if($node['is_shortcut'] == 'N' && !empty($node['url']))
+				if ($node['is_shortcut'] == 'N' && !empty($node['url']))
 				{
 					$oModuleModel = getModel('module');
 					$moduleInfo = $oModuleModel->getModuleInfoByMid($node['url']);
-					if($menu_srl != $moduleInfo->menu_srl)
+					if ($menu_srl != $moduleInfo->menu_srl)
 					{
 						$moduleInfo->menu_srl = $menu_srl;
 						$oModuleController = getController('module');
@@ -1106,7 +1106,7 @@ 
 					}
 				}
 
-				if(count($node['list']) > 0)
+				if (count($node['list']) > 0)
 				{
 					$this->_recursiveUpdateMenuItem($node['list'], $menu_srl);
 				}
@@ -1128,21 +1128,21 @@ 
 		$menuSrl = $itemInfo->menu_srl;
 
 		// get menu properies with child menu
-		$phpFile = sprintf(_XE_PATH_ . "files/cache/menu/%s.php", $menuSrl);
+		$phpFile = sprintf(_XE_PATH_."files/cache/menu/%s.php", $menuSrl);
 		$originMenu = NULL;
 
-		if(is_readable(FileHandler::getRealPath($phpFile)))
+		if (is_readable(FileHandler::getRealPath($phpFile)))
 		{
 			include(FileHandler::getRealPath($phpFile));
 
-			if(is_array($menu->list))
+			if (is_array($menu->list))
 			{
 				$this->_searchMenu($menu->list, $menuItemSrl, $originMenu);
 			}
 		}
 
 		// copy the menu item with recursively
-		if(is_array($originMenu))
+		if (is_array($originMenu))
 		{
 			$this->_copyMenu($menuSrl, $parentSrl, $originMenu);
 		}
@@ -1158,15 +1158,15 @@ 
 	 */
 	private function _searchMenu(&$menuList, $menuItemSrl, &$originMenu)
 	{
-		if(array_key_exists($menuItemSrl, $menuList))
+		if (array_key_exists($menuItemSrl, $menuList))
 		{
 			$originMenu = $menuList[$menuItemSrl];
 			return;
 		}
 
-		foreach($menuList AS $key=>$value)
+		foreach ($menuList AS $key=>$value)
 		{
-			if(count($value['list']) > 0)
+			if (count($value['list']) > 0)
 			{
 				$this->_searchMenu($value['list'], $menuItemSrl, $originMenu);
 			}
@@ -1181,7 +1181,7 @@ 
 		// default argument setting
 		$args = new stdClass();
 		$args->menu_srl = $menuSrl;
-		if($parentSrl == 0) $args->parent_srl = $menuSrl;
+		if ($parentSrl == 0) $args->parent_srl = $menuSrl;
 		else $args->parent_srl = $parentSrl;
 		$args->menu_name_key = $originMenu['text'];
 		$args->menu_name = $originMenu['text'];
@@ -1194,7 +1194,7 @@ 
 
 		$isModuleCopySuccess = false;
 		// if menu have a reference of module instance
-		if($menuItemInfo->is_shortcut == 'N' && strncasecmp('http', $originMenu['url'], 4) !== 0 )
+		if ($menuItemInfo->is_shortcut == 'N' && strncasecmp('http', $originMenu['url'], 4) !== 0)
 		{
 			$oModuleModel = getModel('module');
 			$moduleInfo = $oModuleModel->getModuleInfoByMid($originMenu['url']);
@@ -1214,22 +1214,22 @@ 
 
 			$args->module_srl = $copiedModuleSrl;
 
-			if($copiedModuleSrl)
+			if ($copiedModuleSrl)
 			{
 				$isModuleCopySuccess = true;
 			}
 		}
 		// if menu type is shortcut
-		else if($menuItemInfo->is_shortcut == 'Y')
+		else if ($menuItemInfo->is_shortcut == 'Y')
 		{
 			$args->shortcut_target = $originMenu['url'];
 			$isModuleCopySuccess = true;
 		}
 
-		if($isModuleCopySuccess)
+		if ($isModuleCopySuccess)
 		{
 			// if have a group permission
-			if($menuItemInfo->group_srls)
+			if ($menuItemInfo->group_srls)
 			{
 				$args->group_srls = $menuItemInfo->group_srls;
 			}
@@ -1243,12 +1243,12 @@ 
 
 			// if have a button, copy a button image also
 			$insertedMenuItemSrl = $this->get('menu_item_srl');
-			if($menuItemInfo->normal_btn || $menuItemInfo->hover_btn || $menuItemInfo->active_btn)
+			if ($menuItemInfo->normal_btn || $menuItemInfo->hover_btn || $menuItemInfo->active_btn)
 			{
 				// copy & upate
 				$update_item_info = $oMenuAdminModel->getMenuItemInfo($insertedMenuItemSrl);
-				$copied_info = $this->_copyButton($insertedMenuItemSrl,$update_item_info->menu_srl, $menuItemInfo);
-				if(count($update_item_info->group_srls) == 0)
+				$copied_info = $this->_copyButton($insertedMenuItemSrl, $update_item_info->menu_srl, $menuItemInfo);
+				if (count($update_item_info->group_srls) == 0)
 				{
 					unset($update_item_info->group_srls);
 				}
@@ -1262,7 +1262,7 @@ 
 
 		// if have a child menu, copy child menu also
 		$childMenu = array_shift($originMenu['list']);
-		if(count($childMenu) > 0)
+		if (count($childMenu) > 0)
 		{
 			$this->_copyMenu($menuSrl, $insertedMenuItemSrl, $childMenu);
 		}
@@ -1272,10 +1272,10 @@ 
 	{
 		$time = $_SERVER['REQUEST_TIME'];
 		$randomString = "";
-		for($i=0;$i<4;$i++)
+		for ($i = 0; $i < 4; $i++)
 		{
 			$case = rand(0, 1);
-			if($case) $doc = rand(65, 90);
+			if ($case) $doc = rand(65, 90);
 			else $doc = rand(97, 122);
 
 			$randomString .= chr($doc);
@@ -1299,32 +1299,32 @@ 
 		// menu name update
 		$args->menu_srl = $this->menuSrl;
 		$output = executeQuery('menu.updateMenu', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$this->map = array();
-		if(is_array($parentKeyList))
+		if (is_array($parentKeyList))
 		{
-			foreach($parentKeyList as $no=>$srl)
+			foreach ($parentKeyList as $no=>$srl)
 			{
-				if($srl === 0) continue;
-				if(!is_array($this->map[$srl]))$this->map[$srl] = array();
+				if ($srl === 0) continue;
+				if (!is_array($this->map[$srl]))$this->map[$srl] = array();
 				$this->map[$srl][] = $no;
 			}
 		}
 
 		$result = array();
-		if(is_array($this->itemKeyList))
+		if (is_array($this->itemKeyList))
 		{
-			foreach($this->itemKeyList as $srl)
+			foreach ($this->itemKeyList as $srl)
 			{
-				if(!$this->checked[$srl])
+				if (!$this->checked[$srl])
 				{
 					$target = new stdClass();
 					$this->checked[$srl] = 1;
 					$target->node = $srl;
-					$target->child= array();
+					$target->child = array();
 
-					while(count($this->map[$srl]))
+					while (count($this->map[$srl]))
 					{
 						$this->_setParent($srl, array_shift($this->map[$srl]), $target);
 					}
@@ -1333,13 +1333,13 @@ 
 			}
 		}
 
-		if(is_array($result))
+		if (is_array($result))
 		{
 			$i = 0;
-			foreach($result AS $key=>$node)
+			foreach ($result AS $key=>$node)
 			{
-				$this->moveMenuItem($this->menuSrl, 0, $i, $node->node, 'move');	//move parent node
-				$this->_recursiveMoveMenuItem($node);	//move child node
+				$this->moveMenuItem($this->menuSrl, 0, $i, $node->node, 'move'); //move parent node
+				$this->_recursiveMoveMenuItem($node); //move child node
 				$i = $node->node;
 			}
 		}
@@ -1368,7 +1368,7 @@ 
 		$child_node->child = array();
 		$target->child[] = $child_node;
 
-		while(count($this->map[$child_srl]))
+		while (count($this->map[$child_srl]))
 		{
 			$this->_setParent($child_srl, array_shift($this->map[$child_srl]), $child_node);
 		}
@@ -1383,7 +1383,7 @@ 
 	function _recursiveMoveMenuItem($result)
 	{
 		$i = 0;
-		while(count($result->child))
+		while (count($result->child))
 		{
 			unset($node);
 			$node = array_shift($result->child);
@@ -1403,48 +1403,48 @@ 
 	 * @param string $mode 'move' or 'insert'
 	 * @return void
 	 */
-	function moveMenuItem($menu_srl, $parent_srl, $source_srl, $target_srl, $mode, $isShortcut='Y', $url=NULL)
+	function moveMenuItem($menu_srl, $parent_srl, $source_srl, $target_srl, $mode, $isShortcut = 'Y', $url = NULL)
 	{
 		// Get the original menus
 		$oMenuAdminModel = getAdminModel('menu');
 
 		$target_item = $oMenuAdminModel->getMenuItemInfo($target_srl);
-		if($target_item->menu_item_srl != $target_srl) return new BaseObject(-1,'msg_invalid_request');
+		if ($target_item->menu_item_srl != $target_srl) return new BaseObject(-1, 'msg_invalid_request');
 		// Move the menu location(change the order menu appears)
-		if($mode == 'move')
+		if ($mode == 'move')
 		{
 			$args = new stdClass();
 			$args->parent_srl = $parent_srl;
 			$args->menu_srl = $menu_srl;
 
-			if($source_srl)
+			if ($source_srl)
 			{
 				$source_item = $oMenuAdminModel->getMenuItemInfo($source_srl);
-				if($source_item->menu_item_srl != $source_srl) return new BaseObject(-1,'msg_invalid_request');
-				$args->listorder = $source_item->listorder-1;
+				if ($source_item->menu_item_srl != $source_srl) return new BaseObject(-1, 'msg_invalid_request');
+				$args->listorder = $source_item->listorder - 1;
 			}
 			else
 			{
 				$output = executeQuery('menu.getMaxListorder', $args);
-				if(!$output->toBool()) return $output;
-				$args->listorder = (int)$output->data->listorder;
-				if(!$args->listorder) $args->listorder= 0;
+				if (!$output->toBool()) return $output;
+				$args->listorder = (int) $output->data->listorder;
+				if (!$args->listorder) $args->listorder = 0;
 			}
 			$args->parent_srl = $parent_srl;
 			$output = executeQuery('menu.updateMenuItemListorder', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 
 			$args->parent_srl = $parent_srl;
 			$args->menu_item_srl = $target_srl;
 			$output = executeQuery('menu.updateMenuItemNode', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 
 			//module's menu_srl move also
-			if($isShortcut == 'N' && !empty($url))
+			if ($isShortcut == 'N' && !empty($url))
 			{
 				$oModuleModel = getModel('module');
 				$moduleInfo = $oModuleModel->getModuleInfoByMid($url);
-				if($menu_srl != $moduleInfo->menu_srl)
+				if ($menu_srl != $moduleInfo->menu_srl)
 				{
 					$moduleInfo->menu_srl = $menu_srl;
 					$oModuleController = getController('module');
@@ -1452,13 +1452,13 @@ 
 				}
 
 				// change home menu cache file
-				if($url == $this->homeModuleMid)
+				if ($url == $this->homeModuleMid)
 				{
-					if(file_exists($this->homeMenuCacheFile))
+					if (file_exists($this->homeMenuCacheFile))
 					{
 						include($this->homeMenuCacheFile);
 					}
-					if(!$homeMenuSrl || $homeMenuSrl != $menu_srl)
+					if (!$homeMenuSrl || $homeMenuSrl != $menu_srl)
 					{
 						$this->makeHomemenuCacheFile($menu_srl);
 					}
@@ -1466,13 +1466,13 @@ 
 			}
 			// Add a child
 		}
-		elseif($mode == 'insert')
+		elseif ($mode == 'insert')
 		{
 			$args->menu_item_srl = $target_srl;
 			$args->parent_srl = $parent_srl;
-			$args->listorder = -1*getNextSequence();
+			$args->listorder = -1 * getNextSequence();
 			$output = executeQuery('menu.updateMenuItemNode', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 		}
 
 		$xml_file = $this->makeXmlFile($menu_srl);
@@ -1497,8 +1497,8 @@ 
 		// Re-generate the xml file
 		$xml_file = $this->makeXmlFile($menu_srl);
 		// Set return value
-		$this->add('menu_title',$menu_title);
-		$this->add('xml_file',$xml_file);
+		$this->add('menu_title', $menu_title);
+		$this->add('xml_file', $xml_file);
 	}
 
 	/**
@@ -1512,12 +1512,12 @@ 
 		$target = Context::get('target');
 		$target_file = Context::get($target);
 		// Error occurs when the target is neither a uploaded file nor a valid file
-		if(!$menu_srl || !$menu_item_srl)
+		if (!$menu_srl || !$menu_item_srl)
 		{
 			Context::set('error_messge', Context::getLang('msg_invalid_request'));
 
 		}
-		else if(!$target_file || !is_uploaded_file($target_file['tmp_name']) || !preg_match('/\.(gif|jpeg|jpg|png)$/i',$target_file['name'])  || !checkUploadedFile($target_file['tmp_name']))
+		else if (!$target_file || !is_uploaded_file($target_file['tmp_name']) || !preg_match('/\.(gif|jpeg|jpg|png)$/i', $target_file['name']) || !checkUploadedFile($target_file['tmp_name']))
 		{
 			Context::set('error_messge', Context::getLang('msg_invalid_request'));
 		}
@@ -1525,13 +1525,13 @@ 
 		// Move the file to a specific director if the uploaded file meets requirement
 		else
 		{
-			$tmp_arr = explode('.',$target_file['name']);
-			$ext = $tmp_arr[count($tmp_arr)-1];
+			$tmp_arr = explode('.', $target_file['name']);
+			$ext = $tmp_arr[count($tmp_arr) - 1];
 
 			$path = sprintf('./files/attach/menu_button/%d/', $menu_srl);
 			$filename = sprintf('%s%d.%s.%s', $path, $menu_item_srl, $target, $ext);
 
-			if(!is_dir($path)) FileHandler::makeDir($path);
+			if (!is_dir($path)) FileHandler::makeDir($path);
 
 			move_uploaded_file($target_file['tmp_name'], $filename);
 			Context::set('filename', $filename);
@@ -1564,14 +1564,14 @@ 
 	{
 		$oModuleModel = getModel('module');
 		$installed_module_list = $oModuleModel->getModulesXmlInfo();
-		if(is_array($installed_module_list))
+		if (is_array($installed_module_list))
 		{
 			$currentLang = Context::getLangType();
 			$menuList = array();
-			foreach($installed_module_list AS $key=>$value)
+			foreach ($installed_module_list AS $key=>$value)
 			{
 				$info = $oModuleModel->getModuleActionXml($value->module);
-				if($info->menu) $menuList[$value->module] = $info->menu;
+				if ($info->menu) $menuList[$value->module] = $info->menu;
 				unset($info->menu);
 			}
 		}
@@ -1603,8 +1603,8 @@ 
 		$info = $oModuleModel->getModuleActionXml($moduleName);
 
 		$url = getNotEncodedFullUrl('', 'module', 'admin', 'act', $info->menu->{$menuName}->index);
-		if(empty($url)) $url = getNotEncodedFullUrl('', 'module', 'admin', 'act', $info->admin_index_act);
-		if(empty($url)) $url = getNotEncodedFullUrl('', 'module', 'admin');
+		if (empty($url)) $url = getNotEncodedFullUrl('', 'module', 'admin', 'act', $info->admin_index_act);
+		if (empty($url)) $url = getNotEncodedFullUrl('', 'module', 'admin');
 		$dbInfo = Context::getDBInfo();
 
 		$args = new stdClass();
@@ -1613,7 +1613,7 @@ 
 		$args->menu_srl = $requestArgs->menu_srl;
 		$args->name = sprintf('{$lang->menu_gnb_sub[\'%s\']}', $menuName);
 		//if now page is https...
-		if(strpos($url, 'https') !== false)
+		if (strpos($url, 'https') !== false)
 		{
 			$args->url = str_replace('https'.substr($dbInfo->default_url, 4), '', $url);
 		}
@@ -1627,23 +1627,23 @@ 
 		$args->hover_btn = '';
 		$args->active_btn = '';
 		$args->group_srls = implode(',', array_keys($groupSrlList));
-		$args->listorder = -1*$args->menu_item_srl;
+		$args->listorder = -1 * $args->menu_item_srl;
 
 		// Check if already exists
 		$oMenuModel = getAdminModel('menu');
 		$item_info = $oMenuModel->getMenuItemInfo($args->menu_item_srl);
 		// Update if exists
-		if($item_info->menu_item_srl == $args->menu_item_srl)
+		if ($item_info->menu_item_srl == $args->menu_item_srl)
 		{
 			$output = $this->_updateMenuItem($args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 		}
 		// Insert if not exist
 		else
 		{
-			$args->listorder = -1*$args->menu_item_srl;
+			$args->listorder = -1 * $args->menu_item_srl;
 			$output = executeQuery('menu.insertMenuItem', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 		}
 		// Get information of the menu
 		$menu_info = $oMenuModel->getMenu($args->menu_srl);
@@ -1671,23 +1671,23 @@ 
 
 		// Menu Exposure update
 		// if exposure target is only login user...
-		if(!$exposure)
+		if (!$exposure)
 		{
 			$args->group_srls = '';
 		}
 		else
 		{
 			$exposure = explode(',', $exposure);
-			if(in_array($exposure, array('-1','-3')))
+			if (in_array($exposure, array('-1', '-3')))
 			{
 				$args->group_srls = $exposure;
 			}
 
-			if($exposure) $args->group_srls = implode(',', $exposure);
+			if ($exposure) $args->group_srls = implode(',', $exposure);
 		}
 
 		$output = $this->_updateMenuItem($args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -1701,21 +1701,21 @@ 
 		$moduleInfo = $oModuleModel->getModuleInfoByMid($itemInfo->url, $menuInfo->site_srl);
 		$xml_info = $oModuleModel->getModuleActionXML($moduleInfo->module);
 
-		if($itemInfo->is_shortcut === 'Y')
+		if ($itemInfo->is_shortcut === 'Y')
 		{
 			$moduleGrnatsArgs = new stdClass;
 			$moduleGrnatsArgs->module_srl = $moduleInfo->module_srl;
 			$output = executeQueryArray('module.getModuleGrants', $moduleGrnatsArgs);
-			if(!$output->data) $output->data = array();
+			if (!$output->data) $output->data = array();
 			$moduleGrnats = new stdClass();
-			foreach($output->data as $grant)
+			foreach ($output->data as $grant)
 			{
 				$moduleGrnats->{$grant->name}[] = $grant->group_srl;
 			}
 		}
 
 		$grantList = $xml_info->grant;
-		if(!$grantList) $grantList = new stdClass;
+		if (!$grantList) $grantList = new stdClass;
 
 		$grantList->access = new stdClass();
 		$grantList->access->default = 'guest';
@@ -1723,14 +1723,14 @@ 
 		$grantList->manager->default = 'manager';
 
 		$grant = new stdClass();
-		foreach($grantList AS $grantName=>$grantInfo)
+		foreach ($grantList AS $grantName=>$grantInfo)
 		{
-			if($htPerm[$grantName])
+			if ($htPerm[$grantName])
 			{
 				$htPerm[$grantName] = explode(',', $htPerm[$grantName]);
 
 				// users in a particular group
-				if(is_array($htPerm[$grantName]))
+				if (is_array($htPerm[$grantName]))
 				{
 					$grant->{$grantName} = $htPerm[$grantName];
 					continue;
@@ -1742,13 +1742,13 @@ 
 					continue;
 				}
 			}
-			else if($itemInfo->is_shortcut === 'Y')
+			else if ($itemInfo->is_shortcut === 'Y')
 			{
-				if(isset($moduleGrnats) && $moduleGrnats->{$grantName}) $grant->{$grantName} = $moduleGrnats->{$grantName};
+				if (isset($moduleGrnats) && $moduleGrnats->{$grantName}) $grant->{$grantName} = $moduleGrnats->{$grantName};
 			}
 		}
 
-		if(count($grant))
+		if (count($grant))
 		{
 			$oModuleController = getController('module');
 			$oModuleController->insertModuleGrants($moduleInfo->module_srl, $grant);
@@ -1766,15 +1766,15 @@ 
 	function makeXmlFile($menu_srl)
 	{
 		// Return if there is no information when creating the xml file
-		if(!$menu_srl) return;
+		if (!$menu_srl) return;
 		// Get menu informaton
 		$args = new stdClass();
 		$args->menu_srl = $menu_srl;
 		$output = executeQuery('menu.getMenu', $args);
-		if(!$output->toBool() || !$output->data) return $output;
-		$site_srl = (int)$output->data->site_srl;
+		if (!$output->toBool() || !$output->data) return $output;
+		$site_srl = (int) $output->data->site_srl;
 
-		if($site_srl)
+		if ($site_srl)
 		{
 			$oModuleModel = getModel('module');
 			$columnList = array('sites.domain');
@@ -1785,13 +1785,13 @@ 
 		$args->menu_srl = $menu_srl;
 		$args->sort_index = 'listorder';
 		$output = executeQuery('menu.getMenuItems', $args);
-		if(!$output->toBool()) return;
+		if (!$output->toBool()) return;
 		// Specify the name of the cache file
-		$xml_file = sprintf(_XE_PATH_ . "files/cache/menu/%s.xml.php", $menu_srl);
-		$php_file = sprintf(_XE_PATH_ . "files/cache/menu/%s.php", $menu_srl);
+		$xml_file = sprintf(_XE_PATH_."files/cache/menu/%s.xml.php", $menu_srl);
+		$php_file = sprintf(_XE_PATH_."files/cache/menu/%s.php", $menu_srl);
 		// If no data found, generate an XML file without node data
 		$list = $output->data;
-		if(!$list)
+		if (!$list)
 		{
 			$xml_buff = "<root />";
 			FileHandler::writeFile($xml_file, $xml_buff);
@@ -1799,10 +1799,10 @@ 
 			return $xml_file;
 		}
 		// Change to an array if only a single data is obtained
-		if(!is_array($list)) $list = array($list);
+		if (!is_array($list)) $list = array($list);
 		// Create a tree for loop
 		$list_count = count($list);
-		for($i=0;$i<$list_count;$i++)
+		for ($i = 0; $i < $list_count; $i++)
 		{
 			$node = $list[$i];
 			$menu_item_srl = $node->menu_item_srl;
@@ -1858,7 +1858,7 @@ 
 		$php_buff = sprintf(
 			'<?php '.
 			'if(!defined("__XE__")) exit(); '.
-			'$menu = new stdClass();' .
+			'$menu = new stdClass();'.
 			'%s; '.
 			'%s; '.
 			'$menu->list = array(%s); '.
@@ -1887,26 +1887,26 @@ 
 	 */
 	function getXmlTree($source_node, $tree, $site_srl, $domain)
 	{
-		if(!$source_node) return;
+		if (!$source_node) return;
 
 		$oMenuAdminModel = getAdminModel('menu');
 
-		foreach($source_node as $menu_item_srl => $node)
+		foreach ($source_node as $menu_item_srl => $node)
 		{
 			$child_buff = "";
 			// Get data of the child nodes
-			if($menu_item_srl&&$tree[$menu_item_srl]) $child_buff = $this->getXmlTree($tree[$menu_item_srl], $tree, $site_srl, $domain);
+			if ($menu_item_srl && $tree[$menu_item_srl]) $child_buff = $this->getXmlTree($tree[$menu_item_srl], $tree, $site_srl, $domain);
 			// List variables
 			$names = $oMenuAdminModel->getMenuItemNames($node->name, $site_srl);
-			foreach($names as $key => $val)
+			foreach ($names as $key => $val)
 			{
-				$name_arr_str .= sprintf('"%s"=>%s,',$key, var_export($val, true));
+				$name_arr_str .= sprintf('"%s"=>%s,', $key, var_export($val, true));
 			}
 			$name_str = sprintf('$_names = array(%s); print $_names[$lang_type];', $name_arr_str);
 
-			$url = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$node->url);
-			$desc = str_replace(array('&','"',"'"),array('&amp;','&quot;','\\\''),$node->desc);
-			if(preg_match('/^([0-9a-zA-Z\_\-]+)$/', $node->url))
+			$url = str_replace(array('&', '"', '<', '>'), array('&amp;', '&quot;', '&lt;', '&gt;'), $node->url);
+			$desc = str_replace(array('&', '"', "'"), array('&amp;', '&quot;', '\\\''), $node->desc);
+			if (preg_match('/^([0-9a-zA-Z\_\-]+)$/', $node->url))
 			{
 				$href = "getSiteUrl('$domain', '','mid','$node->url')";
 			}
@@ -1916,21 +1916,21 @@ 
 			$expand = ($node->expand) ? $node->expand : '';
 
 			$normal_btn = ($node->normal_btn) ? $node->normal_btn : '';
-			if($normal_btn && strncasecmp('./files/attach/menu_button', $normal_btn, 26) === 0) $normal_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$normal_btn);
+			if ($normal_btn && strncasecmp('./files/attach/menu_button', $normal_btn, 26) === 0) $normal_btn = str_replace(array('&', '"', '<', '>'), array('&amp;', '&quot;', '&lt;', '&gt;'), $normal_btn);
 			else $normal_btn = '';
 			$hover_btn = ($node->hover_btn) ? $node->hover_btn : '';
-			if($hover_btn && strncasecmp('./files/attach/menu_button', $hover_btn, 26) === 0) $hover_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$hover_btn);
+			if ($hover_btn && strncasecmp('./files/attach/menu_button', $hover_btn, 26) === 0) $hover_btn = str_replace(array('&', '"', '<', '>'), array('&amp;', '&quot;', '&lt;', '&gt;'), $hover_btn);
 			else $hover_btn = '';
 			$active_btn = ($node->active_btn) ? $node->active_btn : '';
-			if($active_btn && strncasecmp('./files/attach/menu_button', $active_btn, 26) === 0) $active_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$active_btn);
+			if ($active_btn && strncasecmp('./files/attach/menu_button', $active_btn, 26) === 0) $active_btn = str_replace(array('&', '"', '<', '>'), array('&amp;', '&quot;', '&lt;', '&gt;'), $active_btn);
 			else $active_btn = '';
 
 			$group_srls = ($node->group_srls) ? $node->group_srls : '';
 
-			if($normal_btn)
+			if ($normal_btn)
 			{
-				if($hover_btn) $hover_str = sprintf('onmouseover=&quot;this.src=\'%s\'&quot;', $hover_btn); else $hover_str = '';
-				if($active_btn) $active_str = sprintf('onmousedown=&quot;this.src=\'%s\'&quot;', $active_btn); else $active_str = '';
+				if ($hover_btn) $hover_str = sprintf('onmouseover=&quot;this.src=\'%s\'&quot;', $hover_btn); else $hover_str = '';
+				if ($active_btn) $active_str = sprintf('onmousedown=&quot;this.src=\'%s\'&quot;', $active_btn); else $active_str = '';
 				$link = sprintf('&lt;img src=&quot;%s&quot; onmouseout=&quot;this.src=\'%s\'&quot; alt=&quot;<?php print htmlspecialchars($_names[$lang_type], ENT_COMPAT | ENT_HTML401, \'UTF-8\', false) ?>&quot; %s %s /&gt;', $normal_btn, $normal_btn, $hover_str, $active_str);
 			}
 			else
@@ -1939,8 +1939,8 @@ 
 			}
 
 			// If the value of node->group_srls exists
-			if($group_srls) {
-				$group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s))))||($is_logged&&%s))',$group_srls,$group_srls == -1?1:0);
+			if ($group_srls) {
+				$group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s))))||($is_logged&&%s))', $group_srls, $group_srls == -1 ? 1 : 0);
 			}
 			else
 			{
@@ -1969,8 +1969,8 @@ 
 				$link
 			);
 
-			if($child_buff) $buff .= sprintf('<node %s>%s</node>', $attribute, $child_buff);
-			else $buff .=  sprintf('<node %s />', $attribute);
+			if ($child_buff) $buff .= sprintf('<node %s>%s</node>', $attribute, $child_buff);
+			else $buff .= sprintf('<node %s />', $attribute);
 		}
 		return $buff;
 	}
@@ -1989,84 +1989,84 @@ 
 	function getPhpCacheCode($source_node, $tree, $site_srl, $domain)
 	{
 		$output = array("buff"=>"", "url_list"=>array());
-		if(!$source_node) return $output;
+		if (!$source_node) return $output;
 
 		$oMenuAdminModel = getAdminModel('menu');
 
-		foreach($source_node as $menu_item_srl => $node)
+		foreach ($source_node as $menu_item_srl => $node)
 		{
 			// Get data from child nodes if exist.
-			if($menu_item_srl&&$tree[$menu_item_srl]) $child_output = $this->getPhpCacheCode($tree[$menu_item_srl], $tree, $site_srl, $domain);
+			if ($menu_item_srl && $tree[$menu_item_srl]) $child_output = $this->getPhpCacheCode($tree[$menu_item_srl], $tree, $site_srl, $domain);
 			else $child_output = array("buff"=>"", "url_list"=>array());
 
 			// List variables
 			$names = $oMenuAdminModel->getMenuItemNames($node->name, $site_srl);
 			unset($name_arr_str);
-			foreach($names as $key => $val)
+			foreach ($names as $key => $val)
 			{
-				if(preg_match('/\{\$lang->menu_gnb(?:_sub)?\[\'([a-zA-Z_]+)\'\]\}/', $val) === 1)
+				if (preg_match('/\{\$lang->menu_gnb(?:_sub)?\[\'([a-zA-Z_]+)\'\]\}/', $val) === 1)
 				{
 					$name_arr_str .= sprintf('"%s"=>"%s",', $key, $val);
 				}
 				else
 				{
-					$name_arr_str .= sprintf('"%s"=>\'%s\',', $key, str_replace(array('\\','\''), array('\\\\','\\\''), removeHackTag($val)));
+					$name_arr_str .= sprintf('"%s"=>\'%s\',', $key, str_replace(array('\\', '\''), array('\\\\', '\\\''), removeHackTag($val)));
 				}
 			}
 			$name_str = sprintf('$_menu_names[%d] = array(%s); %s', $node->menu_item_srl, $name_arr_str, $child_output['name']);
 
 			// If url value is not empty in the current node, put the value into an array url_list
-			if($node->url) $child_output['url_list'][] = $node->url;
+			if ($node->url) $child_output['url_list'][] = $node->url;
 			$output['url_list'] = array_merge($output['url_list'], $child_output['url_list']);
 			// If node->group_srls value exists
-			if($node->group_srls)$group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s))))||($is_logged && %s))',$node->group_srls,$node->group_srls == -1?1:0);
+			if ($node->group_srls)$group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s))))||($is_logged && %s))', $node->group_srls, $node->group_srls == -1 ? 1 : 0);
 			else $group_check_code = "true";
 
 			// List variables
-			$href = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$node->href);
-			$url = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$node->url);
-			$desc = str_replace(array('&','"',"'"),array('&amp;','&quot;','\\\''),$node->desc);
-			if(preg_match('/^([0-9a-zA-Z\_\-]+)$/i', $node->url))
+			$href = str_replace(array('&', '"', '<', '>'), array('&amp;', '&quot;', '&lt;', '&gt;'), $node->href);
+			$url = str_replace(array('&', '"', '<', '>'), array('&amp;', '&quot;', '&lt;', '&gt;'), $node->url);
+			$desc = str_replace(array('&', '"', "'"), array('&amp;', '&quot;', '\\\''), $node->desc);
+			if (preg_match('/^([0-9a-zA-Z\_\-]+)$/i', $node->url))
 			{
 				$href = "getSiteUrl('$domain', '','mid','$node->url')";
 			}
 			else $href = var_export($url, true);
 			$is_shortcut = $node->is_shortcut;
 			$open_window = $node->open_window;
-			$normal_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$node->normal_btn);
-			$hover_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$node->hover_btn);
-			$active_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$node->active_btn);
+			$normal_btn = str_replace(array('&', '"', '<', '>'), array('&amp;', '&quot;', '&lt;', '&gt;'), $node->normal_btn);
+			$hover_btn = str_replace(array('&', '"', '<', '>'), array('&amp;', '&quot;', '&lt;', '&gt;'), $node->hover_btn);
+			$active_btn = str_replace(array('&', '"', '<', '>'), array('&amp;', '&quot;', '&lt;', '&gt;'), $node->active_btn);
 
-			foreach($child_output['url_list'] as $key =>$val)
+			foreach ($child_output['url_list'] as $key =>$val)
 			{
 				$child_output['url_list'][$key] = addslashes($val);
 			}
 
-			$selected = '"'.implode('","',$child_output['url_list']).'"';
+			$selected = '"'.implode('","', $child_output['url_list']).'"';
 			$child_buff = $child_output['buff'];
 			$expand = $node->expand;
 
 			$normal_btn = $node->normal_btn;
-			if($normal_btn && strncasecmp('./files/attach/menu_button', $normal_btn, 26) === 0) $normal_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$normal_btn);
+			if ($normal_btn && strncasecmp('./files/attach/menu_button', $normal_btn, 26) === 0) $normal_btn = str_replace(array('&', '"', '<', '>'), array('&amp;', '&quot;', '&lt;', '&gt;'), $normal_btn);
 			else $normal_btn = '';
 
 			$hover_btn = $node->hover_btn;
-			if($hover_btn && strncasecmp('./files/attach/menu_button', $hover_btn, 26) === 0) $hover_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$hover_btn);
+			if ($hover_btn && strncasecmp('./files/attach/menu_button', $hover_btn, 26) === 0) $hover_btn = str_replace(array('&', '"', '<', '>'), array('&amp;', '&quot;', '&lt;', '&gt;'), $hover_btn);
 			else $hover_btn = '';
 
 			$active_btn = $node->active_btn;
-			if($active_btn && strncasecmp('./files/attach/menu_button', $active_btn, 26) === 0) $active_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$active_btn);
+			if ($active_btn && strncasecmp('./files/attach/menu_button', $active_btn, 26) === 0) $active_btn = str_replace(array('&', '"', '<', '>'), array('&amp;', '&quot;', '&lt;', '&gt;'), $active_btn);
 			else $active_btn = '';
 
 
 			$group_srls = $node->group_srls;
 
-			if($normal_btn)
+			if ($normal_btn)
 			{
-				if($hover_btn) $hover_str = sprintf('onmouseover=\"this.src=\'%s\'\"', $hover_btn); else $hover_str = '';
-				if($active_btn) $active_str = sprintf('onmousedown=\"this.src=\'%s\'\"', $active_btn); else $active_str = '';
+				if ($hover_btn) $hover_str = sprintf('onmouseover=\"this.src=\'%s\'\"', $hover_btn); else $hover_str = '';
+				if ($active_btn) $active_str = sprintf('onmousedown=\"this.src=\'%s\'\"', $active_btn); else $active_str = '';
 				$link = sprintf('"<img src=\"%s\" onmouseout=\"this.src=\'%s\'\" alt=\"".$_menu_names[%d][$lang_type]."\" %s %s />"', $normal_btn, $normal_btn, $node->menu_item_srl, $hover_str, $active_str);
-				if($active_btn) $link_active = sprintf('"<img src=\"%s\" onmouseout=\"this.src=\'%s\'\" alt=\"".$_menu_names[%d][$lang_type]."\" %s />"', $active_btn, $active_btn, $node->menu_item_srl, $hover_str);
+				if ($active_btn) $link_active = sprintf('"<img src=\"%s\" onmouseout=\"this.src=\'%s\'\" alt=\"".$_menu_names[%d][$lang_type]."\" %s />"', $active_btn, $active_btn, $node->menu_item_srl, $hover_str);
 				else $link_active = $link;
 			}
 			else
@@ -2104,7 +2104,7 @@ 
 			);
 
 			// Generate buff data
-			$output['buff'] .=  sprintf('%s=>array(%s),', $node->menu_item_srl, $attribute);
+			$output['buff'] .= sprintf('%s=>array(%s),', $node->menu_item_srl, $attribute);
 			$output['name'] .= $name_str;
 		}
 		return $output;
@@ -2118,19 +2118,19 @@ 
 	 */
 	function updateMenuLayout($layout_srl, $menu_srl_list)
 	{
-		if(!count($menu_srl_list)) return;
+		if (!count($menu_srl_list)) return;
 		// Delete the value of menu_srls
-		$args->menu_srls = implode(',',$menu_srl_list);
+		$args->menu_srls = implode(',', $menu_srl_list);
 		$output = executeQuery('menu.deleteMenuLayout', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$args->layout_srl = $layout_srl;
 		// Mapping menu_srls, layout_srl
-		for($i=0;$i<count($menu_srl_list);$i++)
+		for ($i = 0; $i < count($menu_srl_list); $i++)
 		{
 			$args->menu_srl = $menu_srl_list[$i];
 			$output = executeQuery('menu.insertMenuLayout', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 		}
 	}
 
@@ -2143,47 +2143,47 @@ 
 	{
 		// path setting
 		$path = sprintf('./files/attach/menu_button/%d/', $args->menu_srl);
-		if($args->menu_normal_btn || $args->menu_hover_btn || $args->menu_active_btn && !is_dir($path))
+		if ($args->menu_normal_btn || $args->menu_hover_btn || $args->menu_active_btn && !is_dir($path))
 		{
 			FileHandler::makeDir($path);
 		}
 
-		if($args->isNormalDelete == 'Y' || $args->isHoverDelete == 'Y' || $args->isActiveDelete == 'Y')
+		if ($args->isNormalDelete == 'Y' || $args->isHoverDelete == 'Y' || $args->isActiveDelete == 'Y')
 		{
 			$oMenuModel = getAdminModel('menu');
 			$itemInfo = $oMenuModel->getMenuItemInfo($args->menu_item_srl);
 
-			if($args->isNormalDelete == 'Y' && $itemInfo->normal_btn) FileHandler::removeFile($itemInfo->normal_btn);
-			if($args->isHoverDelete == 'Y' && $itemInfo->hover_btn) FileHandler::removeFile($itemInfo->hover_btn);
-			if($args->isActiveDelete == 'Y' && $itemInfo->active_btn) FileHandler::removeFile($itemInfo->active_btn);
+			if ($args->isNormalDelete == 'Y' && $itemInfo->normal_btn) FileHandler::removeFile($itemInfo->normal_btn);
+			if ($args->isHoverDelete == 'Y' && $itemInfo->hover_btn) FileHandler::removeFile($itemInfo->hover_btn);
+			if ($args->isActiveDelete == 'Y' && $itemInfo->active_btn) FileHandler::removeFile($itemInfo->active_btn);
 		}
 
 		$returnArray = array();
 		$date = date('YmdHis');
 		// normal button
-		if($args->menu_normal_btn)
+		if ($args->menu_normal_btn)
 		{
-			$tmp_arr = explode('.',$args->menu_normal_btn['name']);
-			$ext = $tmp_arr[count($tmp_arr)-1];
+			$tmp_arr = explode('.', $args->menu_normal_btn['name']);
+			$ext = $tmp_arr[count($tmp_arr) - 1];
 
 			$filename = sprintf('%s%d.%s.%s.%s', $path, $args->menu_item_srl, $date, 'menu_normal_btn', $ext);
 
-			if(checkUploadedFile($args->menu_normal_btn['tmp_name']))
+			if (checkUploadedFile($args->menu_normal_btn['tmp_name']))
 			{
-				move_uploaded_file ( $args->menu_normal_btn ['tmp_name'], $filename );
+				move_uploaded_file($args->menu_normal_btn ['tmp_name'], $filename);
 				$returnArray ['normal_btn'] = $filename;
 			}
 		}
 
 		// hover button
-		if($args->menu_hover_btn)
+		if ($args->menu_hover_btn)
 		{
-			$tmp_arr = explode('.',$args->menu_hover_btn['name']);
-			$ext = $tmp_arr[count($tmp_arr)-1];
+			$tmp_arr = explode('.', $args->menu_hover_btn['name']);
+			$ext = $tmp_arr[count($tmp_arr) - 1];
 
 			$filename = sprintf('%s%d.%s.%s.%s', $path, $args->menu_item_srl, $date, 'menu_hover_btn', $ext);
 
-			if(checkUploadedFile($args->menu_hover_btn['tmp_name']))
+			if (checkUploadedFile($args->menu_hover_btn['tmp_name']))
 			{
 				move_uploaded_file($args->menu_hover_btn['tmp_name'], $filename);
 				$returnArray['hover_btn'] = $filename;
@@ -2191,14 +2191,14 @@ 
 		}
 
 		// active button
-		if($args->menu_active_btn)
+		if ($args->menu_active_btn)
 		{
-			$tmp_arr = explode('.',$args->menu_active_btn['name']);
-			$ext = $tmp_arr[count($tmp_arr)-1];
+			$tmp_arr = explode('.', $args->menu_active_btn['name']);
+			$ext = $tmp_arr[count($tmp_arr) - 1];
 
 			$filename = sprintf('%s%d.%s.%s.%s', $path, $args->menu_item_srl, $date, 'menu_active_btn', $ext);
 
-			if(checkUploadedFile($args->menu_active_btn['tmp_name']))
+			if (checkUploadedFile($args->menu_active_btn['tmp_name']))
 			{
 				move_uploaded_file($args->menu_active_btn['tmp_name'], $filename);
 				$returnArray['active_btn'] = $filename;
@@ -2220,7 +2220,7 @@ 
 			"active_btn"=>"",
 		);
 		//normal_btn
-		if($menuItemInfo->normal_btn)
+		if ($menuItemInfo->normal_btn)
 		{
 			$originFile = FileHandler::getRealPath($menuItemInfo->normal_btn);
 			$targetFile = $this->_changeMenuItemSrlInButtonPath($menuItemInfo->normal_btn, $insertedMenuSrl, $insertedMenuItemSrl, 'normal');
@@ -2230,7 +2230,7 @@ 
 		}
 
 		//hover_btn
-		if($menuItemInfo->hover_btn)
+		if ($menuItemInfo->hover_btn)
 		{
 			$originFile = FileHandler::getRealPath($menuItemInfo->hover_btn);
 			$targetFile = $this->_changeMenuItemSrlInButtonPath($menuItemInfo->hover_btn, $insertedMenuSrl, $insertedMenuItemSrl, 'hover');
@@ -2240,7 +2240,7 @@ 
 		}
 
 		//active_btn
-		if($menuItemInfo->active_btn)
+		if ($menuItemInfo->active_btn)
 		{
 			$originFile = FileHandler::getRealPath($menuItemInfo->active_btn);
 			$targetFile = $this->_changeMenuItemSrlInButtonPath($menuItemInfo->active_btn, $insertedMenuSrl, $insertedMenuItemSrl, 'active');
@@ -2255,9 +2255,9 @@ 
 	{
 		$path = sprintf('./files/attach/menu_button/%d/', $menuSrl);
 		$tmp_arr = explode('.', $buttonPath);
-		$ext = $tmp_arr[count($tmp_arr)-1];
+		$ext = $tmp_arr[count($tmp_arr) - 1];
 		$date = date("YmdHis");
-		return sprintf('%s%d.%s.%s.%s', $path, $menuItemSrl,$date,'menu_'.$mode.'_btn', $ext);
+		return sprintf('%s%d.%s.%s.%s', $path, $menuItemSrl, $date, 'menu_'.$mode.'_btn', $ext);
 	}
 
 	public function makeHomemenuCacheFile($menuSrl)

Braces +300 added lines, -134 removed lines

@@ -69,7 +69,9 @@ 
 		$site_module_info = Context::get('site_module_info');
 
 		$output = $this->addMenu(Context::get('title'), (int)$site_module_info->site_srl);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->add('menu_srl', $output->get('menuSrl'));
 		$this->setMessage('success_registed');
@@ -148,8 +150,7 @@ 
 				$moduleConfig->unlinked_menu_srl = $output->get('menuSrl');
 				$oModuleController = getController('module');
 				$oModuleController->updateModuleConfig('menu', $moduleConfig);
-			}
-			else
+			} else
 			{
 				return false;
 			}
@@ -186,8 +187,7 @@ 
 			if($output->toBool() && $output->data)
 			{
 				$moduleInfo->menu_srl = $output->data->menu_srl;
-			}
-			else
+			} else
 			{
 				// create menu item.
 				$item_args->menu_srl = $menuSrl;
@@ -235,7 +235,9 @@ 
 		$args->menu_srl = Context::get('menu_srl');
 
 		$output = executeQuery('menu.updateMenu', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->setMessage('success_registed');
 
@@ -255,8 +257,9 @@ 
 		$menuInfo = $oMenuAdminModel->getMenu($menu_srl);
 
 		$oAdmin = getClass('admin');
-		if($menuInfo->title == $oAdmin->getAdminMenuName())
-			return new BaseObject(-1, 'msg_adminmenu_cannot_delete');
+		if($menuInfo->title == $oAdmin->getAdminMenuName()) {
+					return new BaseObject(-1, 'msg_adminmenu_cannot_delete');
+		}
 
 		// get menu properies with child menu
 		$phpFile = sprintf("./files/cache/menu/%s.php", $menu_srl);
@@ -280,8 +283,9 @@ 
 				$originMenu = $value;
 				$this->_checkHomeMenuInOriginMenu($originMenu, $siteInfo->mid, $isStartmenuInclude);
 
-				if($isStartmenuInclude)
-					break;
+				if($isStartmenuInclude) {
+									break;
+				}
 			}
 		}
 
@@ -366,7 +370,9 @@ 
 			foreach($cache_list as $cache_file)
 			{
 				$pos = strpos($cache_file, $menu_srl.'.');
-				if($pos>0)FileHandler::removeFile($cache_file);
+				if($pos>0) {
+					FileHandler::removeFile($cache_file);
+				}
 			}
 		}
 		// Delete images of menu buttons
@@ -405,8 +411,7 @@ 
 		if($request->is_shortcut == 'Y')
 		{
 			$result = $this->_insertShortcut($request);
-		}
-		else
+		} else
 		{
 			$result = $this->_insertMenu($request, $isProc);
 		}
@@ -464,12 +469,21 @@ 
 			$args->is_shortcut = $request->is_shortcut;
 			$args->url = $request->shortcut_target;
 
-			if(!$args->open_window) $args->open_window = 'N';
-			if(!$args->expand) $args->expand = 'N';
-			if(!$args->is_shortcut) $args->is_shortcut = 'Y';
+			if(!$args->open_window) {
+				$args->open_window = 'N';
+			}
+			if(!$args->expand) {
+				$args->expand = 'N';
+			}
+			if(!$args->is_shortcut) {
+				$args->is_shortcut = 'Y';
+			}
 
-			if($request->menu_name_key) $args->name = $request->menu_name_key;
-			else $args->name = $request->menu_name;
+			if($request->menu_name_key) {
+				$args->name = $request->menu_name_key;
+			} else {
+				$args->name = $request->menu_name;
+			}
 		}
 		// type is module short cut
 		else if(is_numeric($request->shortcut_target))
@@ -504,13 +518,18 @@ 
 			$args->url = '#';
 		}
 
-		if($request->menu_desc) $args->desc = $request->menu_desc;
-		else $args->desc = '';
+		if($request->menu_desc) {
+			$args->desc = $request->menu_desc;
+		} else {
+			$args->desc = '';
+		}
 
 		$args->menu_item_srl = getNextSequence();
 		$args->listorder = -1*$args->menu_item_srl;
 		$output = executeQuery('menu.insertMenuItem', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$oDB->commit();
 
@@ -532,15 +551,27 @@ 
 		$args->expand = $request->menu_expand;
 		$args->is_shortcut = $request->is_shortcut;
 
-		if(!$args->open_window) $args->open_window = 'N';
-		if(!$args->expand) $args->expand = 'N';
-		if(!$args->is_shortcut) $args->is_shortcut = 'N';
+		if(!$args->open_window) {
+			$args->open_window = 'N';
+		}
+		if(!$args->expand) {
+			$args->expand = 'N';
+		}
+		if(!$args->is_shortcut) {
+			$args->is_shortcut = 'N';
+		}
 
-		if($request->menu_name_key) $args->name = $request->menu_name_key;
-		else $args->name = $request->menu_name;
+		if($request->menu_name_key) {
+			$args->name = $request->menu_name_key;
+		} else {
+			$args->name = $request->menu_name;
+		}
 
-		if($request->menu_desc) $args->desc = $request->menu_desc;
-		else $args->desc = '';
+		if($request->menu_desc) {
+			$args->desc = $request->menu_desc;
+		} else {
+			$args->desc = '';
+		}
 
 		if($request->module_id && strncasecmp('http', $request->module_id, 4) === 0)
 		{
@@ -558,9 +589,15 @@ 
 		}
 
 		// if setting button variables, set argument button variables for db insert. but not upload in this method
-		if($request->normal_btn) $args->normal_btn = $request->normal_btn;
-		if($request->hover_btn) $args->hover_btn = $request->hover_btn;
-		if($request->active_btn) $args->active_btn = $request->active_btn;
+		if($request->normal_btn) {
+			$args->normal_btn = $request->normal_btn;
+		}
+		if($request->hover_btn) {
+			$args->hover_btn = $request->hover_btn;
+		}
+		if($request->active_btn) {
+			$args->active_btn = $request->active_btn;
+		}
 
 		if(!$request->module_id)
 		{
@@ -571,7 +608,9 @@ 
 		$args->menu_item_srl = getNextSequence();
 		$args->listorder = -1*$args->menu_item_srl;
 		$output = executeQuery('menu.insertMenuItem', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$oDB->commit();
 
@@ -652,8 +691,12 @@ 
 		}
 
 		// variables set
-		if($request->menu_open_window != "Y") $request->menu_open_window = "N";
-		if($request->menu_expand != "Y") $request->menu_expand = "N";
+		if($request->menu_open_window != "Y") {
+			$request->menu_open_window = "N";
+		}
+		if($request->menu_expand != "Y") {
+			$request->menu_expand = "N";
+		}
 
 		// Get original information
 		$oMenuAdminModel = getAdminModel('menu');
@@ -680,13 +723,11 @@ 
 
 				$args->url = $newItemInfo->url;
 				$args->is_shortcut = 'Y';
-			}
-			else
+			} else
 			{
 				$args->url = '#';
 			}
-		}
-		else
+		} else
 		{
 			// check already created module instance
 			$oModuleModel = getModel('module');
@@ -719,14 +760,16 @@ 
 		if($request->menu_name_key)
 		{
 			$args->name = $request->menu_name_key;
-		}
-		else
+		} else
 		{
 			$args->name = $request->menu_name;
 		}
 
-		if($request->menu_desc) $args->desc = $request->menu_desc;
-		else $args->desc = '';
+		if($request->menu_desc) {
+			$args->desc = $request->menu_desc;
+		} else {
+			$args->desc = '';
+		}
 
 		unset($args->group_srls);
 		$args->open_window = $request->menu_open_window;
@@ -828,8 +871,7 @@ 
 		{
 			$this->setError($returnObj->error);
 			$this->setMessage($returnObj->message);
-		}
-		else
+		} else
 		{
 			$this->setMessage('success_deleted');
 		}
@@ -856,7 +898,9 @@ 
 		if($args->is_force != 'Y')
 		{
 			$output = executeQuery('menu.getChildMenuCount', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 			if($output->data->count > 0)
 			{
 				return new BaseObject(-1001, 'msg_cannot_delete_for_child');
@@ -874,7 +918,9 @@ 
 			return $this->stop('msg_cannot_delete_for_admin_topmenu');
 		}
 
-		if($itemInfo->parent_srl) $parent_srl = $itemInfo->parent_srl;
+		if($itemInfo->parent_srl) {
+			$parent_srl = $itemInfo->parent_srl;
+		}
 
 		// get menu properies with child menu
 		$phpFile = sprintf("./files/cache/menu/%s.php", $args->menu_srl);
@@ -948,9 +994,15 @@ 
 		// Update the xml file and get its location
 		$xml_file = $this->makeXmlFile($args->menu_srl);
 		// Delete all of image buttons
-		if($node['normal_btn']) FileHandler::removeFile($node['normal_btn']);
-		if($node['hover_btn']) FileHandler::removeFile($node['hover_btn']);
-		if($node['active_btn']) FileHandler::removeFile($node['active_btn']);
+		if($node['normal_btn']) {
+			FileHandler::removeFile($node['normal_btn']);
+		}
+		if($node['hover_btn']) {
+			FileHandler::removeFile($node['hover_btn']);
+		}
+		if($node['active_btn']) {
+			FileHandler::removeFile($node['active_btn']);
+		}
 
 		// Delete module
 		if($node['is_shortcut'] != 'Y' && strncasecmp('http', $node['url'], 4) !== 0)
@@ -1017,7 +1069,9 @@ 
 		$source_srl = Context::get('source_srl');	// Same hierarchy's menu item serial number
 		$target_srl = Context::get('target_srl');	// Self menu item serial number
 
-		if(!$mode || !$parent_srl || !$target_srl) return new BaseObject(-1,'msg_invalid_request');
+		if(!$mode || !$parent_srl || !$target_srl) {
+			return new BaseObject(-1,'msg_invalid_request');
+		}
 
 		$oMenuAdminModel = getAdminModel('menu');
 
@@ -1181,8 +1235,11 @@ 
 		// default argument setting
 		$args = new stdClass();
 		$args->menu_srl = $menuSrl;
-		if($parentSrl == 0) $args->parent_srl = $menuSrl;
-		else $args->parent_srl = $parentSrl;
+		if($parentSrl == 0) {
+			$args->parent_srl = $menuSrl;
+		} else {
+			$args->parent_srl = $parentSrl;
+		}
 		$args->menu_name_key = $originMenu['text'];
 		$args->menu_name = $originMenu['text'];
 		$args->menu_open_window = $originMenu['open_window'];
@@ -1275,8 +1332,11 @@ 
 		for($i=0;$i<4;$i++)
 		{
 			$case = rand(0, 1);
-			if($case) $doc = rand(65, 90);
-			else $doc = rand(97, 122);
+			if($case) {
+				$doc = rand(65, 90);
+			} else {
+				$doc = rand(97, 122);
+			}
 
 			$randomString .= chr($doc);
 		}
@@ -1299,15 +1359,21 @@ 
 		// menu name update
 		$args->menu_srl = $this->menuSrl;
 		$output = executeQuery('menu.updateMenu', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->map = array();
 		if(is_array($parentKeyList))
 		{
 			foreach($parentKeyList as $no=>$srl)
 			{
-				if($srl === 0) continue;
-				if(!is_array($this->map[$srl]))$this->map[$srl] = array();
+				if($srl === 0) {
+					continue;
+				}
+				if(!is_array($this->map[$srl])) {
+					$this->map[$srl] = array();
+				}
 				$this->map[$srl][] = $no;
 			}
 		}
@@ -1409,7 +1475,9 @@ 
 		$oMenuAdminModel = getAdminModel('menu');
 
 		$target_item = $oMenuAdminModel->getMenuItemInfo($target_srl);
-		if($target_item->menu_item_srl != $target_srl) return new BaseObject(-1,'msg_invalid_request');
+		if($target_item->menu_item_srl != $target_srl) {
+			return new BaseObject(-1,'msg_invalid_request');
+		}
 		// Move the menu location(change the order menu appears)
 		if($mode == 'move')
 		{
@@ -1420,24 +1488,33 @@ 
 			if($source_srl)
 			{
 				$source_item = $oMenuAdminModel->getMenuItemInfo($source_srl);
-				if($source_item->menu_item_srl != $source_srl) return new BaseObject(-1,'msg_invalid_request');
+				if($source_item->menu_item_srl != $source_srl) {
+					return new BaseObject(-1,'msg_invalid_request');
+				}
 				$args->listorder = $source_item->listorder-1;
-			}
-			else
+			} else
 			{
 				$output = executeQuery('menu.getMaxListorder', $args);
-				if(!$output->toBool()) return $output;
+				if(!$output->toBool()) {
+					return $output;
+				}
 				$args->listorder = (int)$output->data->listorder;
-				if(!$args->listorder) $args->listorder= 0;
+				if(!$args->listorder) {
+					$args->listorder= 0;
+				}
 			}
 			$args->parent_srl = $parent_srl;
 			$output = executeQuery('menu.updateMenuItemListorder', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 
 			$args->parent_srl = $parent_srl;
 			$args->menu_item_srl = $target_srl;
 			$output = executeQuery('menu.updateMenuItemNode', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 
 			//module's menu_srl move also
 			if($isShortcut == 'N' && !empty($url))
@@ -1465,14 +1542,15 @@ 
 				}
 			}
 			// Add a child
-		}
-		elseif($mode == 'insert')
+		} elseif($mode == 'insert')
 		{
 			$args->menu_item_srl = $target_srl;
 			$args->parent_srl = $parent_srl;
 			$args->listorder = -1*getNextSequence();
 			$output = executeQuery('menu.updateMenuItemNode', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 		}
 
 		$xml_file = $this->makeXmlFile($menu_srl);
@@ -1516,8 +1594,7 @@ 
 		{
 			Context::set('error_messge', Context::getLang('msg_invalid_request'));
 
-		}
-		else if(!$target_file || !is_uploaded_file($target_file['tmp_name']) || !preg_match('/\.(gif|jpeg|jpg|png)$/i',$target_file['name'])  || !checkUploadedFile($target_file['tmp_name']))
+		} else if(!$target_file || !is_uploaded_file($target_file['tmp_name']) || !preg_match('/\.(gif|jpeg|jpg|png)$/i',$target_file['name'])  || !checkUploadedFile($target_file['tmp_name']))
 		{
 			Context::set('error_messge', Context::getLang('msg_invalid_request'));
 		}
@@ -1531,7 +1608,9 @@ 
 			$path = sprintf('./files/attach/menu_button/%d/', $menu_srl);
 			$filename = sprintf('%s%d.%s.%s', $path, $menu_item_srl, $target, $ext);
 
-			if(!is_dir($path)) FileHandler::makeDir($path);
+			if(!is_dir($path)) {
+				FileHandler::makeDir($path);
+			}
 
 			move_uploaded_file($target_file['tmp_name'], $filename);
 			Context::set('filename', $filename);
@@ -1571,7 +1650,9 @@ 
 			foreach($installed_module_list AS $key=>$value)
 			{
 				$info = $oModuleModel->getModuleActionXml($value->module);
-				if($info->menu) $menuList[$value->module] = $info->menu;
+				if($info->menu) {
+					$menuList[$value->module] = $info->menu;
+				}
 				unset($info->menu);
 			}
 		}
@@ -1603,8 +1684,12 @@ 
 		$info = $oModuleModel->getModuleActionXml($moduleName);
 
 		$url = getNotEncodedFullUrl('', 'module', 'admin', 'act', $info->menu->{$menuName}->index);
-		if(empty($url)) $url = getNotEncodedFullUrl('', 'module', 'admin', 'act', $info->admin_index_act);
-		if(empty($url)) $url = getNotEncodedFullUrl('', 'module', 'admin');
+		if(empty($url)) {
+			$url = getNotEncodedFullUrl('', 'module', 'admin', 'act', $info->admin_index_act);
+		}
+		if(empty($url)) {
+			$url = getNotEncodedFullUrl('', 'module', 'admin');
+		}
 		$dbInfo = Context::getDBInfo();
 
 		$args = new stdClass();
@@ -1616,8 +1701,7 @@ 
 		if(strpos($url, 'https') !== false)
 		{
 			$args->url = str_replace('https'.substr($dbInfo->default_url, 4), '', $url);
-		}
-		else
+		} else
 		{
 			$args->url = str_replace($dbInfo->default_url, '', $url);
 		}
@@ -1636,14 +1720,18 @@ 
 		if($item_info->menu_item_srl == $args->menu_item_srl)
 		{
 			$output = $this->_updateMenuItem($args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 		}
 		// Insert if not exist
 		else
 		{
 			$args->listorder = -1*$args->menu_item_srl;
 			$output = executeQuery('menu.insertMenuItem', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 		}
 		// Get information of the menu
 		$menu_info = $oMenuModel->getMenu($args->menu_srl);
@@ -1674,8 +1762,7 @@ 
 		if(!$exposure)
 		{
 			$args->group_srls = '';
-		}
-		else
+		} else
 		{
 			$exposure = explode(',', $exposure);
 			if(in_array($exposure, array('-1','-3')))
@@ -1683,7 +1770,9 @@ 
 				$args->group_srls = $exposure;
 			}
 
-			if($exposure) $args->group_srls = implode(',', $exposure);
+			if($exposure) {
+				$args->group_srls = implode(',', $exposure);
+			}
 		}
 
 		$output = $this->_updateMenuItem($args);
@@ -1706,7 +1795,9 @@ 
 			$moduleGrnatsArgs = new stdClass;
 			$moduleGrnatsArgs->module_srl = $moduleInfo->module_srl;
 			$output = executeQueryArray('module.getModuleGrants', $moduleGrnatsArgs);
-			if(!$output->data) $output->data = array();
+			if(!$output->data) {
+				$output->data = array();
+			}
 			$moduleGrnats = new stdClass();
 			foreach($output->data as $grant)
 			{
@@ -1715,7 +1806,9 @@ 
 		}
 
 		$grantList = $xml_info->grant;
-		if(!$grantList) $grantList = new stdClass;
+		if(!$grantList) {
+			$grantList = new stdClass;
+		}
 
 		$grantList->access = new stdClass();
 		$grantList->access->default = 'guest';
@@ -1741,10 +1834,11 @@ 
 					$grant->{$grantName}[] = $htPerm[$grantName];
 					continue;
 				}
-			}
-			else if($itemInfo->is_shortcut === 'Y')
+			} else if($itemInfo->is_shortcut === 'Y')
 			{
-				if(isset($moduleGrnats) && $moduleGrnats->{$grantName}) $grant->{$grantName} = $moduleGrnats->{$grantName};
+				if(isset($moduleGrnats) && $moduleGrnats->{$grantName}) {
+					$grant->{$grantName} = $moduleGrnats->{$grantName};
+				}
 			}
 		}
 
@@ -1766,12 +1860,16 @@ 
 	function makeXmlFile($menu_srl)
 	{
 		// Return if there is no information when creating the xml file
-		if(!$menu_srl) return;
+		if(!$menu_srl) {
+			return;
+		}
 		// Get menu informaton
 		$args = new stdClass();
 		$args->menu_srl = $menu_srl;
 		$output = executeQuery('menu.getMenu', $args);
-		if(!$output->toBool() || !$output->data) return $output;
+		if(!$output->toBool() || !$output->data) {
+			return $output;
+		}
 		$site_srl = (int)$output->data->site_srl;
 
 		if($site_srl)
@@ -1785,7 +1883,9 @@ 
 		$args->menu_srl = $menu_srl;
 		$args->sort_index = 'listorder';
 		$output = executeQuery('menu.getMenuItems', $args);
-		if(!$output->toBool()) return;
+		if(!$output->toBool()) {
+			return;
+		}
 		// Specify the name of the cache file
 		$xml_file = sprintf(_XE_PATH_ . "files/cache/menu/%s.xml.php", $menu_srl);
 		$php_file = sprintf(_XE_PATH_ . "files/cache/menu/%s.php", $menu_srl);
@@ -1799,7 +1899,9 @@ 
 			return $xml_file;
 		}
 		// Change to an array if only a single data is obtained
-		if(!is_array($list)) $list = array($list);
+		if(!is_array($list)) {
+			$list = array($list);
+		}
 		// Create a tree for loop
 		$list_count = count($list);
 		for($i=0;$i<$list_count;$i++)
@@ -1887,7 +1989,9 @@ 
 	 */
 	function getXmlTree($source_node, $tree, $site_srl, $domain)
 	{
-		if(!$source_node) return;
+		if(!$source_node) {
+			return;
+		}
 
 		$oMenuAdminModel = getAdminModel('menu');
 
@@ -1895,7 +1999,9 @@ 
 		{
 			$child_buff = "";
 			// Get data of the child nodes
-			if($menu_item_srl&&$tree[$menu_item_srl]) $child_buff = $this->getXmlTree($tree[$menu_item_srl], $tree, $site_srl, $domain);
+			if($menu_item_srl&&$tree[$menu_item_srl]) {
+				$child_buff = $this->getXmlTree($tree[$menu_item_srl], $tree, $site_srl, $domain);
+			}
 			// List variables
 			$names = $oMenuAdminModel->getMenuItemNames($node->name, $site_srl);
 			foreach($names as $key => $val)
@@ -1909,31 +2015,48 @@ 
 			if(preg_match('/^([0-9a-zA-Z\_\-]+)$/', $node->url))
 			{
 				$href = "getSiteUrl('$domain', '','mid','$node->url')";
+			} else {
+				$href = var_export($url, true);
 			}
-			else $href = var_export($url, true);
 			$is_shortcut = ($node->is_shortcut) ? $node->is_shortcut : '';
 			$open_window = ($node->open_window) ? $node->open_window : '';
 			$expand = ($node->expand) ? $node->expand : '';
 
 			$normal_btn = ($node->normal_btn) ? $node->normal_btn : '';
-			if($normal_btn && strncasecmp('./files/attach/menu_button', $normal_btn, 26) === 0) $normal_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$normal_btn);
-			else $normal_btn = '';
+			if($normal_btn && strncasecmp('./files/attach/menu_button', $normal_btn, 26) === 0) {
+				$normal_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$normal_btn);
+			} else {
+				$normal_btn = '';
+			}
 			$hover_btn = ($node->hover_btn) ? $node->hover_btn : '';
-			if($hover_btn && strncasecmp('./files/attach/menu_button', $hover_btn, 26) === 0) $hover_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$hover_btn);
-			else $hover_btn = '';
+			if($hover_btn && strncasecmp('./files/attach/menu_button', $hover_btn, 26) === 0) {
+				$hover_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$hover_btn);
+			} else {
+				$hover_btn = '';
+			}
 			$active_btn = ($node->active_btn) ? $node->active_btn : '';
-			if($active_btn && strncasecmp('./files/attach/menu_button', $active_btn, 26) === 0) $active_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$active_btn);
-			else $active_btn = '';
+			if($active_btn && strncasecmp('./files/attach/menu_button', $active_btn, 26) === 0) {
+				$active_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$active_btn);
+			} else {
+				$active_btn = '';
+			}
 
 			$group_srls = ($node->group_srls) ? $node->group_srls : '';
 
 			if($normal_btn)
 			{
-				if($hover_btn) $hover_str = sprintf('onmouseover=&quot;this.src=\'%s\'&quot;', $hover_btn); else $hover_str = '';
-				if($active_btn) $active_str = sprintf('onmousedown=&quot;this.src=\'%s\'&quot;', $active_btn); else $active_str = '';
+				if($hover_btn) {
+					$hover_str = sprintf('onmouseover=&quot;this.src=\'%s\'&quot;', $hover_btn);
+				} else {
+					$hover_str = '';
+				}
+				if($active_btn) {
+					$active_str = sprintf('onmousedown=&quot;this.src=\'%s\'&quot;', $active_btn);
+				} else {
+					$active_str = '';
+				}
 				$link = sprintf('&lt;img src=&quot;%s&quot; onmouseout=&quot;this.src=\'%s\'&quot; alt=&quot;<?php print htmlspecialchars($_names[$lang_type], ENT_COMPAT | ENT_HTML401, \'UTF-8\', false) ?>&quot; %s %s /&gt;', $normal_btn, $normal_btn, $hover_str, $active_str);
-			}
-			else
+			} else
 			{
 				$link = '<?php print $_names[$lang_type]; ?>';
 			}
@@ -1941,8 +2064,7 @@ 
 			// If the value of node->group_srls exists
 			if($group_srls) {
 				$group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s))))||($is_logged&&%s))',$group_srls,$group_srls == -1?1:0);
-			}
-			else
+			} else
 			{
 				$group_check_code = "true";
 			}
@@ -1969,8 +2091,11 @@ 
 				$link
 			);
 
-			if($child_buff) $buff .= sprintf('<node %s>%s</node>', $attribute, $child_buff);
-			else $buff .=  sprintf('<node %s />', $attribute);
+			if($child_buff) {
+				$buff .= sprintf('<node %s>%s</node>', $attribute, $child_buff);
+			} else {
+				$buff .=  sprintf('<node %s />', $attribute);
+			}
 		}
 		return $buff;
 	}
@@ -1989,15 +2114,20 @@ 
 	function getPhpCacheCode($source_node, $tree, $site_srl, $domain)
 	{
 		$output = array("buff"=>"", "url_list"=>array());
-		if(!$source_node) return $output;
+		if(!$source_node) {
+			return $output;
+		}
 
 		$oMenuAdminModel = getAdminModel('menu');
 
 		foreach($source_node as $menu_item_srl => $node)
 		{
 			// Get data from child nodes if exist.
-			if($menu_item_srl&&$tree[$menu_item_srl]) $child_output = $this->getPhpCacheCode($tree[$menu_item_srl], $tree, $site_srl, $domain);
-			else $child_output = array("buff"=>"", "url_list"=>array());
+			if($menu_item_srl&&$tree[$menu_item_srl]) {
+				$child_output = $this->getPhpCacheCode($tree[$menu_item_srl], $tree, $site_srl, $domain);
+			} else {
+				$child_output = array("buff"=>"", "url_list"=>array());
+			}
 
 			// List variables
 			$names = $oMenuAdminModel->getMenuItemNames($node->name, $site_srl);
@@ -2007,8 +2137,7 @@ 
 				if(preg_match('/\{\$lang->menu_gnb(?:_sub)?\[\'([a-zA-Z_]+)\'\]\}/', $val) === 1)
 				{
 					$name_arr_str .= sprintf('"%s"=>"%s",', $key, $val);
-				}
-				else
+				} else
 				{
 					$name_arr_str .= sprintf('"%s"=>\'%s\',', $key, str_replace(array('\\','\''), array('\\\\','\\\''), removeHackTag($val)));
 				}
@@ -2016,11 +2145,16 @@ 
 			$name_str = sprintf('$_menu_names[%d] = array(%s); %s', $node->menu_item_srl, $name_arr_str, $child_output['name']);
 
 			// If url value is not empty in the current node, put the value into an array url_list
-			if($node->url) $child_output['url_list'][] = $node->url;
+			if($node->url) {
+				$child_output['url_list'][] = $node->url;
+			}
 			$output['url_list'] = array_merge($output['url_list'], $child_output['url_list']);
 			// If node->group_srls value exists
-			if($node->group_srls)$group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s))))||($is_logged && %s))',$node->group_srls,$node->group_srls == -1?1:0);
-			else $group_check_code = "true";
+			if($node->group_srls) {
+				$group_check_code = sprintf('($is_admin==true||(is_array($group_srls)&&count(array_intersect($group_srls, array(%s))))||($is_logged && %s))',$node->group_srls,$node->group_srls == -1?1:0);
+			} else {
+				$group_check_code = "true";
+			}
 
 			// List variables
 			$href = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$node->href);
@@ -2029,8 +2163,9 @@ 
 			if(preg_match('/^([0-9a-zA-Z\_\-]+)$/i', $node->url))
 			{
 				$href = "getSiteUrl('$domain', '','mid','$node->url')";
+			} else {
+				$href = var_export($url, true);
 			}
-			else $href = var_export($url, true);
 			$is_shortcut = $node->is_shortcut;
 			$open_window = $node->open_window;
 			$normal_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$node->normal_btn);
@@ -2047,29 +2182,48 @@ 
 			$expand = $node->expand;
 
 			$normal_btn = $node->normal_btn;
-			if($normal_btn && strncasecmp('./files/attach/menu_button', $normal_btn, 26) === 0) $normal_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$normal_btn);
-			else $normal_btn = '';
+			if($normal_btn && strncasecmp('./files/attach/menu_button', $normal_btn, 26) === 0) {
+				$normal_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$normal_btn);
+			} else {
+				$normal_btn = '';
+			}
 
 			$hover_btn = $node->hover_btn;
-			if($hover_btn && strncasecmp('./files/attach/menu_button', $hover_btn, 26) === 0) $hover_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$hover_btn);
-			else $hover_btn = '';
+			if($hover_btn && strncasecmp('./files/attach/menu_button', $hover_btn, 26) === 0) {
+				$hover_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$hover_btn);
+			} else {
+				$hover_btn = '';
+			}
 
 			$active_btn = $node->active_btn;
-			if($active_btn && strncasecmp('./files/attach/menu_button', $active_btn, 26) === 0) $active_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$active_btn);
-			else $active_btn = '';
+			if($active_btn && strncasecmp('./files/attach/menu_button', $active_btn, 26) === 0) {
+				$active_btn = str_replace(array('&','"','<','>'),array('&amp;','&quot;','&lt;','&gt;'),$active_btn);
+			} else {
+				$active_btn = '';
+			}
 
 
 			$group_srls = $node->group_srls;
 
 			if($normal_btn)
 			{
-				if($hover_btn) $hover_str = sprintf('onmouseover=\"this.src=\'%s\'\"', $hover_btn); else $hover_str = '';
-				if($active_btn) $active_str = sprintf('onmousedown=\"this.src=\'%s\'\"', $active_btn); else $active_str = '';
+				if($hover_btn) {
+					$hover_str = sprintf('onmouseover=\"this.src=\'%s\'\"', $hover_btn);
+				} else {
+					$hover_str = '';
+				}
+				if($active_btn) {
+					$active_str = sprintf('onmousedown=\"this.src=\'%s\'\"', $active_btn);
+				} else {
+					$active_str = '';
+				}
 				$link = sprintf('"<img src=\"%s\" onmouseout=\"this.src=\'%s\'\" alt=\"".$_menu_names[%d][$lang_type]."\" %s %s />"', $normal_btn, $normal_btn, $node->menu_item_srl, $hover_str, $active_str);
-				if($active_btn) $link_active = sprintf('"<img src=\"%s\" onmouseout=\"this.src=\'%s\'\" alt=\"".$_menu_names[%d][$lang_type]."\" %s />"', $active_btn, $active_btn, $node->menu_item_srl, $hover_str);
-				else $link_active = $link;
-			}
-			else
+				if($active_btn) {
+					$link_active = sprintf('"<img src=\"%s\" onmouseout=\"this.src=\'%s\'\" alt=\"".$_menu_names[%d][$lang_type]."\" %s />"', $active_btn, $active_btn, $node->menu_item_srl, $hover_str);
+				} else {
+					$link_active = $link;
+				}
+			} else
 			{
 				$link_active = $link = sprintf('$_menu_names[%d][$lang_type]', $node->menu_item_srl);
 			}
@@ -2118,11 +2272,15 @@ 
 	 */
 	function updateMenuLayout($layout_srl, $menu_srl_list)
 	{
-		if(!count($menu_srl_list)) return;
+		if(!count($menu_srl_list)) {
+			return;
+		}
 		// Delete the value of menu_srls
 		$args->menu_srls = implode(',',$menu_srl_list);
 		$output = executeQuery('menu.deleteMenuLayout', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$args->layout_srl = $layout_srl;
 		// Mapping menu_srls, layout_srl
@@ -2130,7 +2288,9 @@ 
 		{
 			$args->menu_srl = $menu_srl_list[$i];
 			$output = executeQuery('menu.insertMenuLayout', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 		}
 	}
 
@@ -2153,9 +2313,15 @@ 
 			$oMenuModel = getAdminModel('menu');
 			$itemInfo = $oMenuModel->getMenuItemInfo($args->menu_item_srl);
 
-			if($args->isNormalDelete == 'Y' && $itemInfo->normal_btn) FileHandler::removeFile($itemInfo->normal_btn);
-			if($args->isHoverDelete == 'Y' && $itemInfo->hover_btn) FileHandler::removeFile($itemInfo->hover_btn);
-			if($args->isActiveDelete == 'Y' && $itemInfo->active_btn) FileHandler::removeFile($itemInfo->active_btn);
+			if($args->isNormalDelete == 'Y' && $itemInfo->normal_btn) {
+				FileHandler::removeFile($itemInfo->normal_btn);
+			}
+			if($args->isHoverDelete == 'Y' && $itemInfo->hover_btn) {
+				FileHandler::removeFile($itemInfo->hover_btn);
+			}
+			if($args->isActiveDelete == 'Y' && $itemInfo->active_btn) {
+				FileHandler::removeFile($itemInfo->active_btn);
+			}
 		}
 
 		$returnArray = array();

classes/object/BaseObject.class.php

Spacing +7 added lines, -7 removed lines

@@ -95,7 +95,7 @@ 
 	 */
 	function setMessage($message = 'success', $type = NULL)
 	{
-		if($str = Context::getLang($message))
+		if ($str = Context::getLang($message))
 		{
 			$this->message = $str;
 		}
@@ -138,14 +138,14 @@ 
 	 */
 	function adds($object)
 	{
-		if(is_object($object))
+		if (is_object($object))
 		{
 			$object = get_object_vars($object);
 		}
 
-		if(is_array($object))
+		if (is_array($object))
 		{
-			foreach($object as $key => $val)
+			foreach ($object as $key => $val)
 			{
 				$this->variables[$key] = $val;
 			}
@@ -172,7 +172,7 @@ 
 	{
 		$args = func_get_args();
 		$output = new stdClass();
-		foreach($args as $arg)
+		foreach ($args as $arg)
 		{
 			$output->{$arg} = $this->get($arg);
 		}
@@ -197,7 +197,7 @@ 
 	function getObjectVars()
 	{
 		$output = new stdClass();
-		foreach($this->variables as $key => $val)
+		foreach ($this->variables as $key => $val)
 		{
 			$output->{$key} = $val;
 		}
@@ -226,7 +226,7 @@ 
 	}
 }
 
-if(version_compare(PHP_VERSION, '7.2', '<') && !class_exists('Object', false))
+if (version_compare(PHP_VERSION, '7.2', '<') && !class_exists('Object', false))
 {
 	class_alias('BaseObject', 'Object');
 }