xpressengine / xe-core

modules/document/document.item.php

Spacing +177 added lines, -177 removed lines

@@ -74,7 +74,7 @@ 
 	 */
 	function _loadFromDB($load_extra_vars = true)
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 
 		$document_item = false;
 		$cache_put = false;
@@ -83,11 +83,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');
 			}
@@ -97,12 +97,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);
 			}
@@ -119,9 +119,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;
@@ -131,7 +131,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);
@@ -139,7 +139,7 @@ 
 		}
 
 		$oDocumentModel = getModel('document');
-		if($load_extra_vars)
+		if ($load_extra_vars)
 		{
 			$GLOBALS['XE_DOCUMENT_LIST'][$attribute->document_srl] = $this;
 			$oDocumentModel->setToAllDocumentExtraVars();
@@ -154,18 +154,18 @@ 
 
 	function isGranted()
 	{
-		if($_SESSION['own_document'][$this->document_srl]) return true;
+		if ($_SESSION['own_document'][$this->document_srl]) return true;
 
-		if(!Context::get('is_logged')) return false;
+		if (!Context::get('is_logged')) return false;
 
 		$logged_info = Context::get('logged_info');
-		if($logged_info->is_admin == 'Y') return true;
+		if ($logged_info->is_admin == 'Y') return true;
 
 		$oModuleModel = getModel('module');
 		$grant = $oModuleModel->getGrant($oModuleModel->getModuleInfoByModuleSrl($this->get('module_srl')), $logged_info);
-		if($grant->manager) return true;
+		if ($grant->manager) return true;
 
-		if($this->get('member_srl') && ($this->get('member_srl') == $logged_info->member_srl || $this->get('member_srl')*-1 == $logged_info->member_srl)) return true;
+		if ($this->get('member_srl') && ($this->get('member_srl') == $logged_info->member_srl || $this->get('member_srl') * -1 == $logged_info->member_srl)) return true;
 
 		return false;
 	}
@@ -177,13 +177,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;
 	}
@@ -191,11 +191,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;
 			}
@@ -205,20 +205,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;
 				}
 			}
 		}
@@ -227,14 +227,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;
 	}
 
@@ -251,13 +251,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();
 	}
 
@@ -284,18 +284,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
@@ -310,26 +310,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;
 	}
@@ -359,52 +359,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>';
 		}
@@ -414,26 +414,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]);
 			}
@@ -447,24 +447,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>',
@@ -473,10 +473,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;
 			}
@@ -492,12 +492,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;
@@ -546,7 +546,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;
 	}
@@ -559,13 +559,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()
@@ -575,7 +575,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')
@@ -585,13 +585,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()
@@ -601,21 +601,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('','document_srl',$this->get('document_srl'));
+		return getFullUrl('', '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());
 	}
 
 	/**
@@ -625,24 +625,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);
@@ -651,7 +651,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();
 		}
@@ -664,7 +664,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();
 		}
@@ -678,16 +678,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();
 		}
@@ -701,12 +701,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();
 		}
@@ -730,13 +730,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');
 		}
@@ -744,19 +744,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();
 			}
@@ -765,7 +765,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;
 
 		return $comment_list;
 	}
@@ -777,9 +777,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);
@@ -787,30 +787,30 @@ 
 
 	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
-		if(!$this->get('uploaded_count') && !preg_match("!<img!is", $this->get('content'))) return;
+		if (!$this->get('uploaded_count') && !preg_match("!<img!is", $this->get('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();
@@ -819,13 +819,13 @@ 
 			$thumbnail_type = $config->thumbnail_type;
 		}
 		// 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_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))
+		if (file_exists($thumbnail_file))
 		{
-			if(filesize($thumbnail_file)<1) return false;
+			if (filesize($thumbnail_file) < 1) return false;
 			else return $thumbnail_url;
 		}
 
@@ -834,33 +834,33 @@ 
 		$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;
 			}
@@ -868,32 +868,32 @@ 
 
 		// If not exists, file an image file from the content
 		$is_tmp_file = false;
-		if(!$source_file)
+		if (!$source_file)
 		{
 			$random = new Password();
 			$content = $this->get('content');
 
 			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;
 				}
@@ -904,15 +904,15 @@ 
 			}
 		}
 
-		if($source_file)
+		if ($source_file)
 		{
 			$output = FileHandler::createImageFile($source_file, $thumbnail_file, $width, $height, 'jpg', $thumbnail_type);
 		}
-		if($is_tmp_file) FileHandler::removeFile($source_file);
+		if ($is_tmp_file) FileHandler::removeFile($source_file);
 		// Return its path if a thumbnail is successfully genetated
-		if($output) return $thumbnail_url;
+		if ($output) return $thumbnail_url;
 		// Create an empty file not to re-generate the thumbnail
-		else FileHandler::writeFile($thumbnail_file, '','w');
+		else FileHandler::writeFile($thumbnail_file, '', 'w');
 
 		return;
 	}
@@ -925,21 +925,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');
@@ -962,14 +962,14 @@ 
 		 */
 
 		// Check the attachment
-		if($this->hasUploadedFiles()) $buffs[] = "file";
+		if ($this->hasUploadedFiles()) $buffs[] = "file";
 
 		return $buffs;
 	}
 
 	function getStatus()
 	{
-		if(!$this->get('status')) return $this->getDefaultStatus();
+		if (!$this->get('status')) return $this->getDefaultStatus();
 		return $this->get('status');
 	}
 
@@ -980,15 +980,15 @@ 
 	 */
 	function printExtraImages($time_check = 43200)
 	{
-		if(!$this->document_srl) return;
+		if (!$this->document_srl) return;
 		// Get the icon directory
-		$path = sprintf('%s%s',getUrl(), 'modules/document/tpl/icons/');
+		$path = sprintf('%s%s', getUrl(), 'modules/document/tpl/icons/');
 
 		$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);
 		}
@@ -997,20 +997,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);
@@ -1026,7 +1026,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');
@@ -1041,7 +1041,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;
 	}
@@ -1052,7 +1052,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');
@@ -1064,10 +1064,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;
 	}
@@ -1079,21 +1079,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;
@@ -1106,7 +1106,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]);
 	}
 
 	/**
@@ -1117,19 +1117,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 +276 added lines, -113 removed lines

@@ -74,7 +74,9 @@ 
 	 */
 	function _loadFromDB($load_extra_vars = true)
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 
 		$document_item = false;
 		$cache_put = false;
@@ -106,8 +108,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;
@@ -154,18 +155,28 @@ 
 
 	function isGranted()
 	{
-		if($_SESSION['own_document'][$this->document_srl]) return true;
+		if($_SESSION['own_document'][$this->document_srl]) {
+			return true;
+		}
 
-		if(!Context::get('is_logged')) return false;
+		if(!Context::get('is_logged')) {
+			return false;
+		}
 
 		$logged_info = Context::get('logged_info');
-		if($logged_info->is_admin == 'Y') return true;
+		if($logged_info->is_admin == 'Y') {
+			return true;
+		}
 
 		$oModuleModel = getModel('module');
 		$grant = $oModuleModel->getGrant($oModuleModel->getModuleInfoByModuleSrl($this->get('module_srl')), $logged_info);
-		if($grant->manager) return true;
+		if($grant->manager) {
+			return true;
+		}
 
-		if($this->get('member_srl') && ($this->get('member_srl') == $logged_info->member_srl || $this->get('member_srl')*-1 == $logged_info->member_srl)) return true;
+		if($this->get('member_srl') && ($this->get('member_srl') == $logged_info->member_srl || $this->get('member_srl')*-1 == $logged_info->member_srl)) {
+			return true;
+		}
 
 		return false;
 	}
@@ -183,7 +194,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;
 	}
@@ -198,8 +211,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');
@@ -210,15 +222,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;
+					}
 				}
 			}
 		}
@@ -227,14 +245,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;
 	}
 
@@ -256,9 +278,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()
@@ -284,16 +311,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');
@@ -320,16 +357,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;
 	}
@@ -361,43 +404,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);
 	}
@@ -430,14 +495,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]);
 			}
@@ -447,15 +510,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();
@@ -489,10 +560,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)
@@ -611,11 +683,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());
+		}
 	}
 
 	/**
@@ -634,7 +710,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;
@@ -642,7 +720,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);
@@ -654,8 +734,7 @@ 
 		if(is_array($extra_vars) && array_key_exists($idx,$extra_vars))
 		{
 			return $extra_vars[$idx]->getValue();
-		}
-		else
+		} else
 		{
 			return '';
 		}
@@ -667,8 +746,7 @@ 
 		if(is_array($extra_vars) && array_key_exists($idx,$extra_vars))
 		{
 			return $extra_vars[$idx]->getValueHTML();
-		}
-		else
+		} else
 		{
 			return '';
 		}
@@ -690,8 +768,7 @@ 
 		if(is_array($extra_eid) && array_key_exists($eid,$extra_eid))
 		{
 			return $extra_eid[$eid]->getValue();
-		}
-		else
+		} else
 		{
 			return '';
 		}
@@ -709,8 +786,7 @@ 
 		if(is_array($extra_eid) && array_key_exists($eid,$extra_eid))
 		{
 			return $extra_eid[$eid]->getValueHTML();
-		}
-		else
+		} else
 		{
 			return '';
 		}
@@ -730,8 +806,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);
@@ -744,7 +824,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();
@@ -754,7 +836,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)
 			{
@@ -765,7 +849,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;
+		}
 
 		return $comment_list;
 	}
@@ -777,9 +863,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);
@@ -787,15 +877,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())
 		{
@@ -803,9 +899,13 @@ 
 		}
 
 		// 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
-		if(!$this->get('uploaded_count') && !preg_match("!<img!is", $this->get('content'))) return;
+		if(!$this->get('uploaded_count') && !preg_match("!<img!is", $this->get('content'))) {
+			return;
+		}
 		// Get thumbnai_type information from document module's configuration
 		if(!in_array($thumbnail_type, array('crop','ratio')))
 		{
@@ -825,8 +925,11 @@ 
 		// Return false if thumbnail file exists and its size is 0. Otherwise, return its path
 		if(file_exists($thumbnail_file))
 		{
-			if(filesize($thumbnail_file)<1) return false;
-			else return $thumbnail_url;
+			if(filesize($thumbnail_file)<1) {
+				return false;
+			} else {
+				return $thumbnail_url;
+			}
 		}
 
 		// Target File
@@ -841,7 +944,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))
 				{
@@ -849,7 +954,9 @@ 
 					break;
 				}
 
-				if($first_image) continue;
+				if($first_image) {
+					continue;
+				}
 
 				if(preg_match("/\.(jpe?g|png|gif|bmp)$/i", $file->source_filename))
 				{
@@ -878,7 +985,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))
 				{
@@ -889,7 +998,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;
@@ -908,11 +1019,17 @@ 
 		{
 			$output = FileHandler::createImageFile($source_file, $thumbnail_file, $width, $height, 'jpg', $thumbnail_type);
 		}
-		if($is_tmp_file) FileHandler::removeFile($source_file);
+		if($is_tmp_file) {
+			FileHandler::removeFile($source_file);
+		}
 		// Return its path if a thumbnail is successfully genetated
-		if($output) return $thumbnail_url;
+		if($output) {
+			return $thumbnail_url;
+		}
 		// Create an empty file not to re-generate the thumbnail
-		else FileHandler::writeFile($thumbnail_file, '','w');
+		else {
+			FileHandler::writeFile($thumbnail_file, '','w');
+		}
 
 		return;
 	}
@@ -925,21 +1042,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');
@@ -962,14 +1086,18 @@ 
 		 */
 
 		// Check the attachment
-		if($this->hasUploadedFiles()) $buffs[] = "file";
+		if($this->hasUploadedFiles()) {
+			$buffs[] = "file";
+		}
 
 		return $buffs;
 	}
 
 	function getStatus()
 	{
-		if(!$this->get('status')) return $this->getDefaultStatus();
+		if(!$this->get('status')) {
+			return $this->getDefaultStatus();
+		}
 		return $this->get('status');
 	}
 
@@ -980,12 +1108,16 @@ 
 	 */
 	function printExtraImages($time_check = 43200)
 	{
-		if(!$this->document_srl) return;
+		if(!$this->document_srl) {
+			return;
+		}
 		// Get the icon directory
 		$path = sprintf('%s%s',getUrl(), 'modules/document/tpl/icons/');
 
 		$buffs = $this->getExtraImages($time_check);
-		if(!count($buffs)) return;
+		if(!count($buffs)) {
+			return;
+		}
 
 		$buff = array();
 		foreach($buffs as $key => $val)
@@ -997,18 +1129,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])
 		{
@@ -1026,7 +1168,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');
@@ -1040,8 +1184,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;
 	}
@@ -1052,7 +1200,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');
@@ -1064,10 +1214,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;
 	}
@@ -1079,7 +1233,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'));
@@ -1093,7 +1249,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;
@@ -1117,20 +1275,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/document/document.class.php

Spacing +91 added lines, -91 removed lines

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

modules/comment/comment.item.php

Spacing +69 added lines, -69 removed lines

@@ -49,7 +49,7 @@ 
 	 */
 	function _loadFromDB()
 	{
-		if(!$this->comment_srl)
+		if (!$this->comment_srl)
 		{
 			return;
 		}
@@ -67,7 +67,7 @@ 
 	 */
 	function setAttribute($attribute)
 	{
-		if(!$attribute->comment_srl)
+		if (!$attribute->comment_srl)
 		{
 			$this->comment_srl = NULL;
 			return;
@@ -77,9 +77,9 @@ 
 		$this->adds($attribute);
 
 		// define vars on the object for backward compatibility of skins
-		if(count($attribute))
+		if (count($attribute))
 		{
-			foreach($attribute as $key => $val)
+			foreach ($attribute as $key => $val)
 			{
 				$this->{$key} = $val;
 			}
@@ -93,29 +93,29 @@ 
 
 	function isGranted()
 	{
-		if($_SESSION['own_comment'][$this->comment_srl])
+		if ($_SESSION['own_comment'][$this->comment_srl])
 		{
 			return TRUE;
 		}
 
-		if(!Context::get('is_logged'))
+		if (!Context::get('is_logged'))
 		{
 			return FALSE;
 		}
 
 		$logged_info = Context::get('logged_info');
-		if($logged_info->is_admin == 'Y')
+		if ($logged_info->is_admin == 'Y')
 		{
 			return TRUE;
 		}
 
 		$grant = Context::get('grant');
-		if($grant->manager)
+		if ($grant->manager)
 		{
 			return TRUE;
 		}
 
-		if($this->get('member_srl') && ($this->get('member_srl') == $logged_info->member_srl || $this->get('member_srl') * -1 == $logged_info->member_srl))
+		if ($this->get('member_srl') && ($this->get('member_srl') == $logged_info->member_srl || $this->get('member_srl') * -1 == $logged_info->member_srl))
 		{
 			return TRUE;
 		}
@@ -136,7 +136,7 @@ 
 
 	function isEditable()
 	{
-		if($this->isGranted() || !$this->get('member_srl'))
+		if ($this->isGranted() || !$this->get('member_srl'))
 		{
 			return TRUE;
 		}
@@ -150,12 +150,12 @@ 
 
 	function isAccessible()
 	{
-		if($_SESSION['accessibled_comment'][$this->comment_srl])
+		if ($_SESSION['accessibled_comment'][$this->comment_srl])
 		{
 			return TRUE;
 		}
 
-		if($this->isGranted() || !$this->isSecret())
+		if ($this->isGranted() || !$this->isSecret())
 		{
 			$this->setAccessible();
 			return TRUE;
@@ -163,7 +163,7 @@ 
 
 		$oDocumentModel = getModel('document');
 		$oDocument = $oDocumentModel->getDocument($this->get('document_srl'));
-		if($oDocument->isGranted())
+		if ($oDocument->isGranted())
 		{
 			$this->setAccessible();
 			return TRUE;
@@ -184,20 +184,20 @@ 
 	function notify($type, $content)
 	{
 		// return if not useNotify
-		if(!$this->useNotify())
+		if (!$this->useNotify())
 		{
 			return;
 		}
 
 		// pass if the author is not logged-in user 
-		if(!$this->get('member_srl'))
+		if (!$this->get('member_srl'))
 		{
 			return;
 		}
 
 		// return if the currently logged-in user is an author of the comment.
 		$logged_info = Context::get('logged_info');
-		if($logged_info->member_srl == $this->get('member_srl'))
+		if ($logged_info->member_srl == $this->get('member_srl'))
 		{
 			return;
 		}
@@ -207,9 +207,9 @@ 
 		$oDocument = $oDocumentModel->getDocument($this->get('document_srl'));
 
 		// Variables
-		if($type)
+		if ($type)
 		{
-			$title = "[" . $type . "] ";
+			$title = "[".$type."] ";
 		}
 
 		$title .= cut_str(strip_tags($content), 30, '...');
@@ -224,17 +224,17 @@ 
 
 	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')))
+		if (trim($this->get('homepage')))
 		{
 			return TRUE;
 		}
@@ -245,14 +245,14 @@ 
 	function getHomepageUrl()
 	{
 		$url = trim($this->get('homepage'));
-		if(!$url)
+		if (!$url)
 		{
 			return;
 		}
 
-		if(strncasecmp('http://', $url, 7) !== 0)
+		if (strncasecmp('http://', $url, 7) !== 0)
 		{
-			$url = "http://" . $url;
+			$url = "http://".$url;
 		}
 
 		return htmlspecialchars($url, ENT_COMPAT | ENT_HTML401, 'UTF-8', false);
@@ -284,14 +284,14 @@ 
 	 */
 	function getContentText($strlen = 0)
 	{
-		if($this->isSecret() && !$this->isAccessible())
+		if ($this->isSecret() && !$this->isAccessible())
 		{
 			return Context::getLang('msg_is_secret');
 		}
 
 		$content = $this->get('content');
 
-		if($strlen)
+		if ($strlen)
 		{
 			return cut_str(strip_tags($content), $strlen, '...');
 		}
@@ -305,7 +305,7 @@ 
 	 */
 	function getContent($add_popup_menu = TRUE, $add_content_info = TRUE, $add_xe_content_class = TRUE)
 	{
-		if($this->isSecret() && !$this->isAccessible())
+		if ($this->isSecret() && !$this->isAccessible())
 		{
 			return Context::getLang('msg_is_secret');
 		}
@@ -314,7 +314,7 @@ 
 		stripEmbedTagForAdmin($content, $this->get('member_srl'));
 
 		// when displaying the comment on the pop-up menu
-		if($add_popup_menu && Context::get('is_logged'))
+		if ($add_popup_menu && Context::get('is_logged'))
 		{
 			$content = sprintf(
 					'%s<div class="comment_popup_menu"><a href="#popup_menu_area" class="comment_%d" onclick="return false">%s</a></div>', $content, $this->comment_srl, Context::getLang('cmd_comment_do')
@@ -322,10 +322,10 @@ 
 		}
 
 		// if additional information which can access contents is set
-		if($add_content_info)
+		if ($add_content_info)
 		{
 			$memberSrl = $this->get('member_srl');
-			if($memberSrl < 0)
+			if ($memberSrl < 0)
 			{
 				$memberSrl = 0;
 			}
@@ -336,7 +336,7 @@ 
 		}
 		else
 		{
-			if($add_xe_content_class)
+			if ($add_xe_content_class)
 			{
 				$content = sprintf('<div class="xe_content">%s</div>', $content);
 			}
@@ -396,7 +396,7 @@ 
 
 	function getRegdateGM()
 	{
-		return $this->getRegdate('D, d M Y H:i:s') . ' ' . $GLOBALS['_time_zone'];
+		return $this->getRegdate('D, d M Y H:i:s').' '.$GLOBALS['_time_zone'];
 	}
 
 	function getUpdate($format = 'Y.m.d H:i:s')
@@ -406,7 +406,7 @@ 
 
 	function getPermanentUrl()
 	{
-		return getFullUrl('', 'document_srl', $this->get('document_srl')) . '#comment_' . $this->get('comment_srl');
+		return getFullUrl('', 'document_srl', $this->get('document_srl')).'#comment_'.$this->get('comment_srl');
 	}
 
 	function getUpdateTime()
@@ -427,7 +427,7 @@ 
 
 	function hasUploadedFiles()
 	{
-		if(($this->isSecret() && !$this->isAccessible()) && !$this->isGranted())
+		if (($this->isSecret() && !$this->isAccessible()) && !$this->isGranted())
 		{
 			return FALSE;
 		}
@@ -436,12 +436,12 @@ 
 
 	function getUploadedFiles()
 	{
-		if(($this->isSecret() && !$this->isAccessible()) && !$this->isGranted())
+		if (($this->isSecret() && !$this->isAccessible()) && !$this->isGranted())
 		{
 			return;
 		}
 
-		if(!$this->get('uploaded_count'))
+		if (!$this->get('uploaded_count'))
 		{
 			return;
 		}
@@ -458,7 +458,7 @@ 
 	function getEditor()
 	{
 		$module_srl = $this->get('module_srl');
-		if(!$module_srl)
+		if (!$module_srl)
 		{
 			$module_srl = Context::get('module_srl');
 		}
@@ -472,13 +472,13 @@ 
 	 */
 	function getProfileImage()
 	{
-		if(!$this->isExists() || !$this->get('member_srl'))
+		if (!$this->isExists() || !$this->get('member_srl'))
 		{
 			return;
 		}
 		$oMemberModel = getModel('member');
 		$profile_info = $oMemberModel->getProfileImage($this->get('member_srl'));
-		if(!$profile_info)
+		if (!$profile_info)
 		{
 			return;
 		}
@@ -493,7 +493,7 @@ 
 	function getSignature()
 	{
 		// pass if the posting not exists.
-		if(!$this->isExists() || !$this->get('member_srl'))
+		if (!$this->isExists() || !$this->get('member_srl'))
 		{
 			return;
 		}
@@ -503,7 +503,7 @@ 
 		$signature = $oMemberModel->getSignature($this->get('member_srl'));
 
 		// check if max height of the signiture is specified on 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');
@@ -512,7 +512,7 @@ 
 
 		$max_signature_height = $GLOBALS['__member_signature_max_height'];
 
-		if($max_signature_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);
 		}
@@ -522,12 +522,12 @@ 
 
 	function thumbnailExists($width = 80, $height = 0, $type = '')
 	{
-		if(!$this->comment_srl)
+		if (!$this->comment_srl)
 		{
 			return FALSE;
 		}
 
-		if(!$this->getThumbnail($width, $height, $type))
+		if (!$this->getThumbnail($width, $height, $type))
 		{
 			return FALSE;
 		}
@@ -538,30 +538,30 @@ 
 	function getThumbnail($width = 80, $height = 0, $thumbnail_type = '')
 	{
 		// return false if no doc exists
-		if(!$this->comment_srl)
+		if (!$this->comment_srl)
 		{
 			return;
 		}
 
-		if($this->isSecret() && !$this->isGranted())
+		if ($this->isSecret() && !$this->isGranted())
 		{
 			return;
 		}
 
 		// If signiture height setting is omitted, create a square
-		if(!$height)
+		if (!$height)
 		{
 			$height = $width;
 		}
 
 		// return false if neigher attached file nor image;
-		if(!$this->hasUploadedFiles() && !preg_match("!<img!is", $this->get('content')))
+		if (!$this->hasUploadedFiles() && !preg_match("!<img!is", $this->get('content')))
 		{
 			return;
 		}
 
 		// get thumbail generation info on the doc module configuration.
-		if(!in_array($thumbnail_type, array('crop', 'ratio')))
+		if (!in_array($thumbnail_type, array('crop', 'ratio')))
 		{
 			$thumbnail_type = 'crop';
 		}
@@ -569,12 +569,12 @@ 
 		// Define thumbnail information
 		$thumbnail_path = sprintf('files/thumbnails/%s', getNumberingPath($this->comment_srl, 3));
 		$thumbnail_file = sprintf('%s%dx%d.%s.jpg', $thumbnail_path, $width, $height, $thumbnail_type);
-		$thumbnail_url = Context::getRequestUri() . $thumbnail_file;
+		$thumbnail_url = Context::getRequestUri().$thumbnail_file;
 
 		// return false if a size of existing thumbnail file is 0. otherwise return the file path
-		if(file_exists($thumbnail_file))
+		if (file_exists($thumbnail_file))
 		{
-			if(filesize($thumbnail_file) < 1)
+			if (filesize($thumbnail_file) < 1)
 			{
 				return FALSE;
 			}
@@ -589,33 +589,33 @@ 
 		$is_tmp_file = FALSE;
 
 		// find an image file among attached files
-		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;
 			}
@@ -623,32 +623,32 @@ 
 
 		// get an image file from the doc content if no file attached. 
 		$is_tmp_file = false;
-		if(!$source_file)
+		if (!$source_file)
 		{
 			$random = new Password();
 			$content = $this->get('content');
 
 			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;
 				}
@@ -661,13 +661,13 @@ 
 
 		$output = FileHandler::createImageFile($source_file, $thumbnail_file, $width, $height, 'jpg', $thumbnail_type);
 
-		if($is_tmp_file)
+		if ($is_tmp_file)
 		{
 			FileHandler::removeFile($source_file);
 		}
 
 		// return the thumbnail path if successfully generated.
-		if($output)
+		if ($output)
 		{
 			return $thumbnail_url;
 		}

Braces +14 added lines, -8 removed lines

@@ -333,8 +333,7 @@ 
 					'<!--BeforeComment(%d,%d)--><div class="comment_%d_%d xe_content">%s</div><!--AfterComment(%d,%d)-->', $this->comment_srl, $memberSrl, $this->comment_srl, $memberSrl, $content, $this->comment_srl, $memberSrl
 			);
 			// xe_content class name should be specified although content access is not necessary.
-		}
-		else
+		} else
 		{
 			if($add_xe_content_class)
 			{
@@ -577,8 +576,7 @@ 
 			if(filesize($thumbnail_file) < 1)
 			{
 				return FALSE;
-			}
-			else
+			} else
 			{
 				return $thumbnail_url;
 			}
@@ -596,7 +594,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))
 				{
@@ -604,7 +604,9 @@ 
 					break;
 				}
 
-				if($first_image) continue;
+				if($first_image) {
+					continue;
+				}
 
 				if(preg_match("/\.(jpe?g|png|gif|bmp)$/i", $file->source_filename))
 				{
@@ -633,7 +635,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))
 				{
@@ -644,7 +648,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;

modules/comment/comment.class.php

Spacing +25 added lines, -25 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 
-require_once(_XE_PATH_ . 'modules/comment/comment.item.php');
+require_once(_XE_PATH_.'modules/comment/comment.item.php');
 
 /**
  * comment
@@ -36,7 +36,7 @@ 
 		// 2008. 02. 22 add comment setting when a new module added
 		$oModuleController->insertTrigger('module.dispAdditionSetup', 'comment', 'view', 'triggerDispCommentAdditionSetup', 'before');
 
-		if(!is_dir('./files/cache/tmp'))
+		if (!is_dir('./files/cache/tmp'))
 		{
 			FileHandler::makeDir('./files/cache/tmp');
 		}
@@ -53,55 +53,55 @@ 
 		$oDB = DB::getInstance();
 		$oModuleModel = getModel('module');
 		// 2007. 10. 17 add a trigger to delete comments together with posting deleted
-		if(!$oModuleModel->getTrigger('document.deleteDocument', 'comment', 'controller', 'triggerDeleteDocumentComments', 'after'))
+		if (!$oModuleModel->getTrigger('document.deleteDocument', 'comment', 'controller', 'triggerDeleteDocumentComments', 'after'))
 		{
 			return TRUE;
 		}
 		// 2007. 10. 17 add a trigger to delete all of comments together with module deleted
-		if(!$oModuleModel->getTrigger('module.deleteModule', 'comment', 'controller', 'triggerDeleteModuleComments', 'after'))
+		if (!$oModuleModel->getTrigger('module.deleteModule', 'comment', 'controller', 'triggerDeleteModuleComments', 'after'))
 		{
 			return TRUE;
 		}
 		// 2007. 10. 23 add a column for recommendation votes or notification of the comments
-		if(!$oDB->isColumnExists("comments", "voted_count"))
+		if (!$oDB->isColumnExists("comments", "voted_count"))
 		{
 			return TRUE;
 		}
-		if(!$oDB->isColumnExists("comments", "notify_message"))
+		if (!$oDB->isColumnExists("comments", "notify_message"))
 		{
 			return TRUE;
 		}
 		// 2008. 02. 22 add comment setting when a new module added
-		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'comment', 'view', 'triggerDispCommentAdditionSetup', 'before'))
+		if (!$oModuleModel->getTrigger('module.dispAdditionSetup', 'comment', 'view', 'triggerDispCommentAdditionSetup', 'before'))
 		{
 			return TRUE;
 		}
 		// 2008. 05. 14 add a column for blamed count
-		if(!$oDB->isColumnExists("comments", "blamed_count"))
+		if (!$oDB->isColumnExists("comments", "blamed_count"))
 		{
 			return TRUE;
 		}
-		if(!$oDB->isColumnExists("comment_voted_log", "point"))
+		if (!$oDB->isColumnExists("comment_voted_log", "point"))
 		{
 			return TRUE;
 		}
 
-		if(!$oDB->isIndexExists("comments", "idx_module_list_order"))
+		if (!$oDB->isIndexExists("comments", "idx_module_list_order"))
 		{
 			return TRUE;
 		}
 		//2012. 02. 24 add comment published status column and index
-		if(!$oDB->isColumnExists("comments", "status"))
+		if (!$oDB->isColumnExists("comments", "status"))
 		{
 			return TRUE;
 		}
-		if(!$oDB->isIndexExists("comments", "idx_status"))
+		if (!$oDB->isIndexExists("comments", "idx_status"))
 		{
 			return TRUE;
 		}
 
 		// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-		if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'comment', 'controller', 'triggerCopyModule', 'after'))
+		if (!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'comment', 'controller', 'triggerCopyModule', 'after'))
 		{
 			return TRUE;
 		}
@@ -119,43 +119,43 @@ 
 		$oModuleModel = getModel('module');
 		$oModuleController = getController('module');
 		// 2007. 10. 17 add a trigger to delete comments together with posting deleted
-		if(!$oModuleModel->getTrigger('document.deleteDocument', 'comment', 'controller', 'triggerDeleteDocumentComments', 'after'))
+		if (!$oModuleModel->getTrigger('document.deleteDocument', 'comment', 'controller', 'triggerDeleteDocumentComments', 'after'))
 		{
 			$oModuleController->insertTrigger('document.deleteDocument', 'comment', 'controller', 'triggerDeleteDocumentComments', 'after');
 		}
 		// 2007. 10. 17 add a trigger to delete all of comments together with module deleted
-		if(!$oModuleModel->getTrigger('module.deleteModule', 'comment', 'controller', 'triggerDeleteModuleComments', 'after'))
+		if (!$oModuleModel->getTrigger('module.deleteModule', 'comment', 'controller', 'triggerDeleteModuleComments', 'after'))
 		{
 			$oModuleController->insertTrigger('module.deleteModule', 'comment', 'controller', 'triggerDeleteModuleComments', 'after');
 		}
 		// 2007. 10. 23 add a column for recommendation votes or notification of the comments
-		if(!$oDB->isColumnExists("comments", "voted_count"))
+		if (!$oDB->isColumnExists("comments", "voted_count"))
 		{
 			$oDB->addColumn("comments", "voted_count", "number", "11");
 			$oDB->addIndex("comments", "idx_voted_count", array("voted_count"));
 		}
 
-		if(!$oDB->isColumnExists("comments", "notify_message"))
+		if (!$oDB->isColumnExists("comments", "notify_message"))
 		{
 			$oDB->addColumn("comments", "notify_message", "char", "1");
 		}
 		// 2008. 02. 22 add comment setting when a new module added
-		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'comment', 'view', 'triggerDispCommentAdditionSetup', 'before'))
+		if (!$oModuleModel->getTrigger('module.dispAdditionSetup', 'comment', 'view', 'triggerDispCommentAdditionSetup', 'before'))
 		{
 			$oModuleController->insertTrigger('module.dispAdditionSetup', 'comment', 'view', 'triggerDispCommentAdditionSetup', 'before');
 		}
 		// 2008. 05. 14 add a column for blamed count
-		if(!$oDB->isColumnExists("comments", "blamed_count"))
+		if (!$oDB->isColumnExists("comments", "blamed_count"))
 		{
 			$oDB->addColumn('comments', 'blamed_count', 'number', 11, 0, TRUE);
 			$oDB->addIndex('comments', 'idx_blamed_count', array('blamed_count'));
 		}
-		if(!$oDB->isColumnExists("comment_voted_log", "point"))
+		if (!$oDB->isColumnExists("comment_voted_log", "point"))
 		{
 			$oDB->addColumn('comment_voted_log', 'point', 'number', 11, 0, TRUE);
 		}
 
-		if(!$oDB->isIndexExists("comments", "idx_module_list_order"))
+		if (!$oDB->isIndexExists("comments", "idx_module_list_order"))
 		{
 			$oDB->addIndex(
 					"comments", "idx_module_list_order", array("module_srl", "list_order"), TRUE
@@ -163,11 +163,11 @@ 
 		}
 
 		//2012. 02. 24 add comment published status column and index
-		if(!$oDB->isColumnExists("comments", "status"))
+		if (!$oDB->isColumnExists("comments", "status"))
 		{
 			$oDB->addColumn("comments", "status", "number", 1, 1, TRUE);
 		}
-		if(!$oDB->isIndexExists("comments", "idx_status"))
+		if (!$oDB->isIndexExists("comments", "idx_status"))
 		{
 			$oDB->addIndex(
 					"comments", "idx_status", array("status", "comment_srl", "module_srl", "document_srl"), TRUE
@@ -175,7 +175,7 @@ 
 		}
 
 		// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-		if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'comment', 'controller', 'triggerCopyModule', 'after'))
+		if (!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'comment', 'controller', 'triggerCopyModule', 'after'))
 		{
 			$oModuleController->insertTrigger('module.procModuleAdminCopyModule', 'comment', 'controller', 'triggerCopyModule', 'after');
 		}
@@ -189,7 +189,7 @@ 
 	 */
 	function recompileCache()
 	{
-		if(!is_dir('./files/cache/tmp'))
+		if (!is_dir('./files/cache/tmp'))
 		{
 			FileHandler::makeDir('./files/cache/tmp');
 		}

classes/file/FileHandler.class.php

Spacing +122 added lines, -122 removed lines

@@ -17,9 +17,9 @@ 
 	 */
 	function getRealPath($source)
 	{
-		if(strlen($source) >= 2 && substr_compare($source, './', 0, 2) === 0)
+		if (strlen($source) >= 2 && substr_compare($source, './', 0, 2) === 0)
 		{
-			return _XE_PATH_ . substr($source, 2);
+			return _XE_PATH_.substr($source, 2);
 		}
 
 		return $source;
@@ -40,7 +40,7 @@ 
 	{
 		$source_dir = self::getRealPath($source_dir);
 		$target_dir = self::getRealPath($target_dir);
-		if(!is_dir($source_dir))
+		if (!is_dir($source_dir))
 		{
 			return FALSE;
 		}
@@ -48,44 +48,44 @@ 
 		// generate when no target exists
 		self::makeDir($target_dir);
 
-		if(substr($source_dir, -1) != DIRECTORY_SEPARATOR)
+		if (substr($source_dir, -1) != DIRECTORY_SEPARATOR)
 		{
 			$source_dir .= DIRECTORY_SEPARATOR;
 		}
 
-		if(substr($target_dir, -1) != DIRECTORY_SEPARATOR)
+		if (substr($target_dir, -1) != DIRECTORY_SEPARATOR)
 		{
 			$target_dir .= DIRECTORY_SEPARATOR;
 		}
 
 		$oDir = dir($source_dir);
-		while($file = $oDir->read())
+		while ($file = $oDir->read())
 		{
-			if($file{0} == '.')
+			if ($file{0} == '.')
 			{
 				continue;
 			}
 
-			if($filter && preg_match($filter, $file))
+			if ($filter && preg_match($filter, $file))
 			{
 				continue;
 			}
 
-			if(is_dir($source_dir . $file))
+			if (is_dir($source_dir.$file))
 			{
-				self::copyDir($source_dir . $file, $target_dir . $file, $type);
+				self::copyDir($source_dir.$file, $target_dir.$file, $type);
 			}
 			else
 			{
-				if($type == 'force')
+				if ($type == 'force')
 				{
-					@unlink($target_dir . $file);
+					@unlink($target_dir.$file);
 				}
 				else
 				{
-					if(!file_exists($target_dir . $file))
+					if (!file_exists($target_dir.$file))
 					{
-						@copy($source_dir . $file, $target_dir . $file);
+						@copy($source_dir.$file, $target_dir.$file);
 					}
 				}
 			}
@@ -110,12 +110,12 @@ 
 
 		self::makeDir($target_dir);
 
-		if($force == 'Y')
+		if ($force == 'Y')
 		{
-			@unlink($target_dir . DIRECTORY_SEPARATOR . $target);
+			@unlink($target_dir.DIRECTORY_SEPARATOR.$target);
 		}
 
-		@copy($source, $target_dir . DIRECTORY_SEPARATOR . $target);
+		@copy($source, $target_dir.DIRECTORY_SEPARATOR.$target);
 	}
 
 	/**
@@ -126,7 +126,7 @@ 
 	 */
 	function readFile($filename)
 	{
-		if(($filename = self::exists($filename)) === FALSE || filesize($filename) < 1)
+		if (($filename = self::exists($filename)) === FALSE || filesize($filename) < 1)
 		{
 			return;
 		}
@@ -149,12 +149,12 @@ 
 		self::makeDir($pathinfo['dirname']);
 
 		$flags = 0;
-		if(strtolower($mode) == 'a')
+		if (strtolower($mode) == 'a')
 		{
 			$flags = FILE_APPEND;
 		}
 
-		@file_put_contents($filename, $buff, $flags|LOCK_EX);
+		@file_put_contents($filename, $buff, $flags | LOCK_EX);
 		@chmod($filename, 0644);
 	}
 
@@ -192,7 +192,7 @@ 
 	 */
 	function moveFile($source, $target)
 	{
-		if(($source = self::exists($source)) !== FALSE)
+		if (($source = self::exists($source)) !== FALSE)
 		{
 			self::removeFile($target);
 			return self::rename($source, $target);
@@ -230,35 +230,35 @@ 
 		$path = self::getRealPath($path);
 		$output = array();
 
-		if(substr($path, -1) != '/')
+		if (substr($path, -1) != '/')
 		{
 			$path .= '/';
 		}
 
-		if(!is_dir($path))
+		if (!is_dir($path))
 		{
 			return $output;
 		}
 
 		$files = scandir($path);
-		foreach($files as $file)
+		foreach ($files as $file)
 		{
-			if($file{0} == '.' || ($filter && !preg_match($filter, $file)))
+			if ($file{0} == '.' || ($filter && !preg_match($filter, $file)))
 			{
 				continue;
 			}
 
-			if($to_lower)
+			if ($to_lower)
 			{
 				$file = strtolower($file);
 			}
 
-			if($filter)
+			if ($filter)
 			{
 				$file = preg_replace($filter, '$1', $file);
 			}
 
-			if($concat_prefix)
+			if ($concat_prefix)
 			{
 				$file = sprintf('%s%s', str_replace(_XE_PATH_, '', $path), $file);
 			}
@@ -279,12 +279,12 @@ 
 	 */
 	function makeDir($path_string)
 	{
-		if(self::exists($path_string) !== FALSE)
+		if (self::exists($path_string) !== FALSE)
 		{
 			return TRUE;
 		}
 
-		if(!ini_get('safe_mode'))
+		if (!ini_get('safe_mode'))
 		{
 			@mkdir($path_string, 0755, TRUE);
 			@chmod($path_string, 0755);
@@ -295,35 +295,35 @@ 
 			static $oFtp = NULL;
 
 			$ftp_info = Context::getFTPInfo();
-			if($oFtp == NULL)
+			if ($oFtp == NULL)
 			{
-				if(!Context::isFTPRegisted())
+				if (!Context::isFTPRegisted())
 				{
 					return;
 				}
 
-				require_once(_XE_PATH_ . 'libs/ftp.class.php');
+				require_once(_XE_PATH_.'libs/ftp.class.php');
 				$oFtp = new ftp();
-				if(!$ftp_info->ftp_host)
+				if (!$ftp_info->ftp_host)
 				{
 					$ftp_info->ftp_host = "127.0.0.1";
 				}
-				if(!$ftp_info->ftp_port)
+				if (!$ftp_info->ftp_port)
 				{
 					$ftp_info->ftp_port = 21;
 				}
-				if(!$oFtp->ftp_connect($ftp_info->ftp_host, $ftp_info->ftp_port))
+				if (!$oFtp->ftp_connect($ftp_info->ftp_host, $ftp_info->ftp_port))
 				{
 					return;
 				}
-				if(!$oFtp->ftp_login($ftp_info->ftp_user, $ftp_info->ftp_password))
+				if (!$oFtp->ftp_login($ftp_info->ftp_user, $ftp_info->ftp_password))
 				{
 					$oFtp->ftp_quit();
 					return;
 				}
 			}
 
-			if(!($ftp_path = $ftp_info->ftp_root_path))
+			if (!($ftp_path = $ftp_info->ftp_root_path))
 			{
 				$ftp_path = DIRECTORY_SEPARATOR;
 			}
@@ -332,19 +332,19 @@ 
 			$path_list = explode(DIRECTORY_SEPARATOR, $path_string);
 
 			$path = _XE_PATH_;
-			for($i = 0, $c = count($path_list); $i < $c; $i++)
+			for ($i = 0, $c = count($path_list); $i < $c; $i++)
 			{
-				if(!$path_list[$i])
+				if (!$path_list[$i])
 				{
 					continue;
 				}
 
-				$path .= $path_list[$i] . DIRECTORY_SEPARATOR;
-				$ftp_path .= $path_list[$i] . DIRECTORY_SEPARATOR;
-				if(!is_dir($path))
+				$path .= $path_list[$i].DIRECTORY_SEPARATOR;
+				$ftp_path .= $path_list[$i].DIRECTORY_SEPARATOR;
+				if (!is_dir($path))
 				{
 					$oFtp->ftp_mkdir($ftp_path);
-					$oFtp->ftp_site("CHMOD 777 " . $ftp_path);
+					$oFtp->ftp_site("CHMOD 777 ".$ftp_path);
 				}
 			}
 		}
@@ -360,23 +360,23 @@ 
 	 */
 	function removeDir($path)
 	{
-		if(($path = self::isDir($path)) === FALSE)
+		if (($path = self::isDir($path)) === FALSE)
 		{
 			return;
 		}
 
-		if(self::isDir($path))
+		if (self::isDir($path))
 		{
 			$files = array_diff(scandir($path), array('..', '.'));
 
-			foreach($files as $file)
+			foreach ($files as $file)
 			{
-				if(($target = self::getRealPath($path . DIRECTORY_SEPARATOR . $file)) === FALSE)
+				if (($target = self::getRealPath($path.DIRECTORY_SEPARATOR.$file)) === FALSE)
 				{
 					continue;
 				}
 
-				if(is_dir($target))
+				if (is_dir($target))
 				{
 					self::removeDir($target);
 				}
@@ -401,22 +401,22 @@ 
 	 */
 	function removeBlankDir($path)
 	{
-		if(($path = self::isDir($path)) === FALSE)
+		if (($path = self::isDir($path)) === FALSE)
 		{
 			return;
 		}
 
 		$files = array_diff(scandir($path), array('..', '.'));
 
-		if(count($files) < 1)
+		if (count($files) < 1)
 		{
 			rmdir($path);
 			return;
 		}
 
-		foreach($files as $file)
+		foreach ($files as $file)
 		{
-			if(($target = self::isDir($path . DIRECTORY_SEPARATOR . $file)) === FALSE)
+			if (($target = self::isDir($path.DIRECTORY_SEPARATOR.$file)) === FALSE)
 			{
 				continue;
 			}
@@ -435,23 +435,23 @@ 
 	 */
 	function removeFilesInDir($path)
 	{
-		if(($path = self::getRealPath($path)) === FALSE)
+		if (($path = self::getRealPath($path)) === FALSE)
 		{
 			return;
 		}
 
-		if(is_dir($path))
+		if (is_dir($path))
 		{
 			$files = array_diff(scandir($path), array('..', '.'));
 
-			foreach($files as $file)
+			foreach ($files as $file)
 			{
-				if(($target = self::getRealPath($path . DIRECTORY_SEPARATOR . $file)) === FALSE)
+				if (($target = self::getRealPath($path.DIRECTORY_SEPARATOR.$file)) === FALSE)
 				{
 					continue;
 				}
 
-				if(is_dir($target))
+				if (is_dir($target))
 				{
 					self::removeFilesInDir($target);
 				}
@@ -463,7 +463,7 @@ 
 		}
 		else
 		{
-			if(self::exists($path)) unlink($path);
+			if (self::exists($path)) unlink($path);
 		}
 
 	}
@@ -477,22 +477,22 @@ 
 	 */
 	function filesize($size)
 	{
-		if(!$size)
+		if (!$size)
 		{
 			return '0Byte';
 		}
 
-		if($size === 1)
+		if ($size === 1)
 		{
 			return '1Byte';
 		}
 
-		if($size < 1024)
+		if ($size < 1024)
 		{
-			return $size . 'Bytes';
+			return $size.'Bytes';
 		}
 
-		if($size >= 1024 && $size < 1024 * 1024)
+		if ($size >= 1024 && $size < 1024 * 1024)
 		{
 			return sprintf("%0.1fKB", $size / 1024);
 		}
@@ -517,7 +517,7 @@ 
 	 */
 	function getRemoteResource($url, $body = null, $timeout = 3, $method = 'GET', $content_type = null, $headers = array(), $cookies = array(), $post_data = array(), $request_config = array())
 	{
-		require_once(_XE_PATH_ . 'libs/idna_convert/idna_convert.class.php');
+		require_once(_XE_PATH_.'libs/idna_convert/idna_convert.class.php');
 		$IDN = new idna_convert(array('idn_version' => 2008));
 		$url = $IDN->encode($url);
 
@@ -527,7 +527,7 @@ 
 			require_once('HTTP/Request.php');
 
 			$parsed_url = parse_url(__PROXY_SERVER__);
-			if($parsed_url["host"])
+			if ($parsed_url["host"])
 			{
 				$oRequest = new HTTP_Request(__PROXY_SERVER__);
 				$oRequest->setMethod('POST');
@@ -537,49 +537,49 @@ 
 			{
 				$oRequest = new HTTP_Request($url);
 
-				if(count($request_config) && method_exists($oRequest, 'setConfig'))
+				if (count($request_config) && method_exists($oRequest, 'setConfig'))
 				{
-					foreach($request_config as $key=>$val)
+					foreach ($request_config as $key=>$val)
 					{
 						$oRequest->setConfig($key, $val);
 					}
 				}
 
-				if(count($headers) > 0)
+				if (count($headers) > 0)
 				{
-					foreach($headers as $key => $val)
+					foreach ($headers as $key => $val)
 					{
 						$oRequest->addHeader($key, $val);
 					}
 				}
-				if($cookies[$host])
+				if ($cookies[$host])
 				{
-					foreach($cookies[$host] as $key => $val)
+					foreach ($cookies[$host] as $key => $val)
 					{
 						$oRequest->addCookie($key, $val);
 					}
 				}
-				if(count($post_data) > 0)
+				if (count($post_data) > 0)
 				{
-					foreach($post_data as $key => $val)
+					foreach ($post_data as $key => $val)
 					{
 						$oRequest->addPostData($key, $val);
 					}
 				}
-				if(!$content_type)
+				if (!$content_type)
 					$oRequest->addHeader('Content-Type', 'text/html');
 				else
 					$oRequest->addHeader('Content-Type', $content_type);
 				$oRequest->setMethod($method);
-				if($body)
+				if ($body)
 					$oRequest->setBody($body);
 			}
 			
-			if(method_exists($oRequest, 'setConfig'))
+			if (method_exists($oRequest, 'setConfig'))
 			{
 				$oRequest->setConfig('timeout', $timeout);
 			}
-			elseif(property_exists($oRequest, '_timeout'))
+			elseif (property_exists($oRequest, '_timeout'))
 			{
 				$oRequest->_timeout = $timeout;
 			}
@@ -589,25 +589,25 @@ 
 			$code = $oRequest->getResponseCode();
 			$header = $oRequest->getResponseHeader();
 			$response = $oRequest->getResponseBody();
-			if($c = $oRequest->getResponseCookies())
+			if ($c = $oRequest->getResponseCookies())
 			{
-				foreach($c as $k => $v)
+				foreach ($c as $k => $v)
 				{
 					$cookies[$host][$v['name']] = $v['value'];
 				}
 			}
 
-			if($code > 300 && $code < 399 && $header['location'])
+			if ($code > 300 && $code < 399 && $header['location'])
 			{
 				return self::getRemoteResource($header['location'], $body, $timeout, $method, $content_type, $headers, $cookies, $post_data);
 			}
 
-			if($code != 200)
+			if ($code != 200)
 				return;
 
 			return $response;
 		}
-		catch(Exception $e)
+		catch (Exception $e)
 		{
 			return NULL;
 		}
@@ -627,7 +627,7 @@ 
 	 */
 	function getRemoteFile($url, $target_filename, $body = null, $timeout = 3, $method = 'GET', $content_type = null, $headers = array(), $cookies = array(), $post_data = array(), $request_config = array())
 	{
-		if(!($body = self::getRemoteResource($url, $body, $timeout, $method, $content_type, $headers,$cookies,$post_data,$request_config)))
+		if (!($body = self::getRemoteResource($url, $body, $timeout, $method, $content_type, $headers, $cookies, $post_data, $request_config)))
 		{
 			return FALSE;
 		}
@@ -646,7 +646,7 @@ 
 	function returnBytes($val)
 	{
 		$unit = strtoupper(substr($val, -1));
-		$val = (float)$val;
+		$val = (float) $val;
 
 		switch ($unit)
 		{
@@ -669,13 +669,13 @@ 
 		$K64 = 65536;
 		$TWEAKFACTOR = 2.0;
 		$channels = $imageInfo['channels'];
-		if(!$channels)
+		if (!$channels)
 		{
 			$channels = 6; //for png
 		}
-		$memoryNeeded = round(($imageInfo[0] * $imageInfo[1] * $imageInfo['bits'] * $channels / 8 + $K64 ) * $TWEAKFACTOR);
+		$memoryNeeded = round(($imageInfo[0] * $imageInfo[1] * $imageInfo['bits'] * $channels / 8 + $K64) * $TWEAKFACTOR);
 		$availableMemory = self::returnBytes(ini_get('memory_limit')) - memory_get_usage();
-		if($availableMemory < $memoryNeeded)
+		if ($availableMemory < $memoryNeeded)
 		{
 			return FALSE;
 		}
@@ -702,30 +702,30 @@ 
 		}
 
 		$target_file = self::getRealPath($target_file);
-		if(!$resize_width)
+		if (!$resize_width)
 		{
 			$resize_width = 100;
 		}
 
-		if(!$resize_height)
+		if (!$resize_height)
 		{
 			$resize_height = $resize_width;
 		}
 
 		// retrieve source image's information
 		$imageInfo = getimagesize($source_file);
-		if(!self::checkMemoryLoadImage($imageInfo))
+		if (!self::checkMemoryLoadImage($imageInfo))
 		{
 			return FALSE;
 		}
 
 		list($width, $height, $type, $attrs) = $imageInfo;
-		if($width < 1 || $height < 1)
+		if ($width < 1 || $height < 1)
 		{
 			return;
 		}
 
-		switch($type)
+		switch ($type)
 		{
 			case '1' :
 				$type = 'gif';
@@ -743,7 +743,7 @@ 
 				return;
 		}
 
-		if(!$target_type)
+		if (!$target_type)
 		{
 			$target_type = $type;
 		}
@@ -754,7 +754,7 @@ 
 		$height_per = ($resize_height > 0 && $height >= $resize_height) ? $resize_height / $height : 1;
 
 		$per = NULL;
-		if($thumbnail_type == 'ratio')
+		if ($thumbnail_type == 'ratio')
 		{
 			$per = ($width_per > $height_per) ? $height_per : $width_per;
 			$resize_width = $width * $per;
@@ -767,16 +767,16 @@ 
 
 		// create temporary image with target size
 		$thumb = NULL;
-		if(function_exists('imagecreateTRUEcolor'))
+		if (function_exists('imagecreateTRUEcolor'))
 		{
 			$thumb = imagecreateTRUEcolor($resize_width, $resize_height);
 		}
-		else if(function_exists('imagecreate'))
+		else if (function_exists('imagecreate'))
 		{
 			$thumb = imagecreate($resize_width, $resize_height);
 		}
 
-		if(!$thumb)
+		if (!$thumb)
 		{
 			return FALSE;
 		}
@@ -785,37 +785,37 @@ 
 
 		// create temporary image having original type
 		$source = NULL;
-		switch($type)
+		switch ($type)
 		{
 			case 'gif' :
-				if(function_exists('imagecreatefromgif'))
+				if (function_exists('imagecreatefromgif'))
 				{
 					$source = @imagecreatefromgif($source_file);
 				}
 				break;
 			case 'jpeg' :
 			case 'jpg' :
-				if(function_exists('imagecreatefromjpeg'))
+				if (function_exists('imagecreatefromjpeg'))
 				{
 					$source = @imagecreatefromjpeg($source_file);
 				}
 				break;
 			case 'png' :
-				if(function_exists('imagecreatefrompng'))
+				if (function_exists('imagecreatefrompng'))
 				{
 					$source = @imagecreatefrompng($source_file);
 				}
 				break;
 			case 'wbmp' :
 			case 'bmp' :
-				if(function_exists('imagecreatefromwbmp'))
+				if (function_exists('imagecreatefromwbmp'))
 				{
 					$source = @imagecreatefromwbmp($source_file);
 				}
 				break;
 		}
 
-		if(!$source)
+		if (!$source)
 		{
 			imagedestroy($thumb);
 			return FALSE;
@@ -827,13 +827,13 @@ 
 
 		$x = 0;
 		$y = 0;
-		if($thumbnail_type == 'crop')
+		if ($thumbnail_type == 'crop')
 		{
 			$x = (int) ($resize_width / 2 - $new_width / 2);
 			$y = (int) ($resize_height / 2 - $new_height / 2);
 		}
 
-		if(function_exists('imagecopyresampled'))
+		if (function_exists('imagecopyresampled'))
 		{
 			imagecopyresampled($thumb, $source, $x, $y, 0, 0, $new_width, $new_height, $width, $height);
 		}
@@ -847,30 +847,30 @@ 
 
 		// write into the file
 		$output = NULL;
-		switch($target_type)
+		switch ($target_type)
 		{
 			case 'gif' :
-				if(function_exists('imagegif'))
+				if (function_exists('imagegif'))
 				{
 					$output = imagegif($thumb, $target_file);
 				}
 				break;
 			case 'jpeg' :
 			case 'jpg' :
-				if(function_exists('imagejpeg'))
+				if (function_exists('imagejpeg'))
 				{
 					$output = imagejpeg($thumb, $target_file, 100);
 				}
 				break;
 			case 'png' :
-				if(function_exists('imagepng'))
+				if (function_exists('imagepng'))
 				{
 					$output = imagepng($thumb, $target_file, 9);
 				}
 				break;
 			case 'wbmp' :
 			case 'bmp' :
-				if(function_exists('imagewbmp'))
+				if (function_exists('imagewbmp'))
 				{
 					$output = imagewbmp($thumb, $target_file, 100);
 				}
@@ -880,7 +880,7 @@ 
 		imagedestroy($thumb);
 		imagedestroy($source);
 
-		if(!$output)
+		if (!$output)
 		{
 			return FALSE;
 		}
@@ -898,12 +898,12 @@ 
 	 */
 	function readIniFile($filename)
 	{
-		if(($filename = self::exists($filename)) === FALSE)
+		if (($filename = self::exists($filename)) === FALSE)
 		{
 			return FALSE;
 		}
 		$arr = parse_ini_file($filename, TRUE);
-		if(is_array($arr) && count($arr) > 0)
+		if (is_array($arr) && count($arr) > 0)
 		{
 			return $arr;
 		}
@@ -929,7 +929,7 @@ 
 	 */
 	function writeIniFile($filename, $arr)
 	{
-		if(!is_array($arr) || count($arr) == 0)
+		if (!is_array($arr) || count($arr) == 0)
 		{
 			return FALSE;
 		}
@@ -946,19 +946,19 @@ 
 	function _makeIniBuff($arr)
 	{
 		$return = array();
-		foreach($arr as $key => $val)
+		foreach ($arr as $key => $val)
 		{
 			// section
-			if(is_array($val))
+			if (is_array($val))
 			{
 				$return[] = sprintf("[%s]", $key);
-				foreach($val as $k => $v)
+				foreach ($val as $k => $v)
 				{
 					$return[] = sprintf("%s=\"%s\"", $k, $v);
 				}
 				// value
 			}
-			else if(is_object($val))
+			else if (is_object($val))
 			{
 				continue;
 			}
@@ -1033,15 +1033,15 @@ 
 	function isWritableDir($path)
 	{
 		$path = self::getRealPath($path);
-		if(is_dir($path)==FALSE)
+		if (is_dir($path) == FALSE)
 		{
 			return FALSE;
 		}
 
-		$checkFile = $path . '/_CheckWritableDir';
+		$checkFile = $path.'/_CheckWritableDir';
 
 		$fp = fopen($checkFile, 'w');
-		if(!is_resource($fp))
+		if (!is_resource($fp))
 		{
 			return FALSE;
 		}