silverstripe / silverstripe-framework

tests/model/ArrayListTest.php

Doc Comments +4 added lines

@@ -802,6 +802,10 @@
 	public $First;
 	public $Second;
 
+	/**
+	 * @param string $first
+	 * @param string $second
+	 */
 	public function __construct($first, $second) {
 		$this->First  = $first;
 		$this->Second = $second;

tests/model/DataObjectTest.php

Doc Comments +4 added lines

@@ -1111,6 +1111,10 @@
 		);
 	}
 
+	/**
+	 * @param DataObjectTest_Team $object
+	 * @param string $method
+	 */
 	protected function makeAccessible($object, $method) {
 		$reflectionMethod = new ReflectionMethod($object, $method);
 		$reflectionMethod->setAccessible(true);

tests/model/HierarchyTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -512,7 +512,7 @@
 	 * Get the HTML class attribute from a node in the sitetree
 	 *
 	 * @param $html
-	 * @param $node
+	 * @param DataObject|null $node
 	 * @return string
 	 */
 	protected function getNodeClassFromTree($html, $node) {

tests/security/GroupTest.php

Doc Comments +4 added lines

@@ -175,6 +175,10 @@
 
 class GroupTest_MemberForm extends Form {
 
+	/**
+	 * @param GroupTest $controller
+	 * @param string $name
+	 */
 	public function __construct($controller, $name) {
 		$fields = singleton('GroupTest_Member')->getCMSFields();
 		$actions = new FieldList(

tests/security/MemberTest.php

Doc Comments +3 added lines

@@ -913,6 +913,9 @@
  */
 class MemberTest_ValidatorExtension extends DataExtension implements TestOnly {
 
+	/**
+	 * @param Member_Validator $validator
+	 */
 	public function updateValidator(&$validator) {
 		$validator->addRequiredField('Surname');
 		$validator->removeRequiredField('FirstName');

tests/security/SecurityTest.php

Doc Comments +4 added lines

@@ -577,6 +577,8 @@ 
 	/**
 	 * Execute a log-in form using Director::test().
 	 * Helper method for the tests above
+	 * @param string $email
+	 * @param string $password
 	 */
 	public function doTestLoginForm($email, $password, $backURL = 'test/link') {
 		$this->get(Config::inst()->get('Security', 'logout_url'));
@@ -597,6 +599,8 @@ 
 
 	/**
 	 * Helper method to execute a change password form
+	 * @param string $oldPassword
+	 * @param string $newPassword
 	 */
 	public function doTestChangepasswordForm($oldPassword, $newPassword) {
 		return $this->submitForm(

tests/view/SSViewerCacheBlockTest.php

Doc Comments +6 added lines

@@ -28,6 +28,9 @@ 
 		"Versioned('Stage', 'Live')"
 	);
 
+	/**
+	 * @param string $entropy
+	 */
 	public function setEntropy($entropy) {
 		$this->entropy = $entropy;
 	}
@@ -53,6 +56,9 @@ 
 		SS_Cache::set_cache_lifetime('cacheblock', $cacheOn ? 600 : -1);
 	}
 
+	/**
+	 * @param string $template
+	 */
 	protected function _runtemplate($template, $data = null) {
 		if ($data === null) $data = $this->data;
 		if (is_array($data)) $data = $this->data->customise($data);

thirdparty/jstree/_demo/_inc/class._database_i.php

Doc Comments +3 added lines

@@ -97,6 +97,9 @@
 		return $return;
 	}
 
+	/**
+	 * @param boolean $index
+	 */
 	function f($index) {
 		return stripslashes($this->row[$index]);
 	}

thirdparty/jstree/_demo/_inc/class.tree.php

Doc Comments +16 added lines

@@ -61,9 +61,17 @@ 
 		return $path;
 	}
 
+	/**
+	 * @param integer $parent
+	 * @param integer $position
+	 */
 	function _create($parent, $position) {
 		return $this->_move(0, $parent, $position);
 	}
+
+	/**
+	 * @param integer $id
+	 */
 	function _remove($id) {
 		if((int)$id === 1) { return false; }
 		$data = $this->_get_node($id);
@@ -100,6 +108,10 @@ 
 		);
 		return true;
 	}
+
+	/**
+	 * @param integer $id
+	 */
 	function _move($id, $ref_id, $position = 0, $is_copy = false) {
 		if((int)$ref_id === 0 || (int)$id === 1) { return false; }
 		$sql		= array();						// Queries executed at the end
@@ -238,6 +250,10 @@ 
 		if($is_copy) $this->_fix_copy($ind, $position);
 		return $node === false || $is_copy ? $ind : true;
 	}
+
+	/**
+	 * @param integer $position
+	 */
 	function _fix_copy($id, $position) {
 		$node = $this->_get_node($id);
 		$children = $this->_get_children($id, true);