sminnee / silverstripe-framework

core/ClassInfo.php

Doc Comments +4 added lines

@@ -219,6 +219,7 @@ 
 
 	/**
 	 * Returns true if the given class implements the given interface
+	 * @param string $interfaceName
 	 */
 	public static function classImplements($className, $interfaceName) {
 		return in_array($className, self::implementorsOf($interfaceName));
@@ -269,6 +270,9 @@ 
 
 	private static $method_from_cache = array();
 
+	/**
+	 * @param string $method
+	 */
 	public static function has_method_from($class, $method, $compclass) {
 		$lClass = strtolower($class);
 		$lMethod = strtolower($method);

core/manifest/ConfigStaticManifest.php

Doc Comments +1 added lines

@@ -83,6 +83,7 @@
 	 * Returns the path for a class or interface in the currently active manifest,
 	 * or any previous ones if later manifests aren't set to "exclusive".
 	 *
+	 * @param string $class
 	 * @return String
 	 */
 	public function getItemPath($class) {

dev/Debug.php

Doc Comments +14 added lines, -7 removed lines

@@ -214,6 +214,12 @@ 
 		set_exception_handler('exceptionHandler');
 	}
 
+	/**
+	 * @param integer $errno
+	 * @param string $errstr
+	 * @param string $errfile
+	 * @param integer $errline
+	 */
 	public static function noticeHandler($errno, $errstr, $errfile, $errline, $errcontext) {
 		if(error_reporting() == 0) return;
 		ini_set('display_errors', 0);
@@ -240,10 +246,10 @@ 
 	/**
 	 * Handle a non-fatal warning error thrown by PHP interpreter.
 	 *
-	 * @param unknown_type $errno
-	 * @param unknown_type $errstr
-	 * @param unknown_type $errfile
-	 * @param unknown_type $errline
+	 * @param integer $errno
+	 * @param string $errstr
+	 * @param string $errfile
+	 * @param integer $errline
 	 * @param unknown_type $errcontext
 	 */
 	public static function warningHandler($errno, $errstr, $errfile, $errline, $errcontext) {
@@ -274,8 +280,8 @@ 
 	 *
 	 * Runtime execution dies immediately once the error is generated.
 	 *
-	 * @param unknown_type $errno
-	 * @param unknown_type $errstr
+	 * @param integer $errno
+	 * @param string $errstr
 	 * @param unknown_type $errfile
 	 * @param unknown_type $errline
 	 * @param unknown_type $errcontext
@@ -315,7 +321,7 @@ 
 	 *                                     or "tech-speak". Used in the HTTP Header and ajax responses.
 	 * @param string $friendlyErrorDetail Detailed user-focused message. Is just used if no {@link ErrorPage} is found
 	 *                                    for this specific status code.
-	 * @return string HTML error message for non-ajax requests, plaintext for ajax-request.
+	 * @return boolean HTML error message for non-ajax requests, plaintext for ajax-request.
 	 */
 	public static function friendlyError($statusCode=500, $friendlyErrorMessage=null, $friendlyErrorDetail=null) {
 		if(!$friendlyErrorMessage) {
@@ -388,6 +394,7 @@ 
 	 * @param unknown_type $errfile
 	 * @param unknown_type $errline
 	 * @param unknown_type $errcontext
+	 * @param string $errtype
 	 */
 	public static function showError($errno, $errstr, $errfile, $errline, $errcontext, $errtype) {
 		if(!headers_sent()) {

dev/Deprecation.php

Doc Comments +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
 	 * greater than or equal to this version, a message will be raised
 	 *
 	 * @static
-	 * @param $ver string -
+	 * @param string|null $ver string -
 	 *     A php standard version string, see http://php.net/manual/en/function.version-compare.php for details.
 	 * @param null $forModule string -
 	 *    The name of a module. The passed version will be used as the check value for

filesystem/File.php

Doc Comments +2 added lines, -2 removed lines

@@ -234,7 +234,7 @@ 
 	 * Find a File object by the given filename.
 	 *
 	 * @param String $filename Matched against the "Name" property.
-	 * @return mixed null if not found, File object of found file
+	 * @return null|DataObject null if not found, File object of found file
 	 */
 	public static function find($filename) {
 		// Get the base file if $filename points to a resampled file
@@ -449,6 +449,7 @@ 
 	 * showing icons on filelinks, etc.
 	 * Possible group values are: "audio", "mov", "zip", "image".
 	 *
+	 * @param string $ext
 	 * @return String
 	 */
 	public static function get_app_category($ext) {
@@ -872,7 +873,6 @@ 
 	/**
 	 * Convert a php.ini value (eg: 512M) to bytes
 	 *
-	 * @param string $phpIniValue
 	 * @return int
 	 */
 	public static function ini2bytes($PHPiniValue) {

filesystem/GD.php

Doc Comments +8 added lines

@@ -150,6 +150,7 @@ 
 	 * successfully, the manipulation's cache key is removed.
 	 *
 	 * @param string $filename
+	 * @param string $manipulation
 	 * @return boolean
 	 */
 	public function failedResample($filename, $manipulation) {
@@ -417,6 +418,10 @@ 
 			return $useAsMinimum ? $this->resizeByWidth( $maxWidth ) : $this->resizeByHeight( $maxHeight );
 	}
 
+	/**
+	 * @param resource $image
+	 * @param string $webColor
+	 */
 	public static function color_web2gd($image, $webColor) {
 		if(substr($webColor,0,1) == "#") $webColor = substr($webColor,1);
 		$r = hexdec(substr($webColor,0,2));
@@ -521,6 +526,9 @@ 
 		return $output;
 	}
 
+	/**
+	 * @param string $dirname
+	 */
 	public function makeDir($dirname) {
 		if(!file_exists(dirname($dirname))) $this->makeDir(dirname($dirname));
 		if(!file_exists($dirname)) mkdir($dirname, Config::inst()->get('Filesystem', 'folder_create_mask'));

filesystem/ImagickBackend.php

Doc Comments +9 added lines, -9 removed lines

@@ -80,7 +80,7 @@ 
 	 *
 	 * Get the backend-specific resource handling the manipulations. Replaces Image::getGD()
 	 *
-	 * @return mixed
+	 * @return ImagickBackend
 	 */
 	public function getImageResource() {
 		return $this;
@@ -111,7 +111,7 @@ 
 	 *
 	 * @param int $width
 	 * @param int $height
-	 * @return Image_Backend
+	 * @return null|ImagickBackend
 	 */
 	public function resize($width, $height) {
 		if(!$this->valid()) return;
@@ -141,9 +141,9 @@ 
 	/**
 	 * resizeRatio
 	 *
-	 * @param int $width
-	 * @param int $height
-	 * @return Image_Backend
+	 * @param integer $maxWidth
+	 * @param integer $maxHeight
+	 * @return null|ImagickBackend
 	 */
 	public function resizeRatio($maxWidth, $maxHeight, $useAsMinimum = false) {
 		if(!$this->valid()) return;
@@ -163,7 +163,7 @@ 
 	 * resizeByWidth
 	 *
 	 * @param int $width
-	 * @return Image_Backend
+	 * @return null|ImagickBackend
 	 */
 	public function resizeByWidth($width) {
 		if(!$this->valid()) return;
@@ -178,7 +178,7 @@ 
 	 * resizeByHeight
 	 *
 	 * @param int $height
-	 * @return Image_Backend
+	 * @return null|ImagickBackend
 	 */
 	public function resizeByHeight($height) {
 		if(!$this->valid()) return;
@@ -194,7 +194,7 @@ 
 	 *
 	 * @param int $width
 	 * @param int $height
-	 * @return Image_Backend
+	 * @return null|ImagickBackend
 	 */
 	public function paddedResize($width, $height, $backgroundColor = "FFFFFF") {
 		$new = $this->resizeRatio($width, $height);
@@ -211,7 +211,7 @@ 
 	 *
 	 * @param int $width
 	 * @param int $height
-	 * @return Image_Backend
+	 * @return null|ImagickBackend
 	 */
 	public function croppedResize($width, $height) {
 		if(!$this->valid()) return;

forms/DateField.php

Doc Comments +8 added lines, -1 removed lines

@@ -90,6 +90,10 @@ 
 	 */
 	protected $valueObj = null;
 
+	/**
+	 * @param string $name
+	 * @param string $title
+	 */
 	public function __construct($name, $title = null, $value = null) {
 		if(!$this->locale) {
 			$this->locale = i18n::get_locale();
@@ -288,6 +292,9 @@ 
 		return $field;
 	}
 
+	/**
+	 * @param string $class
+	 */
 	public function castedCopy($class) {
 		$copy = new $class($this->name);
 		if($copy->hasMethod('setConfig')) {
@@ -326,7 +333,7 @@ 
 	 * @deprecated 4.0 Use the "DateField.default_config" config setting instead
 	 * @param String $k
 	 * @param mixed $v
-	 * @return boolean
+	 * @return boolean|null
 	 */
 	public static function set_default_config($k, $v) {
 		Deprecation::notice('4.0', 'Use the "DateField.default_config" config setting instead');

forms/DatetimeField.php

Doc Comments +18 added lines, -3 removed lines

@@ -56,6 +56,10 @@ 
 	 */
 	protected $config;
 
+	/**
+	 * @param string $name
+	 * @param string $title
+	 */
 	public function __construct($name, $title = null, $value = ""){
 		$this->config = $this->config()->default_config;
 
@@ -68,6 +72,9 @@ 
 		parent::__construct($name, $title, $value);
 	}
 
+	/**
+	 * @param Form $form
+	 */
 	public function setForm($form) {
 		parent::setForm($form);
 
@@ -78,6 +85,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @param string $name
+	 */
 	public function setName($name) {
 		parent::setName($name);
 
@@ -90,7 +100,7 @@ 
 
 	/**
 	 * @param array $properties
-	 * @return HTMLText
+	 * @return string
 	 */
 	public function FieldHolder($properties = array()) {
 		$config = array(
@@ -240,6 +250,7 @@ 
 
 	/**
 	 * @param FormField
+	 * @param DateField $field
 	 */
 	public function setDateField($field) {
 		$expected = $this->getName() . '[date]';
@@ -265,6 +276,7 @@ 
 
 	/**
 	 * @param FormField
+	 * @param TimeField $field
 	 */
 	public function setTimeField($field) {
 		$expected = $this->getName() . '[time]';
@@ -303,7 +315,7 @@ 
 	 * to set field-specific config options.
 	 *
 	 * @param string $name
-	 * @param mixed $val
+	 * @param string $val
 	 */
 	public function setConfig($name, $val) {
 		$this->config[$name] = $val;
@@ -321,7 +333,7 @@ 
 	 * to get field-specific config options.
 	 *
 	 * @param String $name Optional, returns the whole configuration array if empty
-	 * @return mixed
+	 * @return string
 	 */
 	public function getConfig($name = null) {
 		if($name) {
@@ -331,6 +343,9 @@ 
 		}
 	}
 
+	/**
+	 * @param RequiredFields $validator
+	 */
 	public function validate($validator) {
 		$dateValid = $this->dateField->validate($validator);
 		$timeValid = $this->timeField->validate($validator);