xpressengine / xe-core

modules/board/board.admin.controller.php

Spacing +32 added lines, -32 removed lines

@@ -27,37 +27,37 @@ 
 		$args = Context::getRequestVars();
 		$args->module = 'board';
 		$args->mid = $args->board_name;
-		if(is_array($args->use_status)) $args->use_status = implode('|@|', $args->use_status);
+		if (is_array($args->use_status)) $args->use_status = implode('|@|', $args->use_status);
 		unset($args->board_name);
 
 		// setup extra_order_target
 		$extra_order_target = array();
-		if($args->module_srl)
+		if ($args->module_srl)
 		{
 			$oDocumentModel = getModel('document');
 			$module_extra_vars = $oDocumentModel->getExtraKeys($args->module_srl);
-			foreach($module_extra_vars as $oExtraItem)
+			foreach ($module_extra_vars as $oExtraItem)
 			{
 				$extra_order_target[$oExtraItem->eid] = $oExtraItem->name;
 			}
 		}
 
 		// setup other variables
-		if($args->except_notice != 'Y') $args->except_notice = 'N';
-		if($args->use_anonymous != 'Y') $args->use_anonymous = 'N';
-		if($args->consultation != 'Y') $args->consultation = 'N';
-		if($args->protect_content!= 'Y') $args->protect_content = 'N';
-		if(!in_array($args->order_target,$this->order_target) && !array_key_exists($args->order_target, $extra_order_target)) $args->order_target = 'list_order';
-		if(!in_array($args->order_type, array('asc', 'desc'))) $args->order_type = 'asc';
+		if ($args->except_notice != 'Y') $args->except_notice = 'N';
+		if ($args->use_anonymous != 'Y') $args->use_anonymous = 'N';
+		if ($args->consultation != 'Y') $args->consultation = 'N';
+		if ($args->protect_content != 'Y') $args->protect_content = 'N';
+		if (!in_array($args->order_target, $this->order_target) && !array_key_exists($args->order_target, $extra_order_target)) $args->order_target = 'list_order';
+		if (!in_array($args->order_type, array('asc', 'desc'))) $args->order_type = 'asc';
 
 		// if there is an existed module
-		if($args->module_srl) {
+		if ($args->module_srl) {
 			$module_info = $oModuleModel->getModuleInfoByModuleSrl($args->module_srl);
-			if($module_info->module_srl != $args->module_srl) unset($args->module_srl);
+			if ($module_info->module_srl != $args->module_srl) unset($args->module_srl);
 		}
 
 		// insert/update the board module based on module_srl
-		if(!$args->module_srl) {
+		if (!$args->module_srl) {
 			$args->hide_category = 'N';
 			$output = $oModuleController->insertModule($args);
 			$msg_code = 'success_registed';
@@ -67,18 +67,18 @@ 
 			$msg_code = 'success_updated';
 		}
 
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		// setup list config
-		$list = explode(',',Context::get('list'));
-		if(count($list))
+		$list = explode(',', Context::get('list'));
+		if (count($list))
 		{
 			$list_arr = array();
-			foreach($list as $val)
+			foreach ($list as $val)
 			{
 				$val = trim($val);
-				if(!$val) continue;
-				if(substr($val,0,10)=='extra_vars') $val = substr($val,10);
+				if (!$val) continue;
+				if (substr($val, 0, 10) == 'extra_vars') $val = substr($val, 10);
 				$list_arr[] = $val;
 			}
 			$oModuleController = getController('module');
@@ -86,10 +86,10 @@ 
 		}
 
 		$this->setMessage($msg_code);
-		if (Context::get('success_return_url')){
+		if (Context::get('success_return_url')) {
 			changeValueInUrl('mid', $args->mid, $module_info->mid);
 			$this->setRedirectUrl(Context::get('success_return_url'));
-		}else{
+		} else {
 			$this->setRedirectUrl(getNotEncodedUrl('', 'module', 'admin', 'act', 'dispBoardAdminBoardInfo', 'module_srl', $output->get('module_srl')));
 		}
 	}
@@ -105,17 +105,17 @@ 
 		// for board info
 		$args->module = 'board';
 		$args->mid = $args->board_name;
-		if(is_array($args->use_status))
+		if (is_array($args->use_status))
 		{
 			$args->use_status = implode('|@|', $args->use_status);
 		}
 		unset($args->board_name);
 
-		if(!in_array($args->order_target, $this->order_target))
+		if (!in_array($args->order_target, $this->order_target))
 		{
 			$args->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 = 'asc';
 		}
@@ -125,13 +125,13 @@ 
 
 		// for grant info, Register Admin ID
 		$oModuleController->deleteAdminId($args->module_srl);
-		if($args->admin_member)
+		if ($args->admin_member)
 		{
-			$admin_members = explode(',',$args->admin_member);
-			for($i=0;$i<count($admin_members);$i++)
+			$admin_members = explode(',', $args->admin_member);
+			for ($i = 0; $i < count($admin_members); $i++)
 			{
 				$admin_id = trim($admin_members[$i]);
-				if(!$admin_id) continue;
+				if (!$admin_id) continue;
 				$oModuleController->insertAdminId($args->module_srl, $admin_id);
 			}
 		}
@@ -146,10 +146,10 @@ 
 		// get the current module
 		$oModuleController = getController('module');
 		$output = $oModuleController->deleteModule($module_srl);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		$this->add('module','board');
-		$this->add('page',Context::get('page'));
+		$this->add('module', 'board');
+		$this->add('page', Context::get('page'));
 		$this->setMessage('success_deleted');
 	}
 
@@ -160,7 +160,7 @@ 
 
 		$oModuleModel = getModel('module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl);
-		if($module_info->mid != $mid)
+		if ($module_info->mid != $mid)
 		{
 			return new Object(-1, 'msg_invalid_request');
 		}
@@ -168,7 +168,7 @@ 
 		$module_info->hide_category = Context::get('hide_category') == 'Y' ? 'Y' : 'N';
 		$oModuleController = getController('module'); /* @var $oModuleController moduleController */
 		$output = $oModuleController->updateModule($module_info);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}

Braces +41 added lines, -16 removed lines

@@ -27,7 +27,9 @@ 
 		$args = Context::getRequestVars();
 		$args->module = 'board';
 		$args->mid = $args->board_name;
-		if(is_array($args->use_status)) $args->use_status = implode('|@|', $args->use_status);
+		if(is_array($args->use_status)) {
+			$args->use_status = implode('|@|', $args->use_status);
+		}
 		unset($args->board_name);
 
 		// setup extra_order_target
@@ -43,17 +45,31 @@ 
 		}
 
 		// setup other variables
-		if($args->except_notice != 'Y') $args->except_notice = 'N';
-		if($args->use_anonymous != 'Y') $args->use_anonymous = 'N';
-		if($args->consultation != 'Y') $args->consultation = 'N';
-		if($args->protect_content!= 'Y') $args->protect_content = 'N';
-		if(!in_array($args->order_target,$this->order_target) && !array_key_exists($args->order_target, $extra_order_target)) $args->order_target = 'list_order';
-		if(!in_array($args->order_type, array('asc', 'desc'))) $args->order_type = 'asc';
+		if($args->except_notice != 'Y') {
+			$args->except_notice = 'N';
+		}
+		if($args->use_anonymous != 'Y') {
+			$args->use_anonymous = 'N';
+		}
+		if($args->consultation != 'Y') {
+			$args->consultation = 'N';
+		}
+		if($args->protect_content!= 'Y') {
+			$args->protect_content = 'N';
+		}
+		if(!in_array($args->order_target,$this->order_target) && !array_key_exists($args->order_target, $extra_order_target)) {
+			$args->order_target = 'list_order';
+		}
+		if(!in_array($args->order_type, array('asc', 'desc'))) {
+			$args->order_type = 'asc';
+		}
 
 		// if there is an existed module
 		if($args->module_srl) {
 			$module_info = $oModuleModel->getModuleInfoByModuleSrl($args->module_srl);
-			if($module_info->module_srl != $args->module_srl) unset($args->module_srl);
+			if($module_info->module_srl != $args->module_srl) {
+				unset($args->module_srl);
+			}
 		}
 
 		// insert/update the board module based on module_srl
@@ -67,7 +83,9 @@ 
 			$msg_code = 'success_updated';
 		}
 
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		// setup list config
 		$list = explode(',',Context::get('list'));
@@ -77,8 +95,12 @@ 
 			foreach($list as $val)
 			{
 				$val = trim($val);
-				if(!$val) continue;
-				if(substr($val,0,10)=='extra_vars') $val = substr($val,10);
+				if(!$val) {
+					continue;
+				}
+				if(substr($val,0,10)=='extra_vars') {
+					$val = substr($val,10);
+				}
 				$list_arr[] = $val;
 			}
 			$oModuleController = getController('module');
@@ -89,7 +111,7 @@ 
 		if (Context::get('success_return_url')){
 			changeValueInUrl('mid', $args->mid, $module_info->mid);
 			$this->setRedirectUrl(Context::get('success_return_url'));
-		}else{
+		} else{
 			$this->setRedirectUrl(getNotEncodedUrl('', 'module', 'admin', 'act', 'dispBoardAdminBoardInfo', 'module_srl', $output->get('module_srl')));
 		}
 	}
@@ -131,7 +153,9 @@ 
 			for($i=0;$i<count($admin_members);$i++)
 			{
 				$admin_id = trim($admin_members[$i]);
-				if(!$admin_id) continue;
+				if(!$admin_id) {
+					continue;
+				}
 				$oModuleController->insertAdminId($args->module_srl, $admin_id);
 			}
 		}
@@ -146,7 +170,9 @@ 
 		// get the current module
 		$oModuleController = getController('module');
 		$output = $oModuleController->deleteModule($module_srl);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->add('module','board');
 		$this->add('page',Context::get('page'));
@@ -177,8 +203,7 @@ 
 		if (Context::get('success_return_url'))
 		{
 			$this->setRedirectUrl(Context::get('success_return_url'));
-		}
-		else
+		} else
 		{
 			$this->setRedirectUrl(getNotEncodedUrl('', 'module', 'admin', 'act', 'dispBoardAdminCategoryInfo', 'module_srl', $output->get('module_srl')));
 		}

modules/board/board.admin.view.php

Spacing +29 added lines, -29 removed lines

@@ -17,7 +17,7 @@ 
 	function init() {
 		// check module_srl is existed or not
 		$module_srl = Context::get('module_srl');
-		if(!$module_srl && $this->module_srl) {
+		if (!$module_srl && $this->module_srl) {
 			$module_srl = $this->module_srl;
 			Context::set('module_srl', $module_srl);
 		}
@@ -26,20 +26,20 @@ 
 		$oModuleModel = getModel('module');
 
 		// get the module infomation based on the module_srl
-		if($module_srl) {
+		if ($module_srl) {
 			$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl);
-			if(!$module_info) {
-				Context::set('module_srl','');
+			if (!$module_info) {
+				Context::set('module_srl', '');
 				$this->act = 'list';
 			} else {
 				ModuleModel::syncModuleToSite($module_info);
 				$this->module_info = $module_info;
 				$this->module_info->use_status = explode('|@|', $module_info->use_status);
-				Context::set('module_info',$module_info);
+				Context::set('module_info', $module_info);
 			}
 		}
 
-		if($module_info && $module_info->module != 'board') return $this->stop("msg_invalid_request");
+		if ($module_info && $module_info->module != 'board') return $this->stop("msg_invalid_request");
 
 		// get the module category list
 		$module_category = $oModuleModel->getModuleCategories();
@@ -50,11 +50,11 @@ 
 		$security->encodeHTML('module_category..');
 
 		// setup template path (board admin panel templates is resided in the tpl folder)
-		$template_path = sprintf("%stpl/",$this->module_path);
+		$template_path = sprintf("%stpl/", $this->module_path);
 		$this->setTemplatePath($template_path);
 
 		// install order (sorting) options
-		foreach($this->order_target as $key) $order_target[$key] = Context::getLang($key);
+		foreach ($this->order_target as $key) $order_target[$key] = Context::getLang($key);
 		$order_target['list_order'] = Context::getLang('document_srl');
 		$order_target['update_order'] = Context::getLang('last_update');
 		Context::set('order_target', $order_target);
@@ -75,7 +75,7 @@ 
 		$search_target = Context::get('search_target');
 		$search_keyword = Context::get('search_keyword');
 
-		switch ($search_target){
+		switch ($search_target) {
 			case 'mid':
 				$args->s_mid = $search_keyword;
 				break;
@@ -90,7 +90,7 @@ 
 		// get the skins path
 		$oModuleModel = getModel('module');
 		$skin_list = $oModuleModel->getSkins($this->module_path);
-		Context::set('skin_list',$skin_list);
+		Context::set('skin_list', $skin_list);
 
 		$mskin_list = $oModuleModel->getSkins($this->module_path, "m.skins");
 		Context::set('mskin_list', $mskin_list);
@@ -100,7 +100,7 @@ 
 		$layout_list = $oLayoutModel->getLayoutList();
 		Context::set('layout_list', $layout_list);
 
-		$mobile_layout_list = $oLayoutModel->getLayoutList(0,"M");
+		$mobile_layout_list = $oLayoutModel->getLayoutList(0, "M");
 		Context::set('mlayout_list', $mobile_layout_list);
 
 		$oModuleAdminModel = getAdminModel('module');
@@ -115,10 +115,10 @@ 
 		Context::set('page_navigation', $output->page_navigation);
 
 		$security = new Security();
-		$security->encodeHTML('board_list..browser_title','board_list..mid');
-		$security->encodeHTML('skin_list..title','mskin_list..title');
-		$security->encodeHTML('layout_list..title','layout_list..layout');
-		$security->encodeHTML('mlayout_list..title','mlayout_list..layout');
+		$security->encodeHTML('board_list..browser_title', 'board_list..mid');
+		$security->encodeHTML('skin_list..title', 'mskin_list..title');
+		$security->encodeHTML('layout_list..title', 'layout_list..layout');
+		$security->encodeHTML('mlayout_list..title', 'mlayout_list..layout');
 
 		// 템플릿 파일 지정
 		$this->setTemplateFile('index');
@@ -135,14 +135,14 @@ 
 	 * @brief display the module insert form
 	 **/
 	function dispBoardAdminInsertBoard() {
-		if(!in_array($this->module_info->module, array('admin', 'board','blog','guestbook'))) {
+		if (!in_array($this->module_info->module, array('admin', 'board', 'blog', 'guestbook'))) {
 			return $this->alertMessage('msg_invalid_request');
 		}
 
 		// get the skins list
 		$oModuleModel = getModel('module');
 		$skin_list = $oModuleModel->getSkins($this->module_path);
-		Context::set('skin_list',$skin_list);
+		Context::set('skin_list', $skin_list);
 
 		$mskin_list = $oModuleModel->getSkins($this->module_path, "m.skins");
 		Context::set('mskin_list', $mskin_list);
@@ -152,13 +152,13 @@ 
 		$layout_list = $oLayoutModel->getLayoutList();
 		Context::set('layout_list', $layout_list);
 
-		$mobile_layout_list = $oLayoutModel->getLayoutList(0,"M");
+		$mobile_layout_list = $oLayoutModel->getLayoutList(0, "M");
 		Context::set('mlayout_list', $mobile_layout_list);
 
 		$security = new Security();
-		$security->encodeHTML('skin_list..title','mskin_list..title');
-		$security->encodeHTML('layout_list..title','layout_list..layout');
-		$security->encodeHTML('mlayout_list..title','mlayout_list..layout');
+		$security->encodeHTML('skin_list..title', 'mskin_list..title');
+		$security->encodeHTML('layout_list..title', 'layout_list..layout');
+		$security->encodeHTML('mlayout_list..title', 'mlayout_list..layout');
 
 		// get document status list
 		$oDocumentModel = getModel('document');
@@ -177,14 +177,14 @@ 
 		// setup extra_order_target
 		$module_extra_vars = $oDocumentModel->getExtraKeys($this->module_info->module_srl);
 		$extra_order_target = array();
-		foreach($module_extra_vars as $oExtraItem)
+		foreach ($module_extra_vars as $oExtraItem)
 		{
 			$extra_order_target[$oExtraItem->eid] = $oExtraItem->name;
 		}
 		Context::set('extra_order_target', $extra_order_target);
 
 		$security = new Security();
-		$security->encodeHTML('extra_vars..name','list_config..name');
+		$security->encodeHTML('extra_vars..name', 'list_config..name');
 
 		// set the template file
 		$this->setTemplateFile('board_insert');
@@ -212,8 +212,8 @@ 
 	 * @brief display the board mdoule delete page
 	 **/
 	function dispBoardAdminDeleteBoard() {
-		if(!Context::get('module_srl')) return $this->dispBoardAdminContent();
-		if(!in_array($this->module_info->module, array('admin', 'board','blog','guestbook'))) {
+		if (!Context::get('module_srl')) return $this->dispBoardAdminContent();
+		if (!in_array($this->module_info->module, array('admin', 'board', 'blog', 'guestbook'))) {
 			return $this->alertMessage('msg_invalid_request');
 		}
 
@@ -223,10 +223,10 @@ 
 		$document_count = $oDocumentModel->getDocumentCount($module_info->module_srl);
 		$module_info->document_count = $document_count;
 
-		Context::set('module_info',$module_info);
+		Context::set('module_info', $module_info);
 
 		$security = new Security();
-		$security->encodeHTML('module_info..mid','module_info..module','module_info..document_count');
+		$security->encodeHTML('module_info..mid', 'module_info..module', 'module_info..document_count');
 
 		// setup the template file
 		$this->setTemplateFile('board_delete');
@@ -295,7 +295,7 @@ 
 	 * @brief board module message
 	 **/
 	function alertMessage($message) {
-		$script =  sprintf('<script> xAddEventListener(window,"load", function() { alert("%s"); } );</script>', Context::getLang($message));
-		Context::addHtmlHeader( $script );
+		$script = sprintf('<script> xAddEventListener(window,"load", function() { alert("%s"); } );</script>', Context::getLang($message));
+		Context::addHtmlHeader($script);
 	}
 }

Braces +9 added lines, -3 removed lines

@@ -39,7 +39,9 @@ 
 			}
 		}
 
-		if($module_info && $module_info->module != 'board') return $this->stop("msg_invalid_request");
+		if($module_info && $module_info->module != 'board') {
+			return $this->stop("msg_invalid_request");
+		}
 
 		// get the module category list
 		$module_category = $oModuleModel->getModuleCategories();
@@ -54,7 +56,9 @@ 
 		$this->setTemplatePath($template_path);
 
 		// install order (sorting) options
-		foreach($this->order_target as $key) $order_target[$key] = Context::getLang($key);
+		foreach($this->order_target as $key) {
+			$order_target[$key] = Context::getLang($key);
+		}
 		$order_target['list_order'] = Context::getLang('document_srl');
 		$order_target['update_order'] = Context::getLang('last_update');
 		Context::set('order_target', $order_target);
@@ -212,7 +216,9 @@ 
 	 * @brief display the board mdoule delete page
 	 **/
 	function dispBoardAdminDeleteBoard() {
-		if(!Context::get('module_srl')) return $this->dispBoardAdminContent();
+		if(!Context::get('module_srl')) {
+			return $this->dispBoardAdminContent();
+		}
 		if(!in_array($this->module_info->module, array('admin', 'board','blog','guestbook'))) {
 			return $this->alertMessage('msg_invalid_request');
 		}

modules/board/board.class.php

Spacing +11 added lines, -11 removed lines

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

Braces +12 added lines, -4 removed lines

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

modules/board/board.controller.php

Spacing +58 added lines, -58 removed lines

@@ -23,11 +23,11 @@ 
 	function procBoardInsertDocument()
 	{
 		// check grant
-		if($this->module_info->module != "board")
+		if ($this->module_info->module != "board")
 		{
 			return new Object(-1, "msg_invalid_request");
 		}
-		if(!$this->grant->write_document)
+		if (!$this->grant->write_document)
 		{
 			return new Object(-1, 'msg_not_permitted');
 		}
@@ -36,16 +36,16 @@ 
 		// setup variables
 		$obj = Context::getRequestVars();
 		$obj->module_srl = $this->module_srl;
-		if($obj->is_notice!='Y'||!$this->grant->manager) $obj->is_notice = 'N';
+		if ($obj->is_notice != 'Y' || !$this->grant->manager) $obj->is_notice = 'N';
 		$obj->commentStatus = $obj->comment_status;
 
 		settype($obj->title, "string");
-		if($obj->title == '') $obj->title = cut_str(trim(strip_tags(nl2br($obj->content))),20,'...');
+		if ($obj->title == '') $obj->title = cut_str(trim(strip_tags(nl2br($obj->content))), 20, '...');
 		//setup dpcument title tp 'Untitled'
-		if($obj->title == '') $obj->title = 'Untitled';
+		if ($obj->title == '') $obj->title = 'Untitled';
 
 		// unset document style if the user is not the document manager
-		if(!$this->grant->manager)
+		if (!$this->grant->manager)
 		{
 			unset($obj->title_color);
 			unset($obj->title_bold);
@@ -62,24 +62,24 @@ 
 
 		// update the document if it is existed
 		$is_update = false;
-		if($oDocument->isExists() && $oDocument->document_srl == $obj->document_srl)
+		if ($oDocument->isExists() && $oDocument->document_srl == $obj->document_srl)
 		{
 			$is_update = true;
 		}
 
 		// if use anonymous is true
-		if($this->module_info->use_anonymous == 'Y')
+		if ($this->module_info->use_anonymous == 'Y')
 		{
 			$this->module_info->admin_mail = '';
 			$obj->notify_message = 'N';
-			if($is_update===false)
+			if ($is_update === false)
 			{
-				$obj->member_srl = -1*$logged_info->member_srl;
+				$obj->member_srl = -1 * $logged_info->member_srl;
 			}
 			$obj->email_address = $obj->homepage = $obj->user_id = '';
 			$obj->user_name = $obj->nick_name = 'anonymous';
 			$bAnonymous = true;
-			if($is_update===false)
+			if ($is_update === false)
 			{
 				$oDocument->add('member_srl', $obj->member_srl);
 			}
@@ -89,10 +89,10 @@ 
 			$bAnonymous = false;
 		}
 
-		if($obj->is_secret == 'Y' || strtoupper($obj->status == 'SECRET'))
+		if ($obj->is_secret == 'Y' || strtoupper($obj->status == 'SECRET'))
 		{
 			$use_status = explode('|@|', $this->module_info->use_status);
-			if(!is_array($use_status) || !in_array('SECRET', $use_status))
+			if (!is_array($use_status) || !in_array('SECRET', $use_status))
 			{
 				unset($obj->is_secret);
 				$obj->status = 'PUBLIC';
@@ -100,19 +100,19 @@ 
 		}
 
 		// update the document if it is existed
-		if($is_update)
+		if ($is_update)
 		{
-			if(!$oDocument->isGranted())
+			if (!$oDocument->isGranted())
 			{
-				return new Object(-1,'msg_not_permitted');
+				return new Object(-1, 'msg_not_permitted');
 			}
 
-			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 new Object(-1,'msg_protect_content');
+				return new Object(-1, 'msg_protect_content');
 			}
 
-			if(!$this->grant->manager)
+			if (!$this->grant->manager)
 			{
 				// notice & document style same as before if not manager
 				$obj->is_notice = $oDocument->get('is_notice');
@@ -121,7 +121,7 @@ 
 			}
 			
 			// modify list_order if document status is temp
-			if($oDocument->get('status') == 'TEMP')
+			if ($oDocument->get('status') == 'TEMP')
 			{
 				$obj->last_update = $obj->regdate = date('YmdHis');
 				$obj->update_order = $obj->list_order = (getNextSequence() * -1);
@@ -137,18 +137,18 @@ 
 			$obj->document_srl = $output->get('document_srl');
 
 			// send an email to admin user
-			if($output->toBool() && $this->module_info->admin_mail)
+			if ($output->toBool() && $this->module_info->admin_mail)
 			{
 				$oMail = new Mail();
 				$oMail->setTitle($obj->title);
-				$oMail->setContent( sprintf("From : <a href=\"%s\">%s</a><br/>\r\n%s", getFullUrl('','document_srl',$obj->document_srl), getFullUrl('','document_srl',$obj->document_srl), $obj->content));
+				$oMail->setContent(sprintf("From : <a href=\"%s\">%s</a><br/>\r\n%s", getFullUrl('', 'document_srl', $obj->document_srl), getFullUrl('', 'document_srl', $obj->document_srl), $obj->content));
 				$oMail->setSender($obj->user_name, $obj->email_address);
 
-				$target_mail = explode(',',$this->module_info->admin_mail);
-				for($i=0;$i<count($target_mail);$i++)
+				$target_mail = explode(',', $this->module_info->admin_mail);
+				for ($i = 0; $i < count($target_mail); $i++)
 				{
 					$email_address = trim($target_mail[$i]);
-					if(!$email_address) continue;
+					if (!$email_address) continue;
 					$oMail->setReceiptor($email_address, $email_address);
 					$oMail->send();
 				}
@@ -156,7 +156,7 @@ 
 		}
 
 		// if there is an error
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -178,7 +178,7 @@ 
 		$document_srl = Context::get('document_srl');
 
 		// if the document is not existed
-		if(!$document_srl)
+		if (!$document_srl)
 		{
 			return $this->doError('msg_invalid_document');
 		}
@@ -186,7 +186,7 @@ 
 		$oDocumentModel = &getModel('document');
 		$oDocument = $oDocumentModel->getDocument($document_srl);
 		// check protect content
-		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 new Object(-1, 'msg_protect_content');
 		}
@@ -196,7 +196,7 @@ 
 
 		// delete the document
 		$output = $oDocumentController->deleteDocument($document_srl, $this->grant->manager);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -226,7 +226,7 @@ 
 	function procBoardInsertComment()
 	{
 		// check grant
-		if(!$this->grant->write_comment)
+		if (!$this->grant->write_comment)
 		{
 			return new Object(-1, 'msg_not_permitted');
 		}
@@ -236,13 +236,13 @@ 
 		$obj = Context::getRequestVars();
 		$obj->module_srl = $this->module_srl;
 
-		if(!$this->module_info->use_status) $this->module_info->use_status = 'PUBLIC';
-		if(!is_array($this->module_info->use_status))
+		if (!$this->module_info->use_status) $this->module_info->use_status = 'PUBLIC';
+		if (!is_array($this->module_info->use_status))
 		{
 			$this->module_info->use_status = explode('|@|', $this->module_info->use_status);
 		}
 
-		if(in_array('SECRET', $this->module_info->use_status))
+		if (in_array('SECRET', $this->module_info->use_status))
 		{
 			$this->module_info->secret = 'Y';
 		}
@@ -255,17 +255,17 @@ 
 		// check if the doument is existed
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument($obj->document_srl);
-		if(!$oDocument->isExists())
+		if (!$oDocument->isExists())
 		{
-			return new Object(-1,'msg_not_founded');
+			return new Object(-1, 'msg_not_founded');
 		}
 
 		// For anonymous use, remove writer's information and notifying information
-		if($this->module_info->use_anonymous == 'Y')
+		if ($this->module_info->use_anonymous == 'Y')
 		{
 			$this->module_info->admin_mail = '';
 			$obj->notify_message = 'N';
-			$obj->member_srl = -1*$logged_info->member_srl;
+			$obj->member_srl = -1 * $logged_info->member_srl;
 			$obj->email_address = $obj->homepage = $obj->user_id = '';
 			$obj->user_name = $obj->nick_name = 'anonymous';
 			$bAnonymous = true;
@@ -283,7 +283,7 @@ 
 
 		// check the comment is existed
 		// if the comment is not existed, then generate a new sequence
-		if(!$obj->comment_srl)
+		if (!$obj->comment_srl)
 		{
 			$obj->comment_srl = getNextSequence();
 		} else {
@@ -291,14 +291,14 @@ 
 		}
 
 		// if comment_srl is not existed, then insert the comment
-		if($comment->comment_srl != $obj->comment_srl)
+		if ($comment->comment_srl != $obj->comment_srl)
 		{
 
 			// parent_srl is existed
-			if($obj->parent_srl)
+			if ($obj->parent_srl)
 			{
 				$parent_comment = $oCommentModel->getComment($obj->parent_srl);
-				if(!$parent_comment->comment_srl)
+				if (!$parent_comment->comment_srl)
 				{
 					return new Object(-1, 'msg_invalid_request');
 				}
@@ -312,9 +312,9 @@ 
 		// update the comment if it is not existed
 		} else {
 			// check the grant
-			if(!$comment->isGranted())
+			if (!$comment->isGranted())
 			{
-				return new Object(-1,'msg_not_permitted');
+				return new Object(-1, 'msg_not_permitted');
 			}
 
 			$obj->parent_srl = $comment->parent_srl;
@@ -322,7 +322,7 @@ 
 			$comment_srl = $obj->comment_srl;
 		}
 
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -340,7 +340,7 @@ 
 	{
 		// get the comment_srl
 		$comment_srl = Context::get('comment_srl');
-		if(!$comment_srl)
+		if (!$comment_srl)
 		{
 			return $this->doError('msg_invalid_request');
 		}
@@ -349,7 +349,7 @@ 
 		$oCommentController = getController('comment');
 
 		$output = $oCommentController->deleteComment($comment_srl, $this->grant->manager);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -370,10 +370,10 @@ 
 		// generate trackback module controller object
 		$oTrackbackController = getController('trackback');
 
-		if(!$oTrackbackController) return;
+		if (!$oTrackbackController) return;
 
 		$output = $oTrackbackController->deleteTrackback($trackback_srl, $this->grant->manager);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
@@ -397,18 +397,18 @@ 
 		$oMemberModel = getModel('member');
 
 		// if the comment exists
-		if($comment_srl)
+		if ($comment_srl)
 		{
 			// get the comment information
 			$oCommentModel = getModel('comment');
 			$oComment = $oCommentModel->getComment($comment_srl);
-			if(!$oComment->isExists())
+			if (!$oComment->isExists())
 			{
 				return new Object(-1, 'msg_invalid_request');
 			}
 
 			// compare the comment password and the user input password
-			if(!$oMemberModel->isValidPassword($oComment->get('password'),$password))
+			if (!$oMemberModel->isValidPassword($oComment->get('password'), $password))
 			{
 				return new Object(-1, 'msg_invalid_password');
 			}
@@ -418,13 +418,13 @@ 
 			 // get the document information
 			$oDocumentModel = getModel('document');
 			$oDocument = $oDocumentModel->getDocument($document_srl);
-			if(!$oDocument->isExists())
+			if (!$oDocument->isExists())
 			{
 				return new Object(-1, 'msg_invalid_request');
 			}
 
 			// compare the document password and the user input password
-			if(!$oMemberModel->isValidPassword($oDocument->get('password'),$password))
+			if (!$oMemberModel->isValidPassword($oDocument->get('password'), $password))
 			{
 				return new Object(-1, 'msg_invalid_password');
 			}
@@ -441,7 +441,7 @@ 
 		$member_srl = Context::get('target_srl');
 		$mid = Context::get('cur_mid');
 
-		if(!$member_srl || !$mid)
+		if (!$member_srl || !$mid)
 		{
 			return new Object();
 		}
@@ -453,13 +453,13 @@ 
 		$columnList = array('module');
 		$cur_module_info = $oModuleModel->getModuleInfoByMid($mid, 0, $columnList);
 
-		if($cur_module_info->module != 'board')
+		if ($cur_module_info->module != 'board')
 		{
 			return new Object();
 		}
 
 		// get the member information
-		if($member_srl == $logged_info->member_srl)
+		if ($member_srl == $logged_info->member_srl)
 		{
 			$member_info = $logged_info;
 		} else {
@@ -467,13 +467,13 @@ 
 			$member_info = $oMemberModel->getMemberInfoByMemberSrl($member_srl);
 		}
 
-		if(!$member_info->user_id)
+		if (!$member_info->user_id)
 		{
 			return new Object();
 		}
 
 		//search
-		$url = getUrl('','mid',$mid,'search_target','nick_name','search_keyword',$member_info->nick_name);
+		$url = getUrl('', 'mid', $mid, 'search_target', 'nick_name', 'search_keyword', $member_info->nick_name);
 		$oMemberController = getController('member');
 		$oMemberController->addMemberPopupMenu($url, 'cmd_view_own_document', '');
 

Braces +21 added lines, -12 removed lines

@@ -36,13 +36,19 @@ 
 		// setup variables
 		$obj = Context::getRequestVars();
 		$obj->module_srl = $this->module_srl;
-		if($obj->is_notice!='Y'||!$this->grant->manager) $obj->is_notice = 'N';
+		if($obj->is_notice!='Y'||!$this->grant->manager) {
+			$obj->is_notice = 'N';
+		}
 		$obj->commentStatus = $obj->comment_status;
 
 		settype($obj->title, "string");
-		if($obj->title == '') $obj->title = cut_str(trim(strip_tags(nl2br($obj->content))),20,'...');
+		if($obj->title == '') {
+			$obj->title = cut_str(trim(strip_tags(nl2br($obj->content))),20,'...');
+		}
 		//setup dpcument title tp 'Untitled'
-		if($obj->title == '') $obj->title = 'Untitled';
+		if($obj->title == '') {
+			$obj->title = 'Untitled';
+		}
 
 		// unset document style if the user is not the document manager
 		if(!$this->grant->manager)
@@ -83,8 +89,7 @@ 
 			{
 				$oDocument->add('member_srl', $obj->member_srl);
 			}
-		}
-		else
+		} else
 		{
 			$bAnonymous = false;
 		}
@@ -148,7 +153,9 @@ 
 				for($i=0;$i<count($target_mail);$i++)
 				{
 					$email_address = trim($target_mail[$i]);
-					if(!$email_address) continue;
+					if(!$email_address) {
+						continue;
+					}
 					$oMail->setReceiptor($email_address, $email_address);
 					$oMail->send();
 				}
@@ -236,7 +243,9 @@ 
 		$obj = Context::getRequestVars();
 		$obj->module_srl = $this->module_srl;
 
-		if(!$this->module_info->use_status) $this->module_info->use_status = 'PUBLIC';
+		if(!$this->module_info->use_status) {
+			$this->module_info->use_status = 'PUBLIC';
+		}
 		if(!is_array($this->module_info->use_status))
 		{
 			$this->module_info->use_status = explode('|@|', $this->module_info->use_status);
@@ -245,8 +254,7 @@ 
 		if(in_array('SECRET', $this->module_info->use_status))
 		{
 			$this->module_info->secret = 'Y';
-		}
-		else
+		} else
 		{
 			unset($obj->is_secret);
 			$this->module_info->secret = 'N';
@@ -269,8 +277,7 @@ 
 			$obj->email_address = $obj->homepage = $obj->user_id = '';
 			$obj->user_name = $obj->nick_name = 'anonymous';
 			$bAnonymous = true;
-		}
-		else
+		} else
 		{
 			$bAnonymous = false;
 		}
@@ -370,7 +377,9 @@ 
 		// generate trackback module controller object
 		$oTrackbackController = getController('trackback');
 
-		if(!$oTrackbackController) return;
+		if(!$oTrackbackController) {
+			return;
+		}
 
 		$output = $oTrackbackController->deleteTrackback($trackback_srl, $this->grant->manager);
 		if(!$output->toBool())

modules/board/board.mobile.php

Spacing +21 added lines, -21 removed lines

@@ -1,7 +1,7 @@ 
 <?php

 /* Copyright (C) NAVER <http://www.navercorp.com> */

 

-require_once(_XE_PATH_.'modules/board/board.view.php');

+require_once(_XE_PATH_ . 'modules/board/board.view.php');

 

 class boardMobile extends boardView

 {

@@ -10,32 +10,32 @@ 
 		$oSecurity = new Security();

 		$oSecurity->encodeHTML('document_srl', 'comment_srl', 'vid', 'mid', 'page', 'category', 'search_target', 'search_keyword', 'sort_index', 'order_type', 'trackback_srl');

 

-		if($this->module_info->list_count) $this->list_count = $this->module_info->list_count;

-		if($this->module_info->mobile_list_count) $this->list_count = $this->module_info->mobile_list_count;

-		if($this->module_info->search_list_count) $this->search_list_count = $this->module_info->search_list_count;

-		if($this->module_info->mobile_search_list_count) $this->list_count = $this->module_info->mobile_search_list_count;

-		if($this->module_info->page_count) $this->page_count = $this->module_info->page_count;

-		if($this->module_info->mobile_page_count) $this->page_count = $this->module_info->mobile_page_count;

+		if ($this->module_info->list_count) $this->list_count = $this->module_info->list_count;

+		if ($this->module_info->mobile_list_count) $this->list_count = $this->module_info->mobile_list_count;

+		if ($this->module_info->search_list_count) $this->search_list_count = $this->module_info->search_list_count;

+		if ($this->module_info->mobile_search_list_count) $this->list_count = $this->module_info->mobile_search_list_count;

+		if ($this->module_info->page_count) $this->page_count = $this->module_info->page_count;

+		if ($this->module_info->mobile_page_count) $this->page_count = $this->module_info->mobile_page_count;

 		$this->except_notice = $this->module_info->except_notice == 'N' ? false : true;

 

 		// $this->_getStatusNameListecret option backward compatibility

 		$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';

 			}

@@ -55,10 +55,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)

+		if ($this->module_info->consultation == 'Y' && !$this->grant->manager)

 		{

 			$this->consultation = true;

-			if(!Context::get('is_logged')) $this->grant->list = $this->grant->write_document = $this->grant->write_comment = $this->grant->view = false;

+			if (!Context::get('is_logged')) $this->grant->list = $this->grant->write_document = $this->grant->write_comment = $this->grant->view = false;

 		} else {

 			$this->consultation = false;

 		}

@@ -67,14 +67,14 @@ 
 		$extra_keys = $oDocumentModel->getExtraKeys($this->module_info->module_srl);

 		Context::set('extra_keys', $extra_keys);

 

-		$template_path = sprintf("%sm.skins/%s/",$this->module_path, $this->module_info->mskin);

-		if(!is_dir($template_path)||!$this->module_info->mskin)

+		$template_path = sprintf("%sm.skins/%s/", $this->module_path, $this->module_info->mskin);

+		if (!is_dir($template_path) || !$this->module_info->mskin)

 		{

 			$this->module_info->mskin = 'default';

-			$template_path = sprintf("%sm.skins/%s/",$this->module_path, $this->module_info->mskin);

+			$template_path = sprintf("%sm.skins/%s/", $this->module_path, $this->module_info->mskin);

 		}

 		$this->setTemplatePath($template_path);

-		Context::addJsFilter($this->module_path.'tpl/filter', 'input_password.xml');

+		Context::addJsFilter($this->module_path . 'tpl/filter', 'input_password.xml');

 	}

 

 	function dispBoardCategory()

@@ -87,13 +87,13 @@ 
 	function getBoardCommentPage()

 	{

 		$document_srl = Context::get('document_srl');

-		$oDocumentModel =& getModel('document');

-		if(!$document_srl)

+		$oDocumentModel = & getModel('document');

+		if (!$document_srl)

 		{

 			return new Object(-1, "msg_invalid_request");

 		}

 		$oDocument = $oDocumentModel->getDocument($document_srl);

-		if(!$oDocument->isExists())

+		if (!$oDocument->isExists())

 		{

 			return new Object(-1, "msg_invalid_request");

 		}

@@ -106,7 +106,7 @@ 
 	function dispBoardMessage($msg_code)

 	{

 		$msg = Context::getLang($msg_code);

-		$oMessageObject = &ModuleHandler::getModuleInstance('message','mobile');

+		$oMessageObject = &ModuleHandler::getModuleInstance('message', 'mobile');

 		$oMessageObject->setError(-1);

 		$oMessageObject->setMessage($msg);

 		$oMessageObject->dispMessage();

Braces +24 added lines, -13 removed lines

@@ -10,12 +10,24 @@ 
 		$oSecurity = new Security();

 		$oSecurity->encodeHTML('document_srl', 'comment_srl', 'vid', 'mid', 'page', 'category', 'search_target', 'search_keyword', 'sort_index', 'order_type', 'trackback_srl');

 

-		if($this->module_info->list_count) $this->list_count = $this->module_info->list_count;

-		if($this->module_info->mobile_list_count) $this->list_count = $this->module_info->mobile_list_count;

-		if($this->module_info->search_list_count) $this->search_list_count = $this->module_info->search_list_count;

-		if($this->module_info->mobile_search_list_count) $this->list_count = $this->module_info->mobile_search_list_count;

-		if($this->module_info->page_count) $this->page_count = $this->module_info->page_count;

-		if($this->module_info->mobile_page_count) $this->page_count = $this->module_info->mobile_page_count;

+		if($this->module_info->list_count) {
+			$this->list_count = $this->module_info->list_count;
+		}
+		if($this->module_info->mobile_list_count) {
+			$this->list_count = $this->module_info->mobile_list_count;
+		}
+		if($this->module_info->search_list_count) {
+			$this->search_list_count = $this->module_info->search_list_count;
+		}
+		if($this->module_info->mobile_search_list_count) {
+			$this->list_count = $this->module_info->mobile_search_list_count;
+		}
+		if($this->module_info->page_count) {
+			$this->page_count = $this->module_info->page_count;
+		}
+		if($this->module_info->mobile_page_count) {
+			$this->page_count = $this->module_info->mobile_page_count;
+		}
 		$this->except_notice = $this->module_info->except_notice == 'N' ? false : true;

 

 		// $this->_getStatusNameListecret option backward compatibility

@@ -34,18 +46,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';

@@ -58,7 +67,9 @@ 
 		if($this->module_info->consultation == 'Y' && !$this->grant->manager)

 		{

 			$this->consultation = true;

-			if(!Context::get('is_logged')) $this->grant->list = $this->grant->write_document = $this->grant->write_comment = $this->grant->view = false;

+			if(!Context::get('is_logged')) {
+				$this->grant->list = $this->grant->write_document = $this->grant->write_comment = $this->grant->view = false;
+			}
 		} else {

 			$this->consultation = false;

 		}

modules/board/board.model.php

Spacing +11 added lines, -11 removed lines

@@ -25,21 +25,21 @@ 
 

 		// get the list config value, if it is not exitsted then setup the default value

 		$list_config = $oModuleModel->getModulePartConfig('board', $module_srl);

-		if(!$list_config || count($list_config) <= 0)

+		if (!$list_config || count($list_config) <= 0)

 		{

-			$list_config = array( 'no', 'title', 'nick_name','regdate','readed_count');

+			$list_config = array('no', 'title', 'nick_name', 'regdate', 'readed_count');

 		}

 

 		// get the extra variables

 		$inserted_extra_vars = $oDocumentModel->getExtraKeys($module_srl);

 

-		foreach($list_config as $key)

+		foreach ($list_config as $key)

 		{

-			if(preg_match('/^([0-9]+)$/',$key))

+			if (preg_match('/^([0-9]+)$/', $key))

 			{

-				if($inserted_extra_vars[$key])

+				if ($inserted_extra_vars[$key])

 				{

-					$output['extra_vars'.$key] = $inserted_extra_vars[$key];

+					$output['extra_vars' . $key] = $inserted_extra_vars[$key];

 				}

 				else

 				{

@@ -60,9 +60,9 @@ 
 	function getDefaultListConfig($module_srl)

 	{

 		// add virtual srl, title, registered date, update date, nickname, ID, name, readed count, voted count etc.

-		$virtual_vars = array( 'no', 'title', 'regdate', 'last_update', 'last_post', 'nick_name',

+		$virtual_vars = array('no', 'title', 'regdate', 'last_update', 'last_post', 'nick_name',

 				'user_id', 'user_name', 'readed_count', 'voted_count', 'blamed_count', 'thumbnail', 'summary', 'comment_status');

-		foreach($virtual_vars as $key)

+		foreach ($virtual_vars as $key)

 		{

 			$extra_vars[$key] = new ExtraItem($module_srl, -1, Context::getLang($key), $key, 'N', 'N', 'N', null);

 		}

@@ -71,11 +71,11 @@ 
 		$oDocumentModel = getModel('document');

 		$inserted_extra_vars = $oDocumentModel->getExtraKeys($module_srl);

 

-		if(count($inserted_extra_vars))

+		if (count($inserted_extra_vars))

 		{

-			foreach($inserted_extra_vars as $obj)

+			foreach ($inserted_extra_vars as $obj)

 			{

-				$extra_vars['extra_vars'.$obj->idx] = $obj;

+				$extra_vars['extra_vars' . $obj->idx] = $obj;

 			}

 		}

 

Braces +2 added lines, -4 removed lines

@@ -40,13 +40,11 @@
 				if($inserted_extra_vars[$key])

 				{

 					$output['extra_vars'.$key] = $inserted_extra_vars[$key];

-				}

-				else

+				} else

 				{

 					continue;

 				}

-			}

-			else

+			} else

 			{

 				$output[$key] = new ExtraItem($module_srl, -1, Context::getLang($key), $key, 'N', 'N', 'N', null);

 			}

modules/board/board.wap.php

Spacing +30 added lines, -30 removed lines

@@ -15,7 +15,7 @@ 
 	function procWAP(&$oMobile)

 	{

 		// check grant

-		if(!$this->grant->list || $this->module_info->consultation == 'Y')

+		if (!$this->grant->list || $this->module_info->consultation == 'Y')

 		{

 			return $oMobile->setContent(Context::getLang('msg_not_permitted'));

 		}

@@ -25,13 +25,13 @@ 
 

 		// if the doument is existed

 		$document_srl = Context::get('document_srl');

-		if($document_srl)

+		if ($document_srl)

 		{

 			$oDocument = $oDocumentModel->getDocument($document_srl);

-			if($oDocument->isExists())

+			if ($oDocument->isExists())

 			{

 				// check the grant

-				if(!$this->grant->view)

+				if (!$this->grant->view)

 				{

 					return $oMobile->setContent(Context::getLang('msg_not_permitted'));

 				}

@@ -40,48 +40,48 @@ 
 				Context::setBrowserTitle($oDocument->getTitleText());

 

 				// if the act is display comment list

-				if($this->act=='dispBoardContentCommentList')

+				if ($this->act == 'dispBoardContentCommentList')

 				{

 

 					$oCommentModel = getModel('comment');

 					$output = $oCommentModel->getCommentList($oDocument->document_srl, 0, false, $oDocument->getCommentCount());

 

 					$content = '';

-					if(count($output->data))

+					if (count($output->data))

 					{

-						foreach($output->data as $key => $val)

+						foreach ($output->data as $key => $val)

 						{

 							$oComment = new commentItem();

 							$oComment->setAttribute($val);

 

-							if(!$oComment->isAccessible()) continue;

+							if (!$oComment->isAccessible()) continue;

 

-							$content .= "<b>".$oComment->getNickName()."</b> (".$oComment->getRegdate("Y-m-d").")<br>\r\n".$oComment->getContent(false,false)."<br>\r\n";

+							$content .= "<b>" . $oComment->getNickName() . "</b> (" . $oComment->getRegdate("Y-m-d") . ")<br>\r\n" . $oComment->getContent(false, false) . "<br>\r\n";

 						}

 					}

 

 					// setup mobile contents

-					$oMobile->setContent( $content );

+					$oMobile->setContent($content);

 

 					// setup upper URL

-					$oMobile->setUpperUrl( getUrl('act',''), Context::getLang('cmd_go_upper') );

+					$oMobile->setUpperUrl(getUrl('act', ''), Context::getLang('cmd_go_upper'));

 

 				// display the document if the act is not display the comment list

 				} else {

 

 					// setup contents (strip all html tags)

-					$content = strip_tags(str_replace('<p>','<br>&nbsp;&nbsp;&nbsp;',$oDocument->getContent(false,false,false)),'<br><b><i><u><em><small><strong><big>');

+					$content = strip_tags(str_replace('<p>', '<br>&nbsp;&nbsp;&nbsp;', $oDocument->getContent(false, false, false)), '<br><b><i><u><em><small><strong><big>');

 

 

 					// setup content information(include the comments link)

-					$content = Context::getLang('replies').' : <a href="'.getUrl('act','dispBoardContentCommentList').'">'.$oDocument->getCommentCount().'</a><br>'."\r\n".$content;

-					$content = '<b>'.$oDocument->getNickName().'</b> ('.$oDocument->getRegdate("Y-m-d").")<br>\r\n".$content;

+					$content = Context::getLang('replies') . ' : <a href="' . getUrl('act', 'dispBoardContentCommentList') . '">' . $oDocument->getCommentCount() . '</a><br>' . "\r\n" . $content;

+					$content = '<b>' . $oDocument->getNickName() . '</b> (' . $oDocument->getRegdate("Y-m-d") . ")<br>\r\n" . $content;

 

 					// setup mobile contents

-					$oMobile->setContent( $content );

+					$oMobile->setContent($content);

 

 					// setup upper URL

-					$oMobile->setUpperUrl( getUrl('document_srl',''), Context::getLang('cmd_list') );

+					$oMobile->setUpperUrl(getUrl('document_srl', ''), Context::getLang('cmd_list'));

 

 				}

 

@@ -92,44 +92,44 @@ 
 		// board index

 		$args = new stdClass;

 		$args->module_srl = $this->module_srl;

-		$args->page = Context::get('page');;

+		$args->page = Context::get('page'); ;

 		$args->list_count = 9;

-		$args->sort_index = $this->module_info->order_target?$this->module_info->order_target:'list_order';

-		$args->order_type = $this->module_info->order_type?$this->module_info->order_type:'asc';

+		$args->sort_index = $this->module_info->order_target ? $this->module_info->order_target : 'list_order';

+		$args->order_type = $this->module_info->order_type ? $this->module_info->order_type : 'asc';

 		$output = $oDocumentModel->getDocumentList($args, $this->except_notice);

 		$document_list = $output->data;

 		$page_navigation = $output->page_navigation;

 

 		$childs = array();

-		if($document_list && count($document_list))

+		if ($document_list && count($document_list))

 		{

-			foreach($document_list as $key => $val)

+			foreach ($document_list as $key => $val)

 			{

-				$href = getUrl('mid',$_GET['mid'],'document_srl',$val->document_srl);

+				$href = getUrl('mid', $_GET['mid'], 'document_srl', $val->document_srl);

 				$obj = null;

 				$obj['href'] = $val->getPermanentUrl();

 

 				$title = htmlspecialchars($val->getTitleText());

-				if($val->getCommentCount()) $title .= ' ['.$val->getCommentCount().']';

-				$obj['link'] = $obj['text'] = '['.$val->getNickName().'] '.$title;

+				if ($val->getCommentCount()) $title .= ' [' . $val->getCommentCount() . ']';

+				$obj['link'] = $obj['text'] = '[' . $val->getNickName() . '] ' . $title;

 				$childs[] = $obj;

 			}

 			$oMobile->setChilds($childs);

 		}

 

 		$totalPage = $page_navigation->last_page;

-		$page = (int)Context::get('page');

-		if(!$page) $page = 1;

+		$page = (int) Context::get('page');

+		if (!$page) $page = 1;

 

 		// next/prevUrl specification

-		if($page > 1)

+		if ($page > 1)

 		{

-			$oMobile->setPrevUrl(getUrl('mid',$_GET['mid'],'page',$page-1), sprintf('%s (%d/%d)', Context::getLang('cmd_prev'), $page-1, $totalPage));

+			$oMobile->setPrevUrl(getUrl('mid', $_GET['mid'], 'page', $page - 1), sprintf('%s (%d/%d)', Context::getLang('cmd_prev'), $page - 1, $totalPage));

 		}

 

-		if($page < $totalPage)

+		if ($page < $totalPage)

 		{

-			$oMobile->setNextUrl(getUrl('mid',$_GET['mid'],'page',$page+1), sprintf('%s (%d/%d)', Context::getLang('cmd_next'), $page+1, $totalPage));

+			$oMobile->setNextUrl(getUrl('mid', $_GET['mid'], 'page', $page + 1), sprintf('%s (%d/%d)', Context::getLang('cmd_next'), $page + 1, $totalPage));

 		}

 

 		$oMobile->mobilePage = $page;

Braces +9 added lines, -3 removed lines

@@ -54,7 +54,9 @@ 
 							$oComment = new commentItem();

 							$oComment->setAttribute($val);

 

-							if(!$oComment->isAccessible()) continue;

+							if(!$oComment->isAccessible()) {
+								continue;
+							}
 

 							$content .= "<b>".$oComment->getNickName()."</b> (".$oComment->getRegdate("Y-m-d").")<br>\r\n".$oComment->getContent(false,false)."<br>\r\n";

 						}

@@ -110,7 +112,9 @@ 
 				$obj['href'] = $val->getPermanentUrl();

 

 				$title = htmlspecialchars($val->getTitleText());

-				if($val->getCommentCount()) $title .= ' ['.$val->getCommentCount().']';

+				if($val->getCommentCount()) {
+					$title .= ' ['.$val->getCommentCount().']';
+				}
 				$obj['link'] = $obj['text'] = '['.$val->getNickName().'] '.$title;

 				$childs[] = $obj;

 			}

@@ -119,7 +123,9 @@ 
 

 		$totalPage = $page_navigation->last_page;

 		$page = (int)Context::get('page');

-		if(!$page) $page = 1;

+		if(!$page) {
+			$page = 1;
+		}
 

 		// next/prevUrl specification

 		if($page > 1)

modules/comment/comment.admin.view.php

Spacing +7 added lines, -7 removed lines

@@ -52,7 +52,7 @@ 
 		// get a list by using comment->getCommentList. 
 		$oCommentModel = getModel('comment');
 		$secretNameList = $oCommentModel->getSecretNameList();
-		$columnList = array('comment_srl', 'document_srl','module_srl','is_secret', 'status', 'content', 'comments.member_srl', 'comments.nick_name', 'comments.regdate', 'ipaddress', 'voted_count', 'blamed_count');
+		$columnList = array('comment_srl', 'document_srl', 'module_srl', 'is_secret', 'status', 'content', 'comments.member_srl', 'comments.nick_name', 'comments.regdate', 'ipaddress', 'voted_count', 'blamed_count');
 		$output = $oCommentModel->getTotalCommentList($args, $columnList);
 
 		// $modules = $oCommentModel->getDistinctModules();
@@ -70,20 +70,20 @@ 
 		$oModuleModel = getModel('module');
 		$module_list = array();
 		$mod_srls = array();
-		foreach($output->data as $val)
+		foreach ($output->data as $val)
 		{
 			$mod_srls[] = $val->module_srl;
 		}
 		$mod_srls = array_unique($mod_srls);
 		// Module List
 		$mod_srls_count = count($mod_srls);
-		if($mod_srls_count)
+		if ($mod_srls_count)
 		{
 			$columnList = array('module_srl', 'mid', 'browser_title');
 			$module_output = $oModuleModel->getModulesInfo($mod_srls, $columnList);
-			if($module_output && is_array($module_output))
+			if ($module_output && is_array($module_output))
 			{
-				foreach($module_output as $module)
+				foreach ($module_output as $module)
 				{
 					$module_list[$module->module_srl] = $module;
 				}
@@ -114,11 +114,11 @@ 
 		$declared_output = executeQuery('comment.getDeclaredList', $args);
 		$oCommentModel = getModel('comment');
 
-		if($declared_output->data && count($declared_output->data))
+		if ($declared_output->data && count($declared_output->data))
 		{
 			$comment_list = array();
 
-			foreach($declared_output->data as $key => $comment)
+			foreach ($declared_output->data as $key => $comment)
 			{
 				$comment_list[$key] = new commentItem();
 				$comment_list[$key]->setAttribute($comment);

modules/comment/comment.class.php

Spacing +22 added lines, -22 removed lines

@@ -48,55 +48,55 @@ 
 		$oDB = DB::getInstance();
 		$oModuleModel = getModel('module');
 		// 2007. 10. 17 add a trigger to delete comments together with posting deleted
-		if(!$oModuleModel->getTrigger('document.deleteDocument', 'comment', 'controller', 'triggerDeleteDocumentComments', 'after'))
+		if (!$oModuleModel->getTrigger('document.deleteDocument', 'comment', 'controller', 'triggerDeleteDocumentComments', 'after'))
 		{
 			return TRUE;
 		}
 		// 2007. 10. 17 add a trigger to delete all of comments together with module deleted
-		if(!$oModuleModel->getTrigger('module.deleteModule', 'comment', 'controller', 'triggerDeleteModuleComments', 'after'))
+		if (!$oModuleModel->getTrigger('module.deleteModule', 'comment', 'controller', 'triggerDeleteModuleComments', 'after'))
 		{
 			return TRUE;
 		}
 		// 2007. 10. 23 add a column for recommendation votes or notification of the comments
-		if(!$oDB->isColumnExists("comments", "voted_count"))
+		if (!$oDB->isColumnExists("comments", "voted_count"))
 		{
 			return TRUE;
 		}
-		if(!$oDB->isColumnExists("comments", "notify_message"))
+		if (!$oDB->isColumnExists("comments", "notify_message"))
 		{
 			return TRUE;
 		}
 		// 2008. 02. 22 add comment setting when a new module added
-		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'comment', 'view', 'triggerDispCommentAdditionSetup', 'before'))
+		if (!$oModuleModel->getTrigger('module.dispAdditionSetup', 'comment', 'view', 'triggerDispCommentAdditionSetup', 'before'))
 		{
 			return TRUE;
 		}
 		// 2008. 05. 14 add a column for blamed count
-		if(!$oDB->isColumnExists("comments", "blamed_count"))
+		if (!$oDB->isColumnExists("comments", "blamed_count"))
 		{
 			return TRUE;
 		}
-		if(!$oDB->isColumnExists("comment_voted_log", "point"))
+		if (!$oDB->isColumnExists("comment_voted_log", "point"))
 		{
 			return TRUE;
 		}
 
-		if(!$oDB->isIndexExists("comments", "idx_module_list_order"))
+		if (!$oDB->isIndexExists("comments", "idx_module_list_order"))
 		{
 			return TRUE;
 		}
 		//2012. 02. 24 add comment published status column and index
-		if(!$oDB->isColumnExists("comments", "status"))
+		if (!$oDB->isColumnExists("comments", "status"))
 		{
 			return TRUE;
 		}
-		if(!$oDB->isIndexExists("comments", "idx_status"))
+		if (!$oDB->isIndexExists("comments", "idx_status"))
 		{
 			return TRUE;
 		}
 
 		// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-		if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'comment', 'controller', 'triggerCopyModule', 'after'))
+		if (!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'comment', 'controller', 'triggerCopyModule', 'after'))
 		{
 			return TRUE;
 		}
@@ -114,43 +114,43 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		// 2007. 10. 17 add a trigger to delete comments together with posting deleted
-		if(!$oModuleModel->getTrigger('document.deleteDocument', 'comment', 'controller', 'triggerDeleteDocumentComments', 'after'))
+		if (!$oModuleModel->getTrigger('document.deleteDocument', 'comment', 'controller', 'triggerDeleteDocumentComments', 'after'))
 		{
 			$oModuleController->insertTrigger('document.deleteDocument', 'comment', 'controller', 'triggerDeleteDocumentComments', 'after');
 		}
 		// 2007. 10. 17 add a trigger to delete all of comments together with module deleted
-		if(!$oModuleModel->getTrigger('module.deleteModule', 'comment', 'controller', 'triggerDeleteModuleComments', 'after'))
+		if (!$oModuleModel->getTrigger('module.deleteModule', 'comment', 'controller', 'triggerDeleteModuleComments', 'after'))
 		{
 			$oModuleController->insertTrigger('module.deleteModule', 'comment', 'controller', 'triggerDeleteModuleComments', 'after');
 		}
 		// 2007. 10. 23 add a column for recommendation votes or notification of the comments
-		if(!$oDB->isColumnExists("comments", "voted_count"))
+		if (!$oDB->isColumnExists("comments", "voted_count"))
 		{
 			$oDB->addColumn("comments", "voted_count", "number", "11");
 			$oDB->addIndex("comments", "idx_voted_count", array("voted_count"));
 		}
 
-		if(!$oDB->isColumnExists("comments", "notify_message"))
+		if (!$oDB->isColumnExists("comments", "notify_message"))
 		{
 			$oDB->addColumn("comments", "notify_message", "char", "1");
 		}
 		// 2008. 02. 22 add comment setting when a new module added
-		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'comment', 'view', 'triggerDispCommentAdditionSetup', 'before'))
+		if (!$oModuleModel->getTrigger('module.dispAdditionSetup', 'comment', 'view', 'triggerDispCommentAdditionSetup', 'before'))
 		{
 			$oModuleController->insertTrigger('module.dispAdditionSetup', 'comment', 'view', 'triggerDispCommentAdditionSetup', 'before');
 		}
 		// 2008. 05. 14 add a column for blamed count
-		if(!$oDB->isColumnExists("comments", "blamed_count"))
+		if (!$oDB->isColumnExists("comments", "blamed_count"))
 		{
 			$oDB->addColumn('comments', 'blamed_count', 'number', 11, 0, TRUE);
 			$oDB->addIndex('comments', 'idx_blamed_count', array('blamed_count'));
 		}
-		if(!$oDB->isColumnExists("comment_voted_log", "point"))
+		if (!$oDB->isColumnExists("comment_voted_log", "point"))
 		{
 			$oDB->addColumn('comment_voted_log', 'point', 'number', 11, 0, TRUE);
 		}
 
-		if(!$oDB->isIndexExists("comments", "idx_module_list_order"))
+		if (!$oDB->isIndexExists("comments", "idx_module_list_order"))
 		{
 			$oDB->addIndex(
 					"comments", "idx_module_list_order", array("module_srl", "list_order"), TRUE
@@ -158,11 +158,11 @@ 
 		}
 
 		//2012. 02. 24 add comment published status column and index
-		if(!$oDB->isColumnExists("comments", "status"))
+		if (!$oDB->isColumnExists("comments", "status"))
 		{
 			$oDB->addColumn("comments", "status", "number", 1, 1, TRUE);
 		}
-		if(!$oDB->isIndexExists("comments", "idx_status"))
+		if (!$oDB->isIndexExists("comments", "idx_status"))
 		{
 			$oDB->addIndex(
 					"comments", "idx_status", array("status", "comment_srl", "module_srl", "document_srl"), TRUE
@@ -170,7 +170,7 @@ 
 		}
 
 		// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-		if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'comment', 'controller', 'triggerCopyModule', 'after'))
+		if (!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'comment', 'controller', 'triggerCopyModule', 'after'))
 		{
 			$oModuleController->insertTrigger('module.procModuleAdminCopyModule', 'comment', 'controller', 'triggerCopyModule', 'after');
 		}