pradosoft / prado

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}';

framework/Web/UI/WebControls/TCaptchaValidator.php

Doc Comments +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
 
 	/**
 	 * @param string $token
-	 * @return string hash
+	 * @return integer hash
 	 */
 	private function generateTokenHash($token)
 	{

framework/Web/UI/WebControls/TCustomValidator.php

Doc Comments +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
 	}
 
 	/**
-	 * @return TControl|null control to be validated. Null if no control is found.
+	 * @return \Prado\Web\UI\TControl|null control to be validated. Null if no control is found.
 	 */
 	public function getValidationTarget()
 	{

framework/Web/UI/WebControls/TRepeater.php

Doc Comments +4 added lines, -4 removed lines

@@ -286,7 +286,7 @@ 
 	}
 
 	/**
-	 * @param ITemplate $value the template for repeater items
+	 * @param \Prado\Web\UI\TTemplate $value the template for repeater items
 	 * @throws TInvalidDataTypeException if the input is not an {@link ITemplate} or not null.
 	 */
 	public function setItemTemplate($value)
@@ -463,7 +463,7 @@ 
 	 * This method invokes {@link createItem} to create a new repeater item.
 	 * @param int $itemIndex zero-based item index.
 	 * @param TListItemType $itemType item type
-	 * @return TControl the created item, null if item is not created
+	 * @return \Prado\TComponent|null the created item, null if item is not created
 	 */
 	private function createItemInternal($itemIndex, $itemType)
 	{
@@ -483,7 +483,7 @@ 
 	 * @param int $itemIndex zero-based item index.
 	 * @param TListItemType $itemType item type
 	 * @param mixed $dataItem data to be associated with the item
-	 * @return TControl the created item, null if item is not created
+	 * @return \Prado\TComponent|null the created item, null if item is not created
 	 */
 	private function createItemWithDataInternal($itemIndex, $itemType, $dataItem)
 	{
@@ -506,7 +506,7 @@ 
 	 * Creates a repeater item instance based on the item type and index.
 	 * @param int $itemIndex zero-based item index
 	 * @param TListItemType $itemType item type
-	 * @return TControl created repeater item
+	 * @return null|\Prado\TComponent created repeater item
 	 */
 	protected function createItem($itemIndex, $itemType)
 	{

framework/Web/UI/ActiveControls/TCallbackErrorHandler.php

Spacing +4 added lines, -4 removed lines

@@ -78,15 +78,15 @@
 		$data['file'] = $this->hidePrivatePathParts($exception->getFile());
 		$data['line'] = $exception->getLine();
 		$data['trace'] = $exception->getTrace();
-		foreach($data['trace'] as $k => $v)
+		foreach ($data['trace'] as $k => $v)
 		{
-			if(isset($v['file']))
+			if (isset($v['file']))
 				$data['trace'][$k]['file'] = $this->hidePrivatePathParts($v['file']);
 
-			if(isset($v['args']))
+			if (isset($v['args']))
 			{
 				$argsout = [];
-				foreach($v['args'] as $kArg => $vArg)
+				foreach ($v['args'] as $kArg => $vArg)
 				{
 					$data['trace'][$k]['args'][$kArg] = TVarDumper::dump($vArg, 0, false);
 				}

framework/Util/TFirebugLogRoute.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 		$first = $logs[0][3];
 		$even = true;
 
-		$blocks = [ [ 'info', 'Tot Time', 'Time    ', '[Level] [Category] [Message]' ] ];
+		$blocks = [['info', 'Tot Time', 'Time    ', '[Level] [Category] [Message]']];
 		for ($i = 0, $n = count($logs); $i < $n; ++$i) {
 			if ($i < $n - 1) {
 				$timing['delta'] = $logs[$i + 1][3] - $logs[$i][3];

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

@@ -83,7 +83,7 @@ 
 
 	console.log ("[Tot Time] [Time    ] [Level] [Category] [Message]");
 
-EOD;
+eod;
 
 		return $string;
 	}
@@ -122,7 +122,7 @@ 
 }
 </script>
 
-EOD;
+eod;
 
 		return $string;
 	}

framework/I18N/core/DateTimeFormatInfo.php

Doc Comments +4 added lines, -2 removed lines

@@ -125,7 +125,6 @@ 
 	 * information. <b>N.B.</b>You should not initialize this class directly
 	 * unless you know what you are doing. Please use use
 	 * DateTimeFormatInfo::getInstance() to create an instance.
-	 * @param array $data ICU data for date time formatting.
 	 * @see getInstance()
 	 */
 	public function __construct($cultureInfo)
@@ -156,7 +155,7 @@ 
 
 	/**
 	 * Returns the DateTimeFormatInfo associated with the specified culture.
-	 * @param null|CultureInfo|string $culture the culture that gets the DateTimeFormat property.
+	 * @param string $culture the culture that gets the DateTimeFormat property.
 	 * @return DateTimeFormatInfo DateTimeFormatInfo for the specified
 	 * culture.
 	 */
@@ -175,6 +174,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $path
+	 */
 	public function getInfoByPath($path)
 	{
 		static $basePath = null;

Spacing +5 added lines, -5 removed lines

@@ -132,7 +132,7 @@ 
 	{
 		$this->properties = get_class_methods($this);
 
-		if(!($cultureInfo instanceof CultureInfo))
+		if (!($cultureInfo instanceof CultureInfo))
 			throw new Exception('Please provide a CultureInfo instance.');
 
 		$this->cultureInfo = $cultureInfo;
@@ -146,7 +146,7 @@ 
 	static public function getInvariantInfo()
 	{
 		static $invariant;
-		if($invariant === null)
+		if ($invariant === null)
 		{
 			$culture = CultureInfo::getInvariantCulture();
 			$invariant = $culture->getDateTimeFormat();
@@ -160,13 +160,13 @@ 
 	 * @return DateTimeFormatInfo DateTimeFormatInfo for the specified
 	 * culture.
 	 */
-	public static function getInstance($culture=null)
+	public static function getInstance($culture = null)
 	{
 
 		if ($culture instanceof CultureInfo)
 		{
 			return $culture->getDateTimeFormat();
-		} elseif(is_string($culture)) {
+		} elseif (is_string($culture)) {
 			$cultureInfo = CultureInfo::getInstance($culture);
 			return $cultureInfo->getDateTimeFormat();
 		} else {
@@ -178,7 +178,7 @@ 
 	public function getInfoByPath($path)
 	{
 		static $basePath = null;
-		if($basePath === null)
+		if ($basePath === null)
 		{
 			$basePath = 'calendar/' . $this->cultureInfo->getCalendar() . '/';
 		}

framework/I18N/core/NumberFormatInfo.php

Doc Comments +5 added lines, -1 removed lines

@@ -113,7 +113,6 @@ 
 	 * formatting information. <b>N.B.</b>You should not initialize this
 	 * class directly unless you know what you are doing. Please use use
 	 * NumberFormatInfo::getInstance() to create an instance.
-	 * @param array $data ICU data for date time formatting.
 	 * @param mixed $type
 	 * @see getInstance()
 	 */
@@ -129,6 +128,11 @@ 
 		$this->setPattern($type);
 	}
 
+	/**
+	 * @param string $path
+	 *
+	 * @return string
+	 */
 	public function getInfoByPath($path)
 	{
 		static $basePath = null;

Spacing +9 added lines, -9 removed lines

@@ -121,7 +121,7 @@ 
 	{
 		$this->properties = get_class_methods($this);
 
-		if(!($cultureInfo instanceof CultureInfo))
+		if (!($cultureInfo instanceof CultureInfo))
 			throw new Exception('Please provide a CultureInfo instance.');
 
 		$this->cultureInfo = $cultureInfo;
@@ -132,7 +132,7 @@ 
 	public function getInfoByPath($path)
 	{
 		static $basePath = null;
-		if($basePath === null)
+		if ($basePath === null)
 		{
 			$basePath = 'NumberElements/' . $this->cultureInfo->getDefaultNumberFormat() . '/';
 		}
@@ -149,7 +149,7 @@ 
 	 */
 	public function setPattern($type = NumberFormatInfo::DECIMAL)
 	{
-		$info = $this->getInfoByPath('patterns/'.$type);
+		$info = $this->getInfoByPath('patterns/' . $type);
 		$this->pattern = $this->parsePattern($info);
 	}
 
@@ -164,10 +164,10 @@ 
 	 * @param mixed $type
 	 * @return NumberFormatInfo default NumberFormatInfo.
 	 */
-	public static function getInvariantInfo($type=NumberFormatInfo::DECIMAL)
+	public static function getInvariantInfo($type = NumberFormatInfo::DECIMAL)
 	{
 		static $invariant;
-		if($invariant === null)
+		if ($invariant === null)
 		{
 			$culture = CultureInfo::getInvariantCulture();
 			$invariant = $culture->NumberFormat;
@@ -187,14 +187,14 @@ 
 	 * @see getPercentageInstance();
 	 * @see getScientificInstance();
 	 */
-	public static function getInstance($culture=null, $type=NumberFormatInfo::DECIMAL)
+	public static function getInstance($culture = null, $type = NumberFormatInfo::DECIMAL)
 	{
 		if ($culture instanceof CultureInfo)
 		{
 			$formatInfo = $culture->NumberFormat;
 			$formatInfo->setPattern($type);
 			return $formatInfo;
-		} elseif(is_string($culture)) {
+		} elseif (is_string($culture)) {
 			$cultureInfo = new CultureInfo($culture);
 			$formatInfo = $cultureInfo->NumberFormat;
 			$formatInfo->setPattern($type);
@@ -494,9 +494,9 @@ 
 	 * Gets the string to use as the currency symbol.
 	 * @return string currency symbol.
 	 */
-	public function getCurrencySymbol($currency='USD')
+	public function getCurrencySymbol($currency = 'USD')
 	{
-		$fmt = new \NumberFormatter( $this->cultureInfo->getName()."@currency=$currency", \NumberFormatter::CURRENCY );
+		$fmt = new \NumberFormatter($this->cultureInfo->getName() . "@currency=$currency", \NumberFormatter::CURRENCY);
 		return $fmt->getSymbol(\NumberFormatter::CURRENCY_SYMBOL);
 	}
 }

framework/Web/UI/WebControls/TDatePicker.php

Doc Comments +2 added lines, -2 removed lines

@@ -483,7 +483,7 @@ 
 	 * Loads date from drop down list data.
 	 * @param string $key the key that can be used to retrieve data from the input data collection
 	 * @param array $values the input data collection
-	 * @return array the date selected
+	 * @return string the date selected
 	 */
 	protected function getDateFromPostData($key, $values)
 	{
@@ -647,7 +647,7 @@ 
 	/**
 	 * Renders the calendar drop down list depending on the DateFormat pattern.
 	 * @param THtmlWriter $writer the Html writer to render the drop down lists.
-	 * @param array $date the current selected date
+	 * @param \DateTime $date the current selected date
 	 */
 	protected function renderCalendarSelections($writer, $date)
 	{