pradosoft / prado

framework/I18N/core/HTTPNegotiator.php

Indentation +14 added lines, -14 removed lines

@@ -58,7 +58,7 @@ 
 		$this->languages = array();
 
 		if (!isset($_SERVER['HTTP_ACCEPT_LANGUAGE']))
-            return $this->languages;
+			return $this->languages;
 
 		//$basedir = CultureInfo::dataDir();
 		//$ext = CultureInfo::fileExt();
@@ -66,18 +66,18 @@ 
 
 		foreach(explode(',', $_SERVER['HTTP_ACCEPT_LANGUAGE']) as $lang)
 		{
-            // Cut off any q-value that might come after a semi-colon
-            if ($pos = strpos($lang, ';'))
-                $lang = trim(substr($lang, 0, $pos));
+			// Cut off any q-value that might come after a semi-colon
+			if ($pos = strpos($lang, ';'))
+				$lang = trim(substr($lang, 0, $pos));
 
 			if (strstr($lang, '-'))
 			{
 				$codes = explode('-',$lang);
 				if($codes[0] == 'i')
 				{
-                    // Language not listed in ISO 639 that are not variants
-                    // of any listed language, which can be registerd with the
-                    // i-prefix, such as i-cherokee
+					// Language not listed in ISO 639 that are not variants
+					// of any listed language, which can be registerd with the
+					// i-prefix, such as i-cherokee
 					if(count($codes)>1)
 						$lang = $codes[1];
 				}
@@ -91,13 +91,13 @@ 
 							$lang .= '_'.strtoupper($codes[$i]);
 					}
 				}
-            }
+			}
 
 
 
 			if($info->validCulture($lang))
 				$this->languages[] = $lang;
-        }
+		}
 
 		return $this->languages;
 	}
@@ -108,20 +108,20 @@ 
 	 */
 	function getCharsets()
 	{
-        if($this->charsets !== null) {
+		if($this->charsets !== null) {
 			return $this->charsets;
 		}
 
 		$this->charsets = array();
 
 		if (!isset($_SERVER['HTTP_ACCEPT_CHARSET']))
-            return $this->charsets;
+			return $this->charsets;
 
 		foreach (explode(',', $_SERVER['HTTP_ACCEPT_CHARSET']) as $charset)
 		{
-            if (!empty($charset))
-                $this->charsets[] = preg_replace('/;.*/', '', $charset);
-        }
+			if (!empty($charset))
+				$this->charsets[] = preg_replace('/;.*/', '', $charset);
+		}
 
 		return $this->charsets;
 	}

Braces +1 added lines, -2 removed lines

@@ -80,8 +80,7 @@
                     // i-prefix, such as i-cherokee
 					if(count($codes)>1)
 						$lang = $codes[1];
-				}
-				else
+				} else
 				{
 					for($i = 0, $k = count($codes); $i<$k; ++$i)
 					{

Spacing +22 added lines, -22 removed lines

@@ -51,44 +51,44 @@ 
 	 */
 	function getLanguages()
 	{
-		if($this->languages !== null) {
+		if($this->languages!==null) {
 			return $this->languages;
 		}
 
-		$this->languages = array();
+		$this->languages=array();
 
-		if (!isset($_SERVER['HTTP_ACCEPT_LANGUAGE']))
+		if(!isset($_SERVER['HTTP_ACCEPT_LANGUAGE']))
             return $this->languages;
 
 		//$basedir = CultureInfo::dataDir();
 		//$ext = CultureInfo::fileExt();
-		$info = new CultureInfo();
+		$info=new CultureInfo();
 
 		foreach(explode(',', $_SERVER['HTTP_ACCEPT_LANGUAGE']) as $lang)
 		{
             // Cut off any q-value that might come after a semi-colon
-            if ($pos = strpos($lang, ';'))
-                $lang = trim(substr($lang, 0, $pos));
+            if($pos=strpos($lang, ';'))
+                $lang=trim(substr($lang, 0, $pos));
 
-			if (strstr($lang, '-'))
+			if(strstr($lang, '-'))
 			{
-				$codes = explode('-',$lang);
-				if($codes[0] == 'i')
+				$codes=explode('-', $lang);
+				if($codes[0]=='i')
 				{
                     // Language not listed in ISO 639 that are not variants
                     // of any listed language, which can be registerd with the
                     // i-prefix, such as i-cherokee
-					if(count($codes)>1)
-						$lang = $codes[1];
+					if(count($codes) > 1)
+						$lang=$codes[1];
 				}
 				else
 				{
-					for($i = 0, $k = count($codes); $i<$k; ++$i)
+					for($i=0, $k=count($codes); $i < $k; ++$i)
 					{
-						if($i == 0)
-							$lang = strtolower($codes[0]);
+						if($i==0)
+							$lang=strtolower($codes[0]);
 						else
-							$lang .= '_'.strtoupper($codes[$i]);
+							$lang.='_'.strtoupper($codes[$i]);
 					}
 				}
             }
@@ -96,7 +96,7 @@ 
 
 
 			if($info->validCulture($lang))
-				$this->languages[] = $lang;
+				$this->languages[]=$lang;
         }
 
 		return $this->languages;
@@ -108,19 +108,19 @@ 
 	 */
 	function getCharsets()
 	{
-        if($this->charsets !== null) {
+        if($this->charsets!==null) {
 			return $this->charsets;
 		}
 
-		$this->charsets = array();
+		$this->charsets=array();
 
-		if (!isset($_SERVER['HTTP_ACCEPT_CHARSET']))
+		if(!isset($_SERVER['HTTP_ACCEPT_CHARSET']))
             return $this->charsets;
 
-		foreach (explode(',', $_SERVER['HTTP_ACCEPT_CHARSET']) as $charset)
+		foreach(explode(',', $_SERVER['HTTP_ACCEPT_CHARSET']) as $charset)
 		{
-            if (!empty($charset))
-                $this->charsets[] = preg_replace('/;.*/', '', $charset);
+            if(!empty($charset))
+                $this->charsets[]=preg_replace('/;.*/', '', $charset);
         }
 
 		return $this->charsets;

framework/Util/TDataFieldAccessor.php

Indentation +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 
 				$tmp = $data;
 				foreach (explode(".", $field) as $f)
-				    $tmp = $tmp[$f];
+					$tmp = $tmp[$f];
 				return $tmp;
 			}
 			else if(is_object($data))

Spacing +10 added lines, -10 removed lines

@@ -48,7 +48,7 @@ 
 	 * @return mixed value at the specified field
 	 * @throws TInvalidDataValueException if field or data is invalid
 	 */
-	public static function getDataFieldValue($data,$field)
+	public static function getDataFieldValue($data, $field)
 	{
 		try
 		{
@@ -57,33 +57,33 @@ 
 				if(isset($data[$field]))
 					return $data[$field];
 
-				$tmp = $data;
-				foreach (explode(".", $field) as $f)
-				    $tmp = $tmp[$f];
+				$tmp=$data;
+				foreach(explode(".", $field) as $f)
+				    $tmp=$tmp[$f];
 				return $tmp;
 			}
 			else if(is_object($data))
 			{
-				if(strpos($field,'.')===false)  // simple field
+				if(strpos($field, '.')===false)  // simple field
 				{
 					if(method_exists($data, 'get'.$field))
-						return call_user_func(array($data,'get'.$field));
+						return call_user_func(array($data, 'get'.$field));
 					else
 						return $data->{$field};
 				}
 				else // field in the format of xxx.yyy.zzz
 				{
 					$object=$data;
-					foreach(explode('.',$field) as $f)
-						$object = TDataFieldAccessor::getDataFieldValue($object, $f);
+					foreach(explode('.', $field) as $f)
+						$object=TDataFieldAccessor::getDataFieldValue($object, $f);
 					return $object;
 				}
 			}
 		}
 		catch(Exception $e)
 		{
-			throw new TInvalidDataValueException('datafieldaccessor_datafield_invalid',$field,$e->getMessage());
+			throw new TInvalidDataValueException('datafieldaccessor_datafield_invalid', $field, $e->getMessage());
 		}
-		throw new TInvalidDataValueException('datafieldaccessor_data_invalid',$field);
+		throw new TInvalidDataValueException('datafieldaccessor_data_invalid', $field);
 	}
 }

Braces +3 added lines, -6 removed lines

@@ -61,8 +61,7 @@ 
 				foreach (explode(".", $field) as $f)
 				    $tmp = $tmp[$f];
 				return $tmp;
-			}
-			else if(is_object($data))
+			} else if(is_object($data))
 			{
 				if(strpos($field,'.')===false)  // simple field
 				{
@@ -70,8 +69,7 @@ 
 						return call_user_func(array($data,'get'.$field));
 					else
 						return $data->{$field};
-				}
-				else // field in the format of xxx.yyy.zzz
+				} else // field in the format of xxx.yyy.zzz
 				{
 					$object=$data;
 					foreach(explode('.',$field) as $f)
@@ -79,8 +77,7 @@ 
 					return $object;
 				}
 			}
-		}
-		catch(Exception $e)
+		} catch(Exception $e)
 		{
 			throw new TInvalidDataValueException('datafieldaccessor_datafield_invalid',$field,$e->getMessage());
 		}

framework/Web/Services/TFeedService.php

Indentation +3 added lines, -3 removed lines

@@ -139,9 +139,9 @@
 			$feed->init($feedConfig);
 
 			$content=$feed->getFeedContent();
-		    //$this->getResponse()->setContentType('application/rss+xml');
-		    $this->getResponse()->setContentType($feed->getContentType());
-		    $this->getResponse()->write($content);
+			//$this->getResponse()->setContentType('application/rss+xml');
+			$this->getResponse()->setContentType($feed->getContentType());
+			$this->getResponse()->write($content);
 		}
 		else
 			throw new THttpException(404,'feedservice_feed_unknown',$id);

Spacing +10 added lines, -10 removed lines

@@ -77,7 +77,7 @@ 
 			if(is_array($config))
 			{
 				foreach($config as $id => $feed)
-					$this->_feeds[$id] = $feed;
+					$this->_feeds[$id]=$feed;
 			}
 		}
 		else
@@ -110,20 +110,20 @@ 
 		if(isset($this->_feeds[$id]))
 		{
 			$feedConfig=$this->_feeds[$id];
-			$properties = array();
-			$feed = null;
+			$properties=array();
+			$feed=null;
 			if($this->getApplication()->getConfigurationType()==TApplication::CONFIG_TYPE_PHP)
 			{
 				if(isset($feedConfig['class']))
 				{
 					$feed=Prado::createComponent($feedConfig['class']);
 					if($service instanceof IFeedContentProvider)
-						$properties=isset($feedConfig['properties'])?$feedConfig['properties']:array();
+						$properties=isset($feedConfig['properties']) ? $feedConfig['properties'] : array();
 					else
-						throw new TConfigurationException('jsonservice_response_type_invalid',$id);
+						throw new TConfigurationException('jsonservice_response_type_invalid', $id);
 				}
 				else
-					throw new TConfigurationException('jsonservice_class_required',$id);
+					throw new TConfigurationException('jsonservice_class_required', $id);
 			}
 			else
 			{
@@ -132,15 +132,15 @@ 
 				{
 					$feed=Prado::createComponent($class);
 					if(!($feed instanceof IFeedContentProvider))
-						throw new TConfigurationException('feedservice_feedtype_invalid',$id);
+						throw new TConfigurationException('feedservice_feedtype_invalid', $id);
 				}
 				else
-					throw new TConfigurationException('feedservice_class_required',$id);
+					throw new TConfigurationException('feedservice_class_required', $id);
 			}
 
 			// init feed properties
 			foreach($properties as $name=>$value)
-				$feed->setSubproperty($name,$value);
+				$feed->setSubproperty($name, $value);
 			$feed->init($feedConfig);
 
 			$content=$feed->getFeedContent();
@@ -149,6 +149,6 @@ 
 		    $this->getResponse()->write($content);
 		}
 		else
-			throw new THttpException(404,'feedservice_feed_unknown',$id);
+			throw new THttpException(404, 'feedservice_feed_unknown', $id);
 	}
 }
\ No newline at end of file

Braces +5 added lines, -10 removed lines

@@ -79,8 +79,7 @@ 
 				foreach($config as $id => $feed)
 					$this->_feeds[$id] = $feed;
 			}
-		}
-		else
+		} else
 		{
 			foreach($config->getElementsByTagName('feed') as $feed)
 			{
@@ -121,11 +120,9 @@ 
 						$properties=isset($feedConfig['properties'])?$feedConfig['properties']:array();
 					else
 						throw new TConfigurationException('jsonservice_response_type_invalid',$id);
-				}
-				else
+				} else
 					throw new TConfigurationException('jsonservice_class_required',$id);
-			}
-			else
+			} else
 			{
 				$properties=$feedConfig->getAttributes();
 				if(($class=$properties->remove('class'))!==null)
@@ -133,8 +130,7 @@ 
 					$feed=Prado::createComponent($class);
 					if(!($feed instanceof IFeedContentProvider))
 						throw new TConfigurationException('feedservice_feedtype_invalid',$id);
-				}
-				else
+				} else
 					throw new TConfigurationException('feedservice_class_required',$id);
 			}
 
@@ -147,8 +143,7 @@ 
 		    //$this->getResponse()->setContentType('application/rss+xml');
 		    $this->getResponse()->setContentType($feed->getContentType());
 		    $this->getResponse()->write($content);
-		}
-		else
+		} else
 			throw new THttpException(404,'feedservice_feed_unknown',$id);
 	}
 }
\ No newline at end of file

framework/Web/UI/TCachePageStatePersister.php

Indentation +4 added lines, -4 removed lines

@@ -144,15 +144,15 @@
 	 */
 	public function getKeyPrefix()
 	{
-	    return $this->_prefix;
+		return $this->_prefix;
 	}
 
 	/**
-     * @param string prefix of cache variable name to avoid conflict with other cache data
-     */
+	 * @param string prefix of cache variable name to avoid conflict with other cache data
+	 */
 	public function setKeyPrefix($value)
 	{
-	    $this->_prefix=$value;
+		$this->_prefix=$value;
 	}
 
 	/**

Spacing +7 added lines, -7 removed lines

@@ -110,7 +110,7 @@ 
 			if($cache===null || !($cache instanceof ICache))
 			{
 				if($this->_cacheModuleID!=='')
-					throw new TConfigurationException('cachepagestatepersister_cachemoduleid_invalid',$this->_cacheModuleID);
+					throw new TConfigurationException('cachepagestatepersister_cachemoduleid_invalid', $this->_cacheModuleID);
 				else
 					throw new TConfigurationException('cachepagestatepersister_cache_required');
 			}
@@ -133,7 +133,7 @@ 
 	 */
 	public function setCacheTimeout($value)
 	{
-		if(($value=TPropertyValue::ensureInteger($value))>=0)
+		if(($value=TPropertyValue::ensureInteger($value)) >= 0)
 			$this->_timeout=$value;
 		else
 			throw new TInvalidDataValueException('cachepagestatepersister_timeout_invalid');
@@ -173,10 +173,10 @@ 
 	 */
 	public function save($data)
 	{
-		$timestamp=(string)microtime(true);
+		$timestamp=(string) microtime(true);
 		$key=$this->calculateKey($timestamp);
-		$this->getCache()->add($key,$data,$this->_timeout);
-		$this->_page->setClientState(TPageStateFormatter::serialize($this->_page,$timestamp));
+		$this->getCache()->add($key, $data, $this->_timeout);
+		$this->_page->setClientState(TPageStateFormatter::serialize($this->_page, $timestamp));
 	}
 
 	/**
@@ -186,13 +186,13 @@ 
 	 */
 	public function load()
 	{
-		if(($timestamp=TPageStateFormatter::unserialize($this->_page,$this->_page->getRequestClientState()))!==null)
+		if(($timestamp=TPageStateFormatter::unserialize($this->_page, $this->_page->getRequestClientState()))!==null)
 		{
 			$key=$this->calculateKey($timestamp);
 			if(($data=$this->getCache()->get($key))!==false)
 				return $data;
 		}
-		throw new THttpException(400,'cachepagestatepersister_pagestate_corrupted');
+		throw new THttpException(400, 'cachepagestatepersister_pagestate_corrupted');
 	}
 }
 

framework/Web/UI/WebControls/TTableCell.php

Indentation +2 added lines, -2 removed lines

@@ -63,7 +63,7 @@ 
 
 	/**
 	 * Sets the horizontal alignment of the contents within the table item.
-     * Valid values include 'NotSet', 'Justify', 'Left', 'Right', 'Center'
+	 * Valid values include 'NotSet', 'Justify', 'Left', 'Right', 'Center'
 	 * @param string the horizontal alignment
 	 */
 	public function setHorizontalAlign($value)
@@ -84,7 +84,7 @@ 
 
 	/**
 	 * Sets the vertical alignment of the contents within the table item.
-     * Valid values include 'NotSet','Top','Bottom','Middle'
+	 * Valid values include 'NotSet','Top','Bottom','Middle'
 	 * @param string the horizontal alignment
 	 */
 	public function setVerticalAlign($value)

Spacing +6 added lines, -6 removed lines

@@ -151,7 +151,7 @@ 
 	 */
 	public function getText()
 	{
-		return $this->getViewState('Text','');
+		return $this->getViewState('Text', '');
 	}
 
 	/**
@@ -161,7 +161,7 @@ 
 	 */
 	public function setText($value)
 	{
-		$this->setViewState('Text',$value,'');
+		$this->setViewState('Text', $value, '');
 	}
 
 	/**
@@ -197,10 +197,10 @@ 
 	protected function addAttributesToRender($writer)
 	{
 		parent::addAttributesToRender($writer);
-		if(($colspan=$this->getColumnSpan())>0)
-			$writer->addAttribute('colspan',"$colspan");
-		if(($rowspan=$this->getRowSpan())>0)
-			$writer->addAttribute('rowspan',"$rowspan");
+		if(($colspan=$this->getColumnSpan()) > 0)
+			$writer->addAttribute('colspan', "$colspan");
+		if(($rowspan=$this->getRowSpan()) > 0)
+			$writer->addAttribute('rowspan', "$rowspan");
 	}
 
 	/**

framework/Web/UI/WebControls/TAccordion.php

Indentation +14 added lines, -14 removed lines

@@ -180,24 +180,24 @@ 
 	}
 
 	/**
-	* @return string URL for the CSS file including all relevant CSS class definitions. Defaults to ''.
-	*/
+	 * @return string URL for the CSS file including all relevant CSS class definitions. Defaults to ''.
+	 */
 	public function getCssUrl()
 	{
 		return $this->getViewState('CssUrl','default');
 	}
 
 	/**
-	* @param string URL for the CSS file including all relevant CSS class definitions.
-	*/
+	 * @param string URL for the CSS file including all relevant CSS class definitions.
+	 */
 	public function setCssUrl($value)
 	{
 		$this->setViewState('CssUrl',TPropertyValue::ensureString($value),'');
 	}
 
 	/**
-		* @return string CSS class for the whole accordion control div.
-	*/
+	 * @return string CSS class for the whole accordion control div.
+	 */
 	public function getCssClass()
 	{
 		$cssClass=parent::getCssClass();
@@ -205,32 +205,32 @@ 
 	}
 
 	/**
-		* @return string CSS class for the currently displayed view div. Defaults to 'accordion-view'.
-	*/
+	 * @return string CSS class for the currently displayed view div. Defaults to 'accordion-view'.
+	 */
 	public function getViewCssClass()
 	{
 		return $this->getViewStyle()->getCssClass();
 	}
 
 	/**
-	* @param string CSS class for the currently displayed view div.
-	*/
+	 * @param string CSS class for the currently displayed view div.
+	 */
 	public function setViewCssClass($value)
 	{
 		$this->getViewStyle()->setCssClass($value);
 	}
 
 	/**
-		* @return string CSS class for the currently displayed view div. Defaults to 'accordion-view'.
-	*/
+	 * @return string CSS class for the currently displayed view div. Defaults to 'accordion-view'.
+	 */
 	public function getAnimationDuration()
 	{
 		return $this->getViewState('AnimationDuration','1');
 	}
 
 	/**
-	* @param string CSS class for the currently displayed view div.
-	*/
+	 * @param string CSS class for the currently displayed view div.
+	 */
 	public function setAnimationDuration($value)
 	{
 		$this->setViewState('AnimationDuration',$value);

Spacing +41 added lines, -41 removed lines

@@ -100,7 +100,7 @@ 
 	 */
 	public function getActiveViewIndex()
 	{
-		return $this->getViewState('ActiveViewIndex',0);
+		return $this->getViewState('ActiveViewIndex', 0);
 	}
 
 	/**
@@ -109,7 +109,7 @@ 
 	 */
 	public function setActiveViewIndex($value)
 	{
-		$this->setViewState('ActiveViewIndex',TPropertyValue::ensureInteger($value),0);
+		$this->setViewState('ActiveViewIndex', TPropertyValue::ensureInteger($value), 0);
 		$this->setActiveViewID('');
 	}
 
@@ -121,7 +121,7 @@ 
 	 */
 	public function getActiveViewID()
 	{
-		return $this->getViewState('ActiveViewID','');
+		return $this->getViewState('ActiveViewID', '');
 	}
 
 	/**
@@ -129,7 +129,7 @@ 
 	 */
 	public function setActiveViewID($value)
 	{
-		$this->setViewState('ActiveViewID',$value,'');
+		$this->setViewState('ActiveViewID', $value, '');
 	}
 
 	/**
@@ -145,17 +145,17 @@ 
 		$views=$this->getViews();
 		if(($id=$this->getActiveViewID())!=='')
 		{
-			if(($index=$views->findIndexByID($id))>=0)
+			if(($index=$views->findIndexByID($id)) >= 0)
 				$activeView=$views->itemAt($index);
 			else
-				throw new TInvalidDataValueException('accordion_activeviewid_invalid',$id);
+				throw new TInvalidDataValueException('accordion_activeviewid_invalid', $id);
 		}
-		else if(($index=$this->getActiveViewIndex())>=0)
+		else if(($index=$this->getActiveViewIndex()) >= 0)
 		{
-			if($index<$views->getCount())
+			if($index < $views->getCount())
 				$activeView=$views->itemAt($index);
 			else
-				throw new TInvalidDataValueException('accordion_activeviewindex_invalid',$index);
+				throw new TInvalidDataValueException('accordion_activeviewindex_invalid', $index);
 		}
 		else
 		{
@@ -179,7 +179,7 @@ 
 	 */
 	public function setActiveView($view)
 	{
-		if($this->getViews()->indexOf($view)>=0)
+		if($this->getViews()->indexOf($view) >= 0)
 			$this->activateView($view);
 		else
 			throw new TInvalidOperationException('accordion_view_inexistent');
@@ -190,7 +190,7 @@ 
 	*/
 	public function getCssUrl()
 	{
-		return $this->getViewState('CssUrl','default');
+		return $this->getViewState('CssUrl', 'default');
 	}
 
 	/**
@@ -198,7 +198,7 @@ 
 	*/
 	public function setCssUrl($value)
 	{
-		$this->setViewState('CssUrl',TPropertyValue::ensureString($value),'');
+		$this->setViewState('CssUrl', TPropertyValue::ensureString($value), '');
 	}
 
 	/**
@@ -207,7 +207,7 @@ 
 	public function getCssClass()
 	{
 		$cssClass=parent::getCssClass();
-			return $cssClass===''?'accordion':$cssClass;
+			return $cssClass==='' ? 'accordion' : $cssClass;
 	}
 
 	/**
@@ -231,7 +231,7 @@ 
 	*/
 	public function getAnimationDuration()
 	{
-		return $this->getViewState('AnimationDuration','1');
+		return $this->getViewState('AnimationDuration', '1');
 	}
 
 	/**
@@ -239,7 +239,7 @@ 
 	*/
 	public function setAnimationDuration($value)
 	{
-		$this->setViewState('AnimationDuration',$value);
+		$this->setViewState('AnimationDuration', $value);
 	}
 
 	/**
@@ -247,11 +247,11 @@ 
 	 */
 	public function getViewStyle()
 	{
-		if(($style=$this->getViewState('ViewStyle',null))===null)
+		if(($style=$this->getViewState('ViewStyle', null))===null)
 		{
 			$style=new TStyle;
 			$style->setCssClass('accordion-view');
-			$this->setViewState('ViewStyle',$style,null);
+			$this->setViewState('ViewStyle', $style, null);
 		}
 		return $style;
 	}
@@ -277,11 +277,11 @@ 
 	 */
 	public function getHeaderStyle()
 	{
-		if(($style=$this->getViewState('HeaderStyle',null))===null)
+		if(($style=$this->getViewState('HeaderStyle', null))===null)
 		{
 			$style=new TStyle;
 			$style->setCssClass('accordion-header');
-			$this->setViewState('HeaderStyle',$style,null);
+			$this->setViewState('HeaderStyle', $style, null);
 		}
 		return $style;
 	}
@@ -307,11 +307,11 @@ 
 	 */
 	public function getActiveHeaderStyle()
 	{
-		if(($style=$this->getViewState('ActiveHeaderStyle',null))===null)
+		if(($style=$this->getViewState('ActiveHeaderStyle', null))===null)
 		{
 			$style=new TStyle;
 			$style->setCssClass('accordion-header-active');
-			$this->setViewState('ActiveHeaderStyle',$style,null);
+			$this->setViewState('ActiveHeaderStyle', $style, null);
 		}
 		return $style;
 	}
@@ -361,11 +361,11 @@ 
 	 * @param array the input data collection
 	 * @return boolean whether the data of the control has been changed
 	 */
-	public function loadPostData($key,$values)
+	public function loadPostData($key, $values)
 	{
 		if(($index=$values[$this->getClientID().'_1'])!==null)
 		{
-			$index=(int)$index;
+			$index=(int) $index;
 			$currentIndex=$this->getActiveViewIndex();
 			if($currentIndex!==$index)
 			{
@@ -405,7 +405,7 @@ 
 	 */
 	protected function addAttributesToRender($writer)
 	{
-		$writer->addAttribute('id',$this->getClientID());
+		$writer->addAttribute('id', $this->getClientID());
 		$this->setCssClass($this->getCssClass());
 		parent::addAttributesToRender($writer);
 	}
@@ -418,7 +418,7 @@ 
 	public function onPreRender($param)
 	{
 		parent::onPreRender($param);
-		$this->getActiveView();  // determine the active view
+		$this->getActiveView(); // determine the active view
 		$this->registerStyleSheet();
 	}
 
@@ -429,17 +429,17 @@ 
 	 */
 	protected function registerStyleSheet()
 	{
-		$url = $this->getCssUrl();
+		$url=$this->getCssUrl();
 
-		if($url === '') {
+		if($url==='') {
 			return;
 		}
 
-		if($url === 'default') {
-			$url = $this->getApplication()->getAssetManager()->publishFilePath(dirname(__FILE__).DIRECTORY_SEPARATOR.'assets'.DIRECTORY_SEPARATOR.'accordion.css');
+		if($url==='default') {
+			$url=$this->getApplication()->getAssetManager()->publishFilePath(dirname(__FILE__).DIRECTORY_SEPARATOR.'assets'.DIRECTORY_SEPARATOR.'accordion.css');
 		}
 
-		if($url !== '') {
+		if($url!=='') {
 			$this->getPage()->getClientScript()->registerStyleSheetFile($url, $url);
 		}
 	}
@@ -458,7 +458,7 @@ 
 		$code="new $className($options);";
 		$cs->registerEndScript("prado:$id", $code);
 		// ensure an item is always active and visible
-		$index = $this->getActiveViewIndex();
+		$index=$this->getActiveViewIndex();
 		if(!$this->getViews()->itemAt($index)->Visible)
 			$index=0;
 		$cs->registerHiddenField($id.'_1', $index);
@@ -481,17 +481,17 @@ 
 	 */
 	protected function getClientOptions()
 	{
-		$options['ID'] = $this->getClientID();
-		$options['ActiveHeaderCssClass'] = $this->getActiveHeaderCssClass();
-		$options['HeaderCssClass'] = $this->getHeaderCssClass();
-		$options['Duration'] = $this->getAnimationDuration();
+		$options['ID']=$this->getClientID();
+		$options['ActiveHeaderCssClass']=$this->getActiveHeaderCssClass();
+		$options['HeaderCssClass']=$this->getHeaderCssClass();
+		$options['Duration']=$this->getAnimationDuration();
 
-		if (($viewheight = $this->getViewHeight())>0)
-			$options['maxHeight'] = $viewheight;
-		$views = array();
+		if(($viewheight=$this->getViewHeight()) > 0)
+			$options['maxHeight']=$viewheight;
+		$views=array();
 		foreach($this->getViews() as $view)
-			$views[$view->getClientID()] = $view->getVisible() ? '1': '0';
-		$options['Views'] = $views;
+			$views[$view->getClientID()]=$view->getVisible() ? '1' : '0';
+		$options['Views']=$views;
 
 		return $options;
 	}
@@ -526,7 +526,7 @@ 
 	public function renderContents($writer)
 	{
 		$views=$this->getViews();
-		if($views->getCount()>0)
+		if($views->getCount() > 0)
 		{
 			$writer->writeLine();
 			foreach($views as $view)

Braces +3 added lines, -6 removed lines

@@ -149,15 +149,13 @@ 
 				$activeView=$views->itemAt($index);
 			else
 				throw new TInvalidDataValueException('accordion_activeviewid_invalid',$id);
-		}
-		else if(($index=$this->getActiveViewIndex())>=0)
+		} else if(($index=$this->getActiveViewIndex())>=0)
 		{
 			if($index<$views->getCount())
 				$activeView=$views->itemAt($index);
 			else
 				throw new TInvalidDataValueException('accordion_activeviewindex_invalid',$index);
-		}
-		else
+		} else
 		{
 			foreach($views as $index=>$view)
 			{
@@ -348,8 +346,7 @@ 
 				$this->setActiveViewIndex($index);
 				$this->setActiveViewID($view->getID(false));
 				$view->setActive(true);
-			}
-			else
+			} else
 				$v->setActive(false);
 		}
 	}

framework/Web/UI/WebControls/TDropDownList.php

Indentation +2 added lines, -2 removed lines

@@ -140,13 +140,13 @@
 	 */
 	public function getIsValid()
 	{
-	    return $this->_isValid;
+		return $this->_isValid;
 	}
 	/**
 	 * @param bool wether this control is valid.
 	 */
 	public function setIsValid($value)
 	{
-	    $this->_isValid=TPropertyValue::ensureBoolean($value);
+		$this->_isValid=TPropertyValue::ensureBoolean($value);
 	}
 }

Spacing +4 added lines, -4 removed lines

@@ -52,7 +52,7 @@ 
 	 */
 	protected function addAttributesToRender($writer)
 	{
-		$writer->addAttribute('name',$this->getUniqueID());
+		$writer->addAttribute('name', $this->getUniqueID());
 		parent::addAttributesToRender($writer);
 	}
 
@@ -73,15 +73,15 @@ 
 	 * @param array the input data collection
 	 * @return boolean whether the data of the component has been changed
 	 */
-	public function loadPostData($key,$values)
+	public function loadPostData($key, $values)
 	{
 		if(!$this->getEnabled(true))
 			return false;
 		$this->ensureDataBound();
-		$selection=isset($values[$key])?$values[$key]:null;
+		$selection=isset($values[$key]) ? $values[$key] : null;
 		if($selection!==null)
 		{
-			$index=$this->getItems()->findIndexByValue($selection,false);
+			$index=$this->getItems()->findIndexByValue($selection, false);
 			if($this->getSelectedIndex()!==$index)
 			{
 				$this->setSelectedIndex($index);

Doc Comments +1 added lines, -1 removed lines

@@ -125,7 +125,7 @@
 	/**
 	 * Returns the value to be validated.
 	 * This methid is required by \Prado\Web\UI\IValidatable interface.
-	 * @return mixed the value of the property to be validated.
+	 * @return string the value of the property to be validated.
 	 */
 	public function getValidationPropertyValue()
 	{

framework/Web/UI/WebControls/TXmlTransform.php

Indentation +69 added lines, -69 removed lines

@@ -46,50 +46,50 @@ 
    * @throws TConfigurationException If XSL extension is not available
    */
   public function __construct() {
-    if(!class_exists('XSLTProcessor', false)) {
-      throw new TConfigurationException('xmltransform_xslextension_required');
-    }
+	if(!class_exists('XSLTProcessor', false)) {
+	  throw new TConfigurationException('xmltransform_xslextension_required');
+	}
   }
 
   /**
    * @return string The path to the XML style sheet.
    */
   public function getTransformPath() {
-    return $this->getViewState('TransformPath', '');
+	return $this->getViewState('TransformPath', '');
   }
 
   /**
    * @param string The path to the XML style sheet.  It must be in namespace format.
    */
   public function setTransformPath($value) {
-    if(!is_file($value)) {
-      $value = Prado::getPathOfNamespace($value, self::EXT_XSL_FILE);
-      if($value === null) {
+	if(!is_file($value)) {
+	  $value = Prado::getPathOfNamespace($value, self::EXT_XSL_FILE);
+	  if($value === null) {
 	throw new TInvalidDataValueException('xmltransform_transformpath_invalid', $value);
-      }
-    }
-    $this->setViewState('TransformPath', $value, '');
+	  }
+	}
+	$this->setViewState('TransformPath', $value, '');
   }
 
   /**
    * @return string XML style sheet as string
    */
   public function getTransformContent() {
-    return $this->getViewState('TransformContent', '');
+	return $this->getViewState('TransformContent', '');
   }
 
   /**
    * @param string $value XML style sheet as string
    */
   public function setTransformContent($value) {
-    $this->setViewState('TransformContent', $value, '');
+	$this->setViewState('TransformContent', $value, '');
   }
 
   /**
    * @return string The path to the XML document. It must be in namespace format.
    */
   public function getDocumentPath() {
-    return $this->getViewState('DocumentPath', '');
+	return $this->getViewState('DocumentPath', '');
   }
 
   /**
@@ -97,27 +97,27 @@ 
    * @throws TInvalidDataValueException
    */
   public function setDocumentPath($value) {
-    if(!is_file($value)) {
-      $value = Prado::getPathOfNamespace($value, self::EXT_XML_FILE);
-      if($value === null) {
+	if(!is_file($value)) {
+	  $value = Prado::getPathOfNamespace($value, self::EXT_XML_FILE);
+	  if($value === null) {
 	throw new TInvalidDataValueException('xmltransform_documentpath_invalid', $value);
-      }
-    }
-    $this->setViewState('DocumentPath', $value, '');
+	  }
+	}
+	$this->setViewState('DocumentPath', $value, '');
   }
 
   /**
    * @return string XML data
    */
   public function getDocumentContent() {
-    return $this->getViewState('DocumentContent', '');
+	return $this->getViewState('DocumentContent', '');
   }
 
   /**
    * @param string $value XML data. If not empty, it takes precedence over {@link setDocumentPath DocumentPath}.
    */
   public function setDocumentContent($value) {
-    $this->setViewState('DocumentContent', $value, '');
+	$this->setViewState('DocumentContent', $value, '');
   }
 
   /**
@@ -125,41 +125,41 @@ 
    * @return TAttributeCollection the list of custom parameters
    */
   public function getParameters() {
-    if($params = $this->getViewState('Parameters',null)) {
-      return $params;
-    } else {
-      $params = new TAttributeCollection();
-      $this->setViewState('Parameters', $params, null);
-      return $params;
-    }
+	if($params = $this->getViewState('Parameters',null)) {
+	  return $params;
+	} else {
+	  $params = new TAttributeCollection();
+	  $this->setViewState('Parameters', $params, null);
+	  return $params;
+	}
   }
 
   private function getTransformXmlDocument() {
-    if(($content = $this->getTransformContent()) !== '') {
-      $document = new DOMDocument();
-      $document->loadXML($content);
-      return $document;
-    } else if(($path = $this->getTransformPath()) !== '') {
-      $document = new DOMDocument();
-      $document->load($path);
-      return $document;
-    } else {
-      throw new TConfigurationException('xmltransform_transform_required');
-    }
+	if(($content = $this->getTransformContent()) !== '') {
+	  $document = new DOMDocument();
+	  $document->loadXML($content);
+	  return $document;
+	} else if(($path = $this->getTransformPath()) !== '') {
+	  $document = new DOMDocument();
+	  $document->load($path);
+	  return $document;
+	} else {
+	  throw new TConfigurationException('xmltransform_transform_required');
+	}
   }
 
   private function getSourceXmlDocument() {
-    if(($content = $this->getDocumentContent()) !== '') {
-      $document = new DOMDocument();
-      $document->loadXML($content);
-      return $document;
-    } else if(($path = $this->getDocumentPath()) !== '') {
-      $document = new DOMDocument();
-      $document->load($path);
-      return $document;
-    } else {
-      return null;
-    }
+	if(($content = $this->getDocumentContent()) !== '') {
+	  $document = new DOMDocument();
+	  $document->loadXML($content);
+	  return $document;
+	} else if(($path = $this->getDocumentPath()) !== '') {
+	  $document = new DOMDocument();
+	  $document->load($path);
+	  return $document;
+	} else {
+	  return null;
+	}
   }
 
   /**
@@ -167,27 +167,27 @@ 
    * @param THtmlWriter The writer used for the rendering purpose
    */
   public function render($writer) {
-    if(($document=$this->getSourceXmlDocument()) === null) {
+	if(($document=$this->getSourceXmlDocument()) === null) {
 	  $htmlWriter = Prado::createComponent($this->GetResponse()->getHtmlWriterType(), new TTextWriter());
 	  parent::render($htmlWriter);
-      $document = new DOMDocument();
-      $document->loadXML($htmlWriter->flush());
-    }
-    $stylesheet = $this->getTransformXmlDocument();
-
-    // Perform XSL transformation
-    $xslt = new XSLTProcessor();
-    $xslt->importStyleSheet($stylesheet);
-
-    // Check for parameters
-    $parameters = $this->getParameters();
-    foreach($parameters as $name => $value) {
-      $xslt->setParameter('', $name, $value);
-    }
-    $output = $xslt->transformToXML($document);
-
-    // Write output
-    $writer->write($output);
+	  $document = new DOMDocument();
+	  $document->loadXML($htmlWriter->flush());
+	}
+	$stylesheet = $this->getTransformXmlDocument();
+
+	// Perform XSL transformation
+	$xslt = new XSLTProcessor();
+	$xslt->importStyleSheet($stylesheet);
+
+	// Check for parameters
+	$parameters = $this->getParameters();
+	foreach($parameters as $name => $value) {
+	  $xslt->setParameter('', $name, $value);
+	}
+	$output = $xslt->transformToXML($document);
+
+	// Write output
+	$writer->write($output);
   }
 }
 

Spacing +23 added lines, -23 removed lines

@@ -45,8 +45,8 @@ 
  */
 class TXmlTransform extends \Prado\Web\UI\TControl {
 
-  const EXT_XML_FILE = '.xml';
-  const EXT_XSL_FILE = '.xsl';
+  const EXT_XML_FILE='.xml';
+  const EXT_XSL_FILE='.xsl';
 
   /**
    * Constructor
@@ -72,8 +72,8 @@ 
    */
   public function setTransformPath($value) {
     if(!is_file($value)) {
-      $value = Prado::getPathOfNamespace($value, self::EXT_XSL_FILE);
-      if($value === null) {
+      $value=Prado::getPathOfNamespace($value, self::EXT_XSL_FILE);
+      if($value===null) {
 	throw new TInvalidDataValueException('xmltransform_transformpath_invalid', $value);
       }
     }
@@ -107,8 +107,8 @@ 
    */
   public function setDocumentPath($value) {
     if(!is_file($value)) {
-      $value = Prado::getPathOfNamespace($value, self::EXT_XML_FILE);
-      if($value === null) {
+      $value=Prado::getPathOfNamespace($value, self::EXT_XML_FILE);
+      if($value===null) {
 	throw new TInvalidDataValueException('xmltransform_documentpath_invalid', $value);
       }
     }
@@ -134,22 +134,22 @@ 
    * @return TAttributeCollection the list of custom parameters
    */
   public function getParameters() {
-    if($params = $this->getViewState('Parameters',null)) {
+    if($params=$this->getViewState('Parameters', null)) {
       return $params;
     } else {
-      $params = new TAttributeCollection();
+      $params=new TAttributeCollection();
       $this->setViewState('Parameters', $params, null);
       return $params;
     }
   }
 
   private function getTransformXmlDocument() {
-    if(($content = $this->getTransformContent()) !== '') {
-      $document = new DOMDocument();
+    if(($content=$this->getTransformContent())!=='') {
+      $document=new DOMDocument();
       $document->loadXML($content);
       return $document;
-    } else if(($path = $this->getTransformPath()) !== '') {
-      $document = new DOMDocument();
+    } else if(($path=$this->getTransformPath())!=='') {
+      $document=new DOMDocument();
       $document->load($path);
       return $document;
     } else {
@@ -158,12 +158,12 @@ 
   }
 
   private function getSourceXmlDocument() {
-    if(($content = $this->getDocumentContent()) !== '') {
-      $document = new DOMDocument();
+    if(($content=$this->getDocumentContent())!=='') {
+      $document=new DOMDocument();
       $document->loadXML($content);
       return $document;
-    } else if(($path = $this->getDocumentPath()) !== '') {
-      $document = new DOMDocument();
+    } else if(($path=$this->getDocumentPath())!=='') {
+      $document=new DOMDocument();
       $document->load($path);
       return $document;
     } else {
@@ -176,24 +176,24 @@ 
    * @param THtmlWriter The writer used for the rendering purpose
    */
   public function render($writer) {
-    if(($document=$this->getSourceXmlDocument()) === null) {
-	  $htmlWriter = Prado::createComponent($this->GetResponse()->getHtmlWriterType(), new TTextWriter());
+    if(($document=$this->getSourceXmlDocument())===null) {
+	  $htmlWriter=Prado::createComponent($this->GetResponse()->getHtmlWriterType(), new TTextWriter());
 	  parent::render($htmlWriter);
-      $document = new DOMDocument();
+      $document=new DOMDocument();
       $document->loadXML($htmlWriter->flush());
     }
-    $stylesheet = $this->getTransformXmlDocument();
+    $stylesheet=$this->getTransformXmlDocument();
 
     // Perform XSL transformation
-    $xslt = new XSLTProcessor();
+    $xslt=new XSLTProcessor();
     $xslt->importStyleSheet($stylesheet);
 
     // Check for parameters
-    $parameters = $this->getParameters();
+    $parameters=$this->getParameters();
     foreach($parameters as $name => $value) {
       $xslt->setParameter('', $name, $value);
     }
-    $output = $xslt->transformToXML($document);
+    $output=$xslt->transformToXML($document);
 
     // Write output
     $writer->write($output);

framework/Web/UI/WebControls/TInlineFrame.php

Indentation +2 added lines, -2 removed lines

@@ -159,7 +159,7 @@ 
 	public function setWidth($value)
 	{
 	  if(($value=TPropertyValue::ensureInteger($value))<0)
-	    $value=-1;
+		$value=-1;
 	  $this->setViewState('Width',$value,-1);
 	}
 	
@@ -177,7 +177,7 @@ 
 	public function setHeight($value)
 	{
 	  if(($value=TPropertyValue::ensureInteger($value))<0)
-	    $value=-1;
+		$value=-1;
 	  $this->setViewState('Height',$value,-1);
 	}
 	

Spacing +36 added lines, -36 removed lines

@@ -47,7 +47,7 @@ 
 	 */
 	public function getAlign()
 	{
-		return $this->getViewState('Align',TInlineFrameAlign::NotSet);
+		return $this->getViewState('Align', TInlineFrameAlign::NotSet);
 	}
 
 	/**
@@ -55,7 +55,7 @@ 
 	 */
 	public function setAlign($value)
 	{
-		$this->setViewState('Align',TPropertyValue::ensureEnum($value,'Prado\\Web\\UI\\WebControls\\TInlineFrameAlign'),TInlineFrameAlign::NotSet);
+		$this->setViewState('Align', TPropertyValue::ensureEnum($value, 'Prado\\Web\\UI\\WebControls\\TInlineFrameAlign'), TInlineFrameAlign::NotSet);
 	}
 
 	/**
@@ -63,7 +63,7 @@ 
 	 */
 	public function getDescriptionUrl()
 	{
-		return $this->getViewState('DescriptionUrl','');
+		return $this->getViewState('DescriptionUrl', '');
 	}
 
 	/**
@@ -71,7 +71,7 @@ 
 	 */
 	public function setDescriptionUrl($value)
 	{
-		$this->setViewState('DescriptionUrl',$value,'');
+		$this->setViewState('DescriptionUrl', $value, '');
 	}
 
 	/**
@@ -79,7 +79,7 @@ 
 	 */
 	public function getShowBorder()
 	{
-		return $this->getViewState('ShowBorder',true);
+		return $this->getViewState('ShowBorder', true);
 	}
 
 	/**
@@ -87,7 +87,7 @@ 
 	 */
 	public function setShowBorder($value)
 	{
-		$this->setViewState('ShowBorder',TPropertyValue::ensureBoolean($value),true);
+		$this->setViewState('ShowBorder', TPropertyValue::ensureBoolean($value), true);
 	}
 
 	/**
@@ -95,7 +95,7 @@ 
 	 */
 	public function getFrameUrl()
 	{
-		return $this->getViewState('FrameUrl','');
+		return $this->getViewState('FrameUrl', '');
 	}
 
 	/**
@@ -103,7 +103,7 @@ 
 	 */
 	public function setFrameUrl($value)
 	{
-		$this->setViewState('FrameUrl',$value,'');
+		$this->setViewState('FrameUrl', $value, '');
 	}
 
 	/**
@@ -137,7 +137,7 @@ 
 	 */
 	public function getScrollBars()
 	{
-		return $this->getViewState('ScrollBars',TInlineFrameScrollBars::Auto);
+		return $this->getViewState('ScrollBars', TInlineFrameScrollBars::Auto);
 	}
 
 	/**
@@ -145,7 +145,7 @@ 
 	 */
 	public function setScrollBars($value)
 	{
-		$this->setViewState('ScrollBars',TPropertyValue::ensureEnum($value,'Prado\\Web\\UI\\WebControls\\TInlineFrameScrollBars'),TInlineFrameScrollBars::Auto);
+		$this->setViewState('ScrollBars', TPropertyValue::ensureEnum($value, 'Prado\\Web\\UI\\WebControls\\TInlineFrameScrollBars'), TInlineFrameScrollBars::Auto);
 	}
 
 	/**
@@ -153,7 +153,7 @@ 
 	 */
 	public function getWidth()
 	{
-	  return $this->getViewState('Width',-1);
+	  return $this->getViewState('Width', -1);
 	}
 	
 	/**
@@ -161,9 +161,9 @@ 
 	 */
 	public function setWidth($value)
 	{
-	  if(($value=TPropertyValue::ensureInteger($value))<0)
+	  if(($value=TPropertyValue::ensureInteger($value)) < 0)
 	    $value=-1;
-	  $this->setViewState('Width',$value,-1);
+	  $this->setViewState('Width', $value, -1);
 	}
 	
 	/**
@@ -171,7 +171,7 @@ 
 	 */
 	public function getHeight()
 	{
-	  return $this->getViewState('Height',-1);
+	  return $this->getViewState('Height', -1);
 	}
 	
 	/**
@@ -179,9 +179,9 @@ 
 	 */
 	public function setHeight($value)
 	{
-	  if(($value=TPropertyValue::ensureInteger($value))<0)
+	  if(($value=TPropertyValue::ensureInteger($value)) < 0)
 	    $value=-1;
-	  $this->setViewState('Height',$value,-1);
+	  $this->setViewState('Height', $value, -1);
 	}
 	
 	/**
@@ -190,7 +190,7 @@ 
 	 */
 	public function getMarginWidth()
 	{
-		return $this->getViewState('MarginWidth',-1);
+		return $this->getViewState('MarginWidth', -1);
 	}
 
 	/**
@@ -199,9 +199,9 @@ 
 	 */
 	public function setMarginWidth($value)
 	{
-		if(($value=TPropertyValue::ensureInteger($value))<0)
+		if(($value=TPropertyValue::ensureInteger($value)) < 0)
 			$value=-1;
-		$this->setViewState('MarginWidth',$value,-1);
+		$this->setViewState('MarginWidth', $value, -1);
 	}
 
 	/**
@@ -210,7 +210,7 @@ 
 	 */
 	public function getMarginHeight()
 	{
-		return $this->getViewState('MarginHeight',-1);
+		return $this->getViewState('MarginHeight', -1);
 	}
 
 	/**
@@ -219,9 +219,9 @@ 
 	 */
 	public function setMarginHeight($value)
 	{
-		if(($value=TPropertyValue::ensureInteger($value))<0)
+		if(($value=TPropertyValue::ensureInteger($value)) < 0)
 			$value=-1;
-		$this->setViewState('MarginHeight',$value,-1);
+		$this->setViewState('MarginHeight', $value, -1);
 	}
 
 	/**
@@ -232,37 +232,37 @@ 
 	protected function addAttributesToRender($writer)
 	{
 		if($this->getID()!=='')
-			$writer->addAttribute('name',$this->getUniqueID());
+			$writer->addAttribute('name', $this->getUniqueID());
 
 		if(($src=$this->getFrameUrl())!=='')
-			$writer->addAttribute('src',$src);
+			$writer->addAttribute('src', $src);
 
 		if(($align=strtolower($this->getAlign()))!=='notset')
-			$writer->addAttribute('align',$align);
+			$writer->addAttribute('align', $align);
 
 		$scrollBars=$this->getScrollBars();
 		if($scrollBars===TInlineFrameScrollBars::None)
-			$writer->addAttribute('scrolling','no');
+			$writer->addAttribute('scrolling', 'no');
 		else if($scrollBars===TInlineFrameScrollBars::Both)
-			$writer->addAttribute('scrolling','yes');
+			$writer->addAttribute('scrolling', 'yes');
 
-		if (!$this->getShowBorder())
-			$writer->addAttribute('frameborder','0');
+		if(!$this->getShowBorder())
+			$writer->addAttribute('frameborder', '0');
 
 		if(($longdesc=$this->getDescriptionUrl())!=='')
-			$writer->addAttribute('longdesc',$longdesc);
+			$writer->addAttribute('longdesc', $longdesc);
 		
-		if (($width=$this->getWidth())!==-1)
-		  $writer->addAttribute('width',$width);
+		if(($width=$this->getWidth())!==-1)
+		  $writer->addAttribute('width', $width);
 		
-		if (($height=$this->getHeight())!==-1)
-		  $writer->addAttribute('height',$height);
+		if(($height=$this->getHeight())!==-1)
+		  $writer->addAttribute('height', $height);
 
 		if(($marginheight=$this->getMarginHeight())!==-1)
-			$writer->addAttribute('marginheight',$marginheight);
+			$writer->addAttribute('marginheight', $marginheight);
 
 		if(($marginwidth=$this->getMarginWidth())!==-1)
-			$writer->addAttribute('marginwidth',$marginwidth);
+			$writer->addAttribute('marginwidth', $marginwidth);
 
 		parent::addAttributesToRender($writer);
 	}

Doc Comments +1 added lines

@@ -84,6 +84,7 @@
 
 	/**
 	 * @param boolean whether there should be a visual separator between the frames.
+	 * @param boolean $value
 	 */
 	public function setShowBorder($value)
 	{