xpressengine / xe-core

modules/message/message.mobile.php

Braces +9 added lines, -3 removed lines

@@ -18,8 +18,12 @@ 
 		// Get configurations (using module model object)
 		$oModuleModel = getModel('module');
 		$config = $oModuleModel->getModuleConfig('message');
-		if(!is_object($config)) $config = new stdClass;
-		if(!$config->mskin) $config->mskin = 'default';
+		if(!is_object($config)) {
+			$config = new stdClass;
+		}
+		if(!$config->mskin) {
+			$config->mskin = 'default';
+		}
 		// Set the template path
 		$template_path = sprintf('%sm.skins/%s', $this->module_path, $config->mskin);
 		// Get the member configuration
@@ -30,7 +34,9 @@ 
 		$ssl_mode = false;
 		if($member_config->enable_ssl == 'Y')
 		{
-			if(strncasecmp('https://', Context::getRequestUri(), 8) === 0) $ssl_mode = true;
+			if(strncasecmp('https://', Context::getRequestUri(), 8) === 0) {
+				$ssl_mode = true;
+			}
 		}
 		Context::set('ssl_mode',$ssl_mode);
 

Spacing +5 added lines, -5 removed lines

@@ -18,8 +18,8 @@ 
 		// Get configurations (using module model object)
 		$oModuleModel = getModel('module');
 		$config = $oModuleModel->getModuleConfig('message');
-		if(!is_object($config)) $config = new stdClass;
-		if(!$config->mskin) $config->mskin = 'default';
+		if (!is_object($config)) $config = new stdClass;
+		if (!$config->mskin) $config->mskin = 'default';
 		// Set the template path
 		$template_path = sprintf('%sm.skins/%s', $this->module_path, $config->mskin);
 		// Get the member configuration
@@ -28,11 +28,11 @@ 
 		Context::set('member_config', $member_config);
 		// Set a flag to check if the https connection is made when using SSL and create https url 
 		$ssl_mode = false;
-		if($member_config->enable_ssl == 'Y')
+		if ($member_config->enable_ssl == 'Y')
 		{
-			if(strncasecmp('https://', Context::getRequestUri(), 8) === 0) $ssl_mode = true;
+			if (strncasecmp('https://', Context::getRequestUri(), 8) === 0) $ssl_mode = true;
 		}
-		Context::set('ssl_mode',$ssl_mode);
+		Context::set('ssl_mode', $ssl_mode);
 
 		Context::set('system_message', nl2br($this->getMessage()));
 

modules/message/message.view.php

Spacing +5 added lines, -5 removed lines

@@ -23,12 +23,12 @@ 
 		$oModuleModel = getModel('module');
 		$this->module_config = $config = $oModuleModel->getModuleConfig('message', $this->module_info->site_srl);
 
-		if(!$config)
+		if (!$config)
 		{
 			$config = new stdClass();
 		}
 
-		if(!$config->skin)
+		if (!$config->skin)
 		{
 			$config->skin = 'xedition';
 			$template_path = sprintf('%sskins/%s', $this->module_path, $config->skin);
@@ -54,11 +54,11 @@ 
 		Context::set('member_config', $member_config);
 		// Set a flag to check if the https connection is made when using SSL and create https url
 		$ssl_mode = false;
-		if($member_config->enable_ssl == 'Y')
+		if ($member_config->enable_ssl == 'Y')
 		{
-			if(strncasecmp('https://', Context::getRequestUri(), 8) === 0) $ssl_mode = true;
+			if (strncasecmp('https://', Context::getRequestUri(), 8) === 0) $ssl_mode = true;
 		}
-		Context::set('ssl_mode',$ssl_mode);
+		Context::set('ssl_mode', $ssl_mode);
 
 		Context::set('system_message', nl2br($this->getMessage()));
 

Braces +5 added lines, -5 removed lines

@@ -32,16 +32,14 @@ 
 		{
 			$config->skin = 'xedition';
 			$template_path = sprintf('%sskins/%s', $this->module_path, $config->skin);
-		}
-		else
+		} else
 		{
 			//check theme
 			$config_parse = explode('|@|', $config->skin);
 			if (count($config_parse) > 1)
 			{
 				$template_path = sprintf('./themes/%s/modules/message/', $config_parse[0]);
-			}
-			else
+			} else
 			{
 				$template_path = sprintf('%sskins/%s', $this->module_path, $config->skin);
 			}
@@ -56,7 +54,9 @@ 
 		$ssl_mode = false;
 		if($member_config->enable_ssl == 'Y')
 		{
-			if(strncasecmp('https://', Context::getRequestUri(), 8) === 0) $ssl_mode = true;
+			if(strncasecmp('https://', Context::getRequestUri(), 8) === 0) {
+				$ssl_mode = true;
+			}
 		}
 		Context::set('ssl_mode',$ssl_mode);
 

modules/module/module.mobile.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 {
 	function dispModuleChangeLang()
 	{
-		$this->setTemplatePath(sprintf("%stpl/",$this->module_path));
+		$this->setTemplatePath(sprintf("%stpl/", $this->module_path));
 		$this->setTemplateFile('lang.html');
 	}
 }

modules/page/page.api.php

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.controller.php

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;
 		}

Spacing +21 added lines, -21 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']:'');
+		$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,14 +95,14 @@ 
 	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
 		{

modules/page/page.wap.php

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.class.php

Spacing +35 added lines, -35 removed lines

@@ -24,9 +24,9 @@ 
 		$config->able_module = 'N';
 		$config->max_level = 30;
 		// Per-level score
-		for($i=1;$i<=30;$i++)
+		for ($i = 1; $i <= 30; $i++)
 		{
-			$config->level_step[$i] = pow($i,2)*90;
+			$config->level_step[$i] = pow($i, 2) * 90;
 		}
 		// Points for registration
 		$config->signup_point = 10;
@@ -84,29 +84,29 @@ 
 
 		$config = $oModuleModel->getModuleConfig('point');
 		// check if module is abled
-		if($config->able_module != 'N')
+		if ($config->able_module != 'N')
 		{
 			// Add a trigger for registration/insert document/insert comment/upload a file/download
-			if(!$oModuleModel->getTrigger('member.insertMember', 'point', 'controller', 'triggerInsertMember', 'after')) return true;
-			if(!$oModuleModel->getTrigger('document.insertDocument', 'point', 'controller', 'triggerInsertDocument', 'after')) return true;
-			if(!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerBeforeDeleteDocument', 'before')) return true;
-			if(!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerDeleteDocument', 'after')) return true;
-			if(!$oModuleModel->getTrigger('comment.insertComment', 'point', 'controller', 'triggerInsertComment', 'after')) return true;
-			if(!$oModuleModel->getTrigger('comment.deleteComment', 'point', 'controller', 'triggerDeleteComment', 'after')) return true;
-			if(!$oModuleModel->getTrigger('file.insertFile', 'point', 'controller', 'triggerInsertFile', 'after')) return true;
-			if(!$oModuleModel->getTrigger('file.deleteFile', 'point', 'controller', 'triggerDeleteFile', 'after')) return true;
-			if(!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerBeforeDownloadFile', 'before')) return true;
-			if(!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerDownloadFile', 'after')) return true;
-			if(!$oModuleModel->getTrigger('member.doLogin', 'point', 'controller', 'triggerAfterLogin', 'after')) return true;
-			if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'point', 'view', 'triggerDispPointAdditionSetup', 'after')) return true;
-			if(!$oModuleModel->getTrigger('document.updateReadedCount', 'point', 'controller', 'triggerUpdateReadedCount', 'after')) return true;
+			if (!$oModuleModel->getTrigger('member.insertMember', 'point', 'controller', 'triggerInsertMember', 'after')) return true;
+			if (!$oModuleModel->getTrigger('document.insertDocument', 'point', 'controller', 'triggerInsertDocument', 'after')) return true;
+			if (!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerBeforeDeleteDocument', 'before')) return true;
+			if (!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerDeleteDocument', 'after')) return true;
+			if (!$oModuleModel->getTrigger('comment.insertComment', 'point', 'controller', 'triggerInsertComment', 'after')) return true;
+			if (!$oModuleModel->getTrigger('comment.deleteComment', 'point', 'controller', 'triggerDeleteComment', 'after')) return true;
+			if (!$oModuleModel->getTrigger('file.insertFile', 'point', 'controller', 'triggerInsertFile', 'after')) return true;
+			if (!$oModuleModel->getTrigger('file.deleteFile', 'point', 'controller', 'triggerDeleteFile', 'after')) return true;
+			if (!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerBeforeDownloadFile', 'before')) return true;
+			if (!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerDownloadFile', 'after')) return true;
+			if (!$oModuleModel->getTrigger('member.doLogin', 'point', 'controller', 'triggerAfterLogin', 'after')) return true;
+			if (!$oModuleModel->getTrigger('module.dispAdditionSetup', 'point', 'view', 'triggerDispPointAdditionSetup', 'after')) return true;
+			if (!$oModuleModel->getTrigger('document.updateReadedCount', 'point', 'controller', 'triggerUpdateReadedCount', 'after')) return true;
 			// Add a trigger for voting up and down 2008.05.13 haneul
-			if(!$oModuleModel->getTrigger('document.updateVotedCount', 'point', 'controller', 'triggerUpdateVotedCount', 'after')) return true;
+			if (!$oModuleModel->getTrigger('document.updateVotedCount', 'point', 'controller', 'triggerUpdateVotedCount', 'after')) return true;
 			// Add a trigger for using points for permanent saving of a temporarily saved document 2009.05.19 zero
-			if(!$oModuleModel->getTrigger('document.updateDocument', 'point', 'controller', 'triggerUpdateDocument', 'before')) return true;
+			if (!$oModuleModel->getTrigger('document.updateDocument', 'point', 'controller', 'triggerUpdateDocument', 'before')) return true;
 
 			// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-			if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'point', 'controller', 'triggerCopyModule', 'after')) return true;
+			if (!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'point', 'controller', 'triggerCopyModule', 'after')) return true;
 		}
 
 		return false;
@@ -121,40 +121,40 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		// Add a trigger for registration/insert document/insert comment/upload a file/download
-		if(!$oModuleModel->getTrigger('member.insertMember', 'point', 'controller', 'triggerInsertMember', 'after')) 
+		if (!$oModuleModel->getTrigger('member.insertMember', 'point', 'controller', 'triggerInsertMember', 'after')) 
 			$oModuleController->insertTrigger('member.insertMember', 'point', 'controller', 'triggerInsertMember', 'after');
-		if(!$oModuleModel->getTrigger('document.insertDocument', 'point', 'controller', 'triggerInsertDocument', 'after')) 
+		if (!$oModuleModel->getTrigger('document.insertDocument', 'point', 'controller', 'triggerInsertDocument', 'after')) 
 			$oModuleController->insertTrigger('document.insertDocument', 'point', 'controller', 'triggerInsertDocument', 'after');
-		if(!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerBeforeDeleteDocument', 'before')) 
+		if (!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerBeforeDeleteDocument', 'before')) 
 			$oModuleController->insertTrigger('document.deleteDocument', 'point', 'controller', 'triggerBeforeDeleteDocument', 'before');
-		if(!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerDeleteDocument', 'after')) 
+		if (!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerDeleteDocument', 'after')) 
 			$oModuleController->insertTrigger('document.deleteDocument', 'point', 'controller', 'triggerDeleteDocument', 'after');
-		if(!$oModuleModel->getTrigger('comment.insertComment', 'point', 'controller', 'triggerInsertComment', 'after')) 
+		if (!$oModuleModel->getTrigger('comment.insertComment', 'point', 'controller', 'triggerInsertComment', 'after')) 
 			$oModuleController->insertTrigger('comment.insertComment', 'point', 'controller', 'triggerInsertComment', 'after');
-		if(!$oModuleModel->getTrigger('comment.deleteComment', 'point', 'controller', 'triggerDeleteComment', 'after')) 
+		if (!$oModuleModel->getTrigger('comment.deleteComment', 'point', 'controller', 'triggerDeleteComment', 'after')) 
 			$oModuleController->insertTrigger('comment.deleteComment', 'point', 'controller', 'triggerDeleteComment', 'after');
-		if(!$oModuleModel->getTrigger('file.insertFile', 'point', 'controller', 'triggerInsertFile', 'after')) 
+		if (!$oModuleModel->getTrigger('file.insertFile', 'point', 'controller', 'triggerInsertFile', 'after')) 
 			$oModuleController->insertTrigger('file.insertFile', 'point', 'controller', 'triggerInsertFile', 'after');
-		if(!$oModuleModel->getTrigger('file.deleteFile', 'point', 'controller', 'triggerDeleteFile', 'after')) 
+		if (!$oModuleModel->getTrigger('file.deleteFile', 'point', 'controller', 'triggerDeleteFile', 'after')) 
 			$oModuleController->insertTrigger('file.deleteFile', 'point', 'controller', 'triggerDeleteFile', 'after');
-		if(!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerBeforeDownloadFile', 'before')) 
+		if (!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerBeforeDownloadFile', 'before')) 
 			$oModuleController->insertTrigger('file.downloadFile', 'point', 'controller', 'triggerBeforeDownloadFile', 'before');
-		if(!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerDownloadFile', 'after')) 
+		if (!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerDownloadFile', 'after')) 
 			$oModuleController->insertTrigger('file.downloadFile', 'point', 'controller', 'triggerDownloadFile', 'after');
-		if(!$oModuleModel->getTrigger('member.doLogin', 'point', 'controller', 'triggerAfterLogin', 'after'))
+		if (!$oModuleModel->getTrigger('member.doLogin', 'point', 'controller', 'triggerAfterLogin', 'after'))
 			$oModuleController->insertTrigger('member.doLogin', 'point', 'controller', 'triggerAfterLogin', 'after');
-		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'point', 'view', 'triggerDispPointAdditionSetup', 'after')) 
+		if (!$oModuleModel->getTrigger('module.dispAdditionSetup', 'point', 'view', 'triggerDispPointAdditionSetup', 'after')) 
 			$oModuleController->insertTrigger('module.dispAdditionSetup', 'point', 'view', 'triggerDispPointAdditionSetup', 'after');
-		if(!$oModuleModel->getTrigger('document.updateReadedCount', 'point', 'controller', 'triggerUpdateReadedCount', 'after')) 
+		if (!$oModuleModel->getTrigger('document.updateReadedCount', 'point', 'controller', 'triggerUpdateReadedCount', 'after')) 
 			$oModuleController->insertTrigger('document.updateReadedCount', 'point', 'controller', 'triggerUpdateReadedCount', 'after');
 		// Add a trigger for voting up and down 2008.05.13 haneul
-		if(!$oModuleModel->getTrigger('document.updateVotedCount', 'point', 'controller', 'triggerUpdateVotedCount', 'after'))
+		if (!$oModuleModel->getTrigger('document.updateVotedCount', 'point', 'controller', 'triggerUpdateVotedCount', 'after'))
 			$oModuleController->insertTrigger('document.updateVotedCount', 'point', 'controller', 'triggerUpdateVotedCount', 'after');
 		// Add a trigger for using points for permanent saving of a temporarily saved document 2009.05.19 zero
-		if(!$oModuleModel->getTrigger('document.updateDocument', 'point', 'controller', 'triggerUpdateDocument', 'before')) 
+		if (!$oModuleModel->getTrigger('document.updateDocument', 'point', 'controller', 'triggerUpdateDocument', 'before')) 
 			$oModuleController->insertTrigger('document.updateDocument', 'point', 'controller', 'triggerUpdateDocument', 'before');
 		// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-		if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'point', 'controller', 'triggerCopyModule', 'after'))
+		if (!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'point', 'controller', 'triggerCopyModule', 'after'))
 		{
 			$oModuleController->insertTrigger('module.procModuleAdminCopyModule', 'point', 'controller', 'triggerCopyModule', 'after');
 		}

Braces +93 added lines, -46 removed lines

@@ -87,26 +87,58 @@ 
 		if($config->able_module != 'N')
 		{
 			// Add a trigger for registration/insert document/insert comment/upload a file/download
-			if(!$oModuleModel->getTrigger('member.insertMember', 'point', 'controller', 'triggerInsertMember', 'after')) return true;
-			if(!$oModuleModel->getTrigger('document.insertDocument', 'point', 'controller', 'triggerInsertDocument', 'after')) return true;
-			if(!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerBeforeDeleteDocument', 'before')) return true;
-			if(!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerDeleteDocument', 'after')) return true;
-			if(!$oModuleModel->getTrigger('comment.insertComment', 'point', 'controller', 'triggerInsertComment', 'after')) return true;
-			if(!$oModuleModel->getTrigger('comment.deleteComment', 'point', 'controller', 'triggerDeleteComment', 'after')) return true;
-			if(!$oModuleModel->getTrigger('file.insertFile', 'point', 'controller', 'triggerInsertFile', 'after')) return true;
-			if(!$oModuleModel->getTrigger('file.deleteFile', 'point', 'controller', 'triggerDeleteFile', 'after')) return true;
-			if(!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerBeforeDownloadFile', 'before')) return true;
-			if(!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerDownloadFile', 'after')) return true;
-			if(!$oModuleModel->getTrigger('member.doLogin', 'point', 'controller', 'triggerAfterLogin', 'after')) return true;
-			if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'point', 'view', 'triggerDispPointAdditionSetup', 'after')) return true;
-			if(!$oModuleModel->getTrigger('document.updateReadedCount', 'point', 'controller', 'triggerUpdateReadedCount', 'after')) return true;
+			if(!$oModuleModel->getTrigger('member.insertMember', 'point', 'controller', 'triggerInsertMember', 'after')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('document.insertDocument', 'point', 'controller', 'triggerInsertDocument', 'after')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerBeforeDeleteDocument', 'before')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerDeleteDocument', 'after')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('comment.insertComment', 'point', 'controller', 'triggerInsertComment', 'after')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('comment.deleteComment', 'point', 'controller', 'triggerDeleteComment', 'after')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('file.insertFile', 'point', 'controller', 'triggerInsertFile', 'after')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('file.deleteFile', 'point', 'controller', 'triggerDeleteFile', 'after')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerBeforeDownloadFile', 'before')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerDownloadFile', 'after')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('member.doLogin', 'point', 'controller', 'triggerAfterLogin', 'after')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'point', 'view', 'triggerDispPointAdditionSetup', 'after')) {
+				return true;
+			}
+			if(!$oModuleModel->getTrigger('document.updateReadedCount', 'point', 'controller', 'triggerUpdateReadedCount', 'after')) {
+				return true;
+			}
 			// Add a trigger for voting up and down 2008.05.13 haneul
-			if(!$oModuleModel->getTrigger('document.updateVotedCount', 'point', 'controller', 'triggerUpdateVotedCount', 'after')) return true;
+			if(!$oModuleModel->getTrigger('document.updateVotedCount', 'point', 'controller', 'triggerUpdateVotedCount', 'after')) {
+				return true;
+			}
 			// Add a trigger for using points for permanent saving of a temporarily saved document 2009.05.19 zero
-			if(!$oModuleModel->getTrigger('document.updateDocument', 'point', 'controller', 'triggerUpdateDocument', 'before')) return true;
+			if(!$oModuleModel->getTrigger('document.updateDocument', 'point', 'controller', 'triggerUpdateDocument', 'before')) {
+				return true;
+			}
 
 			// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-			if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'point', 'controller', 'triggerCopyModule', 'after')) return true;
+			if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'point', 'controller', 'triggerCopyModule', 'after')) {
+				return true;
+			}
 		}
 
 		return false;
@@ -121,38 +153,53 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		// Add a trigger for registration/insert document/insert comment/upload a file/download
-		if(!$oModuleModel->getTrigger('member.insertMember', 'point', 'controller', 'triggerInsertMember', 'after')) 
-			$oModuleController->insertTrigger('member.insertMember', 'point', 'controller', 'triggerInsertMember', 'after');
-		if(!$oModuleModel->getTrigger('document.insertDocument', 'point', 'controller', 'triggerInsertDocument', 'after')) 
-			$oModuleController->insertTrigger('document.insertDocument', 'point', 'controller', 'triggerInsertDocument', 'after');
-		if(!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerBeforeDeleteDocument', 'before')) 
-			$oModuleController->insertTrigger('document.deleteDocument', 'point', 'controller', 'triggerBeforeDeleteDocument', 'before');
-		if(!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerDeleteDocument', 'after')) 
-			$oModuleController->insertTrigger('document.deleteDocument', 'point', 'controller', 'triggerDeleteDocument', 'after');
-		if(!$oModuleModel->getTrigger('comment.insertComment', 'point', 'controller', 'triggerInsertComment', 'after')) 
-			$oModuleController->insertTrigger('comment.insertComment', 'point', 'controller', 'triggerInsertComment', 'after');
-		if(!$oModuleModel->getTrigger('comment.deleteComment', 'point', 'controller', 'triggerDeleteComment', 'after')) 
-			$oModuleController->insertTrigger('comment.deleteComment', 'point', 'controller', 'triggerDeleteComment', 'after');
-		if(!$oModuleModel->getTrigger('file.insertFile', 'point', 'controller', 'triggerInsertFile', 'after')) 
-			$oModuleController->insertTrigger('file.insertFile', 'point', 'controller', 'triggerInsertFile', 'after');
-		if(!$oModuleModel->getTrigger('file.deleteFile', 'point', 'controller', 'triggerDeleteFile', 'after')) 
-			$oModuleController->insertTrigger('file.deleteFile', 'point', 'controller', 'triggerDeleteFile', 'after');
-		if(!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerBeforeDownloadFile', 'before')) 
-			$oModuleController->insertTrigger('file.downloadFile', 'point', 'controller', 'triggerBeforeDownloadFile', 'before');
-		if(!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerDownloadFile', 'after')) 
-			$oModuleController->insertTrigger('file.downloadFile', 'point', 'controller', 'triggerDownloadFile', 'after');
-		if(!$oModuleModel->getTrigger('member.doLogin', 'point', 'controller', 'triggerAfterLogin', 'after'))
-			$oModuleController->insertTrigger('member.doLogin', 'point', 'controller', 'triggerAfterLogin', 'after');
-		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'point', 'view', 'triggerDispPointAdditionSetup', 'after')) 
-			$oModuleController->insertTrigger('module.dispAdditionSetup', 'point', 'view', 'triggerDispPointAdditionSetup', 'after');
-		if(!$oModuleModel->getTrigger('document.updateReadedCount', 'point', 'controller', 'triggerUpdateReadedCount', 'after')) 
-			$oModuleController->insertTrigger('document.updateReadedCount', 'point', 'controller', 'triggerUpdateReadedCount', 'after');
+		if(!$oModuleModel->getTrigger('member.insertMember', 'point', 'controller', 'triggerInsertMember', 'after')) {
+					$oModuleController->insertTrigger('member.insertMember', 'point', 'controller', 'triggerInsertMember', 'after');
+		}
+		if(!$oModuleModel->getTrigger('document.insertDocument', 'point', 'controller', 'triggerInsertDocument', 'after')) {
+					$oModuleController->insertTrigger('document.insertDocument', 'point', 'controller', 'triggerInsertDocument', 'after');
+		}
+		if(!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerBeforeDeleteDocument', 'before')) {
+					$oModuleController->insertTrigger('document.deleteDocument', 'point', 'controller', 'triggerBeforeDeleteDocument', 'before');
+		}
+		if(!$oModuleModel->getTrigger('document.deleteDocument', 'point', 'controller', 'triggerDeleteDocument', 'after')) {
+					$oModuleController->insertTrigger('document.deleteDocument', 'point', 'controller', 'triggerDeleteDocument', 'after');
+		}
+		if(!$oModuleModel->getTrigger('comment.insertComment', 'point', 'controller', 'triggerInsertComment', 'after')) {
+					$oModuleController->insertTrigger('comment.insertComment', 'point', 'controller', 'triggerInsertComment', 'after');
+		}
+		if(!$oModuleModel->getTrigger('comment.deleteComment', 'point', 'controller', 'triggerDeleteComment', 'after')) {
+					$oModuleController->insertTrigger('comment.deleteComment', 'point', 'controller', 'triggerDeleteComment', 'after');
+		}
+		if(!$oModuleModel->getTrigger('file.insertFile', 'point', 'controller', 'triggerInsertFile', 'after')) {
+					$oModuleController->insertTrigger('file.insertFile', 'point', 'controller', 'triggerInsertFile', 'after');
+		}
+		if(!$oModuleModel->getTrigger('file.deleteFile', 'point', 'controller', 'triggerDeleteFile', 'after')) {
+					$oModuleController->insertTrigger('file.deleteFile', 'point', 'controller', 'triggerDeleteFile', 'after');
+		}
+		if(!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerBeforeDownloadFile', 'before')) {
+					$oModuleController->insertTrigger('file.downloadFile', 'point', 'controller', 'triggerBeforeDownloadFile', 'before');
+		}
+		if(!$oModuleModel->getTrigger('file.downloadFile', 'point', 'controller', 'triggerDownloadFile', 'after')) {
+					$oModuleController->insertTrigger('file.downloadFile', 'point', 'controller', 'triggerDownloadFile', 'after');
+		}
+		if(!$oModuleModel->getTrigger('member.doLogin', 'point', 'controller', 'triggerAfterLogin', 'after')) {
+					$oModuleController->insertTrigger('member.doLogin', 'point', 'controller', 'triggerAfterLogin', 'after');
+		}
+		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'point', 'view', 'triggerDispPointAdditionSetup', 'after')) {
+					$oModuleController->insertTrigger('module.dispAdditionSetup', 'point', 'view', 'triggerDispPointAdditionSetup', 'after');
+		}
+		if(!$oModuleModel->getTrigger('document.updateReadedCount', 'point', 'controller', 'triggerUpdateReadedCount', 'after')) {
+					$oModuleController->insertTrigger('document.updateReadedCount', 'point', 'controller', 'triggerUpdateReadedCount', 'after');
+		}
 		// Add a trigger for voting up and down 2008.05.13 haneul
-		if(!$oModuleModel->getTrigger('document.updateVotedCount', 'point', 'controller', 'triggerUpdateVotedCount', 'after'))
-			$oModuleController->insertTrigger('document.updateVotedCount', 'point', 'controller', 'triggerUpdateVotedCount', 'after');
+		if(!$oModuleModel->getTrigger('document.updateVotedCount', 'point', 'controller', 'triggerUpdateVotedCount', 'after')) {
+					$oModuleController->insertTrigger('document.updateVotedCount', 'point', 'controller', 'triggerUpdateVotedCount', 'after');
+		}
 		// Add a trigger for using points for permanent saving of a temporarily saved document 2009.05.19 zero
-		if(!$oModuleModel->getTrigger('document.updateDocument', 'point', 'controller', 'triggerUpdateDocument', 'before')) 
-			$oModuleController->insertTrigger('document.updateDocument', 'point', 'controller', 'triggerUpdateDocument', 'before');
+		if(!$oModuleModel->getTrigger('document.updateDocument', 'point', 'controller', 'triggerUpdateDocument', 'before')) {
+					$oModuleController->insertTrigger('document.updateDocument', 'point', 'controller', 'triggerUpdateDocument', 'before');
+		}
 		// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
 		if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'point', 'controller', 'triggerCopyModule', 'after'))
 		{

modules/poll/poll.admin.model.php

Spacing +5 added lines, -5 removed lines

@@ -20,7 +20,7 @@ 
 	function getPollList($args)
 	{
 		$output = executeQueryArray('poll.getPollList', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		//if($output->data && !is_array($output->data)) $output->data = array($output->data);
 		return $output;
@@ -32,7 +32,7 @@ 
 	function getPollListWithMember($args)
 	{
 		$output = executeQueryArray('poll.getPollListWithMember', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		return $output;
 	}
@@ -50,14 +50,14 @@ 
 
 		$oDocument = $oDocumentModel->getDocument($upload_target_srl);
 
-		if(!$oDocument->isExists()) $oComment = $oCommentModel->getComment($upload_target_srl);
+		if (!$oDocument->isExists()) $oComment = $oCommentModel->getComment($upload_target_srl);
 
-		if($oComment && $oComment->isExists())
+		if ($oComment && $oComment->isExists())
 		{
 			$this->add('document_srl', $oComment->get('document_srl'));
 			$this->add('comment_srl', $oComment->get('comment_srl'));
 		}
-		elseif($oDocument->isExists())
+		elseif ($oDocument->isExists())
 		{
 			$this->add('document_srl', $oDocument->get('document_srl'));
 		}

Braces +12 added lines, -6 removed lines

@@ -20,7 +20,9 @@ 
 	function getPollList($args)
 	{
 		$output = executeQueryArray('poll.getPollList', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		//if($output->data && !is_array($output->data)) $output->data = array($output->data);
 		return $output;
@@ -32,7 +34,9 @@ 
 	function getPollListWithMember($args)
 	{
 		$output = executeQueryArray('poll.getPollListWithMember', $args);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		return $output;
 	}
@@ -50,18 +54,20 @@ 
 
 		$oDocument = $oDocumentModel->getDocument($upload_target_srl);
 
-		if(!$oDocument->isExists()) $oComment = $oCommentModel->getComment($upload_target_srl);
+		if(!$oDocument->isExists()) {
+			$oComment = $oCommentModel->getComment($upload_target_srl);
+		}
 
 		if($oComment && $oComment->isExists())
 		{
 			$this->add('document_srl', $oComment->get('document_srl'));
 			$this->add('comment_srl', $oComment->get('comment_srl'));
-		}
-		elseif($oDocument->isExists())
+		} elseif($oDocument->isExists())
 		{
 			$this->add('document_srl', $oDocument->get('document_srl'));
+		} else {
+			return new BaseObject(-1, 'msg_not_founded');
 		}
-		else return new BaseObject(-1, 'msg_not_founded');
 	}
 }
 /* End of file poll.admin.model.php */

modules/poll/poll.admin.view.php

Braces +18 added lines, -7 removed lines

@@ -29,7 +29,9 @@ 
 			switch($search_target)
 			{
 				case 'title' :
-					if($search_keyword) $search_keyword = str_replace(' ','%',$search_keyword);
+					if($search_keyword) {
+						$search_keyword = str_replace(' ','%',$search_keyword);
+					}
 					$args->s_title= $search_keyword;
 					break;
 				case 'regdate' :
@@ -62,17 +64,22 @@ 
 
 			$oDocumentModel = getModel('document');
 			$targetDocumentOutput = $oDocumentModel->getDocuments($uploadTargetSrlList);
-			if(!is_array($targetDocumentOutput)) $targetDocumentOutput = array();
+			if(!is_array($targetDocumentOutput)) {
+				$targetDocumentOutput = array();
+			}
 
 			$oCommentModel = getModel('comment');
 			$columnList = array('comment_srl', 'document_srl');
 			$targetCommentOutput = $oCommentModel->getComments($uploadTargetSrlList, $columnList);
-			if(!is_array($targetCommentOutput)) $targetCommentOutput = array();
+			if(!is_array($targetCommentOutput)) {
+				$targetCommentOutput = array();
+			}
 
 			foreach($output->data as $value)
 			{
-				if(array_key_exists($value->upload_target_srl, $targetDocumentOutput))
-					$value->document_srl = $value->upload_target_srl;
+				if(array_key_exists($value->upload_target_srl, $targetDocumentOutput)) {
+									$value->document_srl = $value->upload_target_srl;
+				}
 
 				if(array_key_exists($value->upload_target_srl, $targetCommentOutput))
 				{
@@ -110,7 +117,9 @@ 
 		$skin_list = $oModuleModel->getSkins($this->module_path);
 		Context::set('skin_list', $skin_list);
 
-		if(!$skin_list[$config->skin]) $config->skin = "default";
+		if(!$skin_list[$config->skin]) {
+			$config->skin = "default";
+		}
 		// Set the skin colorset once the configurations is completed
 		Context::set('colorset_list', $skin_list[$config->skin]->colorset);
 
@@ -137,7 +146,9 @@ 
 		$args->poll_index_srl = Context::get('poll_index_srl');
 
 		$output = executeQuery('poll.getPoll', $args);
-		if(!$output->data) return $this->stop('msg_poll_not_exists');
+		if(!$output->data) {
+			return $this->stop('msg_poll_not_exists');
+		}
 
 		$poll = new stdClass();
 		$poll->stop_date = $output->data->stop_date;

Spacing +18 added lines, -18 removed lines

@@ -24,19 +24,19 @@ 
 		$search_keyword = trim(Context::get('search_keyword'));
 
 		$args = new stdClass();
-		if($search_target && $search_keyword)
+		if ($search_target && $search_keyword)
 		{
-			switch($search_target)
+			switch ($search_target)
 			{
 				case 'title' :
-					if($search_keyword) $search_keyword = str_replace(' ','%',$search_keyword);
-					$args->s_title= $search_keyword;
+					if ($search_keyword) $search_keyword = str_replace(' ', '%', $search_keyword);
+					$args->s_title = $search_keyword;
 					break;
 				case 'regdate' :
 					$args->s_regdate = $search_keyword;
 					break;
 				case 'ipaddress' :
-					$args->s_ipaddress= $search_keyword;
+					$args->s_ipaddress = $search_keyword;
 					break;
 			}
 		}
@@ -52,29 +52,29 @@ 
 		$output = $oPollAdminModel->getPollListWithMember($args);
 
 		// check poll type. document or comment
-		if(is_array($output->data))
+		if (is_array($output->data))
 		{
 			$uploadTargetSrlList = array();
-			foreach($output->data as $value)
+			foreach ($output->data as $value)
 			{
 				$uploadTargetSrlList[] = $value->upload_target_srl;
 			}
 
 			$oDocumentModel = getModel('document');
 			$targetDocumentOutput = $oDocumentModel->getDocuments($uploadTargetSrlList);
-			if(!is_array($targetDocumentOutput)) $targetDocumentOutput = array();
+			if (!is_array($targetDocumentOutput)) $targetDocumentOutput = array();
 
 			$oCommentModel = getModel('comment');
 			$columnList = array('comment_srl', 'document_srl');
 			$targetCommentOutput = $oCommentModel->getComments($uploadTargetSrlList, $columnList);
-			if(!is_array($targetCommentOutput)) $targetCommentOutput = array();
+			if (!is_array($targetCommentOutput)) $targetCommentOutput = array();
 
-			foreach($output->data as $value)
+			foreach ($output->data as $value)
 			{
-				if(array_key_exists($value->upload_target_srl, $targetDocumentOutput))
+				if (array_key_exists($value->upload_target_srl, $targetDocumentOutput))
 					$value->document_srl = $value->upload_target_srl;
 
-				if(array_key_exists($value->upload_target_srl, $targetCommentOutput))
+				if (array_key_exists($value->upload_target_srl, $targetCommentOutput))
 				{
 					$value->comment_srl = $value->upload_target_srl;
 					$value->document_srl = $targetCommentOutput[$value->comment_srl]->document_srl;
@@ -110,14 +110,14 @@ 
 		$skin_list = $oModuleModel->getSkins($this->module_path);
 		Context::set('skin_list', $skin_list);
 
-		if(!$skin_list[$config->skin]) $config->skin = "default";
+		if (!$skin_list[$config->skin]) $config->skin = "default";
 		// Set the skin colorset once the configurations is completed
 		Context::set('colorset_list', $skin_list[$config->skin]->colorset);
 
 		$security = new Security();
 		$security->encodeHTML('config..');
 		$security->encodeHTML('skin_list..title');
-		$security->encodeHTML('colorset_list..name','colorset_list..title');
+		$security->encodeHTML('colorset_list..name', 'colorset_list..title');
 
 		// Set a template
 		$this->setTemplatePath($this->module_path.'tpl');
@@ -137,14 +137,14 @@ 
 		$args->poll_index_srl = Context::get('poll_index_srl');
 
 		$output = executeQuery('poll.getPoll', $args);
-		if(!$output->data) return $this->stop('msg_poll_not_exists');
+		if (!$output->data) return $this->stop('msg_poll_not_exists');
 
 		$poll = new stdClass();
 		$poll->stop_date = $output->data->stop_date;
 		$poll->poll_count = $output->data->poll_count;
 
 		$output = executeQuery('poll.getPollTitle', $args);
-		if(!$output->data)
+		if (!$output->data)
 		{
 			return $this->stop('msg_poll_not_exists');
 		}
@@ -155,14 +155,14 @@ 
 		$tmp->poll_count = $output->data->poll_count;
 
 		$output = executeQuery('poll.getPollItem', $args);
-		foreach($output->data as $val)
+		foreach ($output->data as $val)
 		{
 			$tmp->item[] = $val;
 		}
 
 		$poll->poll_srl = $poll_srl;
 
-		Context::set('poll',$poll);
+		Context::set('poll', $poll);
 		// Configure the skin and the colorset for the default configuration
 		$oModuleModel = getModel('module');
 		$poll_config = $oModuleModel->getModuleConfig('poll');