xpressengine / xe-core

modules/page/page.admin.controller.php

Indentation +4 added lines, -4 removed lines

@@ -1,10 +1,10 @@
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 /**
- * @class  pageAdminController
- * @author NAVER (developers@xpressengine.com)
- * @brief page of the module admin controller class
- */
+	 * @class  pageAdminController
+	 * @author NAVER (developers@xpressengine.com)
+	 * @brief page of the module admin controller class
+	 */
 class pageAdminController extends page
 {
 	/**

Spacing +37 added lines, -37 removed lines

@@ -25,24 +25,24 @@ 
 		// Set board module
 		$args = Context::getRequestVars();
 		$args->module = 'page';
-		$args->mid = $args->page_name;	//because if mid is empty in context, set start page mid
+		$args->mid = $args->page_name; //because if mid is empty in context, set start page mid
 		$args->path = (!$args->path) ? '' : $args->path;
 		$args->mpath = (!$args->mpath) ? '' : $args->mpath;
 		unset($args->page_name);
 
-		if($args->use_mobile != 'Y') $args->use_mobile = '';
+		if ($args->use_mobile != 'Y') $args->use_mobile = '';
 		// Check if an original module exists by using module_srl
-		if($args->module_srl)
+		if ($args->module_srl)
 		{
 			$columnList = array('module_srl');
 			$module_info = $oModuleModel->getModuleInfoByModuleSrl($args->module_srl, $columnList);
-			if($module_info->module_srl != $args->module_srl)
+			if ($module_info->module_srl != $args->module_srl)
 			{
 				unset($args->module_srl);
 			}
 			else
 			{
-				foreach($args as $key=>$val)
+				foreach ($args as $key=>$val)
 				{
 					$module_info->{$key} = $val;
 				}
@@ -75,7 +75,7 @@ 
 				}
 		}
 		// Insert/update depending on module_srl
-		if(!$args->module_srl)
+		if (!$args->module_srl)
 		{
 			$output = $oModuleController->insertModule($args);
 			$msg_code = 'success_registed';
@@ -86,10 +86,10 @@ 
 			$msg_code = 'success_updated';
 		}
 
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$this->add("page", Context::get('page'));
-		$this->add('module_srl',$output->get('module_srl'));
+		$this->add('module_srl', $output->get('module_srl'));
 		$this->setMessage($msg_code);
 
 		$returnUrl = Context::get('success_return_url') ? Context::get('success_return_url') : getNotEncodedUrl('', 'module', 'admin', 'module_srl', $output->get('module_srl'), 'act', 'dispPageAdminInfo');
@@ -106,16 +106,16 @@ 
 
 	function putDocumentsInPageToArray($target, &$array)
 	{
-		if(!$target) return;
+		if (!$target) return;
 		preg_match_all('!<img hasContent="true" ([^>]+)!is', $target, $matches);
 		$pattern = '!document_srl="(\d+)"!';
-		foreach($matches[1] as $match)
+		foreach ($matches[1] as $match)
 		{
 			$match2 = null;
 			preg_match($pattern, $match, $match2);
-			if(count($match2))
+			if (count($match2))
 			{
-				$array[(int)$match2[1]] = 1;
+				$array[(int) $match2[1]] = 1;
 			}
 		}
 	}
@@ -127,20 +127,20 @@ 
 	{
 		$module_srl = Context::get('module_srl');
 		$content = Context::get('content');
-		if(!$module_srl) return new Object(-1,'msg_invalid_request');
+		if (!$module_srl) return new Object(-1, 'msg_invalid_request');
 		$mcontent = Context::get('mcontent');
 		$type = Context::get('type');
 		// Guhaeom won information page
 		$oModuleModel = getModel('module');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl);
-		if($type == "mobile")
+		if ($type == "mobile")
 		{
-			if(!$mcontent) $mcontent = '';
+			if (!$mcontent) $mcontent = '';
 			$module_info->mcontent = $mcontent;
 		}
 		else
 		{
-			if(!isset($content)) $content ='';
+			if (!isset($content)) $content = '';
 			$module_info->content = $content;
 		}
 
@@ -154,11 +154,11 @@ 
 		$obj->module_srl = $module_srl;
 		$obj->list_count = 99999999;
 		$output = $oDocumentModel->getDocumentList($obj);
-		if(count($output->data))
+		if (count($output->data))
 		{
-			foreach($output->data as $document)
+			foreach ($output->data as $document)
 			{
-				if($document_srls[$document->document_srl]) continue;
+				if ($document_srls[$document->document_srl]) continue;
 				$oDocumentController->deleteDocument($document->document_srl, true);
 			}
 		}
@@ -166,7 +166,7 @@ 
 		$oModuleController = getController('module');
 		// Save
 		$output = $oModuleController->updateModule($module_info);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 		// On the page, change the validity status of the attached file
 		$oFileController = getController('file');
 		$oFileController->setFilesValid($module_info->module_srl);
@@ -189,10 +189,10 @@ 
 		// Get an original
 		$oModuleController = getController('module');
 		$output = $oModuleController->deleteModule($module_srl);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		$this->add('module','page');
-		$this->add('page',Context::get('page'));
+		$this->add('module', 'page');
+		$this->add('page', Context::get('page'));
 		$this->setMessage('success_deleted');
 
 		$returnUrl = Context::get('success_return_url') ? Context::get('success_return_url') : getNotEncodedUrl('', 'module', 'admin', 'act', 'dispPageAdminContent');
@@ -208,7 +208,7 @@ 
 		$args = Context::getRequestVars();
 		// Insert by creating the module Controller object
 		$oModuleController = getController('module');
-		$output = $oModuleController->insertModuleConfig('page',$args);
+		$output = $oModuleController->insertModuleConfig('page', $args);
 		return $output;
 	}
 
@@ -239,7 +239,7 @@ 
 		$file_srl = Context::get('file_srl');
 		// Create the controller object file class
 		$oFileController = getController('file');
-		if($file_srl) $output = $oFileController->deleteFile($file_srl, $this->grant->manager);
+		if ($file_srl) $output = $oFileController->deleteFile($file_srl, $this->grant->manager);
 		// Attachment to the output of the list, java script
 		$oFileController->printUploadedFileList($upload_target_srl);
 	}
@@ -251,7 +251,7 @@ 
 	{
 		$module_srl = Context::get('module_srl');
 
-		if(!$module_srl) return;
+		if (!$module_srl) return;
 
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'content');
@@ -263,27 +263,27 @@ 
 		$oWidgetController = getController('widget');
 		$oWidgetController->recompileWidget($content);
 
-		if($module_info->page_type == 'WIDGET')
+		if ($module_info->page_type == 'WIDGET')
 		{
 			$path = _XE_PATH_ . 'files/cache/page/';
 			$cache_files = FileHandler::readDir($path, '/^' . $module_info->module_srl . '\./');
-			foreach($cache_files as $file_name)
+			foreach ($cache_files as $file_name)
 			{
 				FileHandler::removeFile($path . $module_info->module_srl . '.' . $file_name);
 			}
 		}
-		else if($module_info->page_type == 'OUTSIDE')
+		else if ($module_info->page_type == 'OUTSIDE')
 		{
 			$path = _XE_PATH_ . 'files/cache/page/';
 			$cache_files = FileHandler::readDir($path, '/^' . $module_info->module_srl . './');
-			foreach($cache_files as $file_name)
+			foreach ($cache_files as $file_name)
 			{
 				FileHandler::removeFile($path . $module_info->module_srl . '.' . $file_name);
 			}
 		}
 
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$object_key = 'mid_info:' . $module_info->module_srl;
 			$cache_key = $oCacheHandler->getGroupKey('site_and_module', $object_key);
@@ -301,7 +301,7 @@ 
 		$oModuleModel = getModel('module');
 		$grant = $oModuleModel->getGrant($this->module_info, $logged_info);
 
-		if(!$grant->manager)
+		if (!$grant->manager)
 		{
 			return new Object(-1, 'msg_not_permitted');
 		}
@@ -311,9 +311,9 @@ 
 		$obj->is_notice = 'N';
 
 		settype($obj->title, "string");
-		if($obj->title == '') $obj->title = cut_str(strip_tags($obj->content),20,'...');
+		if ($obj->title == '') $obj->title = cut_str(strip_tags($obj->content), 20, '...');
 		//그래도 없으면 Untitled
-		if($obj->title == '') $obj->title = 'Untitled';
+		if ($obj->title == '') $obj->title = 'Untitled';
 
 		$document_srl = $obj->document_srl;
 
@@ -324,7 +324,7 @@ 
 		$target = ($obj->ismobile == 'Y') ? 'mdocument_srl' : 'document_srl';
 
 		// 이미 존재하는 경우 수정
-		if($oDocument->isExists() && $oDocument->document_srl == $obj->document_srl) 
+		if ($oDocument->isExists() && $oDocument->document_srl == $obj->document_srl) 
 		{
 			$output = $oDocumentController->updateDocument($oDocument, $obj);
 			$msg_code = 'success_updated';
@@ -337,7 +337,7 @@ 
 			$document_srl = $output->get('document_srl');
 		}
 
-		if(!isset($this->module_info->{$target}) || (isset($this->module_info->{$target}) && $this->module_info->{$target} !== $document_srl))
+		if (!isset($this->module_info->{$target}) || (isset($this->module_info->{$target}) && $this->module_info->{$target} !== $document_srl))
 		{
 			$oModuleController = getController('module');
 			$this->module_info->{$target} = $document_srl;
@@ -345,7 +345,7 @@ 
 		}
 
 		// 오류 발생시 멈춤
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		// 결과를 리턴
 		$this->add('mid', Context::get('mid'));

Braces +47 added lines, -24 removed lines

@@ -30,7 +30,9 @@ 
 		$args->mpath = (!$args->mpath) ? '' : $args->mpath;
 		unset($args->page_name);
 
-		if($args->use_mobile != 'Y') $args->use_mobile = '';
+		if($args->use_mobile != 'Y') {
+			$args->use_mobile = '';
+		}
 		// Check if an original module exists by using module_srl
 		if($args->module_srl)
 		{
@@ -39,8 +41,7 @@ 
 			if($module_info->module_srl != $args->module_srl)
 			{
 				unset($args->module_srl);
-			}
-			else
+			} else
 			{
 				foreach($args as $key=>$val)
 				{
@@ -79,14 +80,15 @@ 
 		{
 			$output = $oModuleController->insertModule($args);
 			$msg_code = 'success_registed';
-		}
-		else
+		} else
 		{
 			$output = $oModuleController->updateModule($args);
 			$msg_code = 'success_updated';
 		}
 
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->add("page", Context::get('page'));
 		$this->add('module_srl',$output->get('module_srl'));
@@ -106,7 +108,9 @@ 
 
 	function putDocumentsInPageToArray($target, &$array)
 	{
-		if(!$target) return;
+		if(!$target) {
+			return;
+		}
 		preg_match_all('!<img hasContent="true" ([^>]+)!is', $target, $matches);
 		$pattern = '!document_srl="(\d+)"!';
 		foreach($matches[1] as $match)
@@ -127,7 +131,9 @@ 
 	{
 		$module_srl = Context::get('module_srl');
 		$content = Context::get('content');
-		if(!$module_srl) return new Object(-1,'msg_invalid_request');
+		if(!$module_srl) {
+			return new Object(-1,'msg_invalid_request');
+		}
 		$mcontent = Context::get('mcontent');
 		$type = Context::get('type');
 		// Guhaeom won information page
@@ -135,12 +141,15 @@ 
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl);
 		if($type == "mobile")
 		{
-			if(!$mcontent) $mcontent = '';
+			if(!$mcontent) {
+				$mcontent = '';
+			}
 			$module_info->mcontent = $mcontent;
-		}
-		else
+		} else
 		{
-			if(!isset($content)) $content ='';
+			if(!isset($content)) {
+				$content ='';
+			}
 			$module_info->content = $content;
 		}
 
@@ -158,7 +167,9 @@ 
 		{
 			foreach($output->data as $document)
 			{
-				if($document_srls[$document->document_srl]) continue;
+				if($document_srls[$document->document_srl]) {
+					continue;
+				}
 				$oDocumentController->deleteDocument($document->document_srl, true);
 			}
 		}
@@ -166,7 +177,9 @@ 
 		$oModuleController = getController('module');
 		// Save
 		$output = $oModuleController->updateModule($module_info);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 		// On the page, change the validity status of the attached file
 		$oFileController = getController('file');
 		$oFileController->setFilesValid($module_info->module_srl);
@@ -189,7 +202,9 @@ 
 		// Get an original
 		$oModuleController = getController('module');
 		$output = $oModuleController->deleteModule($module_srl);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$this->add('module','page');
 		$this->add('page',Context::get('page'));
@@ -239,7 +254,9 @@ 
 		$file_srl = Context::get('file_srl');
 		// Create the controller object file class
 		$oFileController = getController('file');
-		if($file_srl) $output = $oFileController->deleteFile($file_srl, $this->grant->manager);
+		if($file_srl) {
+			$output = $oFileController->deleteFile($file_srl, $this->grant->manager);
+		}
 		// Attachment to the output of the list, java script
 		$oFileController->printUploadedFileList($upload_target_srl);
 	}
@@ -251,7 +268,9 @@ 
 	{
 		$module_srl = Context::get('module_srl');
 
-		if(!$module_srl) return;
+		if(!$module_srl) {
+			return;
+		}
 
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'content');
@@ -271,8 +290,7 @@ 
 			{
 				FileHandler::removeFile($path . $module_info->module_srl . '.' . $file_name);
 			}
-		}
-		else if($module_info->page_type == 'OUTSIDE')
+		} else if($module_info->page_type == 'OUTSIDE')
 		{
 			$path = _XE_PATH_ . 'files/cache/page/';
 			$cache_files = FileHandler::readDir($path, '/^' . $module_info->module_srl . './');
@@ -311,9 +329,13 @@ 
 		$obj->is_notice = 'N';
 
 		settype($obj->title, "string");
-		if($obj->title == '') $obj->title = cut_str(strip_tags($obj->content),20,'...');
+		if($obj->title == '') {
+			$obj->title = cut_str(strip_tags($obj->content),20,'...');
+		}
 		//그래도 없으면 Untitled
-		if($obj->title == '') $obj->title = 'Untitled';
+		if($obj->title == '') {
+			$obj->title = 'Untitled';
+		}
 
 		$document_srl = $obj->document_srl;
 
@@ -328,8 +350,7 @@ 
 		{
 			$output = $oDocumentController->updateDocument($oDocument, $obj);
 			$msg_code = 'success_updated';
-		}
-		else
+		} else
 		{
 			// 그렇지 않으면 신규 등록
 			$output = $oDocumentController->insertDocument($obj, $bAnonymous);
@@ -345,7 +366,9 @@ 
 		}
 
 		// 오류 발생시 멈춤
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		// 결과를 리턴
 		$this->add('mid', Context::get('mid'));

modules/page/page.admin.view.php

Indentation +4 added lines, -4 removed lines

@@ -1,10 +1,10 @@
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 /**
- * @class  pageAdminView
- * @author NAVER (developers@xpressengine.com)
- * @brief page admin view of the module class
- */
+	 * @class  pageAdminView
+	 * @author NAVER (developers@xpressengine.com)
+	 * @brief page admin view of the module class
+	 */
 class pageAdminView extends page
 {
 	var $module_srl = 0;

Spacing +27 added lines, -27 removed lines

@@ -21,19 +21,19 @@ 
 		// Create module model object
 		$oModuleModel = getModel('module');
 		// module_srl two come over to save the module, putting the information in advance
-		if($module_srl)
+		if ($module_srl)
 		{
 			$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl);
-			if(!$module_info)
+			if (!$module_info)
 			{
-				Context::set('module_srl','');
+				Context::set('module_srl', '');
 				$this->act = 'list';
 			}
 			else
 			{
 				ModuleModel::syncModuleToSite($module_info);
 				$this->module_info = $module_info;
-				Context::set('module_info',$module_info);
+				Context::set('module_info', $module_info);
 			}
 		}
 		// Get a list of module categories
@@ -44,7 +44,7 @@ 
 		$security->encodeHTML('module_category..title');
 
 		// Get a template path (page in the administrative template tpl putting together)
-		$this->setTemplatePath($this->module_path.'tpl');
+		$this->setTemplatePath($this->module_path . 'tpl');
 	}
 
 	/**
@@ -59,10 +59,10 @@ 
 		$args->page_count = 10;
 		$args->s_module_category_srl = Context::get('module_category_srl');
 
-		$search_target_list = array('s_mid','s_browser_title');
+		$search_target_list = array('s_mid', 's_browser_title');
 		$search_target = Context::get('search_target');
 		$search_keyword = Context::get('search_keyword');
-		if(in_array($search_target,$search_target_list) && $search_keyword) $args->{$search_target} = $search_keyword;
+		if (in_array($search_target, $search_target_list) && $search_keyword) $args->{$search_target} = $search_keyword;
 
 		$output = executeQuery('page.getPageList', $args);
 		$oModuleModel = getModel('module');
@@ -101,9 +101,9 @@ 
 		$module_srl = Context::get('module_srl');
 		$module_info = Context::get('module_info');
 		// If you do not value module_srl just showing the index page
-		if(!$module_srl) return $this->dispPageAdminContent();
+		if (!$module_srl) return $this->dispPageAdminContent();
 		// If the layout is destined to add layout information haejum (layout_title, layout)
-		if($module_info->layout_srl)
+		if ($module_info->layout_srl)
 		{
 			$oLayoutModel = getModel('layout');
 			$layout_info = $oLayoutModel->getLayout($module_info->layout_srl);
@@ -115,15 +115,15 @@ 
 		$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);
 		// Set a template file
 
-		if($this->module_info->page_type == 'ARTICLE')
+		if ($this->module_info->page_type == 'ARTICLE')
 		{
 			$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);
@@ -161,23 +161,23 @@ 
 
 	function dispPageAdminMobileContent()
 	{
-		if($this->module_info->page_type == 'OUTSIDE')
+		if ($this->module_info->page_type == 'OUTSIDE')
 		{
 			return $this->stop(-1, 'msg_invalid_request');
 		}
 
-		if($this->module_srl)
+		if ($this->module_srl)
 		{
-			Context::set('module_srl',$this->module_srl);
+			Context::set('module_srl', $this->module_srl);
 		}
 
 		$oPageMobile = &getMobile('page');
 		$oPageMobile->module_info = $this->module_info;
 		$page_type_name = strtolower($this->module_info->page_type);
 		$method = '_get' . ucfirst($page_type_name) . 'Content';
-		if(method_exists($oPageMobile, $method))
+		if (method_exists($oPageMobile, $method))
 		{
-			if($method == '_getArticleContent' && $this->module_info->is_mskin_fix == 'N')
+			if ($method == '_getArticleContent' && $this->module_info->is_mskin_fix == 'N')
 			{
 				$oModuleModel = getModel('module');
 				$oPageMobile->module_info->mskin = $oModuleModel->getModuleDefaultSkin('page', 'M');
@@ -230,16 +230,16 @@ 
 	function _setWidgetTypeContentModify($isMobile = false)
 	{
 		// Setting contents
-		if($isMobile)
+		if ($isMobile)
 		{
 			$content = Context::get('mcontent');
-			if(!$content) $content = $this->module_info->mcontent;
+			if (!$content) $content = $this->module_info->mcontent;
 			$templateFile = 'page_mobile_content_modify';
 		}
 		else
 		{
 			$content = Context::get('content');
-			if(!$content) $content = $this->module_info->content;
+			if (!$content) $content = $this->module_info->content;
 			$templateFile = 'page_content_modify';
 		}
 
@@ -256,7 +256,7 @@ 
 
 		//Security
 		$security = new Security();
-		$security->encodeHTML('widget_list..title','module_info.mid');
+		$security->encodeHTML('widget_list..title', 'module_info.mid');
 
 		// Set a template file
 		$this->setTemplateFile($templateFile);
@@ -267,7 +267,7 @@ 
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument(0, true);
 
-		if($isMobile)
+		if ($isMobile)
 		{
 			Context::set('isMobile', 'Y');
 			$target = 'mdocument_srl';
@@ -278,13 +278,13 @@ 
 			$target = 'document_srl';
 		}
 
-		if($this->module_info->{$target})
+		if ($this->module_info->{$target})
 		{
 			$document_srl = $this->module_info->{$target};
 			$oDocument->setDocument($document_srl);
 			Context::set('document_srl', $document_srl);
 		} 
-		else if(Context::get('document_srl'))
+		else if (Context::get('document_srl'))
 		{
 			$document_srl = Context::get('document_srl');
 			$oDocument->setDocument($document_srl);
@@ -295,7 +295,7 @@ 
 			$oDocument->add('module_srl', $this->module_info->module_srl);
 		}
 
-		Context::addJsFilter($this->module_path.'tpl/filter', 'insert_article.xml');
+		Context::addJsFilter($this->module_path . 'tpl/filter', 'insert_article.xml');
 		Context::set('oDocument', $oDocument);
 		Context::set('mid', $this->module_info->mid);
 		$this->setTemplateFile('article_content_modify');
@@ -307,12 +307,12 @@ 
 	function dispPageAdminDelete()
 	{
 		$module_srl = Context::get('module_srl');
-		if(!$module_srl) return $this->dispContent();
+		if (!$module_srl) return $this->dispContent();
 
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'module', 'mid');
 		$module_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl, $columnList);
-		Context::set('module_info',$module_info);
+		Context::set('module_info', $module_info);
 		// Set a template file
 		$this->setTemplateFile('page_delete');
 

Braces +23 added lines, -21 removed lines

@@ -28,8 +28,7 @@ 
 			{
 				Context::set('module_srl','');
 				$this->act = 'list';
-			}
-			else
+			} else
 			{
 				ModuleModel::syncModuleToSite($module_info);
 				$this->module_info = $module_info;
@@ -62,7 +61,9 @@ 
 		$search_target_list = array('s_mid','s_browser_title');
 		$search_target = Context::get('search_target');
 		$search_keyword = Context::get('search_keyword');
-		if(in_array($search_target,$search_target_list) && $search_keyword) $args->{$search_target} = $search_keyword;
+		if(in_array($search_target,$search_target_list) && $search_keyword) {
+			$args->{$search_target} = $search_keyword;
+		}
 
 		$output = executeQuery('page.getPageList', $args);
 		$oModuleModel = getModel('module');
@@ -101,7 +102,9 @@ 
 		$module_srl = Context::get('module_srl');
 		$module_info = Context::get('module_info');
 		// If you do not value module_srl just showing the index page
-		if(!$module_srl) return $this->dispPageAdminContent();
+		if(!$module_srl) {
+			return $this->dispPageAdminContent();
+		}
 		// If the layout is destined to add layout information haejum (layout_title, layout)
 		if($module_info->layout_srl)
 		{
@@ -183,8 +186,7 @@ 
 				$oPageMobile->module_info->mskin = $oModuleModel->getModuleDefaultSkin('page', 'M');
 			}
 			$page_content = $oPageMobile->{$method}();
-		}
-		else
+		} else
 		{
 			return new Object(-1, sprintf('%s method is not exists', $method));
 		}
@@ -202,8 +204,7 @@ 
 		if ($this->module_info->page_type == 'WIDGET')
 		{
 			$this->_setWidgetTypeContentModify(true);
-		}
-		else if ($this->module_info->page_type == 'ARTICLE')
+		} else if ($this->module_info->page_type == 'ARTICLE')
 		{
 			$this->_setArticleTypeContentModify(true);
 		}
@@ -220,8 +221,7 @@ 
 		if ($this->module_info->page_type == 'WIDGET')
 		{
 			$this->_setWidgetTypeContentModify();
-		}
-		else if ($this->module_info->page_type == 'ARTICLE')
+		} else if ($this->module_info->page_type == 'ARTICLE')
 		{
 			$this->_setArticleTypeContentModify();
 		}
@@ -233,13 +233,16 @@ 
 		if($isMobile)
 		{
 			$content = Context::get('mcontent');
-			if(!$content) $content = $this->module_info->mcontent;
+			if(!$content) {
+				$content = $this->module_info->mcontent;
+			}
 			$templateFile = 'page_mobile_content_modify';
-		}
-		else
+		} else
 		{
 			$content = Context::get('content');
-			if(!$content) $content = $this->module_info->content;
+			if(!$content) {
+				$content = $this->module_info->content;
+			}
 			$templateFile = 'page_content_modify';
 		}
 
@@ -271,8 +274,7 @@ 
 		{
 			Context::set('isMobile', 'Y');
 			$target = 'mdocument_srl';
-		}
-		else
+		} else
 		{
 			Context::set('isMobile', 'N');
 			$target = 'document_srl';
@@ -283,14 +285,12 @@ 
 			$document_srl = $this->module_info->{$target};
 			$oDocument->setDocument($document_srl);
 			Context::set('document_srl', $document_srl);
-		} 
-		else if(Context::get('document_srl'))
+		} else if(Context::get('document_srl'))
 		{
 			$document_srl = Context::get('document_srl');
 			$oDocument->setDocument($document_srl);
 			Context::set('document_srl', $document_srl);
-		}
-		else
+		} else
 		{
 			$oDocument->add('module_srl', $this->module_info->module_srl);
 		}
@@ -307,7 +307,9 @@ 
 	function dispPageAdminDelete()
 	{
 		$module_srl = Context::get('module_srl');
-		if(!$module_srl) return $this->dispContent();
+		if(!$module_srl) {
+			return $this->dispContent();
+		}
 
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'module', 'mid');

modules/page/page.api.php

Indentation +4 added lines, -4 removed lines

@@ -1,10 +1,10 @@
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 /**
- * @class  pageAPI
- * @author NAVER (developers@xpressengine.com)
- * @brief View Action page for the module API processing
- */
+	 * @class  pageAPI
+	 * @author NAVER (developers@xpressengine.com)
+	 * @brief View Action page for the module API processing
+	 */
 class pageAPI extends page
 {
 	/**

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 		$oWidgetController->triggerWidgetCompile($page_content);
 		Context::setResponseMethod($requestMethod);
 
-		$oModule->add('page_content',$page_content);
+		$oModule->add('page_content', $page_content);
 	}
 }
 /* End of file page.api.php */

modules/page/page.class.php

Indentation +4 added lines, -4 removed lines

@@ -1,10 +1,10 @@
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 /**
- * @class  page
- * @author NAVER (developers@xpressengine.com)
- * @brief high class of the module page
- */
+	 * @class  page
+	 * @author NAVER (developers@xpressengine.com)
+	 * @brief high class of the module page
+	 */
 class page extends ModuleObject
 {
 	/**

Spacing +12 added lines, -12 removed lines

@@ -24,10 +24,10 @@ 
 	function checkUpdate()
 	{
 		$output = executeQuery('page.pageTypeOpageCheck');
-		if($output->toBool() && $output->data) return true;
+		if ($output->toBool() && $output->data) return true;
 
 		$output = executeQuery('page.pageTypeNullCheck');
-		if($output->toBool() && $output->data) return true;
+		if ($output->toBool() && $output->data) return true;
 
 		return false;
 	}
@@ -40,43 +40,43 @@ 
 		$args = new stdClass;
 		// opage module instance update
 		$output = executeQueryArray('page.pageTypeOpageCheck');
-		if($output->toBool() && count($output->data) > 0)
+		if ($output->toBool() && count($output->data) > 0)
 		{
-			foreach($output->data as $val)
+			foreach ($output->data as $val)
 			{
 				$args->module_srl = $val->module_srl;
 				$args->name = 'page_type';
-				$args->value= 'OUTSIDE';
+				$args->value = 'OUTSIDE';
 				$in_out = executeQuery('page.insertPageType', $args);
 			}
 			$output = executeQuery('page.updateAllOpage');
-			if(!$output->toBool()) return $output;
+			if (!$output->toBool()) return $output;
 		}
 
 		// old page module instance update
 		$output = executeQueryArray('page.pageTypeNullCheck');
 		$skin_update_srls = array();
-		if($output->toBool() && $output->data)
+		if ($output->toBool() && $output->data)
 		{
-			foreach($output->data as $val)
+			foreach ($output->data as $val)
 			{
 				$args->module_srl = $val->module_srl;
 				$args->name = 'page_type';
-				$args->value= 'WIDGET';
+				$args->value = 'WIDGET';
 				$in_out = executeQuery('page.insertPageType', $args);
 
 				$skin_update_srls[] = $val->module_srl;
 			}
 		}
 
-		if(count($skin_update_srls)>0)
+		if (count($skin_update_srls) > 0)
 		{
 			$skin_args = new stdClass;
-			$skin_args->module_srls = implode(',',$skin_update_srls);
+			$skin_args->module_srls = implode(',', $skin_update_srls);
 			$skin_args->is_skin_fix = "Y";
 			$ouput = executeQuery('page.updateSkinFix', $skin_args);
 		}
-		return new Object(0,'success_updated');
+		return new Object(0, 'success_updated');
 	}
 
 	/**

Braces +9 added lines, -3 removed lines

@@ -24,10 +24,14 @@ 
 	function checkUpdate()
 	{
 		$output = executeQuery('page.pageTypeOpageCheck');
-		if($output->toBool() && $output->data) return true;
+		if($output->toBool() && $output->data) {
+			return true;
+		}
 
 		$output = executeQuery('page.pageTypeNullCheck');
-		if($output->toBool() && $output->data) return true;
+		if($output->toBool() && $output->data) {
+			return true;
+		}
 
 		return false;
 	}
@@ -50,7 +54,9 @@ 
 				$in_out = executeQuery('page.insertPageType', $args);
 			}
 			$output = executeQuery('page.updateAllOpage');
-			if(!$output->toBool()) return $output;
+			if(!$output->toBool()) {
+				return $output;
+			}
 		}
 
 		// old page module instance update

modules/page/page.controller.php

Indentation +4 added lines, -4 removed lines

@@ -1,10 +1,10 @@
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 /**
- * @class  documentController
- * @author NAVER (developers@xpressengine.com)
- * @brief controller class of the document module
- */
+	 * @class  documentController
+	 * @author NAVER (developers@xpressengine.com)
+	 * @brief controller class of the document module
+	 */
 class pageController extends page
 {
 	var $target_path = '';

Spacing +23 added lines, -23 removed lines

@@ -29,24 +29,24 @@ 
 	function getHeadScript($content)
 	{
 		// remove the title tag
-		$content = preg_replace('!<title([^>]*)>(.*?)<\/title>!is','', $content);
+		$content = preg_replace('!<title([^>]*)>(.*?)<\/title>!is', '', $content);
 		// Remove meta tags
-		$content = preg_replace('!<(\/){0,1}meta([^>]*)>!is','', $content);
+		$content = preg_replace('!<(\/){0,1}meta([^>]*)>!is', '', $content);
 		// Extract information such as <link, <style, <script and so on
 		preg_match_all('!<link([^>]*)>!is', $content, $link_buff);
-		for($i=0;$i<count($link_buff[0]);$i++)
+		for ($i = 0; $i < count($link_buff[0]); $i++)
 		{
 			$tmp_str = trim($link_buff[0][$i]);
-			if(!$tmp_str) continue;
-			$header_script .=  $tmp_str."\n";
+			if (!$tmp_str) continue;
+			$header_script .= $tmp_str . "\n";
 		}
 
 		preg_match_all('!<(style|script)(.*?)<\/(style|script)>!is', $content, $script_buff);
-		for($i=0;$i<count($script_buff[0]);$i++)
+		for ($i = 0; $i < count($script_buff[0]); $i++)
 		{
 			$tmp_str = trim($script_buff[0][$i]);
-			if(!$tmp_str) continue;
-			$header_script .=  $tmp_str."\n";
+			if (!$tmp_str) continue;
+			$header_script .= $tmp_str . "\n";
 		}
 
 		return $header_script;
@@ -72,22 +72,22 @@ 
 	function replaceSrc($content, $path)
 	{
 		$url_info = parse_url($path);
-		$host = sprintf("%s://%s%s",$url_info['scheme'],$url_info['host'],$url_info['port']?':'.$url_info['port']:'');
-		$this->host = $host.'/';
+		$host = sprintf("%s://%s%s", $url_info['scheme'], $url_info['host'], $url_info['port'] ? ':' . $url_info['port'] : '');
+		$this->host = $host . '/';
 		$path = $url_info['path'];
-		if(substr($path,-1)=='/') $path = substr($path,-1);
-		$t = explode('/',$path);
+		if (substr($path, -1) == '/') $path = substr($path, -1);
+		$t = explode('/', $path);
 		$_t = array();
-		for($i=0,$c=count($t)-1;$i<$c;$i++)
+		for ($i = 0, $c = count($t) - 1; $i < $c; $i++)
 		{
 			$v = trim($t[$i]);
-			if(!$v) continue;
+			if (!$v) continue;
 			$_t[] = $v;
 		}
-		$path = $host.'/'.implode('/',$_t);
-		if(substr($path,-1)!='/') $path .= '/';
+		$path = $host . '/' . implode('/', $_t);
+		if (substr($path, -1) != '/') $path .= '/';
 		$this->path = $path;
-		$content = preg_replace_callback('/(src=|href=|url\()("|\')?([^"\'\)]+)("|\'\))?/is',array($this,'_replacePath'),$content);
+		$content = preg_replace_callback('/(src=|href=|url\()("|\')?([^"\'\)]+)("|\'\))?/is', array($this, '_replacePath'), $content);
 
 		return $content;
 	}
@@ -95,18 +95,18 @@ 
 	function _replacePath($matches)
 	{
 		$val = trim($matches[3]);
-		if(preg_match('/^(http|https|ftp|telnet|mms|mailto)/i',$val)) return $matches[0];
-		if(substr($val,0,2)=='./')
+		if (preg_match('/^(http|https|ftp|telnet|mms|mailto)/i', $val)) return $matches[0];
+		if (substr($val, 0, 2) == './')
 		{
-			$path = $this->path.substr($val,2);
+			$path = $this->path . substr($val, 2);
 		}
-		elseif(substr($val,0,1)=='/')
+		elseif (substr($val, 0, 1) == '/')
 		{
-			$path = $this->host.substr($val,1);
+			$path = $this->host . substr($val, 1);
 		}
 		else
 		{
-			$path = $this->path.$val;
+			$path = $this->path . $val;
 		}
 		return sprintf("%s%s%s%s", $matches[1], $matches[2], $path, $matches[4]);
 	}

Braces +20 added lines, -10 removed lines

@@ -37,7 +37,9 @@ 
 		for($i=0;$i<count($link_buff[0]);$i++)
 		{
 			$tmp_str = trim($link_buff[0][$i]);
-			if(!$tmp_str) continue;
+			if(!$tmp_str) {
+				continue;
+			}
 			$header_script .=  $tmp_str."\n";
 		}
 
@@ -45,7 +47,9 @@ 
 		for($i=0;$i<count($script_buff[0]);$i++)
 		{
 			$tmp_str = trim($script_buff[0][$i]);
-			if(!$tmp_str) continue;
+			if(!$tmp_str) {
+				continue;
+			}
 			$header_script .=  $tmp_str."\n";
 		}
 
@@ -75,17 +79,23 @@ 
 		$host = sprintf("%s://%s%s",$url_info['scheme'],$url_info['host'],$url_info['port']?':'.$url_info['port']:'');
 		$this->host = $host.'/';
 		$path = $url_info['path'];
-		if(substr($path,-1)=='/') $path = substr($path,-1);
+		if(substr($path,-1)=='/') {
+			$path = substr($path,-1);
+		}
 		$t = explode('/',$path);
 		$_t = array();
 		for($i=0,$c=count($t)-1;$i<$c;$i++)
 		{
 			$v = trim($t[$i]);
-			if(!$v) continue;
+			if(!$v) {
+				continue;
+			}
 			$_t[] = $v;
 		}
 		$path = $host.'/'.implode('/',$_t);
-		if(substr($path,-1)!='/') $path .= '/';
+		if(substr($path,-1)!='/') {
+			$path .= '/';
+		}
 		$this->path = $path;
 		$content = preg_replace_callback('/(src=|href=|url\()("|\')?([^"\'\)]+)("|\'\))?/is',array($this,'_replacePath'),$content);
 
@@ -95,16 +105,16 @@ 
 	function _replacePath($matches)
 	{
 		$val = trim($matches[3]);
-		if(preg_match('/^(http|https|ftp|telnet|mms|mailto)/i',$val)) return $matches[0];
+		if(preg_match('/^(http|https|ftp|telnet|mms|mailto)/i',$val)) {
+			return $matches[0];
+		}
 		if(substr($val,0,2)=='./')
 		{
 			$path = $this->path.substr($val,2);
-		}
-		elseif(substr($val,0,1)=='/')
+		} elseif(substr($val,0,1)=='/')
 		{
 			$path = $this->host.substr($val,1);
-		}
-		else
+		} else
 		{
 			$path = $this->path.$val;
 		}

modules/page/page.mobile.php

Spacing +19 added lines, -19 removed lines

@@ -1,26 +1,26 @@ 
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
-require_once(_XE_PATH_.'modules/page/page.view.php');
+require_once(_XE_PATH_ . 'modules/page/page.view.php');
 
 class pageMobile extends pageView
 {
 	function init()
 	{
 		// Get a template path (page in the administrative template tpl putting together)
-		$this->setTemplatePath($this->module_path.'tpl');
+		$this->setTemplatePath($this->module_path . 'tpl');
 
-		switch($this->module_info->page_type)
+		switch ($this->module_info->page_type)
 		{
 			case 'WIDGET' :
 				{
 					$this->cache_file = sprintf("%sfiles/cache/page/%d.%s.%s.m.cache.php", _XE_PATH_, $this->module_info->module_srl, Context::getLangType(), Context::getSslStatus());
-					$this->interval = (int)($this->module_info->page_caching_interval);
+					$this->interval = (int) ($this->module_info->page_caching_interval);
 					break;
 				}
 			case 'OUTSIDE' :
 				{
 					$this->cache_file = sprintf("./files/cache/opage/%d.%s.m.cache.php", $this->module_info->module_srl, Context::getSslStatus()); 
-					$this->interval = (int)($this->module_info->page_caching_interval);
+					$this->interval = (int) ($this->module_info->page_caching_interval);
 					$this->path = $this->module_info->mpath;
 					break;
 				}
@@ -30,7 +30,7 @@ 
 	function dispPageIndex()
 	{
 		// Variables used in the template Context:: set()
-		if($this->module_srl) Context::set('module_srl',$this->module_srl);
+		if ($this->module_srl) Context::set('module_srl', $this->module_srl);
 
 		$page_type_name = strtolower($this->module_info->page_type);
 		$method = '_get' . ucfirst($page_type_name) . 'Content';
@@ -46,13 +46,13 @@ 
 	function _getWidgetContent()
 	{
 		// Arrange a widget ryeolro
-		if($this->module_info->mcontent)
+		if ($this->module_info->mcontent)
 		{
 			$cache_file = sprintf("%sfiles/cache/page/%d.%s.m.cache.php", _XE_PATH_, $this->module_info->module_srl, Context::getLangType());
-			$interval = (int)($this->module_info->page_caching_interval);
-			if($interval>0)
+			$interval = (int) ($this->module_info->page_caching_interval);
+			if ($interval > 0)
 			{
-				if(!file_exists($cache_file) || filesize($cache_file) < 1)
+				if (!file_exists($cache_file) || filesize($cache_file) < 1)
 				{
 					$mtime = 0;
 				}
@@ -61,7 +61,7 @@ 
 					$mtime = filemtime($cache_file);
 				}
 
-				if($mtime + $interval*60 > $_SERVER['REQUEST_TIME']) 
+				if ($mtime + $interval * 60 > $_SERVER['REQUEST_TIME']) 
 				{
 					$page_content = FileHandler::readFile($cache_file); 
 					$page_content = preg_replace('@<\!--#Meta:@', '<!--Meta:', $page_content);
@@ -75,7 +75,7 @@ 
 			} 
 			else 
 			{
-				if(file_exists($cache_file))
+				if (file_exists($cache_file))
 				{
 					FileHandler::removeFile($cache_file);
 				}
@@ -97,13 +97,13 @@ 
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument(0, true);
 
-		if($this->module_info->mdocument_srl)
+		if ($this->module_info->mdocument_srl)
 		{
 			$document_srl = $this->module_info->mdocument_srl;
 			$oDocument->setDocument($document_srl);
 			Context::set('document_srl', $document_srl);
 		}
-		if(!$oDocument->isExists())
+		if (!$oDocument->isExists())
 		{
 			$document_srl = $this->module_info->document_srl;
 			$oDocument->setDocument($document_srl);
@@ -111,13 +111,13 @@ 
 		}
 		Context::set('oDocument', $oDocument);
 
-		if($this->module_info->mskin)
+		if ($this->module_info->mskin)
 		{
-			$templatePath = (sprintf($this->module_path.'m.skins/%s', $this->module_info->mskin));
+			$templatePath = (sprintf($this->module_path . 'm.skins/%s', $this->module_info->mskin));
 		}
 		else
 		{
-			$templatePath = ($this->module_path.'m.skins/default');
+			$templatePath = ($this->module_path . 'm.skins/default');
 		}
 
 		$page_content = $oTemplate->compile($templatePath, 'mobile');
@@ -128,9 +128,9 @@ 
 	function _getOutsideContent()
 	{
 		// check if it is http or internal file
-		if($this->path)
+		if ($this->path)
 		{
-			if(preg_match("/^([a-z]+):\/\//i",$this->path)) $content = $this->getHtmlPage($this->path, $this->interval, $this->cache_file);
+			if (preg_match("/^([a-z]+):\/\//i", $this->path)) $content = $this->getHtmlPage($this->path, $this->interval, $this->cache_file);
 			else $content = $this->executeFile($this->path, $this->interval, $this->cache_file);
 		}
 

Braces +18 added lines, -15 removed lines

@@ -30,12 +30,17 @@ 
 	function dispPageIndex()
 	{
 		// Variables used in the template Context:: set()
-		if($this->module_srl) Context::set('module_srl',$this->module_srl);
+		if($this->module_srl) {
+			Context::set('module_srl',$this->module_srl);
+		}
 
 		$page_type_name = strtolower($this->module_info->page_type);
 		$method = '_get' . ucfirst($page_type_name) . 'Content';
-		if (method_exists($this, $method)) $page_content = $this->{$method}();
-		else return new Object(-1, sprintf('%s method is not exists', $method));
+		if (method_exists($this, $method)) {
+			$page_content = $this->{$method}();
+		} else {
+			return new Object(-1, sprintf('%s method is not exists', $method));
+		}
 
 		Context::set('module_info', $this->module_info);
 		Context::set('page_content', $page_content);
@@ -55,8 +60,7 @@ 
 				if(!file_exists($cache_file) || filesize($cache_file) < 1)
 				{
 					$mtime = 0;
-				}
-				else
+				} else
 				{
 					$mtime = filemtime($cache_file);
 				}
@@ -65,15 +69,13 @@ 
 				{
 					$page_content = FileHandler::readFile($cache_file); 
 					$page_content = preg_replace('@<\!--#Meta:@', '<!--Meta:', $page_content);
-				} 
-				else 
+				} else 
 				{
 					$oWidgetController = getController('widget');
 					$page_content = $oWidgetController->transWidgetCode($this->module_info->mcontent);
 					FileHandler::writeFile($cache_file, $page_content);
 				}
-			} 
-			else 
+			} else 
 			{
 				if(file_exists($cache_file))
 				{
@@ -81,8 +83,7 @@ 
 				}
 				$page_content = $this->module_info->mcontent;
 			}
-		}
-		else
+		} else
 		{
 			$page_content = $this->module_info->content;
 		}
@@ -114,8 +115,7 @@ 
 		if($this->module_info->mskin)
 		{
 			$templatePath = (sprintf($this->module_path.'m.skins/%s', $this->module_info->mskin));
-		}
-		else
+		} else
 		{
 			$templatePath = ($this->module_path.'m.skins/default');
 		}
@@ -130,8 +130,11 @@ 
 		// check if it is http or internal file
 		if($this->path)
 		{
-			if(preg_match("/^([a-z]+):\/\//i",$this->path)) $content = $this->getHtmlPage($this->path, $this->interval, $this->cache_file);
-			else $content = $this->executeFile($this->path, $this->interval, $this->cache_file);
+			if(preg_match("/^([a-z]+):\/\//i",$this->path)) {
+				$content = $this->getHtmlPage($this->path, $this->interval, $this->cache_file);
+			} else {
+				$content = $this->executeFile($this->path, $this->interval, $this->cache_file);
+			}
 		}
 
 		return $content;

modules/page/page.wap.php

Indentation +4 added lines, -4 removed lines

@@ -1,10 +1,10 @@
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 /**
- * @class  pageWap
- * @author NAVER (developers@xpressengine.com)
- * @brief wap class page of the module
- */
+	 * @class  pageWap
+	 * @author NAVER (developers@xpressengine.com)
+	 * @brief wap class page of the module
+	 */
 class pageWap extends page
 {
 	/**

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
 	function procWAP(&$oMobile)
 	{
 		// Check permissions
-		if(!$this->grant->access) return $oMobile->setContent(Context::getLang('msg_not_permitted'));
+		if (!$this->grant->access) return $oMobile->setContent(Context::getLang('msg_not_permitted'));
 		// The contents of the widget chuchulham
 		$oWidgetController = getController('widget');
 		$content = $oWidgetController->transWidgetCode($this->module_info->content);

Braces +3 added lines, -1 removed lines

@@ -15,7 +15,9 @@
 	function procWAP(&$oMobile)
 	{
 		// Check permissions
-		if(!$this->grant->access) return $oMobile->setContent(Context::getLang('msg_not_permitted'));
+		if(!$this->grant->access) {
+			return $oMobile->setContent(Context::getLang('msg_not_permitted'));
+		}
 		// The contents of the widget chuchulham
 		$oWidgetController = getController('widget');
 		$content = $oWidgetController->transWidgetCode($this->module_info->content);

modules/point/point.admin.controller.php

Indentation +4 added lines, -4 removed lines

@@ -1,10 +1,10 @@
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 /**
- * @class  pointAdminController
- * @author NAVER (developers@xpressengine.com)
- * @brief The admin controller class of the point module
- */
+	 * @class  pointAdminController
+	 * @author NAVER (developers@xpressengine.com)
+	 * @brief The admin controller class of the point module
+	 */
 class pointAdminController extends point
 {
 	/**

Spacing +89 added lines, -89 removed lines

@@ -26,10 +26,10 @@ 
 		$args = Context::getRequestVars();
 
 		//if module IO config is off
-		if($args->able_module == 'Y')
+		if ($args->able_module == 'Y')
 		{
 			// Re-install triggers, if it was disabled.
-			if($config->able_module == 'N')
+			if ($config->able_module == 'N')
 			{
 				$this->moduleUpdate();
 			}
@@ -39,55 +39,55 @@ 
 
 			// Check the point name
 			$config->point_name = $args->point_name;
-			if(!$config->point_name) $config->point_name = 'point';
+			if (!$config->point_name) $config->point_name = 'point';
 			// Specify the default points
-			$config->signup_point = (int)$args->signup_point;
-			$config->login_point = (int)$args->login_point;
-			$config->insert_document = (int)$args->insert_document;
-			$config->read_document = (int)$args->read_document;
-			$config->insert_comment = (int)$args->insert_comment;
-			$config->upload_file = (int)$args->upload_file;
-			$config->download_file = (int)$args->download_file;
-			$config->voted = (int)$args->voted;
-			$config->blamed = (int)$args->blamed;
+			$config->signup_point = (int) $args->signup_point;
+			$config->login_point = (int) $args->login_point;
+			$config->insert_document = (int) $args->insert_document;
+			$config->read_document = (int) $args->read_document;
+			$config->insert_comment = (int) $args->insert_comment;
+			$config->upload_file = (int) $args->upload_file;
+			$config->download_file = (int) $args->download_file;
+			$config->voted = (int) $args->voted;
+			$config->blamed = (int) $args->blamed;
 			// The highest level
 			$config->max_level = $args->max_level;
-			if($config->max_level>1000) $config->max_level = 1000;
-			if($config->max_level<1) $config->max_level = 1;
+			if ($config->max_level > 1000) $config->max_level = 1000;
+			if ($config->max_level < 1) $config->max_level = 1;
 			// Set the level icon
 			$config->level_icon = $args->level_icon;
 			// Check if downloads are not allowed
-			if($args->disable_download == 'Y') $config->disable_download = 'Y';
+			if ($args->disable_download == 'Y') $config->disable_download = 'Y';
 			else $config->disable_download = 'N';
 			// Check if reading a document is not allowed
-			if($args->disable_read_document == 'Y') $config->disable_read_document = 'Y';
+			if ($args->disable_read_document == 'Y') $config->disable_read_document = 'Y';
 			else $config->disable_read_document = 'N';
 
 			$oMemberModel = getModel('member');
 			$group_list = $oMemberModel->getGroups();
 
 			// Per-level group configurations
-			foreach($group_list as $group)
+			foreach ($group_list as $group)
 			{
 				// Admin group should not be connected to point.
-				if($group->is_admin == 'Y' || $group->is_default == 'Y') continue;
+				if ($group->is_admin == 'Y' || $group->is_default == 'Y') continue;
 
 				$group_srl = $group->group_srl;
 
-				if(isset($args->{'point_group_'.$group_srl}))
+				if (isset($args->{'point_group_' . $group_srl}))
 				{
 					//if group level is higher than max level, change to max level
-					if($args->{'point_group_'.$group_srl} > $args->max_level)
+					if ($args->{'point_group_' . $group_srl} > $args->max_level)
 					{
-						$args->{'point_group_'.$group_srl} = $args->max_level;
+						$args->{'point_group_' . $group_srl} = $args->max_level;
 					}
 
 					//if group level is lower than 1, change to 1
-					if($args->{'point_group_'.$group_srl} < 1)
+					if ($args->{'point_group_' . $group_srl} < 1)
 					{
-						$args->{'point_group_'.$group_srl} = 1;
+						$args->{'point_group_' . $group_srl} = 1;
 					}
-					$config->point_group[$group_srl] = $args->{'point_group_'.$group_srl};
+					$config->point_group[$group_srl] = $args->{'point_group_' . $group_srl};
 				}
 				else
 				{
@@ -98,10 +98,10 @@ 
 			$config->group_reset = $args->group_reset;
 			// Per-level point configurations
 			unset($config->level_step);
-			for($i=1;$i<=$config->max_level;$i++)
+			for ($i = 1; $i <= $config->max_level; $i++)
 			{
-				$key = "level_step_".$i;
-				$config->level_step[$i] = (int)$args->{$key};
+				$key = "level_step_" . $i;
+				$config->level_step[$i] = (int) $args->{$key};
 			}
 			// A function to calculate per-level points
 			$config->expression = $args->expression;
@@ -133,11 +133,11 @@ 
 		$args = Context::getRequestVars();
 
 		$configTypeList = array('insert_document', 'insert_comment', 'upload_file', 'download_file', 'read_document', 'voted', 'blamed');
-		foreach($configTypeList AS $config)
+		foreach ($configTypeList AS $config)
 		{
-			if(is_array($args->{$config}))
+			if (is_array($args->{$config}))
 			{
-				foreach($args->{$config} AS $key=>$value)
+				foreach ($args->{$config} AS $key=>$value)
 				{
 					$module_config[$key][$config] = $value;
 				}
@@ -145,19 +145,19 @@ 
 		}
 
 		$oModuleController = getController('module');
-		if(count($module_config))
+		if (count($module_config))
 		{
-			foreach($module_config as $module_srl => $config)
+			foreach ($module_config as $module_srl => $config)
 			{
-				$oModuleController->insertModulePartConfig('point',$module_srl,$config);
+				$oModuleController->insertModulePartConfig('point', $module_srl, $config);
 			}
 		}
 
 		$this->setMessage('success_updated');
-		if(!in_array(Context::getRequestMethod(),array('XMLRPC','JSON')))
+		if (!in_array(Context::getRequestMethod(), array('XMLRPC', 'JSON')))
 		{
 			$returnUrl = Context::get('success_return_url') ? Context::get('success_return_url') : getNotEncodedUrl('', 'module', 'admin', 'act', 'dispPointAdminModuleConfig');
-			header('location:'.$returnUrl);
+			header('location:' . $returnUrl);
 			return;
 		}
 	}
@@ -168,24 +168,24 @@ 
 	function procPointAdminInsertPointModuleConfig()
 	{
 		$module_srl = Context::get('target_module_srl');
-		if(!$module_srl) return new Object(-1, 'msg_invalid_request');
+		if (!$module_srl) return new Object(-1, 'msg_invalid_request');
 		// In case of batch configuration of several modules
-		if(preg_match('/^([0-9,]+)$/',$module_srl)) $module_srl = explode(',',$module_srl);
+		if (preg_match('/^([0-9,]+)$/', $module_srl)) $module_srl = explode(',', $module_srl);
 		else $module_srl = array($module_srl);
 		// Save configurations
 		$oModuleController = getController('module');
-		for($i=0;$i<count($module_srl);$i++)
+		for ($i = 0; $i < count($module_srl); $i++)
 		{
 			$srl = trim($module_srl[$i]);
-			if(!$srl) continue;
+			if (!$srl) continue;
 			unset($config);
-			$config['insert_document'] = (int)Context::get('insert_document');
-			$config['insert_comment'] = (int)Context::get('insert_comment');
-			$config['upload_file'] = (int)Context::get('upload_file');
-			$config['download_file'] = (int)Context::get('download_file');
-			$config['read_document'] = (int)Context::get('read_document');
-			$config['voted'] = (int)Context::get('voted');
-			$config['blamed'] = (int)Context::get('blamed');
+			$config['insert_document'] = (int) Context::get('insert_document');
+			$config['insert_comment'] = (int) Context::get('insert_comment');
+			$config['upload_file'] = (int) Context::get('upload_file');
+			$config['download_file'] = (int) Context::get('download_file');
+			$config['read_document'] = (int) Context::get('read_document');
+			$config['voted'] = (int) Context::get('voted');
+			$config['blamed'] = (int) Context::get('blamed');
 			$oModuleController->insertModulePartConfig('point', $srl, $config);
 		}
 
@@ -207,7 +207,7 @@ 
 		preg_match('/^(\+|-)?([1-9][0-9]*)$/', $point, $m);
 
 		$action = '';
-		switch($m[1])
+		switch ($m[1])
 		{
 			case '+':
 				$action = 'add';
@@ -222,7 +222,7 @@ 
 		$point = $m[2];
 
 		$oPointController = getController('point');
-		$output = $oPointController->setPoint($member_srl, (int)$point, $action);
+		$output = $oPointController->setPoint($member_srl, (int) $point, $action);
 
 		$this->setError(-1);
 		$this->setMessage('success_updated', 'info');
@@ -247,29 +247,29 @@ 
 
 		// Get member infomation
 		$output = executeQueryArray('point.getMemberCount');
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		if($output->data)
+		if ($output->data)
 		{
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
-				if(!$val->member_srl) continue;
+				if (!$val->member_srl) continue;
 				$member[$val->member_srl] = 0;
 			}
 		}
 
 		// Get post information
 		$output = executeQueryArray('point.getDocumentPoint');
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		if($output->data)
+		if ($output->data)
 		{
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
-				if($module_config[$val->module_srl]['insert_document']) $insert_point = $module_config[$val->module_srl]['insert_document'];
+				if ($module_config[$val->module_srl]['insert_document']) $insert_point = $module_config[$val->module_srl]['insert_document'];
 				else $insert_point = $config->insert_document;
 
-				if(!$val->member_srl) continue;
+				if (!$val->member_srl) continue;
 				$point = $insert_point * $val->count;
 				$member[$val->member_srl] += $point;
 			}
@@ -278,16 +278,16 @@ 
 		$output = null;
 		// Get comments information
 		$output = executeQueryArray('point.getCommentPoint');
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		if($output->data)
+		if ($output->data)
 		{
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
-				if($module_config[$val->module_srl]['insert_comment']) $insert_point = $module_config[$val->module_srl]['insert_comment'];
+				if ($module_config[$val->module_srl]['insert_comment']) $insert_point = $module_config[$val->module_srl]['insert_comment'];
 				else $insert_point = $config->insert_comment;
 
-				if(!$val->member_srl) continue;
+				if (!$val->member_srl) continue;
 				$point = $insert_point * $val->count;
 				$member[$val->member_srl] += $point;
 			}
@@ -295,16 +295,16 @@ 
 		$output = null;
 		// Get the attached files' information
 		$output = executeQueryArray('point.getFilePoint');
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
-		if($output->data)
+		if ($output->data)
 		{
-			foreach($output->data as $key => $val)
+			foreach ($output->data as $key => $val)
 			{
-				if($module_config[$val->module_srl]['upload_file']) $insert_point = $module_config[$val->module_srl]['upload_file'];
+				if ($module_config[$val->module_srl]['upload_file']) $insert_point = $module_config[$val->module_srl]['upload_file'];
 				else $insert_point = $config->upload_file;
 
-				if(!$val->member_srl) continue;
+				if (!$val->member_srl) continue;
 				$point = $insert_point * $val->count;
 				$member[$val->member_srl] += $point;
 			}
@@ -312,21 +312,21 @@ 
 		$output = null;
 		// Set all members' points to 0
 		$output = executeQuery("point.initMemberPoint");
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 		// Save the file temporarily
 		
 		$str = '';
-		foreach($member as $key => $val)
+		foreach ($member as $key => $val)
 		{
-			$val += (int)$config->signup_point;
-			$str .= $key.','.$val."\r\n";
+			$val += (int) $config->signup_point;
+			$str .= $key . ',' . $val . "\r\n";
 		}
 
 		@file_put_contents('./files/cache/pointRecal.txt', $str, LOCK_EX);
 
 		$this->add('total', count($member));
 		$this->add('position', 0);
-		$this->setMessage( sprintf(Context::getLang('point_recal_message'), 0, $this->get('total')) );
+		$this->setMessage(sprintf(Context::getLang('point_recal_message'), 0, $this->get('total')));
 	}
 
 	/**
@@ -334,35 +334,35 @@ 
 	 */
 	function procPointAdminApplyPoint()
 	{
-		$position = (int)Context::get('position');
-		$total = (int)Context::get('total');
+		$position = (int) Context::get('position');
+		$total = (int) Context::get('total');
 
-		if(!file_exists('./files/cache/pointRecal.txt')) return new Object(-1, 'msg_invalid_request');
+		if (!file_exists('./files/cache/pointRecal.txt')) return new Object(-1, 'msg_invalid_request');
 
 		$idx = 0;
-		$f = fopen("./files/cache/pointRecal.txt","r");
-		while(!feof($f))
+		$f = fopen("./files/cache/pointRecal.txt", "r");
+		while (!feof($f))
 		{
 			$str = trim(fgets($f, 1024));
-			$idx ++;
-			if($idx > $position)
+			$idx++;
+			if ($idx > $position)
 			{
-				list($member_srl, $point) = explode(',',$str);
+				list($member_srl, $point) = explode(',', $str);
 
 				$args = new stdClass();
 				$args->member_srl = $member_srl;
 				$args->point = $point;
-				$output = executeQuery('point.insertPoint',$args);
-				if($idx%5000==0) break;
+				$output = executeQuery('point.insertPoint', $args);
+				if ($idx % 5000 == 0) break;
 			}
 		}
 
-		if(feof($f))
+		if (feof($f))
 		{
 			FileHandler::removeFile('./files/cache/pointRecal.txt');
 			$idx = $total;
 
-			FileHandler::rename('./files/member_extra_info/point','./files/member_extra_info/point.old');
+			FileHandler::rename('./files/member_extra_info/point', './files/member_extra_info/point.old');
 
 			FileHandler::removeDir('./files/member_extra_info/point.old');
 		}
@@ -380,16 +380,16 @@ 
 	function procPointAdminReset()
 	{
 		$module_srl = Context::get('module_srls');
-		if(!$module_srl) return new Object(-1, 'msg_invalid_request');
+		if (!$module_srl) return new Object(-1, 'msg_invalid_request');
 		// In case of batch configuration of several modules
-		if(preg_match('/^([0-9,]+)$/',$module_srl)) $module_srl = explode(',',$module_srl);
+		if (preg_match('/^([0-9,]+)$/', $module_srl)) $module_srl = explode(',', $module_srl);
 		else $module_srl = array($module_srl);
 		// Save configurations
 		$oModuleController = getController('module');
-		for($i=0;$i<count($module_srl);$i++)
+		for ($i = 0; $i < count($module_srl); $i++)
 		{
 			$srl = trim($module_srl[$i]);
-			if(!$srl) continue;
+			if (!$srl) continue;
 			$args = new stdClass();
 			$args->module = 'point';
 			$args->module_srl = $srl;
@@ -397,7 +397,7 @@ 
 		}
 
 		$oCacheHandler = CacheHandler::getInstance('object', null, true);
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$oCacheHandler->invalidateGroupKey('site_and_module');
 		}

Braces +94 added lines, -37 removed lines

@@ -39,7 +39,9 @@ 
 
 			// Check the point name
 			$config->point_name = $args->point_name;
-			if(!$config->point_name) $config->point_name = 'point';
+			if(!$config->point_name) {
+				$config->point_name = 'point';
+			}
 			// Specify the default points
 			$config->signup_point = (int)$args->signup_point;
 			$config->login_point = (int)$args->login_point;
@@ -52,16 +54,26 @@ 
 			$config->blamed = (int)$args->blamed;
 			// The highest level
 			$config->max_level = $args->max_level;
-			if($config->max_level>1000) $config->max_level = 1000;
-			if($config->max_level<1) $config->max_level = 1;
+			if($config->max_level>1000) {
+				$config->max_level = 1000;
+			}
+			if($config->max_level<1) {
+				$config->max_level = 1;
+			}
 			// Set the level icon
 			$config->level_icon = $args->level_icon;
 			// Check if downloads are not allowed
-			if($args->disable_download == 'Y') $config->disable_download = 'Y';
-			else $config->disable_download = 'N';
+			if($args->disable_download == 'Y') {
+				$config->disable_download = 'Y';
+			} else {
+				$config->disable_download = 'N';
+			}
 			// Check if reading a document is not allowed
-			if($args->disable_read_document == 'Y') $config->disable_read_document = 'Y';
-			else $config->disable_read_document = 'N';
+			if($args->disable_read_document == 'Y') {
+				$config->disable_read_document = 'Y';
+			} else {
+				$config->disable_read_document = 'N';
+			}
 
 			$oMemberModel = getModel('member');
 			$group_list = $oMemberModel->getGroups();
@@ -70,7 +82,9 @@ 
 			foreach($group_list as $group)
 			{
 				// Admin group should not be connected to point.
-				if($group->is_admin == 'Y' || $group->is_default == 'Y') continue;
+				if($group->is_admin == 'Y' || $group->is_default == 'Y') {
+					continue;
+				}
 
 				$group_srl = $group->group_srl;
 
@@ -88,8 +102,7 @@ 
 						$args->{'point_group_'.$group_srl} = 1;
 					}
 					$config->point_group[$group_srl] = $args->{'point_group_'.$group_srl};
-				}
-				else
+				} else
 				{
 					unset($config->point_group[$group_srl]);
 				}
@@ -105,8 +118,7 @@ 
 			}
 			// A function to calculate per-level points
 			$config->expression = $args->expression;
-		}
-		else
+		} else
 		{
 			//module IO config is OFF, Other settings will not be modified.
 			$config->able_module = 'N';
@@ -168,16 +180,23 @@ 
 	function procPointAdminInsertPointModuleConfig()
 	{
 		$module_srl = Context::get('target_module_srl');
-		if(!$module_srl) return new Object(-1, 'msg_invalid_request');
+		if(!$module_srl) {
+			return new Object(-1, 'msg_invalid_request');
+		}
 		// In case of batch configuration of several modules
-		if(preg_match('/^([0-9,]+)$/',$module_srl)) $module_srl = explode(',',$module_srl);
-		else $module_srl = array($module_srl);
+		if(preg_match('/^([0-9,]+)$/',$module_srl)) {
+			$module_srl = explode(',',$module_srl);
+		} else {
+			$module_srl = array($module_srl);
+		}
 		// Save configurations
 		$oModuleController = getController('module');
 		for($i=0;$i<count($module_srl);$i++)
 		{
 			$srl = trim($module_srl[$i]);
-			if(!$srl) continue;
+			if(!$srl) {
+				continue;
+			}
 			unset($config);
 			$config['insert_document'] = (int)Context::get('insert_document');
 			$config['insert_comment'] = (int)Context::get('insert_comment');
@@ -247,29 +266,40 @@ 
 
 		// Get member infomation
 		$output = executeQueryArray('point.getMemberCount');
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		if($output->data)
 		{
 			foreach($output->data as $key => $val)
 			{
-				if(!$val->member_srl) continue;
+				if(!$val->member_srl) {
+					continue;
+				}
 				$member[$val->member_srl] = 0;
 			}
 		}
 
 		// Get post information
 		$output = executeQueryArray('point.getDocumentPoint');
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		if($output->data)
 		{
 			foreach($output->data as $key => $val)
 			{
-				if($module_config[$val->module_srl]['insert_document']) $insert_point = $module_config[$val->module_srl]['insert_document'];
-				else $insert_point = $config->insert_document;
+				if($module_config[$val->module_srl]['insert_document']) {
+					$insert_point = $module_config[$val->module_srl]['insert_document'];
+				} else {
+					$insert_point = $config->insert_document;
+				}
 
-				if(!$val->member_srl) continue;
+				if(!$val->member_srl) {
+					continue;
+				}
 				$point = $insert_point * $val->count;
 				$member[$val->member_srl] += $point;
 			}
@@ -278,16 +308,23 @@ 
 		$output = null;
 		// Get comments information
 		$output = executeQueryArray('point.getCommentPoint');
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		if($output->data)
 		{
 			foreach($output->data as $key => $val)
 			{
-				if($module_config[$val->module_srl]['insert_comment']) $insert_point = $module_config[$val->module_srl]['insert_comment'];
-				else $insert_point = $config->insert_comment;
+				if($module_config[$val->module_srl]['insert_comment']) {
+					$insert_point = $module_config[$val->module_srl]['insert_comment'];
+				} else {
+					$insert_point = $config->insert_comment;
+				}
 
-				if(!$val->member_srl) continue;
+				if(!$val->member_srl) {
+					continue;
+				}
 				$point = $insert_point * $val->count;
 				$member[$val->member_srl] += $point;
 			}
@@ -295,16 +332,23 @@ 
 		$output = null;
 		// Get the attached files' information
 		$output = executeQueryArray('point.getFilePoint');
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		if($output->data)
 		{
 			foreach($output->data as $key => $val)
 			{
-				if($module_config[$val->module_srl]['upload_file']) $insert_point = $module_config[$val->module_srl]['upload_file'];
-				else $insert_point = $config->upload_file;
+				if($module_config[$val->module_srl]['upload_file']) {
+					$insert_point = $module_config[$val->module_srl]['upload_file'];
+				} else {
+					$insert_point = $config->upload_file;
+				}
 
-				if(!$val->member_srl) continue;
+				if(!$val->member_srl) {
+					continue;
+				}
 				$point = $insert_point * $val->count;
 				$member[$val->member_srl] += $point;
 			}
@@ -312,7 +356,9 @@ 
 		$output = null;
 		// Set all members' points to 0
 		$output = executeQuery("point.initMemberPoint");
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 		// Save the file temporarily
 		
 		$str = '';
@@ -337,7 +383,9 @@ 
 		$position = (int)Context::get('position');
 		$total = (int)Context::get('total');
 
-		if(!file_exists('./files/cache/pointRecal.txt')) return new Object(-1, 'msg_invalid_request');
+		if(!file_exists('./files/cache/pointRecal.txt')) {
+			return new Object(-1, 'msg_invalid_request');
+		}
 
 		$idx = 0;
 		$f = fopen("./files/cache/pointRecal.txt","r");
@@ -353,7 +401,9 @@ 
 				$args->member_srl = $member_srl;
 				$args->point = $point;
 				$output = executeQuery('point.insertPoint',$args);
-				if($idx%5000==0) break;
+				if($idx%5000==0) {
+					break;
+				}
 			}
 		}
 
@@ -380,16 +430,23 @@ 
 	function procPointAdminReset()
 	{
 		$module_srl = Context::get('module_srls');
-		if(!$module_srl) return new Object(-1, 'msg_invalid_request');
+		if(!$module_srl) {
+			return new Object(-1, 'msg_invalid_request');
+		}
 		// In case of batch configuration of several modules
-		if(preg_match('/^([0-9,]+)$/',$module_srl)) $module_srl = explode(',',$module_srl);
-		else $module_srl = array($module_srl);
+		if(preg_match('/^([0-9,]+)$/',$module_srl)) {
+			$module_srl = explode(',',$module_srl);
+		} else {
+			$module_srl = array($module_srl);
+		}
 		// Save configurations
 		$oModuleController = getController('module');
 		for($i=0;$i<count($module_srl);$i++)
 		{
 			$srl = trim($module_srl[$i]);
-			if(!$srl) continue;
+			if(!$srl) {
+				continue;
+			}
 			$args = new stdClass();
 			$args->module = 'point';
 			$args->module_srl = $srl;

modules/point/point.admin.view.php

Indentation +4 added lines, -4 removed lines

@@ -1,10 +1,10 @@
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 /**
- * @class  pointAdminView
- * @author NAVER (developers@xpressengine.com)
- * @brief The admin view class of the point module
- */
+	 * @class  pointAdminView
+	 * @author NAVER (developers@xpressengine.com)
+	 * @brief The admin view class of the point module
+	 */
 class pointAdminView extends point
 {
 	/**

Spacing +7 added lines, -7 removed lines

@@ -20,11 +20,11 @@ 
 
 		//Security
 		$security = new Security();
-		$security->encodeHTML('config.point_name','config.level_icon');
+		$security->encodeHTML('config.point_name', 'config.level_icon');
 		$security->encodeHTML('module_info..');
 
 		// Set the template path
-		$this->setTemplatePath($this->module_path.'tpl');
+		$this->setTemplatePath($this->module_path . 'tpl');
 	}
 
 	/**
@@ -39,9 +39,9 @@ 
 		$oMemberModel = getModel('member');
 		$group_list = $oMemberModel->getGroups();
 		$selected_group_list = array();
-		if(count($group_list))
+		if (count($group_list))
 		{
-			foreach($group_list as $key => $val)
+			foreach ($group_list as $key => $val)
 			{
 				$selected_group_list[$key] = $val;
 			}
@@ -49,7 +49,7 @@ 
 		Context::set('group_list', $selected_group_list);
 		//Security
 		$security = new Security();
-		$security->encodeHTML('group_list..title','group_list..description');
+		$security->encodeHTML('group_list..title', 'group_list..description');
 
 		// Set the template
 		$this->setTemplateFile('config');
@@ -69,7 +69,7 @@ 
 		Context::set('module_config', $oModuleModel->getModulePartConfigs('point'));
 		//Security
 		$security = new Security();			
-		$security->encodeHTML('mid_list..browser_title','mid_list..mid');
+		$security->encodeHTML('mid_list..browser_title', 'mid_list..mid');
 
 		// Set the template
 		$this->setTemplateFile('module_config');
@@ -115,7 +115,7 @@ 
 		Context::set('group_list', $this->group_list);
 		//Security
 		$security = new Security();			
-		$security->encodeHTML('group_list..title','group_list..description');
+		$security->encodeHTML('group_list..title', 'group_list..description');
 		$security->encodeHTML('member_list..');
 
 		// Set the template