pradosoft / prado

framework/Collections/TPriorityMap.php

Doc Comments +4 added lines, -4 removed lines

@@ -96,7 +96,7 @@ 
 	 * Initializes the array with an array or an iterable object.
 	 * @param null|array|Iterator|map|TPriorityMap $data the intial data. Default is null, meaning no initialization.
 	 * @param bool $readOnly whether the list is read-only
-	 * @param numeric $defaultPriority the default priority of items without specified priorities.
+	 * @param integer $defaultPriority the default priority of items without specified priorities.
 	 * @param int $precision the precision of the numeric priorities
 	 * @throws TInvalidDataTypeException If data is not null and neither an array nor an iterator.
 	 */
@@ -136,7 +136,7 @@ 
 
 	/**
 	 * This must be called internally or when instantiated.
-	 * @param numeric $value sets the default priority of inserted items without a specified priority
+	 * @param integer $value sets the default priority of inserted items without a specified priority
 	 */
 	protected function setDefaultPriority($value)
 	{
@@ -163,7 +163,7 @@ 
 	/**
 	 * Returns an iterator for traversing the items in the map.
 	 * This method is required by the interface \IteratorAggregate.
-	 * @return Iterator an iterator for traversing the items in the map.
+	 * @return \ArrayIterator an iterator for traversing the items in the map.
 	 */
 	public function getIterator()
 	{
@@ -358,7 +358,7 @@ 
 	 * @param mixed $value
 	 * @param null|numeric $priority default: null, filled in with default priority
 	 * @throws TInvalidOperationException if the map is read-only
-	 * @return numeric priority at which the pair was added
+	 * @return string priority at which the pair was added
 	 */
 	public function add($key, $value, $priority = null)
 	{

framework/Data/ActiveRecord/Relations/TActiveRecordRelation.php

Doc Comments +5 added lines, -1 removed lines

@@ -30,6 +30,9 @@ 
 	private $_context;
 	private $_criteria;
 
+	/**
+	 * @param \Prado\Data\ActiveRecord\TActiveRecordCriteria $criteria
+	 */
 	public function __construct(TActiveRecordRelationContext $context, $criteria)
 	{
 		$this->_context = $context;
@@ -95,7 +98,7 @@ 
 
 	/**
 	 * Fetch results for current relationship.
-	 * @param mixed $obj
+	 * @param TActiveRecord $obj
 	 * @return bool always true.
 	 */
 	public function fetchResultsInto($obj)
@@ -223,6 +226,7 @@ 
 	 * @param array $properties source property names
 	 * @param array &$fkObjects foreign objects
 	 * @param array $fields foreign object field names.
+	 * @param TActiveRecord[] $fkObjects
 	 */
 	protected function populateResult(&$results, $properties, &$fkObjects, $fields)
 	{

framework/Data/Common/TDbCommandBuilder.php

Doc Comments +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
 	}
 
 	/**
-	 * @param TDbConnection $value database connection.
+	 * @param null|TDbConnection $value database connection.
 	 */
 	public function setDbConnection($value)
 	{
@@ -194,7 +194,7 @@ 
 	 * array('col1' => 'NULL', '*')
 	 * // SELECT `col1`, `col2`, `col3`, NULL AS `col1` FROM...
 	 * </code>
-	 * @param mixed $data
+	 * @param string $data
 	 * @return array of generated fields - use implode(', ', $selectfieldlist) to collapse field list for usage
 	 * @since 3.1.7
 	 * @todo add support for table aliasing
@@ -420,7 +420,7 @@ 
 	/**
 	 * Returns a list of insert field name and a list of binding names.
 	 * @param object $values array or object to be inserted.
-	 * @return array tuple ($fields, $bindings)
+	 * @return string[] tuple ($fields, $bindings)
 	 */
 	protected function getInsertFieldBindings($values)
 	{

framework/I18N/core/MessageCache.php

Doc Comments +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
 	 * @param string $culture The translation locale, e.g. "en_AU".
 	 * @param int $lastmodified If the source is a file, this file's modified
 	 * time is newer than the cache's modified time, no cache hit.
-	 * @return mixed bool FALSE if no cache hit. Otherwise, translation
+	 * @return false|string bool FALSE if no cache hit. Otherwise, translation
 	 * table data for the specified section and locale.
 	 */
 	public function get($catalogue, $culture, $lastmodified = 0)

framework/Web/Services/TPageService.php

Doc Comments +2 added lines, -2 removed lines

@@ -470,7 +470,7 @@ 
 	 * @param string $pagePath requested page path
 	 * @throws THttpException if requested page path is invalid
 	 * @throws TConfigurationException if the page class cannot be found
-	 * @return TPage the requested page instance
+	 * @return \Prado\TComponent the requested page instance
 	 */
 	protected function createPage($pagePath)
 	{
@@ -517,7 +517,7 @@ 
 
 	/**
 	 * Executes a page.
-	 * @param TPage $page the page instance to be run
+	 * @param \Prado\TComponent $page the page instance to be run
 	 * @param array $properties list of initial page properties
 	 */
 	protected function runPage($page, $properties)

bin/prado-cli.php

Doc Comments +3 added lines, -3 removed lines

@@ -333,7 +333,7 @@ 
 	}
 
 	/**
-	 * @param array $appName application id
+	 * @param string $appName application id
 	 * @return string
 	 */
 	protected function renderConfigFile($appName)
@@ -590,7 +590,7 @@ 
 
 	/**
 	 * @param string $app_dir application directory
-	 * @return string|false
+	 * @return string
 	 */
 	protected function getActiveRecordConfig($app_dir)
 	{
@@ -633,7 +633,7 @@ 
 	 * @param string $config database configuration
 	 * @param string $tablename table name
 	 * @param string $output output file name
-	 * @return bool
+	 * @return false|null
 	 */
 	protected function generateActiveRecord($config, $tablename, $output)
 	{

framework/Caching/TCache.php

Doc Comments +2 added lines, -2 removed lines

@@ -146,7 +146,7 @@ 
 	 * @param mixed $value the value to be cached
 	 * @param int $expire the number of seconds in which the cached value will expire. 0 means never expire.
 	 * @param ICacheDependency $dependency dependency of the cached item. If the dependency changes, the item is labeled invalid.
-	 * @return bool true if the value is successfully stored into cache, false otherwise
+	 * @return null|boolean true if the value is successfully stored into cache, false otherwise
 	 */
 	public function set($id, $value, $expire = 0, $dependency = null)
 	{
@@ -283,7 +283,7 @@ 
 	 * Deletes the value with the specified key from cache
 	 * This method is required by the interface \ArrayAccess.
 	 * @param string $id the key of the value to be deleted
-	 * @return bool if no error happens during deletion
+	 * @return boolean|null if no error happens during deletion
 	 */
 	public function offsetUnset($id)
 	{

framework/Data/SqlMap/Statements/TMappedStatement.php

Doc Comments +3 added lines, -3 removed lines

@@ -468,7 +468,7 @@ 
 	 * Execute the select key statement, used to obtain last insert ID.
 	 * @param IDbConnection $connection database connection
 	 * @param mixed $parameter insert statement parameter
-	 * @param TSqlMapSelectKey $selectKey select key statement
+	 * @param \Prado\Data\SqlMap\Configuration\TSqlMapSelectKey $selectKey select key statement
 	 * @return string last insert ID.
 	 */
 	protected function executeSelectKey($connection, $parameter, $selectKey)
@@ -595,7 +595,7 @@ 
 	/**
 	 * Apply the result to a TList or an array.
 	 * @param array $row a result set row retrieved from the database
-	 * @param object $resultObject result object, array or list
+	 * @param \ArrayAccess $resultObject result object, array or list
 	 * @return object result filled with data.
 	 */
 	protected function fillResultArrayList($row, $resultObject)
@@ -640,7 +640,7 @@ 
 	 * Fills the result object according to result mappings.
 	 * @param string $resultMapName result map name.
 	 * @param array $row a result set row retrieved from the database
-	 * @param null|mixed $parentGroup
+	 * @param null|string $parentGroup
 	 * @param null|&object $resultObject result object to fill, will create new instances if required.
 	 * @return object result object filled with data.
 	 */

framework/Exceptions/TErrorHandler.php

Doc Comments +13 added lines, -1 removed lines

@@ -118,7 +118,7 @@ 
 	 * raised in {@link TApplication}.
 	 * The method mainly uses appropriate template to display the error/exception.
 	 * It terminates the application immediately after the error is displayed.
-	 * @param mixed $sender sender of the event
+	 * @param null|\Prado\TApplication $sender sender of the event
 	 * @param mixed $param event parameter (if the event is raised by TApplication, it refers to the exception instance)
 	 */
 	public function handleError($sender, $param)
@@ -244,6 +244,9 @@ 
 		}
 	}
 
+	/**
+	 * @return string
+	 */
 	protected function hidePrivatePathParts($value)
 	{
 		static $aRpl;
@@ -401,6 +404,9 @@ 
 		return $result;
 	}
 
+	/**
+	 * @return string
+	 */
 	private function getExactTraceAsString($exception)
 	{
 		if ($exception instanceof TPhpFatalErrorException &&
@@ -427,6 +433,9 @@ 
 		return $this->hidePrivatePathParts($exception->getTraceAsString());
 	}
 
+	/**
+	 * @param string $pattern
+	 */
 	private function getPropertyAccessTrace($trace, $pattern)
 	{
 		$result = null;
@@ -457,6 +466,9 @@ 
 		return $source;
 	}
 
+	/**
+	 * @param string $message
+	 */
 	private function addLink($message)
 	{
 		if (null !== ($class = $this->getErrorClassNameSpace($message))) {