pradosoft / prado

framework/Data/SqlMap/DataMapper/TObjectProxy.php

Doc Comments +1 added lines

@@ -116,6 +116,7 @@
 	/**
 	 * @param object handler to method calls.
 	 * @param object the object to by proxied.
+	 * @param TLazyLoadList $handler
 	 */
 	public function __construct($handler, $object)
 	{

Spacing +3 added lines, -3 removed lines

@@ -30,8 +30,8 @@ 
 	 */
 	public function __construct($handler, $object)
 	{
-		$this->_handler = $handler;
-		$this->_object = $object;
+		$this->_handler=$handler;
+		$this->_object=$object;
 	}
 
 	/**
@@ -41,7 +41,7 @@ 
 	 * @param array method arguments
 	 * @return mixed method return value.
 	 */
-	public function __call($method,$params)
+	public function __call($method, $params)
 	{
 		if($this->_handler->hasMethod($method))
 			return $this->_handler->intercept($method, $params);

framework/Data/SqlMap/DataMapper/TSqlMapApplicationCache.php

Doc Comments +3 added lines

@@ -205,6 +205,9 @@
 		return $keyList;
 	}
 
+	/**
+	 * @param TList $keyList
+	 */
 	protected function setKeyList($keyList)
 	{
 		$this->getCache()->set($this->getKeyListId(), $keyList);

Spacing +11 added lines, -11 removed lines

@@ -43,7 +43,7 @@ 
 	protected function getKeyListId()
 	{
 		$id='keyList';
-		if ($this->_cacheModel instanceof TSqlMapCacheModel)
+		if($this->_cacheModel instanceof TSqlMapCacheModel)
 				$id.='_'.$this->_cacheModel->getId();
 		return $id;
 	}
@@ -53,7 +53,7 @@ 
 	 */
 	protected function getKeyList()
 	{
-		if (($keyList=$this->getCache()->get($this->getKeyListId()))===false)
+		if(($keyList=$this->getCache()->get($this->getKeyListId()))===false)
 		{
 			$keyList=new TList();
 			$this->getCache()->set($this->getKeyListId(), $keyList);
@@ -84,7 +84,7 @@ 
 	{
 		$keyList=$this->getKeyList();
 		$cache=$this->getCache();
-		foreach ($keyList as $key)
+		foreach($keyList as $key)
 		{
 			$cache->delete($key);
 		}
@@ -97,18 +97,18 @@ 
 	 */
 	public function get($key)
 	{
-		$result = $this->getCache()->get($key);
-		if ($result === false)
+		$result=$this->getCache()->get($key);
+		if($result===false)
 		{
 			// if the key has not been found in cache (e.g expired), remove from keylist
 			$keyList=$this->getKeyList();
-			if ($keyList->contains($key))
+			if($keyList->contains($key))
 			{
 				$keyList->remove($key);
 				$this->setKeyList($keyList);
 			}
 		}
-		return $result === false ? null : $result;
+		return $result===false ? null : $result;
 	}
 
 	/**
@@ -116,11 +116,11 @@ 
 	 * @param string the key identifying the value to be cached
 	 * @param mixed the value to be cached
 	 */
-	public function set($key, $value,$expire=0,$dependency=null)
+	public function set($key, $value, $expire=0, $dependency=null)
 	{
-		$this->getCache()->set($key, $value, $expire,$dependency);
+		$this->getCache()->set($key, $value, $expire, $dependency);
 		$keyList=$this->getKeyList();
-		if (!$keyList->contains($key))
+		if(!$keyList->contains($key))
 		{
 			$keyList->add($key);
 			$this->setKeyList($keyList);
@@ -138,7 +138,7 @@ 
 	/**
 	 * @throws TSqlMapException not implemented.
 	 */
-	public function add($id,$value,$expire=0,$dependency=null)
+	public function add($id, $value, $expire=0, $dependency=null)
 	{
 		throw new TSqlMapException('sqlmap_use_set_to_store_cache');
 	}

framework/Data/SqlMap/Statements/TMappedStatement.php

Doc Comments +6 added lines, -1 removed lines

@@ -96,7 +96,7 @@ 
 	}
 
 	/**
-	 * @return TSqlMapper The SqlMap used by this MappedStatement
+	 * @return TSqlMapManager The SqlMap used by this MappedStatement
 	 */
 	public function getManager()
 	{
@@ -481,6 +481,7 @@ 
 	 * @param IDbConnection database connection
 	 * @param mixed insert statement parameter
 	 * @param TSqlMapSelectKey select key statement
+	 * @param \Prado\Data\SqlMap\Configuration\TSqlMapSelectKey $selectKey
 	 * @return string last insert ID.
 	 */
 	protected function executeSelectKey($connection, $parameter, $selectKey)
@@ -581,6 +582,7 @@ 
 	 * @param string result object class name
 	 * @param array a result set row retrieved from the database
 	 * @param object the result object, will create if necessary.
+	 * @param string $resultClass
 	 * @return object result object filled with data
 	 */
 	protected function fillResultClass($resultClass, $row, $resultObject)
@@ -603,6 +605,7 @@ 
 	 * Apply the result to a TList or an array.
 	 * @param array a result set row retrieved from the database
 	 * @param object result object, array or list
+	 * @param \ArrayAccess $resultObject
 	 * @return object result filled with data.
 	 */
 	protected function fillResultArrayList($row, $resultObject)
@@ -645,6 +648,7 @@ 
 	 * @param string result map name.
 	 * @param array a result set row retrieved from the database
 	 * @param object result object to fill, will create new instances if required.
+	 * @param string $parentGroup
 	 * @return object result object filled with data.
 	 */
 	protected function fillResultMap($resultMapName, $row, $parentGroup=null, &$resultObject=null)
@@ -784,6 +788,7 @@ 
 	 * Converts the first array value to scalar value of given type.
 	 * @param array list of results
 	 * @param string scalar type.
+	 * @param string $type
 	 * @return mixed scalar value.
 	 */
 	protected function getScalarResult($result, $type)

Spacing +137 added lines, -140 removed lines

@@ -56,7 +56,7 @@ 
 	/**
 	 * @var boolean true when data is mapped to a particular row.
 	 */
-	private $_IsRowDataFound = false;
+	private $_IsRowDataFound=false;
 
 	/**
 	 * @var TSQLMapObjectCollectionTree group by object collection tree
@@ -66,17 +66,17 @@ 
 	/**
 	 * @var Post select is to query for list.
 	 */
-	const QUERY_FOR_LIST = 0;
+	const QUERY_FOR_LIST=0;
 
 	/**
 	 * @var Post select is to query for list.
 	 */
-	const QUERY_FOR_ARRAY = 1;
+	const QUERY_FOR_ARRAY=1;
 
 	/**
 	 * @var Post select is to query for object.
 	 */
-	const QUERY_FOR_OBJECT = 2;
+	const QUERY_FOR_OBJECT=2;
 
 	/**
 	 * @return string Name used to identify the TMappedStatement amongst the others.
@@ -116,7 +116,7 @@ 
 	 */
 	protected function initialGroupByResults()
 	{
-		$this->_groupBy = new TSqlMapObjectCollectionTree();
+		$this->_groupBy=new TSqlMapObjectCollectionTree();
 	}
 
 	/**
@@ -126,9 +126,9 @@ 
 	 */
 	public function __construct(TSqlMapManager $sqlMap, TSqlMapStatement $statement)
 	{
-		$this->_manager = $sqlMap;
-		$this->_statement = $statement;
-		$this->_command = new TPreparedCommand();
+		$this->_manager=$sqlMap;
+		$this->_statement=$statement;
+		$this->_command=new TPreparedCommand();
 		$this->initialGroupByResults();
 	}
 
@@ -173,10 +173,10 @@ 
 	 */
 	protected function executeSQLQueryLimit($connection, $command, $max, $skip)
 	{
-		if($max>-1 || $skip > -1)
+		if($max > -1 || $skip > -1)
 		{
-			$maxStr=$max>0?' LIMIT '.$max:'';
-			$skipStr=$skip>0?' OFFSET '.$skip:'';
+			$maxStr=$max > 0 ? ' LIMIT '.$max : '';
+			$skipStr=$skip > 0 ? ' OFFSET '.$skip : '';
 			$command->setText($command->getText().$maxStr.$skipStr);
 		}
 		$connection->setActive(true);
@@ -213,7 +213,7 @@ 
 	 */
 	public function executeQueryForList($connection, $parameter, $result=null, $skip=-1, $max=-1, $delegate=null)
 	{
-		$sql = $this->_command->create($this->_manager, $connection, $this->_statement, $parameter,$skip,$max);
+		$sql=$this->_command->create($this->_manager, $connection, $this->_statement, $parameter, $skip, $max);
 		return $this->runQueryForList($connection, $parameter, $sql, $result, $delegate);
 	}
 
@@ -236,17 +236,16 @@ 
 	public function runQueryForList($connection, $parameter, $sql, $result, $delegate=null)
 	{
 		$registry=$this->getManager()->getTypeHandlers();
-		$list = $result instanceof \ArrayAccess ? $result :
-							$this->_statement->createInstanceOfListClass($registry);
+		$list=$result instanceof \ArrayAccess ? $result : $this->_statement->createInstanceOfListClass($registry);
 		$connection->setActive(true);
-		$reader = $sql->query();
+		$reader=$sql->query();
 		//$reader = $this->executeSQLQueryLimit($connection, $sql, $max, $skip);
 		if($delegate!==null)
 		{
 			foreach($reader as $row)
 			{
-				$obj = $this->applyResultMap($row);
-				$param = new TResultSetListItemParameter($obj, $parameter, $list);
+				$obj=$this->applyResultMap($row);
+				$param=new TResultSetListItemParameter($obj, $parameter, $list);
 				$this->raiseRowDelegate($delegate, $param);
 			}
 		}
@@ -256,13 +255,13 @@ 
 			foreach($reader as $row)
 			{
 //				var_dump($row);
-				$list[] = $this->applyResultMap($row);
+				$list[]=$this->applyResultMap($row);
 			}
 		}
 
 		if(!$this->_groupBy->isEmpty())
 		{
-			$list = $this->_groupBy->collect();
+			$list=$this->_groupBy->collect();
 			$this->initialGroupByResults();
 		}
 
@@ -284,9 +283,9 @@ 
 	 * @param callback row delegate handler
 	 * @return array An array of object containing the rows keyed by keyProperty.
 	 */
-	public function executeQueryForMap($connection, $parameter, $keyProperty, $valueProperty=null,  $skip=-1, $max=-1, $delegate=null)
+	public function executeQueryForMap($connection, $parameter, $keyProperty, $valueProperty=null, $skip=-1, $max=-1, $delegate=null)
 	{
-		$sql = $this->_command->create($this->_manager, $connection, $this->_statement, $parameter, $skip, $max);
+		$sql=$this->_command->create($this->_manager, $connection, $this->_statement, $parameter, $skip, $max);
 		return $this->runQueryForMap($connection, $parameter, $sql, $keyProperty, $valueProperty, $delegate);
 	}
 
@@ -310,20 +309,19 @@ 
 	 */
 	public function runQueryForMap($connection, $parameter, $command, $keyProperty, $valueProperty=null, $delegate=null)
 	{
-		$map = array();
+		$map=array();
 		//$recordSet = $this->executeSQLQuery($connection, $sql);
 		$connection->setActive(true);
-		$reader = $command->query();
+		$reader=$command->query();
 		if($delegate!==null)
 		{
 			//while($row = $recordSet->fetchRow())
 			foreach($reader as $row)
 			{
-				$obj = $this->applyResultMap($row);
-				$key = TPropertyAccess::get($obj, $keyProperty);
-				$value = ($valueProperty===null) ? $obj :
-							TPropertyAccess::get($obj, $valueProperty);
-				$param = new TResultSetMapItemParameter($key, $value, $parameter, $map);
+				$obj=$this->applyResultMap($row);
+				$key=TPropertyAccess::get($obj, $keyProperty);
+				$value=($valueProperty===null) ? $obj : TPropertyAccess::get($obj, $valueProperty);
+				$param=new TResultSetMapItemParameter($key, $value, $parameter, $map);
 				$this->raiseRowDelegate($delegate, $param);
 			}
 		}
@@ -332,10 +330,9 @@ 
 			//while($row = $recordSet->fetchRow())
 			foreach($reader as $row)
 			{
-				$obj = $this->applyResultMap($row);
-				$key = TPropertyAccess::get($obj, $keyProperty);
-				$map[$key] = ($valueProperty===null) ? $obj :
-								TPropertyAccess::get($obj, $valueProperty);
+				$obj=$this->applyResultMap($row);
+				$key=TPropertyAccess::get($obj, $keyProperty);
+				$map[$key]=($valueProperty===null) ? $obj : TPropertyAccess::get($obj, $valueProperty);
 			}
 		}
 		$this->onExecuteQuery($command);
@@ -352,22 +349,22 @@ 
 	{
 		if(is_string($handler))
 		{
-			call_user_func($handler,$this,$param);
+			call_user_func($handler, $this, $param);
 		}
-		else if(is_callable($handler,true))
+		else if(is_callable($handler, true))
 		{
 			// an array: 0 - object, 1 - method name/path
-			list($object,$method)=$handler;
+			list($object, $method)=$handler;
 			if(is_string($object))	// static method call
-				call_user_func($handler,$this,$param);
+				call_user_func($handler, $this, $param);
 			else
 			{
-				if(($pos=strrpos($method,'.'))!==false)
+				if(($pos=strrpos($method, '.'))!==false)
 				{
-					$object=$this->getSubProperty(substr($method,0,$pos));
-					$method=substr($method,$pos+1);
+					$object=$this->getSubProperty(substr($method, 0, $pos));
+					$method=substr($method, $pos + 1);
 				}
-				$object->$method($this,$param);
+				$object->$method($this, $param);
 			}
 		}
 		else
@@ -384,7 +381,7 @@ 
 	 */
 	public function executeQueryForObject($connection, $parameter, $result=null)
 	{
-		$sql = $this->_command->create($this->_manager, $connection, $this->_statement, $parameter);
+		$sql=$this->_command->create($this->_manager, $connection, $this->_statement, $parameter);
 		return $this->runQueryForObject($connection, $sql, $result);
 	}
 
@@ -403,16 +400,16 @@ 
 	 */
 	public function runQueryForObject($connection, $command, &$result)
 	{
-		$object = null;
+		$object=null;
 		$connection->setActive(true);
 		foreach($command->query() as $row)
-			$object = $this->applyResultMap($row, $result);
+			$object=$this->applyResultMap($row, $result);
 
 		if(!$this->_groupBy->isEmpty())
 		{
-			$list = $this->_groupBy->collect();
+			$list=$this->_groupBy->collect();
 			$this->initialGroupByResults();
-			$object = $list[0];
+			$object=$list[0];
 		}
 
 		$this->executePostSelect($connection);
@@ -430,14 +427,14 @@ 
 	 */
 	public function executeInsert($connection, $parameter)
 	{
-		$generatedKey = $this->getPreGeneratedSelectKey($connection, $parameter);
+		$generatedKey=$this->getPreGeneratedSelectKey($connection, $parameter);
 
-		$command = $this->_command->create($this->_manager, $connection, $this->_statement, $parameter);
+		$command=$this->_command->create($this->_manager, $connection, $this->_statement, $parameter);
 //		var_dump($command,$parameter);
-		$result = $command->execute();
+		$result=$command->execute();
 
 		if($generatedKey===null)
-			$generatedKey = $this->getPostGeneratedSelectKey($connection, $parameter);
+			$generatedKey=$this->getPostGeneratedSelectKey($connection, $parameter);
 
 		$this->executePostSelect($connection);
 		$this->onExecuteQuery($command);
@@ -454,7 +451,7 @@ 
 	{
 		if($this->_statement instanceof TSqlMapInsert)
 		{
-			$selectKey = $this->_statement->getSelectKey();
+			$selectKey=$this->_statement->getSelectKey();
 			if(($selectKey!==null) && !$selectKey->getIsAfter())
 				return $this->executeSelectKey($connection, $parameter, $selectKey);
 		}
@@ -470,7 +467,7 @@ 
 	{
 		if($this->_statement instanceof TSqlMapInsert)
 		{
-			$selectKey = $this->_statement->getSelectKey();
+			$selectKey=$this->_statement->getSelectKey();
 			if(($selectKey!==null) && $selectKey->getIsAfter())
 				return $this->executeSelectKey($connection, $parameter, $selectKey);
 		}
@@ -485,10 +482,10 @@ 
 	 */
 	protected function executeSelectKey($connection, $parameter, $selectKey)
 	{
-		$mappedStatement = $this->getManager()->getMappedStatement($selectKey->getID());
-		$generatedKey = $mappedStatement->executeQueryForObject(
+		$mappedStatement=$this->getManager()->getMappedStatement($selectKey->getID());
+		$generatedKey=$mappedStatement->executeQueryForObject(
 									$connection, $parameter, null);
-		if(strlen($prop = $selectKey->getProperty()) > 0)
+		if(strlen($prop=$selectKey->getProperty()) > 0)
 				TPropertyAccess::set($parameter, $prop, $generatedKey);
 		return $generatedKey;
 	}
@@ -502,8 +499,8 @@ 
 	 */
 	public function executeUpdate($connection, $parameter)
 	{
-		$sql = $this->_command->create($this->getManager(),$connection, $this->_statement, $parameter);
-		$affectedRows = $sql->execute();
+		$sql=$this->_command->create($this->getManager(), $connection, $this->_statement, $parameter);
+		$affectedRows=$sql->execute();
 		//$this->executeSQLQuery($connection, $sql);
 		$this->executePostSelect($connection);
 		$this->onExecuteQuery($sql);
@@ -518,24 +515,24 @@ 
 	{
 		while(count($this->_selectQueue))
 		{
-			$postSelect = array_shift($this->_selectQueue);
-			$method = $postSelect->getMethod();
-			$statement = $postSelect->getStatement();
-			$property = $postSelect->getResultProperty()->getProperty();
-			$keys = $postSelect->getKeys();
-			$resultObject = $postSelect->getResultObject();
-
-			if($method == self::QUERY_FOR_LIST || $method == self::QUERY_FOR_ARRAY)
+			$postSelect=array_shift($this->_selectQueue);
+			$method=$postSelect->getMethod();
+			$statement=$postSelect->getStatement();
+			$property=$postSelect->getResultProperty()->getProperty();
+			$keys=$postSelect->getKeys();
+			$resultObject=$postSelect->getResultObject();
+
+			if($method==self::QUERY_FOR_LIST || $method==self::QUERY_FOR_ARRAY)
 			{
-				$values = $statement->executeQueryForList($connection, $keys, null);
+				$values=$statement->executeQueryForList($connection, $keys, null);
 
-				if($method == self::QUERY_FOR_ARRAY)
-					$values = $values->toArray();
+				if($method==self::QUERY_FOR_ARRAY)
+					$values=$values->toArray();
 				TPropertyAccess::set($resultObject, $property, $values);
 			}
-			else if($method == self::QUERY_FOR_OBJECT)
+			else if($method==self::QUERY_FOR_OBJECT)
 			{
-				$value = $statement->executeQueryForObject($connection, $keys, null);
+				$value=$statement->executeQueryForObject($connection, $keys, null);
 				TPropertyAccess::set($resultObject, $property, $value);
 			}
 		}
@@ -558,21 +555,21 @@ 
 	 */
 	protected function applyResultMap($row, &$resultObject=null)
 	{
-		if($row === false) return null;
+		if($row===false) return null;
 
-		$resultMapName = $this->_statement->getResultMap();
-		$resultClass = $this->_statement->getResultClass();
+		$resultMapName=$this->_statement->getResultMap();
+		$resultClass=$this->_statement->getResultClass();
 
 		$obj=null;
 		if($this->getManager()->getResultMaps()->contains($resultMapName))
-			$obj = $this->fillResultMap($resultMapName, $row, null, $resultObject);
+			$obj=$this->fillResultMap($resultMapName, $row, null, $resultObject);
 		else if(strlen($resultClass) > 0)
-			$obj = $this->fillResultClass($resultClass, $row, $resultObject);
+			$obj=$this->fillResultClass($resultClass, $row, $resultObject);
 		else
-			$obj = $this->fillDefaultResultMap(null, $row, $resultObject);
-		if(class_exists('TActiveRecord',false) && $obj instanceof TActiveRecord)
+			$obj=$this->fillDefaultResultMap(null, $row, $resultObject);
+		if(class_exists('TActiveRecord', false) && $obj instanceof TActiveRecord)
 			//Create a new clean active record.
-			$obj=TActiveRecord::createRecord(get_class($obj),$obj);
+			$obj=TActiveRecord::createRecord(get_class($obj), $obj);
 		return $obj;
 	}
 
@@ -587,8 +584,8 @@ 
 	{
 		if($resultObject===null)
 		{
-			$registry = $this->getManager()->getTypeHandlers();
-			$resultObject = $this->_statement->createInstanceOfResultClass($registry,$row);
+			$registry=$this->getManager()->getTypeHandlers();
+			$resultObject=$this->_statement->createInstanceOfResultClass($registry, $row);
 		}
 
 		if($resultObject instanceOf \ArrayAccess)
@@ -609,10 +606,10 @@ 
 	{
 		if($resultObject instanceof TList)
 			foreach($row as $v)
-				$resultObject[] = $v;
+				$resultObject[]=$v;
 		else
 			foreach($row as $k => $v)
-				$resultObject[$k] = $v;
+				$resultObject[$k]=$v;
 		return $resultObject;
 	}
 
@@ -624,18 +621,18 @@ 
 	 */
 	protected function fillResultObjectProperty($row, $resultObject)
 	{
-		$index = 0;
+		$index=0;
 		$registry=$this->getManager()->getTypeHandlers();
 		foreach($row as $k=>$v)
 		{
-			$property = new TResultProperty;
+			$property=new TResultProperty;
 			if(is_string($k) && strlen($k) > 0)
 				$property->setColumn($k);
 			$property->setColumnIndex(++$index);
-			$type = gettype(TPropertyAccess::get($resultObject,$k));
+			$type=gettype(TPropertyAccess::get($resultObject, $k));
 			$property->setType($type);
-			$value = $property->getPropertyValue($registry,$row);
-			TPropertyAccess::set($resultObject, $k,$value);
+			$value=$property->getPropertyValue($registry, $row);
+			TPropertyAccess::set($resultObject, $k, $value);
 		}
 		return $resultObject;
 	}
@@ -649,12 +646,12 @@ 
 	 */
 	protected function fillResultMap($resultMapName, $row, $parentGroup=null, &$resultObject=null)
 	{
-		$resultMap = $this->getManager()->getResultMap($resultMapName);
-		$registry = $this->getManager()->getTypeHandlers();
-		$resultMap = $resultMap->resolveSubMap($registry,$row);
+		$resultMap=$this->getManager()->getResultMap($resultMapName);
+		$registry=$this->getManager()->getTypeHandlers();
+		$resultMap=$resultMap->resolveSubMap($registry, $row);
 
 		if($resultObject===null)
-			$resultObject = $resultMap->createInstanceOfResult($registry);
+			$resultObject=$resultMap->createInstanceOfResult($registry);
 
 		if(is_object($resultObject))
 		{
@@ -666,7 +663,7 @@ 
 		}
 		else
 		{
-			$resultObject = $this->fillDefaultResultMap($resultMap, $row, $resultObject);
+			$resultObject=$this->fillDefaultResultMap($resultMap, $row, $resultObject);
 		}
 		return $resultObject;
 	}
@@ -681,11 +678,11 @@ 
 	 */
 	protected function addResultMapGroupBy($resultMap, $row, $parent, &$resultObject)
 	{
-		$group = $this->getResultMapGroupKey($resultMap, $row);
+		$group=$this->getResultMapGroupKey($resultMap, $row);
 
 		if(empty($parent))
 		{
-			$rootObject = array('object'=>$resultObject, 'property' => null);
+			$rootObject=array('object'=>$resultObject, 'property' => null);
 			$this->_groupBy->add(null, $group, $rootObject);
 		}
 
@@ -693,23 +690,23 @@ 
 		{
 			//set properties.
 			$this->setObjectProperty($resultMap, $property, $row, $resultObject);
-			$nested = $property->getResultMapping();
+			$nested=$property->getResultMapping();
 
 			//nested property
 			if($this->getManager()->getResultMaps()->contains($nested))
 			{
-				$nestedMap = $this->getManager()->getResultMap($nested);
-				$groupKey = $this->getResultMapGroupKey($nestedMap, $row);
+				$nestedMap=$this->getManager()->getResultMap($nested);
+				$groupKey=$this->getResultMapGroupKey($nestedMap, $row);
 
 				//add the node reference first
 				if(empty($parent))
 					$this->_groupBy->add($group, $groupKey, '');
 
 				//get the nested result mapping value
-				$value = $this->fillResultMap($nested, $row, $groupKey);
+				$value=$this->fillResultMap($nested, $row, $groupKey);
 
 				//add it to the object tree graph
-				$groupObject = array('object'=>$value, 'property' => $property->getProperty());
+				$groupObject=array('object'=>$value, 'property' => $property->getProperty());
 				if(empty($parent))
 					$this->_groupBy->add($group, $groupKey, $groupObject);
 				else
@@ -727,7 +724,7 @@ 
 	 */
 	protected function getResultMapGroupKey($resultMap, $row)
 	{
-		$groupBy = $resultMap->getGroupBy();
+		$groupBy=$resultMap->getGroupBy();
 		if(isset($row[$groupBy]))
 			return $resultMap->getID().$row[$groupBy];
 		else
@@ -748,12 +745,12 @@ 
 			$resultObject='';
 
 		if($resultMap!==null)
-			$result = $this->fillArrayResultMap($resultMap, $row, $resultObject);
+			$result=$this->fillArrayResultMap($resultMap, $row, $resultObject);
 		else
-			$result = $row;
+			$result=$row;
 
 		//if scalar result types
-		if(count($result) == 1 && ($type = gettype($resultObject))!= 'array')
+		if(count($result)==1 && ($type=gettype($resultObject))!='array')
 			return $this->getScalarResult($result, $type);
 		else
 			return $result;
@@ -768,14 +765,14 @@ 
 	 */
 	protected function fillArrayResultMap($resultMap, $row, $resultObject)
 	{
-		$result = array();
+		$result=array();
 		$registry=$this->getManager()->getTypeHandlers();
 		foreach($resultMap->getColumns() as $column)
 		{
 			if(($column->getType()===null)
 				&& ($resultObject!==null) && !is_object($resultObject))
 			$column->setType(gettype($resultObject));
-			$result[$column->getProperty()] = $column->getPropertyValue($registry,$row);
+			$result[$column->getProperty()]=$column->getPropertyValue($registry, $row);
 		}
 		return $result;
 	}
@@ -788,7 +785,7 @@ 
 	 */
 	protected function getScalarResult($result, $type)
 	{
-		$scalar = array_shift($result);
+		$scalar=array_shift($result);
 		settype($scalar, $type);
 		return $scalar;
 	}
@@ -802,23 +799,23 @@ 
 	 */
 	protected function setObjectProperty($resultMap, $property, $row, &$resultObject)
 	{
-		$select = $property->getSelect();
-		$key = $property->getProperty();
-		$nested = $property->getNestedResultMap();
+		$select=$property->getSelect();
+		$key=$property->getProperty();
+		$nested=$property->getNestedResultMap();
 		$registry=$this->getManager()->getTypeHandlers();
-		if($key === '')
+		if($key==='')
 		{
-			$resultObject = $property->getPropertyValue($registry,$row);
+			$resultObject=$property->getPropertyValue($registry, $row);
 		}
-		else if(strlen($select) == 0 && ($nested===null))
+		else if(strlen($select)==0 && ($nested===null))
 		{
-			$value = $property->getPropertyValue($registry,$row);
+			$value=$property->getPropertyValue($registry, $row);
 
-			$this->_IsRowDataFound = $this->_IsRowDataFound || ($value != null);
+			$this->_IsRowDataFound=$this->_IsRowDataFound || ($value!=null);
 			if(is_array($resultObject) || is_object($resultObject))
 				TPropertyAccess::set($resultObject, $key, $value);
 			else
-				$resultObject = $value;
+				$resultObject=$value;
 		}
 		else if($nested!==null)
 		{
@@ -831,9 +828,9 @@ 
 			}
 			else
 			{
-				$obj = $nested->createInstanceOfResult($this->getManager()->getTypeHandlers());
-				if($this->fillPropertyWithResultMap($nested, $row, $obj) == false)
-					$obj = null;
+				$obj=$nested->createInstanceOfResult($this->getManager()->getTypeHandlers());
+				if($this->fillPropertyWithResultMap($nested, $row, $obj)==false)
+					$obj=null;
 				TPropertyAccess::set($resultObject, $key, $obj);
 			}
 		}
@@ -853,9 +850,9 @@ 
 	 */
 	protected function enquequePostSelect($select, $resultMap, $property, $row, $resultObject)
 	{
-		$statement = $this->getManager()->getMappedStatement($select);
-		$key = $this->getPostSelectKeys($resultMap, $property, $row);
-		$postSelect = new TPostSelectBinding;
+		$statement=$this->getManager()->getMappedStatement($select);
+		$key=$this->getPostSelectKeys($resultMap, $property, $row);
+		$postSelect=new TPostSelectBinding;
 		$postSelect->setStatement($statement);
 		$postSelect->setResultObject($resultObject);
 		$postSelect->setResultProperty($property);
@@ -863,10 +860,10 @@ 
 
 		if($property->instanceOfListType($resultObject))
 		{
-			$values = null;
+			$values=null;
 			if($property->getLazyLoad())
 			{
-				$values = TLazyLoadList::newInstance($statement, $key,
+				$values=TLazyLoadList::newInstance($statement, $key,
 								$resultObject, $property->getProperty());
 				TPropertyAccess::set($resultObject, $property->getProperty(), $values);
 			}
@@ -879,7 +876,7 @@ 
 			$postSelect->setMethod(self::QUERY_FOR_OBJECT);
 
 		if(!$property->getLazyLoad())
-			$this->_selectQueue[] = $postSelect;
+			$this->_selectQueue[]=$postSelect;
 	}
 
 	/**
@@ -889,23 +886,23 @@ 
 	 * @param array current row data.
 	 * @return array list of primary key values.
 	 */
-	protected function getPostSelectKeys($resultMap, $property,$row)
+	protected function getPostSelectKeys($resultMap, $property, $row)
 	{
-		$value = $property->getColumn();
-		if(is_int(strpos($value.',',0)) || is_int(strpos($value, '=',0)))
+		$value=$property->getColumn();
+		if(is_int(strpos($value.',', 0)) || is_int(strpos($value, '=', 0)))
 		{
-			$keys = array();
+			$keys=array();
 			foreach(explode(',', $value) as $entry)
 			{
-				$pair =explode('=',$entry);
-				$keys[trim($pair[0])] = $row[trim($pair[1])];
+				$pair=explode('=', $entry);
+				$keys[trim($pair[0])]=$row[trim($pair[1])];
 			}
 			return $keys;
 		}
 		else
 		{
 			$registry=$this->getManager()->getTypeHandlers();
-			return $property->getPropertyValue($registry,$row);
+			return $property->getPropertyValue($registry, $row);
 		}
 	}
 
@@ -918,28 +915,28 @@ 
 	 */
 	protected function fillPropertyWithResultMap($resultMap, $row, &$resultObject)
 	{
-		$dataFound = false;
+		$dataFound=false;
 		foreach($resultMap->getColumns() as $property)
 		{
-			$this->_IsRowDataFound = false;
+			$this->_IsRowDataFound=false;
 			$this->setObjectProperty($resultMap, $property, $row, $resultObject);
-			$dataFound = $dataFound || $this->_IsRowDataFound;
+			$dataFound=$dataFound || $this->_IsRowDataFound;
 		}
-		$this->_IsRowDataFound = $dataFound;
+		$this->_IsRowDataFound=$dataFound;
 		return $dataFound;
 	}
 
 	public function __wakeup()
 	{
-		if (is_null($this->_selectQueue)) $this->_selectQueue = array();
+		if(is_null($this->_selectQueue)) $this->_selectQueue=array();
 	}
 
 	public function __sleep()
 	{
-		$exprops = array(); $cn = __CLASS__;
-		if (!count($this->_selectQueue)) $exprops[] = "\0$cn\0_selectQueue";
-		if (is_null($this->_groupBy)) $exprops[] = "\0$cn\0_groupBy";
-		if (!$this->_IsRowDataFound) $exprops[] = "\0$cn\0_IsRowDataFound";
-		return array_diff(parent::__sleep(),$exprops);
+		$exprops=array(); $cn=__CLASS__;
+		if(!count($this->_selectQueue)) $exprops[]="\0$cn\0_selectQueue";
+		if(is_null($this->_groupBy)) $exprops[]="\0$cn\0_groupBy";
+		if(!$this->_IsRowDataFound) $exprops[]="\0$cn\0_IsRowDataFound";
+		return array_diff(parent::__sleep(), $exprops);
 	}
 }
\ No newline at end of file

Braces +13 added lines, -26 removed lines

@@ -249,8 +249,7 @@ 
 				$param = new TResultSetListItemParameter($obj, $parameter, $list);
 				$this->raiseRowDelegate($delegate, $param);
 			}
-		}
-		else
+		} else
 		{
 			//var_dump($sql,$parameter);
 			foreach($reader as $row)
@@ -326,8 +325,7 @@ 
 				$param = new TResultSetMapItemParameter($key, $value, $parameter, $map);
 				$this->raiseRowDelegate($delegate, $param);
 			}
-		}
-		else
+		} else
 		{
 			//while($row = $recordSet->fetchRow())
 			foreach($reader as $row)
@@ -353,8 +351,7 @@ 
 		if(is_string($handler))
 		{
 			call_user_func($handler,$this,$param);
-		}
-		else if(is_callable($handler,true))
+		} else if(is_callable($handler,true))
 		{
 			// an array: 0 - object, 1 - method name/path
 			list($object,$method)=$handler;
@@ -369,8 +366,7 @@ 
 				}
 				$object->$method($this,$param);
 			}
-		}
-		else
+		} else
 			throw new TInvalidDataValueException('sqlmap_invalid_delegate', $this->getID(), $handler);
 	}
 
@@ -532,8 +528,7 @@ 
 				if($method == self::QUERY_FOR_ARRAY)
 					$values = $values->toArray();
 				TPropertyAccess::set($resultObject, $property, $values);
-			}
-			else if($method == self::QUERY_FOR_OBJECT)
+			} else if($method == self::QUERY_FOR_OBJECT)
 			{
 				$value = $statement->executeQueryForObject($connection, $keys, null);
 				TPropertyAccess::set($resultObject, $property, $value);
@@ -663,8 +658,7 @@ 
 			else
 				foreach($resultMap->getColumns() as $property)
 					$this->setObjectProperty($resultMap, $property, $row, $resultObject);
-		}
-		else
+		} else
 		{
 			$resultObject = $this->fillDefaultResultMap($resultMap, $row, $resultObject);
 		}
@@ -809,8 +803,7 @@ 
 		if($key === '')
 		{
 			$resultObject = $property->getPropertyValue($registry,$row);
-		}
-		else if(strlen($select) == 0 && ($nested===null))
+		} else if(strlen($select) == 0 && ($nested===null))
 		{
 			$value = $property->getPropertyValue($registry,$row);
 
@@ -819,8 +812,7 @@ 
 				TPropertyAccess::set($resultObject, $key, $value);
 			else
 				$resultObject = $value;
-		}
-		else if($nested!==null)
+		} else if($nested!==null)
 		{
 			if($property->instanceOfListType($resultObject) || $property->instanceOfArrayType($resultObject))
 			{
@@ -828,16 +820,14 @@ 
 					throw new TSqlMapExecutionException(
 						'sqlmap_non_groupby_array_list_type', $resultMap->getID(),
 						get_class($resultObject), $key);
-			}
-			else
+			} else
 			{
 				$obj = $nested->createInstanceOfResult($this->getManager()->getTypeHandlers());
 				if($this->fillPropertyWithResultMap($nested, $row, $obj) == false)
 					$obj = null;
 				TPropertyAccess::set($resultObject, $key, $obj);
 			}
-		}
-		else //'select' ResultProperty
+		} else //'select' ResultProperty
 		{
 			$this->enquequePostSelect($select, $resultMap, $property, $row, $resultObject);
 		}
@@ -869,11 +859,9 @@ 
 				$values = TLazyLoadList::newInstance($statement, $key,
 								$resultObject, $property->getProperty());
 				TPropertyAccess::set($resultObject, $property->getProperty(), $values);
-			}
-			else
+			} else
 				$postSelect->setMethod(self::QUERY_FOR_LIST);
-		}
-		else if($property->instanceOfArrayType($resultObject))
+		} else if($property->instanceOfArrayType($resultObject))
 			$postSelect->setMethod(self::QUERY_FOR_ARRAY);
 		else
 			$postSelect->setMethod(self::QUERY_FOR_OBJECT);
@@ -901,8 +889,7 @@ 
 				$keys[trim($pair[0])] = $row[trim($pair[1])];
 			}
 			return $keys;
-		}
-		else
+		} else
 		{
 			$registry=$this->getManager()->getTypeHandlers();
 			return $property->getPropertyValue($registry,$row);

framework/Data/SqlMap/Statements/TPostSelectBinding.php

Doc Comments +4 added lines

@@ -39,5 +39,9 @@
 	public function setKeys($value){ $this->_keys = $value; }
 
 	public function getMethod(){ return $this->_method; }
+
+	/**
+	 * @param integer $value
+	 */
 	public function setMethod($value){ $this->_method = $value; }
 }
\ No newline at end of file

Spacing +10 added lines, -10 removed lines

@@ -26,18 +26,18 @@
 	private $_keys=null;
 	private $_method=TMappedStatement::QUERY_FOR_LIST;
 
-	public function getStatement(){ return $this->_statement; }
-	public function setStatement($value){ $this->_statement = $value; }
+	public function getStatement() { return $this->_statement; }
+	public function setStatement($value) { $this->_statement=$value; }
 
-	public function getResultProperty(){ return $this->_property; }
-	public function setResultProperty($value){ $this->_property = $value; }
+	public function getResultProperty() { return $this->_property; }
+	public function setResultProperty($value) { $this->_property=$value; }
 
-	public function getResultObject(){ return $this->_resultObject; }
-	public function setResultObject($value){ $this->_resultObject = $value; }
+	public function getResultObject() { return $this->_resultObject; }
+	public function setResultObject($value) { $this->_resultObject=$value; }
 
-	public function getKeys(){ return $this->_keys; }
-	public function setKeys($value){ $this->_keys = $value; }
+	public function getKeys() { return $this->_keys; }
+	public function setKeys($value) { $this->_keys=$value; }
 
-	public function getMethod(){ return $this->_method; }
-	public function setMethod($value){ $this->_method = $value; }
+	public function getMethod() { return $this->_method; }
+	public function setMethod($value) { $this->_method=$value; }
 }
\ No newline at end of file

framework/Data/SqlMap/Statements/TResultSetListItemParameter.php

Doc Comments +3 added lines

@@ -24,6 +24,9 @@
 	private $_parameterObject;
 	private $_list;
 
+	/**
+	 * @param \ArrayAccess $list
+	 */
 	public function __construct($result, $parameter, &$list)
 	{
 		$this->_resultObject = $result;

Spacing +3 added lines, -3 removed lines

@@ -26,9 +26,9 @@
 
 	public function __construct($result, $parameter, &$list)
 	{
-		$this->_resultObject = $result;
-		$this->_parameterObject = $parameter;
-		$this->_list = &$list;
+		$this->_resultObject=$result;
+		$this->_parameterObject=$parameter;
+		$this->_list=&$list;
 	}
 
 	public function getResult()

framework/Data/SqlMap/Statements/TSqlMapObjectCollectionTree.php

Doc Comments +1 added lines

@@ -53,6 +53,7 @@
 	 * @param string parent node id
 	 * @param string new node id
 	 * @param mixed node value
+	 * @param string $node
 	 */
 	public function add($parent, $node, $object='')
 	{

Spacing +36 added lines, -36 removed lines

@@ -30,22 +30,22 @@ 
 	/**
 	 * @var array object graph as tree
 	 */
-	private $_tree = array();
+	private $_tree=array();
 	/**
 	 * @var array tree node values
 	 */
-	private $_entries = array();
+	private $_entries=array();
 	/**
 	 * @var array resulting object collection
 	 */
-	private $_list = array();
+	private $_list=array();
 
 	/**
 	 * @return boolean true if the graph is empty
 	 */
 	public function isEmpty()
 	{
-		return count($this->_entries) == 0;
+		return count($this->_entries)==0;
 	}
 
 	/**
@@ -59,22 +59,22 @@ 
 		if(isset($this->_entries[$parent]) && ($this->_entries[$parent]!==null)
 			&& isset($this->_entries[$node]) && ($this->_entries[$node]!==null))
 		{
-			$this->_entries[$node] = $object;
+			$this->_entries[$node]=$object;
 			return;
 		}
-		$this->_entries[$node] = $object;
+		$this->_entries[$node]=$object;
 		if(empty($parent))
 		{
 			if(isset($this->_entries[$node]))
 				return;
-			$this->_tree[$node] = array();
+			$this->_tree[$node]=array();
 		}
-		$found = $this->addNode($this->_tree, $parent, $node);
+		$found=$this->addNode($this->_tree, $parent, $node);
 		if(!$found && !empty($parent))
 		{
-			$this->_tree[$parent] = array();
-			if(!isset($this->_entries[$parent]) || $object !== '')
-				$this->_entries[$parent] = $object;
+			$this->_tree[$parent]=array();
+			if(!isset($this->_entries[$parent]) || $object!=='')
+				$this->_entries[$parent]=$object;
 			$this->addNode($this->_tree, $parent, $node);
 		}
 	}
@@ -88,20 +88,20 @@ 
 	 */
 	protected function addNode(&$childs, $parent, $node)
 	{
-		$found = false;
+		$found=false;
 		reset($childs);
-		for($i = 0, $k = count($childs); $i < $k; $i++)
+		for($i=0, $k=count($childs); $i < $k; $i++)
 		{
-			$key = key($childs);
+			$key=key($childs);
 			next($childs);
-			if($key == $parent)
+			if($key==$parent)
 			{
-				$found = true;
-				$childs[$key][$node] = array();
+				$found=true;
+				$childs[$key][$node]=array();
 			}
 			else
 			{
-				$found = $found || $this->addNode($childs[$key], $parent, $node);
+				$found=$found || $this->addNode($childs[$key], $parent, $node);
 			}
 		}
 		return $found;
@@ -123,9 +123,9 @@ 
 	 */
 	protected function hasChildren(&$nodes)
 	{
-		$hasChildren = false;
+		$hasChildren=false;
 		foreach($nodes as $node)
-			if(count($node) != 0)
+			if(count($node)!=0)
 				return true;
 		return $hasChildren;
 	}
@@ -137,14 +137,14 @@ 
 	 */
 	protected function collectChildren($parent, &$nodes)
 	{
-		$noChildren = !$this->hasChildren($nodes);
-		$childs = array();
-		for(reset($nodes); $key = key($nodes);)
+		$noChildren=!$this->hasChildren($nodes);
+		$childs=array();
+		for(reset($nodes); $key=key($nodes);)
 		{
 			next($nodes);
 			if($noChildren)
 			{
-				$childs[] = $key;
+				$childs[]=$key;
 				unset($nodes[$key]);
 			}
 			else
@@ -164,17 +164,17 @@ 
 		if(empty($parent) || empty($this->_entries[$parent]))
 			return;
 
-		$parentObject = $this->_entries[$parent]['object'];
-		$property = $this->_entries[$nodes[0]]['property'];
+		$parentObject=$this->_entries[$parent]['object'];
+		$property=$this->_entries[$nodes[0]]['property'];
 
-		$list = TPropertyAccess::get($parentObject, $property);
+		$list=TPropertyAccess::get($parentObject, $property);
 
 		foreach($nodes as $node)
 		{
 			if($list instanceof TList)
-				$parentObject->{$property}[] = $this->_entries[$node]['object'];
+				$parentObject->{$property}[]=$this->_entries[$node]['object'];
 			else if(is_array($list))
-				$list[] = $this->_entries[$node]['object'];
+				$list[]=$this->_entries[$node]['object'];
 			else
 				throw new TSqlMapExecutionException(
 					'sqlmap_property_must_be_list');
@@ -183,8 +183,8 @@ 
 		if(is_array($list))
 			TPropertyAccess::set($parentObject, $property, $list);
 
-		if($this->_entries[$parent]['property'] === null)
-			$this->_list[] = $parentObject;
+		if($this->_entries[$parent]['property']===null)
+			$this->_list[]=$parentObject;
 	}
 
 	/**
@@ -197,10 +197,10 @@ 
 
 	public function __sleep()
 	{
-		$exprops = array(); $cn = __CLASS__;
-		if (!count($this->_tree)) $exprops[] = "\0$cn\0_tree";
-		if (!count($this->_entries)) $exprops[] = "\0$cn\0_entries";
-		if (!count($this->_list)) $exprops[] = "\0$cn\0_list";
-		return array_diff(parent::__sleep(),$exprops);
+		$exprops=array(); $cn=__CLASS__;
+		if(!count($this->_tree)) $exprops[]="\0$cn\0_tree";
+		if(!count($this->_entries)) $exprops[]="\0$cn\0_entries";
+		if(!count($this->_list)) $exprops[]="\0$cn\0_list";
+		return array_diff(parent::__sleep(), $exprops);
 	}
 }
\ No newline at end of file

Braces +5 added lines, -6 removed lines

@@ -98,8 +98,7 @@ 
 			{
 				$found = true;
 				$childs[$key][$node] = array();
-			}
-			else
+			} else
 			{
 				$found = $found || $this->addNode($childs[$key], $parent, $node);
 			}
@@ -112,8 +111,9 @@ 
 	 */
 	public function collect()
 	{
-		while(count($this->_tree) > 0)
-			$this->collectChildren(null, $this->_tree);
+		while(count($this->_tree) > 0) {
+					$this->collectChildren(null, $this->_tree);
+		}
 		return $this->getCollection();
 	}
 
@@ -146,8 +146,7 @@ 
 			{
 				$childs[] = $key;
 				unset($nodes[$key]);
-			}
-			else
+			} else
 				$this->collectChildren($key, $nodes[$key]);
 		}
 		if(count($childs) > 0)

framework/Exceptions/THttpException.php

Doc Comments +2 added lines

@@ -393,6 +393,8 @@
 	 * in the message file. If so, the message in the preferred language
 	 * will be used as the error message. Any rest parameters will be used
 	 * to replace placeholders ({0}, {1}, {2}, etc.) in the message.
+	 * @param integer $statusCode
+	 * @param string $errorMessage
 	 */
 	public function __construct($statusCode,$errorMessage)
 	{

Spacing +3 added lines, -3 removed lines

@@ -35,7 +35,7 @@ 
 	 * will be used as the error message. Any rest parameters will be used
 	 * to replace placeholders ({0}, {1}, {2}, etc.) in the message.
 	 */
-	public function __construct($statusCode,$errorMessage)
+	public function __construct($statusCode, $errorMessage)
 	{
 		$this->_statusCode=$statusCode;
 		$this->setErrorCode($errorMessage);
@@ -45,9 +45,9 @@ 
 		array_shift($args);
 		$n=count($args);
 		$tokens=array();
-		for($i=0;$i<$n;++$i)
+		for($i=0; $i < $n; ++$i)
 			$tokens['{'.$i.'}']=TPropertyValue::ensureString($args[$i]);
-		parent::__construct(strtr($errorMessage,$tokens));
+		parent::__construct(strtr($errorMessage, $tokens));
 	}
 
 	/**

framework/I18N/core/Gettext/TGettext_MO.php

Doc Comments +3 added lines, -3 removed lines

@@ -86,7 +86,7 @@ 
      * _read
      *
      * @access  private
-     * @return  mixed
+     * @return  string|null
      * @param   int     $bytes
      */
     function _read($bytes = 1)
@@ -165,7 +165,7 @@ 
      * Load MO file
      *
      * @access   public
-     * @return   mixed   Returns true on success or PEAR_Error on failure.
+     * @return   boolean   Returns true on success or PEAR_Error on failure.
      * @param    string  $file
      */
     function load($file = null)
@@ -261,7 +261,7 @@ 
      * Save MO file
      *
      * @access  public
-     * @return  mixed   Returns true on success or PEAR_Error on failure.
+     * @return  boolean   Returns true on success or PEAR_Error on failure.
      * @param   string  $file
      */
     function save($file = null)

Indentation +295 added lines, -295 removed lines

@@ -52,304 +52,304 @@
  */
 class TGettext_MO extends TGettext
 {
-    /**
-     * file handle
-     *
-     * @access  private
-     * @var     resource
-     */
-    protected $_handle = null;
-
-    /**
-     * big endianess
-     *
-     * Whether to write with big endian byte order.
-     *
-     * @access  public
-     * @var     bool
-     */
-    protected $writeBigEndian = false;
-
-    /**
-     * Constructor
-     *
-     * @access  public
-     * @return  object      File_Gettext_MO
-     * @param   string      $file   path to GNU MO file
-     */
-    function __construct($file = '')
-    {
-        $this->file = $file;
-    }
-
-    /**
-     * _read
-     *
-     * @access  private
-     * @return  mixed
-     * @param   int     $bytes
-     */
-    function _read($bytes = 1)
-    {
-        if (0 < $bytes = abs($bytes)) {
-            return fread($this->_handle, $bytes);
-        }
-        return null;
-    }
-
-    /**
-     * _readInt
-     *
-     * @access  private
-     * @return  int
-     * @param   bool    $bigendian
-     */
-    function _readInt($bigendian = false)
-    {
+	/**
+	 * file handle
+	 *
+	 * @access  private
+	 * @var     resource
+	 */
+	protected $_handle = null;
+
+	/**
+	 * big endianess
+	 *
+	 * Whether to write with big endian byte order.
+	 *
+	 * @access  public
+	 * @var     bool
+	 */
+	protected $writeBigEndian = false;
+
+	/**
+	 * Constructor
+	 *
+	 * @access  public
+	 * @return  object      File_Gettext_MO
+	 * @param   string      $file   path to GNU MO file
+	 */
+	function __construct($file = '')
+	{
+		$this->file = $file;
+	}
+
+	/**
+	 * _read
+	 *
+	 * @access  private
+	 * @return  mixed
+	 * @param   int     $bytes
+	 */
+	function _read($bytes = 1)
+	{
+		if (0 < $bytes = abs($bytes)) {
+			return fread($this->_handle, $bytes);
+		}
+		return null;
+	}
+
+	/**
+	 * _readInt
+	 *
+	 * @access  private
+	 * @return  int
+	 * @param   bool    $bigendian
+	 */
+	function _readInt($bigendian = false)
+	{
 		//unpack returns a reference????
 		$unpacked = unpack($bigendian ? 'N' : 'V', $this->_read(4));
-        return array_shift($unpacked);
-    }
-
-    /**
-     * _writeInt
-     *
-     * @access  private
-     * @return  int
-     * @param   int     $int
-     */
-    function _writeInt($int)
-    {
-        return $this->_write(pack($this->writeBigEndian ? 'N' : 'V', (int) $int));
-    }
-
-    /**
-     * _write
-     *
-     * @access  private
-     * @return  int
-     * @param   string  $data
-     */
-    function _write($data)
-    {
-        return fwrite($this->_handle, $data);
-    }
-
-    /**
-     * _writeStr
-     *
-     * @access  private
-     * @return  int
-     * @param   string  $string
-     */
-    function _writeStr($string)
-    {
-        return $this->_write($string . "\0");
-    }
-
-    /**
-     * _readStr
-     *
-     * @access  private
-     * @return  string
-     * @param   array   $params     associative array with offset and length
-     *                              of the string
-     */
-    function _readStr($params)
-    {
-        fseek($this->_handle, $params['offset']);
-        return $this->_read($params['length']);
-    }
-
-    /**
-     * Load MO file
-     *
-     * @access   public
-     * @return   mixed   Returns true on success or PEAR_Error on failure.
-     * @param    string  $file
-     */
-    function load($file = null)
-    {
-        if (!isset($file)) {
-            $file = $this->file;
-        }
-
-        // open MO file
-        if (!is_resource($this->_handle = @fopen($file, 'rb'))) {
-            return false;
-        }
-        // lock MO file shared
-        if (!@flock($this->_handle, LOCK_SH)) {
-            @fclose($this->_handle);
-            return false;
-        }
-
-        // read (part of) magic number from MO file header and define endianess
+		return array_shift($unpacked);
+	}
+
+	/**
+	 * _writeInt
+	 *
+	 * @access  private
+	 * @return  int
+	 * @param   int     $int
+	 */
+	function _writeInt($int)
+	{
+		return $this->_write(pack($this->writeBigEndian ? 'N' : 'V', (int) $int));
+	}
+
+	/**
+	 * _write
+	 *
+	 * @access  private
+	 * @return  int
+	 * @param   string  $data
+	 */
+	function _write($data)
+	{
+		return fwrite($this->_handle, $data);
+	}
+
+	/**
+	 * _writeStr
+	 *
+	 * @access  private
+	 * @return  int
+	 * @param   string  $string
+	 */
+	function _writeStr($string)
+	{
+		return $this->_write($string . "\0");
+	}
+
+	/**
+	 * _readStr
+	 *
+	 * @access  private
+	 * @return  string
+	 * @param   array   $params     associative array with offset and length
+	 *                              of the string
+	 */
+	function _readStr($params)
+	{
+		fseek($this->_handle, $params['offset']);
+		return $this->_read($params['length']);
+	}
+
+	/**
+	 * Load MO file
+	 *
+	 * @access   public
+	 * @return   mixed   Returns true on success or PEAR_Error on failure.
+	 * @param    string  $file
+	 */
+	function load($file = null)
+	{
+		if (!isset($file)) {
+			$file = $this->file;
+		}
+
+		// open MO file
+		if (!is_resource($this->_handle = @fopen($file, 'rb'))) {
+			return false;
+		}
+		// lock MO file shared
+		if (!@flock($this->_handle, LOCK_SH)) {
+			@fclose($this->_handle);
+			return false;
+		}
+
+		// read (part of) magic number from MO file header and define endianess
 
 		//unpack returns a reference????
 		$unpacked = unpack('c', $this->_read(4));
-        switch ($magic = array_shift($unpacked))
-        {
-            case -34:
-                $be = false;
-            break;
-
-            case -107:
-                $be = true;
-            break;
-
-            default:
-                return false;
-        }
-
-        // check file format revision - we currently only support 0
-        if (0 !== ($_rev = $this->_readInt($be))) {
-            return false;
-        }
-
-        // count of strings in this file
-        $count = $this->_readInt($be);
-
-        // offset of hashing table of the msgids
-        $offset_original = $this->_readInt($be);
-        // offset of hashing table of the msgstrs
-        $offset_translat = $this->_readInt($be);
-
-        // move to msgid hash table
-        fseek($this->_handle, $offset_original);
-        // read lengths and offsets of msgids
-        $original = array();
-        for ($i = 0; $i < $count; $i++) {
-            $original[$i] = array(
-                'length' => $this->_readInt($be),
-                'offset' => $this->_readInt($be)
-            );
-        }
-
-        // move to msgstr hash table
-        fseek($this->_handle, $offset_translat);
-        // read lengths and offsets of msgstrs
-        $translat = array();
-        for ($i = 0; $i < $count; $i++) {
-            $translat[$i] = array(
-                'length' => $this->_readInt($be),
-                'offset' => $this->_readInt($be)
-            );
-        }
-
-        // read all
-        for ($i = 0; $i < $count; $i++) {
-            $this->strings[$this->_readStr($original[$i])] =
-                $this->_readStr($translat[$i]);
-        }
-
-        // done
-        @flock($this->_handle, LOCK_UN);
-        @fclose($this->_handle);
-        $this->_handle = null;
-
-        // check for meta info
-        if (isset($this->strings[''])) {
-            $this->meta = parent::meta2array($this->strings['']);
-            unset($this->strings['']);
-        }
-
-        return true;
-    }
-
-    /**
-     * Save MO file
-     *
-     * @access  public
-     * @return  mixed   Returns true on success or PEAR_Error on failure.
-     * @param   string  $file
-     */
-    function save($file = null)
-    {
-        if (!isset($file)) {
-            $file = $this->file;
-        }
-
-        // open MO file
-        if (!is_resource($this->_handle = @fopen($file, 'wb'))) {
-            return false;
-        }
-        // lock MO file exclusively
-        if (!@flock($this->_handle, LOCK_EX)) {
-            @fclose($this->_handle);
-            return false;
-        }
-
-        // write magic number
-        if ($this->writeBigEndian) {
-            $this->_write(pack('c*', 0x95, 0x04, 0x12, 0xde));
-        } else {
-            $this->_write(pack('c*', 0xde, 0x12, 0x04, 0x95));
-        }
-
-        // write file format revision
-        $this->_writeInt(0);
-
-        $count = count($this->strings) + ($meta = (count($this->meta) ? 1 : 0));
-        // write count of strings
-        $this->_writeInt($count);
-
-        $offset = 28;
-        // write offset of orig. strings hash table
-        $this->_writeInt($offset);
-
-        $offset += ($count * 8);
-        // write offset transl. strings hash table
-        $this->_writeInt($offset);
-
-        // write size of hash table (we currently ommit the hash table)
-        $this->_writeInt(0);
-
-        $offset += ($count * 8);
-        // write offset of hash table
-        $this->_writeInt($offset);
-
-        // unshift meta info
-        if ($this->meta) {
-            $meta = '';
-            foreach ($this->meta as $key => $val) {
-                $meta .= $key . ': ' . $val . "\n";
-            }
-            $strings = array('' => $meta) + $this->strings;
-        } else {
-            $strings = $this->strings;
-        }
-
-        // write offsets for original strings
-        foreach (array_keys($strings) as $o) {
-            $len = strlen($o);
-            $this->_writeInt($len);
-            $this->_writeInt($offset);
-            $offset += $len + 1;
-        }
-
-        // write offsets for translated strings
-        foreach ($strings as $t) {
-            $len = strlen($t);
-            $this->_writeInt($len);
-            $this->_writeInt($offset);
-            $offset += $len + 1;
-        }
-
-        // write original strings
-        foreach (array_keys($strings) as $o) {
-            $this->_writeStr($o);
-        }
-
-        // write translated strings
-        foreach ($strings as $t) {
-            $this->_writeStr($t);
-        }
-
-        // done
-        @flock($this->_handle, LOCK_UN);
-        @fclose($this->_handle);
-        chmod($file,PRADO_CHMOD);
-        return true;
-    }
+		switch ($magic = array_shift($unpacked))
+		{
+			case -34:
+				$be = false;
+			break;
+
+			case -107:
+				$be = true;
+			break;
+
+			default:
+				return false;
+		}
+
+		// check file format revision - we currently only support 0
+		if (0 !== ($_rev = $this->_readInt($be))) {
+			return false;
+		}
+
+		// count of strings in this file
+		$count = $this->_readInt($be);
+
+		// offset of hashing table of the msgids
+		$offset_original = $this->_readInt($be);
+		// offset of hashing table of the msgstrs
+		$offset_translat = $this->_readInt($be);
+
+		// move to msgid hash table
+		fseek($this->_handle, $offset_original);
+		// read lengths and offsets of msgids
+		$original = array();
+		for ($i = 0; $i < $count; $i++) {
+			$original[$i] = array(
+				'length' => $this->_readInt($be),
+				'offset' => $this->_readInt($be)
+			);
+		}
+
+		// move to msgstr hash table
+		fseek($this->_handle, $offset_translat);
+		// read lengths and offsets of msgstrs
+		$translat = array();
+		for ($i = 0; $i < $count; $i++) {
+			$translat[$i] = array(
+				'length' => $this->_readInt($be),
+				'offset' => $this->_readInt($be)
+			);
+		}
+
+		// read all
+		for ($i = 0; $i < $count; $i++) {
+			$this->strings[$this->_readStr($original[$i])] =
+				$this->_readStr($translat[$i]);
+		}
+
+		// done
+		@flock($this->_handle, LOCK_UN);
+		@fclose($this->_handle);
+		$this->_handle = null;
+
+		// check for meta info
+		if (isset($this->strings[''])) {
+			$this->meta = parent::meta2array($this->strings['']);
+			unset($this->strings['']);
+		}
+
+		return true;
+	}
+
+	/**
+	 * Save MO file
+	 *
+	 * @access  public
+	 * @return  mixed   Returns true on success or PEAR_Error on failure.
+	 * @param   string  $file
+	 */
+	function save($file = null)
+	{
+		if (!isset($file)) {
+			$file = $this->file;
+		}
+
+		// open MO file
+		if (!is_resource($this->_handle = @fopen($file, 'wb'))) {
+			return false;
+		}
+		// lock MO file exclusively
+		if (!@flock($this->_handle, LOCK_EX)) {
+			@fclose($this->_handle);
+			return false;
+		}
+
+		// write magic number
+		if ($this->writeBigEndian) {
+			$this->_write(pack('c*', 0x95, 0x04, 0x12, 0xde));
+		} else {
+			$this->_write(pack('c*', 0xde, 0x12, 0x04, 0x95));
+		}
+
+		// write file format revision
+		$this->_writeInt(0);
+
+		$count = count($this->strings) + ($meta = (count($this->meta) ? 1 : 0));
+		// write count of strings
+		$this->_writeInt($count);
+
+		$offset = 28;
+		// write offset of orig. strings hash table
+		$this->_writeInt($offset);
+
+		$offset += ($count * 8);
+		// write offset transl. strings hash table
+		$this->_writeInt($offset);
+
+		// write size of hash table (we currently ommit the hash table)
+		$this->_writeInt(0);
+
+		$offset += ($count * 8);
+		// write offset of hash table
+		$this->_writeInt($offset);
+
+		// unshift meta info
+		if ($this->meta) {
+			$meta = '';
+			foreach ($this->meta as $key => $val) {
+				$meta .= $key . ': ' . $val . "\n";
+			}
+			$strings = array('' => $meta) + $this->strings;
+		} else {
+			$strings = $this->strings;
+		}
+
+		// write offsets for original strings
+		foreach (array_keys($strings) as $o) {
+			$len = strlen($o);
+			$this->_writeInt($len);
+			$this->_writeInt($offset);
+			$offset += $len + 1;
+		}
+
+		// write offsets for translated strings
+		foreach ($strings as $t) {
+			$len = strlen($t);
+			$this->_writeInt($len);
+			$this->_writeInt($offset);
+			$offset += $len + 1;
+		}
+
+		// write original strings
+		foreach (array_keys($strings) as $o) {
+			$this->_writeStr($o);
+		}
+
+		// write translated strings
+		foreach ($strings as $t) {
+			$this->_writeStr($t);
+		}
+
+		// done
+		@flock($this->_handle, LOCK_UN);
+		@fclose($this->_handle);
+		chmod($file,PRADO_CHMOD);
+		return true;
+	}
 }

Spacing +58 added lines, -58 removed lines

@@ -58,7 +58,7 @@ 
      * @access  private
      * @var     resource
      */
-    protected $_handle = null;
+    protected $_handle=null;
 
     /**
      * big endianess
@@ -68,7 +68,7 @@ 
      * @access  public
      * @var     bool
      */
-    protected $writeBigEndian = false;
+    protected $writeBigEndian=false;
 
     /**
      * Constructor
@@ -77,9 +77,9 @@ 
      * @return  object      File_Gettext_MO
      * @param   string      $file   path to GNU MO file
      */
-    function __construct($file = '')
+    function __construct($file='')
     {
-        $this->file = $file;
+        $this->file=$file;
     }
 
     /**
@@ -89,9 +89,9 @@ 
      * @return  mixed
      * @param   int     $bytes
      */
-    function _read($bytes = 1)
+    function _read($bytes=1)
     {
-        if (0 < $bytes = abs($bytes)) {
+        if(0 < $bytes=abs($bytes)) {
             return fread($this->_handle, $bytes);
         }
         return null;
@@ -104,10 +104,10 @@ 
      * @return  int
      * @param   bool    $bigendian
      */
-    function _readInt($bigendian = false)
+    function _readInt($bigendian=false)
     {
 		//unpack returns a reference????
-		$unpacked = unpack($bigendian ? 'N' : 'V', $this->_read(4));
+		$unpacked=unpack($bigendian ? 'N' : 'V', $this->_read(4));
         return array_shift($unpacked);
     }
 
@@ -144,7 +144,7 @@ 
      */
     function _writeStr($string)
     {
-        return $this->_write($string . "\0");
+        return $this->_write($string."\0");
     }
 
     /**
@@ -168,18 +168,18 @@ 
      * @return   mixed   Returns true on success or PEAR_Error on failure.
      * @param    string  $file
      */
-    function load($file = null)
+    function load($file=null)
     {
-        if (!isset($file)) {
-            $file = $this->file;
+        if(!isset($file)) {
+            $file=$this->file;
         }
 
         // open MO file
-        if (!is_resource($this->_handle = @fopen($file, 'rb'))) {
+        if(!is_resource($this->_handle=@fopen($file, 'rb'))) {
             return false;
         }
         // lock MO file shared
-        if (!@flock($this->_handle, LOCK_SH)) {
+        if(!@flock($this->_handle, LOCK_SH)) {
             @fclose($this->_handle);
             return false;
         }
@@ -187,15 +187,15 @@ 
         // read (part of) magic number from MO file header and define endianess
 
 		//unpack returns a reference????
-		$unpacked = unpack('c', $this->_read(4));
-        switch ($magic = array_shift($unpacked))
+		$unpacked=unpack('c', $this->_read(4));
+        switch($magic=array_shift($unpacked))
         {
             case -34:
-                $be = false;
+                $be=false;
             break;
 
             case -107:
-                $be = true;
+                $be=true;
             break;
 
             default:
@@ -203,24 +203,24 @@ 
         }
 
         // check file format revision - we currently only support 0
-        if (0 !== ($_rev = $this->_readInt($be))) {
+        if(0!==($_rev=$this->_readInt($be))) {
             return false;
         }
 
         // count of strings in this file
-        $count = $this->_readInt($be);
+        $count=$this->_readInt($be);
 
         // offset of hashing table of the msgids
-        $offset_original = $this->_readInt($be);
+        $offset_original=$this->_readInt($be);
         // offset of hashing table of the msgstrs
-        $offset_translat = $this->_readInt($be);
+        $offset_translat=$this->_readInt($be);
 
         // move to msgid hash table
         fseek($this->_handle, $offset_original);
         // read lengths and offsets of msgids
-        $original = array();
-        for ($i = 0; $i < $count; $i++) {
-            $original[$i] = array(
+        $original=array();
+        for($i=0; $i < $count; $i++) {
+            $original[$i]=array(
                 'length' => $this->_readInt($be),
                 'offset' => $this->_readInt($be)
             );
@@ -229,28 +229,28 @@ 
         // move to msgstr hash table
         fseek($this->_handle, $offset_translat);
         // read lengths and offsets of msgstrs
-        $translat = array();
-        for ($i = 0; $i < $count; $i++) {
-            $translat[$i] = array(
+        $translat=array();
+        for($i=0; $i < $count; $i++) {
+            $translat[$i]=array(
                 'length' => $this->_readInt($be),
                 'offset' => $this->_readInt($be)
             );
         }
 
         // read all
-        for ($i = 0; $i < $count; $i++) {
-            $this->strings[$this->_readStr($original[$i])] =
+        for($i=0; $i < $count; $i++) {
+            $this->strings[$this->_readStr($original[$i])]=
                 $this->_readStr($translat[$i]);
         }
 
         // done
         @flock($this->_handle, LOCK_UN);
         @fclose($this->_handle);
-        $this->_handle = null;
+        $this->_handle=null;
 
         // check for meta info
-        if (isset($this->strings[''])) {
-            $this->meta = parent::meta2array($this->strings['']);
+        if(isset($this->strings[''])) {
+            $this->meta=parent::meta2array($this->strings['']);
             unset($this->strings['']);
         }
 
@@ -264,24 +264,24 @@ 
      * @return  mixed   Returns true on success or PEAR_Error on failure.
      * @param   string  $file
      */
-    function save($file = null)
+    function save($file=null)
     {
-        if (!isset($file)) {
-            $file = $this->file;
+        if(!isset($file)) {
+            $file=$this->file;
         }
 
         // open MO file
-        if (!is_resource($this->_handle = @fopen($file, 'wb'))) {
+        if(!is_resource($this->_handle=@fopen($file, 'wb'))) {
             return false;
         }
         // lock MO file exclusively
-        if (!@flock($this->_handle, LOCK_EX)) {
+        if(!@flock($this->_handle, LOCK_EX)) {
             @fclose($this->_handle);
             return false;
         }
 
         // write magic number
-        if ($this->writeBigEndian) {
+        if($this->writeBigEndian) {
             $this->_write(pack('c*', 0x95, 0x04, 0x12, 0xde));
         } else {
             $this->_write(pack('c*', 0xde, 0x12, 0x04, 0x95));
@@ -290,66 +290,66 @@ 
         // write file format revision
         $this->_writeInt(0);
 
-        $count = count($this->strings) + ($meta = (count($this->meta) ? 1 : 0));
+        $count=count($this->strings) + ($meta=(count($this->meta) ? 1 : 0));
         // write count of strings
         $this->_writeInt($count);
 
-        $offset = 28;
+        $offset=28;
         // write offset of orig. strings hash table
         $this->_writeInt($offset);
 
-        $offset += ($count * 8);
+        $offset+=($count * 8);
         // write offset transl. strings hash table
         $this->_writeInt($offset);
 
         // write size of hash table (we currently ommit the hash table)
         $this->_writeInt(0);
 
-        $offset += ($count * 8);
+        $offset+=($count * 8);
         // write offset of hash table
         $this->_writeInt($offset);
 
         // unshift meta info
-        if ($this->meta) {
-            $meta = '';
-            foreach ($this->meta as $key => $val) {
-                $meta .= $key . ': ' . $val . "\n";
+        if($this->meta) {
+            $meta='';
+            foreach($this->meta as $key => $val) {
+                $meta.=$key.': '.$val."\n";
             }
-            $strings = array('' => $meta) + $this->strings;
+            $strings=array('' => $meta) + $this->strings;
         } else {
-            $strings = $this->strings;
+            $strings=$this->strings;
         }
 
         // write offsets for original strings
-        foreach (array_keys($strings) as $o) {
-            $len = strlen($o);
+        foreach(array_keys($strings) as $o) {
+            $len=strlen($o);
             $this->_writeInt($len);
             $this->_writeInt($offset);
-            $offset += $len + 1;
+            $offset+=$len + 1;
         }
 
         // write offsets for translated strings
-        foreach ($strings as $t) {
-            $len = strlen($t);
+        foreach($strings as $t) {
+            $len=strlen($t);
             $this->_writeInt($len);
             $this->_writeInt($offset);
-            $offset += $len + 1;
+            $offset+=$len + 1;
         }
 
         // write original strings
-        foreach (array_keys($strings) as $o) {
+        foreach(array_keys($strings) as $o) {
             $this->_writeStr($o);
         }
 
         // write translated strings
-        foreach ($strings as $t) {
+        foreach($strings as $t) {
             $this->_writeStr($t);
         }
 
         // done
         @flock($this->_handle, LOCK_UN);
         @fclose($this->_handle);
-        chmod($file,PRADO_CHMOD);
+        chmod($file, PRADO_CHMOD);
         return true;
     }
 }

framework/I18N/core/Gettext/TGettext_PO.php

Doc Comments +2 added lines, -2 removed lines

@@ -67,7 +67,7 @@ 
      * Load PO file
      *
      * @access  public
-     * @return  mixed   Returns true on success or PEAR_Error on failure.
+     * @return  boolean   Returns true on success or PEAR_Error on failure.
      * @param   string  $file
      */
     function load($file = null)
@@ -116,7 +116,7 @@ 
      * Save PO file
      *
      * @access  public
-     * @return  mixed   Returns true on success or PEAR_Error on failure.
+     * @return  boolean   Returns true on success or PEAR_Error on failure.
      * @param   string  $file
      */
     function save($file = null)

Indentation +106 added lines, -106 removed lines

@@ -51,110 +51,110 @@
  */
 class TGettext_PO extends TGettext
 {
-    /**
-     * Constructor
-     *
-     * @access  public
-     * @return  object      File_Gettext_PO
-     * @param   string      path to GNU PO file
-     */
-    function __construct($file = '')
-    {
-        $this->file = $file;
-    }
-
-    /**
-     * Load PO file
-     *
-     * @access  public
-     * @return  mixed   Returns true on success or PEAR_Error on failure.
-     * @param   string  $file
-     */
-    function load($file = null)
-    {
-        if (!isset($file)) {
-            $file = $this->file;
-        }
-
-        // load file
-        if (!$contents = @file($file)) {
-            return false;
-        }
-        $contents = implode('', $contents);
-
-        // match all msgid/msgstr entries
-        $matched = preg_match_all(
-            '/(msgid\s+("([^"]|\\\\")*?"\s*)+)\s+' .
-            '(msgstr\s+("([^"]|\\\\")*?"\s*)+)/',
-            $contents, $matches
-        );
-        unset($contents);
-
-        if (!$matched) {
-            return false;
-        }
-
-        // get all msgids and msgtrs
-        for ($i = 0; $i < $matched; $i++) {
-            $msgid = preg_replace(
-                '/\s*msgid\s*"(.*)"\s*/s', '\\1', $matches[1][$i]);
-            $msgstr= preg_replace(
-                '/\s*msgstr\s*"(.*)"\s*/s', '\\1', $matches[4][$i]);
-            $this->strings[parent::prepare($msgid)] = parent::prepare($msgstr);
-        }
-
-        // check for meta info
-        if (isset($this->strings[''])) {
-            $this->meta = parent::meta2array($this->strings['']);
-            unset($this->strings['']);
-        }
-
-        return true;
-    }
-
-    /**
-     * Save PO file
-     *
-     * @access  public
-     * @return  mixed   Returns true on success or PEAR_Error on failure.
-     * @param   string  $file
-     */
-    function save($file = null)
-    {
-        if (!isset($file)) {
-            $file = $this->file;
-        }
-
-        // open PO file
-        if (!is_resource($fh = @fopen($file, 'w'))) {
-            return false;
-        }
-
-        // lock PO file exclusively
-        if (!flock($fh, LOCK_EX)) {
-            fclose($fh);
-            return false;
-        }
-        // write meta info
-        if (count($this->meta)) {
-            $meta = 'msgid ""' . "\nmsgstr " . '""' . "\n";
-            foreach ($this->meta as $k => $v) {
-                $meta .= '"' . $k . ': ' . $v . '\n"' . "\n";
-            }
-            fwrite($fh, $meta . "\n");
-        }
-        // write strings
-        foreach ($this->strings as $o => $t) {
-            fwrite($fh,
-                'msgid "'  . parent::prepare($o, true) . '"' . "\n" .
-                'msgstr "' . parent::prepare($t, true) . '"' . "\n\n"
-            );
-        }
-
-        //done
-        @flock($fh, LOCK_UN);
-        @fclose($fh);
-        chmod($file,PRADO_CHMOD);
-        return true;
-    }
+	/**
+	 * Constructor
+	 *
+	 * @access  public
+	 * @return  object      File_Gettext_PO
+	 * @param   string      path to GNU PO file
+	 */
+	function __construct($file = '')
+	{
+		$this->file = $file;
+	}
+
+	/**
+	 * Load PO file
+	 *
+	 * @access  public
+	 * @return  mixed   Returns true on success or PEAR_Error on failure.
+	 * @param   string  $file
+	 */
+	function load($file = null)
+	{
+		if (!isset($file)) {
+			$file = $this->file;
+		}
+
+		// load file
+		if (!$contents = @file($file)) {
+			return false;
+		}
+		$contents = implode('', $contents);
+
+		// match all msgid/msgstr entries
+		$matched = preg_match_all(
+			'/(msgid\s+("([^"]|\\\\")*?"\s*)+)\s+' .
+			'(msgstr\s+("([^"]|\\\\")*?"\s*)+)/',
+			$contents, $matches
+		);
+		unset($contents);
+
+		if (!$matched) {
+			return false;
+		}
+
+		// get all msgids and msgtrs
+		for ($i = 0; $i < $matched; $i++) {
+			$msgid = preg_replace(
+				'/\s*msgid\s*"(.*)"\s*/s', '\\1', $matches[1][$i]);
+			$msgstr= preg_replace(
+				'/\s*msgstr\s*"(.*)"\s*/s', '\\1', $matches[4][$i]);
+			$this->strings[parent::prepare($msgid)] = parent::prepare($msgstr);
+		}
+
+		// check for meta info
+		if (isset($this->strings[''])) {
+			$this->meta = parent::meta2array($this->strings['']);
+			unset($this->strings['']);
+		}
+
+		return true;
+	}
+
+	/**
+	 * Save PO file
+	 *
+	 * @access  public
+	 * @return  mixed   Returns true on success or PEAR_Error on failure.
+	 * @param   string  $file
+	 */
+	function save($file = null)
+	{
+		if (!isset($file)) {
+			$file = $this->file;
+		}
+
+		// open PO file
+		if (!is_resource($fh = @fopen($file, 'w'))) {
+			return false;
+		}
+
+		// lock PO file exclusively
+		if (!flock($fh, LOCK_EX)) {
+			fclose($fh);
+			return false;
+		}
+		// write meta info
+		if (count($this->meta)) {
+			$meta = 'msgid ""' . "\nmsgstr " . '""' . "\n";
+			foreach ($this->meta as $k => $v) {
+				$meta .= '"' . $k . ': ' . $v . '\n"' . "\n";
+			}
+			fwrite($fh, $meta . "\n");
+		}
+		// write strings
+		foreach ($this->strings as $o => $t) {
+			fwrite($fh,
+				'msgid "'  . parent::prepare($o, true) . '"' . "\n" .
+				'msgstr "' . parent::prepare($t, true) . '"' . "\n\n"
+			);
+		}
+
+		//done
+		@flock($fh, LOCK_UN);
+		@fclose($fh);
+		chmod($file,PRADO_CHMOD);
+		return true;
+	}
 }

Spacing +30 added lines, -30 removed lines

@@ -58,9 +58,9 @@ 
      * @return  object      File_Gettext_PO
      * @param   string      path to GNU PO file
      */
-    function __construct($file = '')
+    function __construct($file='')
     {
-        $this->file = $file;
+        $this->file=$file;
     }
 
     /**
@@ -70,42 +70,42 @@ 
      * @return  mixed   Returns true on success or PEAR_Error on failure.
      * @param   string  $file
      */
-    function load($file = null)
+    function load($file=null)
     {
-        if (!isset($file)) {
-            $file = $this->file;
+        if(!isset($file)) {
+            $file=$this->file;
         }
 
         // load file
-        if (!$contents = @file($file)) {
+        if(!$contents=@file($file)) {
             return false;
         }
-        $contents = implode('', $contents);
+        $contents=implode('', $contents);
 
         // match all msgid/msgstr entries
-        $matched = preg_match_all(
-            '/(msgid\s+("([^"]|\\\\")*?"\s*)+)\s+' .
+        $matched=preg_match_all(
+            '/(msgid\s+("([^"]|\\\\")*?"\s*)+)\s+'.
             '(msgstr\s+("([^"]|\\\\")*?"\s*)+)/',
             $contents, $matches
         );
         unset($contents);
 
-        if (!$matched) {
+        if(!$matched) {
             return false;
         }
 
         // get all msgids and msgtrs
-        for ($i = 0; $i < $matched; $i++) {
-            $msgid = preg_replace(
+        for($i=0; $i < $matched; $i++) {
+            $msgid=preg_replace(
                 '/\s*msgid\s*"(.*)"\s*/s', '\\1', $matches[1][$i]);
-            $msgstr= preg_replace(
+            $msgstr=preg_replace(
                 '/\s*msgstr\s*"(.*)"\s*/s', '\\1', $matches[4][$i]);
-            $this->strings[parent::prepare($msgid)] = parent::prepare($msgstr);
+            $this->strings[parent::prepare($msgid)]=parent::prepare($msgstr);
         }
 
         // check for meta info
-        if (isset($this->strings[''])) {
-            $this->meta = parent::meta2array($this->strings['']);
+        if(isset($this->strings[''])) {
+            $this->meta=parent::meta2array($this->strings['']);
             unset($this->strings['']);
         }
 
@@ -119,42 +119,42 @@ 
      * @return  mixed   Returns true on success or PEAR_Error on failure.
      * @param   string  $file
      */
-    function save($file = null)
+    function save($file=null)
     {
-        if (!isset($file)) {
-            $file = $this->file;
+        if(!isset($file)) {
+            $file=$this->file;
         }
 
         // open PO file
-        if (!is_resource($fh = @fopen($file, 'w'))) {
+        if(!is_resource($fh=@fopen($file, 'w'))) {
             return false;
         }
 
         // lock PO file exclusively
-        if (!flock($fh, LOCK_EX)) {
+        if(!flock($fh, LOCK_EX)) {
             fclose($fh);
             return false;
         }
         // write meta info
-        if (count($this->meta)) {
-            $meta = 'msgid ""' . "\nmsgstr " . '""' . "\n";
-            foreach ($this->meta as $k => $v) {
-                $meta .= '"' . $k . ': ' . $v . '\n"' . "\n";
+        if(count($this->meta)) {
+            $meta='msgid ""'."\nmsgstr ".'""'."\n";
+            foreach($this->meta as $k => $v) {
+                $meta.='"'.$k.': '.$v.'\n"'."\n";
             }
-            fwrite($fh, $meta . "\n");
+            fwrite($fh, $meta."\n");
         }
         // write strings
-        foreach ($this->strings as $o => $t) {
+        foreach($this->strings as $o => $t) {
             fwrite($fh,
-                'msgid "'  . parent::prepare($o, true) . '"' . "\n" .
-                'msgstr "' . parent::prepare($t, true) . '"' . "\n\n"
+                'msgid "'.parent::prepare($o, true).'"'."\n".
+                'msgstr "'.parent::prepare($t, true).'"'."\n\n"
             );
         }
 
         //done
         @flock($fh, LOCK_UN);
         @fclose($fh);
-        chmod($file,PRADO_CHMOD);
+        chmod($file, PRADO_CHMOD);
         return true;
     }
 }