pradosoft / prado

framework/Web/UI/WebControls/THtmlArea4.php

Doc Comments +2 added lines

@@ -422,6 +422,7 @@ 
 
 	/**
 	 * Parse additional options set in the Options property.
+	 * @param string $string
 	 * @return array additional custom options
 	 */
 	protected function parseEditorOptions($string)
@@ -446,6 +447,7 @@ 
 	}
 
 	/**
+	 * @param string $culture
 	 * @return string localized editor interface language extension.
 	 */
 	protected function getLanguageSuffix($culture)

Spacing +43 added lines, -43 removed lines

@@ -62,7 +62,7 @@ 
 	/**
 	 * @var array list of locale => language file pairs.
 	 */
-	private static $_langs = array(
+	private static $_langs=array(
 			'ar' => 'ar',
 			'bg_BG' => 'bg_BG',
 			'bs' => 'bs',
@@ -120,7 +120,7 @@ 
 	/**
 	 * @var array list of default plugins to load, override using getAvailablePlugins();
 	 */
-	private static $_plugins = array(
+	private static $_plugins=array(
 		'advlist',
 		'anchor',
 		'autolink',
@@ -164,7 +164,7 @@ 
 	/**
 	 * @var array default themes to load
 	 */
-	private static $_themes = array(
+	private static $_themes=array(
 		'modern',
 	);
 
@@ -211,7 +211,7 @@ 
 	 */
 	public function getEnableVisualEdit()
 	{
-		return $this->getViewState('EnableVisualEdit',true);
+		return $this->getViewState('EnableVisualEdit', true);
 	}
 
 	/**
@@ -220,7 +220,7 @@ 
 	 */
 	public function setEnableVisualEdit($value)
 	{
-		$this->setViewState('EnableVisualEdit',TPropertyValue::ensureBoolean($value),true);
+		$this->setViewState('EnableVisualEdit', TPropertyValue::ensureBoolean($value), true);
 	}
 
 	/**
@@ -305,7 +305,7 @@ 
 	{
 		if($this->getEnableVisualEdit() && $this->getEnabled(true))
 		{
-			$writer->addAttribute('id',$this->getClientID());
+			$writer->addAttribute('id', $this->getClientID());
 			$this->registerEditorClientScript($writer);
 		}
 
@@ -340,7 +340,7 @@ 
 
 	protected function loadJavascriptLibrary()
 	{
-		$scripts = $this->getPage()->getClientScript();
+		$scripts=$this->getPage()->getClientScript();
 		$scripts->registerPradoScript('htmlarea4');
 		$this->copyCustomPlugins();
 	}
@@ -351,25 +351,25 @@ 
 	protected function registerEditorClientScript($writer)
 	{
 		$this->loadJavascriptLibrary();
-		$scripts = $this->getPage()->getClientScript();
-		$options = array(
+		$scripts=$this->getPage()->getClientScript();
+		$options=array(
 			'EditorOptions' => $this->getEditorOptions()
 		);
 
-		$options = TJavaScript::encode($options,true,true);
-		$script = "new {$this->getClientClassName()}($options)";
-		$scripts->registerEndScript('prado:THtmlArea4'.$this->ClientID,$script);
+		$options=TJavaScript::encode($options, true, true);
+		$script="new {$this->getClientClassName()}($options)";
+		$scripts->registerEndScript('prado:THtmlArea4'.$this->ClientID, $script);
 	}
 
 	protected function copyCustomPlugins()
 	{
-		if($plugins = $this->getCustomPluginPath())
+		if($plugins=$this->getCustomPluginPath())
 		{
-			$basepath = $this->getPage()->getClientScript()->getPradoScriptAssetPath('tinymce');
-			$assets = $this->getApplication()->getAssetManager();
-			$path = is_dir($plugins) ? $plugins : Prado::getPathOfNameSpace($plugins);
-			$name = basename($path);
-			$dest = $basepath.'/plugins/'.$name;
+			$basepath=$this->getPage()->getClientScript()->getPradoScriptAssetPath('tinymce');
+			$assets=$this->getApplication()->getAssetManager();
+			$path=is_dir($plugins) ? $plugins : Prado::getPathOfNameSpace($plugins);
+			$name=basename($path);
+			$dest=$basepath.'/plugins/'.$name;
 			if(!is_dir($dest) || $this->getApplication()->getMode()!==TApplicationMode::Performance)
 				$assets->copyDirectory($path, $dest);
 		}
@@ -381,26 +381,26 @@ 
 	 */
 	protected function getEditorOptions()
 	{
-		$options['mode'] = 'exact';
-		$options['elements'] = $this->getClientID();
-		$options['language'] = $this->getLanguageSuffix($this->getCulture());
+		$options['mode']='exact';
+		$options['elements']=$this->getClientID();
+		$options['language']=$this->getLanguageSuffix($this->getCulture());
 		//$options['theme'] = 'modern'; //default
 		// mimic previous (tinyMCE3) sizing behaviour
-		$options['width'] = $this->getWidth();
-		$options['height'] = $this->getHeight();
-		$options['resize'] = 'both';
-		$options['menubar'] = false;
+		$options['width']=$this->getWidth();
+		$options['height']=$this->getHeight();
+		$options['resize']='both';
+		$options['menubar']=false;
 		if($this->getReadOnly())
 		{
-			$options['readonly'] = true;
-			$options['toolbar'] = false;
-			$options['menubar'] = false;
-			$options['statusbar'] = false;
+			$options['readonly']=true;
+			$options['toolbar']=false;
+			$options['menubar']=false;
+			$options['statusbar']=false;
 		}
 
-		$options['extended_valid_elements'] = 'a[name|href|target|title|onclick],img[class|src|border=0|alt|title|hspace|vspace|width|height|align|onmouseover|onmouseout|name],hr[class|width|size|noshade],font[face|size|color|style],span[class|align|style]';
+		$options['extended_valid_elements']='a[name|href|target|title|onclick],img[class|src|border=0|alt|title|hspace|vspace|width|height|align|onmouseover|onmouseout|name],hr[class|width|size|noshade],font[face|size|color|style],span[class|align|style]';
 
-		$options = array_merge($options, $this->parseEditorOptions($this->getOptions()));
+		$options=array_merge($options, $this->parseEditorOptions($this->getOptions()));
 		return $options;
 	}
 
@@ -410,20 +410,20 @@ 
 	 */
 	protected function parseEditorOptions($string)
 	{
-		$options = array();
-		$substrings = preg_split('/,\s*\n|\n/', trim($string));
+		$options=array();
+		$substrings=preg_split('/,\s*\n|\n/', trim($string));
 		foreach($substrings as $bits)
 		{
-			$option = explode(":",$bits,2);
+			$option=explode(":", $bits, 2);
 
-			if(count($option) == 2)
+			if(count($option)==2)
 			{
-				$value=trim(trim($option[1]),"'\"");
-				if (($s=strtolower($value))==='false')
+				$value=trim(trim($option[1]), "'\"");
+				if(($s=strtolower($value))==='false')
 					$value=false;
-				elseif ($s==='true')
+				elseif($s==='true')
 					$value=true;
-				$options[trim($option[0])] = $value;
+				$options[trim($option[0])]=$value;
 			}
 		}
 		return $options;
@@ -434,12 +434,12 @@ 
 	 */
 	protected function getLanguageSuffix($culture)
 	{
-		$app = $this->getApplication()->getGlobalization();
+		$app=$this->getApplication()->getGlobalization();
 		if(empty($culture) && ($app!==null))
-			$culture = $app->getCulture();
-		$variants = array();
+			$culture=$app->getCulture();
+		$variants=array();
 		if($app!==null)
-			$variants = $app->getCultureVariants($culture);
+			$variants=$app->getCultureVariants($culture);
 
 		foreach($variants as $variant)
 		{

framework/Web/UI/TClientScriptManager.php

Spacing +110 added lines, -110 removed lines

@@ -159,7 +159,7 @@ 
 	{
 		$this->registerPradoScriptInternal($name);
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerPradoScript',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerPradoScript', $params);
 	}
 
 	/**
@@ -170,42 +170,42 @@ 
 		// $this->checkIfNotInRender();
 		if(!isset($this->_registeredScripts[$name]))
 		{
-			if(self::$_scripts === null)
+			if(self::$_scripts===null)
 			{
-				$packageFile = Prado::getFrameworkPath().DIRECTORY_SEPARATOR.self::PACKAGES_FILE;
-				list($folders, $packages, $deps)= include($packageFile);
-				self::$_scriptsFolders = $folders;
-				self::$_scripts = $deps;
-				self::$_scriptsPackages = $packages;
+				$packageFile=Prado::getFrameworkPath().DIRECTORY_SEPARATOR.self::PACKAGES_FILE;
+				list($folders, $packages, $deps)=include($packageFile);
+				self::$_scriptsFolders=$folders;
+				self::$_scripts=$deps;
+				self::$_scriptsPackages=$packages;
 			}
 
-			if (isset(self::$_scripts[$name]))
+			if(isset(self::$_scripts[$name]))
 				$this->_registeredScripts[$name]=true;
 			else
-				throw new TInvalidOperationException('csmanager_pradoscript_invalid',$name);
+				throw new TInvalidOperationException('csmanager_pradoscript_invalid', $name);
 
 			if(($packages=array_keys($this->_registeredScripts))!==array())
 			{
 				$packagesUrl=array();
 				$isDebug=$this->getApplication()->getMode()===TApplicationMode::Debug;
-				foreach ($packages as $p)
+				foreach($packages as $p)
 				{
-					foreach (self::$_scripts[$p] as $dep)
+					foreach(self::$_scripts[$p] as $dep)
 					{
-						foreach (self::$_scriptsPackages[$dep] as $script)
+						foreach(self::$_scriptsPackages[$dep] as $script)
 						{
-							if (!isset($this->_expandedScripts[$script]))
+							if(!isset($this->_expandedScripts[$script]))
 							{
-								list($base, $subPath) = $this->getScriptPackageFolder($script);
-								list($path, $baseUrl) = $this->getPackagePathUrl($base);
+								list($base, $subPath)=$this->getScriptPackageFolder($script);
+								list($path, $baseUrl)=$this->getPackagePathUrl($base);
 
-								$this->_expandedScripts[$script] = true;
+								$this->_expandedScripts[$script]=true;
 								if($isDebug)
 								{
-									if (!in_array($url=$baseUrl.'/'.$subPath, $packagesUrl))
+									if(!in_array($url=$baseUrl.'/'.$subPath, $packagesUrl))
 										$packagesUrl[]=$url;
 								} else {
-									if (!in_array($url=$baseUrl.'/min/'.$subPath, $packagesUrl))
+									if(!in_array($url=$baseUrl.'/min/'.$subPath, $packagesUrl))
 									{
 										if(!is_file($filePath=$path.'/min/'.$subPath))
 										{
@@ -223,7 +223,7 @@ 
 					}
 				}
 				foreach($packagesUrl as $url)
-					$this->registerScriptFile($url,$url);
+					$this->registerScriptFile($url, $url);
 			}
 		}
 	}
@@ -234,10 +234,10 @@ 
 	public function getPradoScriptAssetUrl($script='prado')
 	{
 		if(!array_key_exists($script, self::$_scriptsFolders))
-			throw new TInvalidOperationException('csmanager_pradostyle_invalid',$script);
+			throw new TInvalidOperationException('csmanager_pradostyle_invalid', $script);
 		
-		$base = Prado::getPathOfNameSpace(self::$_scriptsFolders[$script]);
-		$assets = Prado::getApplication()->getAssetManager();
+		$base=Prado::getPathOfNameSpace(self::$_scriptsFolders[$script]);
+		$assets=Prado::getApplication()->getAssetManager();
 		return $assets->getPublishedUrl($base);
 	}
 
@@ -247,10 +247,10 @@ 
 	public function getPradoScriptAssetPath($script='prado')
 	{
 		if(!array_key_exists($script, self::$_scriptsFolders))
-			throw new TInvalidOperationException('csmanager_pradostyle_invalid',$script);
+			throw new TInvalidOperationException('csmanager_pradostyle_invalid', $script);
 		
-		$base = Prado::getPathOfNameSpace(self::$_scriptsFolders[$script]);
-		$assets = Prado::getApplication()->getAssetManager();
+		$base=Prado::getPathOfNameSpace(self::$_scriptsFolders[$script]);
+		$assets=Prado::getApplication()->getAssetManager();
 		return $assets->getPublishedPath($base);
 	}
 
@@ -260,9 +260,9 @@ 
 	 */
 	public function getScriptUrls()
 	{
-		$scripts = array_values($this->_headScriptFiles);
-		$scripts = array_merge($scripts, array_values($this->_scriptFiles));
-		$scripts = array_unique($scripts);
+		$scripts=array_values($this->_headScriptFiles);
+		$scripts=array_merge($scripts, array_values($this->_scriptFiles));
+		$scripts=array_unique($scripts);
 
 		return $scripts;
 	}
@@ -273,17 +273,17 @@ 
 	 */
 	protected function getPackagePathUrl($base)
 	{
-		$assets = Prado::getApplication()->getAssetManager();
+		$assets=Prado::getApplication()->getAssetManager();
 		if(strpos($base, $assets->getBaseUrl())===false)
 		{
-			if(($dir = Prado::getPathOfNameSpace($base)) !== null) {
-				$base = $dir;
+			if(($dir=Prado::getPathOfNameSpace($base))!==null) {
+				$base=$dir;
 			}
 			return array($assets->getPublishedPath($base), $assets->publishFilePath($base));
 		}
 		else
 		{
-			return array($assets->getBasePath().str_replace($assets->getBaseUrl(),'',$base), $base);
+			return array($assets->getBasePath().str_replace($assets->getBaseUrl(), '', $base), $base);
 		}
 	}
 
@@ -293,10 +293,10 @@ 
 	 */
 	protected function getScriptPackageFolder($script)
 	{
-		list($base, $subPath) = explode("/", $script, 2);
+		list($base, $subPath)=explode("/", $script, 2);
 
 		if(!array_key_exists($base, self::$_scriptsFolders))
-			throw new TInvalidOperationException('csmanager_pradostyle_invalid',$base);
+			throw new TInvalidOperationException('csmanager_pradostyle_invalid', $base);
 		
 		return array(self::$_scriptsFolders[$base], $subPath);
 	}
@@ -307,10 +307,10 @@ 
 	 */
 	protected function getStylePackageFolder($script)
 	{
-		list($base, $subPath) = explode("/", $script, 2);
+		list($base, $subPath)=explode("/", $script, 2);
 
 		if(!array_key_exists($base, self::$_stylesFolders))
-			throw new TInvalidOperationException('csmanager_pradostyle_invalid',$base);
+			throw new TInvalidOperationException('csmanager_pradostyle_invalid', $base);
 		
 		return array(self::$_stylesFolders[$base], $subPath);
 	}
@@ -323,13 +323,13 @@ 
 	 */
 	public function getCallbackReference(ICallbackEventHandler $callbackHandler, $options=null)
 	{
-		$options = !is_array($options) ? array() : $options;
-		$class = new \ReflectionClass($callbackHandler);
-		$clientSide = $callbackHandler->getActiveControl()->getClientSide();
-		$options = array_merge($options, $clientSide->getOptions()->toArray());
-		$optionString = TJavaScript::encode($options);
+		$options=!is_array($options) ? array() : $options;
+		$class=new \ReflectionClass($callbackHandler);
+		$clientSide=$callbackHandler->getActiveControl()->getClientSide();
+		$options=array_merge($options, $clientSide->getOptions()->toArray());
+		$optionString=TJavaScript::encode($options);
 		$this->registerPradoScriptInternal('ajax');
-		$id = $callbackHandler->getUniqueID();
+		$id=$callbackHandler->getUniqueID();
 		return "new Prado.CallbackRequest('{$id}',{$optionString})";
 	}
 
@@ -346,7 +346,7 @@ 
 		$this->registerPradoScriptInternal('ajax');
 
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerCallbackControl',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerCallbackControl', $params);
 	}
 
 	/**
@@ -355,9 +355,9 @@ 
 	 * @param string javascript class responsible for the control being registered for postback
 	 * @param array postback options
 	 */
-	public function registerPostBackControl($class,$options)
+	public function registerPostBackControl($class, $options)
 	{
-		if($class === null) {
+		if($class===null) {
 			return;
 		}
 		if(!isset($options['FormID']) && ($form=$this->_page->getForm())!==null)
@@ -369,7 +369,7 @@ 
 		$this->registerPradoScriptInternal('prado');
 
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerPostBackControl',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerPostBackControl', $params);
 	}
 
 	/**
@@ -380,7 +380,7 @@ 
 	 */
 	public function registerDefaultButton($panel, $button)
 	{
-		$panelID=is_string($panel)?$panel:$panel->getUniqueID();
+		$panelID=is_string($panel) ? $panel : $panel->getUniqueID();
 
 		if(is_string($button))
 			$buttonID=$button;
@@ -389,14 +389,14 @@ 
 			$button->setIsDefaultButton(true);
 			$buttonID=$button->getUniqueID();
 		}
-		$options = TJavaScript::encode($this->getDefaultButtonOptions($panelID, $buttonID));
-		$code = "new Prado.WebUI.DefaultButton($options);";
+		$options=TJavaScript::encode($this->getDefaultButtonOptions($panelID, $buttonID));
+		$code="new Prado.WebUI.DefaultButton($options);";
 
 		$this->_endScripts['prado:'.$panelID]=$code;
 		$this->registerPradoScriptInternal('prado');
 
-		$params=array($panelID,$buttonID);
-		$this->_page->registerCachingAction('Page.ClientScript','registerDefaultButton',$params);
+		$params=array($panelID, $buttonID);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerDefaultButton', $params);
 	}
 
 	/**
@@ -406,11 +406,11 @@ 
 	 */
 	protected function getDefaultButtonOptions($panelID, $buttonID)
 	{
-		$options['ID'] = TControl::convertUniqueIdToClientId($panelID);
-		$options['Panel'] = TControl::convertUniqueIdToClientId($panelID);
-		$options['Target'] = TControl::convertUniqueIdToClientId($buttonID);
-		$options['EventTarget'] = $buttonID;
-		$options['Event'] = 'click';
+		$options['ID']=TControl::convertUniqueIdToClientId($panelID);
+		$options['Panel']=TControl::convertUniqueIdToClientId($panelID);
+		$options['Target']=TControl::convertUniqueIdToClientId($buttonID);
+		$options['EventTarget']=$buttonID;
+		$options['Event']='click';
 		return $options;
 	}
 
@@ -423,10 +423,10 @@ 
 		$this->registerPradoScriptInternal('jquery');
 		if($target instanceof TControl)
 			$target=$target->getClientID();
-		$this->_endScripts['prado:focus'] = 'jQuery(\'#'.$target.'\').focus();';
+		$this->_endScripts['prado:focus']='jQuery(\'#'.$target.'\').focus();';
 
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerFocusControl',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerFocusControl', $params);
 	}
 
 	/**
@@ -438,7 +438,7 @@ 
 	{
 		$this->registerPradoStyleInternal($name);
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerPradoStyle',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerPradoStyle', $params);
 	}
 
 	/**
@@ -449,47 +449,47 @@ 
 		// $this->checkIfNotInRender();
 		if(!isset($this->_registeredStyles[$name]))
 		{
-			$base = $this->getPradoScriptAssetUrl();
+			$base=$this->getPradoScriptAssetUrl();
 
-			if(self::$_styles === null)
+			if(self::$_styles===null)
 			{
-				$packageFile = Prado::getFrameworkPath().DIRECTORY_SEPARATOR.self::CSS_PACKAGES_FILE;
-				list($folders, $packages,$deps)= include($packageFile);
-				self::$_stylesFolders = $folders;
-				self::$_styles = $deps;
-				self::$_stylesPackages = $packages;
+				$packageFile=Prado::getFrameworkPath().DIRECTORY_SEPARATOR.self::CSS_PACKAGES_FILE;
+				list($folders, $packages, $deps)=include($packageFile);
+				self::$_stylesFolders=$folders;
+				self::$_styles=$deps;
+				self::$_stylesPackages=$packages;
 			}
 
-			if (isset(self::$_styles[$name]))
+			if(isset(self::$_styles[$name]))
 				$this->_registeredStyles[$name]=true;
 			else
-				throw new TInvalidOperationException('csmanager_pradostyle_invalid',$name);
+				throw new TInvalidOperationException('csmanager_pradostyle_invalid', $name);
 
 			if(($packages=array_keys($this->_registeredStyles))!==array())
 			{
 				$packagesUrl=array();
 				$isDebug=$this->getApplication()->getMode()===TApplicationMode::Debug;
-				foreach ($packages as $p)
+				foreach($packages as $p)
 				{
-					foreach (self::$_styles[$p] as $dep)
+					foreach(self::$_styles[$p] as $dep)
 					{
-						foreach (self::$_stylesPackages[$dep] as $style)
+						foreach(self::$_stylesPackages[$dep] as $style)
 						{
-							if (!isset($this->_expandedStyles[$style]))
+							if(!isset($this->_expandedStyles[$style]))
 							{
-								list($base, $subPath) = $this->getStylePackageFolder($style);
-								list($path, $baseUrl) = $this->getPackagePathUrl($base);
+								list($base, $subPath)=$this->getStylePackageFolder($style);
+								list($path, $baseUrl)=$this->getPackagePathUrl($base);
 
-								$this->_expandedStyles[$style] = true;
+								$this->_expandedStyles[$style]=true;
 								// TODO minify css?
-								if (!in_array($url=$baseUrl.'/'.$subPath, $packagesUrl))
+								if(!in_array($url=$baseUrl.'/'.$subPath, $packagesUrl))
 									$packagesUrl[]=$url;
 							}
 						}
 					}
 				}
 				foreach($packagesUrl as $url)
-					$this->registerStyleSheetFile($url,$url);
+					$this->registerStyleSheetFile($url, $url);
 			}
 		}
 	}
@@ -516,15 +516,15 @@ 
 	 * @param string URL to the CSS file
 	 * @param string media type of the CSS (such as 'print', 'screen', etc.). Defaults to empty, meaning the CSS applies to all media types.
 	 */
-	public function registerStyleSheetFile($key,$url,$media='')
+	public function registerStyleSheetFile($key, $url, $media='')
 	{
 		if($media==='')
 			$this->_styleSheetFiles[$key]=$url;
 		else
-			$this->_styleSheetFiles[$key]=array($url,$media);
+			$this->_styleSheetFiles[$key]=array($url, $media);
 
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerStyleSheetFile',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerStyleSheetFile', $params);
 	}
 
 	/**
@@ -532,12 +532,12 @@ 
 	 * @param string a unique key identifying the CSS block
 	 * @param string CSS block
 	 */
-	public function registerStyleSheet($key,$css,$media='')
+	public function registerStyleSheet($key, $css, $media='')
 	{
 		$this->_styleSheets[$key]=$css;
 
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerStyleSheet',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerStyleSheet', $params);
 	}
 
 	/**
@@ -546,17 +546,17 @@ 
 	 */
 	public function getStyleSheetUrls()
 	{
-		$stylesheets = array_values(
+		$stylesheets=array_values(
 			array_map(function($e) {
 				return is_array($e) ? $e[0] : $e;
 			}, $this->_styleSheetFiles)
 		);
 
 		foreach(Prado::getApplication()->getAssetManager()->getPublished() as $path=>$url)
-			if (substr($url,strlen($url)-4)=='.css')
-				$stylesheets[] = $url;
+			if(substr($url, strlen($url) - 4)=='.css')
+				$stylesheets[]=$url;
 
-		$stylesheets = array_unique($stylesheets);
+		$stylesheets=array_unique($stylesheets);
 
 		return $stylesheets;
 	}
@@ -575,13 +575,13 @@ 
 	 * @param string a unique key identifying the file
 	 * @param string URL to the javascript file
 	 */
-	public function registerHeadScriptFile($key,$url)
+	public function registerHeadScriptFile($key, $url)
 	{
 		$this->checkIfNotInRender();
 		$this->_headScriptFiles[$key]=$url;
 
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerHeadScriptFile',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerHeadScriptFile', $params);
 	}
 
 	/**
@@ -589,13 +589,13 @@ 
 	 * @param string a unique key identifying the script block
 	 * @param string javascript block
 	 */
-	public function registerHeadScript($key,$script)
+	public function registerHeadScript($key, $script)
 	{
 		$this->checkIfNotInRender();
 		$this->_headScripts[$key]=$script;
 
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerHeadScript',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerHeadScript', $params);
 	}
 
 	/**
@@ -608,7 +608,7 @@ 
 		$this->_scriptFiles[$key]=$url;
 
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerScriptFile',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerScriptFile', $params);
 	}
 
 	/**
@@ -616,13 +616,13 @@ 
 	 * @param string a unique key identifying the script block
 	 * @param string javascript block
 	 */
-	public function registerBeginScript($key,$script)
+	public function registerBeginScript($key, $script)
 	{
 		$this->checkIfNotInRender();
 		$this->_beginScripts[$key]=$script;
 
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerBeginScript',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerBeginScript', $params);
 	}
 
 	/**
@@ -630,12 +630,12 @@ 
 	 * @param string a unique key identifying the script block
 	 * @param string javascript block
 	 */
-	public function registerEndScript($key,$script)
+	public function registerEndScript($key, $script)
 	{
 		$this->_endScripts[$key]=$script;
 
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerEndScript',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerEndScript', $params);
 	}
 
 	/**
@@ -644,12 +644,12 @@ 
 	 * @param string|array hidden field value, if the value is an array, every element
 	 * in the array will be rendered as a hidden field value.
 	 */
-	public function registerHiddenField($name,$value)
+	public function registerHiddenField($name, $value)
 	{
 		$this->_hiddenFields[$name]=$value;
 
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','registerHiddenField',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'registerHiddenField', $params);
 	}
 
 	/**
@@ -762,7 +762,7 @@ 
 	public function renderStyleSheets($writer)
 	{
 		if(count($this->_styleSheets))
-			$writer->write("<style type=\"text/css\">\n/*<![CDATA[*/\n".implode("\n",$this->_styleSheets)."\n/*]]>*/\n</style>\n");
+			$writer->write("<style type=\"text/css\">\n/*<![CDATA[*/\n".implode("\n", $this->_styleSheets)."\n/*]]>*/\n</style>\n");
 	}
 
 	/**
@@ -770,7 +770,7 @@ 
 	 */
 	public function renderHeadScriptFiles($writer)
 	{
-		$this->renderScriptFiles($writer,$this->_headScriptFiles);
+		$this->renderScriptFiles($writer, $this->_headScriptFiles);
 	}
 
 	/**
@@ -793,9 +793,9 @@ 
 
 	public function markScriptFileAsRendered($url)
 	{
-		$this->_renderedScriptFiles[$url] = $url;
+		$this->_renderedScriptFiles[$url]=$url;
 		$params=func_get_args();
-		$this->_page->registerCachingAction('Page.ClientScript','markScriptFileAsRendered',$params);
+		$this->_page->registerCachingAction('Page.ClientScript', 'markScriptFileAsRendered', $params);
 	}
 
 	protected function renderScriptFiles($writer, Array $scripts)
@@ -819,8 +819,8 @@ 
 	{
 		if(!empty($this->_scriptFiles))
 		{
-			$addedScripts = array_diff($this->_scriptFiles,$this->getRenderedScriptFiles());
-			$this->renderScriptFiles($writer,$addedScripts);
+			$addedScripts=array_diff($this->_scriptFiles, $this->getRenderedScriptFiles());
+			$this->renderScriptFiles($writer, $addedScripts);
 		}
 	}
 
@@ -858,12 +858,12 @@ 
 
 	public function renderHiddenFieldsBegin($writer)
 	{
-		$this->renderHiddenFieldsInt($writer,true);
+		$this->renderHiddenFieldsInt($writer, true);
 	}
 
 	public function renderHiddenFieldsEnd($writer)
 	{
-		$this->renderHiddenFieldsInt($writer,false);
+		$this->renderHiddenFieldsInt($writer, false);
 	}
 
 	/**
@@ -885,12 +885,12 @@ 
 	 */
 	protected function renderHiddenFieldsInt($writer, $initial)
  	{
-		if ($initial) $this->_renderedHiddenFields = array();
+		if($initial) $this->_renderedHiddenFields=array();
 		$str='';
 		foreach($this->_hiddenFields as $name=>$value)
 		{
-			if (in_array($name,$this->_renderedHiddenFields)) continue;
-			$id=strtr($name,':','_');
+			if(in_array($name, $this->_renderedHiddenFields)) continue;
+			$id=strtr($name, ':', '_');
 			if(is_array($value))
 			{
 				foreach($value as $v)
@@ -900,7 +900,7 @@ 
 			{
 				$str.='<input type="hidden" name="'.$name.'" id="'.$id.'" value="'.THttpUtility::htmlEncode($value)."\" />\n";
 			}
-			$this->_renderedHiddenFields[] = $name;
+			$this->_renderedHiddenFields[]=$name;
 		}
 		if($str!=='')
 			$writer->write("<div style=\"visibility:hidden;\">\n".$str."</div>\n");
@@ -916,7 +916,7 @@ 
 	 */
 	protected function checkIfNotInRender()
 	{
-		if ($form = $this->_page->InFormRender)
+		if($form=$this->_page->InFormRender)
 			throw new \Exception('Operation invalid when page is already rendering');
 	}
 }