pradosoft / prado

framework/Web/UI/WebControls/TImageButton.php

Doc Comments +1 added lines, -1 removed lines

@@ -243,7 +243,7 @@
 	 * invoke the page's {@link TPage::validate validate} method first.
 	 * It will raise {@link onClick OnClick} and {@link onCommand OnCommand} events.
 	 * This method is mainly used by framework and control developers.
-	 * @param TEventParameter $param the event parameter
+	 * @param \Prado\Web\UI\ActiveControls\TCallbackEventParameter $param the event parameter
 	 */
 	public function raisePostBackEvent($param)
 	{

framework/Web/UI/WebControls/TLinkButton.php

Doc Comments +1 added lines, -1 removed lines

@@ -243,7 +243,7 @@
 	 * invoke the page's {@link TPage::validate validate} method first.
 	 * It will raise {@link onClick OnClick} and {@link onCommand OnCommand} events.
 	 * This method is mainly used by framework and control developers.
-	 * @param TEventParameter $param the event parameter
+	 * @param \Prado\Web\UI\ActiveControls\TCallbackEventParameter $param the event parameter
 	 */
 	public function raisePostBackEvent($param)
 	{

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/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)
 	{

Upper-Lower-Casing +5 added lines, -5 removed lines

@@ -206,7 +206,7 @@ 
   {$this->action} {$parameters}{$optional}
 {$description}
 
-EOD;
+eod;
 	}
 
 	/**
@@ -378,7 +378,7 @@ 
   </parameters>
   -->
 </application>
-EOD;
+eod;
 	}
 
 	/**
@@ -403,7 +403,7 @@ 
 <h1>Welcome to PRADO!</h1>
 </body>
 </html>
-EOD;
+eod;
 	}
 }
 
@@ -678,7 +678,7 @@ 
 	 * @soapproperty
 	 */
 
-EOD;
+eod;
 		}
 		$prop .= "\tpublic $name;";
 		return $prop;
@@ -711,7 +711,7 @@ 
 	}
 }
 
-EOD;
+eod;
 	}
 }
 

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/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))) {

Spacing +1 added lines, -1 removed lines

@@ -247,7 +247,7 @@
 	protected function hidePrivatePathParts($value)
 	{
 		static $aRpl;
-		if($aRpl === null)
+		if ($aRpl === null)
 		{
 			$aRpl[$_SERVER['DOCUMENT_ROOT']] = '${DocumentRoot}';
 			$aRpl[str_replace('/', DIRECTORY_SEPARATOR, $_SERVER['DOCUMENT_ROOT'])] = '${DocumentRoot}';