xpressengine / xe-core

modules/board/board.view.php

Doc Comments +3 added lines

@@ -747,6 +747,9 @@
 		$this->setTemplateFile('write_form');
 	}
 
+	/**
+	 * @param ModuleObject $oDocumentModel
+	 */
 	function _getStatusNameList(&$oDocumentModel)
 	{
 		$resultList = array();

Braces +62 added lines, -38 removed lines

@@ -53,18 +53,15 @@ 
 			if($this->module_info->hide_category)
 			{
 				$this->module_info->use_category = ($this->module_info->hide_category == 'Y') ? 'N' : 'Y';
-			}
-			else if($this->module_info->use_category)
+			} else if($this->module_info->use_category)
 			{
 				$this->module_info->hide_category = ($this->module_info->use_category == 'Y') ? 'N' : 'Y';
-			}
-			else
+			} else
 			{
 				$this->module_info->hide_category = 'N';
 				$this->module_info->use_category = 'Y';
 			}
-		}
-		else
+		} else
 		{
 			$this->module_info->hide_category = 'Y';
 			$this->module_info->use_category = 'N';
@@ -84,8 +81,7 @@ 
 				$this->grant->write_comment = FALSE;
 				$this->grant->view = FALSE;
 			}
-		}
-		else
+		} else
 		{
 			$this->consultation = FALSE;
 		}
@@ -160,13 +156,17 @@ 
 		 * add extra vaiables to the search options
 		 **/
 		// use search options on the template (the search options key has been declared, based on the language selected)
-		foreach($this->search_option as $opt) $search_option[$opt] = Context::getLang($opt);
+		foreach($this->search_option as $opt) {
+			$search_option[$opt] = Context::getLang($opt);
+		}
 		$extra_keys = Context::get('extra_keys');
 		if($extra_keys)
 		{
 			foreach($extra_keys as $key => $val)
 			{
-				if($val->search == 'Y') $search_option['extra_vars'.$val->idx] = $val->name;
+				if($val->search == 'Y') {
+					$search_option['extra_vars'.$val->idx] = $val->name;
+				}
 			}
 		}
 		// remove a search option that is not public in member config
@@ -175,8 +175,9 @@ 
 		{
 			if(in_array($signupFormElement->title, $search_option))
 			{
-				if($signupFormElement->isPublic == 'N')
-					unset($search_option[$signupFormElement->name]);
+				if($signupFormElement->isPublic == 'N') {
+									unset($search_option[$signupFormElement->name]);
+				}
 			}
 		}
 		Context::set('search_option', $search_option);
@@ -194,7 +195,9 @@ 
 		// list config, columnList setting
 		$oBoardModel = getModel('board');
 		$this->listConfig = $oBoardModel->getListConfig($this->module_info->module_srl);
-		if(!$this->listConfig) $this->listConfig = array();
+		if(!$this->listConfig) {
+			$this->listConfig = array();
+		}
 		$this->_makeListColumnList();
 
 		// display the notice list
@@ -265,7 +268,9 @@ 
 				}
 
 				// check the manage grant
-				if($this->grant->manager) $oDocument->setGrant();
+				if($this->grant->manager) {
+					$oDocument->setGrant();
+				}
 
 				// if the consultation function is enabled, and the document is not a notice
 				if($this->consultation && !$oDocument->isNotice())
@@ -286,8 +291,7 @@ 
 					}
 				}
 
-			}
-			else
+			} else
 			{
 				// if the document is not existed, then alert a warning message
 				Context::set('document_srl','',true);
@@ -297,8 +301,7 @@ 
 		/**
 		 * if the document is not existed, get an empty document
 		 **/
-		}
-		else
+		} else
 		{
 			$oDocument = $oDocumentModel->getDocument(0);
 		}
@@ -313,8 +316,7 @@ 
 				$oDocument = $oDocumentModel->getDocument(0);
 				Context::set('document_srl','',true);
 				$this->alertMessage('msg_not_permitted');
-			}
-			else
+			} else
 			{
 				// add the document title to the browser
 				Context::addBrowserTitle($oDocument->getTitleText());
@@ -368,13 +370,16 @@ 
 		$downloadGrantCount = 0;
 		if(is_array($file_module_config->download_grant))
 		{
-			foreach($file_module_config->download_grant AS $value)
-				if($value) $downloadGrantCount++;
+			foreach($file_module_config->download_grant AS $value) {
+							if($value) $downloadGrantCount++;
+			}
 		}
 
 		if(is_array($file_module_config->download_grant) && $downloadGrantCount>0)
 		{
-			if(!Context::get('is_logged')) return $this->stop('msg_not_permitted_download');
+			if(!Context::get('is_logged')) {
+				return $this->stop('msg_not_permitted_download');
+			}
 			$logged_info = Context::get('logged_info');
 			if($logged_info->is_admin != 'Y')
 			{
@@ -397,7 +402,9 @@ 
 							break;
 						}
 					}
-					if(!$is_permitted) return $this->stop('msg_not_permitted_download');
+					if(!$is_permitted) {
+						return $this->stop('msg_not_permitted_download');
+					}
 				}
 			}
 		}
@@ -560,7 +567,9 @@ 
 				'allow_trackback', 'notify_message', 'status', 'comment_status');
 		$this->columnList = array_intersect($configColumList, $tableColumnList);
 
-		if(in_array('summary', $configColumList)) array_push($this->columnList, 'content');
+		if(in_array('summary', $configColumList)) {
+			array_push($this->columnList, 'content');
+		}
 
 		// default column list add
 		$defaultColumn = array('document_srl', 'module_srl', 'category_srl', 'lang_code', 'member_srl', 'last_update', 'comment_count', 'trackback_count', 'uploaded_count', 'status', 'regdate', 'title_bold', 'title_color');
@@ -654,8 +663,7 @@ 
 			{
 				$logged_info = Context::get('logged_info');
 				$group_srls = array_keys($logged_info->group_list);
-			}
-			else
+			} else
 			{
 				$group_srls = array();
 			}
@@ -672,10 +680,14 @@ 
 					{
 						$category_group_srls = explode(',',$category->group_srls);
 						$is_granted = FALSE;
-						if(count(array_intersect($group_srls, $category_group_srls))) $is_granted = TRUE;
+						if(count(array_intersect($group_srls, $category_group_srls))) {
+							$is_granted = TRUE;
+						}
 
 					}
-					if($is_granted) $category_list[$category_srl] = $category;
+					if($is_granted) {
+						$category_list[$category_srl] = $category;
+					}
 				}
 			}
 			Context::set('category_list', $category_list);
@@ -686,7 +698,9 @@ 
 		$oDocument = $oDocumentModel->getDocument(0, $this->grant->manager);
 		$oDocument->setDocument($document_srl);
 
-		if($oDocument->get('module_srl') == $oDocument->get('member_srl')) $savedDoc = TRUE;
+		if($oDocument->get('module_srl') == $oDocument->get('member_srl')) {
+			$savedDoc = TRUE;
+		}
 		$oDocument->add('module_srl', $this->module_srl);
 
 		if($oDocument->isExists() && $this->module_info->protect_content=="Y" && $oDocument->get('comment_count')>0 && $this->grant->manager==false)
@@ -712,17 +726,20 @@ 
 				if( !$logged_info )
 				{
 					return $this->dispBoardMessage('msg_not_permitted');
-				}
-				else if (($oPointModel->getPoint($logged_info->member_srl) + $pointForInsert )< 0 )
+				} else if (($oPointModel->getPoint($logged_info->member_srl) + $pointForInsert )< 0 )
 				{
 					return $this->dispBoardMessage('msg_not_enough_point');
 				}
 			}
 		}
-		if(!$oDocument->get('status')) $oDocument->add('status', $oDocumentModel->getDefaultStatus());
+		if(!$oDocument->get('status')) {
+			$oDocument->add('status', $oDocumentModel->getDefaultStatus());
+		}
 
 		$statusList = $this->_getStatusNameList($oDocumentModel);
-		if(count($statusList) > 0) Context::set('status_list', $statusList);
+		if(count($statusList) > 0) {
+			Context::set('status_list', $statusList);
+		}
 
 		// get Document status config value
 		Context::set('document_srl',$document_srl);
@@ -733,13 +750,18 @@ 
 		$oDocumentController->addXmlJsFilter($this->module_info->module_srl);
 
 		// if the document exists, then setup extra variabels on context
-		if($oDocument->isExists() && !$savedDoc) Context::set('extra_keys', $oDocument->getExtraVars());
+		if($oDocument->isExists() && !$savedDoc) {
+			Context::set('extra_keys', $oDocument->getExtraVars());
+		}
 
 		/**
 		 * add JS filters
 		 **/
-		if(Context::get('logged_info')->is_admin=='Y') Context::addJsFilter($this->module_path.'tpl/filter', 'insert_admin.xml');
-		else Context::addJsFilter($this->module_path.'tpl/filter', 'insert.xml');
+		if(Context::get('logged_info')->is_admin=='Y') {
+			Context::addJsFilter($this->module_path.'tpl/filter', 'insert_admin.xml');
+		} else {
+			Context::addJsFilter($this->module_path.'tpl/filter', 'insert.xml');
+		}
 
 		$oSecurity = new Security();
 		$oSecurity->encodeHTML('category_list.text', 'category_list.title');
@@ -1054,7 +1076,9 @@ 
 	function dispBoardMessage($msg_code)
 	{
 		$msg = Context::getLang($msg_code);
-		if(!$msg) $msg = $msg_code;
+		if(!$msg) {
+			$msg = $msg_code;
+		}
 		Context::set('message', $msg);
 		$this->setTemplateFile('message');
 	}

Spacing +150 added lines, -150 removed lines

@@ -23,15 +23,15 @@ 
 		/**
 		 * setup the module general information
 		 **/
-		if($this->module_info->list_count)
+		if ($this->module_info->list_count)
 		{
 			$this->list_count = $this->module_info->list_count;
 		}
-		if($this->module_info->search_list_count)
+		if ($this->module_info->search_list_count)
 		{
 			$this->search_list_count = $this->module_info->search_list_count;
 		}
-		if($this->module_info->page_count)
+		if ($this->module_info->page_count)
 		{
 			$this->page_count = $this->module_info->page_count;
 		}
@@ -41,20 +41,20 @@ 
 		$oDocumentModel = getModel('document');
 
 		$statusList = $this->_getStatusNameList($oDocumentModel);
-		if(isset($statusList['SECRET']))
+		if (isset($statusList['SECRET']))
 		{
 			$this->module_info->secret = 'Y';
 		}
 
 		// use_category <=1.5.x, hide_category >=1.7.x
 		$count_category = count($oDocumentModel->getCategoryList($this->module_info->module_srl));
-		if($count_category)
+		if ($count_category)
 		{
-			if($this->module_info->hide_category)
+			if ($this->module_info->hide_category)
 			{
 				$this->module_info->use_category = ($this->module_info->hide_category == 'Y') ? 'N' : 'Y';
 			}
-			else if($this->module_info->use_category)
+			else if ($this->module_info->use_category)
 			{
 				$this->module_info->hide_category = ($this->module_info->use_category == 'Y') ? 'N' : 'Y';
 			}
@@ -74,10 +74,10 @@ 
 		 * check the consultation function, if the user is admin then swich off consultation function
 		 * if the user is not logged, then disppear write document/write comment./ view document
 		 **/
-		if($this->module_info->consultation == 'Y' && !$this->grant->manager && !$this->grant->consultation_read)
+		if ($this->module_info->consultation == 'Y' && !$this->grant->manager && !$this->grant->consultation_read)
 		{
 			$this->consultation = TRUE;
-			if(!Context::get('is_logged'))
+			if (!Context::get('is_logged'))
 			{
 				$this->grant->list = FALSE;
 				$this->grant->write_document = FALSE;
@@ -94,11 +94,11 @@ 
 		 * setup the template path based on the skin
 		 * the default skin is default
 		 **/
-		$template_path = sprintf("%sskins/%s/",$this->module_path, $this->module_info->skin);
-		if(!is_dir($template_path)||!$this->module_info->skin)
+		$template_path = sprintf("%sskins/%s/", $this->module_path, $this->module_info->skin);
+		if (!is_dir($template_path) || !$this->module_info->skin)
 		{
 			$this->module_info->skin = 'default';
-			$template_path = sprintf("%sskins/%s/",$this->module_path, $this->module_info->skin);
+			$template_path = sprintf("%sskins/%s/", $this->module_path, $this->module_info->skin);
 		}
 		$this->setTemplatePath($template_path);
 
@@ -114,7 +114,7 @@ 
 		 **/
 		if (is_array($extra_keys))
 		{
-			foreach($extra_keys as $val)
+			foreach ($extra_keys as $val)
 			{
 				$this->order_target[] = $val->eid;
 			}
@@ -127,9 +127,9 @@ 
 
 		// remove [document_srl]_cpage from get_vars
 		$args = Context::getRequestVars();
-		foreach($args as $name => $value)
+		foreach ($args as $name => $value)
 		{
-			if(preg_match('/[0-9]+_cpage/', $name))
+			if (preg_match('/[0-9]+_cpage/', $name))
 			{
 				Context::set($name, '', TRUE);
 				Context::set($name, $value);
@@ -145,7 +145,7 @@ 
 		/**
 		 * check the access grant (all the grant has been set by the module object)
 		 **/
-		if(!$this->grant->access || !$this->grant->list)
+		if (!$this->grant->access || !$this->grant->list)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -160,22 +160,22 @@ 
 		 * add extra vaiables to the search options
 		 **/
 		// use search options on the template (the search options key has been declared, based on the language selected)
-		foreach($this->search_option as $opt) $search_option[$opt] = Context::getLang($opt);
+		foreach ($this->search_option as $opt) $search_option[$opt] = Context::getLang($opt);
 		$extra_keys = Context::get('extra_keys');
-		if($extra_keys)
+		if ($extra_keys)
 		{
-			foreach($extra_keys as $key => $val)
+			foreach ($extra_keys as $key => $val)
 			{
-				if($val->search == 'Y') $search_option['extra_vars'.$val->idx] = $val->name;
+				if ($val->search == 'Y') $search_option['extra_vars'.$val->idx] = $val->name;
 			}
 		}
 		// remove a search option that is not public in member config
 		$memberConfig = getModel('module')->getModuleConfig('member');
-		foreach($memberConfig->signupForm as $signupFormElement)
+		foreach ($memberConfig->signupForm as $signupFormElement)
 		{
-			if(in_array($signupFormElement->title, $search_option))
+			if (in_array($signupFormElement->title, $search_option))
 			{
-				if($signupFormElement->isPublic == 'N')
+				if ($signupFormElement->isPublic == 'N')
 					unset($search_option[$signupFormElement->name]);
 			}
 		}
@@ -183,7 +183,7 @@ 
 
 		$oDocumentModel = getModel('document');
 		$statusNameList = $this->_getStatusNameList($oDocumentModel);
-		if(count($statusNameList) > 0)
+		if (count($statusNameList) > 0)
 		{
 			Context::set('status_list', $statusNameList);
 		}
@@ -194,7 +194,7 @@ 
 		// list config, columnList setting
 		$oBoardModel = getModel('board');
 		$this->listConfig = $oBoardModel->getListConfig($this->module_info->module_srl);
-		if(!$this->listConfig) $this->listConfig = array();
+		if (!$this->listConfig) $this->listConfig = array();
 		$this->_makeListColumnList();
 
 		// display the notice list
@@ -218,12 +218,12 @@ 
 	/**
 	 * @brief display the category list
 	 **/
-	function dispBoardCategoryList(){
+	function dispBoardCategoryList() {
 		// check if the use_category option is enabled
-		if($this->module_info->use_category=='Y')
+		if ($this->module_info->use_category == 'Y')
 		{
 			// check the grant
-			if(!$this->grant->list)
+			if (!$this->grant->list)
 			{
 				Context::set('category_list', array());
 				return;
@@ -240,7 +240,7 @@ 
 	/**
 	 * @brief display the board conent view
 	 **/
-	function dispBoardContentView(){
+	function dispBoardContentView() {
 		// get the variable value
 		$document_srl = Context::get('document_srl');
 		$page = Context::get('page');
@@ -251,36 +251,36 @@ 
 		/**
 		 * if the document exists, then get the document information
 		 **/
-		if($document_srl)
+		if ($document_srl)
 		{
 			$oDocument = $oDocumentModel->getDocument($document_srl, false, true);
 
 			// if the document is existed
-			if($oDocument->isExists())
+			if ($oDocument->isExists())
 			{
 				// if the module srl is not consistent
-				if($oDocument->get('module_srl')!=$this->module_info->module_srl )
+				if ($oDocument->get('module_srl') != $this->module_info->module_srl)
 				{
 					return $this->stop('msg_invalid_request');
 				}
 
 				// check the manage grant
-				if($this->grant->manager) $oDocument->setGrant();
+				if ($this->grant->manager) $oDocument->setGrant();
 
 				// if the consultation function is enabled, and the document is not a notice
-				if($this->consultation && !$oDocument->isNotice())
+				if ($this->consultation && !$oDocument->isNotice())
 				{
 					$logged_info = Context::get('logged_info');
-					if($oDocument->get('member_srl')!=$logged_info->member_srl)
+					if ($oDocument->get('member_srl') != $logged_info->member_srl)
 					{
 						$oDocument = $oDocumentModel->getDocument(0);
 					}
 				}
 
 				// if the document is TEMP saved, check Grant
-				if($oDocument->getStatus() == 'TEMP')
+				if ($oDocument->getStatus() == 'TEMP')
 				{
-					if(!$oDocument->isGranted())
+					if (!$oDocument->isGranted())
 					{
 						$oDocument = $oDocumentModel->getDocument(0);
 					}
@@ -290,7 +290,7 @@ 
 			else
 			{
 				// if the document is not existed, then alert a warning message
-				Context::set('document_srl','',true);
+				Context::set('document_srl', '', true);
 				$this->alertMessage('msg_not_founded');
 			}
 
@@ -306,12 +306,12 @@ 
 		/**
 		 *check the document view grant
 		 **/
-		if($oDocument->isExists())
+		if ($oDocument->isExists())
 		{
-			if(!$this->grant->view && !$oDocument->isGranted())
+			if (!$this->grant->view && !$oDocument->isGranted())
 			{
 				$oDocument = $oDocumentModel->getDocument(0);
-				Context::set('document_srl','',true);
+				Context::set('document_srl', '', true);
 				$this->alertMessage('msg_not_permitted');
 			}
 			else
@@ -320,15 +320,15 @@ 
 				Context::addBrowserTitle($oDocument->getTitleText());
 
 				// update the document view count (if the document is not secret)
-				if(!$oDocument->isSecret() || $oDocument->isGranted())
+				if (!$oDocument->isSecret() || $oDocument->isGranted())
 				{
 					$oDocument->updateReadedCount();
 				}
 
 				// disappear the document if it is secret
-				if($oDocument->isSecret() && !$oDocument->isGranted())
+				if ($oDocument->isSecret() && !$oDocument->isGranted())
 				{
-					$oDocument->add('content',Context::getLang('thisissecret'));
+					$oDocument->add('content', Context::getLang('thisissecret'));
 				}
 			}
 		}
@@ -348,11 +348,11 @@ 
 	/**
 	 * @brief  display the document file list (can be used by API)
 	 **/
-	function dispBoardContentFileList(){
+	function dispBoardContentFileList() {
 		/**
 		 * check the access grant (all the grant has been set by the module object)
 		 **/
-		if(!$this->grant->access)
+		if (!$this->grant->access)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -363,41 +363,41 @@ 
 		// Check if a permission for file download is granted
 		// Get configurations (using module model object)
 		$oModuleModel = getModel('module');
-		$file_module_config = $oModuleModel->getModulePartConfig('file',$this->module_srl);
+		$file_module_config = $oModuleModel->getModulePartConfig('file', $this->module_srl);
 		
 		$downloadGrantCount = 0;
-		if(is_array($file_module_config->download_grant))
+		if (is_array($file_module_config->download_grant))
 		{
-			foreach($file_module_config->download_grant AS $value)
-				if($value) $downloadGrantCount++;
+			foreach ($file_module_config->download_grant AS $value)
+				if ($value) $downloadGrantCount++;
 		}
 
-		if(is_array($file_module_config->download_grant) && $downloadGrantCount>0)
+		if (is_array($file_module_config->download_grant) && $downloadGrantCount > 0)
 		{
-			if(!Context::get('is_logged')) return $this->stop('msg_not_permitted_download');
+			if (!Context::get('is_logged')) return $this->stop('msg_not_permitted_download');
 			$logged_info = Context::get('logged_info');
-			if($logged_info->is_admin != 'Y')
+			if ($logged_info->is_admin != 'Y')
 			{
-				$oModuleModel =& getModel('module');
+				$oModuleModel = & getModel('module');
 				$columnList = array('module_srl', 'site_srl');
 				$module_info = $oModuleModel->getModuleInfoByModuleSrl($this->module_srl, $columnList);
 
-				if(!$oModuleModel->isSiteAdmin($logged_info, $module_info->site_srl))
+				if (!$oModuleModel->isSiteAdmin($logged_info, $module_info->site_srl))
 				{
-					$oMemberModel =& getModel('member');
+					$oMemberModel = & getModel('member');
 					$member_groups = $oMemberModel->getMemberGroups($logged_info->member_srl, $module_info->site_srl);
 
 					$is_permitted = false;
-					for($i=0;$i<count($file_module_config->download_grant);$i++)
+					for ($i = 0; $i < count($file_module_config->download_grant); $i++)
 					{
 						$group_srl = $file_module_config->download_grant[$i];
-						if($member_groups[$group_srl])
+						if ($member_groups[$group_srl])
 						{
 							$is_permitted = true;
 							break;
 						}
 					}
-					if(!$is_permitted) return $this->stop('msg_not_permitted_download');
+					if (!$is_permitted) return $this->stop('msg_not_permitted_download');
 				}
 			}
 		}
@@ -405,7 +405,7 @@ 
 		$oDocumentModel = getModel('document');
 		$document_srl = Context::get('document_srl');
 		$oDocument = $oDocumentModel->getDocument($document_srl);
-		Context::set('file_list',$oDocument->getUploadedFiles());
+		Context::set('file_list', $oDocument->getUploadedFiles());
 
 		$oSecurity = new Security();
 		$oSecurity->encodeHTML('file_list..source_filename');
@@ -414,7 +414,7 @@ 
 	/**
 	 * @brief display the document comment list (can be used by API)
 	 **/
-	function dispBoardContentCommentList(){
+	function dispBoardContentCommentList() {
 		// check document view grant
 		$this->dispBoardContentView();
 
@@ -424,26 +424,26 @@ 
 		$comment_list = $oDocument->getComments();
 
 		// setup the comment list
-		if(is_array($comment_list))
+		if (is_array($comment_list))
 		{
-			foreach($comment_list as $key => $val)
+			foreach ($comment_list as $key => $val)
 			{
-				if(!$val->isAccessible())
+				if (!$val->isAccessible())
 				{
-					$val->add('content',Context::getLang('thisissecret'));
+					$val->add('content', Context::getLang('thisissecret'));
 				}
 			}
 		}
-		Context::set('comment_list',$comment_list);
+		Context::set('comment_list', $comment_list);
 
 	}
 
 	/**
 	 * @brief display notice list (can be used by API)
 	 **/
-	function dispBoardNoticeList(){
+	function dispBoardNoticeList() {
 		// check the grant
-		if(!$this->grant->list)
+		if (!$this->grant->list)
 		{
 			Context::set('notice_list', array());
 			return;
@@ -459,15 +459,15 @@ 
 	/**
 	 * @brief display board content list
 	 **/
-	function dispBoardContentList(){
+	function dispBoardContentList() {
 		// check the grant
-		if(!$this->grant->list)
+		if (!$this->grant->list)
 		{
 			Context::set('document_list', array());
 			Context::set('total_count', 0);
 			Context::set('total_page', 1);
 			Context::set('page', 1);
-			Context::set('page_navigation', new PageHandler(0,0,1,10));
+			Context::set('page_navigation', new PageHandler(0, 0, 1, 10));
 			return;
 		}
 
@@ -485,17 +485,17 @@ 
 		$args->search_keyword = Context::get('search_keyword');
 
 		$search_option = Context::get('search_option');
-		if($search_option==FALSE)
+		if ($search_option == FALSE)
 		{
 			$search_option = $this->search_option;
 		}
-		if(isset($search_option[$args->search_target])==FALSE)
+		if (isset($search_option[$args->search_target]) == FALSE)
 		{
 			$args->search_target = '';
 		}
 
 		// if the category is enabled, then get the category
-		if($this->module_info->use_category=='Y')
+		if ($this->module_info->use_category == 'Y')
 		{
 			$args->category_srl = Context::get('category');
 		}
@@ -503,21 +503,21 @@ 
 		// setup the sort index and order index
 		$args->sort_index = Context::get('sort_index');
 		$args->order_type = Context::get('order_type');
-		if(!in_array($args->sort_index, $this->order_target))
+		if (!in_array($args->sort_index, $this->order_target))
 		{
-			$args->sort_index = $this->module_info->order_target?$this->module_info->order_target:'list_order';
+			$args->sort_index = $this->module_info->order_target ? $this->module_info->order_target : 'list_order';
 		}
-		if(!in_array($args->order_type, array('asc','desc')))
+		if (!in_array($args->order_type, array('asc', 'desc')))
 		{
-			$args->order_type = $this->module_info->order_type?$this->module_info->order_type:'asc';
+			$args->order_type = $this->module_info->order_type ? $this->module_info->order_type : 'asc';
 		}
 
 		// set the current page of documents
 		$document_srl = Context::get('document_srl');
-		if(!$args->page && $document_srl)
+		if (!$args->page && $document_srl)
 		{
 			$oDocument = $oDocumentModel->getDocument($document_srl);
-			if($oDocument->isExists() && !$oDocument->isNotice())
+			if ($oDocument->isExists() && !$oDocument->isNotice())
 			{
 				$page = $oDocumentModel->getDocumentPage($oDocument, $args);
 				Context::set('page', $page);
@@ -526,13 +526,13 @@ 
 		}
 
 		// setup the list count to be serach list count, if the category or search keyword has been set
-		if($args->category_srl || $args->search_keyword)
+		if ($args->category_srl || $args->search_keyword)
 		{
 			$args->list_count = $this->search_list_count;
 		}
 
 		// if the consultation function is enabled,  the get the logged user information
-		if($this->consultation)
+		if ($this->consultation)
 		{
 			$logged_info = Context::get('logged_info');
 			$args->member_srl = $logged_info->member_srl;
@@ -560,18 +560,18 @@ 
 				'allow_trackback', 'notify_message', 'status', 'comment_status');
 		$this->columnList = array_intersect($configColumList, $tableColumnList);
 
-		if(in_array('summary', $configColumList)) array_push($this->columnList, 'content');
+		if (in_array('summary', $configColumList)) array_push($this->columnList, 'content');
 
 		// default column list add
 		$defaultColumn = array('document_srl', 'module_srl', 'category_srl', 'lang_code', 'member_srl', 'last_update', 'comment_count', 'trackback_count', 'uploaded_count', 'status', 'regdate', 'title_bold', 'title_color');
 
 		//TODO guestbook, blog style supports legacy codes.
-		if($this->module_info->skin == 'xe_guestbook' || $this->module_info->default_style == 'blog')
+		if ($this->module_info->skin == 'xe_guestbook' || $this->module_info->default_style == 'blog')
 		{
 			$defaultColumn = $tableColumnList;
 		}
 
-		if (in_array('last_post', $configColumList)){
+		if (in_array('last_post', $configColumList)) {
 			array_push($this->columnList, 'last_updater');
 		}
 
@@ -583,9 +583,9 @@ 
 		$this->columnList = array_unique(array_merge($this->columnList, $defaultColumn));
 
 		// add table name
-		foreach($this->columnList as $no => $value)
+		foreach ($this->columnList as $no => $value)
 		{
-			$this->columnList[$no] = 'documents.' . $value;
+			$this->columnList[$no] = 'documents.'.$value;
 		}
 	}
 
@@ -595,7 +595,7 @@ 
 	function dispBoardTagList()
 	{
 		// check if there is not grant fot view list, then alert an warning message
-		if(!$this->grant->list)
+		if (!$this->grant->list)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -609,14 +609,14 @@ 
 		$output = $oTagModel->getTagList($obj);
 
 		// automatically order
-		if(count($output->data))
+		if (count($output->data))
 		{
 			$numbers = array_keys($output->data);
 			shuffle($numbers);
 
-			if(count($output->data))
+			if (count($output->data))
 			{
-				foreach($numbers as $k => $v)
+				foreach ($numbers as $k => $v)
 				{
 					$tag_list[] = $output->data[$v];
 				}
@@ -637,7 +637,7 @@ 
 	function dispBoardWrite()
 	{
 		// check grant
-		if(!$this->grant->write_document)
+		if (!$this->grant->write_document)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -647,10 +647,10 @@ 
 		/**
 		 * check if the category option is enabled not not
 		 **/
-		if($this->module_info->use_category=='Y')
+		if ($this->module_info->use_category == 'Y')
 		{
 			// get the user group information
-			if(Context::get('is_logged'))
+			if (Context::get('is_logged'))
 			{
 				$logged_info = Context::get('logged_info');
 				$group_srls = array_keys($logged_info->group_list);
@@ -663,19 +663,19 @@ 
 
 			// check the grant after obtained the category list
 			$normal_category_list = $oDocumentModel->getCategoryList($this->module_srl);
-			if(count($normal_category_list))
+			if (count($normal_category_list))
 			{
-				foreach($normal_category_list as $category_srl => $category)
+				foreach ($normal_category_list as $category_srl => $category)
 				{
 					$is_granted = TRUE;
-					if($category->group_srls)
+					if ($category->group_srls)
 					{
-						$category_group_srls = explode(',',$category->group_srls);
+						$category_group_srls = explode(',', $category->group_srls);
 						$is_granted = FALSE;
-						if(count(array_intersect($group_srls, $category_group_srls))) $is_granted = TRUE;
+						if (count(array_intersect($group_srls, $category_group_srls))) $is_granted = TRUE;
 
 					}
-					if($is_granted) $category_list[$category_srl] = $category;
+					if ($is_granted) $category_list[$category_srl] = $category;
 				}
 			}
 			Context::set('category_list', $category_list);
@@ -686,46 +686,46 @@ 
 		$oDocument = $oDocumentModel->getDocument(0, $this->grant->manager);
 		$oDocument->setDocument($document_srl);
 
-		if($oDocument->get('module_srl') == $oDocument->get('member_srl')) $savedDoc = TRUE;
+		if ($oDocument->get('module_srl') == $oDocument->get('member_srl')) $savedDoc = TRUE;
 		$oDocument->add('module_srl', $this->module_srl);
 
-		if($oDocument->isExists() && $this->module_info->protect_content=="Y" && $oDocument->get('comment_count')>0 && $this->grant->manager==false)
+		if ($oDocument->isExists() && $this->module_info->protect_content == "Y" && $oDocument->get('comment_count') > 0 && $this->grant->manager == false)
 		{
 			return new Object(-1, 'msg_protect_content');
 		}
 
 		// if the document is not granted, then back to the password input form
 		$oModuleModel = getModel('module');
-		if($oDocument->isExists()&&!$oDocument->isGranted())
+		if ($oDocument->isExists() && !$oDocument->isGranted())
 		{
 			return $this->setTemplateFile('input_password_form');
 		}
 
-		if(!$oDocument->isExists())
+		if (!$oDocument->isExists())
 		{
-			$point_config = $oModuleModel->getModulePartConfig('point',$this->module_srl);
+			$point_config = $oModuleModel->getModulePartConfig('point', $this->module_srl);
 			$logged_info = Context::get('logged_info');
 			$oPointModel = getModel('point');
 			$pointForInsert = $point_config["insert_document"];
-			if($pointForInsert < 0)
+			if ($pointForInsert < 0)
 			{
-				if( !$logged_info )
+				if (!$logged_info)
 				{
 					return $this->dispBoardMessage('msg_not_permitted');
 				}
-				else if (($oPointModel->getPoint($logged_info->member_srl) + $pointForInsert )< 0 )
+				else if (($oPointModel->getPoint($logged_info->member_srl) + $pointForInsert) < 0)
 				{
 					return $this->dispBoardMessage('msg_not_enough_point');
 				}
 			}
 		}
-		if(!$oDocument->get('status')) $oDocument->add('status', $oDocumentModel->getDefaultStatus());
+		if (!$oDocument->get('status')) $oDocument->add('status', $oDocumentModel->getDefaultStatus());
 
 		$statusList = $this->_getStatusNameList($oDocumentModel);
-		if(count($statusList) > 0) Context::set('status_list', $statusList);
+		if (count($statusList) > 0) Context::set('status_list', $statusList);
 
 		// get Document status config value
-		Context::set('document_srl',$document_srl);
+		Context::set('document_srl', $document_srl);
 		Context::set('oDocument', $oDocument);
 
 		// apply xml_js_filter on header
@@ -733,12 +733,12 @@ 
 		$oDocumentController->addXmlJsFilter($this->module_info->module_srl);
 
 		// if the document exists, then setup extra variabels on context
-		if($oDocument->isExists() && !$savedDoc) Context::set('extra_keys', $oDocument->getExtraVars());
+		if ($oDocument->isExists() && !$savedDoc) Context::set('extra_keys', $oDocument->getExtraVars());
 
 		/**
 		 * add JS filters
 		 **/
-		if(Context::get('logged_info')->is_admin=='Y') Context::addJsFilter($this->module_path.'tpl/filter', 'insert_admin.xml');
+		if (Context::get('logged_info')->is_admin == 'Y') Context::addJsFilter($this->module_path.'tpl/filter', 'insert_admin.xml');
 		else Context::addJsFilter($this->module_path.'tpl/filter', 'insert.xml');
 
 		$oSecurity = new Security();
@@ -750,14 +750,14 @@ 
 	function _getStatusNameList(&$oDocumentModel)
 	{
 		$resultList = array();
-		if(!empty($this->module_info->use_status))
+		if (!empty($this->module_info->use_status))
 		{
 			$statusNameList = $oDocumentModel->getStatusNameList();
 			$statusList = explode('|@|', $this->module_info->use_status);
 
-			if(is_array($statusList))
+			if (is_array($statusList))
 			{
-				foreach($statusList as $key => $value)
+				foreach ($statusList as $key => $value)
 				{
 					$resultList[$value] = $statusNameList[$value];
 				}
@@ -772,7 +772,7 @@ 
 	function dispBoardDelete()
 	{
 		// check grant
-		if(!$this->grant->write_document)
+		if (!$this->grant->write_document)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -781,30 +781,30 @@ 
 		$document_srl = Context::get('document_srl');
 
 		// if document exists, get the document information
-		if($document_srl)
+		if ($document_srl)
 		{
 			$oDocumentModel = getModel('document');
 			$oDocument = $oDocumentModel->getDocument($document_srl);
 		}
 
 		// if the document is not existed, then back to the board content page
-		if(!$oDocument || !$oDocument->isExists())
+		if (!$oDocument || !$oDocument->isExists())
 		{
 			return $this->dispBoardContent();
 		}
 
 		// if the document is not granted, then back to the password input form
-		if(!$oDocument->isGranted())
+		if (!$oDocument->isGranted())
 		{
 			return $this->setTemplateFile('input_password_form');
 		}
 
-		if($this->module_info->protect_content=="Y" && $oDocument->get('comment_count')>0 && $this->grant->manager==false)
+		if ($this->module_info->protect_content == "Y" && $oDocument->get('comment_count') > 0 && $this->grant->manager == false)
 		{
 			return $this->dispBoardMessage('msg_protect_content');
 		}
 
-		Context::set('oDocument',$oDocument);
+		Context::set('oDocument', $oDocument);
 
 		/**
 		 * add JS filters
@@ -822,7 +822,7 @@ 
 		$document_srl = Context::get('document_srl');
 
 		// check grant
-		if(!$this->grant->write_comment)
+		if (!$this->grant->write_comment)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -830,13 +830,13 @@ 
 		// get the document information
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument($document_srl);
-		if(!$oDocument->isExists())
+		if (!$oDocument->isExists())
 		{
 			return $this->dispBoardMessage('msg_invalid_request');
 		}
 
 		// Check allow comment
-		if(!$oDocument->allowComment())
+		if (!$oDocument->allowComment())
 		{
 			return $this->dispBoardMessage('msg_not_allow_comment');
 		}
@@ -848,9 +848,9 @@ 
 		$oComment->add('module_srl', $this->module_srl);
 
 		// setup document variables on context
-		Context::set('oDocument',$oDocument);
-		Context::set('oSourceComment',$oSourceComment);
-		Context::set('oComment',$oComment);
+		Context::set('oDocument', $oDocument);
+		Context::set('oSourceComment', $oSourceComment);
+		Context::set('oComment', $oComment);
 
 		/**
 		 * add JS filter
@@ -866,7 +866,7 @@ 
 	function dispBoardReplyComment()
 	{
 		// check grant
-		if(!$this->grant->write_comment)
+		if (!$this->grant->write_comment)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -875,7 +875,7 @@ 
 		$parent_srl = Context::get('comment_srl');
 
 		// if the parent comment is not existed
-		if(!$parent_srl)
+		if (!$parent_srl)
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
@@ -885,11 +885,11 @@ 
 		$oSourceComment = $oCommentModel->getComment($parent_srl, $this->grant->manager);
 
 		// if the comment is not existed, opoup an error message
-		if(!$oSourceComment->isExists())
+		if (!$oSourceComment->isExists())
 		{
 			return $this->dispBoardMessage('msg_invalid_request');
 		}
-		if(Context::get('document_srl') && $oSourceComment->get('document_srl') != Context::get('document_srl'))
+		if (Context::get('document_srl') && $oSourceComment->get('document_srl') != Context::get('document_srl'))
 		{
 			return $this->dispBoardMessage('msg_invalid_request');
 		}
@@ -897,7 +897,7 @@ 
 		// Check allow comment
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument($oSourceComment->get('document_srl'));
-		if(!$oDocument->allowComment())
+		if (!$oDocument->allowComment())
 		{
 			return $this->dispBoardMessage('msg_not_allow_comment');
 		}
@@ -908,9 +908,9 @@ 
 		$oComment->add('document_srl', $oSourceComment->get('document_srl'));
 
 		// setup comment variables
-		Context::set('oSourceComment',$oSourceComment);
-		Context::set('oComment',$oComment);
-		Context::set('module_srl',$this->module_info->module_srl);
+		Context::set('oSourceComment', $oSourceComment);
+		Context::set('oComment', $oComment);
+		Context::set('module_srl', $this->module_info->module_srl);
 
 		/**
 		 * add JS filters
@@ -926,7 +926,7 @@ 
 	function dispBoardModifyComment()
 	{
 		// check grant
-		if(!$this->grant->write_comment)
+		if (!$this->grant->write_comment)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -936,7 +936,7 @@ 
 		$comment_srl = Context::get('comment_srl');
 
 		// if the comment is not existed
-		if(!$comment_srl)
+		if (!$comment_srl)
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
@@ -946,13 +946,13 @@ 
 		$oComment = $oCommentModel->getComment($comment_srl, $this->grant->manager);
 
 		// if the comment is not exited, alert an error message
-		if(!$oComment->isExists())
+		if (!$oComment->isExists())
 		{
 			return $this->dispBoardMessage('msg_invalid_request');
 		}
 
 		// if the comment is not granted, then back to the password input form
-		if(!$oComment->isGranted())
+		if (!$oComment->isGranted())
 		{
 			return $this->setTemplateFile('input_password_form');
 		}
@@ -975,7 +975,7 @@ 
 	function dispBoardDeleteComment()
 	{
 		// check grant
-		if(!$this->grant->write_comment)
+		if (!$this->grant->write_comment)
 		{
 			return $this->dispBoardMessage('msg_not_permitted');
 		}
@@ -984,25 +984,25 @@ 
 		$comment_srl = Context::get('comment_srl');
 
 		// if the comment exists, then get the comment information
-		if($comment_srl)
+		if ($comment_srl)
 		{
 			$oCommentModel = getModel('comment');
 			$oComment = $oCommentModel->getComment($comment_srl, $this->grant->manager);
 		}
 
 		// if the comment is not existed, then back to the board content page
-		if(!$oComment->isExists() )
+		if (!$oComment->isExists())
 		{
 			return $this->dispBoardContent();
 		}
 
 		// if the comment is not granted, then back to the password input form
-		if(!$oComment->isGranted())
+		if (!$oComment->isGranted())
 		{
 			return $this->setTemplateFile('input_password_form');
 		}
 
-		Context::set('oComment',$oComment);
+		Context::set('oComment', $oComment);
 
 		/**
 		 * add JS filters
@@ -1019,7 +1019,7 @@ 
 	{
 		$oTrackbackModel = getModel('trackback');
 
-		if(!$oTrackbackModel)
+		if (!$oTrackbackModel)
 		{
 			return;
 		}
@@ -1033,7 +1033,7 @@ 
 		$trackback = $output->data;
 
 		// if no trackback, then display the board content
-		if(!$trackback)
+		if (!$trackback)
 		{
 			return $this->dispBoardContent();
 		}
@@ -1054,7 +1054,7 @@ 
 	function dispBoardMessage($msg_code)
 	{
 		$msg = Context::getLang($msg_code);
-		if(!$msg) $msg = $msg_code;
+		if (!$msg) $msg = $msg_code;
 		Context::set('message', $msg);
 		$this->setTemplateFile('message');
 	}
@@ -1065,8 +1065,8 @@ 
 	 **/
 	function alertMessage($message)
 	{
-		$script =  sprintf('<script> jQuery(function(){ alert("%s"); } );</script>', Context::getLang($message));
-		Context::addHtmlFooter( $script );
+		$script = sprintf('<script> jQuery(function(){ alert("%s"); } );</script>', Context::getLang($message));
+		Context::addHtmlFooter($script);
 	}
 
 }

modules/comment/comment.admin.controller.php

Doc Comments +6 added lines

@@ -279,6 +279,9 @@ 
 		$this->setRedirectUrl($returnUrl);
 	}
 
+	/**
+	 * @param string $message_content
+	 */
 	private function _sendMessageForComment($message_content, $comment_srl_list)
 	{
 		$oCommunicationController = getController('communication');
@@ -308,6 +311,9 @@ 
 
 	/**
 	 * comment move to trash
+	 * @param ModuleObject $oCommentController
+	 * @param DB $oDB
+	 * @param string $message_content
 	 * @return void|object
 	 */
 	function _moveCommentToTrash($commentSrlList, &$oCommentController, &$oDB, $message_content = NULL)

Indentation +6 added lines, -6 removed lines

@@ -133,8 +133,8 @@ 
 
 					//mail to author of thread - START
 					/**
-				 	 * @todo Removed code send email to document author.
-					*/
+					 * @todo Removed code send email to document author.
+					 */
 					/*
 					if($document_author_email != $comment->email_address && $logged_info->email_address != $document_author_email)
 					{
@@ -341,10 +341,10 @@ 
 	}
 
 	/**
-	  * @fn procCommentAdminMoveToTrash
-	  * @brief move a comment to trash
-	  * @see commentModel::getCommentMenu
-	  */
+	 * @fn procCommentAdminMoveToTrash
+	 * @brief move a comment to trash
+	 * @see commentModel::getCommentMenu
+	 */
 	function procCommentAdminMoveToTrash()
 	{
 		$oDB = DB::getInstance();

Braces +9 added lines, -13 removed lines

@@ -32,8 +32,7 @@ 
 		if(!is_array($cart))
 		{
 			$comment_srl_list = explode('|@|', $cart);
-		}
-		else
+		} else
 		{
 			$comment_srl_list = $cart;
 		}
@@ -61,8 +60,7 @@ 
 		if(!is_array($cart))
 		{
 			$comment_srl_list = explode('|@|', $cart);
-		}
-		else
+		} else
 		{
 			$comment_srl_list = $cart;
 		}
@@ -74,8 +72,7 @@ 
 		if(!$output->toBool())
 		{
 			return $output;
-		}
-		else
+		} else
 		{
 			//update comment count for document
 			$updated_documents_arr = array();
@@ -202,8 +199,7 @@ 
 		if(!is_array($cart))
 		{
 			$comment_srl_list = explode('|@|', $cart);
-		}
-		else
+		} else
 		{
 			$comment_srl_list = $cart;
 		}
@@ -262,8 +258,7 @@ 
 		if($isTrash == 'true')
 		{
 			$msgCode = 'success_trashed';
-		}
-		else
+		} else
 		{
 			$msgCode = 'success_deleted';
 		}
@@ -355,7 +350,9 @@ 
 		$oCommentController = getController('comment');
 		$oComment = $oCommentModel->getComment($comment_srl, false);
 
-		if(!$oComment->isGranted()) return $this->stop('msg_not_permitted');
+		if(!$oComment->isGranted()) {
+			return $this->stop('msg_not_permitted');
+		}
 
 		$message_content = "";
 		$this->_moveCommentToTrash(array($comment_srl), $oCommentController, $oDB, $message_content);
@@ -410,8 +407,7 @@ 
 				if($_SESSION['comment_management'][$key])
 				{
 					unset($_SESSION['comment_management'][$key]);
-				}
-				else
+				} else
 				{
 					$_SESSION['comment_management'][$key] = TRUE;
 				}

Spacing +42 added lines, -42 removed lines

@@ -29,7 +29,7 @@ 
 	{
 		// Error display if none is selected
 		$cart = Context::get('cart');
-		if(!is_array($cart))
+		if (!is_array($cart))
 		{
 			$comment_srl_list = explode('|@|', $cart);
 		}
@@ -54,11 +54,11 @@ 
 
 		// Error display if none is selected
 		$cart = Context::get('cart');
-		if(!$cart)
+		if (!$cart)
 		{
 			return $this->stop('msg_cart_is_null');
 		}
-		if(!is_array($cart))
+		if (!is_array($cart))
 		{
 			$comment_srl_list = explode('|@|', $cart);
 		}
@@ -71,7 +71,7 @@ 
 		$args->status = $will_publish;
 		$args->comment_srls_list = $comment_srl_list;
 		$output = executeQuery('comment.updatePublishedStatus', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -90,16 +90,16 @@ 
 			//$oMemberModule = getModel("member");
 			//$logged_info = $oMemberModule->getMemberInfoByMemberSrl($logged_member_srl);
 			$new_status = ($will_publish) ? "published" : "unpublished";
-			foreach($comment_srl_list as $comment_srl)
+			foreach ($comment_srl_list as $comment_srl)
 			{
 				// check if comment already exists
 				$comment = $oCommentModel->getComment($comment_srl);
-				if($comment->comment_srl != $comment_srl)
+				if ($comment->comment_srl != $comment_srl)
 				{
 					return new Object(-1, 'msg_invalid_request');
 				}
 				$document_srl = $comment->document_srl;
-				if(!in_array($document_srl, $updated_documents_arr))
+				if (!in_array($document_srl, $updated_documents_arr))
 				{
 					$updated_documents_arr[] = $document_srl;
 					// update the number of comments
@@ -117,13 +117,13 @@ 
 					// send email to comment's author, all admins and thread(document) subscribers - START
 					// -------------------------------------------------------
 					$oMail = new Mail();
-					$mail_title = "[XE - " . $module_info->mid . "] comment(s) status changed to " . $new_status . " on document: \"" . $oDocument->getTitleText() . "\"";
+					$mail_title = "[XE - ".$module_info->mid."] comment(s) status changed to ".$new_status." on document: \"".$oDocument->getTitleText()."\"";
 					$oMail->setTitle($mail_title);
 					$mail_content = "
-						The comment #" . $comment_srl . " on document \"" . $oDocument->getTitleText() . "\" has been " . $new_status . " by admin of <strong><i>" . strtoupper($module_info->mid) . "</i></strong> module.
+						The comment #" . $comment_srl." on document \"".$oDocument->getTitleText()."\" has been ".$new_status." by admin of <strong><i>".strtoupper($module_info->mid)."</i></strong> module.
 						<br />
 						<br />Comment content:
-						" . $comment->content . "
+						" . $comment->content."
 						<br />
 						";
 					$oMail->setContent($mail_content);
@@ -145,17 +145,17 @@ 
 					*/
 					//mail to author of thread - STOP
 					//mail to all emails set for administrators - START
-					if($module_info->admin_mail)
+					if ($module_info->admin_mail)
 					{
 						$target_mail = explode(',', $module_info->admin_mail);
-						for($i = 0; $i < count($target_mail); $i++)
+						for ($i = 0; $i < count($target_mail); $i++)
 						{
 							$email_address = trim($target_mail[$i]);
-							if(!$email_address)
+							if (!$email_address)
 							{
 								continue;
 							}
-							if($author_email != $email_address)
+							if ($author_email != $email_address)
 							{
 								$oMail->setReceiptor($email_address, $email_address);
 								$oMail->send();
@@ -173,12 +173,12 @@ 
 
 		// for message send - start
 		$message_content = Context::get('message_content');
-		if($message_content)
+		if ($message_content)
 		{
 			$message_content = nl2br($message_content);
 		}
 
-		if($message_content)
+		if ($message_content)
 		{
 			$this->_sendMessageForComment($message_content, $comment_srl_list);
 		}
@@ -195,11 +195,11 @@ 
 
 		// Error display if none is selected
 		$cart = Context::get('cart');
-		if(!$cart)
+		if (!$cart)
 		{
 			return $this->stop('msg_cart_is_null');
 		}
-		if(!is_array($cart))
+		if (!is_array($cart))
 		{
 			$comment_srl_list = explode('|@|', $cart);
 		}
@@ -208,7 +208,7 @@ 
 			$comment_srl_list = $cart;
 		}
 		$comment_count = count($comment_srl_list);
-		if(!$comment_count)
+		if (!$comment_count)
 		{
 			return $this->stop('msg_cart_is_null');
 		}
@@ -220,34 +220,34 @@ 
 
 		// for message send - start
 		$message_content = Context::get('message_content');
-		if($message_content)
+		if ($message_content)
 		{
 			$message_content = nl2br($message_content);
 		}
 
-		if($message_content)
+		if ($message_content)
 		{
 			$this->_sendMessageForComment($message_content, $comment_srl_list);
 		}
 		// for message send - end
 		// comment into trash
-		if($isTrash == 'true')
+		if ($isTrash == 'true')
 		{
 			$this->_moveCommentToTrash($comment_srl_list, $oCommentController, $oDB, $message_content);
 		}
 
 		$deleted_count = 0;
 		// Delete the comment posting
-		for($i = 0; $i < $comment_count; $i++)
+		for ($i = 0; $i < $comment_count; $i++)
 		{
 			$comment_srl = trim($comment_srl_list[$i]);
-			if(!$comment_srl)
+			if (!$comment_srl)
 			{
 				continue;
 			}
 
 			$output = $oCommentController->deleteComment($comment_srl, TRUE, $isTrash);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				$oDB->rollback();
 				return $output;
@@ -259,7 +259,7 @@ 
 		$oDB->commit();
 
 		$msgCode = '';
-		if($isTrash == 'true')
+		if ($isTrash == 'true')
 		{
 			$msgCode = 'success_trashed';
 		}
@@ -290,12 +290,12 @@ 
 		$sender_member_srl = $logged_info->member_srl;
 
 		$comment_count = count($comment_srl_list);
-		for($i = 0; $i < $comment_count; $i++)
+		for ($i = 0; $i < $comment_count; $i++)
 		{
 			$comment_srl = $comment_srl_list[$i];
 			$oComment = $oCommentModel->getComment($comment_srl, TRUE);
 
-			if(!$oComment->get('member_srl') || $oComment->get('member_srl') == $sender_member_srl)
+			if (!$oComment->get('member_srl') || $oComment->get('member_srl') == $sender_member_srl)
 			{
 				continue;
 			}
@@ -312,16 +312,16 @@ 
 	 */
 	function _moveCommentToTrash($commentSrlList, &$oCommentController, &$oDB, $message_content = NULL)
 	{
-		require_once(_XE_PATH_ . 'modules/trash/model/TrashVO.php');
+		require_once(_XE_PATH_.'modules/trash/model/TrashVO.php');
 
-		if(is_array($commentSrlList))
+		if (is_array($commentSrlList))
 		{
 			$logged_info = Context::get('logged_info');
 			$oCommentModel = getModel('comment');
 			$commentItemList = $oCommentModel->getComments($commentSrlList);
 			$oTrashAdminController = getAdminController('trash');
 
-			foreach($commentItemList AS $key => $oComment)
+			foreach ($commentItemList AS $key => $oComment)
 			{
 				$oTrashVO = new TrashVO();
 				$oTrashVO->setTrashSrl(getNextSequence());
@@ -331,7 +331,7 @@ 
 				$oTrashVO->setDescription($message_content);
 
 				$output = $oTrashAdminController->insertTrash($oTrashVO);
-				if(!$output->toBool())
+				if (!$output->toBool())
 				{
 					$oDB->rollback();
 					return $output;
@@ -355,7 +355,7 @@ 
 		$oCommentController = getController('comment');
 		$oComment = $oCommentModel->getComment($comment_srl, false);
 
-		if(!$oComment->isGranted()) return $this->stop('msg_not_permitted');
+		if (!$oComment->isGranted()) return $this->stop('msg_not_permitted');
 
 		$message_content = "";
 		$this->_moveCommentToTrash(array($comment_srl), $oCommentController, $oDB, $message_content);
@@ -377,12 +377,12 @@ 
 	{
 		$comment_srl = trim(Context::get('comment_srl'));
 
-		if($comment_srl)
+		if ($comment_srl)
 		{
 			$args = new stdClass();
 			$args->comment_srl = $comment_srl;
 			$output = executeQuery('comment.deleteDeclaredComments', $args);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				return $output;
 			}
@@ -403,11 +403,11 @@ 
 
 		$output = $oCommentModel->getComments($commentSrlList);
 
-		if(is_array($output))
+		if (is_array($output))
 		{
-			foreach($output AS $key => $value)
+			foreach ($output AS $key => $value)
 			{
-				if($_SESSION['comment_management'][$key])
+				if ($_SESSION['comment_management'][$key])
 				{
 					unset($_SESSION['comment_management'][$key]);
 				}
@@ -428,7 +428,7 @@ 
 		$args = new stdClass();
 		$args->module_srl = $module_srl;
 		$output = executeQuery('comment.deleteModuleComments', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -437,7 +437,7 @@ 
 
 		//remove from cache
 		$oCacheHandler = CacheHandler::getInstance('object');
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			// Invalidate newest comments. Per document cache is invalidated inside document admin controller.
 			$oCacheHandler->invalidateGroupKey('newestCommentsList');
@@ -452,7 +452,7 @@ 
 	 */
 	function restoreTrash($originObject)
 	{
-		if(is_array($originObject))
+		if (is_array($originObject))
 		{
 			$originObject = (object) $originObject;
 		}
@@ -485,7 +485,7 @@ 
 	function emptyTrash($originObject)
 	{
 		$originObject = unserialize($originObject);
-		if(is_array($originObject))
+		if (is_array($originObject))
 		{
 			$originObject = (object) $originObject;
 		}

modules/comment/comment.controller.php

Doc Comments +3 added lines, -4 removed lines

@@ -160,7 +160,6 @@ 
 
 	/**
 	 * Check if module is using comment validation system
-	 * @param int $document_srl
 	 * @param int $module_srl
 	 * @return bool
 	 */
@@ -1151,7 +1150,7 @@ 
 
 	/**
 	 * Report a blamed comment
-	 * @param $comment_srl
+	 * @param string $comment_srl
 	 * @return void
 	 */
 	function declaredComment($comment_srl)
@@ -1356,7 +1355,7 @@ 
 	/**
 	 * Comment module config setting
 	 * @param int $srl
-	 * @param object $comment_config
+	 * @param stdClass $comment_config
 	 * @return Object
 	 */
 	function setCommentModuleConfig($srl, $comment_config)
@@ -1368,7 +1367,7 @@ 
 
 	/**
 	 * Get comment all list
-	 * @return void
+	 * @return Object|null
 	 */
 	function procCommentGetList()
 	{

Indentation +1 added lines, -1 removed lines

@@ -581,7 +581,7 @@
 				//mail to author of thread - START
 				/**
 				 * @todo Removed code send email to document author.
-				*/
+				 */
 				/*
 				if($document_author_email != $obj->email_address && $logged_info->email_address != $document_author_email)
 				{

Braces +22 added lines, -44 removed lines

@@ -211,14 +211,12 @@ 
 				if($logged_info->is_admin == 'Y')
 				{
 					$is_admin = TRUE;
-				}
-				else
+				} else
 				{
 					$is_admin = FALSE;
 				}
 			}
-		}
-		else
+		} else
 		{
 			$is_admin = FALSE;
 		}
@@ -226,14 +224,12 @@ 
 		if(!$using_validation)
 		{
 			$obj->status = 1;
-		}
-		else
+		} else
 		{
 			if($is_admin)
 			{
 				$obj->status = 1;
-			}
-			else
+			} else
 			{
 				$obj->status = 0;
 			}
@@ -310,8 +306,7 @@ 
 		if(!$obj->comment_srl)
 		{
 			$obj->comment_srl = getNextSequence();
-		}
-		elseif(!$is_admin && !$manual_inserted && !checkUserSequence($obj->comment_srl)) 
+		} elseif(!$is_admin && !$manual_inserted && !checkUserSequence($obj->comment_srl)) 
 		{
 			return new Object(-1, 'msg_not_permitted');
 		}
@@ -369,8 +364,7 @@ 
 			$list_args->head = $list_args->arrange = $obj->comment_srl;
 			$list_args->depth = 0;
 			// If parent comment exists, get information of the parent comment
-		}
-		else
+		} else
 		{
 			// get information of the parent comment posting
 			$parent_args = new stdClass();
@@ -393,8 +387,7 @@ 
 			{
 				$list_args->arrange = $obj->comment_srl;
 				// if the depth of comments is greater than 2, execute update.
-			}
-			else
+			} else
 			{
 				// get the top listed comment among those in lower depth and same head with parent's.
 				$p_args = new stdClass();
@@ -407,8 +400,7 @@ 
 				{
 					$list_args->arrange = $output->data->arrange;
 					$output = executeQuery('comment.updateCommentListArrange', $list_args);
-				}
-				else
+				} else
 				{
 					$list_args->arrange = $obj->comment_srl;
 				}
@@ -442,8 +434,7 @@ 
 		if(!$using_validation)
 		{
 			$output = $oDocumentController->updateCommentCount($document_srl, $comment_count, $obj->nick_name, TRUE);
-		}
-		else
+		} else
 		{
 			if($is_admin)
 			{
@@ -511,8 +502,7 @@ 
 		if(isset($obj->member_srl) && !is_null($obj->member_srl))
 		{
 			$member_info = $oMemberModel->getMemberInfoByMemberSrl($obj->member_srl);
-		}
-		else
+		} else
 		{
 			$member_info = new stdClass();
 			$member_info->is_admin = "N";
@@ -558,8 +548,7 @@ 
 					<br /><a href=\"" . getFullUrl('', 'module', 'admin', 'act', 'dispCommentAdminList', 'search_target', 'module', 'search_keyword', $obj->module_srl) . "\">" . getFullUrl('', 'module', 'admin', 'act', 'dispCommentAdminList', 'search_target', 'module', 'search_keyword', $obj->module_srl) . "</a>
 					";
 				$oMail->setContent($mail_content);
-			}
-			else
+			} else
 			{
 				$mail_content = "
 					Author: " . $member_info->nick_name . "
@@ -832,8 +821,7 @@ 
 			if(!$deleteAllComment)
 			{
 				return new Object(-1, 'fail_to_delete_have_children');
-			}
-			else
+			} else
 			{
 				foreach($childs as $val)
 				{
@@ -897,8 +885,7 @@ 
 		{
 			$this->_deleteDeclaredComments($args);
 			$this->_deleteVotedComments($args);
-		} 
-		else 
+		} else 
 		{
 			$args = new stdClass();
 			$args->upload_target_srl = $comment_srl;
@@ -941,8 +928,7 @@ 
 		{
 			$oDocument = new documentItem();
 			$oDocument->setAttribute($obj);
-		}
-		else
+		} else
 		{
 			$oDocument = $oDocumentModel->getDocument($document_srl);
 		}
@@ -1035,8 +1021,7 @@ 
 		{
 			$failed_voted = 'failed_voted';
 			$success_message = 'success_voted';
-		}
-		else
+		} else
 		{
 			$failed_voted = 'failed_blamed';
 			$success_message = 'success_blamed';
@@ -1079,8 +1064,7 @@ 
 		if($member_srl)
 		{
 			$args->member_srl = $member_srl;
-		}
-		else
+		} else
 		{
 			$args->ipaddress = $_SERVER['REMOTE_ADDR'];
 		}
@@ -1104,8 +1088,7 @@ 
 		{
 			$args->blamed_count = $oComment->get('blamed_count') + $point;
 			$output = executeQuery('comment.updateBlamedCount', $args);
-		}
-		else
+		} else
 		{
 			$args->voted_count = $oComment->get('voted_count') + $point;
 			$output = executeQuery('comment.updateVotedCount', $args);
@@ -1140,8 +1123,7 @@ 
 		if($point > 0)
 		{
 			$output->add('voted_count', $obj->after_point);
-		}
-		else
+		} else
 		{
 			$output->add('blamed_count', $obj->after_point);
 		}
@@ -1213,8 +1195,7 @@ 
 		if($member_srl)
 		{
 			$args->member_srl = $member_srl;
-		}
-		else
+		} else
 		{
 			$args->ipaddress = $_SERVER['REMOTE_ADDR'];
 		}
@@ -1236,8 +1217,7 @@ 
 		if($output->data->declared_count > 0)
 		{
 			$output = executeQuery('comment.updateDeclaredComment', $args);
-		}
-		else
+		} else
 		{
 			$output = executeQuery('comment.insertDeclaredComment', $args);
 		}
@@ -1304,8 +1284,7 @@ 
 		if(preg_match('/^([0-9,]+)$/', $module_srl))
 		{
 			$module_srl = explode(',', $module_srl);
-		}
-		else
+		} else
 		{
 			$module_srl = array($module_srl);
 		}
@@ -1395,8 +1374,7 @@ 
 					$value->content = strip_tags($value->content);
 				}
 			}
-		}
-		else
+		} else
 		{
 			global $lang;
 			$commentList = array();

Spacing +153 added lines, -153 removed lines

@@ -27,13 +27,13 @@ 
 	 */
 	function procCommentVoteUp()
 	{
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
 
 		$comment_srl = Context::get('target_srl');
-		if(!$comment_srl)
+		if (!$comment_srl)
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
@@ -41,14 +41,14 @@ 
 		$oCommentModel = getModel('comment');
 		$oComment = $oCommentModel->getComment($comment_srl, FALSE, FALSE);
 		$module_srl = $oComment->get('module_srl');
-		if(!$module_srl)
+		if (!$module_srl)
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
 
 		$oModuleModel = getModel('module');
 		$comment_config = $oModuleModel->getModulePartConfig('comment', $module_srl);
-		if($comment_config->use_vote_up == 'N')
+		if ($comment_config->use_vote_up == 'N')
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
@@ -65,13 +65,13 @@ 
 	 */
 	function procCommentVoteDown()
 	{
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
 
 		$comment_srl = Context::get('target_srl');
-		if(!$comment_srl)
+		if (!$comment_srl)
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
@@ -79,14 +79,14 @@ 
 		$oCommentModel = getModel('comment');
 		$oComment = $oCommentModel->getComment($comment_srl, FALSE, FALSE);
 		$module_srl = $oComment->get('module_srl');
-		if(!$module_srl)
+		if (!$module_srl)
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
 
 		$oModuleModel = getModel('module');
 		$comment_config = $oModuleModel->getModulePartConfig('comment', $module_srl);
-		if($comment_config->use_vote_down == 'N')
+		if ($comment_config->use_vote_down == 'N')
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
@@ -103,13 +103,13 @@ 
 	 */
 	function procCommentDeclare()
 	{
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
 
 		$comment_srl = Context::get('target_srl');
-		if(!$comment_srl)
+		if (!$comment_srl)
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
@@ -124,7 +124,7 @@ 
 	function triggerDeleteDocumentComments(&$obj)
 	{
 		$document_srl = $obj->document_srl;
-		if(!$document_srl)
+		if (!$document_srl)
 		{
 			return new Object();
 		}
@@ -139,7 +139,7 @@ 
 	function triggerDeleteModuleComments(&$obj)
 	{
 		$module_srl = $obj->module_srl;
-		if(!$module_srl)
+		if (!$module_srl)
 		{
 			return new Object();
 		}
@@ -166,7 +166,7 @@ 
 	 */
 	function isModuleUsingPublishValidation($module_srl = NULL)
 	{
-		if($module_srl == NULL)
+		if ($module_srl == NULL)
 		{
 			return FALSE;
 		}
@@ -175,7 +175,7 @@ 
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl);
 		$module_part_config = $oModuleModel->getModulePartConfig('comment', $module_info->module_srl);
 		$use_validation = FALSE;
-		if(isset($module_part_config->use_comment_validation) && $module_part_config->use_comment_validation == "Y")
+		if (isset($module_part_config->use_comment_validation) && $module_part_config->use_comment_validation == "Y")
 		{
 			$use_validation = TRUE;
 		}
@@ -190,12 +190,12 @@ 
 	 */
 	function insertComment($obj, $manual_inserted = FALSE)
 	{
-		if(!$manual_inserted && !checkCSRF())
+		if (!$manual_inserted && !checkCSRF())
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
 
-		if(!is_object($obj))
+		if (!is_object($obj))
 		{
 			$obj = new stdClass();
 		}
@@ -203,12 +203,12 @@ 
 		// check if comment's module is using comment validation and set the publish status to 0 (false)
 		// for inserting query, otherwise default is 1 (true - means comment is published)
 		$using_validation = $this->isModuleUsingPublishValidation($obj->module_srl);
-		if(!$manual_inserted)
+		if (!$manual_inserted)
 		{
-			if(Context::get('is_logged'))
+			if (Context::get('is_logged'))
 			{
 				$logged_info = Context::get('logged_info');
-				if($logged_info->is_admin == 'Y')
+				if ($logged_info->is_admin == 'Y')
 				{
 					$is_admin = TRUE;
 				}
@@ -223,13 +223,13 @@ 
 			$is_admin = FALSE;
 		}
 
-		if(!$using_validation)
+		if (!$using_validation)
 		{
 			$obj->status = 1;
 		}
 		else
 		{
-			if($is_admin)
+			if ($is_admin)
 			{
 				$obj->status = 1;
 			}
@@ -242,14 +242,14 @@ 
 
 		// call a trigger (before)
 		$output = ModuleHandler::triggerCall('comment.insertComment', 'before', $obj);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
 
 		// check if a posting of the corresponding document_srl exists
 		$document_srl = $obj->document_srl;
-		if(!$document_srl)
+		if (!$document_srl)
 		{
 			return new Object(-1, 'msg_invalid_document');
 		}
@@ -258,36 +258,36 @@ 
 		$oDocumentModel = getModel('document');
 
 		// even for manual_inserted if password exists, hash it.
-		if($obj->password)
+		if ($obj->password)
 		{
 			$obj->password = getModel('member')->hashPassword($obj->password);
 		}
 
 		// get the original posting
-		if(!$manual_inserted)
+		if (!$manual_inserted)
 		{
 			$oDocument = $oDocumentModel->getDocument($document_srl);
 
-			if($document_srl != $oDocument->document_srl)
+			if ($document_srl != $oDocument->document_srl)
 			{
 				return new Object(-1, 'msg_invalid_document');
 			}
-			if($oDocument->isLocked())
+			if ($oDocument->isLocked())
 			{
 				return new Object(-1, 'msg_invalid_request');
 			}
 
-			if($obj->homepage)
+			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;
 				}
 			}
 
 			// input the member's information if logged-in
-			if(Context::get('is_logged'))
+			if (Context::get('is_logged'))
 			{
 				$logged_info = Context::get('logged_info');
 				$obj->member_srl = $logged_info->member_srl;
@@ -302,16 +302,16 @@ 
 		}
 
 		// error display if neither of log-in info and user name exist.
-		if(!$logged_info->member_srl && !$obj->nick_name)
+		if (!$logged_info->member_srl && !$obj->nick_name)
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
 
-		if(!$obj->comment_srl)
+		if (!$obj->comment_srl)
 		{
 			$obj->comment_srl = getNextSequence();
 		}
-		elseif(!$is_admin && !$manual_inserted && !checkUserSequence($obj->comment_srl)) 
+		elseif (!$is_admin && !$manual_inserted && !checkUserSequence($obj->comment_srl)) 
 		{
 			return new Object(-1, 'msg_not_permitted');
 		}
@@ -322,32 +322,32 @@ 
 		// 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())
+		if (Mobile::isFromMobilePhone())
 		{
-			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);
 		}
 
-		if(!$obj->regdate)
+		if (!$obj->regdate)
 		{
 			$obj->regdate = date("YmdHis");
 		}
 
 		// remove iframe and script if not a top administrator on the session.
-		if($logged_info->is_admin != 'Y')
+		if ($logged_info->is_admin != 'Y')
 		{
 			$obj->content = removeHackTag($obj->content);
 		}
 
-		if(!$obj->notify_message)
+		if (!$obj->notify_message)
 		{
 			$obj->notify_message = 'N';
 		}
 
-		if(!$obj->is_secret)
+		if (!$obj->is_secret)
 		{
 			$obj->is_secret = 'N';
 		}
@@ -364,7 +364,7 @@ 
 		$list_args->regdate = $obj->regdate;
 
 		// If parent comment doesn't exist, set data directly
-		if(!$obj->parent_srl)
+		if (!$obj->parent_srl)
 		{
 			$list_args->head = $list_args->arrange = $obj->comment_srl;
 			$list_args->depth = 0;
@@ -378,7 +378,7 @@ 
 			$parent_output = executeQuery('comment.getCommentListItem', $parent_args);
 
 			// return if no parent comment exists
-			if(!$parent_output->toBool() || !$parent_output->data)
+			if (!$parent_output->toBool() || !$parent_output->data)
 			{
 				return;
 			}
@@ -389,7 +389,7 @@ 
 			$list_args->depth = $parent->depth + 1;
 
 			// if the depth of comments is less than 2, execute insert.
-			if($list_args->depth < 2)
+			if ($list_args->depth < 2)
 			{
 				$list_args->arrange = $obj->comment_srl;
 				// if the depth of comments is greater than 2, execute update.
@@ -403,7 +403,7 @@ 
 				$p_args->depth = $parent->depth;
 				$output = executeQuery('comment.getCommentParentNextSibling', $p_args);
 
-				if($output->data->arrange)
+				if ($output->data->arrange)
 				{
 					$list_args->arrange = $output->data->arrange;
 					$output = executeQuery('comment.updateCommentListArrange', $list_args);
@@ -416,14 +416,14 @@ 
 		}
 
 		$output = executeQuery('comment.insertCommentList', $list_args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
 
 		// insert comment
 		$output = executeQuery('comment.insertComment', $obj);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
@@ -439,29 +439,29 @@ 
 		$oDocumentController = getController('document');
 
 		// Update the number of comments in the post
-		if(!$using_validation)
+		if (!$using_validation)
 		{
 			$output = $oDocumentController->updateCommentCount($document_srl, $comment_count, $obj->nick_name, TRUE);
 		}
 		else
 		{
-			if($is_admin)
+			if ($is_admin)
 			{
 				$output = $oDocumentController->updateCommentCount($document_srl, $comment_count, $obj->nick_name, TRUE);
 			}
 		}
 
 		// grant autority of the comment
-		if(!$manual_inserted)
+		if (!$manual_inserted)
 		{
 			$this->addGrant($obj->comment_srl);
 		}
 
 		// call a trigger(after)
-		if($output->toBool())
+		if ($output->toBool())
 		{
 			$trigger_output = ModuleHandler::triggerCall('comment.insertComment', 'after', $obj);
-			if(!$trigger_output->toBool())
+			if (!$trigger_output->toBool())
 			{
 				$oDB->rollback();
 				return $trigger_output;
@@ -471,16 +471,16 @@ 
 		// commit
 		$oDB->commit();
 
-		if(!$manual_inserted)
+		if (!$manual_inserted)
 		{
 			// send a message if notify_message option in enabled in the original article
 			$oDocument->notify(Context::getLang('comment'), $obj->content);
 
 			// send a message if notify_message option in enabled in the original comment
-			if($obj->parent_srl)
+			if ($obj->parent_srl)
 			{
 				$oParent = $oCommentModel->getComment($obj->parent_srl);
-				if($oParent->get('member_srl') != $oDocument->get('member_srl'))
+				if ($oParent->get('member_srl') != $oDocument->get('member_srl'))
 				{
 					$oParent->notify(Context::getLang('comment'), $obj->content);
 				}
@@ -508,7 +508,7 @@ 
 		$oDocument = $oDocumentModel->getDocument($obj->document_srl);
 
 		$oMemberModel = getModel("member");
-		if(isset($obj->member_srl) && !is_null($obj->member_srl))
+		if (isset($obj->member_srl) && !is_null($obj->member_srl))
 		{
 			$member_info = $oMemberModel->getMemberInfoByMemberSrl($obj->member_srl);
 		}
@@ -528,47 +528,47 @@ 
 		$module_info = $oModuleModel->getModuleInfoByDocumentSrl($obj->document_srl);
 
 		// If there is no problem to register comment then send an email to all admin were set in module admin panel
-		if($module_info->admin_mail && $member_info->is_admin != 'Y')
+		if ($module_info->admin_mail && $member_info->is_admin != 'Y')
 		{
 			$oMail = new Mail();
 			$oMail->setSender($obj->email_address, $obj->email_address);
-			$mail_title = "[XE - " . Context::get('mid') . "] A new comment was posted on document: \"" . $oDocument->getTitleText() . "\"";
+			$mail_title = "[XE - ".Context::get('mid')."] A new comment was posted on document: \"".$oDocument->getTitleText()."\"";
 			$oMail->setTitle($mail_title);
-			$url_comment = getFullUrl('','document_srl',$obj->document_srl).'#comment_'.$obj->comment_srl;
-			if($using_validation)
+			$url_comment = getFullUrl('', 'document_srl', $obj->document_srl).'#comment_'.$obj->comment_srl;
+			if ($using_validation)
 			{
 				$url_approve = getFullUrl('', 'module', 'admin', 'act', 'procCommentAdminChangePublishedStatusChecked', 'cart[]', $obj->comment_srl, 'will_publish', '1', 'search_target', 'is_published', 'search_keyword', 'N');
 				$url_trash = getFullUrl('', 'module', 'admin', 'act', 'procCommentAdminDeleteChecked', 'cart[]', $obj->comment_srl, 'search_target', 'is_trash', 'search_keyword', 'true');
 				$mail_content = "
-					A new comment on the document \"" . $oDocument->getTitleText() . "\" is waiting for your approval.
+					A new comment on the document \"" . $oDocument->getTitleText()."\" is waiting for your approval.
 					<br />
 					<br />
-					Author: " . $member_info->nick_name . "
-					<br />Author e-mail: " . $member_info->email_address . "
-					<br />From : <a href=\"" . $url_comment . "\">" . $url_comment . "</a>
+					Author: " . $member_info->nick_name."
+					<br />Author e-mail: " . $member_info->email_address."
+					<br />From : <a href=\"" . $url_comment."\">".$url_comment."</a>
 					<br />Comment:
-					<br />\"" . $obj->content . "\"
+					<br />\"" . $obj->content."\"
 					<br />Document:
-					<br />\"" . $oDocument->getContentText(). "\"
+					<br />\"" . $oDocument->getContentText()."\"
 					<br />
 					<br />
-					Approve it: <a href=\"" . $url_approve . "\">" . $url_approve . "</a>
-					<br />Trash it: <a href=\"" . $url_trash . "\">" . $url_trash . "</a>
-					<br />Currently " . $nr_comments_not_approved . " comments on \"" . Context::get('mid') . "\" module are waiting for approval. Please visit the moderation panel:
-					<br /><a href=\"" . getFullUrl('', 'module', 'admin', 'act', 'dispCommentAdminList', 'search_target', 'module', 'search_keyword', $obj->module_srl) . "\">" . getFullUrl('', 'module', 'admin', 'act', 'dispCommentAdminList', 'search_target', 'module', 'search_keyword', $obj->module_srl) . "</a>
+					Approve it: <a href=\"" . $url_approve."\">".$url_approve."</a>
+					<br />Trash it: <a href=\"" . $url_trash."\">".$url_trash."</a>
+					<br />Currently " . $nr_comments_not_approved." comments on \"".Context::get('mid')."\" module are waiting for approval. Please visit the moderation panel:
+					<br /><a href=\"" . getFullUrl('', 'module', 'admin', 'act', 'dispCommentAdminList', 'search_target', 'module', 'search_keyword', $obj->module_srl)."\">".getFullUrl('', 'module', 'admin', 'act', 'dispCommentAdminList', 'search_target', 'module', 'search_keyword', $obj->module_srl)."</a>
 					";
 				$oMail->setContent($mail_content);
 			}
 			else
 			{
 				$mail_content = "
-					Author: " . $member_info->nick_name . "
-					<br />Author e-mail: " . $member_info->email_address . "
-					<br />From : <a href=\"" . $url_comment . "\">" . $url_comment . "</a>
+					Author: " . $member_info->nick_name."
+					<br />Author e-mail: " . $member_info->email_address."
+					<br />From : <a href=\"" . $url_comment."\">".$url_comment."</a>
 					<br />Comment:
-					<br />\"" . $obj->content . "\"
+					<br />\"" . $obj->content."\"
 					<br />Document:
-					<br />\"" . $oDocument->getContentText(). "\"
+					<br />\"" . $oDocument->getContentText()."\"
 					";
 				$oMail->setContent($mail_content);
 
@@ -597,10 +597,10 @@ 
 			$target_mail = explode(',', $admins_emails);
 
 			// send email to all admins - START
-			for($i = 0; $i < count($target_mail); $i++)
+			for ($i = 0; $i < count($target_mail); $i++)
 			{
 				$email_address = trim($target_mail[$i]);
-				if(!$email_address)
+				if (!$email_address)
 				{
 					continue;
 				}
@@ -646,12 +646,12 @@ 
 	 */
 	function updateComment($obj, $is_admin = FALSE, $manual_updated = FALSE)
 	{
-		if(!$manual_updated && !checkCSRF())
+		if (!$manual_updated && !checkCSRF())
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
 
-		if(!is_object($obj))
+		if (!is_object($obj))
 		{
 			$obj = new stdClass();
 		}
@@ -660,7 +660,7 @@ 
 
 		// call a trigger (before)
 		$output = ModuleHandler::triggerCall('comment.updateComment', 'before', $obj);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -670,7 +670,7 @@ 
 
 		// get the original data
 		$source_obj = $oCommentModel->getComment($obj->comment_srl);
-		if(!$source_obj->getMemberSrl())
+		if (!$source_obj->getMemberSrl())
 		{
 			$obj->member_srl = $source_obj->get('member_srl');
 			$obj->user_name = $source_obj->get('user_name');
@@ -680,30 +680,30 @@ 
 		}
 
 		// check if permission is granted
-		if(!$is_admin && !$source_obj->isGranted())
+		if (!$is_admin && !$source_obj->isGranted())
 		{
 			return new Object(-1, 'msg_not_permitted');
 		}
 
-		if($obj->password)
+		if ($obj->password)
 		{
 			$obj->password = getModel('member')->hashPassword($obj->password);
 		}
 
-		if($obj->homepage) 
+		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;
 			}
 		}
 
 		// set modifier's information if logged-in and posting author and modifier are matched.
-		if(Context::get('is_logged'))
+		if (Context::get('is_logged'))
 		{
 			$logged_info = Context::get('logged_info');
-			if($source_obj->member_srl == $logged_info->member_srl)
+			if ($source_obj->member_srl == $logged_info->member_srl)
 			{
 				$obj->member_srl = $logged_info->member_srl;
 				$obj->user_name = $logged_info->user_name;
@@ -714,7 +714,7 @@ 
 		}
 
 		// if nick_name of the logged-in author doesn't exist
-		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');
@@ -723,7 +723,7 @@ 
 			$obj->homepage = $source_obj->get('homepage');
 		}
 
-		if(!$obj->content)
+		if (!$obj->content)
 		{
 			$obj->content = $source_obj->get('content');
 		}
@@ -731,9 +731,9 @@ 
 		// remove XE's wn tags from contents
 		$obj->content = preg_replace('!<\!--(Before|After)(Document|Comment)\(([0-9]+),([0-9]+)\)-->!is', '', $obj->content);
 
-		if(Mobile::isFromMobilePhone())
+		if (Mobile::isFromMobilePhone())
 		{
-			if($obj->use_html != 'Y')
+			if ($obj->use_html != 'Y')
 			{
 				$obj->content = htmlspecialchars($obj->content, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 			}
@@ -741,7 +741,7 @@ 
 		}
 
 		// remove iframe and script if not a top administrator on the session
-		if($logged_info->is_admin != 'Y')
+		if ($logged_info->is_admin != 'Y')
 		{
 			$obj->content = removeHackTag($obj->content);
 		}
@@ -752,17 +752,17 @@ 
 
 		// Update
 		$output = executeQuery('comment.updateComment', $obj);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 
 		// call a trigger (after)
-		if($output->toBool())
+		if ($output->toBool())
 		{
 			$trigger_output = ModuleHandler::triggerCall('comment.updateComment', 'after', $obj);
-			if(!$trigger_output->toBool())
+			if (!$trigger_output->toBool())
 			{
 				$oDB->rollback();
 				return $trigger_output;
@@ -791,7 +791,7 @@ 
 
 		// check if comment already exists
 		$comment = $oCommentModel->getComment($comment_srl);
-		if($comment->comment_srl != $comment_srl)
+		if ($comment->comment_srl != $comment_srl)
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
@@ -800,28 +800,28 @@ 
 
 		// call a trigger (before)
 		$output = ModuleHandler::triggerCall('comment.deleteComment', 'before', $comment);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
 
 		// check if permission is granted
-		if(!$is_admin && !$comment->isGranted())
+		if (!$is_admin && !$comment->isGranted())
 		{
 			return new Object(-1, 'msg_not_permitted');
 		}
 
 		// check if child comment exists on the comment
 		$childs = $oCommentModel->getChildComments($comment_srl);
-		if(count($childs) > 0)
+		if (count($childs) > 0)
 		{
 			$deleteAllComment = TRUE;
-			if(!$is_admin)
+			if (!$is_admin)
 			{
 				$logged_info = Context::get('logged_info');
-				foreach($childs as $val)
+				foreach ($childs as $val)
 				{
-					if($val->member_srl != $logged_info->member_srl)
+					if ($val->member_srl != $logged_info->member_srl)
 					{
 						$deleteAllComment = FALSE;
 						break;
@@ -829,16 +829,16 @@ 
 				}
 			}
 
-			if(!$deleteAllComment)
+			if (!$deleteAllComment)
 			{
 				return new Object(-1, 'fail_to_delete_have_children');
 			}
 			else
 			{
-				foreach($childs as $val)
+				foreach ($childs as $val)
 				{
 					$output = $this->deleteComment($val->comment_srl, $is_admin, $isMoveToTrash);
-					if(!$output->toBool())
+					if (!$output->toBool())
 					{
 						return $output;
 					}
@@ -854,7 +854,7 @@ 
 		$args = new stdClass();
 		$args->comment_srl = $comment_srl;
 		$output = executeQuery('comment.deleteComment', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
@@ -866,14 +866,14 @@ 
 		$comment_count = $oCommentModel->getCommentCount($document_srl);
 
 		// only document is exists
-		if(isset($comment_count))
+		if (isset($comment_count))
 		{
 			// create the controller object of the document
 			$oDocumentController = getController('document');
 
 			// update comment count of the article posting
 			$output = $oDocumentController->updateCommentCount($document_srl, $comment_count, NULL, FALSE);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				$oDB->rollback();
 				return $output;
@@ -881,11 +881,11 @@ 
 		}
 
 		// call a trigger (after)
-		if($output->toBool())
+		if ($output->toBool())
 		{
 			$comment->isMoveToTrash = $isMoveToTrash;
 			$trigger_output = ModuleHandler::triggerCall('comment.deleteComment', 'after', $comment);
-			if(!$trigger_output->toBool())
+			if (!$trigger_output->toBool())
 			{
 				$oDB->rollback();
 				return $trigger_output;
@@ -893,7 +893,7 @@ 
 			unset($comment->isMoveToTrash);
 		}
 
-		if(!$isMoveToTrash)
+		if (!$isMoveToTrash)
 		{
 			$this->_deleteDeclaredComments($args);
 			$this->_deleteVotedComments($args);
@@ -937,7 +937,7 @@ 
 		$oCommentModel = getModel('comment');
 
 		// check if permission is granted
-		if(is_object($obj))
+		if (is_object($obj))
 		{
 			$oDocument = new documentItem();
 			$oDocument->setAttribute($obj);
@@ -947,7 +947,7 @@ 
 			$oDocument = $oDocumentModel->getDocument($document_srl);
 		}
 
-		if(!$oDocument->isExists() || !$oDocument->isGranted())
+		if (!$oDocument->isExists() || !$oDocument->isGranted())
 		{
 			return new Object(-1, 'msg_not_permitted');
 		}
@@ -956,23 +956,23 @@ 
 		$args = new stdClass();
 		$args->document_srl = $document_srl;
 		$comments = executeQueryArray('comment.getAllComments', $args);
-		if($comments->data)
+		if ($comments->data)
 		{
 			$commentSrlList = array();
-			foreach($comments->data as $comment)
+			foreach ($comments->data as $comment)
 			{
 				$commentSrlList[] = $comment->comment_srl;
 
 				// call a trigger (before)
 				$output = ModuleHandler::triggerCall('comment.deleteComment', 'before', $comment);
-				if(!$output->toBool())
+				if (!$output->toBool())
 				{
 					continue;
 				}
 
 				// call a trigger (after)
 				$output = ModuleHandler::triggerCall('comment.deleteComment', 'after', $comment);
-				if(!$output->toBool())
+				if (!$output->toBool())
 				{
 					continue;
 				}
@@ -982,7 +982,7 @@ 
 		// delete the comment
 		$args->document_srl = $document_srl;
 		$output = executeQuery('comment.deleteComments', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -991,7 +991,7 @@ 
 		$output = executeQuery('comment.deleteCommentsList', $args);
 
 		//delete declared, declared_log, voted_log
-		if(is_array($commentSrlList) && count($commentSrlList) > 0)
+		if (is_array($commentSrlList) && count($commentSrlList) > 0)
 		{
 			$args = new stdClass();
 			$args->comment_srl = join(',', $commentSrlList);
@@ -1031,7 +1031,7 @@ 
 	 */
 	function updateVotedCount($comment_srl, $point = 1)
 	{
-		if($point > 0)
+		if ($point > 0)
 		{
 			$failed_voted = 'failed_voted';
 			$success_message = 'success_voted';
@@ -1043,7 +1043,7 @@ 
 		}
 
 		// invalid vote if vote info exists in the session info.
-		if($_SESSION['voted_comment'][$comment_srl])
+		if ($_SESSION['voted_comment'][$comment_srl])
 		{
 			return new Object(-1, $failed_voted);
 		}
@@ -1052,21 +1052,21 @@ 
 		$oComment = $oCommentModel->getComment($comment_srl, FALSE, FALSE);
 
 		// invalid vote if both ip addresses between author's and the current user are same.
-		if($oComment->get('ipaddress') == $_SERVER['REMOTE_ADDR'])
+		if ($oComment->get('ipaddress') == $_SERVER['REMOTE_ADDR'])
 		{
 			$_SESSION['voted_comment'][$comment_srl] = TRUE;
 			return new Object(-1, $failed_voted);
 		}
 
 		// if the comment author is a member
-		if($oComment->get('member_srl'))
+		if ($oComment->get('member_srl'))
 		{
 			// create the member model object
 			$oMemberModel = getModel('member');
 			$member_srl = $oMemberModel->getLoggedMemberSrl();
 
 			// session registered if the author information matches to the current logged-in user's.
-			if($member_srl && $member_srl == abs($oComment->get('member_srl')))
+			if ($member_srl && $member_srl == abs($oComment->get('member_srl')))
 			{
 				$_SESSION['voted_comment'][$comment_srl] = TRUE;
 				return new Object(-1, $failed_voted);
@@ -1076,7 +1076,7 @@ 
 		$args = new stdClass();
 
 		// If logged-in, use the member_srl. otherwise use the ipaddress.
-		if($member_srl)
+		if ($member_srl)
 		{
 			$args->member_srl = $member_srl;
 		}
@@ -1089,7 +1089,7 @@ 
 		$output = executeQuery('comment.getCommentVotedLogInfo', $args);
 
 		// session registered if log info contains recommendation vote log.
-		if($output->data->count)
+		if ($output->data->count)
 		{
 			$_SESSION['voted_comment'][$comment_srl] = TRUE;
 			return new Object(-1, $failed_voted);
@@ -1100,7 +1100,7 @@ 
 		$oDB->begin();
 
 		// update the number of votes
-		if($point < 0)
+		if ($point < 0)
 		{
 			$args->blamed_count = $oComment->get('blamed_count') + $point;
 			$output = executeQuery('comment.updateBlamedCount', $args);
@@ -1124,7 +1124,7 @@ 
 		$obj->before_point = ($point < 0) ? $oComment->get('blamed_count') : $oComment->get('voted_count');
 		$obj->after_point = ($point < 0) ? $args->blamed_count : $args->voted_count;
 		$trigger_output = ModuleHandler::triggerCall('comment.updateVotedCount', 'after', $obj);
-		if(!$trigger_output->toBool())
+		if (!$trigger_output->toBool())
 		{
 			$oDB->rollback();
 			return $trigger_output;
@@ -1137,7 +1137,7 @@ 
 
 		// Return the result
 		$output = new Object(0, $success_message);
-		if($point > 0)
+		if ($point > 0)
 		{
 			$output->add('voted_count', $obj->after_point);
 		}
@@ -1157,7 +1157,7 @@ 
 	function declaredComment($comment_srl)
 	{
 		// Fail if session information already has a reported document
-		if($_SESSION['declared_comment'][$comment_srl])
+		if ($_SESSION['declared_comment'][$comment_srl])
 		{
 			return new Object(-1, 'failed_declared');
 		}
@@ -1166,7 +1166,7 @@ 
 		$args = new stdClass();
 		$args->comment_srl = $comment_srl;
 		$output = executeQuery('comment.getDeclaredComment', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -1178,7 +1178,7 @@ 
 
 		// Call a trigger (before)
 		$trigger_output = ModuleHandler::triggerCall('comment.declaredComment', 'before', $trigger_obj);
-		if(!$trigger_output->toBool())
+		if (!$trigger_output->toBool())
 		{
 			return $trigger_output;
 		}
@@ -1188,21 +1188,21 @@ 
 		$oComment = $oCommentModel->getComment($comment_srl, FALSE, FALSE);
 
 		// failed if both ip addresses between author's and the current user are same.
-		if($oComment->get('ipaddress') == $_SERVER['REMOTE_ADDR'])
+		if ($oComment->get('ipaddress') == $_SERVER['REMOTE_ADDR'])
 		{
 			$_SESSION['declared_comment'][$comment_srl] = TRUE;
 			return new Object(-1, 'failed_declared');
 		}
 
 		// if the comment author is a member
-		if($oComment->get('member_srl'))
+		if ($oComment->get('member_srl'))
 		{
 			// create the member model object
 			$oMemberModel = getModel('member');
 			$member_srl = $oMemberModel->getLoggedMemberSrl();
 
 			// session registered if the author information matches to the current logged-in user's.
-			if($member_srl && $member_srl == abs($oComment->get('member_srl')))
+			if ($member_srl && $member_srl == abs($oComment->get('member_srl')))
 			{
 				$_SESSION['declared_comment'][$comment_srl] = TRUE;
 				return new Object(-1, 'failed_declared');
@@ -1210,7 +1210,7 @@ 
 		}
 
 		// If logged-in, use the member_srl. otherwise use the ipaddress.
-		if($member_srl)
+		if ($member_srl)
 		{
 			$args->member_srl = $member_srl;
 		}
@@ -1222,7 +1222,7 @@ 
 		$log_output = executeQuery('comment.getCommentDeclaredLogInfo', $args);
 
 		// session registered if log info contains report log.
-		if($log_output->data->count)
+		if ($log_output->data->count)
 		{
 			$_SESSION['declared_comment'][$comment_srl] = TRUE;
 			return new Object(-1, 'failed_declared');
@@ -1233,7 +1233,7 @@ 
 		$oDB->begin();
 
 		// execute insert
-		if($output->data->declared_count > 0)
+		if ($output->data->declared_count > 0)
 		{
 			$output = executeQuery('comment.updateDeclaredComment', $args);
 		}
@@ -1242,7 +1242,7 @@ 
 			$output = executeQuery('comment.insertDeclaredComment', $args);
 		}
 
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
@@ -1254,7 +1254,7 @@ 
 		// Call a trigger (after)
 		$trigger_obj->declared_count = $declared_count + 1;
 		$trigger_output = ModuleHandler::triggerCall('comment.declaredComment', 'after', $trigger_obj);
-		if(!$trigger_output->toBool())
+		if (!$trigger_output->toBool())
 		{
 			$oDB->rollback();
 			return $trigger_output;
@@ -1279,7 +1279,7 @@ 
 	function addCommentPopupMenu($url, $str, $icon = '', $target = 'self')
 	{
 		$comment_popup_menu_list = Context::get('comment_popup_menu_list');
-		if(!is_array($comment_popup_menu_list))
+		if (!is_array($comment_popup_menu_list))
 		{
 			$comment_popup_menu_list = array();
 		}
@@ -1301,7 +1301,7 @@ 
 	function procCommentInsertModuleConfig()
 	{
 		$module_srl = Context::get('target_module_srl');
-		if(preg_match('/^([0-9,]+)$/', $module_srl))
+		if (preg_match('/^([0-9,]+)$/', $module_srl))
 		{
 			$module_srl = explode(',', $module_srl);
 		}
@@ -1312,33 +1312,33 @@ 
 
 		$comment_config = new stdClass();
 		$comment_config->comment_count = (int) Context::get('comment_count');
-		if(!$comment_config->comment_count)
+		if (!$comment_config->comment_count)
 		{
 			$comment_config->comment_count = 50;
 		}
 
 		$comment_config->use_vote_up = Context::get('use_vote_up');
-		if(!$comment_config->use_vote_up)
+		if (!$comment_config->use_vote_up)
 		{
 			$comment_config->use_vote_up = 'Y';
 		}
 
 		$comment_config->use_vote_down = Context::get('use_vote_down');
-		if(!$comment_config->use_vote_down)
+		if (!$comment_config->use_vote_down)
 		{
 			$comment_config->use_vote_down = 'Y';
 		}
 
 		$comment_config->use_comment_validation = Context::get('use_comment_validation');
-		if(!$comment_config->use_comment_validation)
+		if (!$comment_config->use_comment_validation)
 		{
 			$comment_config->use_comment_validation = 'N';
 		}
 
-		for($i = 0; $i < count($module_srl); $i++)
+		for ($i = 0; $i < count($module_srl); $i++)
 		{
 			$srl = trim($module_srl[$i]);
-			if(!$srl)
+			if (!$srl)
 			{
 				continue;
 			}
@@ -1372,25 +1372,25 @@ 
 	 */
 	function procCommentGetList()
 	{
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return new Object(-1, 'msg_not_permitted');
 		}
 
 		$commentSrls = Context::get('comment_srls');
-		if($commentSrls)
+		if ($commentSrls)
 		{
 			$commentSrlList = explode(',', $commentSrls);
 		}
 
-		if(count($commentSrlList) > 0)
+		if (count($commentSrlList) > 0)
 		{
 			$oCommentModel = getModel('comment');
 			$commentList = $oCommentModel->getComments($commentSrlList);
 
-			if(is_array($commentList))
+			if (is_array($commentList))
 			{
-				foreach($commentList as $value)
+				foreach ($commentList as $value)
 				{
 					$value->content = strip_tags($value->content);
 				}
@@ -1415,9 +1415,9 @@ 
 		$commentConfig = $oModuleModel->getModulePartConfig('comment', $obj->originModuleSrl);
 
 		$oModuleController = getController('module');
-		if(is_array($obj->moduleSrlList))
+		if (is_array($obj->moduleSrlList))
 		{
-			foreach($obj->moduleSrlList as $moduleSrl)
+			foreach ($obj->moduleSrlList as $moduleSrl)
 			{
 				$oModuleController->insertModulePartConfig('comment', $moduleSrl, $commentConfig);
 			}

modules/comment/comment.view.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 	/**
 	 * Add a form fot comment setting on the additional setting of module
 	 * @param string $obj
-	 * @return string
+	 * @return Object
 	 */
 	function triggerDispCommentAdditionSetup(&$obj)
 	{

Spacing +3 added lines, -3 removed lines

@@ -31,12 +31,12 @@ 
 		$current_module_srl = Context::get('module_srl');
 		$current_module_srls = Context::get('module_srls');
 
-		if(!$current_module_srl && !$current_module_srls)
+		if (!$current_module_srl && !$current_module_srls)
 		{
 			// get information of the selected module
 			$current_module_info = Context::get('current_module_info');
 			$current_module_srl = $current_module_info->module_srl;
-			if(!$current_module_srl)
+			if (!$current_module_srl)
 			{
 				return new Object();
 			}
@@ -54,7 +54,7 @@ 
 
 		// Set a template file
 		$oTemplate = TemplateHandler::getInstance();
-		$tpl = $oTemplate->compile($this->module_path . 'tpl', 'comment_module_config');
+		$tpl = $oTemplate->compile($this->module_path.'tpl', 'comment_module_config');
 		$obj .= $tpl;
 
 		return new Object();

modules/communication/communication.mobile.php

Doc Comments +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
 
 	/**
 	 * Display message box
-	 * @return void|Object (void : success, Object : fail)
+	 * @return ModuleObject|null (void : success, Object : fail)
 	 */
 	function dispCommunicationMessages()
 	{
@@ -130,7 +130,7 @@ 
 
 	/**
 	 * Display message sending
-	 * @return void|Object (void : success, Object : fail)
+	 * @return ModuleObject|null (void : success, Object : fail)
 	 */
 	function dispCommunicationSendMessage()
 	{

Braces +1 added lines, -2 removed lines

@@ -21,8 +21,7 @@
 		if($absolute_url)
 		{
 			return getFullUrl('','vid',$vid, 'mid',$mid, 'act',$format);
-		}
-		else
+		} else
 		{
 			return getUrl('','vid',$vid, 'mid',$mid, 'act',$format);
 		}

Spacing +19 added lines, -19 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 
-require_once(_XE_PATH_ . 'modules/communication/communication.view.php');
+require_once(_XE_PATH_.'modules/communication/communication.view.php');
 
 /**
  * @class  communicationMobile
@@ -25,7 +25,7 @@ 
 
 		$oLayoutModel = getModel('layout');
 		$layout_info = $oLayoutModel->getLayout($this->communication_config->mlayout_srl);
-		if($layout_info)
+		if ($layout_info)
 		{
 			$this->module_info->mlayout_srl = $this->communication_config->mlayout_srl;
 			$this->setLayoutPath($layout_info->path);
@@ -39,13 +39,13 @@ 
 	function dispCommunicationMessages()
 	{
 		// Error appears if not logged-in
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return $this->stop('msg_not_logged');
 		}
 
 		$logged_info = Context::get('logged_info');
-		if(!array_key_exists('dispCommunicationMessages', $logged_info->menu_list))
+		if (!array_key_exists('dispCommunicationMessages', $logged_info->menu_list))
 		{
 			return $this->stop('msg_invalid_request');
 		}
@@ -53,7 +53,7 @@ 
 		// Set the variables
 		$message_srl = Context::get('message_srl');
 		$message_type = Context::get('message_type');
-		if(!in_array($message_type, array('R', 'S', 'T')))
+		if (!in_array($message_type, array('R', 'S', 'T')))
 		{
 			$message_type = 'R';
 			Context::set('message_type', $message_type);
@@ -61,37 +61,37 @@ 
 		$oCommunicationModel = getModel('communication');
 
 		// extract contents if message_srl exists
-		if($message_srl)
+		if ($message_srl)
 		{
 			$templateFile = 'read_message';
 			$columnList = array('message_srl', 'sender_srl', 'receiver_srl', 'message_type', 'title', 'content', 'readed', 'regdate');
 			$message = $oCommunicationModel->getSelectedMessage($message_srl, $columnList);
 
-			switch($message->message_type)
+			switch ($message->message_type)
 			{
 				case 'R':
-					if($message->receiver_srl != $logged_info->member_srl)
+					if ($message->receiver_srl != $logged_info->member_srl)
 					{
 						return $this->stop('msg_invalid_request');
 					}
 					break;
 
 				case 'S':
-					if($message->sender_srl != $logged_info->member_srl)
+					if ($message->sender_srl != $logged_info->member_srl)
 					{
 						return $this->stop('msg_invalid_request');
 					}
 					break;
 
 				case 'T':
-					if($message->receiver_srl != $logged_info->member_srl && $message->sender_srl != $logged_info->member_srl)
+					if ($message->receiver_srl != $logged_info->member_srl && $message->sender_srl != $logged_info->member_srl)
 					{
 						return $this->stop('msg_invalid_request');
 					}
 					break;
 			}
 
-			if($message->message_srl == $message_srl && ($message->receiver_srl == $logged_info->member_srl || $message->sender_srl == $logged_info->member_srl))
+			if ($message->message_srl == $message_srl && ($message->receiver_srl == $logged_info->member_srl || $message->sender_srl == $logged_info->member_srl))
 			{
 				stripEmbedTagForAdmin($message->content, $message->sender_srl);
 				Context::set('message', $message);
@@ -138,7 +138,7 @@ 
 		$oMemberModel = getModel('member');
 
 		// Error appears if not logged-in
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return $this->stop('msg_not_logged');
 		}
@@ -148,32 +148,32 @@ 
 		// get receipient's information
 		// check inalid request
 		$receiver_srl = Context::get('receiver_srl');
-		if(!$receiver_srl)
+		if (!$receiver_srl)
 		{
 			return $this->stop('msg_invalid_request');
 		}
 
 		// check receiver and sender are same
-		if($logged_info->member_srl == $receiver_srl)
+		if ($logged_info->member_srl == $receiver_srl)
 		{
 			return $this->stop('msg_cannot_send_to_yourself');
 		}
 
 		// get message_srl of the original message if it is a reply
 		$message_srl = Context::get('message_srl');
-		if($message_srl)
+		if ($message_srl)
 		{
 			$source_message = $oCommunicationModel->getSelectedMessage($message_srl);
-			if($source_message->message_srl == $message_srl && $source_message->sender_srl == $receiver_srl)
+			if ($source_message->message_srl == $message_srl && $source_message->sender_srl == $receiver_srl)
 			{
-				$source_message->title = "[re] " . $source_message->title;
-				$source_message->content = "\r\n<br />\r\n<br /><div style=\"padding-left:5px; border-left:5px solid #DDDDDD;\">" . trim($source_message->content) . "</div>";
+				$source_message->title = "[re] ".$source_message->title;
+				$source_message->content = "\r\n<br />\r\n<br /><div style=\"padding-left:5px; border-left:5px solid #DDDDDD;\">".trim($source_message->content)."</div>";
 				Context::set('source_message', $source_message);
 			}
 		}
 
 		$receiver_info = $oMemberModel->getMemberInfoByMemberSrl($receiver_srl);
-		if(!$receiver_info)
+		if (!$receiver_info)
 		{
 			return $this->stop('msg_invalid_request');
 		}

modules/communication/communication.model.php

Doc Comments -1 removed lines

@@ -101,7 +101,6 @@
 
 	/**
 	  * @brief check member's grant
-	  * @param object $member_info
 	  * @param array $arrGrant
 	  * @return boolean
 	  */

Indentation +10 added lines, -10 removed lines

@@ -61,11 +61,11 @@ 
 	}
 
 	/**
-	  * @brief get grant array for insert to database. table module_config's config field 
-	  * @param string $default
-	  * @param array $group
-	  * @return array
-	  */
+	 * @brief get grant array for insert to database. table module_config's config field 
+	 * @param string $default
+	 * @param array $group
+	 * @return array
+	 */
 	function getGrantArray($default, $group)
 	{
 		$grant = array();
@@ -100,11 +100,11 @@ 
 	}
 
 	/**
-	  * @brief check member's grant
-	  * @param object $member_info
-	  * @param array $arrGrant
-	  * @return boolean
-	  */
+	 * @brief check member's grant
+	 * @param object $member_info
+	 * @param array $arrGrant
+	 * @return boolean
+	 */
 	function checkGrant($arrGrant)
 	{
 		if(!$arrGrant)

Spacing +35 added lines, -35 removed lines

@@ -27,32 +27,32 @@ 
 		$oModuleModel = getModel('module');
 		$communication_config = $oModuleModel->getModuleConfig('communication');
 
-		if(!is_object($communication_config))
+		if (!is_object($communication_config))
 		{
 			$communication_config = new stdClass();
 		}
 
-		if(!$communication_config->skin)
+		if (!$communication_config->skin)
 		{
 			$communication_config->skin = 'default';
 		}
 
-		if(!$communication_config->colorset)
+		if (!$communication_config->colorset)
 		{
 			$communication_config->colorset = 'white';
 		}
 
-		if(!$communication_config->editor_skin)
+		if (!$communication_config->editor_skin)
 		{
 			$communication_config->editor_skin = 'ckeditor';
 		}
 
-		if(!$communication_config->mskin)
+		if (!$communication_config->mskin)
 		{
 			$communication_config->mskin = 'default';
 		}
 		
-		if(!$communication_config->grant_write)
+		if (!$communication_config->grant_write)
 		{
 			$communication_config->grant_write = array('default_grant'=>'member');
 		}
@@ -69,9 +69,9 @@ 
 	function getGrantArray($default, $group)
 	{
 		$grant = array();
-		if($default!="")
+		if ($default != "")
 		{
-			switch($default)
+			switch ($default)
 			{
 				case "-2":
 					$grant = array("default_grant"=>"site");
@@ -84,13 +84,13 @@ 
 					break;
 			}
 		} 
-		else if(is_array($group)) 
+		else if (is_array($group)) 
 		{
 			$oMemberModel = getModel('member');
 			$group_list = $oMemberModel->getGroups($this->site_srl);
 
 			$group_grant = array();
-			foreach($group as $group_srl)
+			foreach ($group as $group_srl)
 			{
 				$group_grant[$group_srl] = $group_list[$group_srl]->title;
 			}
@@ -107,37 +107,37 @@ 
 	  */
 	function checkGrant($arrGrant)
 	{
-		if(!$arrGrant)
+		if (!$arrGrant)
 			return false;
 
 		$logged_info = Context::get('logged_info');
-		if(!$logged_info)
+		if (!$logged_info)
 			return false;
 
-		if($logged_info->is_admin == "Y")
+		if ($logged_info->is_admin == "Y")
 			return true;
 
-		if($arrGrant['default_grant'])
+		if ($arrGrant['default_grant'])
 		{
-			if($arrGrant['default_grant'] == "member" && $logged_info)
+			if ($arrGrant['default_grant'] == "member" && $logged_info)
 				return true;
 
-			if($arrGrant['default_grant'] == "site" && $this->site_srl == $logged_info->site_srl)
+			if ($arrGrant['default_grant'] == "site" && $this->site_srl == $logged_info->site_srl)
 				return true;
 
-			if($arrGrant['default_grant'] == "manager" && $logged_info->is_admin == "Y")
+			if ($arrGrant['default_grant'] == "manager" && $logged_info->is_admin == "Y")
 				return true;
 		}
 
-		if($arrGrant['group_grant'])
+		if ($arrGrant['group_grant'])
 		{
 			$group_grant = $arrGrant['group_grant'];
-			if(!is_array($group_grant))
+			if (!is_array($group_grant))
 				return false;
 
-			foreach($logged_info->group_list as $group_srl=>$title)
+			foreach ($logged_info->group_list as $group_srl=>$title)
 			{
-				if(isset($group_grant[$group_srl])&&$group_grant[$group_srl]==$title)
+				if (isset($group_grant[$group_srl]) && $group_grant[$group_srl] == $title)
 					return true;
 			}
 
@@ -162,7 +162,7 @@ 
 		$output = executeQuery('communication.getMessage', $args, $columnList);
 
 		$message = $output->data;
-		if(!$message)
+		if (!$message)
 		{
 			return;
 		}
@@ -170,7 +170,7 @@ 
 		// get recipient's information if it is a sent message
 		$oMemberModel = getModel('member');
 
-		if($message->sender_srl == $logged_info->member_srl && $message->message_type == 'S')
+		if ($message->sender_srl == $logged_info->member_srl && $message->message_type == 'S')
 		{
 			$member_info = $oMemberModel->getMemberInfoByMemberSrl($message->receiver_srl);
 		}
@@ -180,22 +180,22 @@ 
 			$member_info = $oMemberModel->getMemberInfoByMemberSrl($message->sender_srl);
 		}
 
-		if($member_info)
+		if ($member_info)
 		{
-			foreach($member_info as $key => $val)
+			foreach ($member_info as $key => $val)
 			{
-				if($key === 'title') continue;
-				if($key === 'content') continue;
-				if($key === 'sender_srl') continue;
-				if($key === 'password') continue;
-				if($key === 'regdate') continue;
+				if ($key === 'title') continue;
+				if ($key === 'content') continue;
+				if ($key === 'sender_srl') continue;
+				if ($key === 'password') continue;
+				if ($key === 'regdate') continue;
 
 				$message->{$key} = $val;
 			}
 		}
 
 		// change the status if is a received and not yet read message
-		if($message->message_type == 'R' && $message->readed != 'Y')
+		if ($message->message_type == 'R' && $message->readed != 'Y')
 		{
 			$oCommunicationController = getController('communication');
 			$oCommunicationController->setMessageReaded($message_srl);
@@ -218,7 +218,7 @@ 
 		$args->readed = 'N';
 
 		$output = executeQuery('communication.getNewMessage', $args, $columnList);
-		if(!count($output->data))
+		if (!count($output->data))
 		{
 			return;
 		}
@@ -242,7 +242,7 @@ 
 		$logged_info = Context::get('logged_info');
 		$args = new stdClass();
 
-		switch($message_type)
+		switch ($message_type)
 		{
 			case 'R' :
 				$args->member_srl = $logged_info->member_srl;
@@ -347,7 +347,7 @@ 
 		$output = executeQueryArray('communication.getFriendGroups', $args);
 
 		$group_list = $output->data;
-		if(!$group_list)
+		if (!$group_list)
 		{
 			return;
 		}
@@ -370,7 +370,7 @@ 
 
 		$output = executeQuery('communication.isAddedFriend', $args);
 
-		if($output->data->count)
+		if ($output->data->count)
 		{
 			return TRUE;
 		}

Braces +40 added lines, -23 removed lines

@@ -83,8 +83,7 @@ 
 					$grant = array("default_grant"=>"member");
 					break;
 			}
-		} 
-		else if(is_array($group)) 
+		} else if(is_array($group)) 
 		{
 			$oMemberModel = getModel('member');
 			$group_list = $oMemberModel->getGroups($this->site_srl);
@@ -107,38 +106,46 @@ 
 	  */
 	function checkGrant($arrGrant)
 	{
-		if(!$arrGrant)
-			return false;
+		if(!$arrGrant) {
+					return false;
+		}
 
 		$logged_info = Context::get('logged_info');
-		if(!$logged_info)
-			return false;
+		if(!$logged_info) {
+					return false;
+		}
 
-		if($logged_info->is_admin == "Y")
-			return true;
+		if($logged_info->is_admin == "Y") {
+					return true;
+		}
 
 		if($arrGrant['default_grant'])
 		{
-			if($arrGrant['default_grant'] == "member" && $logged_info)
-				return true;
+			if($arrGrant['default_grant'] == "member" && $logged_info) {
+							return true;
+			}
 
-			if($arrGrant['default_grant'] == "site" && $this->site_srl == $logged_info->site_srl)
-				return true;
+			if($arrGrant['default_grant'] == "site" && $this->site_srl == $logged_info->site_srl) {
+							return true;
+			}
 
-			if($arrGrant['default_grant'] == "manager" && $logged_info->is_admin == "Y")
-				return true;
+			if($arrGrant['default_grant'] == "manager" && $logged_info->is_admin == "Y") {
+							return true;
+			}
 		}
 
 		if($arrGrant['group_grant'])
 		{
 			$group_grant = $arrGrant['group_grant'];
-			if(!is_array($group_grant))
-				return false;
+			if(!is_array($group_grant)) {
+							return false;
+			}
 
 			foreach($logged_info->group_list as $group_srl=>$title)
 			{
-				if(isset($group_grant[$group_srl])&&$group_grant[$group_srl]==$title)
-					return true;
+				if(isset($group_grant[$group_srl])&&$group_grant[$group_srl]==$title) {
+									return true;
+				}
 			}
 
 		}
@@ -184,11 +191,21 @@ 
 		{
 			foreach($member_info as $key => $val)
 			{
-				if($key === 'title') continue;
-				if($key === 'content') continue;
-				if($key === 'sender_srl') continue;
-				if($key === 'password') continue;
-				if($key === 'regdate') continue;
+				if($key === 'title') {
+					continue;
+				}
+				if($key === 'content') {
+					continue;
+				}
+				if($key === 'sender_srl') {
+					continue;
+				}
+				if($key === 'password') {
+					continue;
+				}
+				if($key === 'regdate') {
+					continue;
+				}
 
 				$message->{$key} = $val;
 			}

modules/communication/communication.view.php

Doc Comments +6 added lines, -6 removed lines

@@ -46,7 +46,7 @@ 
 
 	/**
 	 * Display message box
-	 * @return void|Object (void : success, Object : fail)
+	 * @return ModuleObject|null (void : success, Object : fail)
 	 */
 	function dispCommunicationMessages()
 	{
@@ -131,7 +131,7 @@ 
 
 	/**
 	 * display a new message
-	 * @return void|Object (void : success, Object : fail)
+	 * @return ModuleObject|null (void : success, Object : fail)
 	 */
 	function dispCommunicationNewMessage()
 	{
@@ -167,7 +167,7 @@ 
 
 	/**
 	 * Display message sending
-	 * @return void|Object (void : success, Object : fail)
+	 * @return ModuleObject|null (void : success, Object : fail)
 	 */
 	function dispCommunicationSendMessage()
 	{
@@ -242,7 +242,7 @@ 
 
 	/**
 	 * display a list of friends
-	 * @return void|Object (void : success, Object : fail)
+	 * @return ModuleObject|null (void : success, Object : fail)
 	 */
 	function dispCommunicationFriend()
 	{
@@ -298,7 +298,7 @@ 
 
 	/**
 	 * display Add a friend
-	 * @return void|Object (void : success, Object : fail)
+	 * @return ModuleObject|null (void : success, Object : fail)
 	 */
 	function dispCommunicationAddFriend()
 	{
@@ -340,7 +340,7 @@ 
 
 	/**
 	 * display add a group of friends
-	 * @return void|Object (void : success, Object : fail)
+	 * @return ModuleObject|null (void : success, Object : fail)
 	 */
 	function dispCommunicationAddFriendGroup()
 	{

Braces +1 added lines, -2 removed lines

@@ -21,8 +21,7 @@
 		if($absolute_url)
 		{
 			return getFullUrl('','vid',$vid, 'mid',$mid, 'act',$format);
-		}
-		else
+		} else
 		{
 			return getUrl('','vid',$vid, 'mid',$mid, 'act',$format);
 		}

Spacing +33 added lines, -33 removed lines

@@ -24,7 +24,7 @@ 
 
 		$config_parse = explode('|@|', $skin);
 
-		if(count($config_parse) > 1)
+		if (count($config_parse) > 1)
 		{
 			$tpl_path = sprintf('./themes/%s/modules/communication/', $config_parse[0]);
 		}
@@ -37,7 +37,7 @@ 
 
 		$oLayoutModel = getModel('layout');
 		$layout_info = $oLayoutModel->getLayout($this->communication_config->layout_srl);
-		if($layout_info)
+		if ($layout_info)
 		{
 			$this->module_info->layout_srl = $this->communication_config->layout_srl;
 			$this->setLayoutPath($layout_info->path);
@@ -51,14 +51,14 @@ 
 	function dispCommunicationMessages()
 	{
 		// Error appears if not logged-in
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return $this->stop('msg_not_logged');
 		}
 
 		$logged_info = Context::get('logged_info');
 
-		if(!array_key_exists('dispCommunicationMessages', $logged_info->menu_list))
+		if (!array_key_exists('dispCommunicationMessages', $logged_info->menu_list))
 		{
 			return $this->stop('msg_invalid_request');
 		}
@@ -67,7 +67,7 @@ 
 		$message_srl = Context::get('message_srl');
 		$message_type = Context::get('message_type');
 
-		if(!in_array($message_type, array('R', 'S', 'T')))
+		if (!in_array($message_type, array('R', 'S', 'T')))
 		{
 			$message_type = 'R';
 			Context::set('message_type', $message_type);
@@ -76,36 +76,36 @@ 
 		$oCommunicationModel = getModel('communication');
 
 		// extract contents if message_srl exists
-		if($message_srl)
+		if ($message_srl)
 		{
 			$columnList = array('message_srl', 'sender_srl', 'receiver_srl', 'message_type', 'title', 'content', 'readed', 'regdate');
 			$message = $oCommunicationModel->getSelectedMessage($message_srl, $columnList);
 
-			switch($message->message_type)
+			switch ($message->message_type)
 			{
 				case 'R':
-					if($message->receiver_srl != $logged_info->member_srl)
+					if ($message->receiver_srl != $logged_info->member_srl)
 					{
 						return $this->stop('msg_invalid_request');
 					}
 					break;
 
 				case 'S':
-					if($message->sender_srl != $logged_info->member_srl)
+					if ($message->sender_srl != $logged_info->member_srl)
 					{
 						return $this->stop('msg_invalid_request');
 					}
 					break;
 
 				case 'T':
-					if($message->receiver_srl != $logged_info->member_srl && $message->sender_srl != $logged_info->member_srl)
+					if ($message->receiver_srl != $logged_info->member_srl && $message->sender_srl != $logged_info->member_srl)
 					{
 						return $this->stop('msg_invalid_request');
 					}
 					break;
 			}
 
-			if($message->message_srl == $message_srl && ($message->receiver_srl == $logged_info->member_srl || $message->sender_srl == $logged_info->member_srl))
+			if ($message->message_srl == $message_srl && ($message->receiver_srl == $logged_info->member_srl || $message->sender_srl == $logged_info->member_srl))
 			{
 				stripEmbedTagForAdmin($message->content, $message->sender_srl);
 				Context::set('message', $message);
@@ -139,7 +139,7 @@ 
 		$this->setLayoutFile('popup_layout');
 
 		// Error appears if not logged-in
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return $this->stop('msg_not_logged');
 		}
@@ -151,14 +151,14 @@ 
 		// get a new message
 		$columnList = array('message_srl', 'member_srl', 'nick_name', 'title', 'content', 'sender_srl');
 		$message = $oCommunicationModel->getNewMessage($columnList);
-		if($message)
+		if ($message)
 		{
 			stripEmbedTagForAdmin($message->content, $message->sender_srl);
 			Context::set('message', $message);
 		}
 
 		// Delete a flag
-		$flag_path = './files/communication_extra_info/new_message_flags/' . getNumberingPath($logged_info->member_srl);
+		$flag_path = './files/communication_extra_info/new_message_flags/'.getNumberingPath($logged_info->member_srl);
 		$flag_file = sprintf('%s%s', $flag_path, $logged_info->member_srl);
 		FileHandler::removeFile($flag_file);
 
@@ -178,7 +178,7 @@ 
 		$oMemberModel = getModel('member');
 
 		// Error appears if not logged-in
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return $this->stop('msg_not_logged');
 		}
@@ -188,32 +188,32 @@ 
 		// get receipient's information
 		// check inalid request
 		$receiver_srl = Context::get('receiver_srl');
-		if(!$receiver_srl)
+		if (!$receiver_srl)
 		{
 			return $this->stop('msg_invalid_request');
 		}
 
 		// check receiver and sender are same
-		if($logged_info->member_srl == $receiver_srl)
+		if ($logged_info->member_srl == $receiver_srl)
 		{
 			return $this->stop('msg_cannot_send_to_yourself');
 		}
 
 		// get message_srl of the original message if it is a reply
 		$message_srl = Context::get('message_srl');
-		if($message_srl)
+		if ($message_srl)
 		{
 			$source_message = $oCommunicationModel->getSelectedMessage($message_srl);
-			if($source_message->message_srl == $message_srl && $source_message->sender_srl == $receiver_srl)
+			if ($source_message->message_srl == $message_srl && $source_message->sender_srl == $receiver_srl)
 			{
-				$source_message->title = "[re] " . $source_message->title;
-				$source_message->content = "\r\n<br />\r\n<br /><div style=\"padding-left:5px; border-left:5px solid #DDDDDD;\">" . trim($source_message->content) . "</div>";
+				$source_message->title = "[re] ".$source_message->title;
+				$source_message->content = "\r\n<br />\r\n<br /><div style=\"padding-left:5px; border-left:5px solid #DDDDDD;\">".trim($source_message->content)."</div>";
 				Context::set('source_message', $source_message);
 			}
 		}
 
 		$receiver_info = $oMemberModel->getMemberInfoByMemberSrl($receiver_srl);
-		if(!$receiver_info)
+		if (!$receiver_info)
 		{
 			return $this->stop('msg_invalid_request');
 		}
@@ -247,7 +247,7 @@ 
 	function dispCommunicationFriend()
 	{
 		// Error appears if not logged-in
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return $this->stop('msg_not_logged');
 		}
@@ -258,7 +258,7 @@ 
 		$tmp_group_list = $oCommunicationModel->getFriendGroups();
 		$group_count = count($tmp_group_list);
 
-		for($i = 0; $i < $group_count; $i++)
+		for ($i = 0; $i < $group_count; $i++)
 		{
 			$friend_group_list[$tmp_group_list[$i]->friend_group_srl] = $tmp_group_list[$i];
 		}
@@ -272,13 +272,13 @@ 
 		$output = $oCommunicationModel->getFriends($friend_group_srl, $columnList);
 		$friend_count = count($output->data);
 
-		if($friend_count)
+		if ($friend_count)
 		{
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
 				$group_srl = $val->friend_group_srl;
 				$group_title = $friend_group_list[$group_srl]->title;
-				if(!$group_title)
+				if (!$group_title)
 				{
 					$group_title = Context::get('default_friend_group');
 				}
@@ -306,7 +306,7 @@ 
 		$this->setLayoutFile("popup_layout");
 
 		// error appears if not logged-in
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return $this->stop('msg_not_logged');
 		}
@@ -314,7 +314,7 @@ 
 		$logged_info = Context::get('logged_info');
 		$target_srl = Context::get('target_srl');
 
-		if(!$target_srl)
+		if (!$target_srl)
 		{
 			return $this->stop('msg_invalid_request');
 		}
@@ -324,7 +324,7 @@ 
 		$oCommunicationModel = getModel('communication');
 		$communication_info = $oMemberModel->getMemberInfoByMemberSrl($target_srl);
 
-		if($communication_info->member_srl != $target_srl)
+		if ($communication_info->member_srl != $target_srl)
 		{
 			return $this->stop('msg_invalid_request');
 		}
@@ -348,7 +348,7 @@ 
 		$this->setLayoutFile("popup_layout");
 
 		// error apprears if not logged-in
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return $this->stop('msg_not_logged');
 		}
@@ -357,11 +357,11 @@ 
 
 		// change to edit mode when getting the group_srl
 		$friend_group_srl = Context::get('friend_group_srl');
-		if($friend_group_srl)
+		if ($friend_group_srl)
 		{
 			$oCommunicationModel = getModel('communication');
 			$friend_group = $oCommunicationModel->getFriendGroupInfo($friend_group_srl);
-			if($friend_group->friend_group_srl == $friend_group_srl)
+			if ($friend_group->friend_group_srl == $friend_group_srl)
 			{
 				Context::set('friend_group', $friend_group);
 			}

modules/document/document.admin.controller.php

Doc Comments +3 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
 
 	/**
 	 * Remove the selected docs from admin page
-	 * @return void
+	 * @return ModuleObject|null
 	 */
 	function procDocumentAdminDeleteChecked()
 	{
@@ -519,6 +519,7 @@ 
 
 	/**
 	 * Delete thumbnails with subdirectory
+	 * @param string $path
 	 * @return void
 	 */
 	function deleteThumbnailFile($path)
@@ -861,7 +862,7 @@ 
 	/**
 	 * Restore document from trash module, called by trash module
 	 * This method is passived
-	 * @param object|array $originObject
+	 * @param string $originObject
 	 * @return object
 	 */
 	function restoreTrash($originObject)

Indentation +4 added lines, -4 removed lines

@@ -721,10 +721,10 @@
 	}
 
 	/**
-	  * @fn procDocumentAdminMoveToTrash
-	  * @brief move a document to trash.
-	  * @see documentModel::getDocumentMenu
-	  */
+	 * @fn procDocumentAdminMoveToTrash
+	 * @brief move a document to trash.
+	 * @see documentModel::getDocumentMenu
+	 */
 	function procDocumentAdminMoveToTrash()
 	{
 		$document_srl = Context::get('document_srl');

Braces +136 added lines, -59 removed lines

@@ -27,16 +27,22 @@ 
 	{
 		// error appears if no doc is selected
 		$cart = Context::get('cart');
-		if(!$cart) return $this->stop('msg_cart_is_null');
+		if(!$cart) {
+			return $this->stop('msg_cart_is_null');
+		}
 		$document_srl_list= explode('|@|', $cart);
 		$document_count = count($document_srl_list);
-		if(!$document_count) return $this->stop('msg_cart_is_null');
+		if(!$document_count) {
+			return $this->stop('msg_cart_is_null');
+		}
 		// Delete a doc
 		$oDocumentController = getController('document');
 		for($i=0;$i<$document_count;$i++)
 		{
 			$document_srl = trim($document_srl_list[$i]);
-			if(!$document_srl) continue;
+			if(!$document_srl) {
+				continue;
+			}
 
 			$oDocumentController->deleteDocument($document_srl, true);
 		}
@@ -53,7 +59,9 @@ 
 	 */
 	function moveDocumentModule($document_srl_list, $module_srl, $category_srl)
 	{
-		if(!count($document_srl_list)) return;
+		if(!count($document_srl_list)) {
+			return;
+		}
 
 		$oDocumentModel = getModel('document');
 		$oDocumentController = getController('document');
@@ -77,7 +85,9 @@ 
 		{
 			$document_srl = $document_srl_list[$i];
 			$oDocument = $oDocumentModel->getDocument($document_srl);
-			if(!$oDocument->isExists()) continue;
+			if(!$oDocument->isExists()) {
+				continue;
+			}
 
 			$source_category_srl = $oDocument->get('category_srl');
 
@@ -113,8 +123,7 @@ 
 								$target_filename = substr($inserted_file->get('uploaded_filename'),2);
 								$obj->content = str_replace($source_filename, $target_filename, $obj->content);
 								// For binary files
-							}
-							else
+							} else
 							{
 								$obj->content = str_replace('file_srl='.$val->file_srl, 'file_srl='.$inserted_file->get('file_srl'), $obj->content);
 								$obj->content = str_replace('sid='.$val->sid, 'sid='.$inserted_file->get('sid'), $obj->content);
@@ -151,8 +160,12 @@ 
 			// Set 0 if a new category doesn't exist after catergory change
 			if($source_category_srl != $category_srl)
 			{
-				if($source_category_srl) $oDocumentController->updateCategoryCount($oDocument->get('module_srl'), $source_category_srl);
-				if($category_srl) $oDocumentController->updateCategoryCount($module_srl, $category_srl);
+				if($source_category_srl) {
+					$oDocumentController->updateCategoryCount($oDocument->get('module_srl'), $source_category_srl);
+				}
+				if($category_srl) {
+					$oDocumentController->updateCategoryCount($module_srl, $category_srl);
+				}
 			}
 		}
 
@@ -223,7 +236,9 @@ 
 	 */
 	function copyDocumentModule($document_srl_list, $module_srl, $category_srl)
 	{
-		if(count($document_srl_list) < 1) return;
+		if(count($document_srl_list) < 1) {
+			return;
+		}
 
 		$oDocumentModel = getModel('document');
 		$oDocumentController = getController('document');
@@ -264,7 +279,9 @@ 
 		{
 			$document_srl = $document_srl_list[$i];
 			$oDocument = $oDocumentModel->getDocument($document_srl);
-			if(!$oDocument->isExists()) continue;
+			if(!$oDocument->isExists()) {
+				continue;
+			}
 
 			$obj = $oDocument->getObjectVars();
 
@@ -275,7 +292,9 @@ 
 				{
 					foreach($extraVars as $extraItem)
 					{
-						if($extraItem->var_idx >= 0) $obj->{'extra_vars'.$extraItem->var_idx} = $extraItem->value;
+						if($extraItem->var_idx >= 0) {
+							$obj->{'extra_vars'.$extraItem->var_idx} = $extraItem->value;
+						}
 					}
 				}
 			}
@@ -304,8 +323,7 @@ 
 						$target_filename = substr($inserted_file->get('uploaded_filename'),2);
 						$obj->content = str_replace($source_filename, $target_filename, $obj->content);
 						// If binary file
-					}
-					else
+					} else
 					{
 						$obj->content = str_replace('file_srl='.$val->file_srl, 'file_srl='.$inserted_file->get('file_srl'), $obj->content);
 						$obj->content = str_replace('sid='.$val->sid, 'sid='.$inserted_file->get('sid'), $obj->content);
@@ -372,8 +390,7 @@ 
 									$target_filename = substr($inserted_file->get('uploaded_filename'),2);
 									$comment_obj->content = str_replace($source_filename, $target_filename, $comment_obj->content);
 									// If binary file
-								}
-								else
+								} else
 								{
 									$comment_obj->content = str_replace('file_srl='.$val->file_srl, 'file_srl='.$inserted_file->get('file_srl'), $comment_obj->content);
 									$comment_obj->content = str_replace('sid='.$val->sid, 'sid='.$inserted_file->get('sid'), $comment_obj->content);
@@ -385,10 +402,14 @@ 
 						$comment_obj->document_srl = $obj->document_srl;
 						$comment_obj->comment_srl = $comment_srl;
 
-						if($comment_obj->parent_srl) $comment_obj->parent_srl = $p_comment_srl[$comment_obj->parent_srl];
+						if($comment_obj->parent_srl) {
+							$comment_obj->parent_srl = $p_comment_srl[$comment_obj->parent_srl];
+						}
 
 						$output = $oCommentController->insertComment($comment_obj, true);
-						if($output->toBool()) $success_count ++;
+						if($output->toBool()) {
+							$success_count ++;
+						}
 					}
 					$oDocumentController->updateCommentCount($obj->document_srl, $success_count, $comment_obj->nick_name, true);
 				}
@@ -408,7 +429,9 @@ 
 						$trackback_obj->module_srl = $obj->module_srl;
 						$trackback_obj->document_srl = $obj->document_srl;
 						$output = executeQuery('trackback.insertTrackback', $trackback_obj);
-						if($output->toBool()) $success_count++;
+						if($output->toBool()) {
+							$success_count++;
+						}
 					}
 					// Update the number of trackbacks
 					$oDocumentController->updateTrackbackCount($obj->document_srl, $success_count);
@@ -499,7 +522,9 @@ 
 		{
 			$args->document_srl = $document_srl;
 			$output = executeQuery('document.deleteDeclaredDocuments', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 		}
 	}
 
@@ -529,7 +554,9 @@ 
 				if (is_dir($path."/".$entry)) {
 					$this->deleteThumbnailFile($path."/".$entry);
 				} else {
-					if(!preg_match('/^thumbnail_([^\.]*)\.jpg$/i',$entry)) continue;
+					if(!preg_match('/^thumbnail_([^\.]*)\.jpg$/i',$entry)) {
+						continue;
+					}
 					FileHandler::removeFile($path.'/'.$entry);
 				}
 			}
@@ -554,7 +581,9 @@ 
 		$eid = Context::get('eid');
 		$obj = new stdClass();
 
-		if(!$module_srl || !$name || !$eid) return new Object(-1,'msg_invalid_request');
+		if(!$module_srl || !$name || !$eid) {
+			return new Object(-1,'msg_invalid_request');
+		}
 		// set the max value if idx is not specified
 		if(!$var_idx)
 		{
@@ -576,7 +605,9 @@ 
 		// insert or update
 		$oDocumentController = getController('document');
 		$output = $oDocumentController->insertDocumentExtraKey($module_srl, $var_idx, $name, $type, $is_required, $search, $default, $desc, $eid);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->setMessage('success_registed');
 
@@ -592,11 +623,15 @@ 
 	{
 		$module_srl = Context::get('module_srl');
 		$var_idx = Context::get('var_idx');
-		if(!$module_srl || !$var_idx) return new Object(-1,'msg_invalid_request');
+		if(!$module_srl || !$var_idx) {
+			return new Object(-1,'msg_invalid_request');
+		}
 
 		$oDocumentController = getController('document');
 		$output = $oDocumentController->deleteDocumentExtraKeys($module_srl, $var_idx);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->setMessage('success_deleted');
 	}
@@ -611,26 +646,41 @@ 
 		$module_srl = Context::get('module_srl');
 		$var_idx = Context::get('var_idx');
 
-		if(!$type || !$module_srl || !$var_idx) return new Object(-1,'msg_invalid_request');
+		if(!$type || !$module_srl || !$var_idx) {
+			return new Object(-1,'msg_invalid_request');
+		}
 
 		$oModuleModel = getModel('module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl);
-		if(!$module_info->module_srl) return new Object(-1,'msg_invalid_request');
+		if(!$module_info->module_srl) {
+			return new Object(-1,'msg_invalid_request');
+		}
 
 		$oDocumentModel = getModel('document');
 		$extra_keys = $oDocumentModel->getExtraKeys($module_srl);
-		if(!$extra_keys[$var_idx]) return new Object(-1,'msg_invalid_request');
+		if(!$extra_keys[$var_idx]) {
+			return new Object(-1,'msg_invalid_request');
+		}
 
-		if($type == 'up') $new_idx = $var_idx-1;
-		else $new_idx = $var_idx+1;
-		if($new_idx<1) return new Object(-1,'msg_invalid_request');
+		if($type == 'up') {
+			$new_idx = $var_idx-1;
+		} else {
+			$new_idx = $var_idx+1;
+		}
+		if($new_idx<1) {
+			return new Object(-1,'msg_invalid_request');
+		}
 
 		$args = new stdClass();
 		$args->module_srl = $module_srl;
 		$args->var_idx = $new_idx;
 		$output = executeQuery('document.getDocumentExtraKeys', $args);
-		if (!$output->toBool()) return $output;
-		if (!$output->data) return new Object(-1, 'msg_invalid_request');
+		if (!$output->toBool()) {
+			return $output;
+		}
+		if (!$output->data) {
+			return new Object(-1, 'msg_invalid_request');
+		}
 		unset($args);
 
 		// update immediately if there is no idx to change
@@ -641,35 +691,50 @@ 
 			$args->var_idx = $var_idx;
 			$args->new_idx = $new_idx;
 			$output = executeQuery('document.updateDocumentExtraKeyIdx', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 			$output = executeQuery('document.updateDocumentExtraVarIdx', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 			// replace if exists
-		}
-		else
+		} else
 		{
 			$args = new stdClass();
 			$args->module_srl = $module_srl;
 			$args->var_idx = $new_idx;
 			$args->new_idx = -10000;
 			$output = executeQuery('document.updateDocumentExtraKeyIdx', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 			$output = executeQuery('document.updateDocumentExtraVarIdx', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 
 			$args->var_idx = $var_idx;
 			$args->new_idx = $new_idx;
 			$output = executeQuery('document.updateDocumentExtraKeyIdx', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 			$output = executeQuery('document.updateDocumentExtraVarIdx', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 
 			$args->var_idx = -10000;
 			$args->new_idx = $var_idx;
 			$output = executeQuery('document.updateDocumentExtraKeyIdx', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 			$output = executeQuery('document.updateDocumentExtraVarIdx', $args);
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 		}
 
 		$oCacheHandler = CacheHandler::getInstance('object', NULL, TRUE);
@@ -693,8 +758,7 @@ 
 		{
 			$args->alias_srl = getNextSequence();
 			$query = "document.insertAlias";
-		}
-		else
+		} else
 		{
 			$args->alias_srl = $alias_srl;
 			$query = "document.updateAlias";
@@ -732,7 +796,9 @@ 
 		$oDocumentModel = getModel('document');
 		$oDocumentController = getController('document');
 		$oDocument = $oDocumentModel->getDocument($document_srl, false, false);
-		if(!$oDocument->isGranted()) return $this->stop('msg_not_permitted');
+		if(!$oDocument->isGranted()) {
+			return $this->stop('msg_not_permitted');
+		}
 	
 		$oModuleModel = getModel('module');
 		$module_info = $oModuleModel->getModuleInfoByDocumentSrl($document_srl);
@@ -754,8 +820,9 @@ 
 				parse_str($arrUrl['query'], $arrQuery);
 
 				// set query
-				if(isset($arrQuery['document_srl']))
-					unset($arrQuery['document_srl']);
+				if(isset($arrQuery['document_srl'])) {
+									unset($arrQuery['document_srl']);
+				}
 
 				$searchArgs = new stdClass;
 				foreach($arrQuery as $key=>$val)
@@ -763,13 +830,15 @@ 
 					$searchArgs->{$key} = $val;
 				}
 
-				if(!isset($searchArgs->sort_index))
-					$searchArgs->sort_index = $module_info->order_target;
+				if(!isset($searchArgs->sort_index)) {
+									$searchArgs->sort_index = $module_info->order_target;
+				}
 
 				foreach($module_info as $key=>$val)
 				{
-					if(!isset($searchArgs->{$key}))
-						$searchArgs->{$key} = $val;
+					if(!isset($searchArgs->{$key})) {
+											$searchArgs->{$key} = $val;
+					}
 				}
 
 				$oDocumentModel = getModel('document');
@@ -781,12 +850,14 @@ 
 				}
 
 
-				if($cur_page>1 && count($output->data) == 0)
-					$arrQuery['page'] = $cur_page - 1;
+				if($cur_page>1 && count($output->data) == 0) {
+									$arrQuery['page'] = $cur_page - 1;
+				}
 
 				$query = "?";
-				foreach($arrQuery as $key=>$val)
-					$query .= sprintf("%s=%s&", $key, $val);
+				foreach($arrQuery as $key=>$val) {
+									$query .= sprintf("%s=%s&", $key, $val);
+				}
 				$query = substr($query, 0, -1);
 			}
 			$returnUrl = $arrUrl['path'] . $query;
@@ -866,7 +937,9 @@ 
 	 */
 	function restoreTrash($originObject)
 	{
-		if(is_array($originObject)) $originObject = (object)$originObject;
+		if(is_array($originObject)) {
+			$originObject = (object)$originObject;
+		}
 
 		$oDocumentController = getController('document');
 		$oDocumentModel = getModel('document');
@@ -876,7 +949,9 @@ 
 
 		//DB restore
 		$output = $oDocumentController->insertDocument($originObject, false, true, false);
-		if(!$output->toBool()) return new Object(-1, $output->getMessage());
+		if(!$output->toBool()) {
+			return new Object(-1, $output->getMessage());
+		}
 
 		//FILE restore
 		$oDocument = $oDocumentModel->getDocument($originObject->document_srl);
@@ -914,7 +989,9 @@ 
 	function emptyTrash($originObject)
 	{
 		$originObject = unserialize($originObject);
-		if(is_array($originObject)) $originObject = (object) $originObject;
+		if(is_array($originObject)) {
+			$originObject = (object) $originObject;
+		}
 
 		$oDocument = new documentItem();
 		$oDocument->setAttribute($originObject);

Spacing +125 added lines, -125 removed lines

@@ -27,21 +27,21 @@ 
 	{
 		// error appears if no doc is selected
 		$cart = Context::get('cart');
-		if(!$cart) return $this->stop('msg_cart_is_null');
-		$document_srl_list= explode('|@|', $cart);
+		if (!$cart) return $this->stop('msg_cart_is_null');
+		$document_srl_list = explode('|@|', $cart);
 		$document_count = count($document_srl_list);
-		if(!$document_count) return $this->stop('msg_cart_is_null');
+		if (!$document_count) return $this->stop('msg_cart_is_null');
 		// Delete a doc
 		$oDocumentController = getController('document');
-		for($i=0;$i<$document_count;$i++)
+		for ($i = 0; $i < $document_count; $i++)
 		{
 			$document_srl = trim($document_srl_list[$i]);
-			if(!$document_srl) continue;
+			if (!$document_srl) continue;
 
 			$oDocumentController->deleteDocument($document_srl, true);
 		}
 
-		$this->setMessage(sprintf(Context::getLang('msg_checked_document_is_deleted'), $document_count) );
+		$this->setMessage(sprintf(Context::getLang('msg_checked_document_is_deleted'), $document_count));
 	}
 
 	/**
@@ -53,7 +53,7 @@ 
 	 */
 	function moveDocumentModule($document_srl_list, $module_srl, $category_srl)
 	{
-		if(!count($document_srl_list)) return;
+		if (!count($document_srl_list)) return;
 
 		$oDocumentModel = getModel('document');
 		$oDocumentController = getController('document');
@@ -62,22 +62,22 @@ 
 		$oDB->begin();
 
 		$triggerObj = new stdClass();
-		$triggerObj->document_srls = implode(',',$document_srl_list);
+		$triggerObj->document_srls = implode(',', $document_srl_list);
 		$triggerObj->module_srl = $module_srl;
 		$triggerObj->category_srl = $category_srl;
 		// Call a trigger (before)
 		$output = ModuleHandler::triggerCall('document.moveDocumentModule', 'before', $triggerObj);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 
-		for($i=count($document_srl_list)-1;$i>=0;$i--)
+		for ($i = count($document_srl_list) - 1; $i >= 0; $i--)
 		{
 			$document_srl = $document_srl_list[$i];
 			$oDocument = $oDocumentModel->getDocument($document_srl);
-			if(!$oDocument->isExists()) continue;
+			if (!$oDocument->isExists()) continue;
 
 			$source_category_srl = $oDocument->get('category_srl');
 
@@ -90,27 +90,27 @@ 
 			$obj->content = $output_ori->data->content;
 
 			// Move the attached file if the target module is different
-			if($module_srl != $obj->module_srl && $oDocument->hasUploadedFiles())
+			if ($module_srl != $obj->module_srl && $oDocument->hasUploadedFiles())
 			{
 				$oFileController = getController('file');
 
 				$files = $oDocument->getUploadedFiles();
 				$delete_file_srls = array();
-				if(is_array($files))
+				if (is_array($files))
 				{
-					foreach($files as $val)
+					foreach ($files as $val)
 					{
 						$file_info = array();
 						$file_info['tmp_name'] = $val->uploaded_filename;
 						$file_info['name'] = $val->source_filename;
 						$inserted_file = $oFileController->insertFile($file_info, $module_srl, $obj->document_srl, $val->download_count, true);
-						if($inserted_file && $inserted_file->toBool())
+						if ($inserted_file && $inserted_file->toBool())
 						{
 							// for image/video files
-							if($val->direct_download == 'Y')
+							if ($val->direct_download == 'Y')
 							{
-								$source_filename = substr($val->uploaded_filename,2);
-								$target_filename = substr($inserted_file->get('uploaded_filename'),2);
+								$source_filename = substr($val->uploaded_filename, 2);
+								$target_filename = substr($inserted_file->get('uploaded_filename'), 2);
 								$obj->content = str_replace($source_filename, $target_filename, $obj->content);
 								// For binary files
 							}
@@ -129,7 +129,7 @@ 
 				$oFileController->setFilesValid($obj->document_srl);
 			}
 
-			if($module_srl != $obj->module_srl)
+			if ($module_srl != $obj->module_srl)
 			{
 				$oDocumentController->deleteDocumentAliasByDocument($obj->document_srl);
 			}
@@ -137,48 +137,48 @@ 
 			$obj->module_srl = $module_srl;
 			$obj->category_srl = $category_srl;
 			$output = executeQuery('document.updateDocumentModule', $obj);
-			if(!$output->toBool()) {
+			if (!$output->toBool()) {
 				$oDB->rollback();
 				return $output;
 			}
 
 			//Move a module of the extra vars
 			$output = executeQuery('document.moveDocumentExtraVars', $obj);
-			if(!$output->toBool()) {
+			if (!$output->toBool()) {
 				$oDB->rollback();
 				return $output;
 			}
 			// Set 0 if a new category doesn't exist after catergory change
-			if($source_category_srl != $category_srl)
+			if ($source_category_srl != $category_srl)
 			{
-				if($source_category_srl) $oDocumentController->updateCategoryCount($oDocument->get('module_srl'), $source_category_srl);
-				if($category_srl) $oDocumentController->updateCategoryCount($module_srl, $category_srl);
+				if ($source_category_srl) $oDocumentController->updateCategoryCount($oDocument->get('module_srl'), $source_category_srl);
+				if ($category_srl) $oDocumentController->updateCategoryCount($module_srl, $category_srl);
 			}
 		}
 
 		$args = new stdClass();
-		$args->document_srls = implode(',',$document_srl_list);
+		$args->document_srls = implode(',', $document_srl_list);
 		$args->module_srl = $module_srl;
 		// move the comment
 		$output = executeQuery('comment.updateCommentModule', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 
 		$output = executeQuery('comment.updateCommentListModule', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 		
 		// move the trackback
-		if(getClass('trackback'))
+		if (getClass('trackback'))
 		{
 			$output = executeQuery('trackback.updateTrackbackModule', $args);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				$oDB->rollback();
 				return $output;
@@ -187,14 +187,14 @@ 
 
 		// Tags
 		$output = executeQuery('tag.updateTagModule', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
 		}
 		// Call a trigger (before)
 		$output = ModuleHandler::triggerCall('document.moveDocumentModule', 'after', $triggerObj);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
@@ -203,11 +203,11 @@ 
 		$oDB->commit();
 		//remove from cache
 		$oCacheHandler = CacheHandler::getInstance('object');
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
-			foreach($document_srl_list as $document_srl)
+			foreach ($document_srl_list as $document_srl)
 			{
-				$cache_key_item = 'document_item:'. getNumberingPath($document_srl) . $document_srl;
+				$cache_key_item = 'document_item:'.getNumberingPath($document_srl).$document_srl;
 				$oCacheHandler->delete($cache_key_item);
 			}
 		}
@@ -223,7 +223,7 @@ 
 	 */
 	function copyDocumentModule($document_srl_list, $module_srl, $category_srl)
 	{
-		if(count($document_srl_list) < 1) return;
+		if (count($document_srl_list) < 1) return;
 
 		$oDocumentModel = getModel('document');
 		$oDocumentController = getController('document');
@@ -234,12 +234,12 @@ 
 		$oDB->begin();
 
 		$triggerObj = new stdClass();
-		$triggerObj->document_srls = implode(',',$document_srl_list);
+		$triggerObj->document_srls = implode(',', $document_srl_list);
 		$triggerObj->module_srl = $module_srl;
 		$triggerObj->category_srl = $category_srl;
 		// Call a trigger (before)
 		$output = ModuleHandler::triggerCall('document.copyDocumentModule', 'before', $triggerObj);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
@@ -247,11 +247,11 @@ 
 
 		$extraVarsList = $oDocumentModel->getDocumentExtraVarsFromDB($document_srl_list);
 		$extraVarsListByDocumentSrl = array();
-		if(is_array($extraVarsList->data))
+		if (is_array($extraVarsList->data))
 		{
-			foreach($extraVarsList->data as $value)
+			foreach ($extraVarsList->data as $value)
 			{
-				if(!isset($extraVarsListByDocumentSrl[$value->document_srl]))
+				if (!isset($extraVarsListByDocumentSrl[$value->document_srl]))
 				{
 					$extraVarsListByDocumentSrl[$value->document_srl] = array();
 				}
@@ -260,22 +260,22 @@ 
 			}
 		}
 
-		for($i=count($document_srl_list)-1;$i>=0;$i--)
+		for ($i = count($document_srl_list) - 1; $i >= 0; $i--)
 		{
 			$document_srl = $document_srl_list[$i];
 			$oDocument = $oDocumentModel->getDocument($document_srl);
-			if(!$oDocument->isExists()) continue;
+			if (!$oDocument->isExists()) continue;
 
 			$obj = $oDocument->getObjectVars();
 
 			$extraVars = $extraVarsListByDocumentSrl[$document_srl];
-			if($module_srl == $obj->module_srl)
+			if ($module_srl == $obj->module_srl)
 			{
-				if(is_array($extraVars))
+				if (is_array($extraVars))
 				{
-					foreach($extraVars as $extraItem)
+					foreach ($extraVars as $extraItem)
 					{
-						if($extraItem->var_idx >= 0) $obj->{'extra_vars'.$extraItem->var_idx} = $extraItem->value;
+						if ($extraItem->var_idx >= 0) $obj->{'extra_vars'.$extraItem->var_idx} = $extraItem->value;
 					}
 				}
 			}
@@ -287,10 +287,10 @@ 
 			$obj->trackback_count = 0;
 
 			// Pre-register the attachment
-			if($oDocument->hasUploadedFiles())
+			if ($oDocument->hasUploadedFiles())
 			{
 				$files = $oDocument->getUploadedFiles();
-				foreach($files as $val)
+				foreach ($files as $val)
 				{
 					$file_info = array();
 					$file_info['tmp_name'] = $val->uploaded_filename;
@@ -298,10 +298,10 @@ 
 					$oFileController = getController('file');
 					$inserted_file = $oFileController->insertFile($file_info, $module_srl, $obj->document_srl, 0, true);
 					// if image/video files
-					if($val->direct_download == 'Y')
+					if ($val->direct_download == 'Y')
 					{
-						$source_filename = substr($val->uploaded_filename,2);
-						$target_filename = substr($inserted_file->get('uploaded_filename'),2);
+						$source_filename = substr($val->uploaded_filename, 2);
+						$target_filename = substr($inserted_file->get('uploaded_filename'), 2);
 						$obj->content = str_replace($source_filename, $target_filename, $obj->content);
 						// If binary file
 					}
@@ -315,23 +315,23 @@ 
 
 			// Write a post
 			$output = $oDocumentController->insertDocument($obj, true, true);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				$oDB->rollback();
 				return $output;
 			}
 
 			// copy multi language contents
-			if(is_array($extraVars))
+			if (is_array($extraVars))
 			{
-				foreach($extraVars as $value)
+				foreach ($extraVars as $value)
 				{
-					if($value->idx >= 0 && $value->lang_code == Context::getLangType())
+					if ($value->idx >= 0 && $value->lang_code == Context::getLangType())
 					{
 						continue;
 					}
 
-					if( $value->var_idx < 0 || ($module_srl == $value->module_srl && $value->var_idx >= 0) )
+					if ($value->var_idx < 0 || ($module_srl == $value->module_srl && $value->var_idx >= 0))
 					{
 						$oDocumentController->insertDocumentExtraVar($value->module_srl, $obj->document_srl, $value->var_idx, $value->value, $value->eid, $value->lang_code);
 					}
@@ -339,26 +339,26 @@ 
 			}
 
 			// Move the comments
-			if($oDocument->getCommentCount())
+			if ($oDocument->getCommentCount())
 			{
 				$oCommentModel = getModel('comment');
 				$comment_output = $oCommentModel->getCommentList($document_srl, 0, true, 99999999);
 				$comments = $comment_output->data;
-				if(count($comments) > 0)
+				if (count($comments) > 0)
 				{
 					$oCommentController = getController('comment');
 					$success_count = 0;
 					$p_comment_srl = array();
-					foreach($comments as $comment_obj)
+					foreach ($comments as $comment_obj)
 					{
 						$comment_srl = getNextSequence();
 						$p_comment_srl[$comment_obj->comment_srl] = $comment_srl;
 
 						// Pre-register the attachment
-						if($comment_obj->uploaded_count)
+						if ($comment_obj->uploaded_count)
 						{
 							$files = $oFileModel->getFiles($comment_obj->comment_srl, true);
-							foreach($files as $val)
+							foreach ($files as $val)
 							{
 								$file_info = array();
 								$file_info['tmp_name'] = $val->uploaded_filename;
@@ -366,10 +366,10 @@ 
 								$oFileController = getController('file');
 								$inserted_file = $oFileController->insertFile($file_info, $module_srl, $comment_srl, 0, true);
 								// if image/video files
-								if($val->direct_download == 'Y')
+								if ($val->direct_download == 'Y')
 								{
-									$source_filename = substr($val->uploaded_filename,2);
-									$target_filename = substr($inserted_file->get('uploaded_filename'),2);
+									$source_filename = substr($val->uploaded_filename, 2);
+									$target_filename = substr($inserted_file->get('uploaded_filename'), 2);
 									$comment_obj->content = str_replace($source_filename, $target_filename, $comment_obj->content);
 									// If binary file
 								}
@@ -385,10 +385,10 @@ 
 						$comment_obj->document_srl = $obj->document_srl;
 						$comment_obj->comment_srl = $comment_srl;
 
-						if($comment_obj->parent_srl) $comment_obj->parent_srl = $p_comment_srl[$comment_obj->parent_srl];
+						if ($comment_obj->parent_srl) $comment_obj->parent_srl = $p_comment_srl[$comment_obj->parent_srl];
 
 						$output = $oCommentController->insertComment($comment_obj, true);
-						if($output->toBool()) $success_count ++;
+						if ($output->toBool()) $success_count++;
 					}
 					$oDocumentController->updateCommentCount($obj->document_srl, $success_count, $comment_obj->nick_name, true);
 				}
@@ -396,19 +396,19 @@ 
 
 			// Move the trackbacks
 			$oTrackbackModel = getModel('trackback');
-			if($oTrackbackModel && $oDocument->getTrackbackCount())
+			if ($oTrackbackModel && $oDocument->getTrackbackCount())
 			{
 				$trackbacks = $oTrackbackModel->getTrackbackList($oDocument->document_srl);
-				if(count($trackbacks))
+				if (count($trackbacks))
 				{
 					$success_count = 0;
-					foreach($trackbacks as $trackback_obj)
+					foreach ($trackbacks as $trackback_obj)
 					{
 						$trackback_obj->trackback_srl = getNextSequence();
 						$trackback_obj->module_srl = $obj->module_srl;
 						$trackback_obj->document_srl = $obj->document_srl;
 						$output = executeQuery('trackback.insertTrackback', $trackback_obj);
-						if($output->toBool()) $success_count++;
+						if ($output->toBool()) $success_count++;
 					}
 					// Update the number of trackbacks
 					$oDocumentController->updateTrackbackCount($obj->document_srl, $success_count);
@@ -421,7 +421,7 @@ 
 		// Call a trigger (before)
 		$triggerObj->copied_srls = $copied_srls;
 		$output = ModuleHandler::triggerCall('document.copyDocumentModule', 'after', $triggerObj);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$oDB->rollback();
 			return $output;
@@ -448,7 +448,7 @@ 
 		$document_list = $oDocumentModel->getDocumentList($args);
 		$documents = $document_list->data;
 		$output = executeQuery('document.deleteModuleDocument', $args);
-		if(is_array($documents))
+		if (is_array($documents))
 		{
 			foreach ($documents as $oDocument)
 			{
@@ -457,13 +457,13 @@ 
 		}
 		//remove from cache
 		$oCacheHandler = CacheHandler::getInstance('object');
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
-			if(is_array($document_srl_list))
+			if (is_array($document_srl_list))
 			{
-				foreach($document_srl_list as $document_srl)
+				foreach ($document_srl_list as $document_srl)
 				{
-					$cache_key_item = 'document_item:'. getNumberingPath($document_srl) . $document_srl;
+					$cache_key_item = 'document_item:'.getNumberingPath($document_srl).$document_srl;
 					$oCacheHandler->delete($cache_key_item);
 				}
 			}
@@ -481,7 +481,7 @@ 
 		$config = Context::gets('thumbnail_type');
 		// Insert by creating the module Controller object
 		$oModuleController = getController('module');
-		$output = $oModuleController->insertModuleConfig('document',$config);
+		$output = $oModuleController->insertModuleConfig('document', $config);
 
 		$returnUrl = Context::get('success_return_url') ? Context::get('success_return_url') : getNotEncodedUrl('', 'module', 'admin', 'act', 'dispDocumentAdminConfig');
 		return $this->setRedirectUrl($returnUrl, $output);
@@ -495,11 +495,11 @@ 
 	{
 		$document_srl = trim(Context::get('document_srl'));
 
-		if($document_srl)
+		if ($document_srl)
 		{
 			$args->document_srl = $document_srl;
 			$output = executeQuery('document.deleteDeclaredDocuments', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 		}
 	}
 
@@ -524,12 +524,12 @@ 
 	function deleteThumbnailFile($path)
 	{
 		$directory = dir($path);
-		while($entry = $directory->read()) {
+		while ($entry = $directory->read()) {
 			if ($entry != "." && $entry != "..") {
 				if (is_dir($path."/".$entry)) {
 					$this->deleteThumbnailFile($path."/".$entry);
 				} else {
-					if(!preg_match('/^thumbnail_([^\.]*)\.jpg$/i',$entry)) continue;
+					if (!preg_match('/^thumbnail_([^\.]*)\.jpg$/i', $entry)) continue;
 					FileHandler::removeFile($path.'/'.$entry);
 				}
 			}
@@ -554,13 +554,13 @@ 
 		$eid = Context::get('eid');
 		$obj = new stdClass();
 
-		if(!$module_srl || !$name || !$eid) return new Object(-1,'msg_invalid_request');
+		if (!$module_srl || !$name || !$eid) return new Object(-1, 'msg_invalid_request');
 		// set the max value if idx is not specified
-		if(!$var_idx)
+		if (!$var_idx)
 		{
 			$obj->module_srl = $module_srl;
 			$output = executeQuery('document.getDocumentMaxExtraKeyIdx', $obj);
-			$var_idx = $output->data->var_idx+1;
+			$var_idx = $output->data->var_idx + 1;
 		}
 
 		// Check if the module name already exists
@@ -568,7 +568,7 @@ 
 		$obj->var_idx = $var_idx;
 		$obj->eid = $eid;
 		$output = executeQuery('document.isExistsExtraKey', $obj);
-		if(!$output->toBool() || $output->data->count)
+		if (!$output->toBool() || $output->data->count)
 		{
 			return new Object(-1, 'msg_extra_name_exists');
 		}
@@ -576,7 +576,7 @@ 
 		// insert or update
 		$oDocumentController = getController('document');
 		$output = $oDocumentController->insertDocumentExtraKey($module_srl, $var_idx, $name, $type, $is_required, $search, $default, $desc, $eid);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$this->setMessage('success_registed');
 
@@ -592,11 +592,11 @@ 
 	{
 		$module_srl = Context::get('module_srl');
 		$var_idx = Context::get('var_idx');
-		if(!$module_srl || !$var_idx) return new Object(-1,'msg_invalid_request');
+		if (!$module_srl || !$var_idx) return new Object(-1, 'msg_invalid_request');
 
 		$oDocumentController = getController('document');
 		$output = $oDocumentController->deleteDocumentExtraKeys($module_srl, $var_idx);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$this->setMessage('success_deleted');
 	}
@@ -611,19 +611,19 @@ 
 		$module_srl = Context::get('module_srl');
 		$var_idx = Context::get('var_idx');
 
-		if(!$type || !$module_srl || !$var_idx) return new Object(-1,'msg_invalid_request');
+		if (!$type || !$module_srl || !$var_idx) return new Object(-1, 'msg_invalid_request');
 
 		$oModuleModel = getModel('module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl);
-		if(!$module_info->module_srl) return new Object(-1,'msg_invalid_request');
+		if (!$module_info->module_srl) return new Object(-1, 'msg_invalid_request');
 
 		$oDocumentModel = getModel('document');
 		$extra_keys = $oDocumentModel->getExtraKeys($module_srl);
-		if(!$extra_keys[$var_idx]) return new Object(-1,'msg_invalid_request');
+		if (!$extra_keys[$var_idx]) return new Object(-1, 'msg_invalid_request');
 
-		if($type == 'up') $new_idx = $var_idx-1;
-		else $new_idx = $var_idx+1;
-		if($new_idx<1) return new Object(-1,'msg_invalid_request');
+		if ($type == 'up') $new_idx = $var_idx - 1;
+		else $new_idx = $var_idx + 1;
+		if ($new_idx < 1) return new Object(-1, 'msg_invalid_request');
 
 		$args = new stdClass();
 		$args->module_srl = $module_srl;
@@ -634,16 +634,16 @@ 
 		unset($args);
 
 		// update immediately if there is no idx to change
-		if(!$extra_keys[$new_idx])
+		if (!$extra_keys[$new_idx])
 		{
 			$args = new stdClass();
 			$args->module_srl = $module_srl;
 			$args->var_idx = $var_idx;
 			$args->new_idx = $new_idx;
 			$output = executeQuery('document.updateDocumentExtraKeyIdx', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 			$output = executeQuery('document.updateDocumentExtraVarIdx', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 			// replace if exists
 		}
 		else
@@ -653,27 +653,27 @@ 
 			$args->var_idx = $new_idx;
 			$args->new_idx = -10000;
 			$output = executeQuery('document.updateDocumentExtraKeyIdx', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 			$output = executeQuery('document.updateDocumentExtraVarIdx', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 
 			$args->var_idx = $var_idx;
 			$args->new_idx = $new_idx;
 			$output = executeQuery('document.updateDocumentExtraKeyIdx', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 			$output = executeQuery('document.updateDocumentExtraVarIdx', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 
 			$args->var_idx = -10000;
 			$args->new_idx = $var_idx;
 			$output = executeQuery('document.updateDocumentExtraKeyIdx', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 			$output = executeQuery('document.updateDocumentExtraVarIdx', $args);
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 		}
 
 		$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);
@@ -687,9 +687,9 @@ 
 	 */
 	function procDocumentAdminInsertAlias()
 	{
-		$args = Context::gets('module_srl','document_srl', 'alias_title');
+		$args = Context::gets('module_srl', 'document_srl', 'alias_title');
 		$alias_srl = Context::get('alias_srl');
-		if(!$alias_srl)
+		if (!$alias_srl)
 		{
 			$args->alias_srl = getNextSequence();
 			$query = "document.insertAlias";
@@ -732,7 +732,7 @@ 
 		$oDocumentModel = getModel('document');
 		$oDocumentController = getController('document');
 		$oDocument = $oDocumentModel->getDocument($document_srl, false, false);
-		if(!$oDocument->isGranted()) return $this->stop('msg_not_permitted');
+		if (!$oDocument->isGranted()) return $this->stop('msg_not_permitted');
 	
 		$oModuleModel = getModel('module');
 		$module_info = $oModuleModel->getModuleInfoByDocumentSrl($document_srl);
@@ -744,31 +744,31 @@ 
 		$oDocumentController->moveDocumentToTrash($args);
 
 		$returnUrl = Context::get('success_return_url');
-		if(!$returnUrl)	
+		if (!$returnUrl)	
 		{
 			$arrUrl = parse_url(Context::get('cur_url'));
 			$query = "";
 
-			if($arrUrl['query'])
+			if ($arrUrl['query'])
 			{
 				parse_str($arrUrl['query'], $arrQuery);
 
 				// set query
-				if(isset($arrQuery['document_srl']))
+				if (isset($arrQuery['document_srl']))
 					unset($arrQuery['document_srl']);
 
 				$searchArgs = new stdClass;
-				foreach($arrQuery as $key=>$val)
+				foreach ($arrQuery as $key=>$val)
 				{
 					$searchArgs->{$key} = $val;
 				}
 
-				if(!isset($searchArgs->sort_index))
+				if (!isset($searchArgs->sort_index))
 					$searchArgs->sort_index = $module_info->order_target;
 
-				foreach($module_info as $key=>$val)
+				foreach ($module_info as $key=>$val)
 				{
-					if(!isset($searchArgs->{$key}))
+					if (!isset($searchArgs->{$key}))
 						$searchArgs->{$key} = $val;
 				}
 
@@ -776,20 +776,20 @@ 
 				$output = $oDocumentModel->getDocumentList($searchArgs, $module_info->except_notice, TRUE, array('document_srl'));
 
 				$cur_page = 1;
-				if(isset($arrQuery['page'])) {
-					$cur_page = (int)$arrQuery['page'];
+				if (isset($arrQuery['page'])) {
+					$cur_page = (int) $arrQuery['page'];
 				}
 
 
-				if($cur_page>1 && count($output->data) == 0)
+				if ($cur_page > 1 && count($output->data) == 0)
 					$arrQuery['page'] = $cur_page - 1;
 
 				$query = "?";
-				foreach($arrQuery as $key=>$val)
+				foreach ($arrQuery as $key=>$val)
 					$query .= sprintf("%s=%s&", $key, $val);
 				$query = substr($query, 0, -1);
 			}
-			$returnUrl = $arrUrl['path'] . $query;
+			$returnUrl = $arrUrl['path'].$query;
 		}
 
 		$this->add('redirect_url', $returnUrl);
@@ -866,7 +866,7 @@ 
 	 */
 	function restoreTrash($originObject)
 	{
-		if(is_array($originObject)) $originObject = (object)$originObject;
+		if (is_array($originObject)) $originObject = (object) $originObject;
 
 		$oDocumentController = getController('document');
 		$oDocumentModel = getModel('document');
@@ -876,12 +876,12 @@ 
 
 		//DB restore
 		$output = $oDocumentController->insertDocument($originObject, false, true, false);
-		if(!$output->toBool()) return new Object(-1, $output->getMessage());
+		if (!$output->toBool()) return new Object(-1, $output->getMessage());
 
 		//FILE restore
 		$oDocument = $oDocumentModel->getDocument($originObject->document_srl);
 		// If the post was not temorarily saved, set the attachment's status to be valid
-		if($oDocument->hasUploadedFiles() && $originObject->member_srl != $originObject->module_srl)
+		if ($oDocument->hasUploadedFiles() && $originObject->member_srl != $originObject->module_srl)
 		{
 			$args = new stdClass();
 			$args->upload_target_srl = $oDocument->document_srl;
@@ -890,10 +890,10 @@ 
 		}
 
 		// call a trigger (after)
-		if($output->toBool())
+		if ($output->toBool())
 		{
 			$trigger_output = ModuleHandler::triggerCall('document.restoreTrash', 'after', $originObject);
-			if(!$trigger_output->toBool())
+			if (!$trigger_output->toBool())
 			{
 				$oDB->rollback();
 				return $trigger_output;
@@ -914,7 +914,7 @@ 
 	function emptyTrash($originObject)
 	{
 		$originObject = unserialize($originObject);
-		if(is_array($originObject)) $originObject = (object) $originObject;
+		if (is_array($originObject)) $originObject = (object) $originObject;
 
 		$oDocument = new documentItem();
 		$oDocument->setAttribute($originObject);

modules/document/document.class.php

Doc Comments +1 added lines

@@ -351,6 +351,7 @@
 
 	/**
 	 * Return status by key
+	 * @param string $key
 	 * @return string
 	 */
 	function getConfigStatus($key)

Braces +140 added lines, -54 removed lines

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

Spacing +91 added lines, -91 removed lines

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