pradosoft / prado

framework/Collections/TPriorityMap.php

Doc Comments +8 added lines, -1 removed lines

@@ -114,6 +114,7 @@ 
 
 	/**
 	 * @param boolean whether this list is read-only or not
+	 * @param boolean $value
 	 */
 	protected function setReadOnly($value)
 	{
@@ -131,6 +132,7 @@ 
 	/**
 	 * This must be called internally or when instantiated.
 	 * @param numeric sets the default priority of inserted items without a specified priority
+	 * @param integer $value
 	 */
 	protected function setDefaultPriority($value)
 	{
@@ -148,6 +150,7 @@ 
 	/**
 	 * This must be called internally or when instantiated.
 	 * @param integer The precision of numeric priorities.
+	 * @param integer $value
 	 */
 	protected function setPrecision($value)
 	{
@@ -270,6 +273,8 @@ 
 	 * This method is exactly the same as {@link offsetGet}.
 	 * @param mixed the key
 	 * @param numeric|null the priority.  default: null, filled in with the default priority numeric.
+	 * @param string $key
+	 * @param integer $priority
 	 * @return numeric old priority of the item
 	 */
 	public function setPriorityAt($key,$priority=null)
@@ -338,7 +343,7 @@ 
 	 * @param mixed key
 	 * @param mixed value
 	 * @param numeric|null priority, default: null, filled in with default priority
-	 * @return numeric priority at which the pair was added
+	 * @return string priority at which the pair was added
 	 * @throws TInvalidOperationException if the map is read-only
 	 */
 	public function add($key,$value,$priority=null)
@@ -467,6 +472,7 @@ 
 	 * Combines the map elements which have a priority below the parameter value
 	 * @param numeric the cut-off priority.  All items of priority less than this are returned.
 	 * @param boolean whether or not the input cut-off priority is inclusive.  Default: false, not inclusive.
+	 * @param integer $priority
 	 * @return array the array of priorities keys with values of arrays of items that are below a specified priority.
 	 *  The priorities are sorted so important priorities, lower numerics, are first.
 	 */
@@ -487,6 +493,7 @@ 
 	 * Combines the map elements which have a priority above the parameter value
 	 * @param numeric the cut-off priority.  All items of priority greater than this are returned.
 	 * @param boolean whether or not the input cut-off priority is inclusive.  Default: true, inclusive.
+	 * @param integer $priority
 	 * @return array the array of priorities keys with values of arrays of items that are above a specified priority.
 	 *  The priorities are sorted so important priorities, lower numerics, are first.
 	 */

Spacing +113 added lines, -113 removed lines

@@ -60,31 +60,31 @@ 
 	/**
 	 * @var array internal data storage
 	 */
-	private $_d=array();
+	private $_d = array();
 	/**
 	 * @var boolean whether this list is read-only
 	 */
-	private $_r=false;
+	private $_r = false;
 	/**
 	 * @var boolean indicates if the _d is currently ordered.
 	 */
-	private $_o=false;
+	private $_o = false;
 	/**
 	 * @var array cached flattened internal data storage
 	 */
-	private $_fd=null;
+	private $_fd = null;
 	/**
 	 * @var integer number of items contain within the map
 	 */
-	private $_c=0;
+	private $_c = 0;
 	/**
 	 * @var numeric the default priority of items without specified priorities
 	 */
-	private $_dp=10;
+	private $_dp = 10;
 	/**
 	 * @var integer the precision of the numeric priorities within this priority list.
 	 */
-	private $_p=8;
+	private $_p = 8;
 
 	/**
 	 * Constructor.
@@ -95,9 +95,9 @@ 
 	 * @param integer the precision of the numeric priorities
 	 * @throws TInvalidDataTypeException If data is not null and neither an array nor an iterator.
 	 */
-	public function __construct($data=null,$readOnly=false,$defaultPriority=10,$precision=8)
+	public function __construct($data = null, $readOnly = false, $defaultPriority = 10, $precision = 8)
 	{
-		if($data!==null)
+		if ($data !== null)
 			$this->copyFrom($data);
 		$this->setReadOnly($readOnly);
 		$this->setPrecision($precision);
@@ -117,7 +117,7 @@ 
 	 */
 	protected function setReadOnly($value)
 	{
-		$this->_r=TPropertyValue::ensureBoolean($value);
+		$this->_r = TPropertyValue::ensureBoolean($value);
 	}
 
 	/**
@@ -134,7 +134,7 @@ 
 	 */
 	protected function setDefaultPriority($value)
 	{
-		$this->_dp = (string)round(TPropertyValue::ensureFloat($value), $this->_p);
+		$this->_dp = (string) round(TPropertyValue::ensureFloat($value), $this->_p);
 	}
 
 	/**
@@ -151,7 +151,7 @@ 
 	 */
 	protected function setPrecision($value)
 	{
-		$this->_p=TPropertyValue::ensureInteger($value);
+		$this->_p = TPropertyValue::ensureInteger($value);
 	}
 
 	/**
@@ -169,9 +169,9 @@ 
 	 * Orders the priority list internally.
 	 */
 	protected function sortPriorities() {
-		if(!$this->_o) {
+		if (!$this->_o) {
 			ksort($this->_d, SORT_NUMERIC);
-			$this->_o=true;
+			$this->_o = true;
 		}
 	}
 
@@ -180,12 +180,12 @@ 
 	 * @return array array of items in the list in priority and index order
 	 */
 	protected function flattenPriorities() {
-		if(is_array($this->_fd))
+		if (is_array($this->_fd))
 			return $this->_fd;
 
 		$this->sortPriorities();
 		$this->_fd = array();
-		foreach($this->_d as $priority => $itemsatpriority)
+		foreach ($this->_d as $priority => $itemsatpriority)
 			$this->_fd = array_merge($this->_fd, $itemsatpriority);
 		return $this->_fd;
 	}
@@ -214,13 +214,13 @@ 
 	 * it will be set to the default {@link getDefaultPriority}
 	 * @return integer the number of items in the map at the specified priority
 	 */
-	public function getPriorityCount($priority=null)
+	public function getPriorityCount($priority = null)
 	{
-		if($priority===null)
-			$priority=$this->getDefaultPriority();
-		$priority=(string)round(TPropertyValue::ensureFloat($priority),$this->_p);
+		if ($priority === null)
+			$priority = $this->getDefaultPriority();
+		$priority = (string) round(TPropertyValue::ensureFloat($priority), $this->_p);
 
-		if(!isset($this->_d[$priority])||!is_array($this->_d[$priority]))
+		if (!isset($this->_d[$priority]) || !is_array($this->_d[$priority]))
 			return false;
 		return count($this->_d[$priority]);
 	}
@@ -252,16 +252,16 @@ 
 	 * and numeric is a specific priority.  default: false, any priority.
 	 * @return mixed the element at the offset, null if no element is found at the offset
 	 */
-	public function itemAt($key,$priority=false)
+	public function itemAt($key, $priority = false)
 	{
-		if($priority===false){
-			$map=$this->flattenPriorities();
-			return isset($map[$key])?$map[$key]:null;
+		if ($priority === false) {
+			$map = $this->flattenPriorities();
+			return isset($map[$key]) ? $map[$key] : null;
 		} else {
-			if($priority===null)
-				$priority=$this->getDefaultPriority();
-			$priority=(string)round(TPropertyValue::ensureFloat($priority),$this->_p);
-			return (isset($this->_d[$priority])&&isset($this->_d[$priority][$key]))?$this->_d[$priority][$key]:null;
+			if ($priority === null)
+				$priority = $this->getDefaultPriority();
+			$priority = (string) round(TPropertyValue::ensureFloat($priority), $this->_p);
+			return (isset($this->_d[$priority]) && isset($this->_d[$priority][$key])) ? $this->_d[$priority][$key] : null;
 		}
 	}
 
@@ -272,16 +272,16 @@ 
 	 * @param numeric|null the priority.  default: null, filled in with the default priority numeric.
 	 * @return numeric old priority of the item
 	 */
-	public function setPriorityAt($key,$priority=null)
+	public function setPriorityAt($key, $priority = null)
 	{
-		if($priority===null)
-			$priority=$this->getDefaultPriority();
-		$priority=(string)round(TPropertyValue::ensureFloat($priority),$this->_p);
-
-		$oldpriority=$this->priorityAt($key);
-		if($oldpriority!==false&&$oldpriority!=$priority) {
-			$value=$this->remove($key,$oldpriority);
-			$this->add($key,$value,$priority);
+		if ($priority === null)
+			$priority = $this->getDefaultPriority();
+		$priority = (string) round(TPropertyValue::ensureFloat($priority), $this->_p);
+
+		$oldpriority = $this->priorityAt($key);
+		if ($oldpriority !== false && $oldpriority != $priority) {
+			$value = $this->remove($key, $oldpriority);
+			$this->add($key, $value, $priority);
 		}
 		return $oldpriority;
 	}
@@ -291,13 +291,13 @@ 
 	 * @param numeric priority of the items to get.  Defaults to null, filled in with the default priority, if left blank.
 	 * @return array all items at priority in index order, null if there are no items at that priority
 	 */
-	public function itemsAtPriority($priority=null)
+	public function itemsAtPriority($priority = null)
 	{
-		if($priority===null)
-			$priority=$this->getDefaultPriority();
-		$priority=(string)round(TPropertyValue::ensureFloat($priority),$this->_p);
+		if ($priority === null)
+			$priority = $this->getDefaultPriority();
+		$priority = (string) round(TPropertyValue::ensureFloat($priority), $this->_p);
 
-		return isset($this->_d[$priority])?$this->_d[$priority]:null;
+		return isset($this->_d[$priority]) ? $this->_d[$priority] : null;
 	}
 
 	/**
@@ -308,8 +308,8 @@ 
 	public function priorityOf($item)
 	{
 		$this->sortPriorities();
-		foreach($this->_d as $priority=>$items)
-			if(($index=array_search($item,$items,true))!==false)
+		foreach ($this->_d as $priority=>$items)
+			if (($index = array_search($item, $items, true)) !== false)
 				return $priority;
 		return false;
 	}
@@ -322,8 +322,8 @@ 
 	public function priorityAt($key)
 	{
 		$this->sortPriorities();
-		foreach($this->_d as $priority=>$items)
-			if(array_key_exists($key,$items))
+		foreach ($this->_d as $priority=>$items)
+			if (array_key_exists($key, $items))
 				return $priority;
 		return false;
 	}
@@ -341,33 +341,33 @@ 
 	 * @return numeric priority at which the pair was added
 	 * @throws TInvalidOperationException if the map is read-only
 	 */
-	public function add($key,$value,$priority=null)
+	public function add($key, $value, $priority = null)
 	{
-		if($priority===null)
-			$priority=$this->getDefaultPriority();
-		$priority=(string)round(TPropertyValue::ensureFloat($priority),$this->_p);
+		if ($priority === null)
+			$priority = $this->getDefaultPriority();
+		$priority = (string) round(TPropertyValue::ensureFloat($priority), $this->_p);
 
-		if(!$this->_r)
+		if (!$this->_r)
 		{
-			foreach($this->_d as $innerpriority=>$items)
-				if(array_key_exists($key,$items))
+			foreach ($this->_d as $innerpriority=>$items)
+				if (array_key_exists($key, $items))
 				{
 					unset($this->_d[$innerpriority][$key]);
 					$this->_c--;
-					if(count($this->_d[$innerpriority])===0)
+					if (count($this->_d[$innerpriority]) === 0)
 						unset($this->_d[$innerpriority]);
 				}
-			if(!isset($this->_d[$priority])) {
-				$this->_d[$priority]=array($key=>$value);
-				$this->_o=false;
+			if (!isset($this->_d[$priority])) {
+				$this->_d[$priority] = array($key=>$value);
+				$this->_o = false;
 			}
 			else
-				$this->_d[$priority][$key]=$value;
+				$this->_d[$priority][$key] = $value;
 			$this->_c++;
-			$this->_fd=null;
+			$this->_fd = null;
 		}
 		else
-			throw new TInvalidOperationException('map_readonly',get_class($this));
+			throw new TInvalidOperationException('map_readonly', get_class($this));
 		return $priority;
 	}
 
@@ -383,45 +383,45 @@ 
 	 * @return mixed the removed value, null if no such key exists.
 	 * @throws TInvalidOperationException if the map is read-only
 	 */
-	public function remove($key,$priority=false)
+	public function remove($key, $priority = false)
 	{
-		if(!$this->_r)
+		if (!$this->_r)
 		{
-			if($priority===null)
-				$priority=$this->getDefaultPriority();
+			if ($priority === null)
+				$priority = $this->getDefaultPriority();
 
-			if($priority===false)
+			if ($priority === false)
 			{
 				$this->sortPriorities();
-				foreach($this->_d as $priority=>$items)
-					if(array_key_exists($key,$items))
+				foreach ($this->_d as $priority=>$items)
+					if (array_key_exists($key, $items))
 					{
-						$value=$this->_d[$priority][$key];
+						$value = $this->_d[$priority][$key];
 						unset($this->_d[$priority][$key]);
 						$this->_c--;
-						if(count($this->_d[$priority])===0)
+						if (count($this->_d[$priority]) === 0)
 						{
 							unset($this->_d[$priority]);
-							$this->_o=false;
+							$this->_o = false;
 						}
-						$this->_fd=null;
+						$this->_fd = null;
 						return $value;
 					}
 				return null;
 			}
 			else
 			{
-				$priority=(string)round(TPropertyValue::ensureFloat($priority),$this->_p);
-				if(isset($this->_d[$priority])&&(isset($this->_d[$priority][$key])||array_key_exists($key,$this->_d[$priority])))
+				$priority = (string) round(TPropertyValue::ensureFloat($priority), $this->_p);
+				if (isset($this->_d[$priority]) && (isset($this->_d[$priority][$key]) || array_key_exists($key, $this->_d[$priority])))
 				{
-					$value=$this->_d[$priority][$key];
+					$value = $this->_d[$priority][$key];
 					unset($this->_d[$priority][$key]);
 					$this->_c--;
-					if(count($this->_d[$priority])===0) {
+					if (count($this->_d[$priority]) === 0) {
 						unset($this->_d[$priority]);
-						$this->_o=false;
+						$this->_o = false;
 					}
-					$this->_fd=null;
+					$this->_fd = null;
 					return $value;
 				}
 				else
@@ -429,7 +429,7 @@ 
 			}
 		}
 		else
-			throw new TInvalidOperationException('map_readonly',get_class($this));
+			throw new TInvalidOperationException('map_readonly', get_class($this));
 	}
 
 	/**
@@ -437,8 +437,8 @@ 
 	 */
 	public function clear()
 	{
-		foreach($this->_d as $priority=>$items)
-			foreach(array_keys($items) as $key)
+		foreach ($this->_d as $priority=>$items)
+			foreach (array_keys($items) as $key)
 				$this->remove($key);
 	}
 
@@ -448,8 +448,8 @@ 
 	 */
 	public function contains($key)
 	{
-		$map=$this->flattenPriorities();
-		return isset($map[$key])||array_key_exists($key,$map);
+		$map = $this->flattenPriorities();
+		return isset($map[$key]) || array_key_exists($key, $map);
 	}
 
 	/**
@@ -470,15 +470,15 @@ 
 	 * @return array the array of priorities keys with values of arrays of items that are below a specified priority.
 	 *  The priorities are sorted so important priorities, lower numerics, are first.
 	 */
-	public function toArrayBelowPriority($priority,$inclusive=false)
+	public function toArrayBelowPriority($priority, $inclusive = false)
 	{
 		$this->sortPriorities();
-		$items=array();
-		foreach($this->_d as $itemspriority=>$itemsatpriority)
+		$items = array();
+		foreach ($this->_d as $itemspriority=>$itemsatpriority)
 		{
-			if((!$inclusive&&$itemspriority>=$priority)||$itemspriority>$priority)
+			if ((!$inclusive && $itemspriority >= $priority) || $itemspriority > $priority)
 				break;
-			$items=array_merge($items,$itemsatpriority);
+			$items = array_merge($items, $itemsatpriority);
 		}
 		return $items;
 	}
@@ -490,15 +490,15 @@ 
 	 * @return array the array of priorities keys with values of arrays of items that are above a specified priority.
 	 *  The priorities are sorted so important priorities, lower numerics, are first.
 	 */
-	public function toArrayAbovePriority($priority,$inclusive=true)
+	public function toArrayAbovePriority($priority, $inclusive = true)
 	{
 		$this->sortPriorities();
-		$items=array();
-		foreach($this->_d as $itemspriority=>$itemsatpriority)
+		$items = array();
+		foreach ($this->_d as $itemspriority=>$itemsatpriority)
 		{
-			if((!$inclusive&&$itemspriority<=$priority)||$itemspriority<$priority)
+			if ((!$inclusive && $itemspriority <= $priority) || $itemspriority < $priority)
 				continue;
-			$items=array_merge($items,$itemsatpriority);
+			$items = array_merge($items, $itemsatpriority);
 		}
 		return $items;
 	}
@@ -512,24 +512,24 @@ 
 	 */
 	public function copyFrom($data)
 	{
-		if($data instanceof TPriorityMap)
+		if ($data instanceof TPriorityMap)
 		{
-			if($this->getCount()>0)
+			if ($this->getCount() > 0)
 				$this->clear();
-			foreach($data->getPriorities() as $priority) {
-				foreach($data->itemsAtPriority($priority) as $key => $value) {
-					$this->add($key,$value,$priority);
+			foreach ($data->getPriorities() as $priority) {
+				foreach ($data->itemsAtPriority($priority) as $key => $value) {
+					$this->add($key, $value, $priority);
 				}
 			}
 		}
-		else if(is_array($data)||$data instanceof Traversable)
+		else if (is_array($data) || $data instanceof Traversable)
 		{
-			if($this->getCount()>0)
+			if ($this->getCount() > 0)
 				$this->clear();
-			foreach($data as $key=>$value)
-				$this->add($key,$value);
+			foreach ($data as $key=>$value)
+				$this->add($key, $value);
 		}
-		else if($data!==null)
+		else if ($data !== null)
 			throw new TInvalidDataTypeException('map_data_not_iterable');
 	}
 
@@ -542,20 +542,20 @@ 
 	 */
 	public function mergeWith($data)
 	{
-		if($data instanceof TPriorityMap)
+		if ($data instanceof TPriorityMap)
 		{
-			foreach($data->getPriorities() as $priority)
+			foreach ($data->getPriorities() as $priority)
 			{
-				foreach($data->itemsAtPriority($priority) as $key => $value)
-					$this->add($key,$value,$priority);
+				foreach ($data->itemsAtPriority($priority) as $key => $value)
+					$this->add($key, $value, $priority);
 			}
 		}
-		else if(is_array($data)||$data instanceof Traversable)
+		else if (is_array($data) || $data instanceof Traversable)
 		{
-			foreach($data as $key=>$value)
-				$this->add($key,$value);
+			foreach ($data as $key=>$value)
+				$this->add($key, $value);
 		}
-		else if($data!==null)
+		else if ($data !== null)
 			throw new TInvalidDataTypeException('map_data_not_iterable');
 	}
 
@@ -587,9 +587,9 @@ 
 	 * @param integer the offset to set element
 	 * @param mixed the element value
 	 */
-	public function offsetSet($offset,$item)
+	public function offsetSet($offset, $item)
 	{
-		$this->add($offset,$item);
+		$this->add($offset, $item);
 	}
 
 	/**

Braces +84 added lines, -64 removed lines

@@ -97,8 +97,9 @@ 
 	 */
 	public function __construct($data=null,$readOnly=false,$defaultPriority=10,$precision=8)
 	{
-		if($data!==null)
-			$this->copyFrom($data);
+		if($data!==null) {
+					$this->copyFrom($data);
+		}
 		$this->setReadOnly($readOnly);
 		$this->setPrecision($precision);
 		$this->setDefaultPriority($defaultPriority);
@@ -180,13 +181,15 @@ 
 	 * @return array array of items in the list in priority and index order
 	 */
 	protected function flattenPriorities() {
-		if(is_array($this->_fd))
-			return $this->_fd;
+		if(is_array($this->_fd)) {
+					return $this->_fd;
+		}
 
 		$this->sortPriorities();
 		$this->_fd = array();
-		foreach($this->_d as $priority => $itemsatpriority)
-			$this->_fd = array_merge($this->_fd, $itemsatpriority);
+		foreach($this->_d as $priority => $itemsatpriority) {
+					$this->_fd = array_merge($this->_fd, $itemsatpriority);
+		}
 		return $this->_fd;
 	}
 
@@ -216,12 +219,14 @@ 
 	 */
 	public function getPriorityCount($priority=null)
 	{
-		if($priority===null)
-			$priority=$this->getDefaultPriority();
+		if($priority===null) {
+					$priority=$this->getDefaultPriority();
+		}
 		$priority=(string)round(TPropertyValue::ensureFloat($priority),$this->_p);
 
-		if(!isset($this->_d[$priority])||!is_array($this->_d[$priority]))
-			return false;
+		if(!isset($this->_d[$priority])||!is_array($this->_d[$priority])) {
+					return false;
+		}
 		return count($this->_d[$priority]);
 	}
 
@@ -258,8 +263,9 @@ 
 			$map=$this->flattenPriorities();
 			return isset($map[$key])?$map[$key]:null;
 		} else {
-			if($priority===null)
-				$priority=$this->getDefaultPriority();
+			if($priority===null) {
+							$priority=$this->getDefaultPriority();
+			}
 			$priority=(string)round(TPropertyValue::ensureFloat($priority),$this->_p);
 			return (isset($this->_d[$priority])&&isset($this->_d[$priority][$key]))?$this->_d[$priority][$key]:null;
 		}
@@ -274,8 +280,9 @@ 
 	 */
 	public function setPriorityAt($key,$priority=null)
 	{
-		if($priority===null)
-			$priority=$this->getDefaultPriority();
+		if($priority===null) {
+					$priority=$this->getDefaultPriority();
+		}
 		$priority=(string)round(TPropertyValue::ensureFloat($priority),$this->_p);
 
 		$oldpriority=$this->priorityAt($key);
@@ -293,8 +300,9 @@ 
 	 */
 	public function itemsAtPriority($priority=null)
 	{
-		if($priority===null)
-			$priority=$this->getDefaultPriority();
+		if($priority===null) {
+					$priority=$this->getDefaultPriority();
+		}
 		$priority=(string)round(TPropertyValue::ensureFloat($priority),$this->_p);
 
 		return isset($this->_d[$priority])?$this->_d[$priority]:null;
@@ -308,9 +316,10 @@ 
 	public function priorityOf($item)
 	{
 		$this->sortPriorities();
-		foreach($this->_d as $priority=>$items)
-			if(($index=array_search($item,$items,true))!==false)
+		foreach($this->_d as $priority=>$items) {
+					if(($index=array_search($item,$items,true))!==false)
 				return $priority;
+		}
 		return false;
 	}
 
@@ -322,9 +331,10 @@ 
 	public function priorityAt($key)
 	{
 		$this->sortPriorities();
-		foreach($this->_d as $priority=>$items)
-			if(array_key_exists($key,$items))
+		foreach($this->_d as $priority=>$items) {
+					if(array_key_exists($key,$items))
 				return $priority;
+		}
 		return false;
 	}
 
@@ -343,31 +353,34 @@ 
 	 */
 	public function add($key,$value,$priority=null)
 	{
-		if($priority===null)
-			$priority=$this->getDefaultPriority();
+		if($priority===null) {
+					$priority=$this->getDefaultPriority();
+		}
 		$priority=(string)round(TPropertyValue::ensureFloat($priority),$this->_p);
 
 		if(!$this->_r)
 		{
-			foreach($this->_d as $innerpriority=>$items)
-				if(array_key_exists($key,$items))
+			foreach($this->_d as $innerpriority=>$items) {
+							if(array_key_exists($key,$items))
 				{
 					unset($this->_d[$innerpriority][$key]);
+			}
 					$this->_c--;
-					if(count($this->_d[$innerpriority])===0)
-						unset($this->_d[$innerpriority]);
+					if(count($this->_d[$innerpriority])===0) {
+											unset($this->_d[$innerpriority]);
+					}
 				}
 			if(!isset($this->_d[$priority])) {
 				$this->_d[$priority]=array($key=>$value);
 				$this->_o=false;
+			} else {
+							$this->_d[$priority][$key]=$value;
 			}
-			else
-				$this->_d[$priority][$key]=$value;
 			$this->_c++;
 			$this->_fd=null;
+		} else {
+					throw new TInvalidOperationException('map_readonly',get_class($this));
 		}
-		else
-			throw new TInvalidOperationException('map_readonly',get_class($this));
 		return $priority;
 	}
 
@@ -387,16 +400,18 @@ 
 	{
 		if(!$this->_r)
 		{
-			if($priority===null)
-				$priority=$this->getDefaultPriority();
+			if($priority===null) {
+							$priority=$this->getDefaultPriority();
+			}
 
 			if($priority===false)
 			{
 				$this->sortPriorities();
-				foreach($this->_d as $priority=>$items)
-					if(array_key_exists($key,$items))
+				foreach($this->_d as $priority=>$items) {
+									if(array_key_exists($key,$items))
 					{
 						$value=$this->_d[$priority][$key];
+				}
 						unset($this->_d[$priority][$key]);
 						$this->_c--;
 						if(count($this->_d[$priority])===0)
@@ -408,8 +423,7 @@ 
 						return $value;
 					}
 				return null;
-			}
-			else
+			} else
 			{
 				$priority=(string)round(TPropertyValue::ensureFloat($priority),$this->_p);
 				if(isset($this->_d[$priority])&&(isset($this->_d[$priority][$key])||array_key_exists($key,$this->_d[$priority])))
@@ -423,13 +437,13 @@ 
 					}
 					$this->_fd=null;
 					return $value;
+				} else {
+									return null;
 				}
-				else
-					return null;
 			}
+		} else {
+					throw new TInvalidOperationException('map_readonly',get_class($this));
 		}
-		else
-			throw new TInvalidOperationException('map_readonly',get_class($this));
 	}
 
 	/**
@@ -437,9 +451,10 @@ 
 	 */
 	public function clear()
 	{
-		foreach($this->_d as $priority=>$items)
-			foreach(array_keys($items) as $key)
+		foreach($this->_d as $priority=>$items) {
+					foreach(array_keys($items) as $key)
 				$this->remove($key);
+		}
 	}
 
 	/**
@@ -476,8 +491,9 @@ 
 		$items=array();
 		foreach($this->_d as $itemspriority=>$itemsatpriority)
 		{
-			if((!$inclusive&&$itemspriority>=$priority)||$itemspriority>$priority)
-				break;
+			if((!$inclusive&&$itemspriority>=$priority)||$itemspriority>$priority) {
+							break;
+			}
 			$items=array_merge($items,$itemsatpriority);
 		}
 		return $items;
@@ -496,8 +512,9 @@ 
 		$items=array();
 		foreach($this->_d as $itemspriority=>$itemsatpriority)
 		{
-			if((!$inclusive&&$itemspriority<=$priority)||$itemspriority<$priority)
-				continue;
+			if((!$inclusive&&$itemspriority<=$priority)||$itemspriority<$priority) {
+							continue;
+			}
 			$items=array_merge($items,$itemsatpriority);
 		}
 		return $items;
@@ -514,23 +531,25 @@ 
 	{
 		if($data instanceof TPriorityMap)
 		{
-			if($this->getCount()>0)
-				$this->clear();
+			if($this->getCount()>0) {
+							$this->clear();
+			}
 			foreach($data->getPriorities() as $priority) {
 				foreach($data->itemsAtPriority($priority) as $key => $value) {
 					$this->add($key,$value,$priority);
 				}
 			}
-		}
-		else if(is_array($data)||$data instanceof Traversable)
+		} else if(is_array($data)||$data instanceof Traversable)
 		{
-			if($this->getCount()>0)
-				$this->clear();
-			foreach($data as $key=>$value)
-				$this->add($key,$value);
+			if($this->getCount()>0) {
+							$this->clear();
+			}
+			foreach($data as $key=>$value) {
+							$this->add($key,$value);
+			}
+		} else if($data!==null) {
+					throw new TInvalidDataTypeException('map_data_not_iterable');
 		}
-		else if($data!==null)
-			throw new TInvalidDataTypeException('map_data_not_iterable');
 	}
 
 	/**
@@ -546,17 +565,18 @@ 
 		{
 			foreach($data->getPriorities() as $priority)
 			{
-				foreach($data->itemsAtPriority($priority) as $key => $value)
-					$this->add($key,$value,$priority);
+				foreach($data->itemsAtPriority($priority) as $key => $value) {
+									$this->add($key,$value,$priority);
+				}
 			}
-		}
-		else if(is_array($data)||$data instanceof Traversable)
+		} else if(is_array($data)||$data instanceof Traversable)
 		{
-			foreach($data as $key=>$value)
-				$this->add($key,$value);
+			foreach($data as $key=>$value) {
+							$this->add($key,$value);
+			}
+		} else if($data!==null) {
+					throw new TInvalidDataTypeException('map_data_not_iterable');
 		}
-		else if($data!==null)
-			throw new TInvalidDataTypeException('map_data_not_iterable');
 	}
 
 	/**

framework/Collections/TQueue.php

Doc Comments +3 added lines

@@ -49,6 +49,7 @@ 
 	 * Constructor.
 	 * Initializes the queue with an array or an iterable object.
 	 * @param array|Iterator the intial data. Default is null, meaning no initialization.
+	 * @param integer[] $data
 	 * @throws TInvalidDataTypeException If data is not null and neither an array nor an iterator.
 	 */
 	public function __construct($data=null)
@@ -97,6 +98,7 @@ 
 
 	/**
 	 * @param mixed the item
+	 * @param integer $item
 	 * @return boolean whether the queue contains the item
 	 */
 	public function contains($item)
@@ -137,6 +139,7 @@ 
 	/**
 	 * Adds an object to the end of the queue.
 	 * @param mixed the item to be appended into the queue
+	 * @param integer $item
 	 */
 	public function enqueue($item)
 	{

Spacing +20 added lines, -20 removed lines

@@ -32,18 +32,18 @@ 
  * @package System.Collections
  * @since 3.1
  */
-class TQueue extends TComponent implements IteratorAggregate,Countable
+class TQueue extends TComponent implements IteratorAggregate, Countable
 {
 	/**
 	 * internal data storage
 	 * @var array
 	 */
-	private $_d=array();
+	private $_d = array();
 	/**
 	 * number of items
 	 * @var integer
 	 */
-	private $_c=0;
+	private $_c = 0;
 
 	/**
 	 * Constructor.
@@ -51,9 +51,9 @@ 
 	 * @param array|Iterator the intial data. Default is null, meaning no initialization.
 	 * @throws TInvalidDataTypeException If data is not null and neither an array nor an iterator.
 	 */
-	public function __construct($data=null)
+	public function __construct($data = null)
 	{
-		if($data!==null)
+		if ($data !== null)
 			$this->copyFrom($data);
 	}
 
@@ -73,16 +73,16 @@ 
 	 */
 	public function copyFrom($data)
 	{
-		if(is_array($data) || ($data instanceof Traversable))
+		if (is_array($data) || ($data instanceof Traversable))
 		{
 			$this->clear();
-			foreach($data as $item)
+			foreach ($data as $item)
 			{
-				$this->_d[]=$item;
+				$this->_d[] = $item;
 				++$this->_c;
 			}
 		}
-		else if($data!==null)
+		else if ($data !== null)
 			throw new TInvalidDataTypeException('queue_data_not_iterable');
 	}
 
@@ -91,8 +91,8 @@ 
 	 */
 	public function clear()
 	{
-		$this->_c=0;
-		$this->_d=array();
+		$this->_c = 0;
+		$this->_d = array();
 	}
 
 	/**
@@ -101,7 +101,7 @@ 
 	 */
 	public function contains($item)
 	{
-		return array_search($item,$this->_d,true)!==false;
+		return array_search($item, $this->_d, true) !== false;
 	}
 
 	/**
@@ -112,7 +112,7 @@ 
 	 */
 	public function peek()
 	{
-		if($this->_c===0)
+		if ($this->_c === 0)
 			throw new TInvalidOperationException('queue_empty');
 		else
 			return $this->_d[0];
@@ -125,7 +125,7 @@ 
 	 */
 	public function dequeue()
 	{
-		if($this->_c===0)
+		if ($this->_c === 0)
 			throw new TInvalidOperationException('queue_empty');
 		else
 		{
@@ -151,7 +151,7 @@ 
 	 */
 	public function getIterator()
 	{
-		return new ArrayIterator( $this->_d );
+		return new ArrayIterator($this->_d);
 	}
 
 	/**
@@ -207,9 +207,9 @@ 
 	 */
 	public function __construct(&$data)
 	{
-		$this->_d=&$data;
-		$this->_i=0;
-		$this->_c=count($this->_d);
+		$this->_d = &$data;
+		$this->_i = 0;
+		$this->_c = count($this->_d);
 	}
 
 	/**
@@ -218,7 +218,7 @@ 
 	 */
 	public function rewind()
 	{
-		$this->_i=0;
+		$this->_i = 0;
 	}
 
 	/**
@@ -257,7 +257,7 @@ 
 	 */
 	public function valid()
 	{
-		return $this->_i<$this->_c;
+		return $this->_i < $this->_c;
 	}
 }
 

Braces +13 added lines, -11 removed lines

@@ -53,8 +53,9 @@ 
 	 */
 	public function __construct($data=null)
 	{
-		if($data!==null)
-			$this->copyFrom($data);
+		if($data!==null) {
+					$this->copyFrom($data);
+		}
 	}
 
 	/**
@@ -81,9 +82,9 @@ 
 				$this->_d[]=$item;
 				++$this->_c;
 			}
+		} else if($data!==null) {
+					throw new TInvalidDataTypeException('queue_data_not_iterable');
 		}
-		else if($data!==null)
-			throw new TInvalidDataTypeException('queue_data_not_iterable');
 	}
 
 	/**
@@ -112,10 +113,11 @@ 
 	 */
 	public function peek()
 	{
-		if($this->_c===0)
-			throw new TInvalidOperationException('queue_empty');
-		else
-			return $this->_d[0];
+		if($this->_c===0) {
+					throw new TInvalidOperationException('queue_empty');
+		} else {
+					return $this->_d[0];
+		}
 	}
 
 	/**
@@ -125,9 +127,9 @@ 
 	 */
 	public function dequeue()
 	{
-		if($this->_c===0)
-			throw new TInvalidOperationException('queue_empty');
-		else
+		if($this->_c===0) {
+					throw new TInvalidOperationException('queue_empty');
+		} else
 		{
 			--$this->_c;
 			return array_shift($this->_d);

framework/Collections/TStack.php

Doc Comments +1 added lines

@@ -46,6 +46,7 @@
 	 * Constructor.
 	 * Initializes the stack with an array or an iterable object.
 	 * @param array|Iterator the initial data. Default is null, meaning no initialization.
+	 * @param integer[] $data
 	 * @throws TInvalidDataTypeException If data is not null and neither an array nor an iterator.
 	 */
 	public function __construct($data=null)

Indentation +8 added lines, -8 removed lines

@@ -1,13 +1,13 @@
 <?php
 /**
- * TStack, TStackIterator classes
- *
- * @author Qiang Xue <qiang.xue@gmail.com>
- * @link https://github.com/pradosoft/prado
- * @copyright Copyright &copy; 2005-2015 The PRADO Group
- * @license https://github.com/pradosoft/prado/blob/master/COPYRIGHT
- * @package System.Collections
- */
+	 * TStack, TStackIterator classes
+	 *
+	 * @author Qiang Xue <qiang.xue@gmail.com>
+	 * @link https://github.com/pradosoft/prado
+	 * @copyright Copyright &copy; 2005-2015 The PRADO Group
+	 * @license https://github.com/pradosoft/prado/blob/master/COPYRIGHT
+	 * @package System.Collections
+	 */
 
 /**
  * TStack class

Spacing +21 added lines, -21 removed lines

@@ -29,18 +29,18 @@ 
  * @package System.Collections
  * @since 3.0
  */
-class TStack extends TComponent implements IteratorAggregate,Countable
+class TStack extends TComponent implements IteratorAggregate, Countable
 {
 	/**
 	 * internal data storage
 	 * @var array
 	 */
-	private $_d=array();
+	private $_d = array();
 	/**
 	 * number of items
 	 * @var integer
 	 */
-	private $_c=0;
+	private $_c = 0;
 
 	/**
 	 * Constructor.
@@ -48,9 +48,9 @@ 
 	 * @param array|Iterator the initial data. Default is null, meaning no initialization.
 	 * @throws TInvalidDataTypeException If data is not null and neither an array nor an iterator.
 	 */
-	public function __construct($data=null)
+	public function __construct($data = null)
 	{
-		if($data!==null)
+		if ($data !== null)
 			$this->copyFrom($data);
 	}
 
@@ -70,16 +70,16 @@ 
 	 */
 	public function copyFrom($data)
 	{
-		if(is_array($data) || ($data instanceof Traversable))
+		if (is_array($data) || ($data instanceof Traversable))
 		{
 			$this->clear();
-			foreach($data as $item)
+			foreach ($data as $item)
 			{
-				$this->_d[]=$item;
+				$this->_d[] = $item;
 				++$this->_c;
 			}
 		}
-		else if($data!==null)
+		else if ($data !== null)
 			throw new TInvalidDataTypeException('stack_data_not_iterable');
 	}
 
@@ -88,8 +88,8 @@ 
 	 */
 	public function clear()
 	{
-		$this->_c=0;
-		$this->_d=array();
+		$this->_c = 0;
+		$this->_d = array();
 	}
 
 	/**
@@ -98,7 +98,7 @@ 
 	 */
 	public function contains($item)
 	{
-		return array_search($item,$this->_d,true)!==false;
+		return array_search($item, $this->_d, true) !== false;
 	}
 
 	/**
@@ -109,10 +109,10 @@ 
 	 */
 	public function peek()
 	{
-		if($this->_c===0)
+		if ($this->_c === 0)
 			throw new TInvalidOperationException('stack_empty');
 		else
-			return $this->_d[$this->_c-1];
+			return $this->_d[$this->_c - 1];
 	}
 
 	/**
@@ -122,7 +122,7 @@ 
 	 */
 	public function pop()
 	{
-		if($this->_c===0)
+		if ($this->_c === 0)
 			throw new TInvalidOperationException('stack_empty');
 		else
 		{
@@ -148,7 +148,7 @@ 
 	 */
 	public function getIterator()
 	{
-		return new ArrayIterator( $this->_d );
+		return new ArrayIterator($this->_d);
 	}
 
 	/**
@@ -204,9 +204,9 @@ 
 	 */
 	public function __construct(&$data)
 	{
-		$this->_d=&$data;
-		$this->_i=0;
-		$this->_c=count($this->_d);
+		$this->_d = &$data;
+		$this->_i = 0;
+		$this->_c = count($this->_d);
 	}
 
 	/**
@@ -215,7 +215,7 @@ 
 	 */
 	public function rewind()
 	{
-		$this->_i=0;
+		$this->_i = 0;
 	}
 
 	/**
@@ -254,7 +254,7 @@ 
 	 */
 	public function valid()
 	{
-		return $this->_i<$this->_c;
+		return $this->_i < $this->_c;
 	}
 }
 

Braces +13 added lines, -11 removed lines

@@ -50,8 +50,9 @@ 
 	 */
 	public function __construct($data=null)
 	{
-		if($data!==null)
-			$this->copyFrom($data);
+		if($data!==null) {
+					$this->copyFrom($data);
+		}
 	}
 
 	/**
@@ -78,9 +79,9 @@ 
 				$this->_d[]=$item;
 				++$this->_c;
 			}
+		} else if($data!==null) {
+					throw new TInvalidDataTypeException('stack_data_not_iterable');
 		}
-		else if($data!==null)
-			throw new TInvalidDataTypeException('stack_data_not_iterable');
 	}
 
 	/**
@@ -109,10 +110,11 @@ 
 	 */
 	public function peek()
 	{
-		if($this->_c===0)
-			throw new TInvalidOperationException('stack_empty');
-		else
-			return $this->_d[$this->_c-1];
+		if($this->_c===0) {
+					throw new TInvalidOperationException('stack_empty');
+		} else {
+					return $this->_d[$this->_c-1];
+		}
 	}
 
 	/**
@@ -122,9 +124,9 @@ 
 	 */
 	public function pop()
 	{
-		if($this->_c===0)
-			throw new TInvalidOperationException('stack_empty');
-		else
+		if($this->_c===0) {
+					throw new TInvalidOperationException('stack_empty');
+		} else
 		{
 			--$this->_c;
 			return array_pop($this->_d);

framework/Data/ActiveRecord/Relations/TActiveRecordHasManyAssociation.php

Doc Comments +6 added lines

@@ -208,6 +208,7 @@ 
 	 * @param string active record class name.
 	 * @param array row data
 	 * @param array foreign key column names
+	 * @param string $type
 	 * @return TActiveRecord
 	 */
 	protected function createFkObject($type,$row,$foreignKeys)
@@ -227,6 +228,7 @@ 
 	 * @param TTableInfo association table info
 	 * @param array field names
 	 * @param array field values
+	 * @param TActiveRecordCriteria $criteria
 	 */
 	public function createCommand($criteria, $foreignKeys,$indexValues,$sourceKeys)
 	{
@@ -341,6 +343,10 @@ 
 		return $this->getCommandBuilder()->onExecuteCommand($command, $command->execute()) > 0;
 	}
 
+	/**
+	 * @param TActiveRecord $obj
+	 * @param TDbCommandBuilder $builder
+	 */
 	private function updateAssociationTable($obj,$fkObjects, $builder)
 	{
 		$source = $this->getSourceRecordValues($obj);

Indentation +4 added lines, -4 removed lines

@@ -90,10 +90,10 @@
 	private $_association_columns=array();
 
 	/**
-	* Get the foreign key index values from the results and make calls to the
-	* database to find the corresponding foreign objects using association table.
-	* @param array original results.
-	*/
+	 * Get the foreign key index values from the results and make calls to the
+	 * database to find the corresponding foreign objects using association table.
+	 * @param array original results.
+	 */
 	protected function collectForeignObjects(&$results)
 	{
 		list($sourceKeys, $foreignKeys) = $this->getRelationForeignKeys();

Spacing +52 added lines, -52 removed lines

@@ -87,7 +87,7 @@ 
 	private $_association;
 	private $_sourceTable;
 	private $_foreignTable;
-	private $_association_columns=array();
+	private $_association_columns = array();
 
 	/**
 	* Get the foreign key index values from the results and make calls to the
@@ -99,7 +99,7 @@ 
 		list($sourceKeys, $foreignKeys) = $this->getRelationForeignKeys();
 		$properties = array_values($sourceKeys);
 		$indexValues = $this->getIndexValues($properties, $results);
-		$this->fetchForeignObjects($results, $foreignKeys,$indexValues,$sourceKeys);
+		$this->fetchForeignObjects($results, $foreignKeys, $indexValues, $sourceKeys);
 	}
 
 	/**
@@ -119,16 +119,16 @@ 
 	 */
 	protected function getAssociationTable()
 	{
-		if($this->_association===null)
+		if ($this->_association === null)
 		{
 			$gateway = $this->getSourceRecord()->getRecordGateway();
 			$conn = $this->getSourceRecord()->getDbConnection();
 			//table name may include the fk column name separated with a dot.
 			$table = explode('.', $this->getContext()->getAssociationTable());
-			if(count($table)>1)
+			if (count($table) > 1)
 			{
 				$columns = preg_replace('/^\((.*)\)/', '\1', $table[1]);
-				$this->_association_columns = preg_split('/\s*[, ]\*/',$columns);
+				$this->_association_columns = preg_split('/\s*[, ]\*/', $columns);
 			}
 			$this->_association = $gateway->getTableInfo($conn, $table[0]);
 		}
@@ -140,7 +140,7 @@ 
 	 */
 	protected function getSourceTable()
 	{
-		if($this->_sourceTable===null)
+		if ($this->_sourceTable === null)
 		{
 			$gateway = $this->getSourceRecord()->getRecordGateway();
 			$this->_sourceTable = $gateway->getRecordTableInfo($this->getSourceRecord());
@@ -153,7 +153,7 @@ 
 	 */
 	protected function getForeignTable()
 	{
-		if($this->_foreignTable===null)
+		if ($this->_foreignTable === null)
 		{
 			$gateway = $this->getSourceRecord()->getRecordGateway();
 			$fkObject = $this->getContext()->getForeignRecordFinder();
@@ -185,20 +185,20 @@ 
 	 * @param array field names
 	 * @param array foreign key index values.
 	 */
-	protected function fetchForeignObjects(&$results,$foreignKeys,$indexValues,$sourceKeys)
+	protected function fetchForeignObjects(&$results, $foreignKeys, $indexValues, $sourceKeys)
 	{
 		$criteria = $this->getCriteria();
 		$finder = $this->getContext()->getForeignRecordFinder();
 		$type = get_class($finder);
-		$command = $this->createCommand($criteria, $foreignKeys,$indexValues,$sourceKeys);
+		$command = $this->createCommand($criteria, $foreignKeys, $indexValues, $sourceKeys);
 		$srcProps = array_keys($sourceKeys);
-		$collections=array();
-		foreach($this->getCommandBuilder()->onExecuteCommand($command, $command->query()) as $row)
+		$collections = array();
+		foreach ($this->getCommandBuilder()->onExecuteCommand($command, $command->query()) as $row)
 		{
 			$hash = $this->getObjectHash($row, $srcProps);
-			foreach($srcProps as $column)
+			foreach ($srcProps as $column)
 				unset($row[$column]);
-			$obj = $this->createFkObject($type,$row,$foreignKeys);
+			$obj = $this->createFkObject($type, $row, $foreignKeys);
 			$collections[$hash][] = $obj;
 		}
 		$this->setResultCollection($results, $collections, array_values($sourceKeys));
@@ -210,13 +210,13 @@ 
 	 * @param array foreign key column names
 	 * @return TActiveRecord
 	 */
-	protected function createFkObject($type,$row,$foreignKeys)
+	protected function createFkObject($type, $row, $foreignKeys)
 	{
 		$obj = TActiveRecord::createRecord($type, $row);
-		if(count($this->_association_columns) > 0)
+		if (count($this->_association_columns) > 0)
 		{
-			$i=0;
-			foreach($foreignKeys as $ref=>$fk)
+			$i = 0;
+			foreach ($foreignKeys as $ref=>$fk)
 				$obj->setColumnValue($ref, $row[$this->_association_columns[$i++]]);
 		}
 		return $obj;
@@ -228,13 +228,13 @@ 
 	 * @param array field names
 	 * @param array field values
 	 */
-	public function createCommand($criteria, $foreignKeys,$indexValues,$sourceKeys)
+	public function createCommand($criteria, $foreignKeys, $indexValues, $sourceKeys)
 	{
-		$innerJoin = $this->getAssociationJoin($foreignKeys,$indexValues,$sourceKeys);
+		$innerJoin = $this->getAssociationJoin($foreignKeys, $indexValues, $sourceKeys);
 		$fkTable = $this->getForeignTable()->getTableFullName();
 		$srcColumns = $this->getSourceColumns($sourceKeys);
-		if(($where=$criteria->getCondition())===null)
-			$where='1=1';
+		if (($where = $criteria->getCondition()) === null)
+			$where = '1=1';
 		$sql = "SELECT {$fkTable}.*, {$srcColumns} FROM {$fkTable} {$innerJoin} WHERE {$where}";
 
 		$parameters = $criteria->getParameters()->toArray();
@@ -243,7 +243,7 @@ 
 		$offset = $criteria->getOffset();
 
 		$builder = $this->getForeignCommandBuilder()->getBuilder();
-		$command = $builder->applyCriterias($sql,$parameters,$ordering,$limit,$offset);
+		$command = $builder->applyCriterias($sql, $parameters, $ordering, $limit, $offset);
 		$this->getCommandBuilder()->onCreateCommand($command, $criteria);
 		return $command;
 	}
@@ -254,12 +254,12 @@ 
 	 */
 	protected function getSourceColumns($sourceKeys)
 	{
-		$columns=array();
+		$columns = array();
 		$table = $this->getAssociationTable();
 		$tableName = $table->getTableFullName();
-		$columnNames = array_merge(array_keys($sourceKeys),$this->_association_columns);
-		foreach($columnNames as $name)
-			$columns[] = $tableName.'.'.$table->getColumn($name)->getColumnName();
+		$columnNames = array_merge(array_keys($sourceKeys), $this->_association_columns);
+		foreach ($columnNames as $name)
+			$columns[] = $tableName . '.' . $table->getColumn($name)->getColumnName();
 		return implode(', ', $columns);
 	}
 
@@ -270,9 +270,9 @@ 
 	 * @param array source table column names.
 	 * @return string inner join condition for M-N relationship via association table.
 	 */
-	protected function getAssociationJoin($foreignKeys,$indexValues,$sourceKeys)
+	protected function getAssociationJoin($foreignKeys, $indexValues, $sourceKeys)
 	{
-		$refInfo= $this->getAssociationTable();
+		$refInfo = $this->getAssociationTable();
 		$fkInfo = $this->getForeignTable();
 
 		$refTable = $refInfo->getTableFullName();
@@ -280,10 +280,10 @@ 
 
 		$joins = array();
 		$hasAssociationColumns = count($this->_association_columns) > 0;
-		$i=0;
-		foreach($foreignKeys as $ref=>$fk)
+		$i = 0;
+		foreach ($foreignKeys as $ref=>$fk)
 		{
-			if($hasAssociationColumns)
+			if ($hasAssociationColumns)
 				$refField = $refInfo->getColumn($this->_association_columns[$i++])->getColumnName();
 			else
 				$refField = $refInfo->getColumn($ref)->getColumnName();
@@ -291,7 +291,7 @@ 
 			$joins[] = "{$fkTable}.{$fkField} = {$refTable}.{$refField}";
 		}
 		$joinCondition = implode(' AND ', $joins);
-		$index = $this->getCommandBuilder()->getIndexKeyCondition($refInfo,array_keys($sourceKeys), $indexValues);
+		$index = $this->getCommandBuilder()->getIndexKeyCondition($refInfo, array_keys($sourceKeys), $indexValues);
 		return "INNER JOIN {$refTable} ON ({$joinCondition}) AND {$index}";
 	}
 
@@ -303,12 +303,12 @@ 
 	{
 		$obj = $this->getContext()->getSourceRecord();
 		$fkObjects = &$obj->{$this->getContext()->getProperty()};
-		$success=true;
-		if(($total = count($fkObjects))> 0)
+		$success = true;
+		if (($total = count($fkObjects)) > 0)
 		{
 			$source = $this->getSourceRecord();
 			$builder = $this->getAssociationTableCommandBuilder();
-			for($i=0;$i<$total;$i++)
+			for ($i = 0; $i < $total; $i++)
 				$success = $fkObjects[$i]->save() && $success;
 			return $this->updateAssociationTable($obj, $fkObjects, $builder) && $success;
 		}
@@ -324,33 +324,33 @@ 
 		return $this->getAssociationTable()->createCommandBuilder($conn);
 	}
 
-	private function hasAssociationData($builder,$data)
+	private function hasAssociationData($builder, $data)
 	{
-		$condition=array();
+		$condition = array();
 		$table = $this->getAssociationTable();
-		foreach($data as $name=>$value)
-			$condition[] = $table->getColumn($name)->getColumnName().' = ?';
-		$command = $builder->createCountCommand(implode(' AND ', $condition),array_values($data));
+		foreach ($data as $name=>$value)
+			$condition[] = $table->getColumn($name)->getColumnName() . ' = ?';
+		$command = $builder->createCountCommand(implode(' AND ', $condition), array_values($data));
 		$result = $this->getCommandBuilder()->onExecuteCommand($command, intval($command->queryScalar()));
 		return intval($result) > 0;
 	}
 
-	private function addAssociationData($builder,$data)
+	private function addAssociationData($builder, $data)
 	{
 		$command = $builder->createInsertCommand($data);
 		return $this->getCommandBuilder()->onExecuteCommand($command, $command->execute()) > 0;
 	}
 
-	private function updateAssociationTable($obj,$fkObjects, $builder)
+	private function updateAssociationTable($obj, $fkObjects, $builder)
 	{
 		$source = $this->getSourceRecordValues($obj);
 		$foreignKeys = $this->findForeignKeys($this->getAssociationTable(), $fkObjects[0]);
-		$success=true;
-		foreach($fkObjects as $fkObject)
+		$success = true;
+		foreach ($fkObjects as $fkObject)
 		{
-			$data = array_merge($source, $this->getForeignObjectValues($foreignKeys,$fkObject));
-			if(!$this->hasAssociationData($builder,$data))
-				$success = $this->addAssociationData($builder,$data) && $success;
+			$data = array_merge($source, $this->getForeignObjectValues($foreignKeys, $fkObject));
+			if (!$this->hasAssociationData($builder, $data))
+				$success = $this->addAssociationData($builder, $data) && $success;
 		}
 		return $success;
 	}
@@ -360,16 +360,16 @@ 
 		$sourceKeys = $this->findForeignKeys($this->getAssociationTable(), $obj);
 		$indexValues = $this->getIndexValues(array_values($sourceKeys), $obj);
 		$data = array();
-		$i=0;
-		foreach($sourceKeys as $name=>$srcKey)
+		$i = 0;
+		foreach ($sourceKeys as $name=>$srcKey)
 			$data[$name] = $indexValues[0][$i++];
 		return $data;
 	}
 
-	private function getForeignObjectValues($foreignKeys,$fkObject)
+	private function getForeignObjectValues($foreignKeys, $fkObject)
 	{
-		$data=array();
-		foreach($foreignKeys as $name=>$fKey)
+		$data = array();
+		foreach ($foreignKeys as $name=>$fKey)
 			$data[$name] = $fkObject->getColumnValue($fKey);
 		return $data;
 	}

Braces +32 added lines, -22 removed lines

@@ -196,8 +196,9 @@ 
 		foreach($this->getCommandBuilder()->onExecuteCommand($command, $command->query()) as $row)
 		{
 			$hash = $this->getObjectHash($row, $srcProps);
-			foreach($srcProps as $column)
-				unset($row[$column]);
+			foreach($srcProps as $column) {
+							unset($row[$column]);
+			}
 			$obj = $this->createFkObject($type,$row,$foreignKeys);
 			$collections[$hash][] = $obj;
 		}
@@ -216,8 +217,9 @@ 
 		if(count($this->_association_columns) > 0)
 		{
 			$i=0;
-			foreach($foreignKeys as $ref=>$fk)
-				$obj->setColumnValue($ref, $row[$this->_association_columns[$i++]]);
+			foreach($foreignKeys as $ref=>$fk) {
+							$obj->setColumnValue($ref, $row[$this->_association_columns[$i++]]);
+			}
 		}
 		return $obj;
 	}
@@ -233,8 +235,9 @@ 
 		$innerJoin = $this->getAssociationJoin($foreignKeys,$indexValues,$sourceKeys);
 		$fkTable = $this->getForeignTable()->getTableFullName();
 		$srcColumns = $this->getSourceColumns($sourceKeys);
-		if(($where=$criteria->getCondition())===null)
-			$where='1=1';
+		if(($where=$criteria->getCondition())===null) {
+					$where='1=1';
+		}
 		$sql = "SELECT {$fkTable}.*, {$srcColumns} FROM {$fkTable} {$innerJoin} WHERE {$where}";
 
 		$parameters = $criteria->getParameters()->toArray();
@@ -258,8 +261,9 @@ 
 		$table = $this->getAssociationTable();
 		$tableName = $table->getTableFullName();
 		$columnNames = array_merge(array_keys($sourceKeys),$this->_association_columns);
-		foreach($columnNames as $name)
-			$columns[] = $tableName.'.'.$table->getColumn($name)->getColumnName();
+		foreach($columnNames as $name) {
+					$columns[] = $tableName.'.'.$table->getColumn($name)->getColumnName();
+		}
 		return implode(', ', $columns);
 	}
 
@@ -283,10 +287,11 @@ 
 		$i=0;
 		foreach($foreignKeys as $ref=>$fk)
 		{
-			if($hasAssociationColumns)
-				$refField = $refInfo->getColumn($this->_association_columns[$i++])->getColumnName();
-			else
-				$refField = $refInfo->getColumn($ref)->getColumnName();
+			if($hasAssociationColumns) {
+							$refField = $refInfo->getColumn($this->_association_columns[$i++])->getColumnName();
+			} else {
+							$refField = $refInfo->getColumn($ref)->getColumnName();
+			}
 			$fkField = $fkInfo->getColumn($fk)->getColumnName();
 			$joins[] = "{$fkTable}.{$fkField} = {$refTable}.{$refField}";
 		}
@@ -308,8 +313,9 @@ 
 		{
 			$source = $this->getSourceRecord();
 			$builder = $this->getAssociationTableCommandBuilder();
-			for($i=0;$i<$total;$i++)
-				$success = $fkObjects[$i]->save() && $success;
+			for($i=0;$i<$total;$i++) {
+							$success = $fkObjects[$i]->save() && $success;
+			}
 			return $this->updateAssociationTable($obj, $fkObjects, $builder) && $success;
 		}
 		return $success;
@@ -328,8 +334,9 @@ 
 	{
 		$condition=array();
 		$table = $this->getAssociationTable();
-		foreach($data as $name=>$value)
-			$condition[] = $table->getColumn($name)->getColumnName().' = ?';
+		foreach($data as $name=>$value) {
+					$condition[] = $table->getColumn($name)->getColumnName().' = ?';
+		}
 		$command = $builder->createCountCommand(implode(' AND ', $condition),array_values($data));
 		$result = $this->getCommandBuilder()->onExecuteCommand($command, intval($command->queryScalar()));
 		return intval($result) > 0;
@@ -349,8 +356,9 @@ 
 		foreach($fkObjects as $fkObject)
 		{
 			$data = array_merge($source, $this->getForeignObjectValues($foreignKeys,$fkObject));
-			if(!$this->hasAssociationData($builder,$data))
-				$success = $this->addAssociationData($builder,$data) && $success;
+			if(!$this->hasAssociationData($builder,$data)) {
+							$success = $this->addAssociationData($builder,$data) && $success;
+			}
 		}
 		return $success;
 	}
@@ -361,16 +369,18 @@ 
 		$indexValues = $this->getIndexValues(array_values($sourceKeys), $obj);
 		$data = array();
 		$i=0;
-		foreach($sourceKeys as $name=>$srcKey)
-			$data[$name] = $indexValues[0][$i++];
+		foreach($sourceKeys as $name=>$srcKey) {
+					$data[$name] = $indexValues[0][$i++];
+		}
 		return $data;
 	}
 
 	private function getForeignObjectValues($foreignKeys,$fkObject)
 	{
 		$data=array();
-		foreach($foreignKeys as $name=>$fKey)
-			$data[$name] = $fkObject->getColumnValue($fKey);
+		foreach($foreignKeys as $name=>$fKey) {
+					$data[$name] = $fkObject->getColumnValue($fKey);
+		}
 		return $data;
 	}
 }

framework/Data/ActiveRecord/Relations/TActiveRecordRelation.php

Doc Comments +2 added lines, -1 removed lines

@@ -92,6 +92,7 @@ 
 
 	/**
 	 * Fetch results for current relationship.
+	 * @param TActiveRecord $obj
 	 * @return boolean always true.
 	 */
 	public function fetchResultsInto($obj)
@@ -104,7 +105,6 @@ 
 	 * Returns foreign keys in $fromRecord with source column names as key
 	 * and foreign column names in the corresponding $matchesRecord as value.
 	 * The method returns the first matching foreign key between these 2 records.
-	 * @param TActiveRecord $fromRecord
 	 * @param TActiveRecord $matchesRecord
 	 * @return array foreign keys with source column names as key and foreign column names as value.
 	 */
@@ -212,6 +212,7 @@ 
 	 * @param array source property names
 	 * @param array foreign objects
 	 * @param array foreign object field names.
+	 * @param TActiveRecord[] $fkObjects
 	 */
 	protected function populateResult(&$results,$properties,&$fkObjects,$fields)
 	{

Spacing +29 added lines, -29 removed lines

@@ -71,21 +71,21 @@ 
 	 * @param array method arguments
 	 * @return mixed TActiveRecord or array of TActiveRecord results depending on the method called.
 	 */
-	public function __call($method,$args)
+	public function __call($method, $args)
 	{
-		static $stack=array();
+		static $stack = array();
 
-		$results = call_user_func_array(array($this->getSourceRecord(),$method),$args);
+		$results = call_user_func_array(array($this->getSourceRecord(), $method), $args);
 		$validArray = is_array($results) && count($results) > 0;
-		if($validArray || $results instanceof ArrayAccess || $results instanceof TActiveRecord)
+		if ($validArray || $results instanceof ArrayAccess || $results instanceof TActiveRecord)
 		{
 			$this->collectForeignObjects($results);
-			while($obj = array_pop($stack))
+			while ($obj = array_pop($stack))
 				$obj->collectForeignObjects($results);
 		}
-		else if($results instanceof TActiveRecordRelation)
+		else if ($results instanceof TActiveRecordRelation)
 			$stack[] = $this; //call it later
-		else if($results === null || !$validArray)
+		else if ($results === null || !$validArray)
 			$stack = array();
 		return $results;
 	}
@@ -108,24 +108,24 @@ 
 	 * @param TActiveRecord $matchesRecord
 	 * @return array foreign keys with source column names as key and foreign column names as value.
 	 */
-	protected function findForeignKeys($from, $matchesRecord, $loose=false)
+	protected function findForeignKeys($from, $matchesRecord, $loose = false)
 	{
 		$gateway = $matchesRecord->getRecordGateway();
 		$recordTableInfo = $gateway->getRecordTableInfo($matchesRecord);
 		$matchingTableName = strtolower($recordTableInfo->getTableName());
 		$matchingFullTableName = strtolower($recordTableInfo->getTableFullName());
-		$tableInfo=$from;
-		if($from instanceof TActiveRecord)
+		$tableInfo = $from;
+		if ($from instanceof TActiveRecord)
 			$tableInfo = $gateway->getRecordTableInfo($from);
 		//find first non-empty FK
-		foreach($tableInfo->getForeignKeys() as $fkeys)
+		foreach ($tableInfo->getForeignKeys() as $fkeys)
 		{
 			$fkTable = strtolower($fkeys['table']);
-			if($fkTable===$matchingTableName || $fkTable===$matchingFullTableName)
+			if ($fkTable === $matchingTableName || $fkTable === $matchingFullTableName)
 			{
 				$hasFkField = !$loose && $this->getContext()->hasFkField();
 				$key = $hasFkField ? $this->getFkFields($fkeys['keys']) : $fkeys['keys'];
-				if(!empty($key))
+				if (!empty($key))
 					return $key;
 			}
 		}
@@ -152,9 +152,9 @@ 
 		$matching = array();
 		preg_match_all('/\s*(\S+\.)?([\w-]+)\s*/', $this->getContext()->getFkField(), $matching);
 		$fields = array();
-		foreach($fkeys as $fkName => $field)
+		foreach ($fkeys as $fkName => $field)
 		{
-			if(in_array($fkName, $matching[2]))
+			if (in_array($fkName, $matching[2]))
 				$fields[$fkName] = $field;
 		}
 		return $fields;
@@ -167,9 +167,9 @@ 
 	 */
 	protected function getObjectHash($obj, $properties)
 	{
-		$ids=array();
-		foreach($properties as $property)
-			$ids[] = is_object($obj) ? (string)$obj->getColumnValue($property) : (string)$obj[$property];
+		$ids = array();
+		foreach ($properties as $property)
+			$ids[] = is_object($obj) ? (string) $obj->getColumnValue($property) : (string) $obj[$property];
 		return serialize($ids);
 	}
 
@@ -193,13 +193,13 @@ 
 	 */
 	protected function getIndexValues($keys, $results)
 	{
-		if(!is_array($results) && !$results instanceof ArrayAccess)
+		if (!is_array($results) && !$results instanceof ArrayAccess)
 			$results = array($results);
-		$values=array();
-		foreach($results as $result)
+		$values = array();
+		foreach ($results as $result)
 		{
 			$value = array();
-			foreach($keys as $name)
+			foreach ($keys as $name)
 				$value[] = $result->getColumnValue($name);
 			$values[] = $value;
 		}
@@ -213,11 +213,11 @@ 
 	 * @param array foreign objects
 	 * @param array foreign object field names.
 	 */
-	protected function populateResult(&$results,$properties,&$fkObjects,$fields)
+	protected function populateResult(&$results, $properties, &$fkObjects, $fields)
 	{
-		$collections=array();
-		foreach($fkObjects as $fkObject)
-			$collections[$this->getObjectHash($fkObject, $fields)][]=$fkObject;
+		$collections = array();
+		foreach ($fkObjects as $fkObject)
+			$collections[$this->getObjectHash($fkObject, $fields)][] = $fkObject;
 		$this->setResultCollection($results, $collections, $properties);
 	}
 
@@ -229,9 +229,9 @@ 
 	 */
 	protected function setResultCollection(&$results, &$collections, $properties)
 	{
-		if(is_array($results) || $results instanceof ArrayAccess)
+		if (is_array($results) || $results instanceof ArrayAccess)
 		{
-			for($i=0,$k=count($results);$i<$k;$i++)
+			for ($i = 0, $k = count($results); $i < $k; $i++)
 				$this->setObjectProperty($results[$i], $properties, $collections);
 		}
 		else
@@ -248,7 +248,7 @@ 
 	{
 		$hash = $this->getObjectHash($source, $properties);
 		$prop = $this->getContext()->getProperty();
-		$source->$prop=isset($collections[$hash]) ? $collections[$hash] : array();
+		$source->$prop = isset($collections[$hash]) ? $collections[$hash] : array();
 	}
 }
 

Braces +35 added lines, -24 removed lines

@@ -80,13 +80,16 @@ 
 		if($validArray || $results instanceof ArrayAccess || $results instanceof TActiveRecord)
 		{
 			$this->collectForeignObjects($results);
-			while($obj = array_pop($stack))
-				$obj->collectForeignObjects($results);
+			while($obj = array_pop($stack)) {
+							$obj->collectForeignObjects($results);
+			}
+		} else if($results instanceof TActiveRecordRelation) {
+					$stack[] = $this;
+		}
+		//call it later
+		else if($results === null || !$validArray) {
+					$stack = array();
 		}
-		else if($results instanceof TActiveRecordRelation)
-			$stack[] = $this; //call it later
-		else if($results === null || !$validArray)
-			$stack = array();
 		return $results;
 	}
 
@@ -115,8 +118,9 @@ 
 		$matchingTableName = strtolower($recordTableInfo->getTableName());
 		$matchingFullTableName = strtolower($recordTableInfo->getTableFullName());
 		$tableInfo=$from;
-		if($from instanceof TActiveRecord)
-			$tableInfo = $gateway->getRecordTableInfo($from);
+		if($from instanceof TActiveRecord) {
+					$tableInfo = $gateway->getRecordTableInfo($from);
+		}
 		//find first non-empty FK
 		foreach($tableInfo->getForeignKeys() as $fkeys)
 		{
@@ -125,8 +129,9 @@ 
 			{
 				$hasFkField = !$loose && $this->getContext()->hasFkField();
 				$key = $hasFkField ? $this->getFkFields($fkeys['keys']) : $fkeys['keys'];
-				if(!empty($key))
-					return $key;
+				if(!empty($key)) {
+									return $key;
+				}
 			}
 		}
 
@@ -154,8 +159,9 @@ 
 		$fields = array();
 		foreach($fkeys as $fkName => $field)
 		{
-			if(in_array($fkName, $matching[2]))
-				$fields[$fkName] = $field;
+			if(in_array($fkName, $matching[2])) {
+							$fields[$fkName] = $field;
+			}
 		}
 		return $fields;
 	}
@@ -168,8 +174,9 @@ 
 	protected function getObjectHash($obj, $properties)
 	{
 		$ids=array();
-		foreach($properties as $property)
-			$ids[] = is_object($obj) ? (string)$obj->getColumnValue($property) : (string)$obj[$property];
+		foreach($properties as $property) {
+					$ids[] = is_object($obj) ? (string)$obj->getColumnValue($property) : (string)$obj[$property];
+		}
 		return serialize($ids);
 	}
 
@@ -193,14 +200,16 @@ 
 	 */
 	protected function getIndexValues($keys, $results)
 	{
-		if(!is_array($results) && !$results instanceof ArrayAccess)
-			$results = array($results);
+		if(!is_array($results) && !$results instanceof ArrayAccess) {
+					$results = array($results);
+		}
 		$values=array();
 		foreach($results as $result)
 		{
 			$value = array();
-			foreach($keys as $name)
-				$value[] = $result->getColumnValue($name);
+			foreach($keys as $name) {
+							$value[] = $result->getColumnValue($name);
+			}
 			$values[] = $value;
 		}
 		return $values;
@@ -216,8 +225,9 @@ 
 	protected function populateResult(&$results,$properties,&$fkObjects,$fields)
 	{
 		$collections=array();
-		foreach($fkObjects as $fkObject)
-			$collections[$this->getObjectHash($fkObject, $fields)][]=$fkObject;
+		foreach($fkObjects as $fkObject) {
+					$collections[$this->getObjectHash($fkObject, $fields)][]=$fkObject;
+		}
 		$this->setResultCollection($results, $collections, $properties);
 	}
 
@@ -231,11 +241,12 @@ 
 	{
 		if(is_array($results) || $results instanceof ArrayAccess)
 		{
-			for($i=0,$k=count($results);$i<$k;$i++)
-				$this->setObjectProperty($results[$i], $properties, $collections);
+			for($i=0,$k=count($results);$i<$k;$i++) {
+							$this->setObjectProperty($results[$i], $properties, $collections);
+			}
+		} else {
+					$this->setObjectProperty($results, $properties, $collections);
 		}
-		else
-			$this->setObjectProperty($results, $properties, $collections);
 	}
 
 	/**

framework/Data/ActiveRecord/Relations/TActiveRecordRelationContext.php

Doc Comments +5 added lines, -1 removed lines

@@ -30,6 +30,9 @@ 
 	private $_relation; //data from an entry of TActiveRecord::$RELATION
 	private $_fkeys;
 
+	/**
+	 * @param TActiveRecord $record
+	 */
 	public function __construct($record, $property=null, $relation=null)
 	{
 		$this->_record=$record;
@@ -163,6 +166,7 @@ 
 	 * An instance of TActiveRecordHasOne, TActiveRecordBelongsTo, TActiveRecordHasMany,
 	 * or TActiveRecordHasManyAssocation will be returned.
 	 * @param TActiveRecordCriteria search criteria
+	 * @param TSqlCriteria $criteria
 	 * @return TActiveRecordRelation record relationship handler instnace.
 	 * @throws TActiveRecordException if property is not defined or missing.
 	 */
@@ -195,7 +199,7 @@ 
 	}
 
 	/**
-	 * @return TActiveRecordRelationCommand
+	 * @return boolean
 	 */
 	public function updateAssociatedRecords($updateBelongsTo=false)
 	{

Indentation +9 added lines, -9 removed lines

@@ -1,14 +1,14 @@
 <?php
 /**
- * TActiveRecordRelationContext class.
- *
- * @author Wei Zhuo <weizhuo[at]gmail[dot]com>
- * @link https://github.com/pradosoft/prado
- * @copyright Copyright &copy; 2005-2015 The PRADO Group
- * @license https://github.com/pradosoft/prado/blob/master/COPYRIGHT
- * @version $Id$
- * @package System.Data.ActiveRecord.Relations
- */
+	 * TActiveRecordRelationContext class.
+	 *
+	 * @author Wei Zhuo <weizhuo[at]gmail[dot]com>
+	 * @link https://github.com/pradosoft/prado
+	 * @copyright Copyright &copy; 2005-2015 The PRADO Group
+	 * @license https://github.com/pradosoft/prado/blob/master/COPYRIGHT
+	 * @version $Id$
+	 * @package System.Data.ActiveRecord.Relations
+	 */
 
 /**
  * TActiveRecordRelationContext holds information regarding record relationships

Spacing +23 added lines, -23 removed lines

@@ -30,11 +30,11 @@ 
 	private $_relation; //data from an entry of TActiveRecord::$RELATION
 	private $_fkeys;
 
-	public function __construct($record, $property=null, $relation=null)
+	public function __construct($record, $property = null, $relation = null)
 	{
-		$this->_record=$record;
-		$this->_property=$property;
-		$this->_relation=$relation;
+		$this->_record = $record;
+		$this->_property = $property;
+		$this->_relation = $relation;
 	}
 
 	/**
@@ -43,7 +43,7 @@ 
 	 */
 	public function hasRecordRelation()
 	{
-		return $this->_relation!==null;
+		return $this->_relation !== null;
 	}
 
 	public function getPropertyValue()
@@ -75,8 +75,8 @@ 
 	 */
 	public function getRelationForeignKeys()
 	{
-		if($this->_fkeys===null)
-			$this->_fkeys=$this->getRelationHandler()->getRelationForeignKeys();
+		if ($this->_fkeys === null)
+			$this->_fkeys = $this->getRelationHandler()->getRelationForeignKeys();
 		return $this->_fkeys;
 	}
 
@@ -111,7 +111,7 @@ 
 	 */
 	public function getCondition()
 	{
-		return isset($this->_relation[3])?$this->_relation[3]:null;
+		return isset($this->_relation[3]) ? $this->_relation[3] : null;
 	}
 
 	/**
@@ -120,7 +120,7 @@ 
 	 */
 	public function getParameters()
 	{
-		return isset($this->_relation[4])?$this->_relation[4]:array();
+		return isset($this->_relation[4]) ? $this->_relation[4] : array();
 	}
 
 	/**
@@ -166,16 +166,16 @@ 
 	 * @return TActiveRecordRelation record relationship handler instnace.
 	 * @throws TActiveRecordException if property is not defined or missing.
 	 */
-	public function getRelationHandler($criteria=null)
+	public function getRelationHandler($criteria = null)
 	{
-		if(!$this->hasRecordRelation())
+		if (!$this->hasRecordRelation())
 		{
 			throw new TActiveRecordException('ar_undefined_relation_prop',
 				$this->_property, get_class($this->_record), 'RELATIONS');
 		}
-		if($criteria===null)
+		if ($criteria === null)
 			$criteria = new TActiveRecordCriteria($this->getCondition(), $this->getParameters());
-		switch($this->getRelationType())
+		switch ($this->getRelationType())
 		{
 			case TActiveRecord::HAS_MANY:
 				Prado::using('System.Data.ActiveRecord.Relations.TActiveRecordHasMany');
@@ -197,19 +197,19 @@ 
 	/**
 	 * @return TActiveRecordRelationCommand
 	 */
-	public function updateAssociatedRecords($updateBelongsTo=false)
+	public function updateAssociatedRecords($updateBelongsTo = false)
 	{
-		$success=true;
-		foreach($this->_record->getRecordRelations() as $data)
+		$success = true;
+		foreach ($this->_record->getRecordRelations() as $data)
 		{
 			list($property, $relation) = $data;
-			$belongsTo = $relation[0]==TActiveRecord::BELONGS_TO;
-			if(($updateBelongsTo && $belongsTo) || (!$updateBelongsTo && !$belongsTo))
+			$belongsTo = $relation[0] == TActiveRecord::BELONGS_TO;
+			if (($updateBelongsTo && $belongsTo) || (!$updateBelongsTo && !$belongsTo))
 			{
 				$obj = $this->getSourceRecord();
-				if(!$this->isEmptyFkObject($obj->getColumnValue($property)))
+				if (!$this->isEmptyFkObject($obj->getColumnValue($property)))
 				{
-					$context = new TActiveRecordRelationContext($this->getSourceRecord(),$property,$relation);
+					$context = new TActiveRecordRelationContext($this->getSourceRecord(), $property, $relation);
 					$success = $context->getRelationHandler()->updateAssociatedRecords() && $success;
 				}
 			}
@@ -219,10 +219,10 @@ 
 
 	protected function isEmptyFkObject($obj)
 	{
-		if(is_object($obj))
+		if (is_object($obj))
 			return $obj instanceof TList ? $obj->count() === 0 : false;
-		else if(is_array($obj))
-			return count($obj)===0;
+		else if (is_array($obj))
+			return count($obj) === 0;
 		else
 			return empty($obj);
 	}

Braces +13 added lines, -10 removed lines

@@ -75,8 +75,9 @@ 
 	 */
 	public function getRelationForeignKeys()
 	{
-		if($this->_fkeys===null)
-			$this->_fkeys=$this->getRelationHandler()->getRelationForeignKeys();
+		if($this->_fkeys===null) {
+					$this->_fkeys=$this->getRelationHandler()->getRelationForeignKeys();
+		}
 		return $this->_fkeys;
 	}
 
@@ -173,8 +174,9 @@ 
 			throw new TActiveRecordException('ar_undefined_relation_prop',
 				$this->_property, get_class($this->_record), 'RELATIONS');
 		}
-		if($criteria===null)
-			$criteria = new TActiveRecordCriteria($this->getCondition(), $this->getParameters());
+		if($criteria===null) {
+					$criteria = new TActiveRecordCriteria($this->getCondition(), $this->getParameters());
+		}
 		switch($this->getRelationType())
 		{
 			case TActiveRecord::HAS_MANY:
@@ -219,12 +221,13 @@ 
 
 	protected function isEmptyFkObject($obj)
 	{
-		if(is_object($obj))
-			return $obj instanceof TList ? $obj->count() === 0 : false;
-		else if(is_array($obj))
-			return count($obj)===0;
-		else
-			return empty($obj);
+		if(is_object($obj)) {
+					return $obj instanceof TList ? $obj->count() === 0 : false;
+		} else if(is_array($obj)) {
+					return count($obj)===0;
+		} else {
+					return empty($obj);
+		}
 	}
 }
 

framework/Data/ActiveRecord/Scaffold/InputBuilder/TScaffoldInputBase.php

Doc Comments +9 added lines

@@ -46,6 +46,10 @@ 
 		}
 	}
 
+	/**
+	 * @param TScaffoldEditView $parent
+	 * @param TActiveRecord $record
+	 */
 	public function createScaffoldInput($parent, $item, $column, $record)
 	{
 		$this->_parent=$parent;
@@ -62,6 +66,11 @@ 
 		$label->setForControl(self::DEFAULT_ID);
 	}
 
+	/**
+	 * @param TScaffoldEditView $parent
+	 * @param TDbTableColumn $column
+	 * @param TActiveRecord $record
+	 */
 	public function loadScaffoldInput($parent, $item, $column, $record)
 	{
 		$this->_parent=$parent;

Spacing +13 added lines, -13 removed lines

@@ -21,51 +21,51 @@ 
 	{
 		$record->getDbConnection()->setActive(true); //must be connected before retrieving driver name!
 		$driver = $record->getDbConnection()->getDriverName();
-		switch(strtolower($driver))
+		switch (strtolower($driver))
 		{
 			case 'sqlite': //sqlite 3
 			case 'sqlite2': //sqlite 2
-				require_once(dirname(__FILE__).'/TSqliteScaffoldInput.php');
+				require_once(dirname(__FILE__) . '/TSqliteScaffoldInput.php');
 				return new TSqliteScaffoldInput($conn);
 			case 'mysqli':
 			case 'mysql':
-				require_once(dirname(__FILE__).'/TMysqlScaffoldInput.php');
+				require_once(dirname(__FILE__) . '/TMysqlScaffoldInput.php');
 				return new TMysqlScaffoldInput($conn);
 			case 'pgsql':
-				require_once(dirname(__FILE__).'/TPgsqlScaffoldInput.php');
+				require_once(dirname(__FILE__) . '/TPgsqlScaffoldInput.php');
 				return new TPgsqlScaffoldInput($conn);
 			case 'mssql':
-				require_once(dirname(__FILE__).'/TMssqlScaffoldInput.php');
+				require_once(dirname(__FILE__) . '/TMssqlScaffoldInput.php');
 				return new TMssqlScaffoldInput($conn);
 			case 'ibm':
-				require_once(dirname(__FILE__).'/TIbmScaffoldInput.php');
+				require_once(dirname(__FILE__) . '/TIbmScaffoldInput.php');
 				return new TIbmScaffoldInput($conn);
 			default:
 				throw new TConfigurationException(
-					'scaffold_invalid_database_driver',$driver);
+					'scaffold_invalid_database_driver', $driver);
 		}
 	}
 
 	public function createScaffoldInput($parent, $item, $column, $record)
 	{
-		$this->_parent=$parent;
+		$this->_parent = $parent;
 		$item->setCustomData($column->getColumnId());
 		$this->createControl($item->_input, $column, $record);
-		if($item->_input->findControl(self::DEFAULT_ID))
+		if ($item->_input->findControl(self::DEFAULT_ID))
 			$this->createControlLabel($item->_label, $column, $record);
 	}
 
 	protected function createControlLabel($label, $column, $record)
 	{
-		$fieldname = ucwords(str_replace('_', ' ', $column->getColumnId())).':';
+		$fieldname = ucwords(str_replace('_', ' ', $column->getColumnId())) . ':';
 		$label->setText($fieldname);
 		$label->setForControl(self::DEFAULT_ID);
 	}
 
 	public function loadScaffoldInput($parent, $item, $column, $record)
 	{
-		$this->_parent=$parent;
-		if($this->getIsEnabled($column, $record))
+		$this->_parent = $parent;
+		if ($this->getIsEnabled($column, $record))
 		{
 			$prop = $column->getColumnId();
 			$record->setColumnValue($prop, $this->getControlValue($item->_input, $column, $record));
@@ -81,7 +81,7 @@ 
 	protected function getRecordPropertyValue($column, $record)
 	{
 		$value = $record->getColumnValue($column->getColumnId());
-		if($column->getDefaultValue()!==TDbTableColumn::UNDEFINED_VALUE && $value===null)
+		if ($column->getDefaultValue() !== TDbTableColumn::UNDEFINED_VALUE && $value === null)
 			return $column->getDefaultValue();
 		else
 			return $value;

Braces +8 added lines, -6 removed lines

@@ -51,8 +51,9 @@ 
 		$this->_parent=$parent;
 		$item->setCustomData($column->getColumnId());
 		$this->createControl($item->_input, $column, $record);
-		if($item->_input->findControl(self::DEFAULT_ID))
-			$this->createControlLabel($item->_label, $column, $record);
+		if($item->_input->findControl(self::DEFAULT_ID)) {
+					$this->createControlLabel($item->_label, $column, $record);
+		}
 	}
 
 	protected function createControlLabel($label, $column, $record)
@@ -81,10 +82,11 @@ 
 	protected function getRecordPropertyValue($column, $record)
 	{
 		$value = $record->getColumnValue($column->getColumnId());
-		if($column->getDefaultValue()!==TDbTableColumn::UNDEFINED_VALUE && $value===null)
-			return $column->getDefaultValue();
-		else
-			return $value;
+		if($column->getDefaultValue()!==TDbTableColumn::UNDEFINED_VALUE && $value===null) {
+					return $column->getDefaultValue();
+		} else {
+					return $value;
+		}
 	}
 
 	protected function setRecordPropertyValue($item, $record, $input)

framework/Data/ActiveRecord/Scaffold/InputBuilder/TScaffoldInputCommon.php

Doc Comments +3 added lines

@@ -47,6 +47,9 @@
 		return $control;
 	}
 
+	/**
+	 * @return string
+	 */
 	protected function getDefaultControlValue($container,$column, $record)
 	{
 		$control = $container->findControl(self::DEFAULT_ID);

Spacing +35 added lines, -35 removed lines

@@ -21,7 +21,7 @@ 
 	protected function setNotNullProperty($container, $control, $column, $record)
 	{
 		$this->setDefaultProperty($container, $control, $column, $record);
-		if(!$column->getAllowNull() && !$column->hasSequence())
+		if (!$column->getAllowNull() && !$column->hasSequence())
 			$this->createRequiredValidator($container, $column, $record);
 	}
 
@@ -41,18 +41,18 @@ 
 		$control = new TTextBox();
 		$control->setText($value);
 		$control->setCssClass('default-textbox scaffold_input');
-		if(($len=$column->getColumnSize())!==null)
+		if (($len = $column->getColumnSize()) !== null)
 			$control->setMaxLength($len);
 		$this->setNotNullProperty($container, $control, $column, $record);
 		return $control;
 	}
 
-	protected function getDefaultControlValue($container,$column, $record)
+	protected function getDefaultControlValue($container, $column, $record)
 	{
 		$control = $container->findControl(self::DEFAULT_ID);
-		if($control instanceof TCheckBox)
+		if ($control instanceof TCheckBox)
 			return $control->getChecked();
-		else if($control instanceof TControl)
+		else if ($control instanceof TControl)
 			return $control->getText();
 	}
 
@@ -73,8 +73,8 @@ 
 		$control = new TDropDownList();
 		$years = array();
 		$current = intval(@date('Y'));
-		$from = $current-10; $to=$current+10;
-		for($i = $from; $i <= $to; $i++)
+		$from = $current - 10; $to = $current + 10;
+		for ($i = $from; $i <= $to; $i++)
 			$years[$i] = $i;
 		$control->setDataSource($years);
 		$control->setSelectedValue(empty($value) ? $current : $value);
@@ -98,13 +98,13 @@ 
 		$val = $this->createTypeValidator($container, $column, $record);
 		$val->setDataType(TValidationDataType::Float);
 		$val->setErrorMessage('Please entery a decimal number.');
-		if(($max= $column->getMaxiumNumericConstraint())!==null)
+		if (($max = $column->getMaxiumNumericConstraint()) !== null)
 		{
-			$val = $this->createRangeValidator($container,$column,$record);
+			$val = $this->createRangeValidator($container, $column, $record);
 			$val->setDataType(TValidationDataType::Float);
 			$val->setMaxValue($max);
 			$val->setStrictComparison(true);
-			$val->setErrorMessage('Please entery a decimal number strictly less than '.$max.'.');
+			$val->setErrorMessage('Please entery a decimal number strictly less than ' . $max . '.');
 		}
 		return $control;
 	}
@@ -149,22 +149,22 @@ 
 	protected function createTimeControl($container, $column, $record)
 	{
 		$value = $this->getRecordPropertyValue($column, $record);
-		$hours=array();
-		for($i=0;$i<24;$i++) $hours[] = str_pad($i,2,'0',STR_PAD_LEFT);
-		$mins=array();
-		for($i=0;$i<60;$i++) $mins[] = str_pad($i,2,'0',STR_PAD_LEFT);
+		$hours = array();
+		for ($i = 0; $i < 24; $i++) $hours[] = str_pad($i, 2, '0', STR_PAD_LEFT);
+		$mins = array();
+		for ($i = 0; $i < 60; $i++) $mins[] = str_pad($i, 2, '0', STR_PAD_LEFT);
 		$hour = intval(@date('H'));
 		$min = intval(@date('i'));
 		$sec = intval(@date('s'));
-		if(!empty($value))
+		if (!empty($value))
 		{
-			$match=array();
-			if(preg_match('/(\d+):(\d+):?(\d+)?/', $value, $match))
+			$match = array();
+			if (preg_match('/(\d+):(\d+):?(\d+)?/', $value, $match))
 			{
 				$hour = $match[1];
 				$min = $match[2];
-				if(isset($match[3]))
-					$sec=$match[3];
+				if (isset($match[3]))
+					$sec = $match[3];
 			}
 		}
 
@@ -191,7 +191,7 @@ 
 		$scontrol->setSelectedValue(intval($sec));
 		$container->Controls[] = $scontrol;
 
-		return array($hcontrol,$mcontrol,$scontrol);
+		return array($hcontrol, $mcontrol, $scontrol);
 	}
 
 
@@ -202,8 +202,8 @@ 
 		$control->setFromYear(1900);
 		$control->setInputMode(TDatePickerInputMode::DropDownList);
 		$control->setDateFormat('yyyy-MM-dd');
-		if(!empty($value))
-			$control->setDate(substr($value,0,10));
+		if (!empty($value))
+			$control->setDate(substr($value, 0, 10));
 		$control->setCssClass('date-dropdown');
 		$this->setNotNullProperty($container, $control, $column, $record);
 		return $control;
@@ -215,14 +215,14 @@ 
 		$control = $this->createDateControl($container, $column, $record);
 		$container->Controls[] = ' @ ';
 		$time = $this->createTimeControl($container, $column, $record);
-		if(!empty($value))
+		if (!empty($value))
 		{
-			$match=array();
-			if(preg_match('/(\d+):(\d+):?(\d+)?/', substr($value, 11), $match))
+			$match = array();
+			if (preg_match('/(\d+):(\d+):?(\d+)?/', substr($value, 11), $match))
 			{
 				$time[0]->setSelectedValue(intval($match[1]));
 				$time[1]->setSelectedValue(intval($match[2]));
-				if(isset($match[3]))
+				if (isset($match[3]))
 					$time[2]->setSelectedValue(intval($match[3]));
 			}
 		}
@@ -255,7 +255,7 @@ 
 		$values = $column->getDbTypeValues();
 		$control->setDataSource($values);
 		$control->dataBind();
-		$control->setSelectedIndices($this->getMatchingIndices($values,$selectedValues));
+		$control->setSelectedIndices($this->getMatchingIndices($values, $selectedValues));
 		$control->setID(self::DEFAULT_ID);
 		$control->setCssClass('set-checkboxes');
 		$this->setNotNullProperty($container, $control, $column, $record);
@@ -264,10 +264,10 @@ 
 
 	protected function getMatchingIndices($checks, $values)
 	{
-		$index=array();
-		for($i=0, $k=count($checks); $i<$k; $i++)
+		$index = array();
+		for ($i = 0, $k = count($checks); $i < $k; $i++)
 		{
-			if(in_array($checks[$i], $values))
+			if (in_array($checks[$i], $values))
 				$index[] = $i;
 		}
 		return $index;
@@ -281,8 +281,8 @@ 
 		$values = $column->getDbTypeValues();
 		$control->setDataSource($values);
 		$control->dataBind();
-		$index = $this->getMatchingIndices($values,$selectedValues);
-		if(count($index) > 0)
+		$index = $this->getMatchingIndices($values, $selectedValues);
+		if (count($index) > 0)
 			$control->setSelectedIndex($index[0]);
 		$control->setID(self::DEFAULT_ID);
 		$control->setCssClass('enum-radio-buttons');
@@ -292,10 +292,10 @@ 
 
 	protected function getSetValue($container, $column, $record)
 	{
-		$value=array();
-		foreach($container->findControl(self::DEFAULT_ID)->getItems() as $item)
+		$value = array();
+		foreach ($container->findControl(self::DEFAULT_ID)->getItems() as $item)
 		{
-			if($item->getSelected())
+			if ($item->getSelected())
 				$value[] = $item->getText();
 		}
 		return implode(',', $value);

Braces +38 added lines, -24 removed lines

@@ -21,8 +21,9 @@ 
 	protected function setNotNullProperty($container, $control, $column, $record)
 	{
 		$this->setDefaultProperty($container, $control, $column, $record);
-		if(!$column->getAllowNull() && !$column->hasSequence())
-			$this->createRequiredValidator($container, $column, $record);
+		if(!$column->getAllowNull() && !$column->hasSequence()) {
+					$this->createRequiredValidator($container, $column, $record);
+		}
 	}
 
 	protected function createBooleanControl($container, $column, $record)
@@ -41,8 +42,9 @@ 
 		$control = new TTextBox();
 		$control->setText($value);
 		$control->setCssClass('default-textbox scaffold_input');
-		if(($len=$column->getColumnSize())!==null)
-			$control->setMaxLength($len);
+		if(($len=$column->getColumnSize())!==null) {
+					$control->setMaxLength($len);
+		}
 		$this->setNotNullProperty($container, $control, $column, $record);
 		return $control;
 	}
@@ -50,10 +52,11 @@ 
 	protected function getDefaultControlValue($container,$column, $record)
 	{
 		$control = $container->findControl(self::DEFAULT_ID);
-		if($control instanceof TCheckBox)
-			return $control->getChecked();
-		else if($control instanceof TControl)
-			return $control->getText();
+		if($control instanceof TCheckBox) {
+					return $control->getChecked();
+		} else if($control instanceof TControl) {
+					return $control->getText();
+		}
 	}
 
 	protected function createMultiLineControl($container, $column, $record)
@@ -74,8 +77,9 @@ 
 		$years = array();
 		$current = intval(@date('Y'));
 		$from = $current-10; $to=$current+10;
-		for($i = $from; $i <= $to; $i++)
-			$years[$i] = $i;
+		for($i = $from; $i <= $to; $i++) {
+					$years[$i] = $i;
+		}
 		$control->setDataSource($years);
 		$control->setSelectedValue(empty($value) ? $current : $value);
 		$control->setCssClass('year-dropdown');
@@ -150,9 +154,13 @@ 
 	{
 		$value = $this->getRecordPropertyValue($column, $record);
 		$hours=array();
-		for($i=0;$i<24;$i++) $hours[] = str_pad($i,2,'0',STR_PAD_LEFT);
+		for($i=0;$i<24;$i++) {
+			$hours[] = str_pad($i,2,'0',STR_PAD_LEFT);
+		}
 		$mins=array();
-		for($i=0;$i<60;$i++) $mins[] = str_pad($i,2,'0',STR_PAD_LEFT);
+		for($i=0;$i<60;$i++) {
+			$mins[] = str_pad($i,2,'0',STR_PAD_LEFT);
+		}
 		$hour = intval(@date('H'));
 		$min = intval(@date('i'));
 		$sec = intval(@date('s'));
@@ -163,8 +171,9 @@ 
 			{
 				$hour = $match[1];
 				$min = $match[2];
-				if(isset($match[3]))
-					$sec=$match[3];
+				if(isset($match[3])) {
+									$sec=$match[3];
+				}
 			}
 		}
 
@@ -202,8 +211,9 @@ 
 		$control->setFromYear(1900);
 		$control->setInputMode(TDatePickerInputMode::DropDownList);
 		$control->setDateFormat('yyyy-MM-dd');
-		if(!empty($value))
-			$control->setDate(substr($value,0,10));
+		if(!empty($value)) {
+					$control->setDate(substr($value,0,10));
+		}
 		$control->setCssClass('date-dropdown');
 		$this->setNotNullProperty($container, $control, $column, $record);
 		return $control;
@@ -222,8 +232,9 @@ 
 			{
 				$time[0]->setSelectedValue(intval($match[1]));
 				$time[1]->setSelectedValue(intval($match[2]));
-				if(isset($match[3]))
-					$time[2]->setSelectedValue(intval($match[3]));
+				if(isset($match[3])) {
+									$time[2]->setSelectedValue(intval($match[3]));
+				}
 			}
 		}
 		$time[0]->setID('scaffold_time_hour');
@@ -267,8 +278,9 @@ 
 		$index=array();
 		for($i=0, $k=count($checks); $i<$k; $i++)
 		{
-			if(in_array($checks[$i], $values))
-				$index[] = $i;
+			if(in_array($checks[$i], $values)) {
+							$index[] = $i;
+			}
 		}
 		return $index;
 	}
@@ -282,8 +294,9 @@ 
 		$control->setDataSource($values);
 		$control->dataBind();
 		$index = $this->getMatchingIndices($values,$selectedValues);
-		if(count($index) > 0)
-			$control->setSelectedIndex($index[0]);
+		if(count($index) > 0) {
+					$control->setSelectedIndex($index[0]);
+		}
 		$control->setID(self::DEFAULT_ID);
 		$control->setCssClass('enum-radio-buttons');
 		$this->setNotNullProperty($container, $control, $column, $record);
@@ -295,8 +308,9 @@ 
 		$value=array();
 		foreach($container->findControl(self::DEFAULT_ID)->getItems() as $item)
 		{
-			if($item->getSelected())
-				$value[] = $item->getText();
+			if($item->getSelected()) {
+							$value[] = $item->getText();
+			}
 		}
 		return implode(',', $value);
 	}

framework/Data/ActiveRecord/Scaffold/TScaffoldBase.php

Doc Comments +3 added lines, -1 removed lines

@@ -36,7 +36,7 @@ 
 	private $_record;
 
 	/**
-	 * @return TDbMetaData table/view information
+	 * @return TDbTableInfo table/view information
 	 */
 	protected function getTableInfo()
 	{
@@ -84,6 +84,7 @@ 
 	/**
 	 * Name of the Active Record class to be viewed or scaffolded.
 	 * @param string Active Record class name.
+	 * @param string $value
 	 */
 	public function setRecordClass($value)
 	{
@@ -183,6 +184,7 @@ 
 
 	/**
 	 * @param boolean enable default stylesheet, default is true.
+	 * @param boolean $value
 	 */
 	public function setEnableDefaultStyle($value)
 	{

Spacing +17 added lines, -17 removed lines

@@ -52,7 +52,7 @@ 
 	protected function getRecordPropertyValues($record)
 	{
 		$data = array();
-		foreach($this->getTableInfo()->getColumns() as $name=>$column)
+		foreach ($this->getTableInfo()->getColumns() as $name=>$column)
 			$data[] = $record->getColumnValue($name);
 		return $data;
 	}
@@ -63,10 +63,10 @@ 
 	 */
 	protected function getRecordPkValues($record)
 	{
-		$data=array();
-		foreach($this->getTableInfo()->getColumns() as $name=>$column)
+		$data = array();
+		foreach ($this->getTableInfo()->getColumns() as $name=>$column)
 		{
-			if($column->getIsPrimaryKey())
+			if ($column->getIsPrimaryKey())
 				$data[] = $record->getColumnValue($name);
 		}
 		return $data;
@@ -106,7 +106,7 @@ 
 	 */
 	protected function clearRecordObject()
 	{
-		$this->_record=null;
+		$this->_record = null;
 	}
 
 	/**
@@ -115,26 +115,26 @@ 
 	 * @param array primary key value
 	 * @return TActiveRecord record instance
 	 */
-	protected function getRecordObject($pk=null)
+	protected function getRecordObject($pk = null)
 	{
-		if($this->_record===null)
+		if ($this->_record === null)
 		{
-			if($pk!==null)
+			if ($pk !== null)
 			{
-				$this->_record=$this->getRecordFinder()->findByPk($pk);
-				if($this->_record===null)
+				$this->_record = $this->getRecordFinder()->findByPk($pk);
+				if ($this->_record === null)
 					throw new TConfigurationException('scaffold_invalid_record_pk',
 						$this->getRecordClass(), $pk);
 			}
 			else
 			{
 				$class = $this->getRecordClass();
-				if($class!==null)
-					$this->_record=Prado::createComponent($class);
+				if ($class !== null)
+					$this->_record = Prado::createComponent($class);
 				else
 				{
 					throw new TConfigurationException('scaffold_invalid_record_class',
-						$this->getRecordClass(),$this->getID());
+						$this->getRecordClass(), $this->getID());
 				}
 			}
 		}
@@ -146,7 +146,7 @@ 
 	 */
 	protected function setRecordObject(TActiveRecord $value)
 	{
-		$this->_record=$value;
+		$this->_record = $value;
 	}
 
 	/**
@@ -195,10 +195,10 @@ 
 	public function onPreRender($param)
 	{
 		parent::onPreRender($param);
-		if($this->getEnableDefaultStyle())
+		if ($this->getEnableDefaultStyle())
 		{
-			$url = $this->publishAsset($this->getDefaultStyle().'.css');
-			$this->getPage()->getClientScript()->registerStyleSheetFile($url,$url);
+			$url = $this->publishAsset($this->getDefaultStyle() . '.css');
+			$this->getPage()->getClientScript()->registerStyleSheetFile($url, $url);
 		}
 	}
 }

Braces +13 added lines, -11 removed lines

@@ -52,8 +52,9 @@ 
 	protected function getRecordPropertyValues($record)
 	{
 		$data = array();
-		foreach($this->getTableInfo()->getColumns() as $name=>$column)
-			$data[] = $record->getColumnValue($name);
+		foreach($this->getTableInfo()->getColumns() as $name=>$column) {
+					$data[] = $record->getColumnValue($name);
+		}
 		return $data;
 	}
 
@@ -66,8 +67,9 @@ 
 		$data=array();
 		foreach($this->getTableInfo()->getColumns() as $name=>$column)
 		{
-			if($column->getIsPrimaryKey())
-				$data[] = $record->getColumnValue($name);
+			if($column->getIsPrimaryKey()) {
+							$data[] = $record->getColumnValue($name);
+			}
 		}
 		return $data;
 	}
@@ -122,16 +124,16 @@ 
 			if($pk!==null)
 			{
 				$this->_record=$this->getRecordFinder()->findByPk($pk);
-				if($this->_record===null)
-					throw new TConfigurationException('scaffold_invalid_record_pk',
+				if($this->_record===null) {
+									throw new TConfigurationException('scaffold_invalid_record_pk',
 						$this->getRecordClass(), $pk);
-			}
-			else
+				}
+			} else
 			{
 				$class = $this->getRecordClass();
-				if($class!==null)
-					$this->_record=Prado::createComponent($class);
-				else
+				if($class!==null) {
+									$this->_record=Prado::createComponent($class);
+				} else
 				{
 					throw new TConfigurationException('scaffold_invalid_record_class',
 						$this->getRecordClass(),$this->getID());