xpressengine / xe-core

modules/widget/widget.controller.php

Doc Comments +3 added lines

@@ -696,6 +696,9 @@
 		return $GLOBALS['_xe_loaded_widgets_'][$widget];
 	}
 
+	/**
+	 * @param boolean $javascript_mode
+	 */
 	function compileWidgetStyle($widgetStyle,$widget,$widget_content_body, $args, $javascript_mode)
 	{
 		if(!$widgetStyle) return $widget_content_body;

Indentation +4 added lines, -4 removed lines

@@ -1,10 +1,10 @@
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 /**
- * @class  widgetController
- * @author NAVER (developers@xpressengine.com)
- * @brief Controller class for widget modules
- */
+	 * @class  widgetController
+	 * @author NAVER (developers@xpressengine.com)
+	 * @brief Controller class for widget modules
+	 */
 class widgetController extends widget
 {
 	// The results are not widget modify/delete and where to use the flag for

Spacing +143 added lines, -143 removed lines

@@ -34,13 +34,13 @@ 
 		$oModuleModel = getModel('module');
 		$skin_info = $oModuleModel->loadSkinInfo($path, $skin);
 
-		for($i=0;$i<count($skin_info->colorset);$i++)
+		for ($i = 0; $i < count($skin_info->colorset); $i++)
 		{
 			$colorset = sprintf('%s|@|%s', $skin_info->colorset[$i]->name, $skin_info->colorset[$i]->title);
 			$colorset_list[] = $colorset;
 		}
 
-		if(count($colorset_list)) $colorsets = implode("\n", $colorset_list);
+		if (count($colorset_list)) $colorsets = implode("\n", $colorset_list);
 		$this->add('colorset_list', $colorsets);
 	}
 
@@ -50,12 +50,12 @@ 
 	function procWidgetGenerateCode()
 	{
 		$widget = Context::get('selected_widget');
-		if(!$widget) return new Object(-1,'msg_invalid_request');
-		if(!Context::get('skin')) return new Object(-1,Context::getLang('msg_widget_skin_is_null'));
+		if (!$widget) return new Object(-1, 'msg_invalid_request');
+		if (!Context::get('skin')) return new Object(-1, Context::getLang('msg_widget_skin_is_null'));
 
 		$attribute = $this->arrangeWidgetVars($widget, Context::getRequestVars(), $vars);
 
-		$widget_code = sprintf('<img class="zbxe_widget_output" widget="%s" %s />', $widget, implode(' ',$attribute));
+		$widget_code = sprintf('<img class="zbxe_widget_output" widget="%s" %s />', $widget, implode(' ', $attribute));
 		// Code output
 		$this->add('widget_code', $widget_code);
 	}
@@ -66,9 +66,9 @@ 
 	function procWidgetGenerateCodeInPage()
 	{
 		$widget = Context::get('selected_widget');
-		if(!$widget) return new Object(-1,'msg_invalid_request');
+		if (!$widget) return new Object(-1, 'msg_invalid_request');
 
-		if(!in_array($widget,array('widgetBox','widgetContent')) && !Context::get('skin')) return new Object(-1,Context::getLang('msg_widget_skin_is_null'));
+		if (!in_array($widget, array('widgetBox', 'widgetContent')) && !Context::get('skin')) return new Object(-1, Context::getLang('msg_widget_skin_is_null'));
 
 		$attribute = $this->arrangeWidgetVars($widget, Context::getRequestVars(), $vars);
 		// Wanted results
@@ -89,7 +89,7 @@ 
 
 		$this->setLayoutPath('./common/tpl');
 		$this->setLayoutFile('default_layout.html');
-		$this->setTemplatePath($this->module_path.'tpl');
+		$this->setTemplatePath($this->module_path . 'tpl');
 		$this->setTemplateFile("top_refresh.html");
 	}
 
@@ -107,28 +107,28 @@ 
 		$err = 0;
 		$oLayoutModel = getModel('layout');
 		$layout_info = $oLayoutModel->getLayout($module_srl);
-		if(!$layout_info || $layout_info->type != 'faceoff') $err++;
+		if (!$layout_info || $layout_info->type != 'faceoff') $err++;
 		// Destination Information Wanted page module
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'module');
 		$page_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl, $columnList);
-		if(!$page_info->module_srl || $page_info->module != 'page') $err++;
+		if (!$page_info->module_srl || $page_info->module != 'page') $err++;
 
-		if($err > 1) return new Object(-1,'msg_invalid_request');
+		if ($err > 1) return new Object(-1, 'msg_invalid_request');
 		// Check permissions
 		$is_logged = Context::get('is_logged');
 		$logged_info = Context::get('logged_info');
 		$user_group = $logged_info->group_list;
 		$is_admin = false;
-		if(count($user_group)&&count($page_info->grants['manager']))
+		if (count($user_group) && count($page_info->grants['manager']))
 		{
 			$manager_group = $page_info->grants['manager'];
-			foreach($user_group as $group_srl => $group_info)
+			foreach ($user_group as $group_srl => $group_info)
 			{
-				if(in_array($group_srl, $manager_group)) $is_admin = true;
+				if (in_array($group_srl, $manager_group)) $is_admin = true;
 			}
 		}
-		if(!$is_admin && !$is_logged && $logged_info->is_admin != 'Y' && !$oModuleModel->isSiteAdmin($logged_info) && !(is_array($page_info->admin_id) && in_array($logged_info->user_id, $page_info->admin_id))) return new Object(-1,'msg_not_permitted');
+		if (!$is_admin && !$is_logged && $logged_info->is_admin != 'Y' && !$oModuleModel->isSiteAdmin($logged_info) && !(is_array($page_info->admin_id) && in_array($logged_info->user_id, $page_info->admin_id))) return new Object(-1, 'msg_not_permitted');
 		// Enter post
 		$oDocumentModel = getModel('document');
 		$oDocumentController = getController('document');
@@ -139,7 +139,7 @@ 
 		$obj->document_srl = $document_srl;
 
 		$oDocument = $oDocumentModel->getDocument($obj->document_srl, true);
-		if($oDocument->isExists() && $oDocument->document_srl == $obj->document_srl)
+		if ($oDocument->isExists() && $oDocument->document_srl == $obj->document_srl)
 		{
 			$output = $oDocumentController->updateDocument($oDocument, $obj);
 		}
@@ -149,7 +149,7 @@ 
 			$obj->document_srl = $output->get('document_srl');
 		}
 		// Stop when an error occurs
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 		// Return results
 		$this->add('document_srl', $obj->document_srl);
 	}
@@ -167,30 +167,30 @@ 
 		$oDocumentAdminController = getAdminController('document');
 
 		$oDocument = $oDocumentModel->getDocument($document_srl, true);
-		if(!$oDocument->isExists()) return new Object(-1,'msg_invalid_request');
+		if (!$oDocument->isExists()) return new Object(-1, 'msg_invalid_request');
 		$module_srl = $oDocument->get('module_srl');
 		// Destination Information Wanted page module
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'module');
 		$page_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl, $columnList);
-		if(!$page_info->module_srl || $page_info->module != 'page') return new Object(-1,'msg_invalid_request');
+		if (!$page_info->module_srl || $page_info->module != 'page') return new Object(-1, 'msg_invalid_request');
 		// Check permissions
 		$is_logged = Context::get('is_logged');
 		$logged_info = Context::get('logged_info');
 		$user_group = $logged_info->group_list;
 		$is_admin = false;
-		if(count($user_group)&&count($page_info->grants['manager']))
+		if (count($user_group) && count($page_info->grants['manager']))
 		{
 			$manager_group = $page_info->grants['manager'];
-			foreach($user_group as $group_srl => $group_info)
+			foreach ($user_group as $group_srl => $group_info)
 			{
-				if(in_array($group_srl, $manager_group)) $is_admin = true;
+				if (in_array($group_srl, $manager_group)) $is_admin = true;
 			}
 		}
-		if(!$is_admin && !$is_logged && $logged_info->is_admin != 'Y' && !$oModuleModel->isSiteAdmin($logged_info) && !(is_array($page_info->admin_id) && in_array($logged_info->user_id, $page_info->admin_id))) return new Object(-1,'msg_not_permitted');
+		if (!$is_admin && !$is_logged && $logged_info->is_admin != 'Y' && !$oModuleModel->isSiteAdmin($logged_info) && !(is_array($page_info->admin_id) && in_array($logged_info->user_id, $page_info->admin_id))) return new Object(-1, 'msg_not_permitted');
 
-		$output = $oDocumentAdminController->copyDocumentModule(array($oDocument->get('document_srl')), $oDocument->get('module_srl'),0);
-		if(!$output->toBool()) return $output;
+		$output = $oDocumentAdminController->copyDocumentModule(array($oDocument->get('document_srl')), $oDocument->get('module_srl'), 0);
+		if (!$output->toBool()) return $output;
 		// Return results
 		$copied_srls = $output->get('copied_srls');
 		$this->add('document_srl', $copied_srls[$oDocument->get('document_srl')]);
@@ -208,29 +208,29 @@ 
 		$oDocumentController = getController('document');
 
 		$oDocument = $oDocumentModel->getDocument($document_srl, true);
-		if(!$oDocument->isExists()) return new Object();
+		if (!$oDocument->isExists()) return new Object();
 		$module_srl = $oDocument->get('module_srl');
 		// Destination Information Wanted page module
 		$oModuleModel = getModel('module');
 		$page_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl);
-		if(!$page_info->module_srl || $page_info->module != 'page') return new Object(-1,'msg_invalid_request');
+		if (!$page_info->module_srl || $page_info->module != 'page') return new Object(-1, 'msg_invalid_request');
 		// Check permissions
 		$is_logged = Context::get('is_logged');
 		$logged_info = Context::get('logged_info');
 		$user_group = $logged_info->group_list;
 		$is_admin = false;
-		if(count($user_group)&&count($page_info->grants['manager']))
+		if (count($user_group) && count($page_info->grants['manager']))
 		{
 			$manager_group = $page_info->grants['manager'];
-			foreach($user_group as $group_srl => $group_info)
+			foreach ($user_group as $group_srl => $group_info)
 			{
-				if(in_array($group_srl, $manager_group)) $is_admin = true;
+				if (in_array($group_srl, $manager_group)) $is_admin = true;
 			}
 		}
-		if(!$is_admin && !$is_logged && $logged_info->is_admin != 'Y' && !$oModuleModel->isSiteAdmin($logged_info) && !(is_array($page_info->admin_id) && in_array($logged_info->user_id, $page_info->admin_id))) return new Object(-1,'msg_not_permitted');
+		if (!$is_admin && !$is_logged && $logged_info->is_admin != 'Y' && !$oModuleModel->isSiteAdmin($logged_info) && !(is_array($page_info->admin_id) && in_array($logged_info->user_id, $page_info->admin_id))) return new Object(-1, 'msg_not_permitted');
 
 		$output = $oDocumentController->deleteDocument($oDocument->get('document_srl'), true);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 	}
 
 	/**
@@ -247,7 +247,7 @@ 
 	 */
 	function triggerWidgetCompile(&$content)
 	{
-		if(Context::getResponseMethod()!='HTML') return new Object();
+		if (Context::getResponseMethod() != 'HTML') return new Object();
 		$content = $this->transWidgetCode($content, $this->layout_javascript_mode);
 		return new Object();
 	}
@@ -263,9 +263,9 @@ 
 		// Check whether to include information about editing
 		$this->javascript_mode = $javascript_mode;
 		// Widget code box change
-		$content = preg_replace_callback('!<div([^\>]*)widget=([^\>]*?)\><div><div>((<img.*?>)*)!is', array($this,'transWidgetBox'), $content);
+		$content = preg_replace_callback('!<div([^\>]*)widget=([^\>]*?)\><div><div>((<img.*?>)*)!is', array($this, 'transWidgetBox'), $content);
 		// Widget code information byeogyeong
-		$content = preg_replace_callback('!<img([^\>]*)widget=([^\>]*?)\>!is', array($this,'transWidget'), $content);
+		$content = preg_replace_callback('!<img([^\>]*)widget=([^\>]*?)\>!is', array($this, 'transWidget'), $content);
 
 		return $content;
 	}
@@ -280,11 +280,11 @@ 
 		$oXmlParser = new XmlParser();
 		$xml_doc = $oXmlParser->parse(trim($buff));
 
-		if($xml_doc->img) $vars = $xml_doc->img->attrs;
+		if ($xml_doc->img) $vars = $xml_doc->img->attrs;
 		else $vars = $xml_doc->attrs;
 
 		$widget = $vars->widget;
-		if(!$widget) return $matches[0];
+		if (!$widget) return $matches[0];
 		unset($vars->widget);
 
 		return $this->execute($widget, $vars, $this->javascript_mode);
@@ -295,13 +295,13 @@ 
 	 */
 	function transWidgetBox($matches)
 	{
-		$buff = preg_replace('/<div><div>(.*)$/i','</div>',$matches[0]);
+		$buff = preg_replace('/<div><div>(.*)$/i', '</div>', $matches[0]);
 		$oXmlParser = new XmlParser();
 		$xml_doc = $oXmlParser->parse($buff);
 
 		$vars = $xml_doc->div->attrs;
 		$widget = $vars->widget;
-		if(!$widget) return $matches[0];
+		if (!$widget) return $matches[0];
 		unset($vars->widget);
 
 		$vars->widgetbox_content = $matches[3];
@@ -322,28 +322,28 @@ 
 		$oXmlParser = new XmlParser();
 
 		$cnt = count($matches[1]);
-		for($i=0;$i<$cnt;$i++)
+		for ($i = 0; $i < $cnt; $i++)
 		{
 			$buff = $matches[0][$i];
 			$xml_doc = $oXmlParser->parse(trim($buff));
 
 			$args = $xml_doc->img->attrs;
-			if(!$args) continue;
+			if (!$args) continue;
 			// If you are not caching path
 			$widget = $args->widget;
 			$sequence = $args->widget_sequence;
 			$cache = $args->widget_cache;
-			if(!$sequence || !$cache) continue;
+			if (!$sequence || !$cache) continue;
 
-			if(count($args))
+			if (count($args))
 			{
-				foreach($args as $k => $v) $args->{$k} = urldecode($v);
+				foreach ($args as $k => $v) $args->{$k} = urldecode($v);
 			}
 			// If the cache file for each language widget regeneration
-			foreach($lang_list as $lang_type => $val)
+			foreach ($lang_list as $lang_type => $val)
 			{
 				$cache_file = sprintf('%s%d.%s.cache', $this->cache_path, $sequence, $lang_type);
-				if(!file_exists($cache_file)) continue;
+				if (!file_exists($cache_file)) continue;
 				$this->getCache($widget, $args, $lang_type, true);
 			}
 		}
@@ -355,7 +355,7 @@ 
 	function getCache($widget, $args, $lang_type = null, $ignore_cache = false)
 	{
 		// If the specified language specifies the current language
-		if(!$lang_type) $lang_type = Context::getLangType();
+		if (!$lang_type) $lang_type = Context::getLangType();
 		// widget, the cache number and cache values are set
 		$widget_sequence = $args->widget_sequence;
 		$widget_cache = $args->widget_cache;
@@ -363,10 +363,10 @@ 
 		/**
 		 * Even if the cache number and value of the cache and return it to extract data
 		 */
-		if(!$ignore_cache && (!$widget_cache || !$widget_sequence))
+		if (!$ignore_cache && (!$widget_cache || !$widget_sequence))
 		{
 			$oWidget = $this->getWidgetObject($widget);
-			if(!$oWidget || !method_exists($oWidget, 'proc')) return;
+			if (!$oWidget || !method_exists($oWidget, 'proc')) return;
 
 			$widget_content = $oWidget->proc($args);
 			$oModuleController = getController('module');
@@ -375,7 +375,7 @@ 
 		}
 
 		$oCacheHandler = CacheHandler::getInstance('template');
-		if($oCacheHandler->isSupport())
+		if ($oCacheHandler->isSupport())
 		{
 			$key = 'widget_cache:' . $widget_sequence;
 
@@ -383,7 +383,7 @@ 
 			$cache_body = preg_replace('@<\!--#Meta:@', '<!--Meta:', $cache_body);
 		}
 
-		if($cache_body)
+		if ($cache_body)
 		{
 			return $cache_body;
 		}
@@ -396,11 +396,11 @@ 
 			// Wanted cache file
 			$cache_file = sprintf('%s%d.%s.cache', $this->cache_path, $widget_sequence, $lang_type);
 			// If the file exists in the cache, the file validation
-			if(!$ignore_cache && file_exists($cache_file))
+			if (!$ignore_cache && file_exists($cache_file))
 			{
 				$filemtime = filemtime($cache_file);
 				// Should be modified compared to the time of the cache or in the future if creating more than widget.controller.php file a return value of the cache
-				if($filemtime + $widget_cache * 60 > $_SERVER['REQUEST_TIME'] && $filemtime > filemtime(_XE_PATH_.'modules/widget/widget.controller.php'))
+				if ($filemtime + $widget_cache * 60 > $_SERVER['REQUEST_TIME'] && $filemtime > filemtime(_XE_PATH_ . 'modules/widget/widget.controller.php'))
 				{
 					$cache_body = FileHandler::readFile($cache_file);
 					$cache_body = preg_replace('@<\!--#Meta:@', '<!--Meta:', $cache_body);
@@ -409,18 +409,18 @@ 
 				}
 			}
 			// cache update and cache renewal of the file mtime
-			if(!$oCacheHandler->isSupport())
+			if (!$oCacheHandler->isSupport())
 			{
 			touch($cache_file);
 			}
 
 			$oWidget = $this->getWidgetObject($widget);
-			if(!$oWidget || !method_exists($oWidget,'proc')) return;
+			if (!$oWidget || !method_exists($oWidget, 'proc')) return;
 
 			$widget_content = $oWidget->proc($args);
 			$oModuleController = getController('module');
 			$oModuleController->replaceDefinedLangCode($widget_content);
-			if($oCacheHandler->isSupport())
+			if ($oCacheHandler->isSupport())
 			{
 				$oCacheHandler->put($key, $widget_content, $widget_cache * 60);
 			}
@@ -442,16 +442,16 @@ 
 	function execute($widget, $args, $javascript_mode = false, $escaped = true)
 	{
 		// Save for debug run-time widget
-		if(__DEBUG__==3) $start = getMicroTime();
+		if (__DEBUG__ == 3) $start = getMicroTime();
 		$before = microtime(true);
 		// urldecode the value of args haejum
 		$object_vars = get_object_vars($args);
-		if(count($object_vars))
+		if (count($object_vars))
 		{
-			foreach($object_vars as $key => $val)
+			foreach ($object_vars as $key => $val)
 			{
-				if(in_array($key, array('widgetbox_content','body','class','style','widget_sequence','widget','widget_padding_left','widget_padding_top','widget_padding_bottom','widget_padding_right','widgetstyle','document_srl'))) continue;
-				if($escaped) $args->{$key} = utf8RawUrlDecode($val);
+				if (in_array($key, array('widgetbox_content', 'body', 'class', 'style', 'widget_sequence', 'widget', 'widget_padding_left', 'widget_padding_top', 'widget_padding_bottom', 'widget_padding_right', 'widgetstyle', 'document_srl'))) continue;
+				if ($escaped) $args->{$key} = utf8RawUrlDecode($val);
 			}
 		}
 
@@ -460,14 +460,14 @@ 
 		 * Widgets widgetContent/widgetBox Wanted If you are not content
 		 */
 		$widget_content = '';
-		if($widget != 'widgetContent' && $widget != 'widgetBox')
+		if ($widget != 'widgetContent' && $widget != 'widgetBox')
 		{
-			if(!is_dir(sprintf(_XE_PATH_.'widgets/%s/',$widget))) return;
+			if (!is_dir(sprintf(_XE_PATH_ . 'widgets/%s/', $widget))) return;
 			// Hold the contents of the widget parameter
 			$widget_content = $this->getCache($widget, $args);
 		}
 
-		if($widget == 'widgetBox')
+		if ($widget == 'widgetBox')
 		{
 			$widgetbox_content = $args->widgetbox_content;
 		}
@@ -476,7 +476,7 @@ 
 		 * Wanted specified by the administrator of the widget style
 		 */
 		// Sometimes the wrong code, background-image: url (none) can be heard but none in this case, the request for the url so unconditionally Removed
-		$style = preg_replace('/url\((.+)(\/?)none\)/is','', $args->style);
+		$style = preg_replace('/url\((.+)(\/?)none\)/is', '', $args->style);
 		// Find a style statement that based on the internal margin dropping pre-change
 		$widget_padding_left = $args->widget_padding_left;
 		$widget_padding_right = $args->widget_padding_right;
@@ -492,18 +492,18 @@ 
 		$widget_content_body = '';
 		$widget_content_footer = '';
 		// If general call is given on page styles should return immediately dreamin '
-		if(!$javascript_mode)
+		if (!$javascript_mode)
 		{
-			if($args->id) $args->id = ' id="'.$args->id.'" ';
-			switch($widget)
+			if ($args->id) $args->id = ' id="' . $args->id . '" ';
+			switch ($widget)
 			{
 				// If a direct orthogonal addition information
 				case 'widgetContent' :
-					if($args->document_srl)
+					if ($args->document_srl)
 					{
 						$oDocumentModel = getModel('document');
 						$oDocument = $oDocumentModel->getDocument($args->document_srl);
-						$body = $oDocument->getContent(false,false,false, false);
+						$body = $oDocument->getContent(false, false, false, false);
 					}
 					else
 					{
@@ -513,21 +513,21 @@ 
 					$oEditorController = getController('editor');
 					$body = $oEditorController->transComponent($body);
 
-					$widget_content_header = sprintf('<div class="xe_content xe-widget-wrapper ' . $args->css_class . '" %sstyle="%s"><div style="%s">', $args->id, $style,  $inner_style);
+					$widget_content_header = sprintf('<div class="xe_content xe-widget-wrapper ' . $args->css_class . '" %sstyle="%s"><div style="%s">', $args->id, $style, $inner_style);
 					$widget_content_body = $body;
 					$widget_content_footer = '</div></div>';
 
 					break;
 					// If the widget box; it could
 				case 'widgetBox' :
-					$widget_content_header = sprintf('<div class="xe-widget-wrapper ' . $args->css_class . '" %sstyle="%s;"><div style="%s"><div>', $args->id, $style,  $inner_style);
+					$widget_content_header = sprintf('<div class="xe-widget-wrapper ' . $args->css_class . '" %sstyle="%s;"><div style="%s"><div>', $args->id, $style, $inner_style);
 					$widget_content_body = $widgetbox_content;
 
 					break;
 					// If the General wijetil
 				default :
-					$widget_content_header = sprintf('<div class="xe-widget-wrapper ' . $args->css_class . '" %sstyle="%s">',$args->id,$style);
-					$widget_content_body = sprintf('<div style="*zoom:1;%s">%s</div>', $inner_style,$widget_content);
+					$widget_content_header = sprintf('<div class="xe-widget-wrapper ' . $args->css_class . '" %sstyle="%s">', $args->id, $style);
+					$widget_content_body = sprintf('<div style="*zoom:1;%s">%s</div>', $inner_style, $widget_content);
 					$widget_content_footer = '</div>';
 					break;
 			}
@@ -535,15 +535,15 @@ 
 		}
 		else
 		{
-			switch($widget)
+			switch ($widget)
 			{
 				// If a direct orthogonal addition information
 				case 'widgetContent' :
-					if($args->document_srl)
+					if ($args->document_srl)
 					{
 						$oDocumentModel = getModel('document');
 						$oDocument = $oDocumentModel->getDocument($args->document_srl);
-						$body = $oDocument->getContent(false,false,false);
+						$body = $oDocument->getContent(false, false, false);
 					}
 					else
 					{
@@ -551,12 +551,12 @@ 
 					}
 					// by args
 					$attribute = array();
-					if($args)
+					if ($args)
 					{
-						foreach($args as $key => $val)
+						foreach ($args as $key => $val)
 						{
-							if(in_array($key, array('class','style','widget_padding_top','widget_padding_right','widget_padding_bottom','widget_padding_left','widget','widgetstyle','document_srl'))) continue;
-							if(strpos($val,'|@|')>0) $val = str_replace('|@|',',',$val);
+							if (in_array($key, array('class', 'style', 'widget_padding_top', 'widget_padding_right', 'widget_padding_bottom', 'widget_padding_left', 'widget', 'widgetstyle', 'document_srl'))) continue;
+							if (strpos($val, '|@|') > 0) $val = str_replace('|@|', ',', $val);
 							$attribute[] = sprintf('%s="%s"', $key, htmlspecialchars($val, ENT_COMPAT | ENT_HTML401, 'UTF-8', false));
 						}
 					}
@@ -564,44 +564,44 @@ 
 					$oWidgetController = getController('widget');
 
 					$widget_content_header = sprintf(
-						'<div class="xe_content widgetOutput ' . $args->css_class . '" widgetstyle="%s" style="%s" widget_padding_left="%s" widget_padding_right="%s" widget_padding_top="%s" widget_padding_bottom="%s" widget="widgetContent" document_srl="%d" %s>'.
-						'<div class="widgetResize"></div>'.
-						'<div class="widgetResizeLeft"></div>'.
-						'<div class="widgetBorder">'.
-						'<div style="%s">',$args->widgetstyle,
+						'<div class="xe_content widgetOutput ' . $args->css_class . '" widgetstyle="%s" style="%s" widget_padding_left="%s" widget_padding_right="%s" widget_padding_top="%s" widget_padding_bottom="%s" widget="widgetContent" document_srl="%d" %s>' .
+						'<div class="widgetResize"></div>' .
+						'<div class="widgetResizeLeft"></div>' .
+						'<div class="widgetBorder">' .
+						'<div style="%s">', $args->widgetstyle,
 						$style,
 						$args->widget_padding_left, $args->widget_padding_right, $args->widget_padding_top, $args->widget_padding_bottom,
 						$args->document_srl,
-						implode(' ',$attribute),
+						implode(' ', $attribute),
 						$inner_style);
 
 					$widget_content_body = $body;
-					$widget_content_footer = sprintf('</div>'.
-						'</div>'.
-						'<div class="widgetContent" style="display:none;width:1px;height:1px;overflow:hidden;">%s</div>'.
-						'</div>',base64_encode($body));
+					$widget_content_footer = sprintf('</div>' .
+						'</div>' .
+						'<div class="widgetContent" style="display:none;width:1px;height:1px;overflow:hidden;">%s</div>' .
+						'</div>', base64_encode($body));
 
 					break;
 					// If the widget box; it could
 				case 'widgetBox' :
 					// by args
 					$attribute = array();
-					if($args)
+					if ($args)
 					{
-						foreach($args as $key => $val)
+						foreach ($args as $key => $val)
 						{
-							if(in_array($key, array('class','style','widget_padding_top','widget_padding_right','widget_padding_bottom','widget_padding_left','widget','widgetstyle','document_srl'))) continue;
-							if(!is_numeric($val) && (!is_string($val) || strlen($val)==0)) continue;
-							if(strpos($val,'|@|')>0) $val = str_replace('|@|',',',$val);
+							if (in_array($key, array('class', 'style', 'widget_padding_top', 'widget_padding_right', 'widget_padding_bottom', 'widget_padding_left', 'widget', 'widgetstyle', 'document_srl'))) continue;
+							if (!is_numeric($val) && (!is_string($val) || strlen($val) == 0)) continue;
+							if (strpos($val, '|@|') > 0) $val = str_replace('|@|', ',', $val);
 							$attribute[] = sprintf('%s="%s"', $key, htmlspecialchars($val, ENT_COMPAT | ENT_HTML401, 'UTF-8', false));
 						}
 					}
 
 					$widget_content_header = sprintf(
-						'<div class="widgetOutput ' . $args->css_class . '" widgetstyle="%s" widget="widgetBox" style="%s;" widget_padding_top="%s" widget_padding_right="%s" widget_padding_bottom="%s" widget_padding_left="%s" %s >'.
-						'<div class="widgetBoxResize"></div>'.
-						'<div class="widgetBoxResizeLeft"></div>'.
-						'<div class="widgetBoxBorder"><div class="nullWidget" style="%s">',$args->widgetstyle,$style, $widget_padding_top, $widget_padding_right, $widget_padding_bottom, $widget_padding_left,implode(' ',$attribute),$inner_style);
+						'<div class="widgetOutput ' . $args->css_class . '" widgetstyle="%s" widget="widgetBox" style="%s;" widget_padding_top="%s" widget_padding_right="%s" widget_padding_bottom="%s" widget_padding_left="%s" %s >' .
+						'<div class="widgetBoxResize"></div>' .
+						'<div class="widgetBoxResizeLeft"></div>' .
+						'<div class="widgetBoxBorder"><div class="nullWidget" style="%s">', $args->widgetstyle, $style, $widget_padding_top, $widget_padding_right, $widget_padding_bottom, $widget_padding_left, implode(' ', $attribute), $inner_style);
 
 					$widget_content_body = $widgetbox_content;
 
@@ -610,26 +610,26 @@ 
 				default :
 					// by args
 					$attribute = array();
-					if($args)
+					if ($args)
 					{
-						$allowed_key = array('class','style','widget_padding_top','widget_padding_right','widget_padding_bottom','widget_padding_left','widget');
-						foreach($args as $key => $val)
+						$allowed_key = array('class', 'style', 'widget_padding_top', 'widget_padding_right', 'widget_padding_bottom', 'widget_padding_left', 'widget');
+						foreach ($args as $key => $val)
 						{
-							if(in_array($key, $allowed_key)) continue;
-							if(!is_numeric($val) && (!is_string($val) || strlen($val)==0)) continue;
-							if(strpos($val,'|@|')>0) $val = str_replace('|@|',',',$val);
+							if (in_array($key, $allowed_key)) continue;
+							if (!is_numeric($val) && (!is_string($val) || strlen($val) == 0)) continue;
+							if (strpos($val, '|@|') > 0) $val = str_replace('|@|', ',', $val);
 							$attribute[] = sprintf('%s="%s"', $key, htmlspecialchars($val, ENT_COMPAT | ENT_HTML401, 'UTF-8', false));
 						}
 					}
 
-					$widget_content_header = sprintf('<div class="widgetOutput ' . $args->css_class . '" widgetstyle="%s" style="%s" widget_padding_top="%s" widget_padding_right="%s" widget_padding_bottom="%s" widget_padding_left="%s" widget="%s" %s >'.
-						'<div class="widgetResize"></div>'.
-						'<div class="widgetResizeLeft"></div>'.
-						'<div class="widgetBorder">',$args->widgetstyle,$style,
+					$widget_content_header = sprintf('<div class="widgetOutput ' . $args->css_class . '" widgetstyle="%s" style="%s" widget_padding_top="%s" widget_padding_right="%s" widget_padding_bottom="%s" widget_padding_left="%s" widget="%s" %s >' .
+						'<div class="widgetResize"></div>' .
+						'<div class="widgetResizeLeft"></div>' .
+						'<div class="widgetBorder">', $args->widgetstyle, $style,
 						$widget_padding_top, $widget_padding_right, $widget_padding_bottom, $widget_padding_left,
-						$widget, implode(' ',$attribute));
+						$widget, implode(' ', $attribute));
 
-					$widget_content_body = sprintf('<div style="%s">%s</div>',$inner_style, $widget_content);
+					$widget_content_body = sprintf('<div style="%s">%s</div>', $inner_style, $widget_content);
 
 					$widget_content_footer = '</div></div>';
 
@@ -637,11 +637,11 @@ 
 			}
 		}
 		// Compile the widget style.
-		if($args->widgetstyle) $widget_content_body = $this->compileWidgetStyle($args->widgetstyle,$widget, $widget_content_body, $args, $javascript_mode);
+		if ($args->widgetstyle) $widget_content_body = $this->compileWidgetStyle($args->widgetstyle, $widget, $widget_content_body, $args, $javascript_mode);
 
 		$output = $widget_content_header . $widget_content_body . $widget_content_footer;
 		// Debug widget creation time information added to the results
-		if(__DEBUG__==3) $GLOBALS['__widget_excute_elapsed__'] += getMicroTime() - $start;
+		if (__DEBUG__ == 3) $GLOBALS['__widget_excute_elapsed__'] += getMicroTime() - $start;
 
 		$after = microtime(true);
 
@@ -662,32 +662,32 @@ 
 	 */
 	function getWidgetObject($widget)
 	{
-		if(!preg_match('/^[a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*$/', $widget))
+		if (!preg_match('/^[a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*$/', $widget))
 		{
 			return Context::getLang('msg_invalid_request');
 		}
 
-		if(!$GLOBALS['_xe_loaded_widgets_'][$widget])
+		if (!$GLOBALS['_xe_loaded_widgets_'][$widget])
 		{
 			// Finding the location of a widget
 			$oWidgetModel = getModel('widget');
 			$path = $oWidgetModel->getWidgetPath($widget);
 			// If you do not find the class file error output widget (html output)
 			$class_file = sprintf('%s%s.class.php', $path, $widget);
-			if(!file_exists($class_file)) return sprintf(Context::getLang('msg_widget_is_not_exists'), $widget);
+			if (!file_exists($class_file)) return sprintf(Context::getLang('msg_widget_is_not_exists'), $widget);
 			// Widget classes include
 			require_once($class_file);
 
 			// Creating Objects
-			if(!class_exists($widget, false))
+			if (!class_exists($widget, false))
 			{
 				return sprintf(Context::getLang('msg_widget_object_is_null'), $widget);
 			}
 
 			$oWidget = new $widget();
-			if(!is_object($oWidget)) return sprintf(Context::getLang('msg_widget_object_is_null'), $widget);
+			if (!is_object($oWidget)) return sprintf(Context::getLang('msg_widget_object_is_null'), $widget);
 
-			if(!method_exists($oWidget, 'proc')) return sprintf(Context::getLang('msg_widget_proc_is_null'), $widget);
+			if (!method_exists($oWidget, 'proc')) return sprintf(Context::getLang('msg_widget_proc_is_null'), $widget);
 
 			$oWidget->widget_path = $path;
 
@@ -696,31 +696,31 @@ 
 		return $GLOBALS['_xe_loaded_widgets_'][$widget];
 	}
 
-	function compileWidgetStyle($widgetStyle,$widget,$widget_content_body, $args, $javascript_mode)
+	function compileWidgetStyle($widgetStyle, $widget, $widget_content_body, $args, $javascript_mode)
 	{
-		if(!$widgetStyle) return $widget_content_body;
+		if (!$widgetStyle) return $widget_content_body;
 
 		$oWidgetModel = getModel('widget');
 		// Bring extra_var widget style tie
 		$widgetstyle_info = $oWidgetModel->getWidgetStyleInfo($widgetStyle);
-		if(!$widgetstyle_info) return $widget_content_body;
+		if (!$widgetstyle_info) return $widget_content_body;
 
 		$widgetstyle_extra_var = new stdClass();
 		$widgetstyle_extra_var_key = get_object_vars($widgetstyle_info);
-		if(count($widgetstyle_extra_var_key['extra_var']))
+		if (count($widgetstyle_extra_var_key['extra_var']))
 		{
-			foreach($widgetstyle_extra_var_key['extra_var'] as $key => $val)
+			foreach ($widgetstyle_extra_var_key['extra_var'] as $key => $val)
 			{
-				$widgetstyle_extra_var->{$key} =  $args->{$key};
+				$widgetstyle_extra_var->{$key} = $args->{$key};
 			}
 		}
 		Context::set('widgetstyle_extra_var', $widgetstyle_extra_var);
 		// #18994272 오타를 수정했으나 하위 호환성을 위해 남겨둠 - deprecated
 		Context::set('widgetstyle_extar_var', $widgetstyle_extra_var);
 
-		if($javascript_mode && $widget=='widgetBox')
+		if ($javascript_mode && $widget == 'widgetBox')
 		{
-			Context::set('widget_content', '<div class="widget_inner">'.$widget_content_body.'</div>');
+			Context::set('widget_content', '<div class="widget_inner">' . $widget_content_body . '</div>');
 		}
 		else
 		{
@@ -742,7 +742,7 @@ 
 		$oWidgetModel = getModel('widget');
 		$widget_info = $oWidgetModel->getWidgetInfo($widget);
 
-		if(!$vars)
+		if (!$vars)
 		{
 			$vars = new stdClass();
 		}
@@ -753,31 +753,31 @@ 
 
 		$vars->skin = trim($request_vars->skin);
 		$vars->colorset = trim($request_vars->colorset);
-		$vars->widget_sequence = (int)($request_vars->widget_sequence);
-		$vars->widget_cache = (int)($request_vars->widget_cache);
+		$vars->widget_sequence = (int) ($request_vars->widget_sequence);
+		$vars->widget_cache = (int) ($request_vars->widget_cache);
 		$vars->style = trim($request_vars->style);
 		$vars->widget_padding_left = trim($request_vars->widget_padding_left);
 		$vars->widget_padding_right = trim($request_vars->widget_padding_right);
 		$vars->widget_padding_top = trim($request_vars->widget_padding_top);
 		$vars->widget_padding_bottom = trim($request_vars->widget_padding_bottom);
-		$vars->document_srl= trim($request_vars->document_srl);
+		$vars->document_srl = trim($request_vars->document_srl);
 
-		if(count($widget_info->extra_var))
+		if (count($widget_info->extra_var))
 		{
-			foreach($widget_info->extra_var as $key=>$val)
+			foreach ($widget_info->extra_var as $key=>$val)
 			{
 				$vars->{$key} = trim($request_vars->{$key});
 			}
 		}
 		// If the widget style
-		if($request_vars->widgetstyle)
+		if ($request_vars->widgetstyle)
 		{
 			$widgetStyle_info = $oWidgetModel->getWidgetStyleInfo($request_vars->widgetstyle);
-			if(count($widgetStyle_info->extra_var))
+			if (count($widgetStyle_info->extra_var))
 			{
-				foreach($widgetStyle_info->extra_var as $key=>$val)
+				foreach ($widgetStyle_info->extra_var as $key=>$val)
 				{
-					if($val->type =='color' || $val->type =='text' || $val->type =='select' || $val->type =='filebox' || $val->type == 'textarea')
+					if ($val->type == 'color' || $val->type == 'text' || $val->type == 'select' || $val->type == 'filebox' || $val->type == 'textarea')
 					{
 						$vars->{$key} = trim($request_vars->{$key});
 					}
@@ -785,23 +785,23 @@ 
 			}
 		}
 
-		if($vars->widget_sequence)
+		if ($vars->widget_sequence)
 		{
 			$cache_file = sprintf('%s%d.%s.cache', $this->cache_path, $vars->widget_sequence, Context::getLangType());
 			FileHandler::removeFile($cache_file);
 		}
 
-		if($vars->widget_cache>0) $vars->widget_sequence = getNextSequence();
+		if ($vars->widget_cache > 0) $vars->widget_sequence = getNextSequence();
 
 		$attribute = array();
-		foreach($vars as $key => $val)
+		foreach ($vars as $key => $val)
 		{
-			if(!$val)
+			if (!$val)
 			{
 				unset($vars->{$key});
 				continue;
 			}
-			if(strpos($val,'|@|') > 0) $val = str_replace('|@|', ',', $val);
+			if (strpos($val, '|@|') > 0) $val = str_replace('|@|', ',', $val);
 			$vars->{$key} = Context::convertEncodingStr($val);
 			$attribute[] = sprintf('%s="%s"', $key, htmlspecialchars(Context::convertEncodingStr($val), ENT_COMPAT | ENT_HTML401, 'UTF-8', false));
 		}

Braces +171 added lines, -69 removed lines

@@ -40,7 +40,9 @@ 
 			$colorset_list[] = $colorset;
 		}
 
-		if(count($colorset_list)) $colorsets = implode("\n", $colorset_list);
+		if(count($colorset_list)) {
+			$colorsets = implode("\n", $colorset_list);
+		}
 		$this->add('colorset_list', $colorsets);
 	}
 
@@ -50,8 +52,12 @@ 
 	function procWidgetGenerateCode()
 	{
 		$widget = Context::get('selected_widget');
-		if(!$widget) return new Object(-1,'msg_invalid_request');
-		if(!Context::get('skin')) return new Object(-1,Context::getLang('msg_widget_skin_is_null'));
+		if(!$widget) {
+			return new Object(-1,'msg_invalid_request');
+		}
+		if(!Context::get('skin')) {
+			return new Object(-1,Context::getLang('msg_widget_skin_is_null'));
+		}
 
 		$attribute = $this->arrangeWidgetVars($widget, Context::getRequestVars(), $vars);
 
@@ -66,9 +72,13 @@ 
 	function procWidgetGenerateCodeInPage()
 	{
 		$widget = Context::get('selected_widget');
-		if(!$widget) return new Object(-1,'msg_invalid_request');
+		if(!$widget) {
+			return new Object(-1,'msg_invalid_request');
+		}
 
-		if(!in_array($widget,array('widgetBox','widgetContent')) && !Context::get('skin')) return new Object(-1,Context::getLang('msg_widget_skin_is_null'));
+		if(!in_array($widget,array('widgetBox','widgetContent')) && !Context::get('skin')) {
+			return new Object(-1,Context::getLang('msg_widget_skin_is_null'));
+		}
 
 		$attribute = $this->arrangeWidgetVars($widget, Context::getRequestVars(), $vars);
 		// Wanted results
@@ -107,14 +117,20 @@ 
 		$err = 0;
 		$oLayoutModel = getModel('layout');
 		$layout_info = $oLayoutModel->getLayout($module_srl);
-		if(!$layout_info || $layout_info->type != 'faceoff') $err++;
+		if(!$layout_info || $layout_info->type != 'faceoff') {
+			$err++;
+		}
 		// Destination Information Wanted page module
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'module');
 		$page_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl, $columnList);
-		if(!$page_info->module_srl || $page_info->module != 'page') $err++;
+		if(!$page_info->module_srl || $page_info->module != 'page') {
+			$err++;
+		}
 
-		if($err > 1) return new Object(-1,'msg_invalid_request');
+		if($err > 1) {
+			return new Object(-1,'msg_invalid_request');
+		}
 		// Check permissions
 		$is_logged = Context::get('is_logged');
 		$logged_info = Context::get('logged_info');
@@ -125,10 +141,14 @@ 
 			$manager_group = $page_info->grants['manager'];
 			foreach($user_group as $group_srl => $group_info)
 			{
-				if(in_array($group_srl, $manager_group)) $is_admin = true;
+				if(in_array($group_srl, $manager_group)) {
+					$is_admin = true;
+				}
 			}
 		}
-		if(!$is_admin && !$is_logged && $logged_info->is_admin != 'Y' && !$oModuleModel->isSiteAdmin($logged_info) && !(is_array($page_info->admin_id) && in_array($logged_info->user_id, $page_info->admin_id))) return new Object(-1,'msg_not_permitted');
+		if(!$is_admin && !$is_logged && $logged_info->is_admin != 'Y' && !$oModuleModel->isSiteAdmin($logged_info) && !(is_array($page_info->admin_id) && in_array($logged_info->user_id, $page_info->admin_id))) {
+			return new Object(-1,'msg_not_permitted');
+		}
 		// Enter post
 		$oDocumentModel = getModel('document');
 		$oDocumentController = getController('document');
@@ -142,14 +162,15 @@ 
 		if($oDocument->isExists() && $oDocument->document_srl == $obj->document_srl)
 		{
 			$output = $oDocumentController->updateDocument($oDocument, $obj);
-		}
-		else
+		} else
 		{
 			$output = $oDocumentController->insertDocument($obj);
 			$obj->document_srl = $output->get('document_srl');
 		}
 		// Stop when an error occurs
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 		// Return results
 		$this->add('document_srl', $obj->document_srl);
 	}
@@ -167,13 +188,17 @@ 
 		$oDocumentAdminController = getAdminController('document');
 
 		$oDocument = $oDocumentModel->getDocument($document_srl, true);
-		if(!$oDocument->isExists()) return new Object(-1,'msg_invalid_request');
+		if(!$oDocument->isExists()) {
+			return new Object(-1,'msg_invalid_request');
+		}
 		$module_srl = $oDocument->get('module_srl');
 		// Destination Information Wanted page module
 		$oModuleModel = getModel('module');
 		$columnList = array('module_srl', 'module');
 		$page_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl, $columnList);
-		if(!$page_info->module_srl || $page_info->module != 'page') return new Object(-1,'msg_invalid_request');
+		if(!$page_info->module_srl || $page_info->module != 'page') {
+			return new Object(-1,'msg_invalid_request');
+		}
 		// Check permissions
 		$is_logged = Context::get('is_logged');
 		$logged_info = Context::get('logged_info');
@@ -184,13 +209,19 @@ 
 			$manager_group = $page_info->grants['manager'];
 			foreach($user_group as $group_srl => $group_info)
 			{
-				if(in_array($group_srl, $manager_group)) $is_admin = true;
+				if(in_array($group_srl, $manager_group)) {
+					$is_admin = true;
+				}
 			}
 		}
-		if(!$is_admin && !$is_logged && $logged_info->is_admin != 'Y' && !$oModuleModel->isSiteAdmin($logged_info) && !(is_array($page_info->admin_id) && in_array($logged_info->user_id, $page_info->admin_id))) return new Object(-1,'msg_not_permitted');
+		if(!$is_admin && !$is_logged && $logged_info->is_admin != 'Y' && !$oModuleModel->isSiteAdmin($logged_info) && !(is_array($page_info->admin_id) && in_array($logged_info->user_id, $page_info->admin_id))) {
+			return new Object(-1,'msg_not_permitted');
+		}
 
 		$output = $oDocumentAdminController->copyDocumentModule(array($oDocument->get('document_srl')), $oDocument->get('module_srl'),0);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 		// Return results
 		$copied_srls = $output->get('copied_srls');
 		$this->add('document_srl', $copied_srls[$oDocument->get('document_srl')]);
@@ -208,12 +239,16 @@ 
 		$oDocumentController = getController('document');
 
 		$oDocument = $oDocumentModel->getDocument($document_srl, true);
-		if(!$oDocument->isExists()) return new Object();
+		if(!$oDocument->isExists()) {
+			return new Object();
+		}
 		$module_srl = $oDocument->get('module_srl');
 		// Destination Information Wanted page module
 		$oModuleModel = getModel('module');
 		$page_info = $oModuleModel->getModuleInfoByModuleSrl($module_srl);
-		if(!$page_info->module_srl || $page_info->module != 'page') return new Object(-1,'msg_invalid_request');
+		if(!$page_info->module_srl || $page_info->module != 'page') {
+			return new Object(-1,'msg_invalid_request');
+		}
 		// Check permissions
 		$is_logged = Context::get('is_logged');
 		$logged_info = Context::get('logged_info');
@@ -224,13 +259,19 @@ 
 			$manager_group = $page_info->grants['manager'];
 			foreach($user_group as $group_srl => $group_info)
 			{
-				if(in_array($group_srl, $manager_group)) $is_admin = true;
+				if(in_array($group_srl, $manager_group)) {
+					$is_admin = true;
+				}
 			}
 		}
-		if(!$is_admin && !$is_logged && $logged_info->is_admin != 'Y' && !$oModuleModel->isSiteAdmin($logged_info) && !(is_array($page_info->admin_id) && in_array($logged_info->user_id, $page_info->admin_id))) return new Object(-1,'msg_not_permitted');
+		if(!$is_admin && !$is_logged && $logged_info->is_admin != 'Y' && !$oModuleModel->isSiteAdmin($logged_info) && !(is_array($page_info->admin_id) && in_array($logged_info->user_id, $page_info->admin_id))) {
+			return new Object(-1,'msg_not_permitted');
+		}
 
 		$output = $oDocumentController->deleteDocument($oDocument->get('document_srl'), true);
-		if(!$output->toBool()) return $output;
+		if(!$output->toBool()) {
+			return $output;
+		}
 	}
 
 	/**
@@ -247,7 +288,9 @@ 
 	 */
 	function triggerWidgetCompile(&$content)
 	{
-		if(Context::getResponseMethod()!='HTML') return new Object();
+		if(Context::getResponseMethod()!='HTML') {
+			return new Object();
+		}
 		$content = $this->transWidgetCode($content, $this->layout_javascript_mode);
 		return new Object();
 	}
@@ -280,11 +323,16 @@ 
 		$oXmlParser = new XmlParser();
 		$xml_doc = $oXmlParser->parse(trim($buff));
 
-		if($xml_doc->img) $vars = $xml_doc->img->attrs;
-		else $vars = $xml_doc->attrs;
+		if($xml_doc->img) {
+			$vars = $xml_doc->img->attrs;
+		} else {
+			$vars = $xml_doc->attrs;
+		}
 
 		$widget = $vars->widget;
-		if(!$widget) return $matches[0];
+		if(!$widget) {
+			return $matches[0];
+		}
 		unset($vars->widget);
 
 		return $this->execute($widget, $vars, $this->javascript_mode);
@@ -301,7 +349,9 @@ 
 
 		$vars = $xml_doc->div->attrs;
 		$widget = $vars->widget;
-		if(!$widget) return $matches[0];
+		if(!$widget) {
+			return $matches[0];
+		}
 		unset($vars->widget);
 
 		$vars->widgetbox_content = $matches[3];
@@ -328,22 +378,30 @@ 
 			$xml_doc = $oXmlParser->parse(trim($buff));
 
 			$args = $xml_doc->img->attrs;
-			if(!$args) continue;
+			if(!$args) {
+				continue;
+			}
 			// If you are not caching path
 			$widget = $args->widget;
 			$sequence = $args->widget_sequence;
 			$cache = $args->widget_cache;
-			if(!$sequence || !$cache) continue;
+			if(!$sequence || !$cache) {
+				continue;
+			}
 
 			if(count($args))
 			{
-				foreach($args as $k => $v) $args->{$k} = urldecode($v);
+				foreach($args as $k => $v) {
+					$args->{$k} = urldecode($v);
+				}
 			}
 			// If the cache file for each language widget regeneration
 			foreach($lang_list as $lang_type => $val)
 			{
 				$cache_file = sprintf('%s%d.%s.cache', $this->cache_path, $sequence, $lang_type);
-				if(!file_exists($cache_file)) continue;
+				if(!file_exists($cache_file)) {
+					continue;
+				}
 				$this->getCache($widget, $args, $lang_type, true);
 			}
 		}
@@ -355,7 +413,9 @@ 
 	function getCache($widget, $args, $lang_type = null, $ignore_cache = false)
 	{
 		// If the specified language specifies the current language
-		if(!$lang_type) $lang_type = Context::getLangType();
+		if(!$lang_type) {
+			$lang_type = Context::getLangType();
+		}
 		// widget, the cache number and cache values are set
 		$widget_sequence = $args->widget_sequence;
 		$widget_cache = $args->widget_cache;
@@ -366,7 +426,9 @@ 
 		if(!$ignore_cache && (!$widget_cache || !$widget_sequence))
 		{
 			$oWidget = $this->getWidgetObject($widget);
-			if(!$oWidget || !method_exists($oWidget, 'proc')) return;
+			if(!$oWidget || !method_exists($oWidget, 'proc')) {
+				return;
+			}
 
 			$widget_content = $oWidget->proc($args);
 			$oModuleController = getController('module');
@@ -386,8 +448,7 @@ 
 		if($cache_body)
 		{
 			return $cache_body;
-		}
-		else
+		} else
 		{
 			/**
 			 * Cache number and cache values are set so that the cache file should call
@@ -415,7 +476,9 @@ 
 			}
 
 			$oWidget = $this->getWidgetObject($widget);
-			if(!$oWidget || !method_exists($oWidget,'proc')) return;
+			if(!$oWidget || !method_exists($oWidget,'proc')) {
+				return;
+			}
 
 			$widget_content = $oWidget->proc($args);
 			$oModuleController = getController('module');
@@ -423,8 +486,7 @@ 
 			if($oCacheHandler->isSupport())
 			{
 				$oCacheHandler->put($key, $widget_content, $widget_cache * 60);
-			}
-			else
+			} else
 			{
 				FileHandler::writeFile($cache_file, $widget_content);
 			}
@@ -442,7 +504,9 @@ 
 	function execute($widget, $args, $javascript_mode = false, $escaped = true)
 	{
 		// Save for debug run-time widget
-		if(__DEBUG__==3) $start = getMicroTime();
+		if(__DEBUG__==3) {
+			$start = getMicroTime();
+		}
 		$before = microtime(true);
 		// urldecode the value of args haejum
 		$object_vars = get_object_vars($args);
@@ -450,8 +514,12 @@ 
 		{
 			foreach($object_vars as $key => $val)
 			{
-				if(in_array($key, array('widgetbox_content','body','class','style','widget_sequence','widget','widget_padding_left','widget_padding_top','widget_padding_bottom','widget_padding_right','widgetstyle','document_srl'))) continue;
-				if($escaped) $args->{$key} = utf8RawUrlDecode($val);
+				if(in_array($key, array('widgetbox_content','body','class','style','widget_sequence','widget','widget_padding_left','widget_padding_top','widget_padding_bottom','widget_padding_right','widgetstyle','document_srl'))) {
+					continue;
+				}
+				if($escaped) {
+					$args->{$key} = utf8RawUrlDecode($val);
+				}
 			}
 		}
 
@@ -462,7 +530,9 @@ 
 		$widget_content = '';
 		if($widget != 'widgetContent' && $widget != 'widgetBox')
 		{
-			if(!is_dir(sprintf(_XE_PATH_.'widgets/%s/',$widget))) return;
+			if(!is_dir(sprintf(_XE_PATH_.'widgets/%s/',$widget))) {
+				return;
+			}
 			// Hold the contents of the widget parameter
 			$widget_content = $this->getCache($widget, $args);
 		}
@@ -494,7 +564,9 @@ 
 		// If general call is given on page styles should return immediately dreamin '
 		if(!$javascript_mode)
 		{
-			if($args->id) $args->id = ' id="'.$args->id.'" ';
+			if($args->id) {
+				$args->id = ' id="'.$args->id.'" ';
+			}
 			switch($widget)
 			{
 				// If a direct orthogonal addition information
@@ -504,8 +576,7 @@ 
 						$oDocumentModel = getModel('document');
 						$oDocument = $oDocumentModel->getDocument($args->document_srl);
 						$body = $oDocument->getContent(false,false,false, false);
-					}
-					else
+					} else
 					{
 						$body = base64_decode($args->body);
 					}
@@ -532,8 +603,7 @@ 
 					break;
 			}
 			// Edit page is called when a widget if you add the code for handling
-		}
-		else
+		} else
 		{
 			switch($widget)
 			{
@@ -544,8 +614,7 @@ 
 						$oDocumentModel = getModel('document');
 						$oDocument = $oDocumentModel->getDocument($args->document_srl);
 						$body = $oDocument->getContent(false,false,false);
-					}
-					else
+					} else
 					{
 						$body = base64_decode($args->body);
 					}
@@ -555,8 +624,12 @@ 
 					{
 						foreach($args as $key => $val)
 						{
-							if(in_array($key, array('class','style','widget_padding_top','widget_padding_right','widget_padding_bottom','widget_padding_left','widget','widgetstyle','document_srl'))) continue;
-							if(strpos($val,'|@|')>0) $val = str_replace('|@|',',',$val);
+							if(in_array($key, array('class','style','widget_padding_top','widget_padding_right','widget_padding_bottom','widget_padding_left','widget','widgetstyle','document_srl'))) {
+								continue;
+							}
+							if(strpos($val,'|@|')>0) {
+								$val = str_replace('|@|',',',$val);
+							}
 							$attribute[] = sprintf('%s="%s"', $key, htmlspecialchars($val, ENT_COMPAT | ENT_HTML401, 'UTF-8', false));
 						}
 					}
@@ -590,9 +663,15 @@ 
 					{
 						foreach($args as $key => $val)
 						{
-							if(in_array($key, array('class','style','widget_padding_top','widget_padding_right','widget_padding_bottom','widget_padding_left','widget','widgetstyle','document_srl'))) continue;
-							if(!is_numeric($val) && (!is_string($val) || strlen($val)==0)) continue;
-							if(strpos($val,'|@|')>0) $val = str_replace('|@|',',',$val);
+							if(in_array($key, array('class','style','widget_padding_top','widget_padding_right','widget_padding_bottom','widget_padding_left','widget','widgetstyle','document_srl'))) {
+								continue;
+							}
+							if(!is_numeric($val) && (!is_string($val) || strlen($val)==0)) {
+								continue;
+							}
+							if(strpos($val,'|@|')>0) {
+								$val = str_replace('|@|',',',$val);
+							}
 							$attribute[] = sprintf('%s="%s"', $key, htmlspecialchars($val, ENT_COMPAT | ENT_HTML401, 'UTF-8', false));
 						}
 					}
@@ -615,9 +694,15 @@ 
 						$allowed_key = array('class','style','widget_padding_top','widget_padding_right','widget_padding_bottom','widget_padding_left','widget');
 						foreach($args as $key => $val)
 						{
-							if(in_array($key, $allowed_key)) continue;
-							if(!is_numeric($val) && (!is_string($val) || strlen($val)==0)) continue;
-							if(strpos($val,'|@|')>0) $val = str_replace('|@|',',',$val);
+							if(in_array($key, $allowed_key)) {
+								continue;
+							}
+							if(!is_numeric($val) && (!is_string($val) || strlen($val)==0)) {
+								continue;
+							}
+							if(strpos($val,'|@|')>0) {
+								$val = str_replace('|@|',',',$val);
+							}
 							$attribute[] = sprintf('%s="%s"', $key, htmlspecialchars($val, ENT_COMPAT | ENT_HTML401, 'UTF-8', false));
 						}
 					}
@@ -637,11 +722,15 @@ 
 			}
 		}
 		// Compile the widget style.
-		if($args->widgetstyle) $widget_content_body = $this->compileWidgetStyle($args->widgetstyle,$widget, $widget_content_body, $args, $javascript_mode);
+		if($args->widgetstyle) {
+			$widget_content_body = $this->compileWidgetStyle($args->widgetstyle,$widget, $widget_content_body, $args, $javascript_mode);
+		}
 
 		$output = $widget_content_header . $widget_content_body . $widget_content_footer;
 		// Debug widget creation time information added to the results
-		if(__DEBUG__==3) $GLOBALS['__widget_excute_elapsed__'] += getMicroTime() - $start;
+		if(__DEBUG__==3) {
+			$GLOBALS['__widget_excute_elapsed__'] += getMicroTime() - $start;
+		}
 
 		$after = microtime(true);
 
@@ -674,7 +763,9 @@ 
 			$path = $oWidgetModel->getWidgetPath($widget);
 			// If you do not find the class file error output widget (html output)
 			$class_file = sprintf('%s%s.class.php', $path, $widget);
-			if(!file_exists($class_file)) return sprintf(Context::getLang('msg_widget_is_not_exists'), $widget);
+			if(!file_exists($class_file)) {
+				return sprintf(Context::getLang('msg_widget_is_not_exists'), $widget);
+			}
 			// Widget classes include
 			require_once($class_file);
 
@@ -685,9 +776,13 @@ 
 			}
 
 			$oWidget = new $widget();
-			if(!is_object($oWidget)) return sprintf(Context::getLang('msg_widget_object_is_null'), $widget);
+			if(!is_object($oWidget)) {
+				return sprintf(Context::getLang('msg_widget_object_is_null'), $widget);
+			}
 
-			if(!method_exists($oWidget, 'proc')) return sprintf(Context::getLang('msg_widget_proc_is_null'), $widget);
+			if(!method_exists($oWidget, 'proc')) {
+				return sprintf(Context::getLang('msg_widget_proc_is_null'), $widget);
+			}
 
 			$oWidget->widget_path = $path;
 
@@ -698,12 +793,16 @@ 
 
 	function compileWidgetStyle($widgetStyle,$widget,$widget_content_body, $args, $javascript_mode)
 	{
-		if(!$widgetStyle) return $widget_content_body;
+		if(!$widgetStyle) {
+			return $widget_content_body;
+		}
 
 		$oWidgetModel = getModel('widget');
 		// Bring extra_var widget style tie
 		$widgetstyle_info = $oWidgetModel->getWidgetStyleInfo($widgetStyle);
-		if(!$widgetstyle_info) return $widget_content_body;
+		if(!$widgetstyle_info) {
+			return $widget_content_body;
+		}
 
 		$widgetstyle_extra_var = new stdClass();
 		$widgetstyle_extra_var_key = get_object_vars($widgetstyle_info);
@@ -721,8 +820,7 @@ 
 		if($javascript_mode && $widget=='widgetBox')
 		{
 			Context::set('widget_content', '<div class="widget_inner">'.$widget_content_body.'</div>');
-		}
-		else
+		} else
 		{
 			Context::set('widget_content', $widget_content_body);
 		}
@@ -791,7 +889,9 @@ 
 			FileHandler::removeFile($cache_file);
 		}
 
-		if($vars->widget_cache>0) $vars->widget_sequence = getNextSequence();
+		if($vars->widget_cache>0) {
+			$vars->widget_sequence = getNextSequence();
+		}
 
 		$attribute = array();
 		foreach($vars as $key => $val)
@@ -801,7 +901,9 @@ 
 				unset($vars->{$key});
 				continue;
 			}
-			if(strpos($val,'|@|') > 0) $val = str_replace('|@|', ',', $val);
+			if(strpos($val,'|@|') > 0) {
+				$val = str_replace('|@|', ',', $val);
+			}
 			$vars->{$key} = Context::convertEncodingStr($val);
 			$attribute[] = sprintf('%s="%s"', $key, htmlspecialchars(Context::convertEncodingStr($val), ENT_COMPAT | ENT_HTML401, 'UTF-8', false));
 		}

tools/dbxml_validator/tests/xmlQueriesTest.php

Doc Comments +11 added lines

@@ -27,6 +27,11 @@ 
  

     // recursive glob

     // On Windows glob() is case-sensitive.

+
+    /**
+     * @param string $sDir
+     * @param integer $nFlags
+     */
     public function globr($sDir, $sPattern, $nFlags = NULL) 

     { 

 	// Get the list of all matching files currently in the 

@@ -129,6 +134,9 @@ 
 	$this->markTestIncomplete('XML Schema Language files should be fixed first.');

     }

 

+    /**
+     * @param integer $err_code
+     */
     public function invoke_testInvalidXmlFiles($filename, $err_code, $args = '')

     {

 	$cmd = $this->validator_cmd . ' '. $args . ' ' . escapeshellarg($filename);

@@ -166,6 +174,9 @@ 
 	return $this->invoke_testInvalidXmlFiles($filename, self::RETCODE_BUILTIN_CHECKS);

     }

 

+    /**
+     * @param string $dir_name
+     */
     public function getDirFilesList($dir_name)

     {

 	$output = array();

Indentation +101 added lines, -101 removed lines

@@ -2,33 +2,33 @@ 
 /* Copyright (C) NAVER <http://www.navercorp.com> */

 

 if (!defined('__DIR__'))

-    define('__DIR__', realpath(dirname(__FILE__))); 

+	define('__DIR__', realpath(dirname(__FILE__))); 

 

 /** The tests here are meant only for the built-in checks in validator.php, 

     and not for the entire syntax expressed by the .xsd files. */

 class XmlQueriesTest extends PHPUnit_Framework_TestCase

 {

-    // taken from validator.php

+	// taken from validator.php

 

-    const RETCODE_VALIDATOR_INTERNAL = 60;

-    const RETCODE_GENERIC_XML_SYNTAX = 50;

-    const RETCODE_QUERY_ELEMENT = 40;

-    const RETCODE_XSD_VALIDATION = 30;

-    const RETCODE_BUILTIN_CHECKS =    20;

-    const RETCODE_DB_SCHEMA_MATCH =10;	// no schema match is currently implemented.

-    const RETCODE_SUCCESS = 0;

+	const RETCODE_VALIDATOR_INTERNAL = 60;

+	const RETCODE_GENERIC_XML_SYNTAX = 50;

+	const RETCODE_QUERY_ELEMENT = 40;

+	const RETCODE_XSD_VALIDATION = 30;

+	const RETCODE_BUILTIN_CHECKS =    20;

+	const RETCODE_DB_SCHEMA_MATCH =10;	// no schema match is currently implemented.

+	const RETCODE_SUCCESS = 0;

 

-    public $validator_cmd;

+	public $validator_cmd;

 

-    public function setUp()

-    {

+	public function setUp()

+	{

 	 $this->validator_cmd = "php " . escapeshellarg(__DIR__ . '/../validate.php') . " ";

-    }

+	}

  

-    // recursive glob

-    // On Windows glob() is case-sensitive.

-    public function globr($sDir, $sPattern, $nFlags = NULL) 

-    { 

+	// recursive glob

+	// On Windows glob() is case-sensitive.

+	public function globr($sDir, $sPattern, $nFlags = NULL) 

+	{ 

 	// Get the list of all matching files currently in the 

 	// directory. 

 

@@ -41,25 +41,25 @@ 
 

 	foreach ($aDirs as $sSubDir) 

 	{ 

-	    if ($sSubDir != '.' && $sSubDir != '..')

-	    {

+		if ($sSubDir != '.' && $sSubDir != '..')

+		{

 		$aSubFiles = $this->globr($sSubDir, $sPattern, $nFlags); 

 		$aFiles = array_merge($aFiles, $aSubFiles); 

-	    }

+		}

 	} 

 

 	// return merged array with all (recursive) files

 	return $aFiles; 

-    } 

+	} 

 

-    /** Tests all XML Query and Schema Language files (in all modules/addons/widgets) in XE */

-    public function invoke_testReleasedXMLLangFiles

+	/** Tests all XML Query and Schema Language files (in all modules/addons/widgets) in XE */

+	public function invoke_testReleasedXMLLangFiles

 	(

-	    $released_files,

-	    $expected_return_code,

-	    $validator_args = ''

+		$released_files,

+		$expected_return_code,

+		$validator_args = ''

 	)

-    {

+	{

 	// this file is in tools/dbxml_validator/tests

 	$xe_dir = __DIR__ . '/../../..';

 

@@ -70,67 +70,67 @@ 
 	$xml_files = array();

 

 	foreach ($released_files as $released_file_mask)

-	    $xml_files =

+		$xml_files =

 		array_merge

 		(

-		    $xml_files,

-		    $this->globr

+			$xml_files,

+			$this->globr

 			(

-			    $xe_dir,

-			    $released_file_mask,

-			    GLOB_NOSORT | GLOB_NOESCAPE | GLOB_ERR

+				$xe_dir,

+				$released_file_mask,

+				GLOB_NOSORT | GLOB_NOESCAPE | GLOB_ERR

 			)

 		);

 

 	while ($cnt < count($xml_files))

 	{

-	    $cmd = $this->validator_cmd . $validator_args;

+		$cmd = $this->validator_cmd . $validator_args;

 

-	    // Validate 50 files at once

-	    foreach (array_slice($xml_files, $cnt, 50) as $xml_file)

+		// Validate 50 files at once

+		foreach (array_slice($xml_files, $cnt, 50) as $xml_file)

 		$cmd .= " " . escapeshellarg($xml_file);

 

-	    exec($cmd . ' 2>&1', $validator_output, $return_code);

+		exec($cmd . ' 2>&1', $validator_output, $return_code);

 

-	    $output_text = trim(trim(implode("\n", $validator_output)), "\n");

+		$output_text = trim(trim(implode("\n", $validator_output)), "\n");

 

-	    // Validator should not crash/exit-with-an-error.

-	    $this->assertLessThanOrEqual

+		// Validator should not crash/exit-with-an-error.

+		$this->assertLessThanOrEqual

 		(

-		    $expected_return_code,

-		    $return_code,

-		    "{$cmd}\n\n{$output_text}\nValidator returned code {$return_code}."

+			$expected_return_code,

+			$return_code,

+			"{$cmd}\n\n{$output_text}\nValidator returned code {$return_code}."

 		);

 

-	    $cnt += 50;

+		$cnt += 50;

+	}

 	}

-    }

 

-    public function testReleasedXMLQueryLangFiles()

-    {

+	public function testReleasedXMLQueryLangFiles()

+	{

 	$this->invoke_testReleasedXMLLangFiles

-	    (

+		(

 		array('queries/*.xml', 'xml_query/*.xml'),

 		self::RETCODE_QUERY_ELEMENT

-	    );

+		);

 

 	$this->markTestIncomplete('XML Query Language files should be fixed first.');

-    }

+	}

 

-    public function testReleasedXMLSchemaLangFiles()

-    {

+	public function testReleasedXMLSchemaLangFiles()

+	{

 	$this->invoke_testReleasedXMLLangFiles

-	    (

+		(

 		array('schemas/*.xml'),

 		self::RETCODE_BUILTIN_CHECKS,

 		' --schema-language'

-	    );

+		);

 

 	$this->markTestIncomplete('XML Schema Language files should be fixed first.');

-    }

+	}

 

-    public function invoke_testInvalidXmlFiles($filename, $err_code, $args = '')

-    {

+	public function invoke_testInvalidXmlFiles($filename, $err_code, $args = '')

+	{

 	$cmd = $this->validator_cmd . ' '. $args . ' ' . escapeshellarg($filename);

 	$validator_output = array();

 	$return_code = 0;

@@ -141,88 +141,88 @@ 
 

 	// Validator should not crash/exit-with-an-error.

 	$this->assertEquals

-	    (

+		(

 		$err_code,

 		$return_code,

 		"{$cmd}\n{$output_text}\nValidator returned code {$return_code}."

-	    );

+		);

 

 	// Validator should output some error on the test files

 	$basefilename = basename($filename);

 	$this->assertNotEmpty($output_text, "Error reporting failed for {$basefilename} validation.");

 

-    }

+	}

 

-    public function testInvalidQueryId()

-    {

+	public function testInvalidQueryId()

+	{

 	return $this->invoke_testInvalidXmlFiles(__DIR__.'/data/wrongQueryId.xml', self::RETCODE_QUERY_ELEMENT);

-    }

+	}

 

-    /**

-     * @dataProvider getFilesList

-     */

-    public function testInvalidXMLQueryFiles($filename)

-    {

+	/**

+	 * @dataProvider getFilesList

+	 */

+	public function testInvalidXMLQueryFiles($filename)

+	{

 	return $this->invoke_testInvalidXmlFiles($filename, self::RETCODE_BUILTIN_CHECKS);

-    }

+	}

 

-    public function getDirFilesList($dir_name)

-    {

+	public function getDirFilesList($dir_name)

+	{

 	$output = array();

 

 	$dir = opendir(__DIR__ . '/' . $dir_name);

 

 	if ($dir)

 	{

-	    $entry = readdir($dir);

+		$entry = readdir($dir);

 

-	    while ($entry !== FALSE)

-	    {

+		while ($entry !== FALSE)

+		{

 		$fname = __DIR__ . '/' . $dir_name .'/' . $entry;

 

 		if (!is_dir($fname)&& $entry != 'wrongQueryId.xml')

-		    $output[] = array($fname);

+			$output[] = array($fname);

 

 		$entry = readdir($dir);

-	    }

+		}

 

-	    closedir($dir);

+		closedir($dir);

 	}

 	else

-	    $this->assertFalse(TRUE);

+		$this->assertFalse(TRUE);

 

-        return $output;

-    }

+		return $output;

+	}

 

-    public function getFilesList()

-    {

+	public function getFilesList()

+	{

 	return $this->getDirFilesList('data');

-    }

+	}

 

-    public function getSchemaFilesList()

-    {

+	public function getSchemaFilesList()

+	{

 	return $this->getDirFilesList('data/schema');

-    }

+	}

 

-    public function getSchemaWarningFilesList()

-    {

+	public function getSchemaWarningFilesList()

+	{

 	return $this->getDirFilesList('data/schema/warnings');

-    }

+	}

 

-    /**

-     * @dataProvider getSchemaFilesList

-     */

-    public function testInvalidXMLSchemaFiles($filename)

-    {

+	/**

+	 * @dataProvider getSchemaFilesList

+	 */

+	public function testInvalidXMLSchemaFiles($filename)

+	{

 	return $this->invoke_testInvalidXmlFiles($filename, self::RETCODE_BUILTIN_CHECKS, '--schema-language');

-    }

+	}

 

-    /**

-     * @dataProvider getSchemaWarningFilesList

-     */

-    public function testWarningXMLSchemaFiles($filename)

-    {

+	/**

+	 * @dataProvider getSchemaWarningFilesList

+	 */

+	public function testWarningXMLSchemaFiles($filename)

+	{

 	return $this->invoke_testInvalidXmlFiles($filename, self::RETCODE_SUCCESS, '--schema-language');

-    }

+	}

 }

 

Spacing +13 added lines, -20 removed lines

@@ -14,8 +14,8 @@ 
     const RETCODE_GENERIC_XML_SYNTAX = 50;

     const RETCODE_QUERY_ELEMENT = 40;

     const RETCODE_XSD_VALIDATION = 30;

-    const RETCODE_BUILTIN_CHECKS =    20;

-    const RETCODE_DB_SCHEMA_MATCH =10;	// no schema match is currently implemented.

+    const RETCODE_BUILTIN_CHECKS = 20;

+    const RETCODE_DB_SCHEMA_MATCH = 10; // no schema match is currently implemented.

     const RETCODE_SUCCESS = 0;

 

     public $validator_cmd;

@@ -53,8 +53,7 @@ 
     } 

 

     /** Tests all XML Query and Schema Language files (in all modules/addons/widgets) in XE */

-    public function invoke_testReleasedXMLLangFiles

-	(

+    public function invoke_testReleasedXMLLangFiles(

 	    $released_files,

 	    $expected_return_code,

 	    $validator_args = ''

@@ -71,11 +70,9 @@ 
 

 	foreach ($released_files as $released_file_mask)

 	    $xml_files =

-		array_merge

-		(

+		array_merge(

 		    $xml_files,

-		    $this->globr

-			(

+		    $this->globr(

 			    $xe_dir,

 			    $released_file_mask,

 			    GLOB_NOSORT | GLOB_NOESCAPE | GLOB_ERR

@@ -95,8 +92,7 @@ 
 	    $output_text = trim(trim(implode("\n", $validator_output)), "\n");

 

 	    // Validator should not crash/exit-with-an-error.

-	    $this->assertLessThanOrEqual

-		(

+	    $this->assertLessThanOrEqual(

 		    $expected_return_code,

 		    $return_code,

 		    "{$cmd}\n\n{$output_text}\nValidator returned code {$return_code}."

@@ -108,8 +104,7 @@ 
 

     public function testReleasedXMLQueryLangFiles()

     {

-	$this->invoke_testReleasedXMLLangFiles

-	    (

+	$this->invoke_testReleasedXMLLangFiles(

 		array('queries/*.xml', 'xml_query/*.xml'),

 		self::RETCODE_QUERY_ELEMENT

 	    );

@@ -119,8 +114,7 @@ 
 

     public function testReleasedXMLSchemaLangFiles()

     {

-	$this->invoke_testReleasedXMLLangFiles

-	    (

+	$this->invoke_testReleasedXMLLangFiles(

 		array('schemas/*.xml'),

 		self::RETCODE_BUILTIN_CHECKS,

 		' --schema-language'

@@ -131,7 +125,7 @@ 
 

     public function invoke_testInvalidXmlFiles($filename, $err_code, $args = '')

     {

-	$cmd = $this->validator_cmd . ' '. $args . ' ' . escapeshellarg($filename);

+	$cmd = $this->validator_cmd . ' ' . $args . ' ' . escapeshellarg($filename);

 	$validator_output = array();

 	$return_code = 0;

 

@@ -140,8 +134,7 @@ 
 	$output_text = trim(trim(implode("\n", $validator_output)), "\n");

 

 	// Validator should not crash/exit-with-an-error.

-	$this->assertEquals

-	    (

+	$this->assertEquals(

 		$err_code,

 		$return_code,

 		"{$cmd}\n{$output_text}\nValidator returned code {$return_code}."

@@ -155,7 +148,7 @@ 
 

     public function testInvalidQueryId()

     {

-	return $this->invoke_testInvalidXmlFiles(__DIR__.'/data/wrongQueryId.xml', self::RETCODE_QUERY_ELEMENT);

+	return $this->invoke_testInvalidXmlFiles(__DIR__ . '/data/wrongQueryId.xml', self::RETCODE_QUERY_ELEMENT);

     }

 

     /**

@@ -178,9 +171,9 @@ 
 

 	    while ($entry !== FALSE)

 	    {

-		$fname = __DIR__ . '/' . $dir_name .'/' . $entry;

+		$fname = __DIR__ . '/' . $dir_name . '/' . $entry;

 

-		if (!is_dir($fname)&& $entry != 'wrongQueryId.xml')

+		if (!is_dir($fname) && $entry != 'wrongQueryId.xml')

 		    $output[] = array($fname);

 

 		$entry = readdir($dir);

Braces +16 added lines, -12 removed lines

@@ -1,8 +1,9 @@ 
 <?php

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

 

-if (!defined('__DIR__'))

-    define('__DIR__', realpath(dirname(__FILE__))); 

+if (!defined('__DIR__')) {
+    define('__DIR__', realpath(dirname(__FILE__)));
+}
 

 /** The tests here are meant only for the built-in checks in validator.php, 

     and not for the entire syntax expressed by the .xsd files. */

@@ -69,8 +70,8 @@ 
 	$cmd = $this->validator_cmd;

 	$xml_files = array();

 

-	foreach ($released_files as $released_file_mask)

-	    $xml_files =

+	foreach ($released_files as $released_file_mask) {
+		    $xml_files =

 		array_merge

 		(

 		    $xml_files,

@@ -80,15 +81,17 @@ 
 			    $released_file_mask,

 			    GLOB_NOSORT | GLOB_NOESCAPE | GLOB_ERR

 			)

-		);

+		);
+	}
 

 	while ($cnt < count($xml_files))

 	{

 	    $cmd = $this->validator_cmd . $validator_args;

 

 	    // Validate 50 files at once

-	    foreach (array_slice($xml_files, $cnt, 50) as $xml_file)

-		$cmd .= " " . escapeshellarg($xml_file);

+	    foreach (array_slice($xml_files, $cnt, 50) as $xml_file) {
+	    		$cmd .= " " . escapeshellarg($xml_file);
+	    }
 

 	    exec($cmd . ' 2>&1', $validator_output, $return_code);

 

@@ -180,16 +183,17 @@ 
 	    {

 		$fname = __DIR__ . '/' . $dir_name .'/' . $entry;

 

-		if (!is_dir($fname)&& $entry != 'wrongQueryId.xml')

-		    $output[] = array($fname);

+		if (!is_dir($fname)&& $entry != 'wrongQueryId.xml') {
+				    $output[] = array($fname);
+		}
 

 		$entry = readdir($dir);

 	    }

 

 	    closedir($dir);

-	}

-	else

-	    $this->assertFalse(TRUE);

+	} else {
+		    $this->assertFalse(TRUE);
+	}
 

         return $output;

     }

tools/dbxml_validator/validate.php

Doc Comments +5 added lines, -5 removed lines

@@ -173,7 +173,7 @@ 
  @return

  @param $xml_file

  @param $node

- @param $child_tag

+ @param string $child_tag

  */

 function checkDuplicateDescendants($xml_file, $node, $child_tag)

 {

@@ -199,13 +199,13 @@ 
 	messages.

  @param $node

 	The XML node with the children to be checked.

- @param $child_tags

+ @param string[] $child_tags

 	Array with tag names for the children elements

- @param $attr_tags

+ @param string[] $attr_tags

 	Array with names of attributes to be checked. If multiple attributes

 	are given, than the first one that is present on a child is included

 	in the check.

- @param $key

+ @param boolean $key

 	True if child elements are required to expose at least one of the 

 	attribute. False if only the child nodes with some of the

 	attributes present are to be checked.

@@ -489,7 +489,7 @@ 
 	@return

 	@param $xml_file

 	@param $container_element

-	@param $child_tag

+	@param string $child_tag

 */

 function checkVarContentsValidation

 	(

Indentation +28 added lines, -29 removed lines

@@ -90,7 +90,7 @@ 
 		@param $line_no

 		@param $message

 		@access

-	*/

+	 */

 	public function __construct($file, $line_no, $message)

 	{

 		parent::__construct("{$file}({$line_no}):\n\t$message");

@@ -112,7 +112,7 @@ 
 	@developer

 	@return

 	@access

-	*/

+	 */

 	public function __destruct()

 	{

 		libxml_clear_errors();

@@ -834,8 +834,7 @@ 
 

 	@brief

 	@developer

-

-	*/

+	 */

 	class RestoreWorkDir

 	{

 		protected $dirname;

@@ -845,7 +844,7 @@ 
 			@developer

 			@return

 			@access

-			*/

+		 */

 		public function __destruct()

 		{

 			try

@@ -868,7 +867,7 @@ 
 			@developer

 			@return

 			@access

-			*/

+		 */

 		public function __construct()

 		{

 			$this->dirname = getcwd();

@@ -1113,7 +1112,7 @@ 
 		@return

 		@access

 		@param $val

-	*/

+	 */

 	public function code($val = -1)

 	{

 		if($val == -1)

@@ -1135,7 +1134,7 @@ 
 		@return

 		@access

 		@param $val

-	*/

+	 */

 	public function push($val)

 	{

 		$this->save = $this->exit_code;

@@ -1147,7 +1146,7 @@ 
 		@developer

 		@access

 		@return

-	*/

+	 */

 	public function pop()

 	{

 		$this->exit_code = $this->save;

@@ -1160,7 +1159,7 @@ 
 		@return

 		@access

 		@param $val

-	*/

+	 */

 	public function __construct($val = 0)

 	{

 		$this->save = self::RETCODE_VALIDATOR_INTERNAL;

@@ -1181,7 +1180,7 @@ 
 		@developer

 		@access

 		@return

-	*/

+	 */

 	public function __destruct()

 	{

 		if($this->file_name)

@@ -1196,7 +1195,7 @@ 
 		@access

 		@return

 		@param $file_name

-	*/

+	 */

 	public function __construct($file_name)

 	{

 		$this->file_name = $file_name;

@@ -1439,7 +1438,7 @@ 
 

 			@brief

 			@developer

-		*/

+		 */

 		class Context

 		{

 			protected static $db_info = NULL;

@@ -1449,7 +1448,7 @@ 
 				@developer

 				@return

 				@access

-			*/

+			 */

 			public static function isInstalled()

 			{

 				return TRUE;

@@ -1460,7 +1459,7 @@ 
 				@developer

 				@return

 				@access

-			*/

+			 */

 			public static function getLangType()

 			{

 				return 'en';

@@ -1471,7 +1470,7 @@ 
 				@developer

 				@return

 				@access

-			*/

+			 */

 			public static function getLang()

 			{

 				return 'en';

@@ -1482,7 +1481,7 @@ 
 				@developer

 				@return

 				@access

-			*/

+			 */

 			public static function getDBType()

 			{

 				if(self::$db_info)

@@ -1501,7 +1500,7 @@ 
 				@return

 				@access

 				@param $db_info

-			*/

+			 */

 			public static function setDBInfo($db_info)

 			{

 				self::$db_info = $db_info;

@@ -1512,7 +1511,7 @@ 
 				@developer

 				@return

 				@access

-			*/

+			 */

 			public static function getDBInfo()

 			{

 				return self::$db_info;

@@ -1524,7 +1523,7 @@ 
 				@return

 				@access

 				@param $str

-			*/

+			 */

 			public static function convertEncodingStr($str)

 			{

 				return $str;

@@ -1535,7 +1534,7 @@ 
 				@developer

 				@return

 				@access

-			*/

+			 */

 			public static function setNoDBInfo()

 			{

 				$db_info = (object)NULL;

@@ -1562,7 +1561,7 @@ 
 				@developer

 				@access

 				@return

-			*/

+			 */

 			public static function setMysqlDBInfo()

 			{

 				$db_info = (object)NULL;

@@ -1604,7 +1603,7 @@ 
 				@developer

 				@return

 				@access

-			*/

+			 */

 			public static function setMysqliDBInfo()

 			{

 				$db_info = (object)NULL;

@@ -1646,7 +1645,7 @@ 
 				@developer

 				@return

 				@access

-			*/

+			 */

 			public static function setCubridDBInfo()

 			{

 				$db_info = (object)NULL;

@@ -1688,7 +1687,7 @@ 
 				@developer

 				@return

 				@access

-			*/

+			 */

 			public static function setMssqlDBInfo()

 			{

 				$db_info = (object)NULL;

@@ -1729,7 +1728,7 @@ 
 		/**

 			@brief

 			@developer

-		*/

+		 */

 		class Any_prop_obj_base

 		{

 			/**

@@ -1738,7 +1737,7 @@ 
 				@return

 				@param $property

 				@access

-			*/

+			 */

 			public function __get($property)

 			{

 				return NULL;

@@ -1748,7 +1747,7 @@ 
 		/**

 			@brief

 			@developer

-			*/

+		 */

 		class LangArgFilterErrorMessage

 		{

 			/**

@@ -1757,7 +1756,7 @@ 
 				@return

 				@param $property

 				@access

-			*/

+			 */

 			public function __get($property)

 			{

 				return 'Argument filter error';

Spacing +186 added lines, -205 removed lines

@@ -95,7 +95,7 @@ 
 	{

 		parent::__construct("{$file}({$line_no}):\n\t$message");

 

-		$this->xml_file	= $file;

+		$this->xml_file = $file;

 		$this->xml_line_no = $line_no;

 		$this->xml_message = $message;

 	}

@@ -133,21 +133,21 @@ 
 

 	$libXmlErrors = libxml_get_errors();

 

-	if(count($libXmlErrors))

+	if (count($libXmlErrors))

 	{

-		if(!$filename)

+		if (!$filename)

 		{

 			$filename = $libXmlErrors[0]->file;

 		}

 

 		$msg = '';

 

-		foreach($libXmlErrors as $libXmlError)

+		foreach ($libXmlErrors as $libXmlError)

 		{

 			$msg .= "{$libXmlError->file}({$libXmlError->line}):\n\t {$libXmlError->message}";

 		}

 

-		if($throw_error)

+		if ($throw_error)

 		{

 			throw new ErrorMessage($msg);

 		}

@@ -158,7 +158,7 @@ 
 	}

 	else

 	{

-		if($throw_error)

+		if ($throw_error)

 		{

 			throw new ErrorMessage('Schema validation failed.');

 		}

@@ -179,7 +179,7 @@ 
 {

 	$children = $node->getElementsByTagName($child_tag);

 

-	if($children->length > 1)

+	if ($children->length > 1)

 	{

 		throw 

 			new XmlSchemaError(

@@ -217,21 +217,21 @@ 
 {

 	$key_values = array();

 

-	foreach($node->childNodes as $child_node)

+	foreach ($node->childNodes as $child_node)

 	{

-		if($child_node->nodeType == XML_ELEMENT_NODE

+		if ($child_node->nodeType == XML_ELEMENT_NODE

 					&&

 				in_array($child_node->tagName, $child_tags))

 		{

 			$key_value = NULL;

 

-			foreach($attr_tags as $attr_tag)

+			foreach ($attr_tags as $attr_tag)

 			{

-				if($child_node->hasAttribute($attr_tag))

+				if ($child_node->hasAttribute($attr_tag))

 				{

 					$key_value = $child_node->getAttribute($attr_tag);

 

-					if(array_key_exists($key_value, $key_values))

+					if (array_key_exists($key_value, $key_values))

 					{

 						throw

 							new XmlSchemaError(

@@ -246,7 +246,7 @@ 
 				}

 			}

 

-			if(!$key_value && $key)

+			if (!$key_value && $key)

 			{

 				throw

 					new XmlSchemaError(

@@ -282,19 +282,19 @@ 
 	$table_name = NULL;

 	$join_type = NULL;

 

-	if($table_element->hasAttribute('name'))

+	if ($table_element->hasAttribute('name'))

 	{

 		$table_name = $table_element->getAttribute('name');

 	}

 

-	if($table_element->hasAttribute('type'))

+	if ($table_element->hasAttribute('type'))

 	{

 		$join_type = $table_element->getAttribute('type');

 	}

 

-	if($table_element->getAttribute('query') == 'true')

+	if ($table_element->getAttribute('query') == 'true')

 	{

-		if($table_name !== NULL)

+		if ($table_name !== NULL)

 		{

 			throw

 				new XmlSchemaError(

@@ -304,7 +304,7 @@ 
 				);

 		}

 

-		if($join_type !== NULL)

+		if ($join_type !== NULL)

 		{

 			throw

 				new XmlSchemaError(

@@ -322,9 +322,9 @@ 
 		// check contents for a select list or a table-specification

 		$has_query_clauses = FALSE;

 

-		foreach($table_element->childNodes as $query_clause)

+		foreach ($table_element->childNodes as $query_clause)

 		{

-			if($query_clause->nodeType == XML_ELEMENT_NODE

+			if ($query_clause->nodeType == XML_ELEMENT_NODE

 						&&

 					(

 						$query_clause->tagName == 'columns' 

@@ -335,7 +335,7 @@ 
 			}

 		}

 

-		if(!$has_query_clauses)

+		if (!$has_query_clauses)

 		{

 			throw

 				new XmlSchemaError(

@@ -351,17 +351,17 @@ 
 	{

 		// base table or view

 

-		if($join_type !== NULL)

+		if ($join_type !== NULL)

 		{

 			$has_conditions_element = FALSE;

 

-			foreach($table_element->childNodes as $child_node)

+			foreach ($table_element->childNodes as $child_node)

 			{

-				if($child_node->nodeType == XML_ELEMENT_NODE)

+				if ($child_node->nodeType == XML_ELEMENT_NODE)

 				{

-					if($child_node->tagName == 'conditions')

+					if ($child_node->tagName == 'conditions')

 					{

-						if($has_conditions_element)

+						if ($has_conditions_element)

 						{

 							throw

 								new XmlSchemaError(

@@ -387,7 +387,7 @@ 
 				}

 			}

 

-			if(!$has_conditions_element)

+			if (!$has_conditions_element)

 			{

 				throw

 					new XmlSchemaError(

@@ -399,9 +399,9 @@ 
 		}

 		else

 		{

-			foreach($table_element->childNodes as $child_node)

+			foreach ($table_element->childNodes as $child_node)

 			{

-				if($child_node->nodeType == XML_ELEMENT_NODE)

+				if ($child_node->nodeType == XML_ELEMENT_NODE)

 				{

 					throw

 						new XmlSchemaError(

@@ -429,26 +429,25 @@ 
  */

 function checkTablesClause($xml_file, $tables_element)

 {

-	checkUniqueKey

-		(

+	checkUniqueKey(

 			$xml_file,

 			$tables_element,

-			array('table'),				// child elements to be checked

-			array('alias', 'name'),		// attributes to be checked

+			array('table'), // child elements to be checked

+			array('alias', 'name'), // attributes to be checked

 			TRUE						// attributes are required

 		);

 

-	foreach($tables_element->childNodes as $table)

+	foreach ($tables_element->childNodes as $table)

 	{

-		if($table->nodeType == XML_ELEMENT_NODE

+		if ($table->nodeType == XML_ELEMENT_NODE

 					&&

 				$table->tagName == 'table')

 		{

 			checkTableExpression($xml_file, $table);

 

-			if($table->getAttribute('query') == 'true')

+			if ($table->getAttribute('query') == 'true')

 			{

-				validate_select_query($xml_file, $table);		// recursive call

+				validate_select_query($xml_file, $table); // recursive call

 			}

 		}

 	}

@@ -470,12 +469,11 @@ 
  */

 function checkSelectListClause($xml_file, $columns_element)

 {

-	checkUniqueKey

-		(

+	checkUniqueKey(

 			$xml_file,

 			$columns_element,

-			array('column', 'query'),	// child elements

-			array('alias', 'name'),		// attributes

+			array('column', 'query'), // child elements

+			array('alias', 'name'), // attributes

 			FALSE						// ignore if no attributes present

 		);

 }

@@ -491,8 +489,7 @@ 
 	@param $container_element

 	@param $child_tag

 */

-function checkVarContentsValidation

-	(

+function checkVarContentsValidation(

 		$xml_file,

 		$container_element,

 		$child_tag

@@ -503,23 +500,22 @@ 
 

 	static

 		$var_attrs =

-			array

-			(

+			array(

 				'filter', 'notnull', 'minlength',

 				'maxlength'

 			);

 

-	foreach($container_element->childNodes as $child_node)

+	foreach ($container_element->childNodes as $child_node)

 	{

-		if($child_node->nodeType == XML_ELEMENT_NODE

+		if ($child_node->nodeType == XML_ELEMENT_NODE

 					&&

 				$child_node->tagName == $child_tag)

 		{

-			if(!$child_node->hasAttribute($key_attr))

+			if (!$child_node->hasAttribute($key_attr))

 			{

-				foreach($var_attrs as $var_attr)

+				foreach ($var_attrs as $var_attr)

 				{

-					if($child_node->hasAttribute($var_attr))

+					if ($child_node->hasAttribute($var_attr))

 					{

 						throw

 							new XmlSchemaError(

@@ -551,9 +547,9 @@ 
 	$has_var_attribute = $condition->hasAttribute('var') || $condition->hasAttribute('default');

 	$query_line_no = -1;

 

-	foreach($condition->childNodes as $query_node)

+	foreach ($condition->childNodes as $query_node)

 	{

-		if($query_node->nodeType == XML_ELEMENT_NODE

+		if ($query_node->nodeType == XML_ELEMENT_NODE

 					&&

 				$query_node->tagName == 'query')

 		{

@@ -564,7 +560,7 @@ 
 		}

 	}

 

-	if($child_query_node && $has_var_attribute)

+	if ($child_query_node && $has_var_attribute)

 	{

 		throw 

 			new XmlSchemaError(

@@ -574,7 +570,7 @@ 
 			);

 	}

 

-	if(!($child_query_node || $has_var_attribute))

+	if (!($child_query_node || $has_var_attribute))

 	{

 		throw

 			new XmlSchemaError(

@@ -601,18 +597,18 @@ 
 {

 	$first_child = TRUE;

 

-	foreach($conditions->childNodes as $child_node)

+	foreach ($conditions->childNodes as $child_node)

 	{

-		if($child_node->nodeType == XML_ELEMENT_NODE)

+		if ($child_node->nodeType == XML_ELEMENT_NODE)

 		{

 			// check for 'pipe' attribute

-			if($first_child)

+			if ($first_child)

 			{

 				$first_child = FALSE;

 			}

 			else

 			{

-				if(!$child_node->hasAttribute('pipe'))

+				if (!$child_node->hasAttribute('pipe'))

 				{

 					throw

 						new XmlSchemaError(

@@ -626,19 +622,19 @@ 
 			}

 

 			// recurse in condition groups/queries

-			if($child_node->tagName == 'group')

+			if ($child_node->tagName == 'group')

 			{

 				checkConditionsGroup($xml_file, $child_node);

 			}

 			else

 			{

-				if($child_node->tagName == 'query')

+				if ($child_node->tagName == 'query')

 				{

 					validate_select_query($xml_file, $child_node);

 				}

 				else

 				{

-					if($child_node->tagName == 'condition')

+					if ($child_node->tagName == 'condition')

 					{

 						checkConditionElement($xml_file, $child_node);

 					}

@@ -664,12 +660,11 @@ 
  */

 function checkNavigationClauses($xml_file, $navigation_element)

 {

-	foreach(array('list_count', 'page_count', 'page')

+	foreach (array('list_count', 'page_count', 'page')

 				as

 			$navigation_el)

 	{

-		checkDuplicateDescendants

-			(

+		checkDuplicateDescendants(

 				$xml_file,

 				$navigation_element,

 				$navigation_el

@@ -693,11 +688,11 @@ 
  */

 function validate_select_query($xml_file, $query_element)

 {

-	foreach($query_element->childNodes as $select_clause)

+	foreach ($query_element->childNodes as $select_clause)

 	{

-		if($select_clause->nodeType == XML_ELEMENT_NODE)

+		if ($select_clause->nodeType == XML_ELEMENT_NODE)

 		{

-			switch($select_clause->tagName)

+			switch ($select_clause->tagName)

 			{

 				case 'columns':

 					checkSelectListClause($xml_file, $select_clause);

@@ -728,11 +723,11 @@ 
  */

 function validate_update_query($xml_file, $query_element)

 {

-	foreach($query_element->childNodes as $update_clause)

+	foreach ($query_element->childNodes as $update_clause)

 	{

-		if($update_clause->nodeType == XML_ELEMENT_NODE)

+		if ($update_clause->nodeType == XML_ELEMENT_NODE)

 		{

-			switch($update_clause->tagName)

+			switch ($update_clause->tagName)

 			{

 				case 'tables':

 					checkTablesClause($xml_file, $update_clause);

@@ -755,11 +750,11 @@ 
  */

 function validate_delete_query($xml_file, $query_element)

 {

-	foreach($query_element->childNodes as $delete_clause)

+	foreach ($query_element->childNodes as $delete_clause)

 	{

-		if($delete_clause->nodeType == XML_ELEMENT_NODE)

+		if ($delete_clause->nodeType == XML_ELEMENT_NODE)

 		{

-			switch($delete_clause->tagName)

+			switch ($delete_clause->tagName)

 			{

 				case 'conditions':

 					checkConditionsGroup($xml_file, $delete_clause);

@@ -778,11 +773,11 @@ 
  */

 function validate_insert_select_query($xml_file, $query_element)

 {

-	foreach($query_element->childNodes as $statement_clause)

+	foreach ($query_element->childNodes as $statement_clause)

 	{

-		if($statement_clause->nodeType == XML_ELEMENT_NODE)

+		if ($statement_clause->nodeType == XML_ELEMENT_NODE)

 		{

-			switch($statement_clause->tagName)

+			switch ($statement_clause->tagName)

 			{

 				case 'query':

 					validate_select_query($xml_file, $statement_clause);

@@ -793,8 +788,7 @@ 
 }

 

 $validate_query_type = 

-	array

-		(

+	array(

 			// 'insert' =>		

 							// there is currently nothing special to check

 							// for a plain insert, all the needed checks

@@ -818,13 +812,13 @@ 
 

 	$action = $query_element->getAttribute('action');

 

-	if(array_key_exists($action, $validate_query_type))

+	if (array_key_exists($action, $validate_query_type))

 	{

 		$validate_query_type[$action]($xml_file, $query_element);

 	}

 }

 

-if(strpos(PHP_SAPI, 'cli') !== FALSE

+if (strpos(PHP_SAPI, 'cli') !== FALSE

 			|| 

 		strpos(PHP_SAPI, 'cgi') !== FALSE)

 {

@@ -857,7 +851,7 @@ 
 				print "Failed to restore working dir {$this->dirname}.";

 			}

 

-			if(!$success)

+			if (!$success)

 			{

 				print "Failed to restore working dir {$this->dirname}.";

 			}

@@ -873,7 +867,7 @@ 
 		{

 			$this->dirname = getcwd();

 

-			if(!$this->dirname)

+			if (!$this->dirname)

 			{

 				throw new ErrorMessage("Failed to get current directory.");

 			}

@@ -903,7 +897,7 @@ 
 	$lowercase_name = strtolower($xml_path_info['basename']);

 	$uppercase_name = strtoupper($xml_path_info['basename']);

 

-	if(strlen($lowercase_name) != $filename_len

+	if (strlen($lowercase_name) != $filename_len

 				||

 			strlen($uppercase_name) != $filename_len)

 	{

@@ -916,9 +910,9 @@ 
 

 	$varing_case_filename = '';

 

-	for($i = 0; $i < $filename_len; $i++)

+	for ($i = 0; $i < $filename_len; $i++)

 	{

-		if($lowercase_name[$i] != $uppercase_name[$i])

+		if ($lowercase_name[$i] != $uppercase_name[$i])

 		{

 			$varing_case_filename .= "[{$lowercase_name[$i]}{$uppercase_name[$i]}]";

 		}

@@ -932,13 +926,13 @@ 
 

 	$restoreWorkDir = new RestoreWorkDir();

 

-	if($glob_pattern)

+	if ($glob_pattern)

 	{

 		// change current dir to the xml file directory to keep

 		// glob pattern shorter (maximum 260 chars).

 		$success = chdir($glob_pattern);

 

-		if(!$success)

+		if (!$success)

 		{

 			throw new ErrorMessage("Failed to change work dir to {$glob_pattern}.");

 		}

@@ -950,14 +944,14 @@ 
 	// realpath() would have the same effect, but it is not documented as such

 	$matched_files = glob($glob_pattern, GLOB_NOSORT | GLOB_NOESCAPE | GLOB_ERR);

 

-	unset($RestoreWorkDir);		// restore work dir after call to glob()

+	unset($RestoreWorkDir); // restore work dir after call to glob()

 

-	if($matched_files === FALSE || !is_array($matched_files))

+	if ($matched_files === FALSE || !is_array($matched_files))

 	{

 		throw new ErrorMessage("Directory listing for $xml_file failed.");

 	}

 

-	switch(count($matched_files))

+	switch (count($matched_files))

 	{

 		case 0:

 			throw new ErrorMessage("Directory listing for $xml_file failed.");

@@ -968,9 +962,9 @@ 
 		default:

 			// more than one files with the same name and different case

 			// case-sensitive file system

-			foreach($mached_files as $matched_file)

+			foreach ($mached_files as $matched_file)

 			{

-				if(pathinfo($matched_file, PATHINFO_BASENAME) == $xml_path_info['basename'])

+				if (pathinfo($matched_file, PATHINFO_BASENAME) == $xml_path_info['basename'])

 				{

 					return ($xml_path_info['filename'] == $query_id);

 				}

@@ -979,7 +973,7 @@ 
 

 	}

 

-	throw new ErrorMessage("Internal application error.");  // unreachable

+	throw new ErrorMessage("Internal application error."); // unreachable

 }

 

 /**

@@ -1000,9 +994,9 @@ 
 */

 function validate_schema_doc($xml_file, $table_element)

 {

-	foreach($table_element->childNodes as $col_node)

+	foreach ($table_element->childNodes as $col_node)

 	{

-		if($col_node->nodeType == XML_ELEMENT_NODE

+		if ($col_node->nodeType == XML_ELEMENT_NODE

 					&&

 				$col_node->tagName == 'column')

 		{

@@ -1010,10 +1004,9 @@ 
 			$col_size = NULL;

 

 			// check auto-increment column

-			if($col_node->hasAttribute('auto_increment'))

+			if ($col_node->hasAttribute('auto_increment'))

 			{

-				fwrite

-					(

+				fwrite(

 						fopen('php://stdout', 'wt'), 

 						$xml_file . '(' . $col_node->getLineNo() . ")\n\t"

 							.

@@ -1023,7 +1016,7 @@ 
 				static

 					$autoinc_types = array('number', 'bignumber');

 

-				if(!in_array($col_type, $autoinc_types))

+				if (!in_array($col_type, $autoinc_types))

 				{

 					throw

 						new XmlSchemaError(

@@ -1037,10 +1030,9 @@ 
 			}

 

 			// check tinytext

-			if($col_type == 'tinytext')

+			if ($col_type == 'tinytext')

 			{

-				fwrite

-					(

+				fwrite(

 						fopen('php://stdout', 'wt'), 

 						$xml_file . '(' . $col_node->getLineNo() . ")\n\t"

 							.

@@ -1049,12 +1041,12 @@ 
 			}

 

 			// check size attribute

-			if($col_node->hasAttribute('size'))

+			if ($col_node->hasAttribute('size'))

 			{

 				$col_size = $col_node->getAttribute('size');

 			}

 

-			if($col_type == 'varchar' && $col_size === NULL)

+			if ($col_type == 'varchar' && $col_size === NULL)

 			{

 				throw 

 					new XmlSchemaError(

@@ -1068,7 +1060,7 @@ 
 				$varsize_types = array('char', 'varchar', 'float');

 					

 

-			if($col_size !== NULL  && !in_array($col_type, $varsize_types))

+			if ($col_size !== NULL && !in_array($col_type, $varsize_types))

 			{

 				throw 

 					new XmlSchemaError(

@@ -1102,8 +1094,8 @@ 
 	const RETCODE_GENERIC_XML_SYNTAX = 50;

 	const RETCODE_QUERY_ELEMENT = 40;

 	const RETCODE_XSD_VALIDATION = 30;

-	const RETCODE_BUILTIN_CHECKS =	20;

-	const RETCODE_DB_SCHEMA_MATCH = 10;	// no schema match is currently implemented.

+	const RETCODE_BUILTIN_CHECKS = 20;

+	const RETCODE_DB_SCHEMA_MATCH = 10; // no schema match is currently implemented.

 	const RETCODE_SUCCESS = 0;

 

 

@@ -1116,13 +1108,13 @@ 
 	*/

 	public function code($val = -1)

 	{

-		if($val == -1)

+		if ($val == -1)

 		{

 			return $this->exit_code;

 		}

 		else

 		{

-			if($this->exit_code < $val)

+			if ($this->exit_code < $val)

 			{

 				$this->exit_code = $val;

 			}

@@ -1184,7 +1176,7 @@ 
 	*/

 	public function __destruct()

 	{

-		if($this->file_name)

+		if ($this->file_name)

 		{

 			unlink($this->file_name);

 			$this->file_name = NULL;

@@ -1214,12 +1206,12 @@ 
 	$cmdname = CMD_NAME;

 

 	// php manual says resources should not normally be declared constant

-	if(!defined('STDERR'))

+	if (!defined('STDERR'))

 	{

 		define('STDERR', fopen('php://stderr', 'wt'));

 	}

 

-	if(!defined('__DIR__'))

+	if (!defined('__DIR__'))

 	{

 		define('__DIR__', dirname(__FILE__)); 

 	}

@@ -1234,7 +1226,7 @@ 
 	$query_args = NULL;

 	$query_args_file = NULL;

 

-	while($argc >= 2 && $argv[1][0] == '-')

+	while ($argc >= 2 && $argv[1][0] == '-')

 	{

 		$option = $argv[1];

 

@@ -1242,17 +1234,17 @@ 
 		$argv = array_values($argv);

 		$argc = count($argv);

 

-		switch($option)

+		switch ($option)

 		{

 			case '-s':

 			case '--schema':

 			case '--schema-language':

-				if($query_args !== NULL)

+				if ($query_args !== NULL)

 				{

 					throw new SyntaxError("Both --args-string and --schema-language options given.");

 				}

 

-				if($query_args_file !== NULL)

+				if ($query_args_file !== NULL)

 				{

 					throw new SyntaxError("Both --args-file and --schema-language options given.");

 				}

@@ -1272,7 +1264,7 @@ 
 

 			case '--xe-path':

 			case '--xe':

-				if($argc < 2)

+				if ($argc < 2)

 				{

 					throw

 						new SyntaxError("Option '{$option}' requires an argument., see `{$cmdname} --help`");

@@ -1290,17 +1282,17 @@ 
 			case '--args-string':

 			case '--arguments':

 			case '--args':

-				if($schema_language !== NULL)

+				if ($schema_language !== NULL)

 				{

 					throw new SyntaxError("Both --schema-language and --args-string options given.");

 				}

 

-				if($query_args_file !== NULL)

+				if ($query_args_file !== NULL)

 				{

 					throw new SyntaxError("Both --args-string and --args-file options given.");

 				}

 

-				if($argc < 2)

+				if ($argc < 2)

 				{

 					throw

 						new SyntaxError("Option '{$option}' requires an argument., see `{$cmdname} --help`");

@@ -1316,17 +1308,17 @@ 
 

 			case '--arguments-file':

 			case '--args-file':

-				if($schema_language !== NULL)

+				if ($schema_language !== NULL)

 				{

 					throw new SyntaxError("Both --schema-language and --args-file options given.");

 				}

 

-				if($query_args !== NULL)

+				if ($query_args !== NULL)

 				{

 					throw new SyntaxError("Both --args-string and --args-file options given.");

 				}

 

-				if($argc < 2)

+				if ($argc < 2)

 				{

 					throw

 						new SyntaxError("Option '{$option}' requires an argument., see `{$cmdname} --help`");

@@ -1357,7 +1349,7 @@ 
 		}

 	}

 

-	if($argc < 2 ||

+	if ($argc < 2 ||

 			(

 				$argc == 2

 					&&

@@ -1415,21 +1407,21 @@ 
 	$query_user_args = array();

 

 	// check $xe_path, $query_args

-	if(!$validate_only)

+	if (!$validate_only)

 	{

-		if($xe_path == NULL)

+		if ($xe_path == NULL)

 		{

 			// assume validator.php is in directory .../xe/tools/dbxml_validator/ in an XE installation

 			$xe_path = dirname(dirname(realpath(__DIR__)));

 		}

 

-		if(!file_exists($xe_path . '/index.php'))

+		if (!file_exists($xe_path . '/index.php'))

 		{

 			throw

 				new ErrorMessage("File index.php not found in {$xe_path}.");

 		}

 

-		if(!defined('_XE_PATH_'))

+		if (!defined('_XE_PATH_'))

 		{

 			define('_XE_PATH_', $xe_path . '/');

 		}

@@ -1485,7 +1477,7 @@ 
 			*/

 			public static function getDBType()

 			{

-				if(self::$db_info)

+				if (self::$db_info)

 				{

 					return self::$db_info->master_db['db_type'];

 				}

@@ -1538,10 +1530,9 @@ 
 			*/

 			public static function setNoDBInfo()

 			{

-				$db_info = (object)NULL;

+				$db_info = (object) NULL;

 				$db_info->master_db =

-					array

-						(

+					array(

 							'db_type' => NULL,

 							'db_hostname' => NULL,

 							'db_port' => NULL,

@@ -1565,10 +1556,9 @@ 
 			*/

 			public static function setMysqlDBInfo()

 			{

-				$db_info = (object)NULL;

+				$db_info = (object) NULL;

 				$db_info->master_db =

-					array

-						(

+					array(

 							'db_type' => 'mysql',

 							'db_hostname' => NULL,

 							'db_port' => NULL,

@@ -1584,7 +1574,7 @@ 
 

 				self::setDBInfo($db_info);

 

-				if(array_key_exists('__DB__', $GLOBALS)

+				if (array_key_exists('__DB__', $GLOBALS)

 							&&

 						array_key_exists($db_info->master_db['db_type'], $GLOBALS['__DB__']))

 				{

@@ -1607,10 +1597,9 @@ 
 			*/

 			public static function setMysqliDBInfo()

 			{

-				$db_info = (object)NULL;

+				$db_info = (object) NULL;

 				$db_info->master_db =

-					array

-						(

+					array(

 							'db_type' => 'mysqli',

 							'db_hostname' => NULL,

 							'db_port' => NULL,

@@ -1626,7 +1615,7 @@ 
 

 				self::setDBInfo($db_info);

 

-				if(array_key_exists('__DB__', $GLOBALS)

+				if (array_key_exists('__DB__', $GLOBALS)

 							&&

 						array_key_exists($db_info->master_db['db_type'], $GLOBALS['__DB__']))

 				{

@@ -1649,10 +1638,9 @@ 
 			*/

 			public static function setCubridDBInfo()

 			{

-				$db_info = (object)NULL;

+				$db_info = (object) NULL;

 				$db_info->master_db =

-					array

-						(

+					array(

 							'db_type' => 'cubrid',

 							'db_hostname' => NULL,

 							'db_port' => NULL,

@@ -1668,7 +1656,7 @@ 
 

 				self::setDBInfo($db_info);

 

-				if(array_key_exists('__DB__', $GLOBALS)

+				if (array_key_exists('__DB__', $GLOBALS)

 							&&

 						array_key_exists($db_info->master_db['db_type'], $GLOBALS['__DB__']))

 				{

@@ -1691,10 +1679,9 @@ 
 			*/

 			public static function setMssqlDBInfo()

 			{

-				$db_info = (object)NULL;

+				$db_info = (object) NULL;

 				$db_info->master_db =

-					array

-						(

+					array(

 							'db_type' => 'mssql',

 							'db_hostname' => NULL,

 							'db_port' => NULL,

@@ -1710,7 +1697,7 @@ 
 

 				self::setDBInfo($db_info);

 

-				if(array_key_exists('__DB__', $GLOBALS)

+				if (array_key_exists('__DB__', $GLOBALS)

 							&&

 						array_key_exists($db_info->master_db['db_type'], $GLOBALS['__DB__']))

 				{

@@ -1765,25 +1752,25 @@ 
 		}

 

 		global $lang;

-		$lang = new Any_prop_obj_base();	// to return NULL on non-existent properties

+		$lang = new Any_prop_obj_base(); // to return NULL on non-existent properties

 		$lang->filter = New LangArgFilterErrorMessage();

 

-		if(!defined('__XE__'))

+		if (!defined('__XE__'))

 		{

 			define('__XE__', TRUE);

 		}

 

-		if(!defined('__ZBXE__'))

+		if (!defined('__ZBXE__'))

 		{

 			define('__ZBXE__', TRUE);

 		}

 

-		if(!defined('__DEBUG__'))

+		if (!defined('__DEBUG__'))

 		{

 			define('__DEBUG__', 0);

 		}

 

-		if(!defined('__DEBUG_QUERY__'))

+		if (!defined('__DEBUG_QUERY__'))

 		{

 			define('__DEBUG_QUERY__', 0);

 		}

@@ -1807,7 +1794,7 @@ 
 		require_once(__DIR__ . '/connect_wrapper.php');

 

 		// check $query_args, $query_args_file

-		if($query_args_file)

+		if ($query_args_file)

 		{

 			try

 			{

@@ -1821,7 +1808,7 @@ 
 		}

 		else

 		{

-			if($query_args)

+			if ($query_args)

 			{

 				try

 				{

@@ -1841,8 +1828,7 @@ 
 

 	$schema_file = NULL;

 	$schemas_set =

-		array

-			(

+		array(

 				'delete' => __DIR__ . '/xml_delete.xsd',

 				'update' => __DIR__ . '/xml_update.xsd',

 				'select' => __DIR__ . '/xml_select.xsd',

@@ -1855,12 +1841,12 @@ 
 

 	$i = 1;

 

-	if(pathinfo($argv[1], PATHINFO_EXTENSION) == 'xsd')

+	if (pathinfo($argv[1], PATHINFO_EXTENSION) == 'xsd')

 	{

 		$schema_file = $argv[$i++];

 	}

 	

-	for(; $i < count($argv); $i++)

+	for (; $i < count($argv); $i++)

 	{

 		try

 		{

@@ -1869,7 +1855,7 @@ 
 			$use_schema_language = $schema_language;

 

 			$retcode->push(ReturnCode::RETCODE_GENERIC_XML_SYNTAX);

-			if($domDocument->load($argv[$i]))

+			if ($domDocument->load($argv[$i]))

 			{

 				$retcode->pop();

 

@@ -1883,7 +1869,7 @@ 
 					}

 				}

 

-				if(!$schema_file && !$use_schema_language

+				if (!$schema_file && !$use_schema_language

 							&&

 						(

 							$queryElement->tagName != 'query'

@@ -1903,7 +1889,7 @@ 
 							);

 				}

 

-				if(!$schema_file && !$use_schema_language && !$skip_query_id

+				if (!$schema_file && !$use_schema_language && !$skip_query_id

 							&&

 						!validate_query_id($argv[$i], $queryElement->getAttribute('id')))

 				{

@@ -1916,24 +1902,24 @@ 
 						);

 				}

 

-				if($use_schema_language)

+				if ($use_schema_language)

 				{

 					$document_schema = $table_schema;

 				}

 				else

 				{

-					if(!$document_schema)

+					if (!$document_schema)

 					{

 						$document_schema = $schemas_set[$queryElement->getAttribute('action')];

 					}

 				}

 

 				$retcode->push(ReturnCode::RETCODE_XSD_VALIDATION);

-				if($domDocument->schemaValidate($document_schema))

+				if ($domDocument->schemaValidate($document_schema))

 				{

 					$retcode->pop();

 

-					if($use_schema_language)

+					if ($use_schema_language)

 					{

 						validate_schema_doc($argv[$i], $domDocument->documentElement);

 					}

@@ -1944,22 +1930,22 @@ 
 					$success = TRUE;

 				}

 

-				if(!$validate_only)

+				if (!$validate_only)

 				{

 					// Generate SQL with the db provider back-ends

 

-					if(function_exists('sys_get_temp_dir'))

+					if (function_exists('sys_get_temp_dir'))

 					{

 						$tmpdir = sys_get_temp_dir();

 					}

 					else

 					{

 						$tmpdir = getenv('TEMP');

-						if(!$tmpdir)

+						if (!$tmpdir)

 						{

 							$tmpdir = getenv('TMP');

 						}

-						if(!$tmpdir)

+						if (!$tmpdir)

 						{

 							$tmpdir = '/tmp';

 						}

@@ -1968,30 +1954,29 @@ 
 

 					global $_SERVER;

 

-					if(!is_array($_SERVER))

+					if (!is_array($_SERVER))

 					{

 						$_SERVER = array();

 					}

 

-					if(!array_key_exists('REMOTE_ADDR', $_SERVER))

+					if (!array_key_exists('REMOTE_ADDR', $_SERVER))

 					{

 						$_SERVER['REMOTE_ADDR'] = '127.0.0.1';

 					}

 

 					$set_db_info_methods =

-						array

-							(

+						array(

 								'mysql'		=> 'setMysqlDBInfo',

 								'mysqli'	=> 'setMysqliDBInfo',

 								'cubrid'	=> 'setCubridDBInfo',

 								'mssql'		=> 'setMssqlDBInfo'

 							);

 

-					foreach($set_db_info_methods as $db_type => $set_info_method)

+					foreach ($set_db_info_methods as $db_type => $set_info_method)

 					{

 						Context::$set_info_method(); // calls setMysqlDBInfo()/setCubridDBInfo()/...

 

-						if($use_schema_language)

+						if ($use_schema_language)

 						{

 							$GLOBALS['__DB__'][$db_type]->queries = '';

 							$GLOBALS['__DB__'][$db_type]->createTableByXmlFile($argv[$i]);

@@ -2010,26 +1995,25 @@ 
 							// copied from classes/db/DB.class.php

 							$oParser = new XmlQueryParser();

 							$args_array =

-								$oParser->parse_xml_query

-									(

+								$oParser->parse_xml_query(

 										pathinfo($argv[$i], PATHINFO_FILENAME), // query id

-										$argv[$i],								// xml file

+										$argv[$i], // xml file

 										$unlink_tmpfile->file_name				// cache file

 									);

 							$args_array = $args_array->queryTag->getArguments();

 

 							$GLOBALS['__DB__'][$db_type]->queries = '';

 							$k = 1;

-							foreach($args_array as $arg)

+							foreach ($args_array as $arg)

 							{

-								if(// why would there be a query arg without a var name ?

+								if (// why would there be a query arg without a var name ?

 										isset($arg->variable_name)

 											&&

 										!array_key_exists($arg->variable_name, $query_user_args))

 								{

-									if(isset($arg->argument_validator))

+									if (isset($arg->argument_validator))

 									{

-										if(FALSE	// some default values are to be parsed by php, some are not...

+										if (FALSE	// some default values are to be parsed by php, some are not...

 													&&

 												isset($arg->argument_validator->default_value)

 													&&

@@ -2040,9 +2024,9 @@ 
 										}

 										else

 										{

-											if($arg->argument_validator->filter)

+											if ($arg->argument_validator->filter)

 											{

-												switch($arg->argument_validator->filter)

+												switch ($arg->argument_validator->filter)

 												{

 													case 'email':

 													case 'email_address':

@@ -2081,18 +2065,17 @@ 
 										}

 									}

 

-									if(!array_key_exists($arg->variable_name, $query_user_args))

+									if (!array_key_exists($arg->variable_name, $query_user_args))

 									{

 										$query_user_args[$arg->variable_name] = sprintf('%06d', $k);

 									}

 

-									if(isset($arg->argument_validator))

+									if (isset($arg->argument_validator))

 									{

-										if(isset($arg->argument_validator->min_length))

+										if (isset($arg->argument_validator->min_length))

 										{

 											$query_user_args[$arg->variable_name] =

-												str_pad

-													(

+												str_pad(

 														$query_user_args[$arg->variable_name],

 														$arg->argument_validator->min_length,

 														isset($arg->argument_validator->filter) &&

@@ -2105,11 +2088,10 @@ 
 													);

 										}

 

-										if(isset($arg->argument_validator->max_length))

+										if (isset($arg->argument_validator->max_length))

 										{

 											$query_user_args[$arg->variable_name] =

-												substr

-													(

+												substr(

 														$query_user_args[$arg->variable_name],

 														0, 

 														$arg->argument_validator->max_length

@@ -2122,15 +2104,14 @@ 
 							}

 

 							$resultset = 

-								$GLOBALS['__DB__'][$db_type]->_executeQuery

-									(

-										$unlink_tmpfile->file_name,		// cache_file

-										(object)$query_user_args,		// source_args

-										basename($argv[$i]),			// query_id

+								$GLOBALS['__DB__'][$db_type]->_executeQuery(

+										$unlink_tmpfile->file_name, // cache_file

+										(object) $query_user_args, // source_args

+										basename($argv[$i]), // query_id

 										array()							// arg_columns

 									);

 

-							if(is_a($resultset, 'Object') && !$resultset->toBool())

+							if (is_a($resultset, 'Object') && !$resultset->toBool())

 							{

 								throw new XmlSchemaError($argv[$i], -1, 'mysql SQL query generation failed');

 							}

@@ -2147,7 +2128,7 @@ 
 				}

 			}

 

-			if(!$success)

+			if (!$success)

 			{

 				libXmlDisplayError($argv[$i], TRUE);

 			}

@@ -2160,7 +2141,7 @@ 
 		}

 		catch (ErrorMessage $exc)

 		{

-			if($retcode->code() == ReturnCode::RETCODE_SUCCESS)

+			if ($retcode->code() == ReturnCode::RETCODE_SUCCESS)

 			{

 				$retcode->code(ReturnCode::RETCODE_VALIDATOR_INTERNAL);

 			}

@@ -2170,7 +2151,7 @@ 
 		}

 		catch (ErrorException $exc)

 		{

-			if($retcode->code() == ReturnCode::RETCODE_SUCCESS)

+			if ($retcode->code() == ReturnCode::RETCODE_SUCCESS)

 			{

 				$retcode->code(ReturnCode::RETCODE_VALIDATOR_INTERNAL);

 			}

@@ -2194,21 +2175,21 @@ 
 catch (SyntaxError $syntax)

 {

 	fwrite(STDERR, $syntax->getMessage() . "\n");

-	exit(254);	// wrong command line

+	exit(254); // wrong command line

 				// 255 is reserved by php (for parse errors, etc.)

 }

 catch (ErrorMessage $exc)

 {

 	fwrite(STDERR, $exc->getMessage() . "\n");

 	libXmlDisplayError();

-	exit(ReturnCode::RETCODE_VALIDATOR_INTERNAL);		// internal validator error

+	exit(ReturnCode::RETCODE_VALIDATOR_INTERNAL); // internal validator error

 }

 catch (Exception $exc)

 {

 	fwrite(STDERR, $exc->getFile() . '(' . $exc->getLine() . ")\n\t" . $exc->getMessage() . "\n");

 	fwrite(STDERR, $exc->getTraceAsString());

 	libXmlDisplayError();

-	exit(ReturnCode::RETCODE_VALIDATOR_INTERNAL);		// internal validator error

+	exit(ReturnCode::RETCODE_VALIDATOR_INTERNAL); // internal validator error

 }

 

 /* End of file validate.php */

Braces +33 added lines, -66 removed lines

@@ -150,13 +150,11 @@ 
 		if($throw_error)

 		{

 			throw new ErrorMessage($msg);

-		}

-		else

+		} else

 		{

 			fwrite(STDERR, $msg . "\n");

 		}

-	}

-	else

+	} else

 	{

 		if($throw_error)

 		{

@@ -346,8 +344,7 @@ 
 					"\nANSI SQL-99 declares the table specification as required."

 				);

 		}

-	}

-	else

+	} else

 	{

 		// base table or view

 

@@ -369,13 +366,11 @@ 
 									$child_node->getLineNo(),

 									'Duplicate <conditions> elements.'

 								);

-						}

-						else

+						} else

 						{

 							$has_conditions_element = TRUE;

 						}

-					}

-					else

+					} else

 					{

 						throw

 							new XmlSchemaError(

@@ -396,8 +391,7 @@ 
 						'Expected <conditions> element as content.'

 					);

 			}

-		}

-		else

+		} else

 		{

 			foreach($table_element->childNodes as $child_node)

 			{

@@ -609,8 +603,7 @@ 
 			if($first_child)

 			{

 				$first_child = FALSE;

-			}

-			else

+			} else

 			{

 				if(!$child_node->hasAttribute('pipe'))

 				{

@@ -629,14 +622,12 @@ 
 			if($child_node->tagName == 'group')

 			{

 				checkConditionsGroup($xml_file, $child_node);

-			}

-			else

+			} else

 			{

 				if($child_node->tagName == 'query')

 				{

 					validate_select_query($xml_file, $child_node);

-				}

-				else

+				} else

 				{

 					if($child_node->tagName == 'condition')

 					{

@@ -851,8 +842,7 @@ 
 			try

 			{

 				$success = chdir($this->dirname);

-			}

-			catch (Exception $e)

+			} catch (Exception $e)

 			{

 				print "Failed to restore working dir {$this->dirname}.";

 			}

@@ -921,8 +911,7 @@ 
 		if($lowercase_name[$i] != $uppercase_name[$i])

 		{

 			$varing_case_filename .= "[{$lowercase_name[$i]}{$uppercase_name[$i]}]";

-		}

-		else

+		} else

 		{

 			$varing_case_filename .= $lowercase_name[$i];

 		}

@@ -1119,8 +1108,7 @@ 
 		if($val == -1)

 		{

 			return $this->exit_code;

-		}

-		else

+		} else

 		{

 			if($this->exit_code < $val)

 			{

@@ -1488,8 +1476,7 @@ 
 				if(self::$db_info)

 				{

 					return self::$db_info->master_db['db_type'];

-				}

-				else

+				} else

 				{

 					return NULL;

 				}

@@ -1588,8 +1575,7 @@ 
 							&&

 						array_key_exists($db_info->master_db['db_type'], $GLOBALS['__DB__']))

 				{

-				}

-				else

+				} else

 				{

 					$GLOBALS['__DB__'][$db_info->master_db['db_type']] =

 						new DBMysqlConnectWrapper();

@@ -1630,8 +1616,7 @@ 
 							&&

 						array_key_exists($db_info->master_db['db_type'], $GLOBALS['__DB__']))

 				{

-				}

-				else

+				} else

 				{

 					$GLOBALS['__DB__'][$db_info->master_db['db_type']] =

 						new DBMysqliConnectWrapper();

@@ -1672,8 +1657,7 @@ 
 							&&

 						array_key_exists($db_info->master_db['db_type'], $GLOBALS['__DB__']))

 				{

-				}

-				else

+				} else

 				{

 					$GLOBALS['__DB__'][$db_info->master_db['db_type']] =

 						new DBCubridConnectWrapper();

@@ -1714,8 +1698,7 @@ 
 							&&

 						array_key_exists($db_info->master_db['db_type'], $GLOBALS['__DB__']))

 				{

-				}

-				else

+				} else

 				{

 					$GLOBALS['__DB__'][$db_info->master_db['db_type']] =

 						new DBMssqlConnectWrapper();

@@ -1812,22 +1795,19 @@ 
 			try

 			{

 				$query_user_args = require($query_args_file);

-			}

-			catch (Exception $exc)

+			} catch (Exception $exc)

 			{

 				fwrite(STDERR, "Error in arguments file.\n");

 				throw $exc;

 			}

-		}

-		else

+		} else

 		{

 			if($query_args)

 			{

 				try

 				{

 					eval('$query_user_args = array(' . $query_args . ');');

-				}

-				catch (Exception $exc)

+				} catch (Exception $exc)

 				{

 					fwrite(STDERR, "Error in arguments string.\n");

 					throw $exc;

@@ -1919,8 +1899,7 @@ 
 				if($use_schema_language)

 				{

 					$document_schema = $table_schema;

-				}

-				else

+				} else

 				{

 					if(!$document_schema)

 					{

@@ -1936,8 +1915,7 @@ 
 					if($use_schema_language)

 					{

 						validate_schema_doc($argv[$i], $domDocument->documentElement);

-					}

-					else

+					} else

 					{

 						validate_xml_query($argv[$i], $domDocument->documentElement);

 					}

@@ -1951,8 +1929,7 @@ 
 					if(function_exists('sys_get_temp_dir'))

 					{

 						$tmpdir = sys_get_temp_dir();

-					}

-					else

+					} else

 					{

 						$tmpdir = getenv('TEMP');

 						if(!$tmpdir)

@@ -2001,8 +1978,7 @@ 
 							print " {$db_type} query:\n";

 							print $GLOBALS['__DB__'][$db_type]->queries;

 							print "\n";

-						}

-						else

+						} else

 						{

 							$unlink_tmpfile =

 								new UnlinkFile(tempnam($tmpdir, 'xe_'));

@@ -2037,8 +2013,7 @@ 
 										{

 											$query_user_args[$arg->variable_name] =

 												eval('return ' . $arg->argument_validator->default_value->toString() . ';');

-										}

-										else

+										} else

 										{

 											if($arg->argument_validator->filter)

 											{

@@ -2133,8 +2108,7 @@ 
 							if(is_a($resultset, 'Object') && !$resultset->toBool())

 							{

 								throw new XmlSchemaError($argv[$i], -1, 'mysql SQL query generation failed');

-							}

-							else

+							} else

 							{

 								print "\n";

 								print pathinfo($argv[$i], PATHINFO_FILENAME);

@@ -2151,14 +2125,12 @@ 
 			{

 				libXmlDisplayError($argv[$i], TRUE);

 			}

-		}

-		catch (XmlSchemaError $exc)

+		} catch (XmlSchemaError $exc)

 		{

 			$retcode->code(ReturnCode::RETCODE_BUILTIN_CHECKS);

 

 			fwrite(STDERR, $exc->getMessage() . "\n");

-		}

-		catch (ErrorMessage $exc)

+		} catch (ErrorMessage $exc)

 		{

 			if($retcode->code() == ReturnCode::RETCODE_SUCCESS)

 			{

@@ -2167,8 +2139,7 @@ 
 

 			fwrite(STDERR, $exc->getMessage() . "\n");

 			libXmlDisplayError($argv[$i]);

-		}

-		catch (ErrorException $exc)

+		} catch (ErrorException $exc)

 		{

 			if($retcode->code() == ReturnCode::RETCODE_SUCCESS)

 			{

@@ -2178,8 +2149,7 @@ 
 			fwrite(STDERR, "{$exc->getFile()}({$exc->getLine()}):\n\t{$exc->getMessage()}.\n");

 			fwrite(STDERR, $exc->getTraceAsString());

 			libXmlDisplayError($argv[$i]);

-		}

-		catch (Exception $exc)

+		} catch (Exception $exc)

 		{

 			$retcode->code(ReturnCode::RETCODE_VALIDATOR_INTERNAL);

 

@@ -2190,20 +2160,17 @@ 
 	}

 

 	exit($retcode->code());

-}

-catch (SyntaxError $syntax)

+} catch (SyntaxError $syntax)

 {

 	fwrite(STDERR, $syntax->getMessage() . "\n");

 	exit(254);	// wrong command line

 				// 255 is reserved by php (for parse errors, etc.)

-}

-catch (ErrorMessage $exc)

+} catch (ErrorMessage $exc)

 {

 	fwrite(STDERR, $exc->getMessage() . "\n");

 	libXmlDisplayError();

 	exit(ReturnCode::RETCODE_VALIDATOR_INTERNAL);		// internal validator error

-}

-catch (Exception $exc)

+} catch (Exception $exc)

 {

 	fwrite(STDERR, $exc->getFile() . '(' . $exc->getLine() . ")\n\t" . $exc->getMessage() . "\n");

 	fwrite(STDERR, $exc->getTraceAsString());

tools/minify/cssmin/CSSmin.php

Doc Comments +11 added lines, -2 removed lines

@@ -606,6 +606,11 @@ 
  * ---------------------------------------------------------------------------------------------
  */
 
+/**
+ * @param double $v1
+ * @param double $v2
+ * @param integer $vh
+ */
 private function hue_to_rgb($v1, $v2, $vh)
 {
 	$vh = $vh < 0 ? $vh + 1 : ($vh > 1 ? $vh - 1 : $vh);
@@ -620,6 +625,10 @@ 
 	return intval(floor(floatval($n) + 0.5), 10);
 }
 
+/**
+ * @param integer $min
+ * @param integer $max
+ */
 private function clamp_number($n, $min, $max)
 {
 	return min(max($n, $min), $max);
@@ -648,7 +657,7 @@ 
  *
  * @param string   $str
  * @param int      $from index
- * @param int|bool $to index (optional)
+ * @param integer $to index (optional)
  * @return string
  */
 private function substring($str, $from = 0, $to = FALSE)
@@ -680,7 +689,7 @@ 
  *
  * @param string   $str
  * @param int      $start index
- * @param int|bool $end index (optional)
+ * @param integer $end index (optional)
  * @return string
  */
 private function str_slice($str, $start = 0, $end = FALSE)

Spacing +15 added lines, -15 removed lines

@@ -47,7 +47,7 @@ 
 		$this->memory_limit = 128 * 1048576; // 128MB in bytes
 		$this->max_execution_time = 60; // 1 min
 		$this->pcre_backtrack_limit = 1000 * 1000;
-		$this->pcre_recursion_limit =  500 * 1000;
+		$this->pcre_recursion_limit = 500 * 1000;
 
 		$this->raise_php_limits = (bool) $raise_php_limits;
 	}
@@ -91,7 +91,7 @@ 
 	}
 
 	// preserve strings so their content doesn't get accidentally minified
-	$css = preg_replace_callback('/(?:"(?:[^\\\\"]|\\\\.|\\\\)*")|'."(?:'(?:[^\\\\']|\\\\.|\\\\)*')/S", array($this, 'replace_string'), $css);
+	$css = preg_replace_callback('/(?:"(?:[^\\\\"]|\\\\.|\\\\)*")|' . "(?:'(?:[^\\\\']|\\\\.|\\\\)*')/S", array($this, 'replace_string'), $css);
 
 	// Let's divide css code in chunks of 25.000 chars aprox.
 	// Reason: PHP's PCRE functions like preg_replace have a "backtrack limit"
@@ -229,8 +229,8 @@ 
 			$token_tring = self::TOKEN . (count($this->preserved_tokens) - 1) . '___';
 			$css = preg_replace($placeholder, $token_tring, $css, 1);
 			// Preserve new lines for /*! important comments
-			$css = preg_replace('/\s*[\n\r\f]+\s*(\/\*'. $token_tring .')/S', self::NL.'$1', $css);
-			$css = preg_replace('/('. $token_tring .'\*\/)\s*[\n\r\f]+\s*/S', '$1'.self::NL, $css);
+			$css = preg_replace('/\s*[\n\r\f]+\s*(\/\*' . $token_tring . ')/S', self::NL . '$1', $css);
+			$css = preg_replace('/(' . $token_tring . '\*\/)\s*[\n\r\f]+\s*/S', '$1' . self::NL, $css);
 			continue;
 		}
 
@@ -238,10 +238,10 @@ 
 		// shorten that to /*\*/ and the next one to /**/
 		if (substr($token, (strlen($token) - 1), 1) === '\\') {
 			$this->preserved_tokens[] = '\\';
-			$css = preg_replace($placeholder,  self::TOKEN . (count($this->preserved_tokens) - 1) . '___', $css, 1);
+			$css = preg_replace($placeholder, self::TOKEN . (count($this->preserved_tokens) - 1) . '___', $css, 1);
 			$i = $i + 1; // attn: advancing the loop
 			$this->preserved_tokens[] = '';
-			$css = preg_replace('/' . self::COMMENT . $i . '___/',  self::TOKEN . (count($this->preserved_tokens) - 1) . '___', $css, 1);
+			$css = preg_replace('/' . self::COMMENT . $i . '___/', self::TOKEN . (count($this->preserved_tokens) - 1) . '___', $css, 1);
 			continue;
 		}
 
@@ -252,7 +252,7 @@ 
 			if ($start_index > 2) {
 				if (substr($css, $start_index - 3, 1) === '>') {
 					$this->preserved_tokens[] = '';
-					$css = preg_replace($placeholder,  self::TOKEN . (count($this->preserved_tokens) - 1) . '___', $css, 1);
+					$css = preg_replace($placeholder, self::TOKEN . (count($this->preserved_tokens) - 1) . '___', $css, 1);
 				}
 			}
 		}
@@ -366,7 +366,7 @@ 
 					$css = preg_replace('/;;+/', ';', $css);
 
 					// Restore new lines for /*! important comments
-					$css = preg_replace('/'. self::NL .'/', "\n", $css);
+					$css = preg_replace('/' . self::NL . '/', "\n", $css);
 
 					// restore preserved comments and strings
 					for ($i = 0, $max = count($this->preserved_tokens); $i < $max; $i++) {
@@ -410,7 +410,7 @@ 
 			$terminator = ')';
 		}
 
-		while ($found_terminator === FALSE && $end_index+1 <= $max_index) {
+		while ($found_terminator === FALSE && $end_index + 1 <= $max_index) {
 			$end_index = $this->index_of($css, $terminator, $end_index + 1);
 
 			// endIndex == 0 doesn't really apply here
@@ -554,7 +554,7 @@ 
 			private function rgb_to_hex($matches)
 			{
 			// Support for percentage values rgb(100%, 0%, 45%);
-			if ($this->index_of($matches[1], '%') >= 0){
+			if ($this->index_of($matches[1], '%') >= 0) {
 				$rgbcolors = explode(',', str_replace('%', '', $matches[1]));
 				for ($i = 0; $i < count($rgbcolors); $i++) {
 					$rgbcolors[$i] = $this->round_number(floatval($rgbcolors[$i]) * 2.55);
@@ -570,7 +570,7 @@ 
 			}
 
 			// Fix for issue #2528093
-			if (!preg_match('/[\s\,\);\}]/', $matches[2])){
+			if (!preg_match('/[\s\,\);\}]/', $matches[2])) {
 				$matches[2] = ' ' . $matches[2];
 			}
 
@@ -594,12 +594,12 @@ 
 	} else {
 		$v2 = $l < 0.5 ? $l * (1 + $s) : ($l + $s) - ($s * $l);
 		$v1 = (2 * $l) - $v2;
-		$r = $this->round_number(255 * $this->hue_to_rgb($v1, $v2, $h + (1/3)));
+		$r = $this->round_number(255 * $this->hue_to_rgb($v1, $v2, $h + (1 / 3)));
 		$g = $this->round_number(255 * $this->hue_to_rgb($v1, $v2, $h));
-		$b = $this->round_number(255 * $this->hue_to_rgb($v1, $v2, $h - (1/3)));
+		$b = $this->round_number(255 * $this->hue_to_rgb($v1, $v2, $h - (1 / 3)));
 	}
 
-	return $this->rgb_to_hex(array('', $r.','.$g.','.$b, $matches[2]));
+	return $this->rgb_to_hex(array('', $r . ',' . $g . ',' . $b, $matches[2]));
 }
 
 /* HELPERS
@@ -611,7 +611,7 @@ 
 	$vh = $vh < 0 ? $vh + 1 : ($vh > 1 ? $vh - 1 : $vh);
 	if ($vh * 6 < 1) return $v1 + ($v2 - $v1) * 6 * $vh;
 	if ($vh * 2 < 1) return $v2;
-	if ($vh * 3 < 2) return $v1 + ($v2 - $v1) * ((2/3) - $vh) * 6;
+	if ($vh * 3 < 2) return $v1 + ($v2 - $v1) * ((2 / 3) - $vh) * 6;
 	return $v1;
 }
 

Braces +9 added lines, -3 removed lines

@@ -609,9 +609,15 @@
 private function hue_to_rgb($v1, $v2, $vh)
 {
 	$vh = $vh < 0 ? $vh + 1 : ($vh > 1 ? $vh - 1 : $vh);
-	if ($vh * 6 < 1) return $v1 + ($v2 - $v1) * 6 * $vh;
-	if ($vh * 2 < 1) return $v2;
-	if ($vh * 3 < 2) return $v1 + ($v2 - $v1) * ((2/3) - $vh) * 6;
+	if ($vh * 6 < 1) {
+		return $v1 + ($v2 - $v1) * 6 * $vh;
+	}
+	if ($vh * 2 < 1) {
+		return $v2;
+	}
+	if ($vh * 3 < 2) {
+		return $v1 + ($v2 - $v1) * ((2/3) - $vh) * 6;
+	}
 	return $v1;
 }
 

tools/minify/jsminplus/jsminplus.php

Doc Comments +35 added lines

@@ -114,6 +114,9 @@ 
 		$this->parser = new JSParser();
 	}
 
+	/**
+	 * @return string
+	 */
 	public static function minify($js, $filename='')
 	{
 		static $instance;
@@ -125,6 +128,9 @@ 
 		return $instance->min($js, $filename);
 	}
 
+	/**
+	 * @param string $filename
+	 */
 	private function min($js, $filename)
 	{
 		try
@@ -140,6 +146,9 @@ 
 		return false;
 	}
 
+	/**
+	 * @param JSNode $n
+	 */
 	private function parseTree($n, $noBlockGrouping = false)
 	{
 		$s = '';
@@ -545,6 +554,9 @@ 
 		return $s;
 	}
 
+	/**
+	 * @param string $string
+	 */
 	private function isValidIdentifier($string)
 	{
 		return preg_match('/^[a-zA-Z_][a-zA-Z0-9_]*$/', $string) && !in_array($string, $this->reserved);
@@ -608,6 +620,9 @@ 
 		$this->t = new JSTokenizer();
 	}
 
+	/**
+	 * @param integer $l
+	 */
 	public function parse($s, $f, $l)
 	{
 		// initialize tokenizer
@@ -621,6 +636,9 @@ 
 		return $n;
 	}
 
+	/**
+	 * @param JSCompilerContext $x
+	 */
 	private function Script($x)
 	{
 		$n = $this->Statements($x);
@@ -971,6 +989,10 @@ 
 		return $n;
 	}
 
+	/**
+	 * @param boolean $requireName
+	 * @param integer $functionForm
+	 */
 	private function FunctionDefinition($x, $requireName, $functionForm)
 	{
 		$f = new JSNode($this->t);
@@ -1437,6 +1459,10 @@ 
 	}
 
 	// Statement stack and nested statement handler.
+
+	/**
+	 * @param JSNode $node
+	 */
 	private function nest($x, $node, $end = false)
 	{
 		array_push($x->stmtStack, $node);
@@ -1501,6 +1527,9 @@ 
 	public $funDecls = array();
 	public $varDecls = array();
 
+	/**
+	 * @param boolean $inFunction
+	 */
 	public function __construct($inFunction)
 	{
 		$this->inFunction = $inFunction;
@@ -1519,6 +1548,9 @@ 
 	public $funDecls = array();
 	public $varDecls = array();
 
+	/**
+	 * @param JSTokenizer $t
+	 */
 	public function __construct($t, $type=0)
 	{
 		if ($token = $t->currentToken())
@@ -1673,6 +1705,9 @@ 
 		$this->scanOperand = true;
 	}
 
+	/**
+	 * @param integer|null $chunksize
+	 */
 	public function getInput($chunksize)
 	{
 		if ($chunksize)

Indentation +6 added lines, -6 removed lines

@@ -786,10 +786,10 @@ 
 			return $n;
 
 			case KEYWORD_WHILE:
-			        $n = new JSNode($this->t);
-			        $n->isLoop = true;
-			        $n->condition = $this->ParenExpression($x);
-			        $n->body = $this->nest($x, $n);
+					$n = new JSNode($this->t);
+					$n->isLoop = true;
+					$n->condition = $this->ParenExpression($x);
+					$n->body = $this->nest($x, $n);
 			return $n;
 
 			case KEYWORD_DO:
@@ -909,7 +909,7 @@ 
 
 			case KEYWORD_VAR:
 			case KEYWORD_CONST:
-			        $n = $this->Variables($x);
+					$n = $this->Variables($x);
 			break;
 
 			case TOKEN_CONDCOMMENT_START:
@@ -1693,7 +1693,7 @@ 
 
 	public function mustMatch($tt)
 	{
-	        if (!$this->match($tt))
+			if (!$this->match($tt))
 			throw $this->newSyntaxError('Unexpected token; token ' . $tt . ' expected');
 
 		return $this->currentToken();

Spacing +16 added lines, -16 removed lines

@@ -114,12 +114,12 @@ 
 		$this->parser = new JSParser();
 	}
 
-	public static function minify($js, $filename='')
+	public static function minify($js, $filename = '')
 	{
 		static $instance;
 
 		// this is a singleton
-		if(!$instance)
+		if (!$instance)
 			$instance = new JSMinPlus();
 
 		return $instance->min($js, $filename);
@@ -132,7 +132,7 @@ 
 			$n = $this->parser->parse($js, $filename, 1);
 			return $this->parseTree($n);
 		}
-		catch(Exception $e)
+		catch (Exception $e)
 		{
 			echo $e->getMessage() . "\n";
 		}
@@ -318,7 +318,7 @@ 
 					$t = $this->parseTree($n->value);
 					if (strlen($t))
 					{
-						if (	$t[0] != '(' && $t[0] != '[' && $t[0] != '{' &&
+						if ($t[0] != '(' && $t[0] != '[' && $t[0] != '{' &&
 							$t[0] != '"' && $t[0] != "'" && $t[0] != '/'
 						)
 							$s .= ' ';
@@ -462,7 +462,7 @@ 
 			case JS_INDEX:
 				$s = $this->parseTree($n->treeNodes[0]);
 				// See if we can replace named index with a dot saving 3 bytes
-				if (	$n->treeNodes[0]->type == TOKEN_IDENTIFIER &&
+				if ($n->treeNodes[0]->type == TOKEN_IDENTIFIER &&
 					$n->treeNodes[1]->type == TOKEN_STRING &&
 					$this->isValidIdentifier(substr($n->treeNodes[1]->value, 1, -1))
 				)
@@ -507,7 +507,7 @@ 
 					if ($t->type == JS_PROPERTY_INIT)
 					{
 						// Ditch the quotes when the index is a valid identifier
-						if (	$t->treeNodes[0]->type == TOKEN_STRING &&
+						if ($t->treeNodes[0]->type == TOKEN_STRING &&
 							$this->isValidIdentifier(substr($t->treeNodes[0]->value, 1, -1))
 						)
 							$s .= substr($t->treeNodes[0]->value, 1, -1);
@@ -1041,7 +1041,7 @@ 
 		return $n;
 	}
 
-	private function Expression($x, $stop=false)
+	private function Expression($x, $stop = false)
 	{
 		$operators = array();
 		$operands = array();
@@ -1076,7 +1076,7 @@ 
 					if ($this->t->scanOperand)
 						break 2;
 
-					while (	!empty($operators) &&
+					while (!empty($operators) &&
 						$this->opPrecedence[end($operators)->type] > $this->opPrecedence[$tt]
 					)
 						$this->reduce($operators, $operands);
@@ -1106,7 +1106,7 @@ 
 						break 2;
 
 					// Use >, not >=, for right-associative ASSIGN
-					while (	!empty($operators) &&
+					while (!empty($operators) &&
 						$this->opPrecedence[end($operators)->type] > $this->opPrecedence[$tt]
 					)
 						$this->reduce($operators, $operands);
@@ -1152,7 +1152,7 @@ 
 					if ($this->t->scanOperand)
 						break 2;
 
-					while (	!empty($operators) &&
+					while (!empty($operators) &&
 						$this->opPrecedence[end($operators)->type] >= $this->opPrecedence[$tt]
 					)
 						$this->reduce($operators, $operands);
@@ -1181,7 +1181,7 @@ 
 				case OP_INCREMENT: case OP_DECREMENT:
 					if ($this->t->scanOperand)
 					{
-						array_push($operators, new JSNode($this->t));  // prefix increment or decrement
+						array_push($operators, new JSNode($this->t)); // prefix increment or decrement
 					}
 					else
 					{
@@ -1338,7 +1338,7 @@ 
 					}
 					else
 					{
-						while (	!empty($operators) &&
+						while (!empty($operators) &&
 							$this->opPrecedence[end($operators)->type] > $this->opPrecedence[KEYWORD_NEW]
 						)
 							$this->reduce($operators, $operands);
@@ -1519,7 +1519,7 @@ 
 	public $funDecls = array();
 	public $varDecls = array();
 
-	public function __construct($t, $type=0)
+	public function __construct($t, $type = 0)
 	{
 		if ($token = $t->currentToken())
 		{
@@ -1537,7 +1537,7 @@ 
 
 		if (($numargs = func_num_args()) > 2)
 		{
-			$args = func_get_args();;
+			$args = func_get_args(); ;
 			for ($i = 2; $i < $numargs; $i++)
 				$this->addNode($args[$i]);
 		}
@@ -1735,7 +1735,7 @@ 
 
 	public function get($chunksize = 1000)
 	{
-		while($this->lookahead)
+		while ($this->lookahead)
 		{
 			$this->lookahead--;
 			$this->tokenIndex = ($this->tokenIndex + 1) & 3;
@@ -1747,7 +1747,7 @@ 
 		$conditional_comment = false;
 
 		// strip whitespace and comments
-		while(true)
+		while (true)
 		{
 			$input = $this->getInput($chunksize);
 

Braces +350 added lines, -274 removed lines

@@ -119,8 +119,9 @@ 
 		static $instance;
 
 		// this is a singleton
-		if(!$instance)
-			$instance = new JSMinPlus();
+		if(!$instance) {
+					$instance = new JSMinPlus();
+		}
 
 		return $instance->min($js, $filename);
 	}
@@ -131,8 +132,7 @@ 
 		{
 			$n = $this->parser->parse($js, $filename, 1);
 			return $this->parseTree($n);
-		}
-		catch(Exception $e)
+		} catch(Exception $e)
 		{
 			echo $e->getMessage() . "\n";
 		}
@@ -149,8 +149,9 @@ 
 			case KEYWORD_FUNCTION:
 				$s .= 'function' . ($n->name ? ' ' . $n->name : '') . '(';
 				$params = $n->params;
-				for ($i = 0, $j = count($params); $i < $j; $i++)
-					$s .= ($i ? ',' : '') . $params[$i];
+				for ($i = 0, $j = count($params); $i < $j; $i++) {
+									$s .= ($i ? ',' : '') . $params[$i];
+				}
 				$s .= '){' . $this->parseTree($n->body, true) . '}';
 			break;
 
@@ -176,13 +177,11 @@ 
 							{
 								// put declared functions on a new line
 								$s .= "\n";
-							}
-							elseif ($type == KEYWORD_VAR && $type == $lastType)
+							} elseif ($type == KEYWORD_VAR && $type == $lastType)
 							{
 								// mutiple var-statements can go into one
 								$t = ',' . substr($t, 4);
-							}
-							else
+							} else
 							{
 								// add terminator
 								$s .= ';';
@@ -208,24 +207,26 @@ 
 				$elsePart = $n->elsePart ? $this->parseTree($n->elsePart) : null;
 
 				// empty if-statement
-				if ($thenPart == '')
-					$thenPart = ';';
+				if ($thenPart == '') {
+									$thenPart = ';';
+				}
 
 				if ($elsePart)
 				{
 					// be carefull and always make a block out of the thenPart; could be more optimized but is a lot of trouble
-					if ($thenPart != ';' && $thenPart[0] != '{')
-						$thenPart = '{' . $thenPart . '}';
+					if ($thenPart != ';' && $thenPart[0] != '{') {
+											$thenPart = '{' . $thenPart . '}';
+					}
 
 					$s .= $thenPart . 'else';
 
 					// we could check for more, but that hardly ever applies so go for performance
-					if ($elsePart[0] != '{')
-						$s .= ' ';
+					if ($elsePart[0] != '{') {
+											$s .= ' ';
+					}
 
 					$s .= $elsePart;
-				}
-				else
+				} else
 				{
 					$s .= $thenPart;
 				}
@@ -237,18 +238,20 @@ 
 				for ($i = 0, $j = count($cases); $i < $j; $i++)
 				{
 					$case = $cases[$i];
-					if ($case->type == KEYWORD_CASE)
-						$s .= 'case' . ($case->caseLabel->type != TOKEN_STRING ? ' ' : '') . $this->parseTree($case->caseLabel) . ':';
-					else
-						$s .= 'default:';
+					if ($case->type == KEYWORD_CASE) {
+											$s .= 'case' . ($case->caseLabel->type != TOKEN_STRING ? ' ' : '') . $this->parseTree($case->caseLabel) . ':';
+					} else {
+											$s .= 'default:';
+					}
 
 					$statement = $this->parseTree($case->statements, true);
 					if ($statement)
 					{
 						$s .= $statement;
 						// no terminator for last statement
-						if ($i + 1 < $j)
-							$s .= ';';
+						if ($i + 1 < $j) {
+													$s .= ';';
+						}
 					}
 				}
 				$s .= '}';
@@ -260,8 +263,9 @@ 
 					. ';' . ($n->update ? $this->parseTree($n->update) : '') . ')';
 
 				$body  = $this->parseTree($n->body);
-				if ($body == '')
-					$body = ';';
+				if ($body == '') {
+									$body = ';';
+				}
 
 				$s .= $body;
 			break;
@@ -270,8 +274,9 @@ 
 				$s = 'while(' . $this->parseTree($n->condition) . ')';
 
 				$body  = $this->parseTree($n->body);
-				if ($body == '')
-					$body = ';';
+				if ($body == '') {
+									$body = ';';
+				}
 
 				$s .= $body;
 			break;
@@ -280,8 +285,9 @@ 
 				$s = 'for(' . ($n->varDecl ? $this->parseTree($n->varDecl) : $this->parseTree($n->iterator)) . ' in ' . $this->parseTree($n->object) . ')';
 
 				$body  = $this->parseTree($n->body);
-				if ($body == '')
-					$body = ';';
+				if ($body == '') {
+									$body = ';';
+				}
 
 				$s .= $body;
 			break;
@@ -303,8 +309,9 @@ 
 					$t = $catchClauses[$i];
 					$s .= 'catch(' . $t->varName . ($t->guard ? ' if ' . $this->parseTree($t->guard) : '') . '){' . $this->parseTree($t->block, true) . '}';
 				}
-				if ($n->finallyBlock)
-					$s .= 'finally{' . $this->parseTree($n->finallyBlock, true) . '}';
+				if ($n->finallyBlock) {
+									$s .= 'finally{' . $this->parseTree($n->finallyBlock, true) . '}';
+				}
 			break;
 
 			case KEYWORD_THROW:
@@ -320,8 +327,9 @@ 
 					{
 						if (	$t[0] != '(' && $t[0] != '[' && $t[0] != '{' &&
 							$t[0] != '"' && $t[0] != "'" && $t[0] != '/'
-						)
-							$s .= ' ';
+						) {
+													$s .= ' ';
+						}
 
 						$s .= $t;
 					}
@@ -341,8 +349,9 @@ 
 					$t = $childs[$i];
 					$s .= ($i ? ',' : '') . $t->name;
 					$u = $t->initializer;
-					if ($u)
-						$s .= '=' . $this->parseTree($u);
+					if ($u) {
+											$s .= '=' . $this->parseTree($u);
+					}
 				}
 			break;
 
@@ -354,13 +363,15 @@ 
 			case TOKEN_CONDCOMMENT_END:
 				$s = $n->value . ($n->type == TOKEN_CONDCOMMENT_START ? ' ' : '');
 				$childs = $n->treeNodes;
-				for ($i = 0, $j = count($childs); $i < $j; $i++)
-					$s .= $this->parseTree($childs[$i]);
+				for ($i = 0, $j = count($childs); $i < $j; $i++) {
+									$s .= $this->parseTree($childs[$i]);
+				}
 			break;
 
 			case OP_SEMICOLON:
-				if ($expression = $n->expression)
-					$s = $this->parseTree($expression);
+				if ($expression = $n->expression) {
+									$s = $this->parseTree($expression);
+				}
 			break;
 
 			case JS_LABEL:
@@ -369,8 +380,9 @@ 
 
 			case OP_COMMA:
 				$childs = $n->treeNodes;
-				for ($i = 0, $j = count($childs); $i < $j; $i++)
-					$s .= ($i ? ',' : '') . $this->parseTree($childs[$i]);
+				for ($i = 0, $j = count($childs); $i < $j; $i++) {
+									$s .= ($i ? ',' : '') . $this->parseTree($childs[$i]);
+				}
 			break;
 
 			case OP_ASSIGN:
@@ -449,10 +461,11 @@ 
 
 			case OP_INCREMENT:
 			case OP_DECREMENT:
-				if ($n->postfix)
-					$s = $this->parseTree($n->treeNodes[0]) . $n->value;
-				else
-					$s = $n->value . $this->parseTree($n->treeNodes[0]);
+				if ($n->postfix) {
+									$s = $this->parseTree($n->treeNodes[0]) . $n->value;
+				} else {
+									$s = $n->value . $this->parseTree($n->treeNodes[0]);
+				}
 			break;
 
 			case OP_DOT:
@@ -465,16 +478,18 @@ 
 				if (	$n->treeNodes[0]->type == TOKEN_IDENTIFIER &&
 					$n->treeNodes[1]->type == TOKEN_STRING &&
 					$this->isValidIdentifier(substr($n->treeNodes[1]->value, 1, -1))
-				)
-					$s .= '.' . substr($n->treeNodes[1]->value, 1, -1);
-				else
-					$s .= '[' . $this->parseTree($n->treeNodes[1]) . ']';
+				) {
+									$s .= '.' . substr($n->treeNodes[1]->value, 1, -1);
+				} else {
+									$s .= '[' . $this->parseTree($n->treeNodes[1]) . ']';
+				}
 			break;
 
 			case JS_LIST:
 				$childs = $n->treeNodes;
-				for ($i = 0, $j = count($childs); $i < $j; $i++)
-					$s .= ($i ? ',' : '') . $this->parseTree($childs[$i]);
+				for ($i = 0, $j = count($childs); $i < $j; $i++) {
+									$s .= ($i ? ',' : '') . $this->parseTree($childs[$i]);
+				}
 			break;
 
 			case JS_CALL:
@@ -502,27 +517,29 @@ 
 				for ($i = 0, $j = count($childs); $i < $j; $i++)
 				{
 					$t = $childs[$i];
-					if ($i)
-						$s .= ',';
+					if ($i) {
+											$s .= ',';
+					}
 					if ($t->type == JS_PROPERTY_INIT)
 					{
 						// Ditch the quotes when the index is a valid identifier
 						if (	$t->treeNodes[0]->type == TOKEN_STRING &&
 							$this->isValidIdentifier(substr($t->treeNodes[0]->value, 1, -1))
-						)
-							$s .= substr($t->treeNodes[0]->value, 1, -1);
-						else
-							$s .= $t->treeNodes[0]->value;
+						) {
+													$s .= substr($t->treeNodes[0]->value, 1, -1);
+						} else {
+													$s .= $t->treeNodes[0]->value;
+						}
 
 						$s .= ':' . $this->parseTree($t->treeNodes[1]);
-					}
-					else
+					} else
 					{
 						$s .= $t->type == JS_GETTER ? 'get' : 'set';
 						$s .= ' ' . $t->name . '(';
 						$params = $t->params;
-						for ($i = 0, $j = count($params); $i < $j; $i++)
-							$s .= ($i ? ',' : '') . $params[$i];
+						for ($i = 0, $j = count($params); $i < $j; $i++) {
+													$s .= ($i ? ',' : '') . $params[$i];
+						}
 						$s .= '){' . $this->parseTree($t->body, true) . '}';
 					}
 				}
@@ -615,8 +632,9 @@ 
 
 		$x = new JSCompilerContext(false);
 		$n = $this->Script($x);
-		if (!$this->t->isDone())
-			throw $this->t->newSyntaxError('Syntax error');
+		if (!$this->t->isDone()) {
+					throw $this->t->newSyntaxError('Syntax error');
+		}
 
 		return $n;
 	}
@@ -636,8 +654,9 @@ 
 		$n = new JSNode($this->t, JS_BLOCK);
 		array_push($x->stmtStack, $n);
 
-		while (!$this->t->isDone() && $this->t->peek() != OP_RIGHT_CURLY)
-			$n->addNode($this->Statement($x));
+		while (!$this->t->isDone() && $this->t->peek() != OP_RIGHT_CURLY) {
+					$n->addNode($this->Statement($x));
+		}
 
 		array_pop($x->stmtStack);
 
@@ -701,15 +720,17 @@ 
 					switch ($tt)
 					{
 						case KEYWORD_DEFAULT:
-							if ($n->defaultIndex >= 0)
-								throw $this->t->newSyntaxError('More than one switch default');
+							if ($n->defaultIndex >= 0) {
+															throw $this->t->newSyntaxError('More than one switch default');
+							}
 							// FALL THROUGH
 						case KEYWORD_CASE:
 							$n2 = new JSNode($this->t);
-							if ($tt == KEYWORD_DEFAULT)
-								$n->defaultIndex = count($n->cases);
-							else
-								$n2->caseLabel = $this->Expression($x, OP_COLON);
+							if ($tt == KEYWORD_DEFAULT) {
+															$n->defaultIndex = count($n->cases);
+							} else {
+															$n2->caseLabel = $this->Expression($x, OP_COLON);
+							}
 								break;
 						default:
 							throw $this->t->newSyntaxError('Invalid switch case');
@@ -717,8 +738,9 @@ 
 
 					$this->t->mustMatch(OP_COLON);
 					$n2->statements = new JSNode($this->t, JS_BLOCK);
-					while (($tt = $this->t->peek()) != KEYWORD_CASE && $tt != KEYWORD_DEFAULT && $tt != OP_RIGHT_CURLY)
-						$n2->statements->addNode($this->Statement($x));
+					while (($tt = $this->t->peek()) != KEYWORD_CASE && $tt != KEYWORD_DEFAULT && $tt != OP_RIGHT_CURLY) {
+											$n2->statements->addNode($this->Statement($x));
+					}
 
 					array_push($n->cases, $n2);
 				}
@@ -738,8 +760,7 @@ 
 					{
 						$this->t->get();
 						$n2 = $this->Variables($x);
-					}
-					else
+					} else
 					{
 						$n2 = $this->Expression($x);
 					}
@@ -763,16 +784,14 @@ 
 						// NB: n2[0].type == IDENTIFIER and n2[0].value == n2[0].name.
 						$n->iterator = $n2->treeNodes[0];
 						$n->varDecl = $n2;
-					}
-					else
+					} else
 					{
 						$n->iterator = $n2;
 						$n->varDecl = null;
 					}
 
 					$n->object = $this->Expression($x);
-				}
-				else
+				} else
 				{
 					$n->setup = $n2 ? $n2 : null;
 					$this->t->mustMatch(OP_SEMICOLON);
@@ -824,17 +843,18 @@ 
 				{
 					do
 					{
-						if (--$i < 0)
-							throw $this->t->newSyntaxError('Label not found');
+						if (--$i < 0) {
+													throw $this->t->newSyntaxError('Label not found');
+						}
 					}
 					while ($ss[$i]->label != $label);
-				}
-				else
+				} else
 				{
 					do
 					{
-						if (--$i < 0)
-							throw $this->t->newSyntaxError('Invalid ' . $tt);
+						if (--$i < 0) {
+													throw $this->t->newSyntaxError('Invalid ' . $tt);
+						}
 					}
 					while (!$ss[$i]->isLoop && ($tt != KEYWORD_BREAK || $ss[$i]->type != KEYWORD_SWITCH));
 				}
@@ -855,15 +875,16 @@ 
 
 					if ($this->t->match(KEYWORD_IF))
 					{
-						if ($x->ecmaStrictMode)
-							throw $this->t->newSyntaxError('Illegal catch guard');
+						if ($x->ecmaStrictMode) {
+													throw $this->t->newSyntaxError('Illegal catch guard');
+						}
 
-						if (count($n->catchClauses) && !end($n->catchClauses)->guard)
-							throw $this->t->newSyntaxError('Guarded catch after unguarded');
+						if (count($n->catchClauses) && !end($n->catchClauses)->guard) {
+													throw $this->t->newSyntaxError('Guarded catch after unguarded');
+						}
 
 						$n2->guard = $this->Expression($x);
-					}
-					else
+					} else
 					{
 						$n2->guard = null;
 					}
@@ -873,11 +894,13 @@ 
 					array_push($n->catchClauses, $n2);
 				}
 
-				if ($this->t->match(KEYWORD_FINALLY))
-					$n->finallyBlock = $this->Block($x);
+				if ($this->t->match(KEYWORD_FINALLY)) {
+									$n->finallyBlock = $this->Block($x);
+				}
 
-				if (!count($n->catchClauses) && !$n->finallyBlock)
-					throw $this->t->newSyntaxError('Invalid try statement');
+				if (!count($n->catchClauses) && !$n->finallyBlock) {
+									throw $this->t->newSyntaxError('Invalid try statement');
+				}
 			return $n;
 
 			case KEYWORD_CATCH:
@@ -890,15 +913,17 @@ 
 			break;
 
 			case KEYWORD_RETURN:
-				if (!$x->inFunction)
-					throw $this->t->newSyntaxError('Invalid return');
+				if (!$x->inFunction) {
+									throw $this->t->newSyntaxError('Invalid return');
+				}
 
 				$n = new JSNode($this->t);
 				$tt = $this->t->peekOnSameLine();
-				if ($tt != TOKEN_END && $tt != TOKEN_NEWLINE && $tt != OP_SEMICOLON && $tt != OP_RIGHT_CURLY)
-					$n->value = $this->Expression($x);
-				else
-					$n->value = null;
+				if ($tt != TOKEN_END && $tt != TOKEN_NEWLINE && $tt != OP_SEMICOLON && $tt != OP_RIGHT_CURLY) {
+									$n->value = $this->Expression($x);
+				} else {
+									$n->value = null;
+				}
 			break;
 
 			case KEYWORD_WITH:
@@ -939,8 +964,9 @@ 
 						$ss = $x->stmtStack;
 						for ($i = count($ss) - 1; $i >= 0; --$i)
 						{
-							if ($ss[$i]->label == $label)
-								throw $this->t->newSyntaxError('Duplicate label');
+							if ($ss[$i]->label == $label) {
+															throw $this->t->newSyntaxError('Duplicate label');
+							}
 						}
 
 						$this->t->get();
@@ -962,8 +988,9 @@ 
 		if ($this->t->lineno == $this->t->currentToken()->lineno)
 		{
 			$tt = $this->t->peekOnSameLine();
-			if ($tt != TOKEN_END && $tt != TOKEN_NEWLINE && $tt != OP_SEMICOLON && $tt != OP_RIGHT_CURLY)
-				throw $this->t->newSyntaxError('Missing ; before statement');
+			if ($tt != TOKEN_END && $tt != TOKEN_NEWLINE && $tt != OP_SEMICOLON && $tt != OP_RIGHT_CURLY) {
+							throw $this->t->newSyntaxError('Missing ; before statement');
+			}
 		}
 
 		$this->t->match(OP_SEMICOLON);
@@ -975,26 +1002,30 @@ 
 	{
 		$f = new JSNode($this->t);
 
-		if ($f->type != KEYWORD_FUNCTION)
-			$f->type = ($f->value == 'get') ? JS_GETTER : JS_SETTER;
+		if ($f->type != KEYWORD_FUNCTION) {
+					$f->type = ($f->value == 'get') ? JS_GETTER : JS_SETTER;
+		}
 
-		if ($this->t->match(TOKEN_IDENTIFIER))
-			$f->name = $this->t->currentToken()->value;
-		elseif ($requireName)
-			throw $this->t->newSyntaxError('Missing function identifier');
+		if ($this->t->match(TOKEN_IDENTIFIER)) {
+					$f->name = $this->t->currentToken()->value;
+		} elseif ($requireName) {
+					throw $this->t->newSyntaxError('Missing function identifier');
+		}
 
 		$this->t->mustMatch(OP_LEFT_PAREN);
 			$f->params = array();
 
 		while (($tt = $this->t->get()) != OP_RIGHT_PAREN)
 		{
-			if ($tt != TOKEN_IDENTIFIER)
-				throw $this->t->newSyntaxError('Missing formal parameter');
+			if ($tt != TOKEN_IDENTIFIER) {
+							throw $this->t->newSyntaxError('Missing formal parameter');
+			}
 
 			array_push($f->params, $this->t->currentToken()->value);
 
-			if ($this->t->peek() != OP_RIGHT_PAREN)
-				$this->t->mustMatch(OP_COMMA);
+			if ($this->t->peek() != OP_RIGHT_PAREN) {
+							$this->t->mustMatch(OP_COMMA);
+			}
 		}
 
 		$this->t->mustMatch(OP_LEFT_CURLY);
@@ -1006,8 +1037,9 @@ 
 		$f->end = $this->t->currentToken()->end;
 
 		$f->functionForm = $functionForm;
-		if ($functionForm == DECLARED_FORM)
-			array_push($x->funDecls, $f);
+		if ($functionForm == DECLARED_FORM) {
+					array_push($x->funDecls, $f);
+		}
 
 		return $f;
 	}
@@ -1025,8 +1057,9 @@ 
 
 			if ($this->t->match(OP_ASSIGN))
 			{
-				if ($this->t->currentToken()->assignOp)
-					throw $this->t->newSyntaxError('Invalid variable initialization');
+				if ($this->t->currentToken()->assignOp) {
+									throw $this->t->newSyntaxError('Invalid variable initialization');
+				}
 
 				$n2->initializer = $this->Expression($x, OP_COMMA);
 			}
@@ -1073,13 +1106,15 @@ 
 					break 2;
 
 				case OP_HOOK:
-					if ($this->t->scanOperand)
-						break 2;
+					if ($this->t->scanOperand) {
+											break 2;
+					}
 
 					while (	!empty($operators) &&
 						$this->opPrecedence[end($operators)->type] > $this->opPrecedence[$tt]
-					)
-						$this->reduce($operators, $operands);
+					) {
+											$this->reduce($operators, $operands);
+					}
 
 					array_push($operators, new JSNode($this->t));
 
@@ -1087,29 +1122,33 @@ 
 					$this->t->scanOperand = true;
 					$n = $this->Expression($x);
 
-					if (!$this->t->match(OP_COLON))
-						break 2;
+					if (!$this->t->match(OP_COLON)) {
+											break 2;
+					}
 
 					--$x->hookLevel;
 					array_push($operands, $n);
 				break;
 
 				case OP_COLON:
-					if ($x->hookLevel)
-						break 2;
+					if ($x->hookLevel) {
+											break 2;
+					}
 
 					throw $this->t->newSyntaxError('Invalid label');
 				break;
 
 				case OP_ASSIGN:
-					if ($this->t->scanOperand)
-						break 2;
+					if ($this->t->scanOperand) {
+											break 2;
+					}
 
 					// Use >, not >=, for right-associative ASSIGN
 					while (	!empty($operators) &&
 						$this->opPrecedence[end($operators)->type] > $this->opPrecedence[$tt]
-					)
-						$this->reduce($operators, $operands);
+					) {
+											$this->reduce($operators, $operands);
+					}
 
 					array_push($operators, new JSNode($this->t));
 					end($operands)->assignOp = $this->t->currentToken()->assignOp;
@@ -1123,8 +1162,9 @@ 
 					if ($x->inForLoopInit && !$x->hookLevel &&
 						!$x->bracketLevel && !$x->curlyLevel &&
 						!$x->parenLevel
-					)
-						break 2;
+					) {
+											break 2;
+					}
 				// FALL THROUGH
 				case OP_COMMA:
 					// A comma operator should not be parsed if we're parsing the then part
@@ -1132,8 +1172,9 @@ 
 					if ($tt == OP_COMMA && $x->hookLevel &&
 						!$x->bracketLevel && !$x->curlyLevel &&
 						!$x->parenLevel
-					)
-						break 2;
+					) {
+											break 2;
+					}
 				// Treat comma as left-associative so reduce can fold left-heavy
 				// COMMA trees into a single array.
 				// FALL THROUGH
@@ -1149,20 +1190,21 @@ 
 				case OP_PLUS: case OP_MINUS:
 				case OP_MUL: case OP_DIV: case OP_MOD:
 				case OP_DOT:
-					if ($this->t->scanOperand)
-						break 2;
+					if ($this->t->scanOperand) {
+											break 2;
+					}
 
 					while (	!empty($operators) &&
 						$this->opPrecedence[end($operators)->type] >= $this->opPrecedence[$tt]
-					)
-						$this->reduce($operators, $operands);
+					) {
+											$this->reduce($operators, $operands);
+					}
 
 					if ($tt == OP_DOT)
 					{
 						$this->t->mustMatch(TOKEN_IDENTIFIER);
 						array_push($operands, new JSNode($this->t, OP_DOT, array_pop($operands), new JSNode($this->t)));
-					}
-					else
+					} else
 					{
 						array_push($operators, new JSNode($this->t));
 						$this->t->scanOperand = true;
@@ -1172,8 +1214,9 @@ 
 				case KEYWORD_DELETE: case KEYWORD_VOID: case KEYWORD_TYPEOF:
 				case OP_NOT: case OP_BITWISE_NOT: case OP_UNARY_PLUS: case OP_UNARY_MINUS:
 				case KEYWORD_NEW:
-					if (!$this->t->scanOperand)
-						break 2;
+					if (!$this->t->scanOperand) {
+											break 2;
+					}
 
 					array_push($operators, new JSNode($this->t));
 				break;
@@ -1182,19 +1225,20 @@ 
 					if ($this->t->scanOperand)
 					{
 						array_push($operators, new JSNode($this->t));  // prefix increment or decrement
-					}
-					else
+					} else
 					{
 						// Don't cross a line boundary for postfix {in,de}crement.
 						$t = $this->t->tokens[($this->t->tokenIndex + $this->t->lookahead - 1) & 3];
-						if ($t && $t->lineno != $this->t->lineno)
-							break 2;
+						if ($t && $t->lineno != $this->t->lineno) {
+													break 2;
+						}
 
 						if (!empty($operators))
 						{
 							// Use >, not >=, so postfix has higher precedence than prefix.
-							while ($this->opPrecedence[end($operators)->type] > $this->opPrecedence[$tt])
-								$this->reduce($operators, $operands);
+							while ($this->opPrecedence[end($operators)->type] > $this->opPrecedence[$tt]) {
+															$this->reduce($operators, $operands);
+							}
 						}
 
 						$n = new JSNode($this->t, $tt, array_pop($operands));
@@ -1204,8 +1248,9 @@ 
 				break;
 
 				case KEYWORD_FUNCTION:
-					if (!$this->t->scanOperand)
-						break 2;
+					if (!$this->t->scanOperand) {
+											break 2;
+					}
 
 					array_push($operands, $this->FunctionDefinition($x, false, EXPRESSED_FORM));
 					$this->t->scanOperand = false;
@@ -1213,8 +1258,9 @@ 
 
 				case KEYWORD_NULL: case KEYWORD_THIS: case KEYWORD_TRUE: case KEYWORD_FALSE:
 				case TOKEN_IDENTIFIER: case TOKEN_NUMBER: case TOKEN_STRING: case TOKEN_REGEXP:
-					if (!$this->t->scanOperand)
-						break 2;
+					if (!$this->t->scanOperand) {
+											break 2;
+					}
 
 					array_push($operands, new JSNode($this->t));
 					$this->t->scanOperand = false;
@@ -1222,10 +1268,11 @@ 
 
 				case TOKEN_CONDCOMMENT_START:
 				case TOKEN_CONDCOMMENT_END:
-					if ($this->t->scanOperand)
-						array_push($operators, new JSNode($this->t));
-					else
-						array_push($operands, new JSNode($this->t));
+					if ($this->t->scanOperand) {
+											array_push($operators, new JSNode($this->t));
+					} else {
+											array_push($operands, new JSNode($this->t));
+					}
 				break;
 
 				case OP_LEFT_BRACKET:
@@ -1244,15 +1291,15 @@ 
 							}
 
 							$n->addNode($this->Expression($x, OP_COMMA));
-							if (!$this->t->match(OP_COMMA))
-								break;
+							if (!$this->t->match(OP_COMMA)) {
+															break;
+							}
 						}
 
 						$this->t->mustMatch(OP_RIGHT_BRACKET);
 						array_push($operands, $n);
 						$this->t->scanOperand = false;
-					}
-					else
+					} else
 					{
 						// Property indexing operator.
 						array_push($operators, new JSNode($this->t, JS_INDEX));
@@ -1262,18 +1309,21 @@ 
 				break;
 
 				case OP_RIGHT_BRACKET:
-					if ($this->t->scanOperand || $x->bracketLevel == $bl)
-						break 2;
+					if ($this->t->scanOperand || $x->bracketLevel == $bl) {
+											break 2;
+					}
 
-					while ($this->reduce($operators, $operands)->type != JS_INDEX)
-						continue;
+					while ($this->reduce($operators, $operands)->type != JS_INDEX) {
+											continue;
+					}
 
 					--$x->bracketLevel;
 				break;
 
 				case OP_LEFT_CURLY:
-					if (!$this->t->scanOperand)
-						break 2;
+					if (!$this->t->scanOperand) {
+											break 2;
+					}
 
 					// Object initialiser.  As for array initialisers (see above),
 					// parse using recursive descent.
@@ -1287,12 +1337,12 @@ 
 							$tv = $this->t->currentToken()->value;
 							if (($tv == 'get' || $tv == 'set') && $this->t->peek() == TOKEN_IDENTIFIER)
 							{
-								if ($x->ecmaStrictMode)
-									throw $this->t->newSyntaxError('Illegal property accessor');
+								if ($x->ecmaStrictMode) {
+																	throw $this->t->newSyntaxError('Illegal property accessor');
+								}
 
 								$n->addNode($this->FunctionDefinition($x, true, EXPRESSED_FORM));
-							}
-							else
+							} else
 							{
 								switch ($tt)
 								{
@@ -1303,8 +1353,9 @@ 
 									break;
 
 									case OP_RIGHT_CURLY:
-										if ($x->ecmaStrictMode)
-											throw $this->t->newSyntaxError('Illegal trailing ,');
+										if ($x->ecmaStrictMode) {
+																					throw $this->t->newSyntaxError('Illegal trailing ,');
+										}
 									break 3;
 
 									default:
@@ -1327,21 +1378,22 @@ 
 				break;
 
 				case OP_RIGHT_CURLY:
-					if (!$this->t->scanOperand && $x->curlyLevel != $cl)
-						throw new Exception('PANIC: right curly botch');
+					if (!$this->t->scanOperand && $x->curlyLevel != $cl) {
+											throw new Exception('PANIC: right curly botch');
+					}
 				break 2;
 
 				case OP_LEFT_PAREN:
 					if ($this->t->scanOperand)
 					{
 						array_push($operators, new JSNode($this->t, JS_GROUP));
-					}
-					else
+					} else
 					{
 						while (	!empty($operators) &&
 							$this->opPrecedence[end($operators)->type] > $this->opPrecedence[KEYWORD_NEW]
-						)
-							$this->reduce($operators, $operands);
+						) {
+													$this->reduce($operators, $operands);
+						}
 
 						// Handle () now, to regularize the n-ary case for n > 0.
 						// We must set scanOperand in case there are arguments and
@@ -1354,8 +1406,7 @@ 
 							{
 								array_pop($operators);
 								$n->addNode(array_pop($operands));
-							}
-							else
+							} else
 							{
 								$n = new JSNode($this->t, JS_CALL, array_pop($operands), new JSNode($this->t, JS_LIST));
 							}
@@ -1365,18 +1416,20 @@ 
 							break;
 						}
 
-						if ($n && $n->type == KEYWORD_NEW)
-							$n->type = JS_NEW_WITH_ARGS;
-						else
-							array_push($operators, new JSNode($this->t, JS_CALL));
+						if ($n && $n->type == KEYWORD_NEW) {
+													$n->type = JS_NEW_WITH_ARGS;
+						} else {
+													array_push($operators, new JSNode($this->t, JS_CALL));
+						}
 					}
 
 					++$x->parenLevel;
 				break;
 
 				case OP_RIGHT_PAREN:
-					if ($this->t->scanOperand || $x->parenLevel == $pl)
-						break 2;
+					if ($this->t->scanOperand || $x->parenLevel == $pl) {
+											break 2;
+					}
 
 					while (($tt = $this->reduce($operators, $operands)->type) != JS_GROUP &&
 						$tt != JS_CALL && $tt != JS_NEW_WITH_ARGS
@@ -1388,10 +1441,11 @@ 
 					if ($tt != JS_GROUP)
 					{
 						$n = end($operands);
-						if ($n->treeNodes[1]->type != OP_COMMA)
-							$n->treeNodes[1] = new JSNode($this->t, JS_LIST, $n->treeNodes[1]);
-						else
-							$n->treeNodes[1]->type = JS_LIST;
+						if ($n->treeNodes[1]->type != OP_COMMA) {
+													$n->treeNodes[1] = new JSNode($this->t, JS_LIST, $n->treeNodes[1]);
+						} else {
+													$n->treeNodes[1]->type = JS_LIST;
+						}
 					}
 
 					--$x->parenLevel;
@@ -1405,24 +1459,29 @@ 
 			}
 		}
 
-		if ($x->hookLevel != $hl)
-			throw $this->t->newSyntaxError('Missing : in conditional expression');
+		if ($x->hookLevel != $hl) {
+					throw $this->t->newSyntaxError('Missing : in conditional expression');
+		}
 
-		if ($x->parenLevel != $pl)
-			throw $this->t->newSyntaxError('Missing ) in parenthetical');
+		if ($x->parenLevel != $pl) {
+					throw $this->t->newSyntaxError('Missing ) in parenthetical');
+		}
 
-		if ($x->bracketLevel != $bl)
-			throw $this->t->newSyntaxError('Missing ] in index expression');
+		if ($x->bracketLevel != $bl) {
+					throw $this->t->newSyntaxError('Missing ] in index expression');
+		}
 
-		if ($this->t->scanOperand)
-			throw $this->t->newSyntaxError('Missing operand');
+		if ($this->t->scanOperand) {
+					throw $this->t->newSyntaxError('Missing operand');
+		}
 
 		// Resume default mode, scanning for operands, not operators.
 		$this->t->scanOperand = true;
 		$this->t->unget();
 
-		while (count($operators))
-			$this->reduce($operators, $operands);
+		while (count($operators)) {
+					$this->reduce($operators, $operands);
+		}
 
 		return array_pop($operands);
 	}
@@ -1443,8 +1502,9 @@ 
 		$n = $this->statement($x);
 		array_pop($x->stmtStack);
 
-		if ($end)
-			$this->t->mustMatch($end);
+		if ($end) {
+					$this->t->mustMatch($end);
+		}
 
 		return $n;
 	}
@@ -1473,13 +1533,15 @@ 
 
 		// Always use push to add operands to n, to update start and end
 		$a = array_splice($operands, $c - $arity);
-		for ($i = 0; $i < $arity; $i++)
-			$n->addNode($a[$i]);
+		for ($i = 0; $i < $arity; $i++) {
+					$n->addNode($a[$i]);
+		}
 
 		// Include closing bracket or postfix operator in [start,end]
 		$te = $this->t->currentToken()->end;
-		if ($n->end < $te)
-			$n->end = $te;
+		if ($n->end < $te) {
+					$n->end = $te;
+		}
 
 		array_push($operands, $n);
 
@@ -1528,8 +1590,7 @@ 
 			$this->lineno = $token->lineno;
 			$this->start = $token->start;
 			$this->end = $token->end;
-		}
-		else
+		} else
 		{
 			$this->type = $type;
 			$this->lineno = $t->lineno;
@@ -1538,8 +1599,9 @@ 
 		if (($numargs = func_num_args()) > 2)
 		{
 			$args = func_get_args();;
-			for ($i = 2; $i < $numargs; $i++)
-				$this->addNode($args[$i]);
+			for ($i = 2; $i < $numargs; $i++) {
+							$this->addNode($args[$i]);
+			}
 		}
 	}
 
@@ -1551,8 +1613,9 @@ 
 
 	public function __get($name)
 	{
-		if (isset($this->$name))
-			return $this->$name;
+		if (isset($this->$name)) {
+					return $this->$name;
+		}
 
 		return null;
 	}
@@ -1561,10 +1624,12 @@ 
 	{
 		if ($node !== null)
 		{
-			if ($node->start < $this->start)
-				$this->start = $node->start;
-			if ($this->end < $node->end)
-				$this->end = $node->end;
+			if ($node->start < $this->start) {
+							$this->start = $node->start;
+			}
+			if ($this->end < $node->end) {
+							$this->end = $node->end;
+			}
 		}
 
 		$this->treeNodes[] = $node;
@@ -1649,14 +1714,16 @@ 
 		$this->opRegExp = '#^(' . implode('|', array_map('preg_quote', array_keys($this->opTypeNames))) . ')#';
 
 		// this is quite a hidden yet convenient place to create the defines for operators and keywords
-		foreach ($this->opTypeNames as $operand => $name)
-			define('OP_' . $name, $operand);
+		foreach ($this->opTypeNames as $operand => $name) {
+					define('OP_' . $name, $operand);
+		}
 
 		define('OP_UNARY_PLUS', 'U+');
 		define('OP_UNARY_MINUS', 'U-');
 
-		foreach ($this->keywords as $keyword)
-			define('KEYWORD_' . strtoupper($keyword), $keyword);
+		foreach ($this->keywords as $keyword) {
+					define('KEYWORD_' . strtoupper($keyword), $keyword);
+		}
 	}
 
 	public function init($source, $filename = '', $lineno = 1)
@@ -1675,8 +1742,9 @@ 
 
 	public function getInput($chunksize)
 	{
-		if ($chunksize)
-			return substr($this->source, $this->cursor, $chunksize);
+		if ($chunksize) {
+					return substr($this->source, $this->cursor, $chunksize);
+		}
 
 		return substr($this->source, $this->cursor);
 	}
@@ -1693,8 +1761,9 @@ 
 
 	public function mustMatch($tt)
 	{
-	        if (!$this->match($tt))
-			throw $this->newSyntaxError('Unexpected token; token ' . $tt . ' expected');
+	        if (!$this->match($tt)) {
+	        			throw $this->newSyntaxError('Unexpected token; token ' . $tt . ' expected');
+	        }
 
 		return $this->currentToken();
 	}
@@ -1704,12 +1773,12 @@ 
 		if ($this->lookahead)
 		{
 			$next = $this->tokens[($this->tokenIndex + $this->lookahead) & 3];
-			if ($this->scanNewlines && $next->lineno != $this->lineno)
-				$tt = TOKEN_NEWLINE;
-			else
-				$tt = $next->type;
-		}
-		else
+			if ($this->scanNewlines && $next->lineno != $this->lineno) {
+							$tt = TOKEN_NEWLINE;
+			} else {
+							$tt = $next->type;
+			}
+		} else
 		{
 			$tt = $this->get();
 			$this->unget();
@@ -1729,8 +1798,9 @@ 
 
 	public function currentToken()
 	{
-		if (!empty($this->tokens))
-			return $this->tokens[$this->tokenIndex];
+		if (!empty($this->tokens)) {
+					return $this->tokens[$this->tokenIndex];
+		}
 	}
 
 	public function get($chunksize = 1000)
@@ -1740,8 +1810,9 @@ 
 			$this->lookahead--;
 			$this->tokenIndex = ($this->tokenIndex + 1) & 3;
 			$token = $this->tokens[$this->tokenIndex];
-			if ($token->type != TOKEN_NEWLINE || $this->scanNewlines)
-				return $token->type;
+			if ($token->type != TOKEN_NEWLINE || $this->scanNewlines) {
+							return $token->type;
+			}
 		}
 
 		$conditional_comment = false;
@@ -1758,22 +1829,27 @@ 
 				$spaces = $match[0];
 				$spacelen = strlen($spaces);
 				$this->cursor += $spacelen;
-				if (!$this->scanNewlines)
-					$this->lineno += substr_count($spaces, "\n");
+				if (!$this->scanNewlines) {
+									$this->lineno += substr_count($spaces, "\n");
+				}
 
-				if ($spacelen == $chunksize)
-					continue; // complete chunk contained whitespace
+				if ($spacelen == $chunksize) {
+									continue;
+				}
+				// complete chunk contained whitespace
 
 				$input = $this->getInput($chunksize);
-				if ($input == '' || $input[0] != '/')
-					break;
+				if ($input == '' || $input[0] != '/') {
+									break;
+				}
 			}
 
 			// Comments
 			if (!preg_match('/^\/(?:\*(@(?:cc_on|if|elif|else|end))?.*?\*\/|\/[^\n]*)/s', $input, $match))
 			{
-				if (!$chunksize)
-					break;
+				if (!$chunksize) {
+									break;
+				}
 
 				// retry with a full chunk fetch; this also prevents breakage of long regular expressions (which will never match a comment)
 				$chunksize = null;
@@ -1786,8 +1862,7 @@ 
 				$match[0] = '/*' . $match[1];
 				$conditional_comment = true;
 				break;
-			}
-			else
+			} else
 			{
 				$this->cursor += strlen($match[0]);
 				$this->lineno += substr_count($match[0], "\n");
@@ -1798,12 +1873,10 @@ 
 		{
 			$tt = TOKEN_END;
 			$match = array('');
-		}
-		elseif ($conditional_comment)
+		} elseif ($conditional_comment)
 		{
 			$tt = TOKEN_CONDCOMMENT_START;
-		}
-		else
+		} else
 		{
 			switch ($input[0])
 			{
@@ -1812,8 +1885,7 @@ 
 					if (preg_match('/^\d+\.\d*(?:[eE][-+]?\d+)?|^\d+(?:\.\d*)?[eE][-+]?\d+/', $input, $match))
 					{
 						$tt = TOKEN_NUMBER;
-					}
-					else if (preg_match('/^0[xX][\da-fA-F]+|^0[0-7]*|^\d+/', $input, $match))
+					} else if (preg_match('/^0[xX][\da-fA-F]+|^0[0-7]*|^\d+/', $input, $match))
 					{
 						// this should always match because of \d+
 						$tt = TOKEN_NUMBER;
@@ -1825,11 +1897,12 @@ 
 					if (preg_match('/^"(?:\\\\(?:.|\r?\n)|[^\\\\"\r\n]+)*"|^\'(?:\\\\(?:.|\r?\n)|[^\\\\\'\r\n]+)*\'/', $input, $match))
 					{
 						$tt = TOKEN_STRING;
-					}
-					else
+					} else
 					{
-						if ($chunksize)
-							return $this->get(null); // retry with a full chunk fetch
+						if ($chunksize) {
+													return $this->get(null);
+						}
+						// retry with a full chunk fetch
 
 						throw $this->newSyntaxError('Unterminated string literal');
 					}
@@ -1861,16 +1934,16 @@ 
 					{
 						$tt = OP_ASSIGN;
 						$match[0] .= '=';
-					}
-					else
+					} else
 					{
 						$tt = $op;
 						if ($this->scanOperand)
 						{
-							if ($op == OP_PLUS)
-								$tt = OP_UNARY_PLUS;
-							elseif ($op == OP_MINUS)
-								$tt = OP_UNARY_MINUS;
+							if ($op == OP_PLUS) {
+															$tt = OP_UNARY_PLUS;
+							} elseif ($op == OP_MINUS) {
+															$tt = OP_UNARY_MINUS;
+							}
 						}
 						$op = null;
 					}
@@ -1906,9 +1979,9 @@ 
 					{
 						$match = array('@*/');
 						$tt = TOKEN_CONDCOMMENT_END;
+					} else {
+											throw $this->newSyntaxError('Illegal token');
 					}
-					else
-						throw $this->newSyntaxError('Illegal token');
 				break;
 
 				case "\n":
@@ -1916,9 +1989,9 @@ 
 					{
 						$match = array("\n");
 						$tt = TOKEN_NEWLINE;
+					} else {
+											throw $this->newSyntaxError('Illegal token');
 					}
-					else
-						throw $this->newSyntaxError('Illegal token');
 				break;
 
 				default:
@@ -1926,22 +1999,24 @@ 
 					if (preg_match('/^[$\w]+/', $input, $match))
 					{
 						$tt = in_array($match[0], $this->keywords) ? $match[0] : TOKEN_IDENTIFIER;
+					} else {
+											throw $this->newSyntaxError('Illegal token');
 					}
-					else
-						throw $this->newSyntaxError('Illegal token');
 			}
 		}
 
 		$this->tokenIndex = ($this->tokenIndex + 1) & 3;
 
-		if (!isset($this->tokens[$this->tokenIndex]))
-			$this->tokens[$this->tokenIndex] = new JSToken();
+		if (!isset($this->tokens[$this->tokenIndex])) {
+					$this->tokens[$this->tokenIndex] = new JSToken();
+		}
 
 		$token = $this->tokens[$this->tokenIndex];
 		$token->type = $tt;
 
-		if ($tt == OP_ASSIGN)
-			$token->assignOp = $op;
+		if ($tt == OP_ASSIGN) {
+					$token->assignOp = $op;
+		}
 
 		$token->start = $this->cursor;
 
@@ -1956,8 +2031,9 @@ 
 
 	public function unget()
 	{
-		if (++$this->lookahead == 4)
-			throw $this->newSyntaxError('PANIC: too much lookahead!');
+		if (++$this->lookahead == 4) {
+					throw $this->newSyntaxError('PANIC: too much lookahead!');
+		}
 
 		$this->tokenIndex = ($this->tokenIndex - 1) & 3;
 	}

tools/minify/minify.php

Doc Comments +6 added lines

@@ -172,6 +172,9 @@ 
 	}
 }
 
+/**
+ * @param string $ext
+ */
 function get_target_files($ext, $dir, $files_to_skip) {
 	$files = glob("{$dir}*.{$ext}");
 	$skips = glob("{$dir}*.min.{$ext}");
@@ -181,6 +184,9 @@ 
 	return $files;
 }
 
+/**
+ * @param string $content
+ */
 function closure_compile($content) {
 	require_once dirname(__FILE__).'/../../classes/httprequest/XEHttpRequest.class.php';
 

Spacing +38 added lines, -38 removed lines

@@ -11,7 +11,7 @@ 
  * @author NAVER(developer@xpressengine.com)
  */
 
-if(version_compare(PHP_VERSION, '5.3.0', '<')) {
+if (version_compare(PHP_VERSION, '5.3.0', '<')) {
 	echo "PHP 5.3.0 or above version is required.";
 	exit(1);
 }
@@ -21,8 +21,8 @@ 
 	$argc = $_SERVER['argc'];
 
 	// get target directories
-	if($argc < 1) exit;
-	elseif($argc < 2) $dirs = array($_SERVER['PWD']);
+	if ($argc < 1) exit;
+	elseif ($argc < 2) $dirs = array($_SERVER['PWD']);
 	else $dirs = array_slice($argv, 1);
 
 	$dirs = array_map('realpath', $dirs);
@@ -33,7 +33,7 @@ 
 
 // add directory separator
 function add_dirsep($path) {
-	if(substr($path,-1) != DIRECTORY_SEPARATOR) $path .= DIRECTORY_SEPARATOR;
+	if (substr($path, -1) != DIRECTORY_SEPARATOR) $path .= DIRECTORY_SEPARATOR;
 	return $path;
 }
 
@@ -46,38 +46,38 @@ 
 	echo " Done\n";
 
 	// merge
-	foreach($config['merge'] as $target=>$files) {
+	foreach ($config['merge'] as $target=>$files) {
 		merge($files, $target, $dir);
 	}
 
 	// files to skip
 	$files_to_skip = $config['skip'];
-	foreach($files_to_skip as $idx=>$file) {
-		if($file) $files_to_skip[$idx] = realpath($dir.trim($file));
+	foreach ($files_to_skip as $idx=>$file) {
+		if ($file) $files_to_skip[$idx] = realpath($dir . trim($file));
 	}
 
 	echo "  Minifying JavaScript files...";
 	$js_files = get_target_files('js', $dir, $files_to_skip);
 
-	if(count($js_files) && !class_exists('JSMinPlus')) {
-		require dirname(__FILE__).'/jsminplus/jsminplus.php';
+	if (count($js_files) && !class_exists('JSMinPlus')) {
+		require dirname(__FILE__) . '/jsminplus/jsminplus.php';
 	}
-	foreach($js_files as $file) {
-		if(!is_readable($file)) continue;
+	foreach ($js_files as $file) {
+		if (!is_readable($file)) continue;
 
 		$target  = preg_replace('@\.js$@', '.min.js', $file);
 		$content = file_get_contents($file);
 
 		// save copyright to preserve it
-		if(preg_match('@^[ \t]*(/\*\*.+?\*/)@s', $content, $matches)) {
-			$copyright = $matches[1]."\n";
+		if (preg_match('@^[ \t]*(/\*\*.+?\*/)@s', $content, $matches)) {
+			$copyright = $matches[1] . "\n";
 		} else {
 			$copyright = '';
 		}
 
-		if($config['use_closure_compiler']) {
+		if ($config['use_closure_compiler']) {
 			$content = closure_compile($content);
-			if(!$content) {
+			if (!$content) {
 				echo "   CANNOT compile the js file with closure compiler.\n";
 				echo "   Trying again with JSMinPlus.\n";
 				$content = JSMinPlus::minify($content);
@@ -86,7 +86,7 @@ 
 			$content = JSMinPlus::minify($content);
 		}
 
-		file_put_contents($target, $copyright.$content, LOCK_EX);
+		file_put_contents($target, $copyright . $content, LOCK_EX);
 
 		echo '.';
 	}
@@ -95,19 +95,19 @@ 
 	echo "  Minifying CSS files...";
 	$css_files = get_target_files('css', $dir, $files_to_skip);
 
-	if(count($css_files) && !class_exists('CSSmin')) {
-		require dirname(__FILE__).'/cssmin/CSSmin.php';
+	if (count($css_files) && !class_exists('CSSmin')) {
+		require dirname(__FILE__) . '/cssmin/CSSmin.php';
 	}
 
 	$oCSSmin = new CSSmin();
 
-	foreach($css_files as $file) {
-		if(!is_readable($file)) continue;
+	foreach ($css_files as $file) {
+		if (!is_readable($file)) continue;
 
 		$target  = preg_replace('@\.css$@', '.min.css', $file);
 		$content = file_get_contents($file);
 
-		file_put_contents($target, $copyright.$oCSSmin->run($content), LOCK_EX);
+		file_put_contents($target, $copyright . $oCSSmin->run($content), LOCK_EX);
 		echo '.';
 	}
 	echo " Done\n";
@@ -115,59 +115,59 @@ 
 
 function read_config($dir) {
 	$default = array('option'=>array(), 'skip'=>array(), 'merge'=>array());
-	$file    = $dir.'minify.ini';
+	$file    = $dir . 'minify.ini';
 
-	if(!is_readable($file)) return $default;
+	if (!is_readable($file)) return $default;
 
 	$config_str = file_get_contents($file);
 	$config_str = preg_replace_callback('/(\[(?:skip|merge *>> *.+?)\])([\s\S]+?)(?=\[|$)/', 'transform_config_str', $config_str);
 
 	$config = parse_ini_string($config_str, 1);
-	if($config === false) return $default;
+	if ($config === false) return $default;
 
-	if(is_array($config['skip'])) $config['skip'] = array_keys($config['skip']);
+	if (is_array($config['skip'])) $config['skip'] = array_keys($config['skip']);
 
-	foreach($config as $section=>$value) {
-		if(preg_match('/merge *>> *(.+)/', $section, $match)) {
-			if(!is_array($config['merge'])) $config['merge'] = array();
+	foreach ($config as $section=>$value) {
+		if (preg_match('/merge *>> *(.+)/', $section, $match)) {
+			if (!is_array($config['merge'])) $config['merge'] = array();
 			$config['merge'][trim($match[1])] = array_keys($value);
 
 			unset($config[$section]);
 		}
 	}
 
-	if(is_array($config['option'])) $config = array_merge($config['option'], $config);
+	if (is_array($config['option'])) $config = array_merge($config['option'], $config);
 	$config = array_merge($default, $config);
 
 	return $config;
 }
 
 function transform_config_str($matches) {
-	if(!$matches[2]) return $matches[0];
+	if (!$matches[2]) return $matches[0];
 	$values = preg_replace('/$/m', '=', trim($matches[2]));
 
 	return "{$matches[1]}\n{$values}\n\n";
 }
 
 function merge($files, $target, $base_dir) {
-	if(!is_array($files)) return false;
+	if (!is_array($files)) return false;
 
 	$body   = '';
 	$is_css = !!preg_match('/\.css$/', $target);
 
-	foreach($files as $file) {
-		$file = $base_dir.trim($file);
-		if(!is_readable($file)) continue;
+	foreach ($files as $file) {
+		$file = $base_dir . trim($file);
+		if (!is_readable($file)) continue;
 
 		$content = trim(file_get_contents($file));
-		if($is_css && $body) $content = preg_replace('/^@.+?;/m', '', $content);
-		if($content) $body .= $content."\n";
+		if ($is_css && $body) $content = preg_replace('/^@.+?;/m', '', $content);
+		if ($content) $body .= $content . "\n";
 	}
 
 	if ($body) {
 		$file_count = count($files);
 		echo "  Merging {$file_count} files to create {$target} file...";
-		file_put_contents($base_dir.$target, $body, LOCK_EX);
+		file_put_contents($base_dir . $target, $body, LOCK_EX);
 		echo " Done\n";
 	}
 }
@@ -182,7 +182,7 @@ 
 }
 
 function closure_compile($content) {
-	require_once dirname(__FILE__).'/../../classes/httprequest/XEHttpRequest.class.php';
+	require_once dirname(__FILE__) . '/../../classes/httprequest/XEHttpRequest.class.php';
 
 	$req = new XEHttpRequest('closure-compiler.appspot.com', 80);
 	$ret = $req->send('/compile', 'POST', 5, array(

Braces +49 added lines, -17 removed lines

@@ -21,9 +21,13 @@ 
 	$argc = $_SERVER['argc'];
 
 	// get target directories
-	if($argc < 1) exit;
-	elseif($argc < 2) $dirs = array($_SERVER['PWD']);
-	else $dirs = array_slice($argv, 1);
+	if($argc < 1) {
+		exit;
+	} elseif($argc < 2) {
+		$dirs = array($_SERVER['PWD']);
+	} else {
+		$dirs = array_slice($argv, 1);
+	}
 
 	$dirs = array_map('realpath', $dirs);
 	$dirs = array_map('add_dirsep', $dirs);
@@ -33,7 +37,9 @@ 
 
 // add directory separator
 function add_dirsep($path) {
-	if(substr($path,-1) != DIRECTORY_SEPARATOR) $path .= DIRECTORY_SEPARATOR;
+	if(substr($path,-1) != DIRECTORY_SEPARATOR) {
+		$path .= DIRECTORY_SEPARATOR;
+	}
 	return $path;
 }
 
@@ -53,7 +59,9 @@ 
 	// files to skip
 	$files_to_skip = $config['skip'];
 	foreach($files_to_skip as $idx=>$file) {
-		if($file) $files_to_skip[$idx] = realpath($dir.trim($file));
+		if($file) {
+			$files_to_skip[$idx] = realpath($dir.trim($file));
+		}
 	}
 
 	echo "  Minifying JavaScript files...";
@@ -63,7 +71,9 @@ 
 		require dirname(__FILE__).'/jsminplus/jsminplus.php';
 	}
 	foreach($js_files as $file) {
-		if(!is_readable($file)) continue;
+		if(!is_readable($file)) {
+			continue;
+		}
 
 		$target  = preg_replace('@\.js$@', '.min.js', $file);
 		$content = file_get_contents($file);
@@ -102,7 +112,9 @@ 
 	$oCSSmin = new CSSmin();
 
 	foreach($css_files as $file) {
-		if(!is_readable($file)) continue;
+		if(!is_readable($file)) {
+			continue;
+		}
 
 		$target  = preg_replace('@\.css$@', '.min.css', $file);
 		$content = file_get_contents($file);
@@ -117,51 +129,71 @@ 
 	$default = array('option'=>array(), 'skip'=>array(), 'merge'=>array());
 	$file    = $dir.'minify.ini';
 
-	if(!is_readable($file)) return $default;
+	if(!is_readable($file)) {
+		return $default;
+	}
 
 	$config_str = file_get_contents($file);
 	$config_str = preg_replace_callback('/(\[(?:skip|merge *>> *.+?)\])([\s\S]+?)(?=\[|$)/', 'transform_config_str', $config_str);
 
 	$config = parse_ini_string($config_str, 1);
-	if($config === false) return $default;
+	if($config === false) {
+		return $default;
+	}
 
-	if(is_array($config['skip'])) $config['skip'] = array_keys($config['skip']);
+	if(is_array($config['skip'])) {
+		$config['skip'] = array_keys($config['skip']);
+	}
 
 	foreach($config as $section=>$value) {
 		if(preg_match('/merge *>> *(.+)/', $section, $match)) {
-			if(!is_array($config['merge'])) $config['merge'] = array();
+			if(!is_array($config['merge'])) {
+				$config['merge'] = array();
+			}
 			$config['merge'][trim($match[1])] = array_keys($value);
 
 			unset($config[$section]);
 		}
 	}
 
-	if(is_array($config['option'])) $config = array_merge($config['option'], $config);
+	if(is_array($config['option'])) {
+		$config = array_merge($config['option'], $config);
+	}
 	$config = array_merge($default, $config);
 
 	return $config;
 }
 
 function transform_config_str($matches) {
-	if(!$matches[2]) return $matches[0];
+	if(!$matches[2]) {
+		return $matches[0];
+	}
 	$values = preg_replace('/$/m', '=', trim($matches[2]));
 
 	return "{$matches[1]}\n{$values}\n\n";
 }
 
 function merge($files, $target, $base_dir) {
-	if(!is_array($files)) return false;
+	if(!is_array($files)) {
+		return false;
+	}
 
 	$body   = '';
 	$is_css = !!preg_match('/\.css$/', $target);
 
 	foreach($files as $file) {
 		$file = $base_dir.trim($file);
-		if(!is_readable($file)) continue;
+		if(!is_readable($file)) {
+			continue;
+		}
 
 		$content = trim(file_get_contents($file));
-		if($is_css && $body) $content = preg_replace('/^@.+?;/m', '', $content);
-		if($content) $body .= $content."\n";
+		if($is_css && $body) {
+			$content = preg_replace('/^@.+?;/m', '', $content);
+		}
+		if($content) {
+			$body .= $content."\n";
+		}
 	}
 
 	if ($body) {

addons/adminlogging/adminlogging.addon.php

Spacing +2 added lines, -2 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 
-if(!defined('__XE__'))
+if (!defined('__XE__'))
 	exit();
 
 /**
@@ -10,7 +10,7 @@ 
  * @brief admin log
  */
 $logged_info = Context::get('logged_info');
-if($logged_info && $logged_info->is_admin == 'Y' && stripos(Context::get('act'), 'admin') !== false && $called_position == 'before_module_proc')
+if ($logged_info && $logged_info->is_admin == 'Y' && stripos(Context::get('act'), 'admin') !== false && $called_position == 'before_module_proc')
 {
 	$oAdminloggingController = getController('adminlogging');
 	$oAdminloggingController->insertLog($this->module, $this->act);

Braces +2 added lines, -1 removed lines

@@ -1,8 +1,9 @@
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 
-if(!defined('__XE__'))
+if(!defined('__XE__')) {
 	exit();
+}
 
 /**
  * @file point.addon.php

addons/autolink/autolink.addon.php

Spacing +3 added lines, -3 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 
-if(!defined('__XE__'))
+if (!defined('__XE__'))
 	exit();
 
 /**
@@ -9,9 +9,9 @@ 
  * @author NAVER (developers@xpressengine.com)
  * @brief Automatic link add-on
  */
-if($called_position == 'after_module_proc' && Context::getResponseMethod() == "HTML")
+if ($called_position == 'after_module_proc' && Context::getResponseMethod() == "HTML")
 {
-	if(Mobile::isFromMobilePhone())
+	if (Mobile::isFromMobilePhone())
 	{
 		Context::addJsFile('./common/js/jquery.min.js', false, '', -1000000);
 		Context::addJsFile('./common/js/xe.min.js', false, '', -1000000);

Braces +2 added lines, -1 removed lines

@@ -1,8 +1,9 @@
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 
-if(!defined('__XE__'))
+if(!defined('__XE__')) {
 	exit();
+}
 
 /**
  * @file point.addon.php

addons/blogapi/blogapi.addon.php

Spacing +78 added lines, -78 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 
-if(!defined('__XE__'))
+if (!defined('__XE__'))
 	exit();
 
 /**
@@ -13,7 +13,7 @@ 
  * It should be called before executing the module(before_module_proc). If not, it is forced to shut down.
  * */
 // Insert a rsd tag when called_position is after_module_proc
-if($called_position == 'after_module_proc')
+if ($called_position == 'after_module_proc')
 {
 	// Create rsd address of the current module
 	$site_module_info = Context::get('site_module_info');
@@ -22,7 +22,7 @@ 
 	Context::addHtmlHeader("    " . '<link rel="EditURI" type="application/rsd+xml" title="RSD" href="' . $rsd_url . '" />');
 }
 // If act isnot api, just return
-if($_REQUEST['act'] != 'api')
+if ($_REQUEST['act'] != 'api')
 {
 	return;
 }
@@ -33,7 +33,7 @@ 
 $xml = $GLOBALS['HTTP_RAW_POST_DATA'];
 
 // If HTTP_RAW_POST_DATA is NULL, Print error message
-if(!$xml)
+if (!$xml)
 {
 	$content = getXmlRpcFailure(1, 'Invalid Method Call');
 	printContent($content);
@@ -41,38 +41,38 @@ 
 
 // xmlprc parsing
 // Parse the requested xmlrpc
-if(Security::detectingXEE($xml))
+if (Security::detectingXEE($xml))
 {
 	header("HTTP/1.0 400 Bad Request");
 	exit;
 }
 
-if(version_compare(PHP_VERSION, '5.2.11', '<=')) libxml_disable_entity_loader(true);
+if (version_compare(PHP_VERSION, '5.2.11', '<=')) libxml_disable_entity_loader(true);
 $xml = new SimpleXMLElement($xml, LIBXML_NONET | LIBXML_NOENT);
 
-$method_name = (string)$xml->methodName;
+$method_name = (string) $xml->methodName;
 $params = $xml->params->param;
 
 // Compatible with some of methodname
-if(in_array($method_name, array('metaWeblog.deletePost', 'metaWeblog.getUsersBlogs', 'metaWeblog.getUserInfo')))
+if (in_array($method_name, array('metaWeblog.deletePost', 'metaWeblog.getUsersBlogs', 'metaWeblog.getUserInfo')))
 {
 	$method_name = str_replace('metaWeblog.', 'blogger.', $method_name);
 }
 
 // Get user_id, password and attempt log-in
-$user_id = trim((string)$params[1]->value->string);
-$password = trim((string)$params[2]->value->string);
+$user_id = trim((string) $params[1]->value->string);
+$password = trim((string) $params[2]->value->string);
 
 // Before executing the module, authentication is processed.
-if($called_position == 'before_module_init')
+if ($called_position == 'before_module_init')
 {
 	// Attempt log-in by using member controller
-	if($user_id && $password)
+	if ($user_id && $password)
 	{
 		$oMemberController = getController('member');
 		$output = $oMemberController->doLogin($user_id, $password);
 		// If login fails, an error message appears
-		if(!$output->toBool())
+		if (!$output->toBool())
 		{
 			$content = getXmlRpcFailure(1, $output->getMessage());
 			printContent($content);
@@ -86,10 +86,10 @@ 
 }
 
 // Before module processing, handle requests from blogapi tool and then terminate.
-if($called_position == 'before_module_proc')
+if ($called_position == 'before_module_proc')
 {
 	// Check writing permission 
-	if(!$this->grant->write_document)
+	if (!$this->grant->write_document)
 	{
 		printContent(getXmlRpcFailure(1, 'no permission'));
 	}
@@ -102,7 +102,7 @@ 
 	$tmp_uploaded_path = sprintf(_XE_PATH_ . 'files/cache/blogapi/%s/%s/', $this->mid, $user_id);
 	$uploaded_target_path = sprintf(_XE_PATH_ . 'files/cache/blogapi/%s/%s/', $this->mid, $user_id);
 
-	switch($method_name)
+	switch ($method_name)
 	{
 		// Blog information
 		case 'blogger.getUsersBlogs' :
@@ -119,9 +119,9 @@ 
 		// Return a list of categories
 		case 'metaWeblog.getCategories' :
 			$category_obj_list = array();
-			if($category_list)
+			if ($category_list)
 			{
-				foreach($category_list as $category_srl => $category_info)
+				foreach ($category_list as $category_srl => $category_info)
 				{
 					$obj = new stdClass();
 					$obj->description = $category_info->title;
@@ -142,35 +142,35 @@ 
 			// Check a file upload permission
 			$oFileModel = getModel('file');
 			$file_module_config = $oFileModel->getFileModuleConfig($this->module_srl);
-			if(is_array($file_module_config->download_grant) && count($file_module_config->download_grant) > 0)
+			if (is_array($file_module_config->download_grant) && count($file_module_config->download_grant) > 0)
 			{
 				$logged_info = Context::get('logged_info');
-				if($logged_info->is_admin != 'Y')
+				if ($logged_info->is_admin != 'Y')
 				{
 					$is_permitted = false;
-					for($i = 0; $i < count($file_module_config->download_grant); $i++)
+					for ($i = 0; $i < count($file_module_config->download_grant); $i++)
 					{
 						$group_srl = $file_module_config->download_grant[$i];
-						if($logged_info->group_list[$group_srl])
+						if ($logged_info->group_list[$group_srl])
 						{
 							$is_permitted = true;
 							break;
 						}
 					}
-					if(!$is_permitted){
+					if (!$is_permitted) {
 						printContent(getXmlRpcFailure(1, 'no permission'));
 					}
 				}
 			}
 
 			$fileinfo = $params[3]->value->struct->member;
-			foreach($fileinfo as $key => $val)
+			foreach ($fileinfo as $key => $val)
 			{
-				$nodename = (string)$val->name;
-				if($nodename == 'bits')
-					$filedata = base64_decode((string)$val->value->base64);
-				elseif($nodename == 'name')
-					$filename = (string)$val->value->string;
+				$nodename = (string) $val->name;
+				if ($nodename == 'bits')
+					$filedata = base64_decode((string) $val->value->base64);
+				elseif ($nodename == 'name')
+					$filename = (string) $val->value->string;
 			}
 
 			$tmp_arr = explode('/', $filename);
@@ -188,8 +188,8 @@ 
 			break;
 		// Get posts
 		case 'metaWeblog.getPost' :
-			$document_srl = (string)$params[0]->value->string;
-			if(!$document_srl)
+			$document_srl = (string) $params[0]->value->string;
+			if (!$document_srl)
 			{
 				printContent(getXmlRpcFailure(1, 'no permission'));
 			}
@@ -197,7 +197,7 @@ 
 			{
 				$oDocumentModel = getModel('document');
 				$oDocument = $oDocumentModel->getDocument($document_srl);
-				if(!$oDocument->isExists() || !$oDocument->isGranted())
+				if (!$oDocument->isExists() || !$oDocument->isGranted())
 				{
 					printContent(getXmlRpcFailure(1, 'no permission'));
 				}
@@ -205,11 +205,11 @@ 
 				{
 					// Get a list of categories and set Context
 					$category = "";
-					if($oDocument->get('category_srl'))
+					if ($oDocument->get('category_srl'))
 					{
 						$oDocumentModel = getModel('document');
 						$category_list = $oDocumentModel->getCategoryList($oDocument->get('module_srl'));
-						if($category_list[$oDocument->get('category_srl')])
+						if ($category_list[$oDocument->get('category_srl')])
 						{
 							$category = $category_list[$oDocument->get('category_srl')]->title;
 						}
@@ -251,35 +251,35 @@ 
 			$obj = new stdClass();
 			$info = $params[3];
 			// Get information of post, title, and category
-			foreach($info->value->struct->member as $val)
+			foreach ($info->value->struct->member as $val)
 			{
-				switch((string)$val->name)
+				switch ((string) $val->name)
 				{
 					case 'title' :
-						$obj->title = (string)$val->value->string;
+						$obj->title = (string) $val->value->string;
 						break;
 					case 'description' :
-						$obj->content = (string)$val->value->string;
+						$obj->content = (string) $val->value->string;
 						break;
 					case 'categories' :
 						$categories = $val->value->array->data->value;
-						$category = (string)$categories[0]->string;
-						if($category && $category_list)
+						$category = (string) $categories[0]->string;
+						if ($category && $category_list)
 						{
-							foreach($category_list as $category_srl => $category_info)
+							foreach ($category_list as $category_srl => $category_info)
 							{
-								if($category_info->title == $category)
+								if ($category_info->title == $category)
 									$obj->category_srl = $category_srl;
 							}
 						}
 						break;
 					case 'tagwords' :
 						$tags = $val->value->array->data->value;
-						foreach($tags as $tag)
+						foreach ($tags as $tag)
 						{
-							$tag_list[] = (string)$tag->string;
+							$tag_list[] = (string) $tag->string;
 						}
-						if(count($tag_list))
+						if (count($tag_list))
 							$obj->tags = implode(',', $tag_list);
 						break;
 				}
@@ -291,14 +291,14 @@ 
 			$obj->module_srl = $this->module_srl;
 
 			// Attachment
-			if(is_dir($tmp_uploaded_path))
+			if (is_dir($tmp_uploaded_path))
 			{
 				$file_list = FileHandler::readDir($tmp_uploaded_path);
 				$file_count = count($file_list);
-				if($file_count)
+				if ($file_count)
 				{
 					$oFileController = getController('file');
-					for($i = 0; $i < $file_count; $i++)
+					for ($i = 0; $i < $file_count; $i++)
 					{
 						$file_info['tmp_name'] = sprintf('%s%s', $tmp_uploaded_path, $file_list[$i]);
 						$file_info['name'] = $file_list[$i];
@@ -324,7 +324,7 @@ 
 			$obj->homepage = $logged_info->homepage;
 			$output = $oDocumentController->insertDocument($obj, TRUE);
 
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				$content = getXmlRpcFailure(1, $output->getMessage());
 			}
@@ -339,17 +339,17 @@ 
 
 		// Edit post
 		case 'metaWeblog.editPost' :
-			$tmp_val = (string)$params[0]->value->string;
-			if(!$tmp_val)
-				$tmp_val = (string)$params[0]->value->i4;
-			if(!$tmp_val)
+			$tmp_val = (string) $params[0]->value->string;
+			if (!$tmp_val)
+				$tmp_val = (string) $params[0]->value->i4;
+			if (!$tmp_val)
 			{
 				$content = getXmlRpcFailure(1, 'no permission');
 				break;
 			}
 			$tmp_arr = explode('/', $tmp_val);
 			$document_srl = array_pop($tmp_arr);
-			if(!$document_srl)
+			if (!$document_srl)
 			{
 				$content = getXmlRpcFailure(1, 'no permission');
 				break;
@@ -358,7 +358,7 @@ 
 			$oDocumentModel = getModel('document');
 			$oDocument = $oDocumentModel->getDocument($document_srl);
 			// Check if a permission to modify a document is granted
-			if(!$oDocument->isGranted())
+			if (!$oDocument->isGranted())
 			{
 				$content = getXmlRpcFailure(1, 'no permission');
 				break;
@@ -368,35 +368,35 @@ 
 
 			$info = $params[3];
 			// Get information of post, title, and category
-			foreach($info->value->struct->member as $val)
+			foreach ($info->value->struct->member as $val)
 			{
-				switch((string)$val->name)
+				switch ((string) $val->name)
 				{
 					case 'title' :
-						$obj->title = (string)$val->value->string;
+						$obj->title = (string) $val->value->string;
 						break;
 					case 'description' :
-						$obj->content = (string)$val->value->string;
+						$obj->content = (string) $val->value->string;
 						break;
 					case 'categories' :
 						$categories = $val->value->array->data->value;
-						$category = (string)$categories[0]->string;
-						if($category && $category_list)
+						$category = (string) $categories[0]->string;
+						if ($category && $category_list)
 						{
-							foreach($category_list as $category_srl => $category_info)
+							foreach ($category_list as $category_srl => $category_info)
 							{
-								if($category_info->title == $category)
+								if ($category_info->title == $category)
 									$obj->category_srl = $category_srl;
 							}
 						}
 						break;
 					case 'tagwords' :
 						$tags = $val->value->array->data->value;
-						foreach($tags as $tag)
+						foreach ($tags as $tag)
 						{
-							$tag_list[] = (string)$tag->string;
+							$tag_list[] = (string) $tag->string;
 						}
-						if(count($tag_list))
+						if (count($tag_list))
 							$obj->tags = implode(',', $tag_list);
 						break;
 				}
@@ -405,20 +405,20 @@ 
 			$obj->document_srl = $document_srl;
 			$obj->module_srl = $this->module_srl;
 			// Attachment
-			if(is_dir($tmp_uploaded_path))
+			if (is_dir($tmp_uploaded_path))
 			{
 				$file_list = FileHandler::readDir($tmp_uploaded_path);
 				$file_count = count($file_list);
-				if($file_count)
+				if ($file_count)
 				{
 					$oFileController = getController('file');
-					for($i = 0; $i < $file_count; $i++)
+					for ($i = 0; $i < $file_count; $i++)
 					{
 						$file_info['tmp_name'] = sprintf('%s%s', $tmp_uploaded_path, $file_list[$i]);
 						$file_info['name'] = $file_list[$i];
 
 						$moved_filename = sprintf('./files/attach/images/%s/%s/%s', $this->module_srl, $document_srl, $file_info['name']);
-						if(file_exists($moved_filename))
+						if (file_exists($moved_filename))
 							continue;
 
 						$fileOutput = $oFileController->insertFile($file_info, $this->module_srl, $document_srl, 0, true);
@@ -433,7 +433,7 @@ 
 			$oDocumentController = getController('document');
 			$output = $oDocumentController->updateDocument($oDocument, $obj, TRUE);
 
-			if(!$output->toBool())
+			if (!$output->toBool())
 			{
 				$content = getXmlRpcFailure(1, $output->getMessage());
 			}
@@ -447,19 +447,19 @@ 
 			break;
 		// Delete the post
 		case 'blogger.deletePost' :
-			$tmp_val = (string)$params[1]->value->string;
+			$tmp_val = (string) $params[1]->value->string;
 			$tmp_arr = explode('/', $tmp_val);
 			$document_srl = array_pop($tmp_arr);
 			// Get a document
 			$oDocumentModel = getModel('document');
 			$oDocument = $oDocumentModel->getDocument($document_srl);
 			// If the document exists
-			if(!$oDocument->isExists())
+			if (!$oDocument->isExists())
 			{
 				$content = getXmlRpcFailure(1, 'not exists');
 				// Check if a permission to delete a document is granted
 			}
-			elseif(!$oDocument->isGranted())
+			elseif (!$oDocument->isGranted())
 			{
 				$content = getXmlRpcFailure(1, 'no permission');
 				break;
@@ -469,7 +469,7 @@ 
 			{
 				$oDocumentController = getController('document');
 				$output = $oDocumentController->deleteDocument($document_srl);
-				if(!$output->toBool())
+				if (!$output->toBool())
 					$content = getXmlRpcFailure(1, $output->getMessage());
 				else
 					$content = getXmlRpcResponse(true);
@@ -489,7 +489,7 @@ 
 			$args->search_target = 'member_srl';
 			$args->search_keyword = $logged_info->member_srl;
 			$output = $oDocumentModel->getDocumentList($args);
-			if(!$output->toBool() || !$output->data)
+			if (!$output->toBool() || !$output->data)
 			{
 				$content = getXmlRpcFailure(1, 'post not founded');
 			}
@@ -498,12 +498,12 @@ 
 				$oEditorController = getController('editor');
 
 				$posts = array();
-				foreach($output->data as $key => $oDocument)
+				foreach ($output->data as $key => $oDocument)
 				{
 					$post = new stdClass();
 					$post->categories = array();
 					$post->dateCreated = date("Ymd", $oDocument->getRegdateTime()) . 'T' . date("H:i:s", $oDocument->getRegdateTime());
-					$post->description = sprintf('<![CDATA[%s]]>',$oEditorController->transComponent($oDocument->getContent(false, false, true, false)));
+					$post->description = sprintf('<![CDATA[%s]]>', $oEditorController->transComponent($oDocument->getContent(false, false, true, false)));
 					$post->link = $post->permaLink = getFullUrl('', 'document_srl', $oDocument->document_srl);
 					$post->postid = $oDocument->document_srl;
 					$post->title = htmlspecialchars($oDocument->get('title'), ENT_COMPAT | ENT_HTML401, 'UTF-8', false);

Braces +41 added lines, -38 removed lines

@@ -1,8 +1,9 @@ 
 <?php
 /* Copyright (C) NAVER <http://www.navercorp.com> */
 
-if(!defined('__XE__'))
+if(!defined('__XE__')) {
 	exit();
+}
 
 /**
  * @file blogapicounter.addon.php
@@ -47,7 +48,9 @@ 
 	exit;
 }
 
-if(version_compare(PHP_VERSION, '5.2.11', '<=')) libxml_disable_entity_loader(true);
+if(version_compare(PHP_VERSION, '5.2.11', '<=')) {
+	libxml_disable_entity_loader(true);
+}
 $xml = new SimpleXMLElement($xml, LIBXML_NONET | LIBXML_NOENT);
 
 $method_name = (string)$xml->methodName;
@@ -77,8 +80,7 @@ 
 			$content = getXmlRpcFailure(1, $output->getMessage());
 			printContent($content);
 		}
-	}
-	else
+	} else
 	{
 		$content = getXmlRpcFailure(1, 'not logged');
 		printContent($content);
@@ -167,10 +169,11 @@ 
 			foreach($fileinfo as $key => $val)
 			{
 				$nodename = (string)$val->name;
-				if($nodename == 'bits')
-					$filedata = base64_decode((string)$val->value->base64);
-				elseif($nodename == 'name')
-					$filename = (string)$val->value->string;
+				if($nodename == 'bits') {
+									$filedata = base64_decode((string)$val->value->base64);
+				} elseif($nodename == 'name') {
+									$filename = (string)$val->value->string;
+				}
 			}
 
 			$tmp_arr = explode('/', $filename);
@@ -192,16 +195,14 @@ 
 			if(!$document_srl)
 			{
 				printContent(getXmlRpcFailure(1, 'no permission'));
-			}
-			else
+			} else
 			{
 				$oDocumentModel = getModel('document');
 				$oDocument = $oDocumentModel->getDocument($document_srl);
 				if(!$oDocument->isExists() || !$oDocument->isGranted())
 				{
 					printContent(getXmlRpcFailure(1, 'no permission'));
-				}
-				else
+				} else
 				{
 					// Get a list of categories and set Context
 					$category = "";
@@ -268,8 +269,9 @@ 
 						{
 							foreach($category_list as $category_srl => $category_info)
 							{
-								if($category_info->title == $category)
-									$obj->category_srl = $category_srl;
+								if($category_info->title == $category) {
+																	$obj->category_srl = $category_srl;
+								}
 							}
 						}
 						break;
@@ -279,8 +281,9 @@ 
 						{
 							$tag_list[] = (string)$tag->string;
 						}
-						if(count($tag_list))
-							$obj->tags = implode(',', $tag_list);
+						if(count($tag_list)) {
+													$obj->tags = implode(',', $tag_list);
+						}
 						break;
 				}
 			}
@@ -327,8 +330,7 @@ 
 			if(!$output->toBool())
 			{
 				$content = getXmlRpcFailure(1, $output->getMessage());
-			}
-			else
+			} else
 			{
 				$content = getXmlRpcResponse(strval($document_srl));
 			}
@@ -340,8 +342,9 @@ 
 		// Edit post
 		case 'metaWeblog.editPost' :
 			$tmp_val = (string)$params[0]->value->string;
-			if(!$tmp_val)
-				$tmp_val = (string)$params[0]->value->i4;
+			if(!$tmp_val) {
+							$tmp_val = (string)$params[0]->value->i4;
+			}
 			if(!$tmp_val)
 			{
 				$content = getXmlRpcFailure(1, 'no permission');
@@ -385,8 +388,9 @@ 
 						{
 							foreach($category_list as $category_srl => $category_info)
 							{
-								if($category_info->title == $category)
-									$obj->category_srl = $category_srl;
+								if($category_info->title == $category) {
+																	$obj->category_srl = $category_srl;
+								}
 							}
 						}
 						break;
@@ -396,8 +400,9 @@ 
 						{
 							$tag_list[] = (string)$tag->string;
 						}
-						if(count($tag_list))
-							$obj->tags = implode(',', $tag_list);
+						if(count($tag_list)) {
+													$obj->tags = implode(',', $tag_list);
+						}
 						break;
 				}
 			}
@@ -418,8 +423,9 @@ 
 						$file_info['name'] = $file_list[$i];
 
 						$moved_filename = sprintf('./files/attach/images/%s/%s/%s', $this->module_srl, $document_srl, $file_info['name']);
-						if(file_exists($moved_filename))
-							continue;
+						if(file_exists($moved_filename)) {
+													continue;
+						}
 
 						$fileOutput = $oFileController->insertFile($file_info, $this->module_srl, $document_srl, 0, true);
 						$uploaded_filename = $fileOutput->get('uploaded_filename');
@@ -436,8 +442,7 @@ 
 			if(!$output->toBool())
 			{
 				$content = getXmlRpcFailure(1, $output->getMessage());
-			}
-			else
+			} else
 			{
 				$content = getXmlRpcResponse(true);
 				FileHandler::removeDir($tmp_uploaded_path);
@@ -458,21 +463,20 @@ 
 			{
 				$content = getXmlRpcFailure(1, 'not exists');
 				// Check if a permission to delete a document is granted
-			}
-			elseif(!$oDocument->isGranted())
+			} elseif(!$oDocument->isGranted())
 			{
 				$content = getXmlRpcFailure(1, 'no permission');
 				break;
 				// Delete
-			}
-			else
+			} else
 			{
 				$oDocumentController = getController('document');
 				$output = $oDocumentController->deleteDocument($document_srl);
-				if(!$output->toBool())
-					$content = getXmlRpcFailure(1, $output->getMessage());
-				else
-					$content = getXmlRpcResponse(true);
+				if(!$output->toBool()) {
+									$content = getXmlRpcFailure(1, $output->getMessage());
+				} else {
+									$content = getXmlRpcResponse(true);
+				}
 			}
 
 			printContent($content);
@@ -492,8 +496,7 @@ 
 			if(!$output->toBool() || !$output->data)
 			{
 				$content = getXmlRpcFailure(1, 'post not founded');
-			}
-			else
+			} else
 			{
 				$oEditorController = getController('editor');