silverstripe / silverstripe-framework

tests/model/DbDatetimeTest.php

Doc Comments +1 added lines

@@ -13,6 +13,7 @@
 	/**
 	 * Check if dates match more or less. This takes into the account the db query
 	 * can overflow to the next second giving offset readings.
+	 * @param integer $offset
 	 */
 	private function matchesRoughly($date1, $date2, $comment = '', $offset) {
 		$allowedDifference = 5 + abs($offset); // seconds

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/parsers/ShortcodeParserTest.php

Doc Comments +4 added lines

@@ -175,6 +175,10 @@
 		$this->assertEquals('', $this->parser->parse('[test_shortcode][test_shortcode]'));
 	}
 	
+	/**
+	 * @param string $a
+	 * @param string $b
+	 */
 	protected function assertEqualsIgnoringWhitespace($a, $b, $message = null) {
 		$this->assertEquals(preg_replace('/\s+/', '', $a), preg_replace('/\s+/', '', $b), $message);
 	}

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);

tests/view/SSViewerTest.php

Doc Comments +13 added lines

@@ -112,6 +112,10 @@ 
 		));
 	}
 
+	/**
+	 * @param HTMLText $result
+	 * @param string[] $expected
+	 */
 	private function assertExpectedStrings($result, $expected) {
 		foreach ($expected as $expectedStr) {
 			$this->assertTrue(
@@ -718,6 +722,10 @@ 
 		$this->assertEquals('A A1 A1 i A1 ii A2 A3', $rationalisedResult);
 	}
 
+	/**
+	 * @param string $a
+	 * @param string $b
+	 */
 	public function assertEqualIgnoringWhitespace($a, $b) {
 		$this->assertEquals(preg_replace('/\s+/', '', $a), preg_replace('/\s+/', '', $b));
 	}
@@ -1328,6 +1336,11 @@ 
 		Config::inst()->update('SSViewer', 'source_file_comments', false);
 		Config::inst()->update('Director', 'environment_type', $origEnv);
 	}
+
+	/**
+	 * @param string $name
+	 * @param string $expected
+	 */
 	private function _renderWithSourceFileComments($name, $expected) {
 		$viewer = new SSViewer(array($name));
 		$data = new ArrayData(array());

thirdparty/jsmin/jsmin.php

Doc Comments +11 added lines

@@ -82,6 +82,10 @@ 
      action treats a string as a single character. Wow!

      action recognizes a regular expression if it is preceded by ( or , or =.

   */

+
+  /**
+   * @param integer $d
+   */
   protected function action($d) {

     switch($d) {

       case 1:

@@ -160,6 +164,9 @@ 
     }

   }

 

+  /**
+   * @return string
+   */
   protected function get() {

     $c = $this->lookAhead;

     $this->lookAhead = null;

@@ -187,6 +194,10 @@ 
   /* isAlphanum -- return true if the character is a letter, digit, underscore,

         dollar sign, or non-ASCII character.

   */

+
+  /**
+   * @param string $c
+   */
   protected function isAlphaNum($c) {

     return ord($c) > 126 || $c === '\\' || preg_match('/^[\w\$]$/', $c) === 1;

   }