xpressengine / xe-core

modules/document/document.item.php

Spacing +184 added lines, -184 removed lines

@@ -79,7 +79,7 @@ 
 	 */
 	function _loadFromDB($load_extra_vars = true)
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 
 		$document_item = false;
 		$cache_put = false;
@@ -88,11 +88,11 @@ 
 
 		// cache controll
 		$oCacheHandler = CacheHandler::getInstance('object');
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
-			$cache_key = 'document_item:' . getNumberingPath($this->document_srl) . $this->document_srl;
+			$cache_key = 'document_item:'.getNumberingPath($this->document_srl).$this->document_srl;
 			$document_item = $oCacheHandler->get($cache_key);
-			if($document_item !== false)
+			if ($document_item !== false)
 			{
 				$columnList = array('readed_count', 'voted_count', 'blamed_count', 'comment_count', 'trackback_count');
 			}
@@ -102,12 +102,12 @@ 
 		$args->document_srl = $this->document_srl;
 		$output = executeQuery('document.getDocument', $args, $columnList);
 
-		if($document_item === false)
+		if ($document_item === false)
 		{
 			$document_item = $output->data;
 
 				//insert in cache
-			if($document_item && $oCacheHandler->isSupport())
+			if ($document_item && $oCacheHandler->isSupport())
 			{
 				$oCacheHandler->put($cache_key, $document_item);
 			}
@@ -124,9 +124,9 @@ 
 		$this->setAttribute($document_item, $load_extra_vars);
 	}
 
-	function setAttribute($attribute, $load_extra_vars=true)
+	function setAttribute($attribute, $load_extra_vars = true)
 	{
-		if(!$attribute->document_srl)
+		if (!$attribute->document_srl)
 		{
 			$this->document_srl = null;
 			return;
@@ -136,7 +136,7 @@ 
 		$this->adds($attribute);
 
 		// Tags
-		if($this->get('tags'))
+		if ($this->get('tags'))
 		{
 			$tag_list = explode(',', $this->get('tags'));
 			$tag_list = array_map('trim', $tag_list);
@@ -144,7 +144,7 @@ 
 		}
 
 		$oDocumentModel = getModel('document');
-		if($load_extra_vars)
+		if ($load_extra_vars)
 		{
 			$GLOBALS['XE_DOCUMENT_LIST'][$attribute->document_srl] = $this;
 			$oDocumentModel->setToAllDocumentExtraVars();
@@ -159,23 +159,23 @@ 
 
 	function isGranted()
 	{
-		if($_SESSION['own_document'][$this->document_srl]) return $this->grant_cache = true;
+		if ($_SESSION['own_document'][$this->document_srl]) return $this->grant_cache = true;
 
-		if($this->grant_cache !== null)
+		if ($this->grant_cache !== null)
 		{
 			return $this->grant_cache;
 		}
 
-		if(!Context::get('is_logged')) return $this->grant_cache = false;
+		if (!Context::get('is_logged')) return $this->grant_cache = false;
 
 		$logged_info = Context::get('logged_info');
-		if($logged_info->is_admin == 'Y') return $this->grant_cache = true;
+		if ($logged_info->is_admin == 'Y') return $this->grant_cache = true;
 
 		$oModuleModel = getModel('module');
 		$grant = $oModuleModel->getGrant($oModuleModel->getModuleInfoByModuleSrl($this->get('module_srl')), $logged_info);
-		if($grant->manager) return $this->grant_cache = true;
+		if ($grant->manager) return $this->grant_cache = true;
 
-		if($this->get('member_srl') && abs($this->get('member_srl')) == $logged_info->member_srl)
+		if ($this->get('member_srl') && abs($this->get('member_srl')) == $logged_info->member_srl)
 		{
 			return $this->grant_cache = true;
 		}
@@ -191,13 +191,13 @@ 
 
 	function isAccessible()
 	{
-		return $_SESSION['accessible'][$this->document_srl]==true?true:false;
+		return $_SESSION['accessible'][$this->document_srl] == true ?true:false;
 	}
 
 	function allowComment()
 	{
 		// init write, document is not exists. so allow comment status is true
-		if(!$this->isExists()) return true;
+		if (!$this->isExists()) return true;
 
 		return $this->get('comment_status') == 'ALLOW' ? true : false;
 	}
@@ -205,11 +205,11 @@ 
 	function allowTrackback()
 	{
 		static $allow_trackback_status = null;
-		if(is_null($allow_trackback_status))
+		if (is_null($allow_trackback_status))
 		{
 			
 			// Check the tarckback module exist
-			if(!getClass('trackback'))
+			if (!getClass('trackback'))
 			{
 				$allow_trackback_status = false;
 			}
@@ -219,20 +219,20 @@ 
 				$oModuleModel = getModel('module');
 				$trackback_config = $oModuleModel->getModuleConfig('trackback');
 				
-				if(!$trackback_config)
+				if (!$trackback_config)
 				{
 					$trackback_config = new stdClass();
 				}
 				
-				if(!isset($trackback_config->enable_trackback)) $trackback_config->enable_trackback = 'Y';
-				if($trackback_config->enable_trackback != 'Y') $allow_trackback_status = false;
+				if (!isset($trackback_config->enable_trackback)) $trackback_config->enable_trackback = 'Y';
+				if ($trackback_config->enable_trackback != 'Y') $allow_trackback_status = false;
 				else
 				{
 					$module_srl = $this->get('module_srl');
 					// Check settings of each module
 					$module_config = $oModuleModel->getModulePartConfig('trackback', $module_srl);
-					if($module_config->enable_trackback == 'N') $allow_trackback_status = false;
-					else if($this->get('allow_trackback')=='Y' || !$this->isExists()) $allow_trackback_status = true;
+					if ($module_config->enable_trackback == 'N') $allow_trackback_status = false;
+					else if ($this->get('allow_trackback') == 'Y' || !$this->isExists()) $allow_trackback_status = true;
 				}
 			}
 		}
@@ -241,14 +241,14 @@ 
 
 	function isLocked()
 	{
-		if(!$this->isExists()) return false;
+		if (!$this->isExists()) return false;
 
 		return $this->get('comment_status') == 'ALLOW' ? false : true;
 	}
 
 	function isEditable()
 	{
-		if($this->isGranted() || !$this->get('member_srl')) return true;
+		if ($this->isGranted() || !$this->get('member_srl')) return true;
 		return false;
 	}
 
@@ -265,13 +265,13 @@ 
 
 	function useNotify()
 	{
-		return $this->get('notify_message')=='Y' ? true : false;
+		return $this->get('notify_message') == 'Y' ? true : false;
 	}
 
 	function doCart()
 	{
-		if(!$this->document_srl) return false;
-		if($this->isCarted()) $this->removeCart();
+		if (!$this->document_srl) return false;
+		if ($this->isCarted()) $this->removeCart();
 		else $this->addCart();
 	}
 
@@ -298,18 +298,18 @@ 
 	 */
 	function notify($type, $content)
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 		// return if it is not useNotify
-		if(!$this->useNotify()) return;
+		if (!$this->useNotify()) return;
 		// Pass if an author is not a logged-in user
-		if(!$this->get('member_srl')) return;
+		if (!$this->get('member_srl')) return;
 		// Return if the currently logged-in user is an author
 		$logged_info = Context::get('logged_info');
-		if($logged_info->member_srl == $this->get('member_srl')) return;
+		if ($logged_info->member_srl == $this->get('member_srl')) return;
 		// List variables
-		if($type) $title = "[".$type."] ";
+		if ($type) $title = "[".$type."] ";
 		$title .= cut_str(strip_tags($content), 10, '...');
-		$content = sprintf('%s<br /><br />from : <a href="%s" target="_blank">%s</a>',$content, getFullUrl('','document_srl',$this->document_srl), getFullUrl('','document_srl',$this->document_srl));
+		$content = sprintf('%s<br /><br />from : <a href="%s" target="_blank">%s</a>', $content, getFullUrl('', 'document_srl', $this->document_srl), getFullUrl('', 'document_srl', $this->document_srl));
 		$receiver_srl = $this->get('member_srl');
 		$sender_member_srl = $logged_info->member_srl;
 		// Send a message
@@ -324,26 +324,26 @@ 
 
 	function getIpAddress()
 	{
-		if($this->isGranted())
+		if ($this->isGranted())
 		{
 			return $this->get('ipaddress');
 		}
 
-		return '*' . strstr($this->get('ipaddress'), '.');
+		return '*'.strstr($this->get('ipaddress'), '.');
 	}
 
 	function isExistsHomepage()
 	{
-		if(trim($this->get('homepage'))) return true;
+		if (trim($this->get('homepage'))) return true;
 		return false;
 	}
 
 	function getHomepageUrl()
 	{
 		$url = trim($this->get('homepage'));
-		if(!$url) return;
+		if (!$url) return;
 
-		if(strncasecmp('http://', $url, 7) !== 0 && strncasecmp('https://', $url, 8) !== 0)  $url = 'http://' . $url;
+		if (strncasecmp('http://', $url, 7) !== 0 && strncasecmp('https://', $url, 8) !== 0)  $url = 'http://'.$url;
 
 		return $url;
 	}
@@ -373,52 +373,52 @@ 
 		return htmlspecialchars($this->get('last_updater'), ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 	}
 
-	function getTitleText($cut_size = 0, $tail='...')
+	function getTitleText($cut_size = 0, $tail = '...')
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 
-		if($cut_size) $title = cut_str($this->get('title'), $cut_size, $tail);
+		if ($cut_size) $title = cut_str($this->get('title'), $cut_size, $tail);
 		else $title = $this->get('title');
 
 		return $title;
 	}
 
-	function getTitle($cut_size = 0, $tail='...')
+	function getTitle($cut_size = 0, $tail = '...')
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 
 		$title = $this->getTitleText($cut_size, $tail);
 
 		$attrs = array();
 		$this->add('title_color', trim($this->get('title_color')));
-		if($this->get('title_bold')=='Y') $attrs[] = "font-weight:bold;";
-		if($this->get('title_color') && $this->get('title_color') != 'N') $attrs[] = "color:#".$this->get('title_color');
+		if ($this->get('title_bold') == 'Y') $attrs[] = "font-weight:bold;";
+		if ($this->get('title_color') && $this->get('title_color') != 'N') $attrs[] = "color:#".$this->get('title_color');
 
-		if(count($attrs)) return sprintf("<span style=\"%s\">%s</span>", implode(';',$attrs), htmlspecialchars($title, ENT_COMPAT | ENT_HTML401, 'UTF-8', false));
+		if (count($attrs)) return sprintf("<span style=\"%s\">%s</span>", implode(';', $attrs), htmlspecialchars($title, ENT_COMPAT | ENT_HTML401, 'UTF-8', false));
 		else return htmlspecialchars($title, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
 	}
 
 	function getContentText($strlen = 0)
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 
-		if($this->isSecret() && !$this->isGranted() && !$this->isAccessible()) return Context::getLang('msg_is_secret');
+		if ($this->isSecret() && !$this->isGranted() && !$this->isAccessible()) return Context::getLang('msg_is_secret');
 
 		$result = $this->_checkAccessibleFromStatus();
-		if($result) $_SESSION['accessible'][$this->document_srl] = true;
+		if ($result) $_SESSION['accessible'][$this->document_srl] = true;
 
 		$content = $this->get('content');
 		$content = preg_replace_callback('/<(object|param|embed)[^>]*/is', array($this, '_checkAllowScriptAccess'), $content);
 		$content = preg_replace_callback('/<object[^>]*>/is', array($this, '_addAllowScriptAccess'), $content);
 
-		if($strlen) return cut_str(strip_tags($content),$strlen,'...');
+		if ($strlen) return cut_str(strip_tags($content), $strlen, '...');
 
 		return htmlspecialchars($content);
 	}
 
 	function _addAllowScriptAccess($m)
 	{
-		if($this->allowscriptaccessList[$this->allowscriptaccessKey] == 1)
+		if ($this->allowscriptaccessList[$this->allowscriptaccessKey] == 1)
 		{
 			$m[0] = $m[0].'<param name="allowscriptaccess" value="never"></param>';
 		}
@@ -428,26 +428,26 @@ 
 
 	function _checkAllowScriptAccess($m)
 	{
-		if($m[1] == 'object')
+		if ($m[1] == 'object')
 		{
 			$this->allowscriptaccessList[] = 1;
 		}
 
-		if($m[1] == 'param')
+		if ($m[1] == 'param')
 		{
-			if(stripos($m[0], 'allowscriptaccess'))
+			if (stripos($m[0], 'allowscriptaccess'))
 			{
 				$m[0] = '<param name="allowscriptaccess" value="never"';
-				if(substr($m[0], -1) == '/')
+				if (substr($m[0], -1) == '/')
 				{
 					$m[0] .= '/';
 				}
-				$this->allowscriptaccessList[count($this->allowscriptaccessList)-1]--;
+				$this->allowscriptaccessList[count($this->allowscriptaccessList) - 1]--;
 			}
 		}
-		else if($m[1] == 'embed')
+		else if ($m[1] == 'embed')
 		{
-			if(stripos($m[0], 'allowscriptaccess'))
+			if (stripos($m[0], 'allowscriptaccess'))
 			{
 				$m[0] = preg_replace('/always|samedomain/i', 'never', $m[0]);
 			}
@@ -461,24 +461,24 @@ 
 
 	function getContent($add_popup_menu = true, $add_content_info = true, $resource_realpath = false, $add_xe_content_class = true, $stripEmbedTagException = false)
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 
-		if($this->isSecret() && !$this->isGranted() && !$this->isAccessible()) return Context::getLang('msg_is_secret');
+		if ($this->isSecret() && !$this->isGranted() && !$this->isAccessible()) return Context::getLang('msg_is_secret');
 
 		$result = $this->_checkAccessibleFromStatus();
-		if($result) $_SESSION['accessible'][$this->document_srl] = true;
+		if ($result) $_SESSION['accessible'][$this->document_srl] = true;
 
 		$content = $this->get('content');
-		if(!$stripEmbedTagException) stripEmbedTagForAdmin($content, $this->get('member_srl'));
+		if (!$stripEmbedTagException) stripEmbedTagForAdmin($content, $this->get('member_srl'));
 
 		// Define a link if using a rewrite module
 		$oContext = &Context::getInstance();
-		if($oContext->allow_rewrite)
+		if ($oContext->allow_rewrite)
 		{
-			$content = preg_replace('/<a([ \t]+)href=("|\')\.\/\?/i',"<a href=\\2". Context::getRequestUri() ."?", $content);
+			$content = preg_replace('/<a([ \t]+)href=("|\')\.\/\?/i', "<a href=\\2".Context::getRequestUri()."?", $content);
 		}
 		// To display a pop-up menu
-		if($add_popup_menu)
+		if ($add_popup_menu)
 		{
 			$content = sprintf(
 				'%s<div class="document_popup_menu"><a href="#popup_menu_area" class="document_%d" onclick="return false">%s</a></div>',
@@ -487,10 +487,10 @@ 
 			);
 		}
 		// If additional content information is set
-		if($add_content_info)
+		if ($add_content_info)
 		{
 			$memberSrl = $this->get('member_srl');
-			if($memberSrl < 0)
+			if ($memberSrl < 0)
 			{
 				$memberSrl = 0;
 			}
@@ -506,12 +506,12 @@ 
 		}
 		else
 		{
-			if($add_xe_content_class) $content = sprintf('<div class="xe_content">%s</div>', $content);
+			if ($add_xe_content_class) $content = sprintf('<div class="xe_content">%s</div>', $content);
 		}
 		// Change the image path to a valid absolute path if resource_realpath is true
-		if($resource_realpath)
+		if ($resource_realpath)
 		{
-			$content = preg_replace_callback('/<img([^>]+)>/i',array($this,'replaceResourceRealPath'), $content);
+			$content = preg_replace_callback('/<img([^>]+)>/i', array($this, 'replaceResourceRealPath'), $content);
 		}
 
 		return $content;
@@ -560,7 +560,7 @@ 
 		$content = trim(cut_str($content, $str_size, $tail));
 
 		// Replace back < , <, "
-		$content = str_replace(array('<', '>', '"'),array('&lt;', '&gt;', '&quot;'), $content);
+		$content = str_replace(array('<', '>', '"'), array('&lt;', '&gt;', '&quot;'), $content);
 
 		return $content;
 	}
@@ -573,13 +573,13 @@ 
 	function getRegdateTime()
 	{
 		$regdate = $this->get('regdate');
-		$year = substr($regdate,0,4);
-		$month = substr($regdate,4,2);
-		$day = substr($regdate,6,2);
-		$hour = substr($regdate,8,2);
-		$min = substr($regdate,10,2);
-		$sec = substr($regdate,12,2);
-		return mktime($hour,$min,$sec,$month,$day,$year);
+		$year = substr($regdate, 0, 4);
+		$month = substr($regdate, 4, 2);
+		$day = substr($regdate, 6, 2);
+		$hour = substr($regdate, 8, 2);
+		$min = substr($regdate, 10, 2);
+		$sec = substr($regdate, 12, 2);
+		return mktime($hour, $min, $sec, $month, $day, $year);
 	}
 
 	function getRegdateGM()
@@ -589,7 +589,7 @@ 
 
 	function getRegdateDT()
 	{
-		return $this->getRegdate('Y-m-d').'T'.$this->getRegdate('H:i:s').substr($GLOBALS['_time_zone'],0,3).':'.substr($GLOBALS['_time_zone'],3,2);
+		return $this->getRegdate('Y-m-d').'T'.$this->getRegdate('H:i:s').substr($GLOBALS['_time_zone'], 0, 3).':'.substr($GLOBALS['_time_zone'], 3, 2);
 	}
 
 	function getUpdate($format = 'Y.m.d H:i:s')
@@ -599,13 +599,13 @@ 
 
 	function getUpdateTime()
 	{
-		$year = substr($this->get('last_update'),0,4);
-		$month = substr($this->get('last_update'),4,2);
-		$day = substr($this->get('last_update'),6,2);
-		$hour = substr($this->get('last_update'),8,2);
-		$min = substr($this->get('last_update'),10,2);
-		$sec = substr($this->get('last_update'),12,2);
-		return mktime($hour,$min,$sec,$month,$day,$year);
+		$year = substr($this->get('last_update'), 0, 4);
+		$month = substr($this->get('last_update'), 4, 2);
+		$day = substr($this->get('last_update'), 6, 2);
+		$hour = substr($this->get('last_update'), 8, 2);
+		$min = substr($this->get('last_update'), 10, 2);
+		$sec = substr($this->get('last_update'), 12, 2);
+		return mktime($hour, $min, $sec, $month, $day, $year);
 	}
 
 	function getUpdateGM()
@@ -615,21 +615,21 @@ 
 
 	function getUpdateDT()
 	{
-		return $this->getUpdate('Y-m-d').'T'.$this->getUpdate('H:i:s').substr($GLOBALS['_time_zone'],0,3).':'.substr($GLOBALS['_time_zone'],3,2);
+		return $this->getUpdate('Y-m-d').'T'.$this->getUpdate('H:i:s').substr($GLOBALS['_time_zone'], 0, 3).':'.substr($GLOBALS['_time_zone'], 3, 2);
 	}
 
 	function getPermanentUrl()
 	{
-		return getFullUrl('','mid', $this->getDocumentMid('document_srl'), 'document_srl', $this->get('document_srl'));
+		return getFullUrl('', 'mid', $this->getDocumentMid('document_srl'), 'document_srl', $this->get('document_srl'));
 	}
 
 	function getTrackbackUrl()
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 
 		// Generate a key to prevent spams
 		$oTrackbackModel = getModel('trackback');
-		if($oTrackbackModel) return $oTrackbackModel->getTrackbackUrl($this->document_srl, $this->getDocumentMid());
+		if ($oTrackbackModel) return $oTrackbackModel->getTrackbackUrl($this->document_srl, $this->getDocumentMid());
 	}
 
 	/**
@@ -639,24 +639,24 @@ 
 	function updateReadedCount()
 	{
 		$oDocumentController = getController('document');
-		if($oDocumentController->updateReadedCount($this))
+		if ($oDocumentController->updateReadedCount($this))
 		{
 			$readed_count = $this->get('readed_count');
-			$this->add('readed_count', $readed_count+1);
+			$this->add('readed_count', $readed_count + 1);
 		}
 	}
 
 	function isExtraVarsExists()
 	{
-		if(!$this->get('module_srl')) return false;
+		if (!$this->get('module_srl')) return false;
 		$oDocumentModel = getModel('document');
 		$extra_keys = $oDocumentModel->getExtraKeys($this->get('module_srl'));
-		return count($extra_keys)?true:false;
+		return count($extra_keys) ?true:false;
 	}
 
 	function getExtraVars()
 	{
-		if(!$this->get('module_srl') || !$this->document_srl) return null;
+		if (!$this->get('module_srl') || !$this->document_srl) return null;
 
 		$oDocumentModel = getModel('document');
 		return $oDocumentModel->getExtraVars($this->get('module_srl'), $this->document_srl);
@@ -665,7 +665,7 @@ 
 	function getExtraValue($idx)
 	{
 		$extra_vars = $this->getExtraVars();
-		if(is_array($extra_vars) && array_key_exists($idx,$extra_vars))
+		if (is_array($extra_vars) && array_key_exists($idx, $extra_vars))
 		{
 			return $extra_vars[$idx]->getValue();
 		}
@@ -678,7 +678,7 @@ 
 	function getExtraValueHTML($idx)
 	{
 		$extra_vars = $this->getExtraVars();
-		if(is_array($extra_vars) && array_key_exists($idx,$extra_vars))
+		if (is_array($extra_vars) && array_key_exists($idx, $extra_vars))
 		{
 			return $extra_vars[$idx]->getValueHTML();
 		}
@@ -692,16 +692,16 @@ 
 	{
 		$extra_vars = $this->getExtraVars();
 
-		if($extra_vars)
+		if ($extra_vars)
 		{
 			// Handle extra variable(eid)
-			foreach($extra_vars as $idx => $key)
+			foreach ($extra_vars as $idx => $key)
 			{
 				$extra_eid[$key->eid] = $key;
 			}
 		}
 		
-		if(is_array($extra_eid) && array_key_exists($eid,$extra_eid))
+		if (is_array($extra_eid) && array_key_exists($eid, $extra_eid))
 		{
 			return $extra_eid[$eid]->getValue();
 		}
@@ -715,12 +715,12 @@ 
 	{
 		$extra_vars = $this->getExtraVars();
 		// Handle extra variable(eid)
-		foreach($extra_vars as $idx => $key)
+		foreach ($extra_vars as $idx => $key)
 		{
 			$extra_eid[$key->eid] = $key;
 		}
 		
-		if(is_array($extra_eid) && array_key_exists($eid,$extra_eid))
+		if (is_array($extra_eid) && array_key_exists($eid, $extra_eid))
 		{
 			return $extra_eid[$eid]->getValueHTML();
 		}
@@ -744,13 +744,13 @@ 
 
 	function getComments()
 	{
-		if(!$this->getCommentCount()) return;
-		if(!$this->isGranted() && $this->isSecret()) return;
+		if (!$this->getCommentCount()) return;
+		if (!$this->isGranted() && $this->isSecret()) return;
 		// cpage is a number of comment pages
 		$cpageStr = sprintf('%d_cpage', $this->document_srl);
 		$cpage = Context::get($cpageStr);
 
-		if(!$cpage)
+		if (!$cpage)
 		{
 			$cpage = Context::get('cpage');
 		}
@@ -758,19 +758,19 @@ 
 		// Get a list of comments
 		$oCommentModel = getModel('comment');
 		$output = $oCommentModel->getCommentList($this->document_srl, $cpage, $is_admin);
-		if(!$output->toBool() || !count($output->data)) return;
+		if (!$output->toBool() || !count($output->data)) return;
 		// Create commentItem object from a comment list
 		// If admin priviledge is granted on parent posts, you can read its child posts.
 		$accessible = array();
 		$comment_list = array();
-		foreach($output->data as $key => $val)
+		foreach ($output->data as $key => $val)
 		{
 			$oCommentItem = new commentItem();
 			$oCommentItem->setAttribute($val);
 			// If permission is granted to the post, you can access it temporarily
-			if($oCommentItem->isGranted()) $accessible[$val->comment_srl] = true;
+			if ($oCommentItem->isGranted()) $accessible[$val->comment_srl] = true;
 			// If the comment is set to private and it belongs child post, it is allowable to read the comment for who has a admin privilege on its parent post
-			if($val->parent_srl>0 && $val->is_secret == 'Y' && !$oCommentItem->isAccessible() && $accessible[$val->parent_srl]===true)
+			if ($val->parent_srl > 0 && $val->is_secret == 'Y' && !$oCommentItem->isAccessible() && $accessible[$val->parent_srl] === true)
 			{
 				$oCommentItem->setAccessible();
 			}
@@ -779,7 +779,7 @@ 
 		// Variable setting to be displayed on the skin
 		Context::set($cpageStr, $output->page_navigation->cur_page);
 		Context::set('cpage', $output->page_navigation->cur_page);
-		if($output->total_page>1) $this->comment_page_navigation = $output->page_navigation;
+		if ($output->total_page > 1) $this->comment_page_navigation = $output->page_navigation;
 
 		// Call trigger (after)
 		$output = ModuleHandler::triggerCall('document.getComments', 'after', $comment_list);
@@ -794,9 +794,9 @@ 
 
 	function getTrackbacks()
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 
-		if(!$this->allowTrackback() || !$this->get('trackback_count')) return;
+		if (!$this->allowTrackback() || !$this->get('trackback_count')) return;
 
 		$oTrackbackModel = getModel('trackback');
 		return $oTrackbackModel->getTrackbackList($this->document_srl, $is_admin);
@@ -804,47 +804,47 @@ 
 
 	function thumbnailExists($width = 80, $height = 0, $type = '')
 	{
-		if(!$this->document_srl) return false;
-		if(!$this->getThumbnail($width, $height, $type)) return false;
+		if (!$this->document_srl) return false;
+		if (!$this->getThumbnail($width, $height, $type)) return false;
 		return true;
 	}
 
 	function getThumbnail($width = 80, $height = 0, $thumbnail_type = '')
 	{
 		// Return false if the document doesn't exist
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 
-		if($this->isSecret() && !$this->isGranted())
+		if ($this->isSecret() && !$this->isGranted())
 		{
 			return;
 		}
 
 		// If not specify its height, create a square
-		if(!$height) $height = $width;
+		if (!$height) $height = $width;
 
 		// Return false if neither attachement nor image files in the document
 		$content = $this->get('content');
-		if(!$this->get('uploaded_count'))
+		if (!$this->get('uploaded_count'))
 		{
-			if(!$content)
+			if (!$content)
 			{
 				$args = new stdClass();
 				$args->document_srl = $this->document_srl;
 				$output = executeQuery('document.getDocument', $args, array('content'));
-				if($output->toBool() && $output->data)
+				if ($output->toBool() && $output->data)
 				{
 					$content = $output->data->content;
 					$this->add('content', $content);
 				}
 			}
 
-			if(!preg_match("!<img!is", $content)) return;
+			if (!preg_match("!<img!is", $content)) return;
 		}
 		// Get thumbnai_type information from document module's configuration
-		if(!in_array($thumbnail_type, array('crop','ratio')))
+		if (!in_array($thumbnail_type, array('crop', 'ratio')))
 		{
 			$config = $GLOBALS['__document_config__'];
-			if(!$config)
+			if (!$config)
 			{
 				$oDocumentModel = getModel('document');
 				$config = $oDocumentModel->getDocumentConfig();
@@ -854,21 +854,21 @@ 
 		}
 
 		// Define thumbnail information
-		$thumbnail_path = sprintf('files/thumbnails/%s',getNumberingPath($this->document_srl, 3));
+		$thumbnail_path = sprintf('files/thumbnails/%s', getNumberingPath($this->document_srl, 3));
 		$thumbnail_file = sprintf('%s%dx%d.%s.jpg', $thumbnail_path, $width, $height, $thumbnail_type);
 		$thumbnail_lockfile = sprintf('%s%dx%d.%s.lock', $thumbnail_path, $width, $height, $thumbnail_type);
 		$thumbnail_url  = Context::getRequestUri().$thumbnail_file;
 
 		// Return false if thumbnail file exists and its size is 0. Otherwise, return its path
-		if(file_exists($thumbnail_file) || file_exists($thumbnail_lockfile))
+		if (file_exists($thumbnail_file) || file_exists($thumbnail_lockfile))
 		{
-			if(filesize($thumbnail_file) < 1)
+			if (filesize($thumbnail_file) < 1)
 			{
 				return FALSE;
 			}
 			else
 			{
-				return $thumbnail_url . '?' . date('YmdHis', filemtime($thumbnail_file));
+				return $thumbnail_url.'?'.date('YmdHis', filemtime($thumbnail_file));
 			}
 		}
 
@@ -880,64 +880,64 @@ 
 		$is_tmp_file = false;
 
 		// Find an iamge file among attached files if exists
-		if($this->hasUploadedFiles())
+		if ($this->hasUploadedFiles())
 		{
 			$file_list = $this->getUploadedFiles();
 
 			$first_image = null;
-			foreach($file_list as $file)
+			foreach ($file_list as $file)
 			{
-				if($file->direct_download !== 'Y') continue;
+				if ($file->direct_download !== 'Y') continue;
 
-				if($file->cover_image === 'Y' && file_exists($file->uploaded_filename))
+				if ($file->cover_image === 'Y' && file_exists($file->uploaded_filename))
 				{
 					$source_file = $file->uploaded_filename;
 					break;
 				}
 
-				if($first_image) continue;
+				if ($first_image) continue;
 
-				if(preg_match("/\.(jpe?g|png|gif|bmp)$/i", $file->source_filename))
+				if (preg_match("/\.(jpe?g|png|gif|bmp)$/i", $file->source_filename))
 				{
-					if(file_exists($file->uploaded_filename))
+					if (file_exists($file->uploaded_filename))
 					{
 						$first_image = $file->uploaded_filename;
 					}
 				}
 			}
 
-			if(!$source_file && $first_image)
+			if (!$source_file && $first_image)
 			{
 				$source_file = $first_image;
 			}
 		}
 		// If not exists, file an image file from the content
 		$is_tmp_file = false;
-		if(!$source_file)
+		if (!$source_file)
 		{
 			$random = new Password();
 
 			preg_match_all("!<img[^>]*src=(?:\"|\')([^\"\']*?)(?:\"|\')!is", $content, $matches, PREG_SET_ORDER);
 
-			foreach($matches as $target_image)
+			foreach ($matches as $target_image)
 			{
 				$target_src = trim($target_image[1]);
-				if(preg_match('/\/(common|modules|widgets|addons|layouts|m\.layouts)\//i', $target_src)) continue;
+				if (preg_match('/\/(common|modules|widgets|addons|layouts|m\.layouts)\//i', $target_src)) continue;
 
-				if(!preg_match('/^(http|https):\/\//i',$target_src))
+				if (!preg_match('/^(http|https):\/\//i', $target_src))
 				{
 					$target_src = Context::getRequestUri().$target_src;
 				}
 
 				$target_src = htmlspecialchars_decode($target_src);
 
-				$tmp_file = _XE_PATH_ . 'files/cache/tmp/' . $random->createSecureSalt(32, 'hex');
+				$tmp_file = _XE_PATH_.'files/cache/tmp/'.$random->createSecureSalt(32, 'hex');
 				FileHandler::getRemoteFile($target_src, $tmp_file);
-				if(!file_exists($tmp_file)) continue;
+				if (!file_exists($tmp_file)) continue;
 
 				$imageinfo = getimagesize($tmp_file);
 				list($_w, $_h) = $imageinfo;
-				if($imageinfo === false || ($_w < ($width * 0.3) && $_h < ($height * 0.3))) {
+				if ($imageinfo === false || ($_w < ($width * 0.3) && $_h < ($height * 0.3))) {
 					FileHandler::removeFile($tmp_file);
 					continue;
 				}
@@ -948,13 +948,13 @@ 
 			}
 		}
 
-		if($source_file)
+		if ($source_file)
 		{
 			$output_file = FileHandler::createImageFile($source_file, $thumbnail_file, $width, $height, 'jpg', $thumbnail_type);
 		}
 
 		// Remove source file if it was temporary
-		if($is_tmp_file)
+		if ($is_tmp_file)
 		{
 			FileHandler::removeFile($source_file);
 		}
@@ -963,12 +963,12 @@ 
 		FileHandler::removeFile($thumbnail_lockfile);
 
 		// Create an empty file if thumbnail generation failed
-		if(!$output_file)
+		if (!$output_file)
 		{
-			FileHandler::writeFile($thumbnail_file, '','w');
+			FileHandler::writeFile($thumbnail_file, '', 'w');
 		}
 
-		return $thumbnail_url . '?' . date('YmdHis', filemtime($thumbnail_file));
+		return $thumbnail_url.'?'.date('YmdHis', filemtime($thumbnail_file));
 	}
 
 	/**
@@ -979,21 +979,21 @@ 
 	 */
 	function getExtraImages($time_interval = 43200)
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 		// variables for icon list
 		$buffs = array();
 
 		$check_files = false;
 
 		// Check if secret post is
-		if($this->isSecret()) $buffs[] = "secret";
+		if ($this->isSecret()) $buffs[] = "secret";
 
 		// Set the latest time
-		$time_check = date("YmdHis", $_SERVER['REQUEST_TIME']-$time_interval);
+		$time_check = date("YmdHis", $_SERVER['REQUEST_TIME'] - $time_interval);
 
 		// Check new post
-		if($this->get('regdate')>$time_check) $buffs[] = "new";
-		else if($this->get('last_update')>$time_check) $buffs[] = "update";
+		if ($this->get('regdate') > $time_check) $buffs[] = "new";
+		else if ($this->get('last_update') > $time_check) $buffs[] = "update";
 
 		/*
 		   $content = $this->get('content');
@@ -1016,14 +1016,14 @@ 
 		 */
 
 		// Check the attachment
-		if($this->hasUploadedFiles()) $buffs[] = "file";
+		if ($this->hasUploadedFiles()) $buffs[] = "file";
 
 		return $buffs;
 	}
 
 	function getStatus()
 	{
-		if(!$this->get('status')) {
+		if (!$this->get('status')) {
 			$oDocumentClass = getClass('document');
 			return $oDocumentClass->getDefaultStatus();
 		}
@@ -1037,11 +1037,11 @@ 
 	 */
 	function printExtraImages($time_check = 43200)
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 
 		$oDocumentModel = getModel('document');
 		$documentConfig = $oDocumentModel->getDocumentConfig();
-		if(Mobile::isFromMobilePhone())
+		if (Mobile::isFromMobilePhone())
 		{
 			$iconSkin = $documentConfig->micons;
 		}
@@ -1049,13 +1049,13 @@ 
 		{
 			$iconSkin = $documentConfig->icons;
 		}
-		$path = sprintf('%s%s',getUrl(), "modules/document/tpl/icons/$iconSkin/");
+		$path = sprintf('%s%s', getUrl(), "modules/document/tpl/icons/$iconSkin/");
 
 		$buffs = $this->getExtraImages($time_check);
-		if(!count($buffs)) return;
+		if (!count($buffs)) return;
 
 		$buff = array();
-		foreach($buffs as $key => $val)
+		foreach ($buffs as $key => $val)
 		{
 			$buff[] = sprintf('<img src="%s%s.gif" alt="%s" title="%s" style="margin-right:2px;" />', $path, $val, $val, $val);
 		}
@@ -1064,20 +1064,20 @@ 
 
 	function hasUploadedFiles()
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 
-		if($this->isSecret() && !$this->isGranted()) return false;
-		return $this->get('uploaded_count')? true : false;
+		if ($this->isSecret() && !$this->isGranted()) return false;
+		return $this->get('uploaded_count') ? true : false;
 	}
 
 	function getUploadedFiles($sortIndex = 'file_srl')
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 
-		if($this->isSecret() && !$this->isGranted()) return;
-		if(!$this->get('uploaded_count')) return;
+		if ($this->isSecret() && !$this->isGranted()) return;
+		if (!$this->get('uploaded_count')) return;
 
-		if(!$this->uploadedFiles[$sortIndex])
+		if (!$this->uploadedFiles[$sortIndex])
 		{
 			$oFileModel = getModel('file');
 			$this->uploadedFiles[$sortIndex] = $oFileModel->getFiles($this->document_srl, array(), $sortIndex, true);
@@ -1093,7 +1093,7 @@ 
 	function getEditor()
 	{
 		$module_srl = $this->get('module_srl');
-		if(!$module_srl) $module_srl = Context::get('module_srl');
+		if (!$module_srl) $module_srl = Context::get('module_srl');
 
 		$oEditorModel = getModel('editor');
 		return $oEditorModel->getModuleEditor('document', $module_srl, $this->document_srl, 'document_srl', 'content');
@@ -1108,7 +1108,7 @@ 
 	{
 		// Return false if not authorized, if a secret document, if the document is set not to allow any comment
 		if (!$this->allowComment()) return false;
-		if(!$this->isGranted() && $this->isSecret()) return false;
+		if (!$this->isGranted() && $this->isSecret()) return false;
 
 		return true;
 	}
@@ -1119,7 +1119,7 @@ 
 	 */
 	function getCommentEditor()
 	{
-		if(!$this->isEnableComment()) return;
+		if (!$this->isEnableComment()) return;
 
 		$oEditorModel = getModel('editor');
 		return $oEditorModel->getModuleEditor('comment', $this->get('module_srl'), $comment_srl, 'comment_srl', 'content');
@@ -1131,10 +1131,10 @@ 
 	 */
 	function getProfileImage()
 	{
-		if(!$this->isExists() || !$this->get('member_srl')) return;
+		if (!$this->isExists() || !$this->get('member_srl')) return;
 		$oMemberModel = getModel('member');
 		$profile_info = $oMemberModel->getProfileImage($this->get('member_srl'));
-		if(!$profile_info) return;
+		if (!$profile_info) return;
 
 		return $profile_info->src;
 	}
@@ -1146,21 +1146,21 @@ 
 	function getSignature()
 	{
 		// Pass if a document doesn't exist
-		if(!$this->isExists() || !$this->get('member_srl')) return;
+		if (!$this->isExists() || !$this->get('member_srl')) return;
 		// Get signature information
 		$oMemberModel = getModel('member');
 		$signature = $oMemberModel->getSignature($this->get('member_srl'));
 		// Check if a maximum height of signiture is set in the member module
-		if(!isset($GLOBALS['__member_signature_max_height']))
+		if (!isset($GLOBALS['__member_signature_max_height']))
 		{
 			$oModuleModel = getModel('module');
 			$member_config = $oModuleModel->getModuleConfig('member');
 			$GLOBALS['__member_signature_max_height'] = $member_config->signature_max_height;
 		}
-		if($signature)
+		if ($signature)
 		{
 			$max_signature_height = $GLOBALS['__member_signature_max_height'];
-			if($max_signature_height) $signature = sprintf('<div style="max-height:%dpx;overflow:auto;overflow-x:hidden;height:expression(this.scrollHeight > %d ? \'%dpx\': \'auto\')">%s</div>', $max_signature_height, $max_signature_height, $max_signature_height, $signature);
+			if ($max_signature_height) $signature = sprintf('<div style="max-height:%dpx;overflow:auto;overflow-x:hidden;height:expression(this.scrollHeight > %d ? \'%dpx\': \'auto\')">%s</div>', $max_signature_height, $max_signature_height, $max_signature_height, $signature);
 		}
 
 		return $signature;
@@ -1173,7 +1173,7 @@ 
 	 */
 	function replaceResourceRealPath($matches)
 	{
-		return preg_replace('/src=(["\']?)files/i','src=$1'.Context::getRequestUri().'files', $matches[0]);
+		return preg_replace('/src=(["\']?)files/i', 'src=$1'.Context::getRequestUri().'files', $matches[0]);
 	}
 
 	/**
@@ -1184,19 +1184,19 @@ 
 	function _checkAccessibleFromStatus()
 	{
 		$logged_info = Context::get('logged_info');
-		if($logged_info->is_admin == 'Y') return true;
+		if ($logged_info->is_admin == 'Y') return true;
 
 		$status = $this->get('status');
-		if(empty($status)) return false;
+		if (empty($status)) return false;
 
 		$oDocumentModel = getModel('document');
 		$configStatusList = $oDocumentModel->getStatusList();
 
-		if($status == $configStatusList['public'] || $status == $configStatusList['publish'])
+		if ($status == $configStatusList['public'] || $status == $configStatusList['publish'])
 			return true;
-		else if($status == $configStatusList['private'] || $status == $configStatusList['secret'])
+		else if ($status == $configStatusList['private'] || $status == $configStatusList['secret'])
 		{
-			if($this->get('member_srl') == $logged_info->member_srl)
+			if ($this->get('member_srl') == $logged_info->member_srl)
 				return true;
 		}
 		return false;

Braces +258 added lines, -110 removed lines

@@ -79,7 +79,9 @@ 
 	 */
 	function _loadFromDB($load_extra_vars = true)
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 
 		$document_item = false;
 		$cache_put = false;
@@ -111,8 +113,7 @@ 
 			{
 				$oCacheHandler->put($cache_key, $document_item);
 			}
-		}
-		else
+		} else
 		{
 			$document_item->readed_count = $output->data->readed_count;
 			$document_item->voted_count = $output->data->voted_count;
@@ -159,21 +160,29 @@ 
 
 	function isGranted()
 	{
-		if($_SESSION['own_document'][$this->document_srl]) return $this->grant_cache = true;
+		if($_SESSION['own_document'][$this->document_srl]) {
+			return $this->grant_cache = true;
+		}
 
 		if($this->grant_cache !== null)
 		{
 			return $this->grant_cache;
 		}
 
-		if(!Context::get('is_logged')) return $this->grant_cache = false;
+		if(!Context::get('is_logged')) {
+			return $this->grant_cache = false;
+		}
 
 		$logged_info = Context::get('logged_info');
-		if($logged_info->is_admin == 'Y') return $this->grant_cache = true;
+		if($logged_info->is_admin == 'Y') {
+			return $this->grant_cache = true;
+		}
 
 		$oModuleModel = getModel('module');
 		$grant = $oModuleModel->getGrant($oModuleModel->getModuleInfoByModuleSrl($this->get('module_srl')), $logged_info);
-		if($grant->manager) return $this->grant_cache = true;
+		if($grant->manager) {
+			return $this->grant_cache = true;
+		}
 
 		if($this->get('member_srl') && abs($this->get('member_srl')) == $logged_info->member_srl)
 		{
@@ -197,7 +206,9 @@ 
 	function allowComment()
 	{
 		// init write, document is not exists. so allow comment status is true
-		if(!$this->isExists()) return true;
+		if(!$this->isExists()) {
+			return true;
+		}
 
 		return $this->get('comment_status') == 'ALLOW' ? true : false;
 	}
@@ -212,8 +223,7 @@ 
 			if(!getClass('trackback'))
 			{
 				$allow_trackback_status = false;
-			}
-			else
+			} else
 			{
 				// If the trackback module is configured to be disabled, do not allow. Otherwise, check the setting of each module.
 				$oModuleModel = getModel('module');
@@ -224,15 +234,21 @@ 
 					$trackback_config = new stdClass();
 				}
 				
-				if(!isset($trackback_config->enable_trackback)) $trackback_config->enable_trackback = 'Y';
-				if($trackback_config->enable_trackback != 'Y') $allow_trackback_status = false;
-				else
+				if(!isset($trackback_config->enable_trackback)) {
+					$trackback_config->enable_trackback = 'Y';
+				}
+				if($trackback_config->enable_trackback != 'Y') {
+					$allow_trackback_status = false;
+				} else
 				{
 					$module_srl = $this->get('module_srl');
 					// Check settings of each module
 					$module_config = $oModuleModel->getModulePartConfig('trackback', $module_srl);
-					if($module_config->enable_trackback == 'N') $allow_trackback_status = false;
-					else if($this->get('allow_trackback')=='Y' || !$this->isExists()) $allow_trackback_status = true;
+					if($module_config->enable_trackback == 'N') {
+						$allow_trackback_status = false;
+					} else if($this->get('allow_trackback')=='Y' || !$this->isExists()) {
+						$allow_trackback_status = true;
+					}
 				}
 			}
 		}
@@ -241,14 +257,18 @@ 
 
 	function isLocked()
 	{
-		if(!$this->isExists()) return false;
+		if(!$this->isExists()) {
+			return false;
+		}
 
 		return $this->get('comment_status') == 'ALLOW' ? false : true;
 	}
 
 	function isEditable()
 	{
-		if($this->isGranted() || !$this->get('member_srl')) return true;
+		if($this->isGranted() || !$this->get('member_srl')) {
+			return true;
+		}
 		return false;
 	}
 
@@ -270,9 +290,14 @@ 
 
 	function doCart()
 	{
-		if(!$this->document_srl) return false;
-		if($this->isCarted()) $this->removeCart();
-		else $this->addCart();
+		if(!$this->document_srl) {
+			return false;
+		}
+		if($this->isCarted()) {
+			$this->removeCart();
+		} else {
+			$this->addCart();
+		}
 	}
 
 	function addCart()
@@ -298,16 +323,26 @@ 
 	 */
 	function notify($type, $content)
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 		// return if it is not useNotify
-		if(!$this->useNotify()) return;
+		if(!$this->useNotify()) {
+			return;
+		}
 		// Pass if an author is not a logged-in user
-		if(!$this->get('member_srl')) return;
+		if(!$this->get('member_srl')) {
+			return;
+		}
 		// Return if the currently logged-in user is an author
 		$logged_info = Context::get('logged_info');
-		if($logged_info->member_srl == $this->get('member_srl')) return;
+		if($logged_info->member_srl == $this->get('member_srl')) {
+			return;
+		}
 		// List variables
-		if($type) $title = "[".$type."] ";
+		if($type) {
+			$title = "[".$type."] ";
+		}
 		$title .= cut_str(strip_tags($content), 10, '...');
 		$content = sprintf('%s<br /><br />from : <a href="%s" target="_blank">%s</a>',$content, getFullUrl('','document_srl',$this->document_srl), getFullUrl('','document_srl',$this->document_srl));
 		$receiver_srl = $this->get('member_srl');
@@ -334,16 +369,22 @@ 
 
 	function isExistsHomepage()
 	{
-		if(trim($this->get('homepage'))) return true;
+		if(trim($this->get('homepage'))) {
+			return true;
+		}
 		return false;
 	}
 
 	function getHomepageUrl()
 	{
 		$url = trim($this->get('homepage'));
-		if(!$url) return;
+		if(!$url) {
+			return;
+		}
 
-		if(strncasecmp('http://', $url, 7) !== 0 && strncasecmp('https://', $url, 8) !== 0)  $url = 'http://' . $url;
+		if(strncasecmp('http://', $url, 7) !== 0 && strncasecmp('https://', $url, 8) !== 0) {
+			$url = 'http://' . $url;
+		}
 
 		return $url;
 	}
@@ -375,43 +416,65 @@ 
 
 	function getTitleText($cut_size = 0, $tail='...')
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 
-		if($cut_size) $title = cut_str($this->get('title'), $cut_size, $tail);
-		else $title = $this->get('title');
+		if($cut_size) {
+			$title = cut_str($this->get('title'), $cut_size, $tail);
+		} else {
+			$title = $this->get('title');
+		}
 
 		return $title;
 	}
 
 	function getTitle($cut_size = 0, $tail='...')
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 
 		$title = $this->getTitleText($cut_size, $tail);
 
 		$attrs = array();
 		$this->add('title_color', trim($this->get('title_color')));
-		if($this->get('title_bold')=='Y') $attrs[] = "font-weight:bold;";
-		if($this->get('title_color') && $this->get('title_color') != 'N') $attrs[] = "color:#".$this->get('title_color');
+		if($this->get('title_bold')=='Y') {
+			$attrs[] = "font-weight:bold;";
+		}
+		if($this->get('title_color') && $this->get('title_color') != 'N') {
+			$attrs[] = "color:#".$this->get('title_color');
+		}
 
-		if(count($attrs)) return sprintf("<span style=\"%s\">%s</span>", implode(';',$attrs), htmlspecialchars($title, ENT_COMPAT | ENT_HTML401, 'UTF-8', false));
-		else return htmlspecialchars($title, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
+		if(count($attrs)) {
+			return sprintf("<span style=\"%s\">%s</span>", implode(';',$attrs), htmlspecialchars($title, ENT_COMPAT | ENT_HTML401, 'UTF-8', false));
+		} else {
+			return htmlspecialchars($title, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
+		}
 	}
 
 	function getContentText($strlen = 0)
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 
-		if($this->isSecret() && !$this->isGranted() && !$this->isAccessible()) return Context::getLang('msg_is_secret');
+		if($this->isSecret() && !$this->isGranted() && !$this->isAccessible()) {
+			return Context::getLang('msg_is_secret');
+		}
 
 		$result = $this->_checkAccessibleFromStatus();
-		if($result) $_SESSION['accessible'][$this->document_srl] = true;
+		if($result) {
+			$_SESSION['accessible'][$this->document_srl] = true;
+		}
 
 		$content = $this->get('content');
 		$content = preg_replace_callback('/<(object|param|embed)[^>]*/is', array($this, '_checkAllowScriptAccess'), $content);
 		$content = preg_replace_callback('/<object[^>]*>/is', array($this, '_addAllowScriptAccess'), $content);
 
-		if($strlen) return cut_str(strip_tags($content),$strlen,'...');
+		if($strlen) {
+			return cut_str(strip_tags($content),$strlen,'...');
+		}
 
 		return htmlspecialchars($content);
 	}
@@ -444,14 +507,12 @@ 
 				}
 				$this->allowscriptaccessList[count($this->allowscriptaccessList)-1]--;
 			}
-		}
-		else if($m[1] == 'embed')
+		} else if($m[1] == 'embed')
 		{
 			if(stripos($m[0], 'allowscriptaccess'))
 			{
 				$m[0] = preg_replace('/always|samedomain/i', 'never', $m[0]);
-			}
-			else
+			} else
 			{
 				$m[0] = preg_replace('/\<embed/i', '<embed allowscriptaccess="never"', $m[0]);
 			}
@@ -461,15 +522,23 @@ 
 
 	function getContent($add_popup_menu = true, $add_content_info = true, $resource_realpath = false, $add_xe_content_class = true, $stripEmbedTagException = false)
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 
-		if($this->isSecret() && !$this->isGranted() && !$this->isAccessible()) return Context::getLang('msg_is_secret');
+		if($this->isSecret() && !$this->isGranted() && !$this->isAccessible()) {
+			return Context::getLang('msg_is_secret');
+		}
 
 		$result = $this->_checkAccessibleFromStatus();
-		if($result) $_SESSION['accessible'][$this->document_srl] = true;
+		if($result) {
+			$_SESSION['accessible'][$this->document_srl] = true;
+		}
 
 		$content = $this->get('content');
-		if(!$stripEmbedTagException) stripEmbedTagForAdmin($content, $this->get('member_srl'));
+		if(!$stripEmbedTagException) {
+			stripEmbedTagForAdmin($content, $this->get('member_srl'));
+		}
 
 		// Define a link if using a rewrite module
 		$oContext = &Context::getInstance();
@@ -503,10 +572,11 @@ 
 				$this->document_srl, $memberSrl
 			);
 			// Add xe_content class although accessing content is not required
-		}
-		else
+		} else
 		{
-			if($add_xe_content_class) $content = sprintf('<div class="xe_content">%s</div>', $content);
+			if($add_xe_content_class) {
+				$content = sprintf('<div class="xe_content">%s</div>', $content);
+			}
 		}
 		// Change the image path to a valid absolute path if resource_realpath is true
 		if($resource_realpath)
@@ -625,11 +695,15 @@ 
 
 	function getTrackbackUrl()
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 
 		// Generate a key to prevent spams
 		$oTrackbackModel = getModel('trackback');
-		if($oTrackbackModel) return $oTrackbackModel->getTrackbackUrl($this->document_srl, $this->getDocumentMid());
+		if($oTrackbackModel) {
+			return $oTrackbackModel->getTrackbackUrl($this->document_srl, $this->getDocumentMid());
+		}
 	}
 
 	/**
@@ -648,7 +722,9 @@ 
 
 	function isExtraVarsExists()
 	{
-		if(!$this->get('module_srl')) return false;
+		if(!$this->get('module_srl')) {
+			return false;
+		}
 		$oDocumentModel = getModel('document');
 		$extra_keys = $oDocumentModel->getExtraKeys($this->get('module_srl'));
 		return count($extra_keys)?true:false;
@@ -656,7 +732,9 @@ 
 
 	function getExtraVars()
 	{
-		if(!$this->get('module_srl') || !$this->document_srl) return null;
+		if(!$this->get('module_srl') || !$this->document_srl) {
+			return null;
+		}
 
 		$oDocumentModel = getModel('document');
 		return $oDocumentModel->getExtraVars($this->get('module_srl'), $this->document_srl);
@@ -668,8 +746,7 @@ 
 		if(is_array($extra_vars) && array_key_exists($idx,$extra_vars))
 		{
 			return $extra_vars[$idx]->getValue();
-		}
-		else
+		} else
 		{
 			return '';
 		}
@@ -681,8 +758,7 @@ 
 		if(is_array($extra_vars) && array_key_exists($idx,$extra_vars))
 		{
 			return $extra_vars[$idx]->getValueHTML();
-		}
-		else
+		} else
 		{
 			return '';
 		}
@@ -704,8 +780,7 @@ 
 		if(is_array($extra_eid) && array_key_exists($eid,$extra_eid))
 		{
 			return $extra_eid[$eid]->getValue();
-		}
-		else
+		} else
 		{
 			return '';
 		}
@@ -723,8 +798,7 @@ 
 		if(is_array($extra_eid) && array_key_exists($eid,$extra_eid))
 		{
 			return $extra_eid[$eid]->getValueHTML();
-		}
-		else
+		} else
 		{
 			return '';
 		}
@@ -744,8 +818,12 @@ 
 
 	function getComments()
 	{
-		if(!$this->getCommentCount()) return;
-		if(!$this->isGranted() && $this->isSecret()) return;
+		if(!$this->getCommentCount()) {
+			return;
+		}
+		if(!$this->isGranted() && $this->isSecret()) {
+			return;
+		}
 		// cpage is a number of comment pages
 		$cpageStr = sprintf('%d_cpage', $this->document_srl);
 		$cpage = Context::get($cpageStr);
@@ -758,7 +836,9 @@ 
 		// Get a list of comments
 		$oCommentModel = getModel('comment');
 		$output = $oCommentModel->getCommentList($this->document_srl, $cpage, $is_admin);
-		if(!$output->toBool() || !count($output->data)) return;
+		if(!$output->toBool() || !count($output->data)) {
+			return;
+		}
 		// Create commentItem object from a comment list
 		// If admin priviledge is granted on parent posts, you can read its child posts.
 		$accessible = array();
@@ -768,7 +848,9 @@ 
 			$oCommentItem = new commentItem();
 			$oCommentItem->setAttribute($val);
 			// If permission is granted to the post, you can access it temporarily
-			if($oCommentItem->isGranted()) $accessible[$val->comment_srl] = true;
+			if($oCommentItem->isGranted()) {
+				$accessible[$val->comment_srl] = true;
+			}
 			// If the comment is set to private and it belongs child post, it is allowable to read the comment for who has a admin privilege on its parent post
 			if($val->parent_srl>0 && $val->is_secret == 'Y' && !$oCommentItem->isAccessible() && $accessible[$val->parent_srl]===true)
 			{
@@ -779,7 +861,9 @@ 
 		// Variable setting to be displayed on the skin
 		Context::set($cpageStr, $output->page_navigation->cur_page);
 		Context::set('cpage', $output->page_navigation->cur_page);
-		if($output->total_page>1) $this->comment_page_navigation = $output->page_navigation;
+		if($output->total_page>1) {
+			$this->comment_page_navigation = $output->page_navigation;
+		}
 
 		// Call trigger (after)
 		$output = ModuleHandler::triggerCall('document.getComments', 'after', $comment_list);
@@ -794,9 +878,13 @@ 
 
 	function getTrackbacks()
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 
-		if(!$this->allowTrackback() || !$this->get('trackback_count')) return;
+		if(!$this->allowTrackback() || !$this->get('trackback_count')) {
+			return;
+		}
 
 		$oTrackbackModel = getModel('trackback');
 		return $oTrackbackModel->getTrackbackList($this->document_srl, $is_admin);
@@ -804,15 +892,21 @@ 
 
 	function thumbnailExists($width = 80, $height = 0, $type = '')
 	{
-		if(!$this->document_srl) return false;
-		if(!$this->getThumbnail($width, $height, $type)) return false;
+		if(!$this->document_srl) {
+			return false;
+		}
+		if(!$this->getThumbnail($width, $height, $type)) {
+			return false;
+		}
 		return true;
 	}
 
 	function getThumbnail($width = 80, $height = 0, $thumbnail_type = '')
 	{
 		// Return false if the document doesn't exist
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 
 		if($this->isSecret() && !$this->isGranted())
 		{
@@ -820,7 +914,9 @@ 
 		}
 
 		// If not specify its height, create a square
-		if(!$height) $height = $width;
+		if(!$height) {
+			$height = $width;
+		}
 
 		// Return false if neither attachement nor image files in the document
 		$content = $this->get('content');
@@ -838,7 +934,9 @@ 
 				}
 			}
 
-			if(!preg_match("!<img!is", $content)) return;
+			if(!preg_match("!<img!is", $content)) {
+				return;
+			}
 		}
 		// Get thumbnai_type information from document module's configuration
 		if(!in_array($thumbnail_type, array('crop','ratio')))
@@ -865,8 +963,7 @@ 
 			if(filesize($thumbnail_file) < 1)
 			{
 				return FALSE;
-			}
-			else
+			} else
 			{
 				return $thumbnail_url . '?' . date('YmdHis', filemtime($thumbnail_file));
 			}
@@ -887,7 +984,9 @@ 
 			$first_image = null;
 			foreach($file_list as $file)
 			{
-				if($file->direct_download !== 'Y') continue;
+				if($file->direct_download !== 'Y') {
+					continue;
+				}
 
 				if($file->cover_image === 'Y' && file_exists($file->uploaded_filename))
 				{
@@ -895,7 +994,9 @@ 
 					break;
 				}
 
-				if($first_image) continue;
+				if($first_image) {
+					continue;
+				}
 
 				if(preg_match("/\.(jpe?g|png|gif|bmp)$/i", $file->source_filename))
 				{
@@ -922,7 +1023,9 @@ 
 			foreach($matches as $target_image)
 			{
 				$target_src = trim($target_image[1]);
-				if(preg_match('/\/(common|modules|widgets|addons|layouts|m\.layouts)\//i', $target_src)) continue;
+				if(preg_match('/\/(common|modules|widgets|addons|layouts|m\.layouts)\//i', $target_src)) {
+					continue;
+				}
 
 				if(!preg_match('/^(http|https):\/\//i',$target_src))
 				{
@@ -933,7 +1036,9 @@ 
 
 				$tmp_file = _XE_PATH_ . 'files/cache/tmp/' . $random->createSecureSalt(32, 'hex');
 				FileHandler::getRemoteFile($target_src, $tmp_file);
-				if(!file_exists($tmp_file)) continue;
+				if(!file_exists($tmp_file)) {
+					continue;
+				}
 
 				$imageinfo = getimagesize($tmp_file);
 				list($_w, $_h) = $imageinfo;
@@ -979,21 +1084,28 @@ 
 	 */
 	function getExtraImages($time_interval = 43200)
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 		// variables for icon list
 		$buffs = array();
 
 		$check_files = false;
 
 		// Check if secret post is
-		if($this->isSecret()) $buffs[] = "secret";
+		if($this->isSecret()) {
+			$buffs[] = "secret";
+		}
 
 		// Set the latest time
 		$time_check = date("YmdHis", $_SERVER['REQUEST_TIME']-$time_interval);
 
 		// Check new post
-		if($this->get('regdate')>$time_check) $buffs[] = "new";
-		else if($this->get('last_update')>$time_check) $buffs[] = "update";
+		if($this->get('regdate')>$time_check) {
+			$buffs[] = "new";
+		} else if($this->get('last_update')>$time_check) {
+			$buffs[] = "update";
+		}
 
 		/*
 		   $content = $this->get('content');
@@ -1016,7 +1128,9 @@ 
 		 */
 
 		// Check the attachment
-		if($this->hasUploadedFiles()) $buffs[] = "file";
+		if($this->hasUploadedFiles()) {
+			$buffs[] = "file";
+		}
 
 		return $buffs;
 	}
@@ -1037,22 +1151,25 @@ 
 	 */
 	function printExtraImages($time_check = 43200)
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 
 		$oDocumentModel = getModel('document');
 		$documentConfig = $oDocumentModel->getDocumentConfig();
 		if(Mobile::isFromMobilePhone())
 		{
 			$iconSkin = $documentConfig->micons;
-		}
-		else
+		} else
 		{
 			$iconSkin = $documentConfig->icons;
 		}
 		$path = sprintf('%s%s',getUrl(), "modules/document/tpl/icons/$iconSkin/");
 
 		$buffs = $this->getExtraImages($time_check);
-		if(!count($buffs)) return;
+		if(!count($buffs)) {
+			return;
+		}
 
 		$buff = array();
 		foreach($buffs as $key => $val)
@@ -1064,18 +1181,28 @@ 
 
 	function hasUploadedFiles()
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 
-		if($this->isSecret() && !$this->isGranted()) return false;
+		if($this->isSecret() && !$this->isGranted()) {
+			return false;
+		}
 		return $this->get('uploaded_count')? true : false;
 	}
 
 	function getUploadedFiles($sortIndex = 'file_srl')
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 
-		if($this->isSecret() && !$this->isGranted()) return;
-		if(!$this->get('uploaded_count')) return;
+		if($this->isSecret() && !$this->isGranted()) {
+			return;
+		}
+		if(!$this->get('uploaded_count')) {
+			return;
+		}
 
 		if(!$this->uploadedFiles[$sortIndex])
 		{
@@ -1093,7 +1220,9 @@ 
 	function getEditor()
 	{
 		$module_srl = $this->get('module_srl');
-		if(!$module_srl) $module_srl = Context::get('module_srl');
+		if(!$module_srl) {
+			$module_srl = Context::get('module_srl');
+		}
 
 		$oEditorModel = getModel('editor');
 		return $oEditorModel->getModuleEditor('document', $module_srl, $this->document_srl, 'document_srl', 'content');
@@ -1107,8 +1236,12 @@ 
 	function isEnableComment()
 	{
 		// Return false if not authorized, if a secret document, if the document is set not to allow any comment
-		if (!$this->allowComment()) return false;
-		if(!$this->isGranted() && $this->isSecret()) return false;
+		if (!$this->allowComment()) {
+			return false;
+		}
+		if(!$this->isGranted() && $this->isSecret()) {
+			return false;
+		}
 
 		return true;
 	}
@@ -1119,7 +1252,9 @@ 
 	 */
 	function getCommentEditor()
 	{
-		if(!$this->isEnableComment()) return;
+		if(!$this->isEnableComment()) {
+			return;
+		}
 
 		$oEditorModel = getModel('editor');
 		return $oEditorModel->getModuleEditor('comment', $this->get('module_srl'), $comment_srl, 'comment_srl', 'content');
@@ -1131,10 +1266,14 @@ 
 	 */
 	function getProfileImage()
 	{
-		if(!$this->isExists() || !$this->get('member_srl')) return;
+		if(!$this->isExists() || !$this->get('member_srl')) {
+			return;
+		}
 		$oMemberModel = getModel('member');
 		$profile_info = $oMemberModel->getProfileImage($this->get('member_srl'));
-		if(!$profile_info) return;
+		if(!$profile_info) {
+			return;
+		}
 
 		return $profile_info->src;
 	}
@@ -1146,7 +1285,9 @@ 
 	function getSignature()
 	{
 		// Pass if a document doesn't exist
-		if(!$this->isExists() || !$this->get('member_srl')) return;
+		if(!$this->isExists() || !$this->get('member_srl')) {
+			return;
+		}
 		// Get signature information
 		$oMemberModel = getModel('member');
 		$signature = $oMemberModel->getSignature($this->get('member_srl'));
@@ -1160,7 +1301,9 @@ 
 		if($signature)
 		{
 			$max_signature_height = $GLOBALS['__member_signature_max_height'];
-			if($max_signature_height) $signature = sprintf('<div style="max-height:%dpx;overflow:auto;overflow-x:hidden;height:expression(this.scrollHeight > %d ? \'%dpx\': \'auto\')">%s</div>', $max_signature_height, $max_signature_height, $max_signature_height, $signature);
+			if($max_signature_height) {
+				$signature = sprintf('<div style="max-height:%dpx;overflow:auto;overflow-x:hidden;height:expression(this.scrollHeight > %d ? \'%dpx\': \'auto\')">%s</div>', $max_signature_height, $max_signature_height, $max_signature_height, $signature);
+			}
 		}
 
 		return $signature;
@@ -1184,20 +1327,25 @@ 
 	function _checkAccessibleFromStatus()
 	{
 		$logged_info = Context::get('logged_info');
-		if($logged_info->is_admin == 'Y') return true;
+		if($logged_info->is_admin == 'Y') {
+			return true;
+		}
 
 		$status = $this->get('status');
-		if(empty($status)) return false;
+		if(empty($status)) {
+			return false;
+		}
 
 		$oDocumentModel = getModel('document');
 		$configStatusList = $oDocumentModel->getStatusList();
 
-		if($status == $configStatusList['public'] || $status == $configStatusList['publish'])
-			return true;
-		else if($status == $configStatusList['private'] || $status == $configStatusList['secret'])
+		if($status == $configStatusList['public'] || $status == $configStatusList['publish']) {
+					return true;
+		} else if($status == $configStatusList['private'] || $status == $configStatusList['secret'])
 		{
-			if($this->get('member_srl') == $logged_info->member_srl)
-				return true;
+			if($this->get('member_srl') == $logged_info->member_srl) {
+							return true;
+			}
 		}
 		return false;
 	}

modules/comment/comment.model.php

Spacing +99 added lines, -99 removed lines

@@ -43,7 +43,7 @@ 
 		$oCommentController = getController('comment');
 
 		// feature that only member can do
-		if($logged_info->member_srl)
+		if ($logged_info->member_srl)
 		{
 			$oCommentModel = getModel('comment');
 			$columnList = array('comment_srl', 'module_srl', 'member_srl', 'ipaddress');
@@ -54,14 +54,14 @@ 
 			$oModuleModel = getModel('module');
 			$comment_config = $oModuleModel->getModulePartConfig('document', $module_srl);
 
-			if($comment_config->use_vote_up != 'N' && $member_srl != $logged_info->member_srl)
+			if ($comment_config->use_vote_up != 'N' && $member_srl != $logged_info->member_srl)
 			{
 				// Add a vote-up button for positive feedback
 				$url = sprintf("doCallModuleAction('comment','procCommentVoteUp','%s')", $comment_srl);
 				$oCommentController->addCommentPopupMenu($url, 'cmd_vote', '', 'javascript');
 			}
 
-			if($comment_config->use_vote_down != 'N' && $member_srl != $logged_info->member_srl)
+			if ($comment_config->use_vote_down != 'N' && $member_srl != $logged_info->member_srl)
 			{
 				// Add a vote-down button for negative feedback
 				$url = sprintf("doCallModuleAction('comment','procCommentVoteDown','%s')", $comment_srl);
@@ -76,20 +76,20 @@ 
 		// call a trigger (after)
 		ModuleHandler::triggerCall('comment.getCommentMenu', 'after', $menu_list);
 
-		if($this->grant->manager){
+		if ($this->grant->manager) {
 			$str_confirm = Context::getLang('confirm_move');
 			$url = sprintf("if(!confirm('%s')) return; var params = new Array(); params['comment_srl']='%s'; params['mid']=current_mid;params['cur_url']=current_url; exec_xml('comment', 'procCommentAdminMoveToTrash', params)", $str_confirm, $comment_srl);
-			$oCommentController->addCommentPopupMenu($url,'cmd_trash','','javascript');
+			$oCommentController->addCommentPopupMenu($url, 'cmd_trash', '', 'javascript');
 
 		}
 
 		// find a comment by IP matching if an administrator.
-		if($logged_info->is_admin == 'Y')
+		if ($logged_info->is_admin == 'Y')
 		{
 			$oCommentModel = getModel('comment');
 			$oComment = $oCommentModel->getComment($comment_srl);
 
-			if($oComment->isExists())
+			if ($oComment->isExists())
 			{
 				// Find a post of the corresponding ip address
 				$url = getUrl('', 'module', 'admin', 'act', 'dispCommentAdminList', 'search_target', 'ipaddress', 'search_keyword', $oComment->getIpAddress());
@@ -104,7 +104,7 @@ 
 		$menus = Context::get('comment_popup_menu_list');
 		$menus_count = count($menus);
 
-		for($i = 0; $i < $menus_count; $i++)
+		for ($i = 0; $i < $menus_count; $i++)
 		{
 			$menus[$i]->str = Context::getLang($menus[$i]->str);
 		}
@@ -160,7 +160,7 @@ 
 	function getComment($comment_srl = 0, $is_admin = FALSE, $columnList = array())
 	{
 		$oComment = new commentItem($comment_srl, $columnList);
-		if($is_admin)
+		if ($is_admin)
 		{
 			$oComment->setGrant();
 		}
@@ -176,7 +176,7 @@ 
 	 */
 	function getComments($comment_srl_list, $columnList = array())
 	{
-		if(is_array($comment_srl_list))
+		if (is_array($comment_srl_list))
 		{
 			$comment_srls = implode(',', $comment_srl_list);
 		}
@@ -185,25 +185,25 @@ 
 		$args = new stdClass();
 		$args->comment_srls = $comment_srls;
 		$output = executeQuery('comment.getComments', $args, $columnList);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return;
 		}
 
 		$comment_list = $output->data;
-		if(!$comment_list)
+		if (!$comment_list)
 		{
 			return;
 		}
-		if(!is_array($comment_list))
+		if (!is_array($comment_list))
 		{
 			$comment_list = array($comment_list);
 		}
 
 		$comment_count = count($comment_list);
-		foreach($comment_list as $key => $attribute)
+		foreach ($comment_list as $key => $attribute)
 		{
-			if(!$attribute->comment_srl)
+			if (!$attribute->comment_srl)
 			{
 				continue;
 			}
@@ -211,7 +211,7 @@ 
 			$oComment = NULL;
 			$oComment = new commentItem();
 			$oComment->setAttribute($attribute);
-			if($is_admin)
+			if ($is_admin)
 			{
 				$oComment->setGrant();
 			}
@@ -238,7 +238,7 @@ 
 		$oDocument = $oDocumentModel->getDocument($document_srl, FALSE, TRUE, $columnList);
 
 		// return if no doc exists.
-		if(!$oDocument->isExists())
+		if (!$oDocument->isExists())
 		{
 			return;
 		}
@@ -250,7 +250,7 @@ 
 		$oCommentController = getController('comment');
 
 		$using_validation = $oCommentController->isModuleUsingPublishValidation($module_srl);
-		if($using_validation)
+		if ($using_validation)
 		{
 			$args->status = 1;
 		}
@@ -269,18 +269,18 @@ 
 	 */
 	function getCommentCountByDate($date = '', $moduleSrlList = array())
 	{
-		if($date)
+		if ($date)
 		{
 			$args->regDate = date('Ymd', strtotime($date));
 		}
 
-		if(count($moduleSrlList) > 0)
+		if (count($moduleSrlList) > 0)
 		{
 			$args->module_srl = $moduleSrlList;
 		}
 
 		$output = executeQuery('comment.getCommentCount', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return 0;
 		}
@@ -299,19 +299,19 @@ 
 		$args = new stdClass();
 		$args->module_srl = $module_srl;
 
-		if(is_null($published))
+		if (is_null($published))
 		{
 			// check if module is using comment validation system
 			$oCommentController = getController("comment");
 			$is_using_validation = $oCommentController->isModuleUsingPublishValidation($module_srl);
-			if($is_using_validation)
+			if ($is_using_validation)
 			{
 				$args->status = 1;
 			}
 		}
 		else
 		{
-			if($published)
+			if ($published)
 			{
 				$args->status = 1;
 			}
@@ -363,12 +363,12 @@ 
 	{
 		$args = new stdClass();
 
-		if(!is_object($obj))
+		if (!is_object($obj))
 		{
 			$obj = new stdClass();
 		}
 
-		if($obj->mid)
+		if ($obj->mid)
 		{
 			$oModuleModel = getModel('module');
 			$obj->module_srl = $oModuleModel->getModuleSrlByMid($obj->mid);
@@ -376,7 +376,7 @@ 
 		}
 
 		// check if module_srl is an arrary.
-		if(is_array($obj->module_srl))
+		if (is_array($obj->module_srl))
 		{
 			$args->module_srl = implode(',', $obj->module_srl);
 		}
@@ -388,14 +388,14 @@ 
 		$args->document_srl = $obj->document_srl;
 		$args->list_count = $obj->list_count;
 
-		if(strpos($args->module_srl, ",") === false)
+		if (strpos($args->module_srl, ",") === false)
 		{
-			if($args->module_srl)
+			if ($args->module_srl)
 			{
 				// check if module is using comment validation system
 				$oCommentController = getController("comment");
 				$is_using_validation = $oCommentController->isModuleUsingPublishValidation($obj->module_srl);
-				if($is_using_validation)
+				if ($is_using_validation)
 				{
 					$args->status = 1;
 				}
@@ -404,24 +404,24 @@ 
 
 		$output = executeQuery('comment.getNewestCommentList', $args, $columnList);
 
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
 
 		$comment_list = $output->data;
-		if($comment_list)
+		if ($comment_list)
 		{
-			if(!is_array($comment_list))
+			if (!is_array($comment_list))
 			{
 				$comment_list = array($comment_list);
 			}
 
 			$comment_count = count($comment_list);
 
-			foreach($comment_list as $key => $attribute)
+			foreach ($comment_list as $key => $attribute)
 			{
-				if(!$attribute->comment_srl)
+				if (!$attribute->comment_srl)
 				{
 					continue;
 				}
@@ -448,7 +448,7 @@ 
 	 */
 	function getCommentList($document_srl, $page = 0, $is_admin = FALSE, $count = 0)
 	{
-		if(!isset($document_srl))
+		if (!isset($document_srl))
 		{
 			return;
 		}
@@ -459,13 +459,13 @@ 
 		$oDocument = $oDocumentModel->getDocument($document_srl, FALSE, TRUE, $columnList);
 
 		// return if no doc exists.
-		if(!$oDocument->isExists())
+		if (!$oDocument->isExists())
 		{
 			return;
 		}
 
 		// return if no comment exists
-		if($oDocument->getCommentCount() < 1)
+		if ($oDocument->getCommentCount() < 1)
 		{
 			return;
 		}
@@ -473,11 +473,11 @@ 
 		// get a list of comments
 		$module_srl = $oDocument->get('module_srl');
 
-		if(!$count)
+		if (!$count)
 		{
 			$comment_config = $this->getCommentConfig($module_srl);
 			$comment_count = $comment_config->comment_count;
-			if(!$comment_count)
+			if (!$comment_count)
 			{
 				$comment_count = 50;
 			}
@@ -488,9 +488,9 @@ 
 		}
 
 		// get a very last page if no page exists
-		if(!$page)
+		if (!$page)
 		{
-			$page = (int) ( ($oDocument->getCommentCount() - 1) / $comment_count) + 1;
+			$page = (int) (($oDocument->getCommentCount() - 1) / $comment_count) + 1;
 		}
 
 		// get a list of comments
@@ -503,14 +503,14 @@ 
 		//check if module is using validation system
 		$oCommentController = getController('comment');
 		$using_validation = $oCommentController->isModuleUsingPublishValidation($module_srl);
-		if($using_validation)
+		if ($using_validation)
 		{
 			$args->status = 1;
 		}
 
 		// call trigger (before)
 		$trigger_output = ModuleHandler::triggerCall('comment.getCommentList', 'before', $args);
-		if($trigger_output instanceof BaseObject && !$trigger_output->toBool())
+		if ($trigger_output instanceof BaseObject && !$trigger_output->toBool())
 		{
 			return $output;
 		}
@@ -518,17 +518,17 @@ 
 		$output = executeQueryArray('comment.getCommentPageList', $args);
 
 		// return if an error occurs in the query results
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return;
 		}
 
 		// insert data into CommentPageList table if the number of results is different from stored comments
-		if(!$output->data)
+		if (!$output->data)
 		{
 			$this->fixCommentList($oDocument->get('module_srl'), $document_srl);
 			$output = executeQueryArray('comment.getCommentPageList', $args);
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				return;
 			}
@@ -536,7 +536,7 @@ 
 
 		// call trigger (after)
 		$trigger_output = ModuleHandler::triggerCall('comment.getCommentList', 'after', $output);
-		if($trigger_output instanceof BaseObject && !$trigger_output->toBool())
+		if ($trigger_output instanceof BaseObject && !$trigger_output->toBool())
 		{
 			return $trigger_output;
 		}
@@ -554,9 +554,9 @@ 
 	function fixCommentList($module_srl, $document_srl)
 	{
 		// create a lock file to prevent repeated work when performing a batch job
-		$lock_file = "./files/cache/tmp/lock." . $document_srl;
+		$lock_file = "./files/cache/tmp/lock.".$document_srl;
 
-		if(file_exists($lock_file) && filemtime($lock_file) + 60 * 60 * 10 < $_SERVER['REQUEST_TIME'])
+		if (file_exists($lock_file) && filemtime($lock_file) + 60 * 60 * 10 < $_SERVER['REQUEST_TIME'])
 		{
 			return;
 		}
@@ -568,13 +568,13 @@ 
 		$args->document_srl = $document_srl;
 		$args->list_order = 'list_order';
 		$output = executeQuery('comment.getCommentList', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
 
 		$source_list = $output->data;
-		if(!is_array($source_list))
+		if (!is_array($source_list))
 		{
 			$source_list = array($source_list);
 		}
@@ -590,11 +590,11 @@ 
 		$logged_info = Context::get('logged_info');
 
 		// generate a hierarchical structure of comments for loop
-		for($i = $comment_count - 1; $i >= 0; $i--)
+		for ($i = $comment_count - 1; $i >= 0; $i--)
 		{
 			$comment_srl = $source_list[$i]->comment_srl;
 			$parent_srl = $source_list[$i]->parent_srl;
-			if(!$comment_srl)
+			if (!$comment_srl)
 			{
 				continue;
 			}
@@ -602,7 +602,7 @@ 
 			// generate a list
 			$list[$comment_srl] = $source_list[$i];
 
-			if($parent_srl)
+			if ($parent_srl)
 			{
 				$list[$parent_srl]->child[] = &$list[$comment_srl];
 			}
@@ -614,9 +614,9 @@ 
 		$this->_arrangeComment($comment_list, $root->child, 0, NULL);
 
 		// insert values to the database
-		if(count($comment_list))
+		if (count($comment_list))
 		{
-			foreach($comment_list as $comment_srl => $item)
+			foreach ($comment_list as $comment_srl => $item)
 			{
 				$comment_args = new stdClass();
 				$comment_args->comment_srl = $comment_srl;
@@ -645,14 +645,14 @@ 
 	 */
 	function _arrangeComment(&$comment_list, $list, $depth, $parent = NULL)
 	{
-		if(!count($list))
+		if (!count($list))
 		{
 			return;
 		}
 
-		foreach($list as $key => $val)
+		foreach ($list as $key => $val)
 		{
-			if($parent)
+			if ($parent)
 			{
 				$val->head = $parent->head;
 			}
@@ -663,7 +663,7 @@ 
 
 			$val->arrange = count($comment_list) + 1;
 
-			if($val->child)
+			if ($val->child)
 			{
 				$val->depth = $depth;
 				$comment_list[$val->comment_srl] = $val;
@@ -700,7 +700,7 @@ 
 		// check if module is using comment validation system
 		$oCommentController = getController("comment");
 		$is_using_validation = $oCommentController->isModuleUsingPublishValidation($obj->module_srl);
-		if($is_using_validation)
+		if ($is_using_validation)
 		{
 			$args->s_is_published = 1;
 		}
@@ -708,12 +708,12 @@ 
 		// Search options
 		$search_target = $obj->search_target ? $obj->search_target : trim(Context::get('search_target'));
 		$search_keyword = $obj->search_keyword ? $obj->search_keyword : trim(Context::get('search_keyword'));
-		if($search_target && $search_keyword)
+		if ($search_target && $search_keyword)
 		{
-			switch($search_target)
+			switch ($search_target)
 			{
 				case 'content' :
-					if($search_keyword)
+					if ($search_keyword)
 					{
 						$search_keyword = str_replace(' ', '%', $search_keyword);
 					}
@@ -722,7 +722,7 @@ 
 					break;
 
 				case 'user_id' :
-					if($search_keyword)
+					if ($search_keyword)
 					{
 						$search_keyword = str_replace(' ', '%', $search_keyword);
 					}
@@ -733,7 +733,7 @@ 
 					break;
 
 				case 'user_name' :
-					if($search_keyword)
+					if ($search_keyword)
 					{
 						$search_keyword = str_replace(' ', '%', $search_keyword);
 					}
@@ -742,7 +742,7 @@ 
 					break;
 
 				case 'nick_name' :
-					if($search_keyword)
+					if ($search_keyword)
 					{
 						$search_keyword = str_replace(' ', '%', $search_keyword);
 					}
@@ -751,7 +751,7 @@ 
 					break;
 
 				case 'email_address' :
-					if($search_keyword)
+					if ($search_keyword)
 					{
 						$search_keyword = str_replace(' ', '%', $search_keyword);
 					}
@@ -760,7 +760,7 @@ 
 					break;
 
 				case 'homepage' :
-					if($search_keyword)
+					if ($search_keyword)
 					{
 						$search_keyword = str_replace(' ', '%', $search_keyword);
 					}
@@ -785,12 +785,12 @@ 
 					break;
 
 				case 'is_published' :
-					if($search_keyword == 'Y')
+					if ($search_keyword == 'Y')
 					{
 						$args->s_is_published = 1;
 					}
 
-					if($search_keyword == 'N')
+					if ($search_keyword == 'N')
 					{
 						$args->s_is_published = 0;
 					}
@@ -802,7 +802,7 @@ 
 					break;
 
 				case 'member_srl' :
-					$args->{"s_" . $search_target} = (int) $search_keyword;
+					$args->{"s_".$search_target} = (int) $search_keyword;
 					break;
 			}
 		}
@@ -811,12 +811,12 @@ 
 		$output = executeQueryArray($query_id, $args, $columnList);
 
 		// return when no result or error occurance
-		if(!$output->toBool() || !count($output->data))
+		if (!$output->toBool() || !count($output->data))
 		{
 			return $output;
 		}
 
-		foreach($output->data as $key => $val)
+		foreach ($output->data as $key => $val)
 		{
 			unset($_oComment);
 			$_oComment = new CommentItem(0);
@@ -845,12 +845,12 @@ 
 		$search_target = $obj->search_target ? $obj->search_target : trim(Context::get('search_target'));
 		$search_keyword = $obj->search_keyword ? $obj->search_keyword : trim(Context::get('search_keyword'));
 
-		if($search_target && $search_keyword)
+		if ($search_target && $search_keyword)
 		{
-			switch($search_target)
+			switch ($search_target)
 			{
 				case 'content' :
-					if($search_keyword)
+					if ($search_keyword)
 					{
 						$search_keyword = str_replace(' ', '%', $search_keyword);
 					}
@@ -859,7 +859,7 @@ 
 					break;
 
 				case 'user_id' :
-					if($search_keyword)
+					if ($search_keyword)
 					{
 						$search_keyword = str_replace(' ', '%', $search_keyword);
 					}
@@ -869,7 +869,7 @@ 
 					break;
 
 				case 'user_name' :
-					if($search_keyword)
+					if ($search_keyword)
 					{
 						$search_keyword = str_replace(' ', '%', $search_keyword);
 					}
@@ -878,7 +878,7 @@ 
 					break;
 
 				case 'nick_name' :
-					if($search_keyword)
+					if ($search_keyword)
 					{
 						$search_keyword = str_replace(' ', '%', $search_keyword);
 					}
@@ -887,7 +887,7 @@ 
 					break;
 
 				case 'email_address' :
-					if($search_keyword)
+					if ($search_keyword)
 					{
 						$search_keyword = str_replace(' ', '%', $search_keyword);
 					}
@@ -896,7 +896,7 @@ 
 					break;
 
 				case 'homepage' :
-					if($search_keyword)
+					if ($search_keyword)
 					{
 						$search_keyword = str_replace(' ', '%', $search_keyword);
 					}
@@ -921,7 +921,7 @@ 
 					break;
 
 				case 'member_srl' :
-					$args->{"s_" . $search_target} = (int) $search_keyword;
+					$args->{"s_".$search_target} = (int) $search_keyword;
 					break;
 			}
 		}
@@ -929,7 +929,7 @@ 
 		$output = executeQueryArray($query_id, $args);
 
 		// return when no result or error occurance
-		if(!$output->toBool() || !count($output->data))
+		if (!$output->toBool() || !count($output->data))
 		{
 			return $output;
 		}
@@ -946,12 +946,12 @@ 
 	{
 		$oModuleModel = getModel('module');
 		$comment_config = $oModuleModel->getModulePartConfig('comment', $module_srl);
-		if(!is_object($comment_config))
+		if (!is_object($comment_config))
 		{
 			$comment_config = new stdClass();
 		}
 
-		if(!isset($comment_config->comment_count))
+		if (!isset($comment_config->comment_count))
 		{
 			$comment_config->comment_count = 50;
 		}
@@ -966,13 +966,13 @@ 
 	function getCommentVotedMemberList()
 	{
 		$comment_srl = Context::get('comment_srl');
-		if(!$comment_srl)
+		if (!$comment_srl)
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
 
 		$point = Context::get('point');
-		if($point != -1)
+		if ($point != -1)
 		{
 			$point = 1;
 		}
@@ -980,7 +980,7 @@ 
 		$oCommentModel = getModel('comment');
 		$oComment = $oCommentModel->getComment($comment_srl, FALSE, FALSE);
 		$module_srl = $oComment->get('module_srl');
-		if(!$module_srl)
+		if (!$module_srl)
 		{
 			return new BaseObject(-1, 'msg_invalid_request');
 		}
@@ -990,9 +990,9 @@ 
 
 		$args = new stdClass();
 
-		if($point == -1)
+		if ($point == -1)
 		{
-			if($comment_config->use_vote_down != 'S')
+			if ($comment_config->use_vote_down != 'S')
 			{
 				return new BaseObject(-1, 'msg_invalid_request');
 			}
@@ -1001,7 +1001,7 @@ 
 		}
 		else
 		{
-			if($comment_config->use_vote_up != 'S')
+			if ($comment_config->use_vote_up != 'S')
 			{
 				return new BaseObject(-1, 'msg_invalid_request');
 			}
@@ -1011,15 +1011,15 @@ 
 
 		$args->comment_srl = $comment_srl;
 		$output = executeQueryArray('comment.getVotedMemberList', $args);
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			return $output;
 		}
 
 		$oMemberModel = getModel('member');
-		if($output->data)
+		if ($output->data)
 		{
-			foreach($output->data as $k => $d)
+			foreach ($output->data as $k => $d)
 			{
 				$profile_image = $oMemberModel->getProfileImage($d->member_srl);
 				$output->data[$k]->src = $profile_image->src;
@@ -1037,7 +1037,7 @@ 
 	{
 		global $lang;
 
-		if(!isset($lang->secret_name_list))
+		if (!isset($lang->secret_name_list))
 		{
 			return array('Y' => 'Secret', 'N' => 'Public');
 		}
@@ -1078,8 +1078,8 @@ 
 		$output = executeQuery('comment.getCommentListByMemberSrl', $args, $columnList);
 		$comment_list = $output->data;
 
-		if(!$comment_list) return array();
-		if(!is_array($comment_list)) $comment_list = array($comment_list);
+		if (!$comment_list) return array();
+		if (!is_array($comment_list)) $comment_list = array($comment_list);
 
 		return $comment_list;