xpressengine / xe-core

modules/document/document.admin.view.php

Braces +7 added lines, -4 removed lines

@@ -168,19 +168,22 @@
 	function dispDocumentAdminAlias()
 	{
 		$args->document_srl = Context::get('document_srl');
-		if(!$args->document_srl) return $this->dispDocumentAdminList();
+		if(!$args->document_srl) {
+			return $this->dispDocumentAdminList();
+		}
 
 		$oModel = getModel('document');
 		$oDocument = $oModel->getDocument($args->document_srl);
-		if(!$oDocument->isExists()) return $this->dispDocumentAdminList();
+		if(!$oDocument->isExists()) {
+			return $this->dispDocumentAdminList();
+		}
 		Context::set('oDocument', $oDocument);
 
 		$output = executeQueryArray('document.getAliases', $args);
 		if(!$output->data)
 		{
 			$aliases = array();
-		}
-		else
+		} else
 		{
 			$aliases = $output->data; 
 		}

Spacing +14 added lines, -14 removed lines

@@ -19,9 +19,9 @@ 
 		// check current location in admin menu
 		$oModuleModel = getModel('module');
 		$info = $oModuleModel->getModuleActionXml('document');
-		foreach($info->menu AS $key => $menu)
+		foreach ($info->menu AS $key => $menu)
 		{
-			if(in_array($this->act, $menu->acts))
+			if (in_array($this->act, $menu->acts))
 			{
 				Context::set('currentMenu', $key);
 				break;
@@ -66,7 +66,7 @@ 
 
 		// set a search option used in the template
 		$count_search_option = count($this->search_option);
-		for($i=0;$i<$count_search_option;$i++)
+		for ($i = 0; $i < $count_search_option; $i++)
 		{
 			$search_option[$this->search_option[$i]] = Context::getLang($this->search_option[$i]);
 		}
@@ -75,20 +75,20 @@ 
 		$oModuleModel = getModel('module');
 		$module_list = array();
 		$mod_srls = array();
-		foreach($output->data as $oDocument)
+		foreach ($output->data as $oDocument)
 		{
 			$mod_srls[] = $oDocument->get('module_srl');
 		}
 		$mod_srls = array_unique($mod_srls);
 		// Module List
 		$mod_srls_count = count($mod_srls);
-		if($mod_srls_count)
+		if ($mod_srls_count)
 		{
 			$columnList = array('module_srl', 'mid', 'browser_title');
 			$module_output = $oModuleModel->getModulesInfo($mod_srls, $columnList);
-			if($module_output && is_array($module_output))
+			if ($module_output && is_array($module_output))
 			{
-				foreach($module_output as $module)
+				foreach ($module_output as $module)
 				{
 					$module_list[$module->module_srl] = $module;
 				}
@@ -109,7 +109,7 @@ 
 	{
 		$oDocumentModel = getModel('document');
 		$config = $oDocumentModel->getDocumentConfig();
-		Context::set('config',$config);
+		Context::set('config', $config);
 
 		// Set the template file
 		$this->setTemplatePath($this->module_path.'tpl');
@@ -123,7 +123,7 @@ 
 	function dispDocumentAdminDeclared()
 	{
 		// option for a list
-		$args =new stdClass();
+		$args = new stdClass();
 		$args->page = Context::get('page'); // /< Page
 		$args->list_count = 30; // /< the number of posts to display on a single page
 		$args->page_count = 10; // /< the number of pages that appear in the page navigation
@@ -137,11 +137,11 @@ 
 
 		// get a list
 		$declared_output = executeQuery('document.getDeclaredList', $args);
-		if($declared_output->data && count($declared_output->data))
+		if ($declared_output->data && count($declared_output->data))
 		{
 			$document_list = array();
 
-			foreach($declared_output->data as $key => $document)
+			foreach ($declared_output->data as $key => $document)
 			{
 				$document_list[$key] = new documentItem();
 				$document_list[$key]->setAttribute($document);
@@ -168,15 +168,15 @@ 
 	function dispDocumentAdminAlias()
 	{
 		$args->document_srl = Context::get('document_srl');
-		if(!$args->document_srl) return $this->dispDocumentAdminList();
+		if (!$args->document_srl) return $this->dispDocumentAdminList();
 
 		$oModel = getModel('document');
 		$oDocument = $oModel->getDocument($args->document_srl);
-		if(!$oDocument->isExists()) return $this->dispDocumentAdminList();
+		if (!$oDocument->isExists()) return $this->dispDocumentAdminList();
 		Context::set('oDocument', $oDocument);
 
 		$output = executeQueryArray('document.getAliases', $args);
-		if(!$output->data)
+		if (!$output->data)
 		{
 			$aliases = array();
 		}

modules/editor/components/emoticon/emoticon.class.php

Braces +18 added lines, -6 removed lines

@@ -28,7 +28,9 @@ 
 	function getEmoticonList()
 	{
 		$emoticon = Context::get('emoticon');
-		if(!$emoticon || !preg_match("/^([a-z0-9\_]+)$/i",$emoticon)) return new Object(-1,'msg_invalid_request');
+		if(!$emoticon || !preg_match("/^([a-z0-9\_]+)$/i",$emoticon)) {
+			return new Object(-1,'msg_invalid_request');
+		}
 
 		$list = $this->getEmoticons($emoticon);
 
@@ -46,11 +48,17 @@ 
 		$oDir = dir($emoticon_path);
 		while($file = $oDir->read())
 		{
-			if(substr($file,0,1)=='.') continue;
-			if(preg_match('/\.(jpg|jpeg|gif|png)$/i',$file)) $output[] = sprintf("%s/%s", $path, str_replace($this->emoticon_path,'',$file));
+			if(substr($file,0,1)=='.') {
+				continue;
+			}
+			if(preg_match('/\.(jpg|jpeg|gif|png)$/i',$file)) {
+				$output[] = sprintf("%s/%s", $path, str_replace($this->emoticon_path,'',$file));
+			}
 		}
 		$oDir->close();
-		if(count($output)) asort($output);
+		if(count($output)) {
+			asort($output);
+		}
 		return $output;
 	}
 
@@ -66,7 +74,9 @@ 
 		{
 			foreach($emoticon_dirs as $emoticon)
 			{
-				if(preg_match("/^([a-z0-9\_]+)$/i", $emoticon)) $emoticon_list[] = $emoticon;
+				if(preg_match("/^([a-z0-9\_]+)$/i", $emoticon)) {
+					$emoticon_list[] = $emoticon;
+				}
 			}
 		}
 		Context::set('emoticon_list', $emoticon_list);
@@ -96,7 +106,9 @@ 
 		}
 
 		$src = str_replace(array('&','"'), array('&','&qout;'), $src);
-		if(!$alt) $alt = $src;
+		if(!$alt) {
+			$alt = $src;
+		}
 
 		$attr_output = array();
 		$attr_output = array("src=\"".$src."\"");

Spacing +16 added lines, -16 removed lines

@@ -19,7 +19,7 @@ 
 	{
 		$this->editor_sequence = $editor_sequence;
 		$this->component_path = $component_path;
-		$this->emoticon_path = sprintf('%s%s/images',preg_replace('/^\.\//i','',$this->component_path),'tpl','images');
+		$this->emoticon_path = sprintf('%s%s/images', preg_replace('/^\.\//i', '', $this->component_path), 'tpl', 'images');
 	}
 
 	/**
@@ -28,11 +28,11 @@ 
 	function getEmoticonList()
 	{
 		$emoticon = Context::get('emoticon');
-		if(!$emoticon || !preg_match("/^([a-z0-9\_]+)$/i",$emoticon)) return new Object(-1,'msg_invalid_request');
+		if (!$emoticon || !preg_match("/^([a-z0-9\_]+)$/i", $emoticon)) return new Object(-1, 'msg_invalid_request');
 
 		$list = $this->getEmoticons($emoticon);
 
-		$this->add('emoticons', implode("\n",$list));
+		$this->add('emoticons', implode("\n", $list));
 	}
 
 	/**
@@ -44,13 +44,13 @@ 
 		$output = array();
 
 		$oDir = dir($emoticon_path);
-		while($file = $oDir->read())
+		while ($file = $oDir->read())
 		{
-			if(substr($file,0,1)=='.') continue;
-			if(preg_match('/\.(jpg|jpeg|gif|png)$/i',$file)) $output[] = sprintf("%s/%s", $path, str_replace($this->emoticon_path,'',$file));
+			if (substr($file, 0, 1) == '.') continue;
+			if (preg_match('/\.(jpg|jpeg|gif|png)$/i', $file)) $output[] = sprintf("%s/%s", $path, str_replace($this->emoticon_path, '', $file));
 		}
 		$oDir->close();
-		if(count($output)) asort($output);
+		if (count($output)) asort($output);
 		return $output;
 	}
 
@@ -62,11 +62,11 @@ 
 		// Bringing a list of emoticons directory
 		$emoticon_dirs = FileHandler::readDir($this->emoticon_path);
 		$emoticon_list = array();
-		if($emoticon_dirs)
+		if ($emoticon_dirs)
 		{
-			foreach($emoticon_dirs as $emoticon)
+			foreach ($emoticon_dirs as $emoticon)
 			{
-				if(preg_match("/^([a-z0-9\_]+)$/i", $emoticon)) $emoticon_list[] = $emoticon;
+				if (preg_match("/^([a-z0-9\_]+)$/i", $emoticon)) $emoticon_list[] = $emoticon;
 			}
 		}
 		Context::set('emoticon_list', $emoticon_list);
@@ -89,24 +89,24 @@ 
 		$src = $xml_obj->attrs->src;
 		$alt = $xml_obj->attrs->alt;
 
-		if(!$alt)
+		if (!$alt)
 		{
-			$tmp_arr = explode('/',$src);
+			$tmp_arr = explode('/', $src);
 			$alt = array_pop($tmp_arr);
 		}
 
-		$src = str_replace(array('&','"'), array('&','&qout;'), $src);
-		if(!$alt) $alt = $src;
+		$src = str_replace(array('&', '"'), array('&', '&qout;'), $src);
+		if (!$alt) $alt = $src;
 
 		$attr_output = array();
 		$attr_output = array("src=\"".$src."\"");
 
-		if($alt)
+		if ($alt)
 		{
 			$attr_output[] = "alt=\"".$alt."\"";
 		}
 
-		$code = sprintf("<img %s style=\"border:0px\" />", implode(" ",$attr_output));
+		$code = sprintf("<img %s style=\"border:0px\" />", implode(" ", $attr_output));
 
 		return $code;
 	}

modules/editor/components/image_gallery/image_gallery.class.php

Braces +8 added lines, -3 removed lines

@@ -75,15 +75,20 @@
 		// HTML gallery output, the output settings via the template for the conversion to generate the html code should
 		preg_match_all('/(width|height)([^[:digit:]]+)([0-9]+)/i',$xml_obj->attrs->style,$matches);
 		$gallery_info->width = trim($matches[3][0]);
-		if(!$gallery_info->width) $gallery_info->width = 400;
+		if(!$gallery_info->width) {
+			$gallery_info->width = 400;
+		}
 
 		Context::set('gallery_info', $gallery_info);
 
 		$tpl_path = $this->component_path.'tpl';
 		Context::set("tpl_path", $tpl_path);
 
-		if($gallery_info->gallery_style == "list") $tpl_file = 'list_gallery.html';
-		else $tpl_file = 'slide_gallery.html';
+		if($gallery_info->gallery_style == "list") {
+			$tpl_file = 'list_gallery.html';
+		} else {
+			$tpl_file = 'slide_gallery.html';
+		}
 
 		$oTemplate = &TemplateHandler::getInstance();
 		return $oTemplate->compile($tpl_path, $tpl_file);

Spacing +9 added lines, -9 removed lines

@@ -45,14 +45,14 @@ 
 	{
 		$gallery_info = new stdClass();
 		$gallery_info->srl = rand(111111, 999999);
-		$gallery_info->border_thickness = (int)$xml_obj->attrs->border_thickness;
+		$gallery_info->border_thickness = (int) $xml_obj->attrs->border_thickness;
 		$gallery_info->gallery_style = $xml_obj->attrs->gallery_style;
 		$color_preg = "/^([a-fA-F0-9]{6})/";
-		$gallery_info->border_color = preg_replace($color_preg,"#$1",$xml_obj->attrs->border_color);
-		$gallery_info->bg_color = preg_replace($color_preg,"#$1",$xml_obj->attrs->bg_color);
+		$gallery_info->border_color = preg_replace($color_preg, "#$1", $xml_obj->attrs->border_color);
+		$gallery_info->bg_color = preg_replace($color_preg, "#$1", $xml_obj->attrs->bg_color);
 		$gallery_info->gallery_align = $xml_obj->attrs->gallery_align;
 
-		if(!in_array($gallery_info->gallery_align, array('left', 'center', 'right'))) {
+		if (!in_array($gallery_info->gallery_align, array('left', 'center', 'right'))) {
 			$gallery_info->gallery_align = 'center';
 		}
 
@@ -62,10 +62,10 @@ 
 		$gallery_info->images_list = preg_grep("/^[a-z0-9\/]+\.(gif|jpe?g|png)+$/", $images_list);
 
 		// If you set the output to output the XML code generated a list of the image
-		if(Context::getResponseMethod() == 'XMLRPC')
+		if (Context::getResponseMethod() == 'XMLRPC')
 		{
 			$output = array();
-			for($i=0;$i<count($gallery_info->images_list);$i++)
+			for ($i = 0; $i < count($gallery_info->images_list); $i++)
 			{
 				$output[] = sprintf('<img src="%s" alt="" />', $gallery_info->images_list[$i]);
 			}
@@ -73,16 +73,16 @@ 
 			return implode('<br />', $output);
 		}
 		// HTML gallery output, the output settings via the template for the conversion to generate the html code should
-		preg_match_all('/(width|height)([^[:digit:]]+)([0-9]+)/i',$xml_obj->attrs->style,$matches);
+		preg_match_all('/(width|height)([^[:digit:]]+)([0-9]+)/i', $xml_obj->attrs->style, $matches);
 		$gallery_info->width = trim($matches[3][0]);
-		if(!$gallery_info->width) $gallery_info->width = 400;
+		if (!$gallery_info->width) $gallery_info->width = 400;
 
 		Context::set('gallery_info', $gallery_info);
 
 		$tpl_path = $this->component_path.'tpl';
 		Context::set("tpl_path", $tpl_path);
 
-		if($gallery_info->gallery_style == "list") $tpl_file = 'list_gallery.html';
+		if ($gallery_info->gallery_style == "list") $tpl_file = 'list_gallery.html';
 		else $tpl_file = 'slide_gallery.html';
 
 		$oTemplate = &TemplateHandler::getInstance();

modules/editor/components/image_link/image_link.class.php

Braces +30 added lines, -12 removed lines

@@ -60,14 +60,19 @@ 
 
 		// Image containing the address to the address conversion request uri (rss output, etc. purposes)
 		$temp_src = explode('/', $src);
-		if(substr($src, 0,2)=='./') $src = Context::getRequestUri().substr($src, 2);
-		else if(substr($src , 0, 1)=='/')
+		if(substr($src, 0,2)=='./') {
+			$src = Context::getRequestUri().substr($src, 2);
+		} else if(substr($src , 0, 1)=='/')
 		{
-			if($_SERVER['HTTPS']=='on') $http_src = 'https://';
-			else $http_src = 'http://';
+			if($_SERVER['HTTPS']=='on') {
+				$http_src = 'https://';
+			} else {
+				$http_src = 'http://';
+			}
 			$src = $http_src.$_SERVER['HTTP_HOST'].$src;
+		} else if(!strpos($temp_src[0],':') && $src) {
+			$src = Context::getRequestUri().$src;
 		}
-		else if(!strpos($temp_src[0],':') && $src) $src = Context::getRequestUri().$src;
 
 		$attr_output = array();
 		$attr_output = array("src=\"".$src."\"");
@@ -81,17 +86,27 @@ 
 		{
 			$style = trim(preg_replace('/margin[a-z\-]*[ ]*:[ ]*[0-9 a-z]+(;| )/i','', $style)).';';
 			$style = str_replace(';;',';',$style);
-			if($style == ';') $style = '';
+			if($style == ';') {
+				$style = '';
+			}
 			$style .= ' margin:'.$margin.'px;';
 		}
-		if($align) $attr_output[] = "align=\"".$align."\"";
-		if($width) $attr_output[] = 'width="'.$width.'"';
-		if($height) $attr_output[] = 'height="'.$height.'"';
+		if($align) {
+			$attr_output[] = "align=\"".$align."\"";
+		}
+		if($width) {
+			$attr_output[] = 'width="'.$width.'"';
+		}
+		if($height) {
+			$attr_output[] = 'height="'.$height.'"';
+		}
 		if($border)
 		{
 			$style = trim(preg_replace('/border[a-z\-]*[ ]*:[ ]*[0-9 a-z]+(;| )/i','', $style)).';';
 			$style = str_replace(';;',';',$style);
-			if($style == ';') $style = '';
+			if($style == ';') {
+				$style = '';
+			}
 			$style .= ' border-style: solid; border-width:'.$border.'px;';
 		}
 
@@ -99,8 +114,11 @@ 
 
 		if($link_url)
 		{
-			if($open_window =='Y') $code = sprintf('<a href="%s" onclick="window.open(this.href);return false;">%s</a>', $link_url, $code);
-			else $code = sprintf('<a href="%s" >%s</a>', $link_url, $code);
+			if($open_window =='Y') {
+				$code = sprintf('<a href="%s" onclick="window.open(this.href);return false;">%s</a>', $link_url, $code);
+			} else {
+				$code = sprintf('<a href="%s" >%s</a>', $link_url, $code);
+			}
 		}
 		return $code;
 	}

Spacing +22 added lines, -22 removed lines

@@ -49,57 +49,57 @@
 		$align = $xml_obj->attrs->align;
 		$alt = $xml_obj->attrs->alt;
 		$title = $xml_obj->attrs->title;
-		$border = (int)$xml_obj->attrs->border;
+		$border = (int) $xml_obj->attrs->border;
 		$link_url = $xml_obj->attrs->link_url;
 		$open_window = $xml_obj->attrs->open_window;
 		$style = $xml_obj->attrs->style;
-		$margin = (int)$xml_obj->attrs->margin;
+		$margin = (int) $xml_obj->attrs->margin;
 
-		$src = str_replace(array('&','"'), array('&','&qout;'), $src);
+		$src = str_replace(array('&', '"'), array('&', '&qout;'), $src);
 		$src = str_replace('&', '&', $src);
 
 		// Image containing the address to the address conversion request uri (rss output, etc. purposes)
 		$temp_src = explode('/', $src);
-		if(substr($src, 0,2)=='./') $src = Context::getRequestUri().substr($src, 2);
-		else if(substr($src , 0, 1)=='/')
+		if (substr($src, 0, 2) == './') $src = Context::getRequestUri().substr($src, 2);
+		else if (substr($src, 0, 1) == '/')
 		{
-			if($_SERVER['HTTPS']=='on') $http_src = 'https://';
+			if ($_SERVER['HTTPS'] == 'on') $http_src = 'https://';
 			else $http_src = 'http://';
 			$src = $http_src.$_SERVER['HTTP_HOST'].$src;
 		}
-		else if(!strpos($temp_src[0],':') && $src) $src = Context::getRequestUri().$src;
+		else if (!strpos($temp_src[0], ':') && $src) $src = Context::getRequestUri().$src;
 
 		$attr_output = array();
 		$attr_output = array("src=\"".$src."\"");
 		$attr_output[] = "alt=\"".$alt."\"";
 
-		if($title)
+		if ($title)
 		{
 			$attr_output[] = "title=\"".$title."\"";
 		}
-		if($margin)
+		if ($margin)
 		{
-			$style = trim(preg_replace('/margin[a-z\-]*[ ]*:[ ]*[0-9 a-z]+(;| )/i','', $style)).';';
-			$style = str_replace(';;',';',$style);
-			if($style == ';') $style = '';
+			$style = trim(preg_replace('/margin[a-z\-]*[ ]*:[ ]*[0-9 a-z]+(;| )/i', '', $style)).';';
+			$style = str_replace(';;', ';', $style);
+			if ($style == ';') $style = '';
 			$style .= ' margin:'.$margin.'px;';
 		}
-		if($align) $attr_output[] = "align=\"".$align."\"";
-		if($width) $attr_output[] = 'width="'.$width.'"';
-		if($height) $attr_output[] = 'height="'.$height.'"';
-		if($border)
+		if ($align) $attr_output[] = "align=\"".$align."\"";
+		if ($width) $attr_output[] = 'width="'.$width.'"';
+		if ($height) $attr_output[] = 'height="'.$height.'"';
+		if ($border)
 		{
-			$style = trim(preg_replace('/border[a-z\-]*[ ]*:[ ]*[0-9 a-z]+(;| )/i','', $style)).';';
-			$style = str_replace(';;',';',$style);
-			if($style == ';') $style = '';
+			$style = trim(preg_replace('/border[a-z\-]*[ ]*:[ ]*[0-9 a-z]+(;| )/i', '', $style)).';';
+			$style = str_replace(';;', ';', $style);
+			if ($style == ';') $style = '';
 			$style .= ' border-style: solid; border-width:'.$border.'px;';
 		}
 
-		$code = sprintf("<img %s style=\"%s\" />", implode(' ',$attr_output), $style);
+		$code = sprintf("<img %s style=\"%s\" />", implode(' ', $attr_output), $style);
 
-		if($link_url)
+		if ($link_url)
 		{
-			if($open_window =='Y') $code = sprintf('<a href="%s" onclick="window.open(this.href);return false;">%s</a>', $link_url, $code);
+			if ($open_window == 'Y') $code = sprintf('<a href="%s" onclick="window.open(this.href);return false;">%s</a>', $link_url, $code);
 			else $code = sprintf('<a href="%s" >%s</a>', $link_url, $code);
 		}
 		return $code;

modules/editor/components/poll_maker/poll_maker.class.php

Braces +6 added lines, -2 removed lines

@@ -47,11 +47,15 @@
 	{
 		$poll_srl = $xml_obj->attrs->poll_srl;
 		$skin = $xml_obj->attrs->skin;
-		if(!$skin) $skin = 'default';
+		if(!$skin) {
+			$skin = 'default';
+		}
 
 		preg_match('/width([^[:digit:]]+)([0-9]+)/i',$xml_obj->attrs->style,$matches);
 		$width = $matches[2];
-		if(!$width) $width = 400;
+		if(!$width) {
+			$width = 400;
+		}
 		$style = sprintf('width:%dpx', $width);
 		// poll model object creation to come get it return html
 		$oPollModel = getModel('poll');

Spacing +4 added lines, -4 removed lines

@@ -27,7 +27,7 @@ 
 	{
 		// Wanted Skins survey
 		$oModuleModel = getModel('module');
-		$skin_list = $oModuleModel->getSkins(_XE_PATH_ . 'modules/poll/');
+		$skin_list = $oModuleModel->getSkins(_XE_PATH_.'modules/poll/');
 		Context::set('skin_list', $skin_list);
 		// Pre-compiled source code to compile template return to
 		$tpl_path = $this->component_path.'tpl';
@@ -47,11 +47,11 @@ 
 	{
 		$poll_srl = $xml_obj->attrs->poll_srl;
 		$skin = $xml_obj->attrs->skin;
-		if(!$skin) $skin = 'default';
+		if (!$skin) $skin = 'default';
 
-		preg_match('/width([^[:digit:]]+)([0-9]+)/i',$xml_obj->attrs->style,$matches);
+		preg_match('/width([^[:digit:]]+)([0-9]+)/i', $xml_obj->attrs->style, $matches);
 		$width = $matches[2];
-		if(!$width) $width = 400;
+		if (!$width) $width = 400;
 		$style = sprintf('width:%dpx', $width);
 		// poll model object creation to come get it return html
 		$oPollModel = getModel('poll');

modules/editor/editor.admin.controller.php

Braces +56 added lines, -26 removed lines

@@ -23,8 +23,12 @@ 
 		$enables = Context::get('enables');
 		$component_names = Context::get('component_names');
 
-		if(!is_array($component_names)) $component_names = array();
-		if(!is_array($enables)) $enables = array();
+		if(!is_array($component_names)) {
+			$component_names = array();
+		}
+		if(!is_array($enables)) {
+			$enables = array();
+		}
 
 		$unables = array_diff($component_names, $enables);
 		$componentList = array();
@@ -39,10 +43,14 @@ 
 		}
 
 		$output = $this->editorListOrder($component_names,$site_module_info->site_srl);
-		if(!$output->toBool()) return new Object();
+		if(!$output->toBool()) {
+			return new Object();
+		}
 
 		$output = $this->editorCheckUse($componentList,$site_module_info->site_srl);
-		if(!$output->toBool()) return new Object();
+		if(!$output->toBool()) {
+			return new Object();
+		}
 
 		$oEditorController = getController('editor');
 		$oEditorController->removeCache($site_module_info->site_srl);
@@ -64,13 +72,14 @@ 
 			if($site_srl == 0)
 			{
 				$output = executeQuery('editor.updateComponent', $args);
-			}
-			else
+			} else
 			{
 				$output = executeQuery('editor.updateSiteComponent', $args);
 			}
 		}
-		if(!$output->toBool()) return new Object();
+		if(!$output->toBool()) {
+			return new Object();
+		}
 
 		unset($componentList);
 		return $output;
@@ -93,13 +102,14 @@ 
 				if($site_srl == 0)
 				{
 					$output = executeQuery('editor.updateComponent', $args);
-				}
-				else
+				} else
 				{
 					$output = executeQuery('editor.updateSiteComponent', $args);
 				}
 
-				if(!$output->toBool()) return new Object();
+				if(!$output->toBool()) {
+					return new Object();
+				}
 				$list_order_num++;
 			}
 		}
@@ -126,9 +136,14 @@ 
 		$args->extra_vars = serialize($extra_vars);
 		$args->site_srl = (int)$site_module_info->site_srl;
 
-		if(!$args->site_srl) $output = executeQuery('editor.updateComponent', $args);
-		else $output = executeQuery('editor.updateSiteComponent', $args);
-		if(!$output->toBool()) return $output;
+		if(!$args->site_srl) {
+			$output = executeQuery('editor.updateComponent', $args);
+		} else {
+			$output = executeQuery('editor.updateSiteComponent', $args);
+		}
+		if(!$output->toBool()) {
+			return $output;
+		}
 
 		$oEditorController = getController('editor');
 		$oEditorController->removeCache($args->site_srl);
@@ -145,13 +160,17 @@ 
 		$oModuleController = getController('module');
 		$configVars = Context::getRequestVars();
 
-		if($configVars->font_defined != 'Y') $config->font_defined = $configVars->font_defined = 'N';
-		else $config->font_defined = 'Y';
+		if($configVars->font_defined != 'Y') {
+			$config->font_defined = $configVars->font_defined = 'N';
+		} else {
+			$config->font_defined = 'Y';
+		}
 
-		if($config->font_defined == 'Y')
-			$config->content_font = $configVars->content_font_defined;
-		else
-			$config->content_font = $configVars->content_font;
+		if($config->font_defined == 'Y') {
+					$config->content_font = $configVars->content_font_defined;
+		} else {
+					$config->content_font = $configVars->content_font;
+		}
 
 		$config->editor_skin = $configVars->editor_skin;
 		$config->editor_height = $configVars->editor_height;
@@ -172,21 +191,32 @@ 
 	 */
 	function insertComponent($component_name, $enabled = false, $site_srl = 0)
 	{
-		if($enabled) $enabled = 'Y';
-		else $enabled = 'N';
+		if($enabled) {
+			$enabled = 'Y';
+		} else {
+			$enabled = 'N';
+		}
 
 		$args = new stdClass;
 		$args->component_name = $component_name;
 		$args->enabled = $enabled;
 		$args->site_srl = $site_srl;
 		// Check if the component exists
-		if(!$site_srl) $output = executeQuery('editor.isComponentInserted', $args);
-		else $output = executeQuery('editor.isSiteComponentInserted', $args);
-		if($output->data->count) return new Object(-1, 'msg_component_is_not_founded');
+		if(!$site_srl) {
+			$output = executeQuery('editor.isComponentInserted', $args);
+		} else {
+			$output = executeQuery('editor.isSiteComponentInserted', $args);
+		}
+		if($output->data->count) {
+			return new Object(-1, 'msg_component_is_not_founded');
+		}
 		// Inert a component
 		$args->list_order = getNextSequence();
-		if(!$site_srl) $output = executeQuery('editor.insertComponent', $args);
-		else $output = executeQuery('editor.insertSiteComponent', $args);
+		if(!$site_srl) {
+			$output = executeQuery('editor.insertComponent', $args);
+		} else {
+			$output = executeQuery('editor.insertSiteComponent', $args);
+		}
 
 		$oEditorController = getController('editor');
 		$oEditorController->removeCache($site_srl);

Spacing +28 added lines, -28 removed lines

@@ -23,26 +23,26 @@ 
 		$enables = Context::get('enables');
 		$component_names = Context::get('component_names');
 
-		if(!is_array($component_names)) $component_names = array();
-		if(!is_array($enables)) $enables = array();
+		if (!is_array($component_names)) $component_names = array();
+		if (!is_array($enables)) $enables = array();
 
 		$unables = array_diff($component_names, $enables);
 		$componentList = array();
 
-		foreach($enables as $component_name)
+		foreach ($enables as $component_name)
 		{
 			$componentList[$component_name] = 'Y';
 		}
-		foreach($unables as $component_name)
+		foreach ($unables as $component_name)
 		{
 			$componentList[$component_name] = 'N';
 		}
 
-		$output = $this->editorListOrder($component_names,$site_module_info->site_srl);
-		if(!$output->toBool()) return new Object();
+		$output = $this->editorListOrder($component_names, $site_module_info->site_srl);
+		if (!$output->toBool()) return new Object();
 
-		$output = $this->editorCheckUse($componentList,$site_module_info->site_srl);
-		if(!$output->toBool()) return new Object();
+		$output = $this->editorCheckUse($componentList, $site_module_info->site_srl);
+		if (!$output->toBool()) return new Object();
 
 		$oEditorController = getController('editor');
 		$oEditorController->removeCache($site_module_info->site_srl);
@@ -57,11 +57,11 @@ 
 		$args = new stdClass();
 		$args->site_srl = $site_srl;
 
-		foreach($componentList as $componentName => $value)
+		foreach ($componentList as $componentName => $value)
 		{
 			$args->component_name = $componentName;
 			$args->enabled = $value;
-			if($site_srl == 0)
+			if ($site_srl == 0)
 			{
 				$output = executeQuery('editor.updateComponent', $args);
 			}
@@ -70,7 +70,7 @@ 
 				$output = executeQuery('editor.updateSiteComponent', $args);
 			}
 		}
-		if(!$output->toBool()) return new Object();
+		if (!$output->toBool()) return new Object();
 
 		unset($componentList);
 		return $output;
@@ -84,13 +84,13 @@ 
 		$args = new stdClass();
 		$args->site_srl = $site_srl;
 		$list_order_num = '30';
-		if(is_array($component_names))
+		if (is_array($component_names))
 		{
-			foreach($component_names as $name)
+			foreach ($component_names as $name)
 			{
 				$args->list_order = $list_order_num;
 				$args->component_name = $name;
-				if($site_srl == 0)
+				if ($site_srl == 0)
 				{
 					$output = executeQuery('editor.updateComponent', $args);
 				}
@@ -99,7 +99,7 @@ 
 					$output = executeQuery('editor.updateSiteComponent', $args);
 				}
 
-				if(!$output->toBool()) return new Object();
+				if (!$output->toBool()) return new Object();
 				$list_order_num++;
 			}
 		}
@@ -124,11 +124,11 @@ 
 		$args = new stdClass;
 		$args->component_name = $component_name;
 		$args->extra_vars = serialize($extra_vars);
-		$args->site_srl = (int)$site_module_info->site_srl;
+		$args->site_srl = (int) $site_module_info->site_srl;
 
-		if(!$args->site_srl) $output = executeQuery('editor.updateComponent', $args);
+		if (!$args->site_srl) $output = executeQuery('editor.updateComponent', $args);
 		else $output = executeQuery('editor.updateSiteComponent', $args);
-		if(!$output->toBool()) return $output;
+		if (!$output->toBool()) return $output;
 
 		$oEditorController = getController('editor');
 		$oEditorController->removeCache($args->site_srl);
@@ -146,10 +146,10 @@ 
 		$configVars = Context::getRequestVars();
 		
 		$config = new stdClass;
-		if($configVars->font_defined != 'Y') $config->font_defined = $configVars->font_defined = 'N';
+		if ($configVars->font_defined != 'Y') $config->font_defined = $configVars->font_defined = 'N';
 		else $config->font_defined = 'Y';
 
-		if($config->font_defined == 'Y')
+		if ($config->font_defined == 'Y')
 			$config->content_font = $configVars->content_font_defined;
 		else
 			$config->content_font = $configVars->content_font;
@@ -160,11 +160,11 @@ 
 		$config->comment_editor_height = $configVars->comment_editor_height;
 		$config->content_style = $configVars->content_style;
 
-		$config->content_font_size= $configVars->content_font_size.'px';
-		$config->sel_editor_colorset= $configVars->sel_editor_colorset;
-		$config->sel_comment_editor_colorset= $configVars->sel_comment_editor_colorset;
+		$config->content_font_size = $configVars->content_font_size.'px';
+		$config->sel_editor_colorset = $configVars->sel_editor_colorset;
+		$config->sel_comment_editor_colorset = $configVars->sel_comment_editor_colorset;
 
-		$oModuleController->insertModuleConfig('editor',$config);
+		$oModuleController->insertModuleConfig('editor', $config);
 		$this->setRedirectUrl(Context::get('error_return_url'));
 	}
 
@@ -173,7 +173,7 @@ 
 	 */
 	function insertComponent($component_name, $enabled = false, $site_srl = 0)
 	{
-		if($enabled) $enabled = 'Y';
+		if ($enabled) $enabled = 'Y';
 		else $enabled = 'N';
 
 		$args = new stdClass;
@@ -181,12 +181,12 @@ 
 		$args->enabled = $enabled;
 		$args->site_srl = $site_srl;
 		// Check if the component exists
-		if(!$site_srl) $output = executeQuery('editor.isComponentInserted', $args);
+		if (!$site_srl) $output = executeQuery('editor.isComponentInserted', $args);
 		else $output = executeQuery('editor.isSiteComponentInserted', $args);
-		if($output->data->count) return new Object(-1, 'msg_component_is_not_founded');
+		if ($output->data->count) return new Object(-1, 'msg_component_is_not_founded');
 		// Inert a component
 		$args->list_order = getNextSequence();
-		if(!$site_srl) $output = executeQuery('editor.insertComponent', $args);
+		if (!$site_srl) $output = executeQuery('editor.insertComponent', $args);
 		else $output = executeQuery('editor.insertSiteComponent', $args);
 
 		$oEditorController = getController('editor');

modules/editor/editor.admin.view.php

Braces +34 added lines, -13 removed lines

@@ -35,12 +35,24 @@ 
 		}
 
 		//editor_config init
-		if(!$editor_config->editor_height) $editor_config->editor_height = 300;
-		if(!$editor_config->comment_editor_height) $editor_config->comment_editor_height = 100;
-		if(!$editor_config->editor_skin) $editor_config->editor_skin = 'ckeditor';
-		if(!$editor_config->comment_editor_skin) $editor_config->comment_editor_skin = 'ckeditor';
-		if(!$editor_config->sel_editor_colorset) $editor_config->sel_editor_colorset= 'moono';
-		if(!$editor_config->sel_comment_editor_colorset) $editor_config->sel_comment_editor_colorset= 'moono';
+		if(!$editor_config->editor_height) {
+			$editor_config->editor_height = 300;
+		}
+		if(!$editor_config->comment_editor_height) {
+			$editor_config->comment_editor_height = 100;
+		}
+		if(!$editor_config->editor_skin) {
+			$editor_config->editor_skin = 'ckeditor';
+		}
+		if(!$editor_config->comment_editor_skin) {
+			$editor_config->comment_editor_skin = 'ckeditor';
+		}
+		if(!$editor_config->sel_editor_colorset) {
+			$editor_config->sel_editor_colorset= 'moono';
+		}
+		if(!$editor_config->sel_comment_editor_colorset) {
+			$editor_config->sel_comment_editor_colorset= 'moono';
+		}
 
 		$component_list = $oEditorModel->getComponentList(false, $site_srl, true);
 		$editor_skin_list = FileHandler::readDir(_XE_PATH_.'modules/editor/skins');
@@ -65,14 +77,20 @@ 
 			$xml_info->path = './modules/editor/components/'.$xml_info->component_name;
 			$xml_info->delete_url = $oAutoinstallModel->getRemoveUrlByPath($xml_info->path);
 			$xml_info->package_srl = $oAutoinstallModel->getPackageSrlByPath($xml_info->path);
-			if($xml_info->package_srl) $targetpackages[$xml_info->package_srl] = 0;
+			if($xml_info->package_srl) {
+				$targetpackages[$xml_info->package_srl] = 0;
+			}
 		}
 
-		if(is_array($targetpackages))	$packages = $oAutoinstallModel->getInstalledPackages(array_keys($targetpackages));
+		if(is_array($targetpackages)) {
+			$packages = $oAutoinstallModel->getInstalledPackages(array_keys($targetpackages));
+		}
 
 		foreach($component_list as $component_name => $xml_info)
 		{
-			if($packages[$xml_info->package_srl])	$xml_info->need_update = $packages[$xml_info->package_srl]->need_update;
+			if($packages[$xml_info->package_srl]) {
+				$xml_info->need_update = $packages[$xml_info->package_srl]->need_update;
+			}
 		}
 		$editor_config_default = array( "editor_height" => "300", "comment_editor_height" => "100","content_font_size"=>"13");
 
@@ -162,13 +180,16 @@ 
 			// Get a list of module category
 			$module_categories = $oModuleModel->getModuleCategories();
 
-			if(!is_array($mid_list)) $mid_list = array($mid_list);
+			if(!is_array($mid_list)) {
+				$mid_list = array($mid_list);
+			}
 			foreach($mid_list as $module_srl => $module)
 			{
-				if($module) $module_categories[$module->module_category_srl]->list[$module_srl] = $module; 
+				if($module) {
+					$module_categories[$module->module_category_srl]->list[$module_srl] = $module;
+				}
 			}
-		}
-		else
+		} else
 		{
 			$module_categories[0]->list = $mid_list;
 		}

Spacing +26 added lines, -26 removed lines

@@ -22,59 +22,59 @@ 
 	{
 		$component_count = 0;
 		$site_module_info = Context::get('site_module_info');
-		$site_srl = (int)$site_module_info->site_srl;
+		$site_srl = (int) $site_module_info->site_srl;
 
 		// Get a type of component
 		$oEditorModel = getModel('editor');
 		$oModuleModel = getModel('module');
 		$editor_config = $oModuleModel->getModuleConfig('editor');
 
-		if(!$editor_config)
+		if (!$editor_config)
 		{
 			$editor_config = new stdClass();
 		}
 
 		//editor_config init
-		if(!$editor_config->editor_height) $editor_config->editor_height = 300;
-		if(!$editor_config->comment_editor_height) $editor_config->comment_editor_height = 100;
-		if(!$editor_config->editor_skin) $editor_config->editor_skin = 'ckeditor';
-		if(!$editor_config->comment_editor_skin) $editor_config->comment_editor_skin = 'ckeditor';
-		if(!$editor_config->sel_editor_colorset) $editor_config->sel_editor_colorset= 'moono';
-		if(!$editor_config->sel_comment_editor_colorset) $editor_config->sel_comment_editor_colorset= 'moono';
+		if (!$editor_config->editor_height) $editor_config->editor_height = 300;
+		if (!$editor_config->comment_editor_height) $editor_config->comment_editor_height = 100;
+		if (!$editor_config->editor_skin) $editor_config->editor_skin = 'ckeditor';
+		if (!$editor_config->comment_editor_skin) $editor_config->comment_editor_skin = 'ckeditor';
+		if (!$editor_config->sel_editor_colorset) $editor_config->sel_editor_colorset = 'moono';
+		if (!$editor_config->sel_comment_editor_colorset) $editor_config->sel_comment_editor_colorset = 'moono';
 
 		$component_list = $oEditorModel->getComponentList(false, $site_srl, true);
 		$editor_skin_list = FileHandler::readDir(_XE_PATH_.'modules/editor/skins');
 
-		$skin_info = $oModuleModel->loadSkinInfo($this->module_path,$editor_config->editor_skin);
+		$skin_info = $oModuleModel->loadSkinInfo($this->module_path, $editor_config->editor_skin);
 
 		$contents = FileHandler::readDir(_XE_PATH_.'modules/editor/styles');
 		$content_style_list = array();
-		for($i=0,$c=count($contents);$i<$c;$i++)
+		for ($i = 0, $c = count($contents); $i < $c; $i++)
 		{
 			$style = $contents[$i];
-			$info = $oModuleModel->loadSkinInfo($this->module_path,$style,'styles');
+			$info = $oModuleModel->loadSkinInfo($this->module_path, $style, 'styles');
 			$content_style_list[$style] = new stdClass();
 			$content_style_list[$style]->title = $info->title;
 		}
 
 		// Get install info, update info, count
 		$oAutoinstallModel = getModel('autoinstall');
-		foreach($component_list as $component_name => $xml_info)
+		foreach ($component_list as $component_name => $xml_info)
 		{
 			$component_count++;
 			$xml_info->path = './modules/editor/components/'.$xml_info->component_name;
 			$xml_info->delete_url = $oAutoinstallModel->getRemoveUrlByPath($xml_info->path);
 			$xml_info->package_srl = $oAutoinstallModel->getPackageSrlByPath($xml_info->path);
-			if($xml_info->package_srl) $targetpackages[$xml_info->package_srl] = 0;
+			if ($xml_info->package_srl) $targetpackages[$xml_info->package_srl] = 0;
 		}
 
-		if(is_array($targetpackages))	$packages = $oAutoinstallModel->getInstalledPackages(array_keys($targetpackages));
+		if (is_array($targetpackages))	$packages = $oAutoinstallModel->getInstalledPackages(array_keys($targetpackages));
 
-		foreach($component_list as $component_name => $xml_info)
+		foreach ($component_list as $component_name => $xml_info)
 		{
-			if($packages[$xml_info->package_srl])	$xml_info->need_update = $packages[$xml_info->package_srl]->need_update;
+			if ($packages[$xml_info->package_srl])	$xml_info->need_update = $packages[$xml_info->package_srl]->need_update;
 		}
-		$editor_config_default = array( "editor_height" => "300", "comment_editor_height" => "100","content_font_size"=>"13");
+		$editor_config_default = array("editor_height" => "300", "comment_editor_height" => "100", "content_font_size"=>"13");
 
 		//editor preview
 		$config = $oEditorModel->getEditorConfig();
@@ -138,12 +138,12 @@ 
 	function dispEditorAdminSetupComponent()
 	{
 		$site_module_info = Context::get('site_module_info');
-		$site_srl = (int)$site_module_info->site_srl;
+		$site_srl = (int) $site_module_info->site_srl;
 
 		$component_name = Context::get('component_name');
 		// Get information of the editor component
 		$oEditorModel = getModel('editor');
-		$component = $oEditorModel->getComponent($component_name,$site_srl);
+		$component = $oEditorModel->getComponent($component_name, $site_srl);
 		Context::set('component', $component);
 		// Get a group list to set a group
 		$oMemberModel = getModel('member');
@@ -152,20 +152,20 @@ 
 		// Get a mid list
 		$oModuleModel = getModel('module');
 
-		$args =new stdClass();
+		$args = new stdClass();
 		$args->site_srl = $site_srl;
 		$columnList = array('module_srl', 'mid', 'module_category_srl', 'browser_title');
 		$mid_list = $oModuleModel->getMidList($args, $columnList);
 		// Combination of module_category and module
-		if(!$args->site_srl)
+		if (!$args->site_srl)
 		{
 			// Get a list of module category
 			$module_categories = $oModuleModel->getModuleCategories();
 
-			if(!is_array($mid_list)) $mid_list = array($mid_list);
-			foreach($mid_list as $module_srl => $module)
+			if (!is_array($mid_list)) $mid_list = array($mid_list);
+			foreach ($mid_list as $module_srl => $module)
 			{
-				if($module) $module_categories[$module->module_category_srl]->list[$module_srl] = $module; 
+				if ($module) $module_categories[$module->module_category_srl]->list[$module_srl] = $module; 
 			}
 		}
 		else
@@ -173,13 +173,13 @@ 
 			$module_categories[0]->list = $mid_list;
 		}
 
-		Context::set('mid_list',$module_categories);
+		Context::set('mid_list', $module_categories);
 
 		//Security
 		$security = new Security();
 		$security->encodeHTML('group_list..title');
 		$security->encodeHTML('component...');
-		$security->encodeHTML('mid_list..title','mid_list..list..browser_title');
+		$security->encodeHTML('mid_list..title', 'mid_list..list..browser_title');
 
 		$this->setTemplatePath($this->module_path.'tpl');
 		$this->setTemplateFile('setup_component');

modules/editor/editor.class.php

Spacing +27 added lines, -27 removed lines

@@ -16,16 +16,16 @@ 
 		$oModuleController = getController('module');
 		// Add the default editor component
 		$oEditorController = getAdminController('editor');
-		$oEditorController->insertComponent('colorpicker_text',true);
-		$oEditorController->insertComponent('colorpicker_bg',true);
-		$oEditorController->insertComponent('emoticon',true);
-		$oEditorController->insertComponent('url_link',true);
-		$oEditorController->insertComponent('image_link',true);
-		$oEditorController->insertComponent('multimedia_link',true);
-		$oEditorController->insertComponent('quotation',true);
-		$oEditorController->insertComponent('table_maker',true);
-		$oEditorController->insertComponent('poll_maker',true);
-		$oEditorController->insertComponent('image_gallery',true);
+		$oEditorController->insertComponent('colorpicker_text', true);
+		$oEditorController->insertComponent('colorpicker_bg', true);
+		$oEditorController->insertComponent('emoticon', true);
+		$oEditorController->insertComponent('url_link', true);
+		$oEditorController->insertComponent('image_link', true);
+		$oEditorController->insertComponent('multimedia_link', true);
+		$oEditorController->insertComponent('quotation', true);
+		$oEditorController->insertComponent('table_maker', true);
+		$oEditorController->insertComponent('poll_maker', true);
+		$oEditorController->insertComponent('image_gallery', true);
 		// Create a directory to use in the editor module
 		FileHandler::makeDir('./files/cache/editor');
 		// 2007. 10. 17 Add a trigger to delete automatically saved document whenever the document(insert or update) is modified
@@ -48,21 +48,21 @@ 
 
 		$oDB = &DB::getInstance();
 		// 2009. 06. 15 Save module_srl when auto-saving
-		if(!$oDB->isColumnExists("editor_autosave","module_srl")) return true;
-		if(!$oDB->isIndexExists("editor_autosave","idx_module_srl")) return true;
+		if (!$oDB->isColumnExists("editor_autosave", "module_srl")) return true;
+		if (!$oDB->isIndexExists("editor_autosave", "idx_module_srl")) return true;
 
 		// 2007. 10. 17 Add a trigger to delete automatically saved document whenever the document(insert or update) is modified
-		if(!$oModuleModel->getTrigger('document.insertDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) return true;
-		if(!$oModuleModel->getTrigger('document.updateDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) return true;
+		if (!$oModuleModel->getTrigger('document.insertDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) return true;
+		if (!$oModuleModel->getTrigger('document.updateDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) return true;
 		// 2007. 10. 23 Add an editor trigger on the module addition setup
-		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'editor', 'view', 'triggerDispEditorAdditionSetup', 'before')) return true;
+		if (!$oModuleModel->getTrigger('module.dispAdditionSetup', 'editor', 'view', 'triggerDispEditorAdditionSetup', 'before')) return true;
 		// 2009. 04. 14 Add a trigger from compiled codes of the editor component
-		if(!$oModuleModel->getTrigger('display', 'editor', 'controller', 'triggerEditorComponentCompile', 'before')) return true;
+		if (!$oModuleModel->getTrigger('display', 'editor', 'controller', 'triggerEditorComponentCompile', 'before')) return true;
 		// 2009. 06. 19 Remove unused trigger
-		if($oModuleModel->getTrigger('file.getIsPermitted', 'editor', 'controller', 'triggerSrlSetting', 'before')) return true;
+		if ($oModuleModel->getTrigger('file.getIsPermitted', 'editor', 'controller', 'triggerSrlSetting', 'before')) return true;
 
 		// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-		if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'editor', 'controller', 'triggerCopyModule', 'after')) return true;
+		if (!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'editor', 'controller', 'triggerCopyModule', 'after')) return true;
 
 		return false;
 	}
@@ -77,29 +77,29 @@ 
 
 		$oDB = &DB::getInstance();
 		// Save module_srl when auto-saving 15/06/2009
-		if(!$oDB->isColumnExists("editor_autosave","module_srl")) 
-			$oDB->addColumn("editor_autosave","module_srl","number",11);
+		if (!$oDB->isColumnExists("editor_autosave", "module_srl")) 
+			$oDB->addColumn("editor_autosave", "module_srl", "number", 11);
 
 		// create an index on module_srl
-		if(!$oDB->isIndexExists("editor_autosave","idx_module_srl")) $oDB->addIndex("editor_autosave","idx_module_srl", "module_srl");
+		if (!$oDB->isIndexExists("editor_autosave", "idx_module_srl")) $oDB->addIndex("editor_autosave", "idx_module_srl", "module_srl");
 
 		// 2007. 10. 17 Add a trigger to delete automatically saved document whenever the document(insert or update) is modified
-		if(!$oModuleModel->getTrigger('document.insertDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) 
+		if (!$oModuleModel->getTrigger('document.insertDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) 
 			$oModuleController->insertTrigger('document.insertDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after');
-		if(!$oModuleModel->getTrigger('document.updateDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) 
+		if (!$oModuleModel->getTrigger('document.updateDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) 
 			$oModuleController->insertTrigger('document.updateDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after');
 		// 2007. 10. Add an editor trigger on the module addition setup
-		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'editor', 'view', 'triggerDispEditorAdditionSetup', 'before')) 
+		if (!$oModuleModel->getTrigger('module.dispAdditionSetup', 'editor', 'view', 'triggerDispEditorAdditionSetup', 'before')) 
 			$oModuleController->insertTrigger('module.dispAdditionSetup', 'editor', 'view', 'triggerDispEditorAdditionSetup', 'before');
 		// 2009. 04. 14 Add a trigger from compiled codes of the editor component
-		if(!$oModuleModel->getTrigger('display', 'editor', 'controller', 'triggerEditorComponentCompile', 'before')) 
+		if (!$oModuleModel->getTrigger('display', 'editor', 'controller', 'triggerEditorComponentCompile', 'before')) 
 			$oModuleController->insertTrigger('display', 'editor', 'controller', 'triggerEditorComponentCompile', 'before');
 		// 2009. 06. 19 Remove unused trigger
-		if($oModuleModel->getTrigger('file.getIsPermitted', 'editor', 'controller', 'triggerSrlSetting', 'before')) 
+		if ($oModuleModel->getTrigger('file.getIsPermitted', 'editor', 'controller', 'triggerSrlSetting', 'before')) 
 			$oModuleController->deleteTrigger('file.getIsPermitted', 'editor', 'controller', 'triggerSrlSetting', 'before');
 
 		// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-		if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'editor', 'controller', 'triggerCopyModule', 'after'))
+		if (!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'editor', 'controller', 'triggerCopyModule', 'after'))
 		{
 			$oModuleController->insertTrigger('module.procModuleAdminCopyModule', 'editor', 'controller', 'triggerCopyModule', 'after');
 		}

Braces +45 added lines, -21 removed lines

@@ -48,21 +48,37 @@ 
 
 		$oDB = &DB::getInstance();
 		// 2009. 06. 15 Save module_srl when auto-saving
-		if(!$oDB->isColumnExists("editor_autosave","module_srl")) return true;
-		if(!$oDB->isIndexExists("editor_autosave","idx_module_srl")) return true;
+		if(!$oDB->isColumnExists("editor_autosave","module_srl")) {
+			return true;
+		}
+		if(!$oDB->isIndexExists("editor_autosave","idx_module_srl")) {
+			return true;
+		}
 
 		// 2007. 10. 17 Add a trigger to delete automatically saved document whenever the document(insert or update) is modified
-		if(!$oModuleModel->getTrigger('document.insertDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) return true;
-		if(!$oModuleModel->getTrigger('document.updateDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) return true;
+		if(!$oModuleModel->getTrigger('document.insertDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) {
+			return true;
+		}
+		if(!$oModuleModel->getTrigger('document.updateDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) {
+			return true;
+		}
 		// 2007. 10. 23 Add an editor trigger on the module addition setup
-		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'editor', 'view', 'triggerDispEditorAdditionSetup', 'before')) return true;
+		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'editor', 'view', 'triggerDispEditorAdditionSetup', 'before')) {
+			return true;
+		}
 		// 2009. 04. 14 Add a trigger from compiled codes of the editor component
-		if(!$oModuleModel->getTrigger('display', 'editor', 'controller', 'triggerEditorComponentCompile', 'before')) return true;
+		if(!$oModuleModel->getTrigger('display', 'editor', 'controller', 'triggerEditorComponentCompile', 'before')) {
+			return true;
+		}
 		// 2009. 06. 19 Remove unused trigger
-		if($oModuleModel->getTrigger('file.getIsPermitted', 'editor', 'controller', 'triggerSrlSetting', 'before')) return true;
+		if($oModuleModel->getTrigger('file.getIsPermitted', 'editor', 'controller', 'triggerSrlSetting', 'before')) {
+			return true;
+		}
 
 		// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
-		if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'editor', 'controller', 'triggerCopyModule', 'after')) return true;
+		if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'editor', 'controller', 'triggerCopyModule', 'after')) {
+			return true;
+		}
 
 		return false;
 	}
@@ -77,26 +93,34 @@ 
 
 		$oDB = &DB::getInstance();
 		// Save module_srl when auto-saving 15/06/2009
-		if(!$oDB->isColumnExists("editor_autosave","module_srl")) 
-			$oDB->addColumn("editor_autosave","module_srl","number",11);
+		if(!$oDB->isColumnExists("editor_autosave","module_srl")) {
+					$oDB->addColumn("editor_autosave","module_srl","number",11);
+		}
 
 		// create an index on module_srl
-		if(!$oDB->isIndexExists("editor_autosave","idx_module_srl")) $oDB->addIndex("editor_autosave","idx_module_srl", "module_srl");
+		if(!$oDB->isIndexExists("editor_autosave","idx_module_srl")) {
+			$oDB->addIndex("editor_autosave","idx_module_srl", "module_srl");
+		}
 
 		// 2007. 10. 17 Add a trigger to delete automatically saved document whenever the document(insert or update) is modified
-		if(!$oModuleModel->getTrigger('document.insertDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) 
-			$oModuleController->insertTrigger('document.insertDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after');
-		if(!$oModuleModel->getTrigger('document.updateDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) 
-			$oModuleController->insertTrigger('document.updateDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after');
+		if(!$oModuleModel->getTrigger('document.insertDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) {
+					$oModuleController->insertTrigger('document.insertDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after');
+		}
+		if(!$oModuleModel->getTrigger('document.updateDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after')) {
+					$oModuleController->insertTrigger('document.updateDocument', 'editor', 'controller', 'triggerDeleteSavedDoc', 'after');
+		}
 		// 2007. 10. Add an editor trigger on the module addition setup
-		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'editor', 'view', 'triggerDispEditorAdditionSetup', 'before')) 
-			$oModuleController->insertTrigger('module.dispAdditionSetup', 'editor', 'view', 'triggerDispEditorAdditionSetup', 'before');
+		if(!$oModuleModel->getTrigger('module.dispAdditionSetup', 'editor', 'view', 'triggerDispEditorAdditionSetup', 'before')) {
+					$oModuleController->insertTrigger('module.dispAdditionSetup', 'editor', 'view', 'triggerDispEditorAdditionSetup', 'before');
+		}
 		// 2009. 04. 14 Add a trigger from compiled codes of the editor component
-		if(!$oModuleModel->getTrigger('display', 'editor', 'controller', 'triggerEditorComponentCompile', 'before')) 
-			$oModuleController->insertTrigger('display', 'editor', 'controller', 'triggerEditorComponentCompile', 'before');
+		if(!$oModuleModel->getTrigger('display', 'editor', 'controller', 'triggerEditorComponentCompile', 'before')) {
+					$oModuleController->insertTrigger('display', 'editor', 'controller', 'triggerEditorComponentCompile', 'before');
+		}
 		// 2009. 06. 19 Remove unused trigger
-		if($oModuleModel->getTrigger('file.getIsPermitted', 'editor', 'controller', 'triggerSrlSetting', 'before')) 
-			$oModuleController->deleteTrigger('file.getIsPermitted', 'editor', 'controller', 'triggerSrlSetting', 'before');
+		if($oModuleModel->getTrigger('file.getIsPermitted', 'editor', 'controller', 'triggerSrlSetting', 'before')) {
+					$oModuleController->deleteTrigger('file.getIsPermitted', 'editor', 'controller', 'triggerSrlSetting', 'before');
+		}
 
 		// 2012. 08. 29 Add a trigger to copy additional setting when the module is copied 
 		if(!$oModuleModel->getTrigger('module.procModuleAdminCopyModule', 'editor', 'controller', 'triggerCopyModule', 'after'))

modules/editor/editor.view.php

Braces +5 added lines, -5 removed lines

@@ -36,8 +36,7 @@ 
 			Context::set('message', sprintf(Context::getLang('msg_component_is_not_founded'), $component));
 			$this->setTemplatePath($this->module_path.'tpl');
 			$this->setTemplateFile('component_not_founded');
-		}
-		else
+		} else
 		{
 			// Get the result after executing a method to display popup url of the component
 			$popup_content = $oComponent->getPopupContent();
@@ -82,7 +81,9 @@ 
 			// Get information of the current module
 			$current_module_info = Context::get('current_module_info');
 			$current_module_srl = $current_module_info->module_srl;
-			if(!$current_module_srl) return new Object();
+			if(!$current_module_srl) {
+				return new Object();
+			}
 		}
 		// Get editors settings
 		$oEditorModel = getModel('editor');
@@ -172,8 +173,7 @@ 
 			$option_com->content_style = $config->comment_content_style;
 			$option_com->colorset = $config->sel_comment_editor_colorset;
 			$editor = $oEditorModel->getEditor(0, $option_com);
-		}
-		else
+		} else
 		{
 			$option = new stdClass();
 			$option->allow_fileupload = false;

Spacing +11 added lines, -11 removed lines

@@ -27,11 +27,11 @@ 
 		$component = Context::get('component');
 
 		$site_module_info = Context::get('site_module_info');
-		$site_srl = (int)$site_module_info->site_srl;
+		$site_srl = (int) $site_module_info->site_srl;
 		// Get compoenet object
 		$oEditorModel = getModel('editor');
 		$oComponent = &$oEditorModel->getComponentObject($component, $editor_sequence, $site_srl);
-		if(!$oComponent->toBool())
+		if (!$oComponent->toBool())
 		{
 			Context::set('message', sprintf(Context::getLang('msg_component_is_not_founded'), $component));
 			$this->setTemplatePath($this->module_path.'tpl');
@@ -58,7 +58,7 @@ 
 		$component_name = Context::get('component_name');
 
 		$site_module_info = Context::get('site_module_info');
-		$site_srl = (int)$site_module_info->site_srl;
+		$site_srl = (int) $site_module_info->site_srl;
 
 		$oEditorModel = getModel('editor');
 		$component = $oEditorModel->getComponent($component_name, $site_srl);
@@ -77,12 +77,12 @@ 
 		$current_module_srl = Context::get('module_srl');
 		$current_module_srls = Context::get('module_srls');
 
-		if(!$current_module_srl && !$current_module_srls)
+		if (!$current_module_srl && !$current_module_srls)
 		{
 			// Get information of the current module
 			$current_module_info = Context::get('current_module_info');
 			$current_module_srl = $current_module_info->module_srl;
-			if(!$current_module_srl) return new Object();
+			if (!$current_module_srl) return new Object();
 		}
 		// Get editors settings
 		$oEditorModel = getModel('editor');
@@ -95,17 +95,17 @@ 
 		$editor_skin_list = FileHandler::readDir(_XE_PATH_.'modules/editor/skins');
 		Context::set('editor_skin_list', $editor_skin_list);
 
-		$skin_info = $oModuleModel->loadSkinInfo($this->module_path,$editor_config->editor_skin);
+		$skin_info = $oModuleModel->loadSkinInfo($this->module_path, $editor_config->editor_skin);
 		Context::set('editor_colorset_list', $skin_info->colorset);
-		$skin_info = $oModuleModel->loadSkinInfo($this->module_path,$editor_config->comment_editor_skin);
+		$skin_info = $oModuleModel->loadSkinInfo($this->module_path, $editor_config->comment_editor_skin);
 		Context::set('editor_comment_colorset_list', $skin_info->colorset);
 
 		$contents = FileHandler::readDir(_XE_PATH_.'modules/editor/styles');
 		$content_style_list = array();
-		for($i=0,$c=count($contents);$i<$c;$i++)
+		for ($i = 0, $c = count($contents); $i < $c; $i++)
 		{
 			$style = $contents[$i];
-			$info = $oModuleModel->loadSkinInfo($this->module_path,$style,'styles');
+			$info = $oModuleModel->loadSkinInfo($this->module_path, $style, 'styles');
 			$content_style_list[$style] = new stdClass();
 			$content_style_list[$style]->title = $info->title;
 		}			
@@ -142,7 +142,7 @@ 
 	{
 		$skin = Context::get('skin');
 		$oModuleModel = getModel('module');
-		$skin_info = $oModuleModel->loadSkinInfo($this->module_path,$skin);
+		$skin_info = $oModuleModel->loadSkinInfo($this->module_path, $skin);
 		$colorset = $skin_info->colorset;
 		Context::set('colorset', $colorset);
 	}
@@ -154,7 +154,7 @@ 
 
 		$mode = Context::get('mode');
 
-		if($mode != 'main')
+		if ($mode != 'main')
 		{
 			$option_com = new stdClass();
 			$option_com->allow_fileupload = false;