silverstripe / silverstripe-framework

model/HTMLValue.php

Doc Comments +2 added lines, -2 removed lines

@@ -152,8 +152,8 @@
 	/**
 	 * Make an xpath query against this HTML
 	 *
-	 * @param $query string - The xpath query string
-	 * @return DOMNodeList
+	 * @param string $query string - The xpath query string
+	 * @return \DOMNodeList
 	 */
 	public function query($query) {
 		$xp = new DOMXPath($this->getDocument());

model/ManyManyList.php

Doc Comments +2 added lines, -2 removed lines

@@ -115,7 +115,7 @@ 
 	 * Create a DataObject from the given SQL row.
 	 *
 	 * @param array $row
-	 * @return DataObject
+	 * @return SilverStripe\Model\DataObject
 	 */
 	protected function createDataObject($row) {
 		// remove any composed fields
@@ -265,7 +265,7 @@ 
 	 * Note that for a ManyManyList, the item is never actually deleted, only
 	 * the join table is affected.
 	 *
-	 * @param DataObject $item
+	 * @param SilverStripe\Model\DataObject $item
 	 */
 	public function remove($item) {
 		if(!($item instanceof $this->dataClass)) {

model/UnsavedRelationList.php

Doc Comments +1 added lines

@@ -56,6 +56,7 @@
 	 * Create a new UnsavedRelationList
 	 *
 	 * @param string $dataClass The DataObject class used in the relation
+	 * @param string $relationName
 	 */
 	public function __construct($baseClass, $relationName, $dataClass) {
 		$this->baseClass = $baseClass;

model/versioning/Versioned.php

Doc Comments +5 added lines, -5 removed lines

@@ -1062,7 +1062,7 @@ 
 	/**
 	 * Find objects in the given relationships, merging them into the given list
 	 *
-	 * @param array $source Config property to extract relationships from
+	 * @param string $source Config property to extract relationships from
 	 * @param bool $recursive True if recursive
 	 * @param ArrayList $list Optional list to add items to
 	 * @return ArrayList The list
@@ -1182,7 +1182,7 @@ 
 	/**
 	 * Check if the current user can delete this record from live
 	 *
-	 * @param null $member
+	 * @param Member|null $member
 	 * @return mixed
 	 */
 	public function canUnpublish($member = null) {
@@ -1215,7 +1215,7 @@ 
 	 * If extended, ensure that both canDelete and canUnpublish are extended also
 	 *
 	 * @param Member $member
-	 * @return bool
+	 * @return null|boolean
 	 */
 	public function canArchive($member = null) {
 		// Skip if invoked by extendedCan()
@@ -1292,7 +1292,7 @@ 
 	 * Extend permissions to include additional security for objects that are not published to live.
 	 *
 	 * @param Member $member
-	 * @return bool|null
+	 * @return false|null
 	 */
 	public function canView($member = null) {
 		// Invoke default version-gnostic canView
@@ -2418,7 +2418,7 @@ 
 	/**
 	 * Returns an array of possible stages.
 	 *
-	 * @return array
+	 * @return string[]
 	 */
 	public function getVersionedStages() {
 		if($this->hasStages()) {

security/PasswordEncryptor.php

Doc Comments +3 added lines, -2 removed lines

@@ -87,6 +87,7 @@ 
 	 * but is necessary for retain compatibility with password hashed
 	 * with flawed algorithms - see {@link PasswordEncryptor_LegacyPHPHash} and
 	 * {@link PasswordEncryptor_Blowfish}
+	 * @param string $password
 	 */
 	public function check($hash, $password, $salt = null, $member = null) {
 		return $hash === $this->encrypt($password, $salt, $member);
@@ -129,8 +130,7 @@ 
 	/**
 	 * Gets the cost that is set for the blowfish algorithm
 	 *
-	 * @param int $cost
-	 * @return null
+	 * @return integer
 	 */
 	public static function get_cost() {
 		return self::$cost;
@@ -242,6 +242,7 @@ 
 
 	/**
 	 * self::$cost param is forced to be two digits with leading zeroes for ints 4-9
+	 * @param string $password
 	 */
 	public function salt($password, $member = null) {
 		$generator = new RandomGenerator();