silverstripe / silverstripe-framework

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/ContentNegotiatorTest.php

Doc Comments +1 added lines

@@ -5,6 +5,7 @@
 	/**
 	 * Small helper to render templates from strings
 	 * Cloned from SSViewerTest
+	 * @param string $templateString
 	 */
 	private function render($templateString, $data = null) {
 		$t = SSViewer::fromString($templateString);

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/php-peg/Compiler.php

Doc Comments +43 added lines

@@ -48,6 +48,9 @@ 
 		return $str ;
 	}
 
+	/**
+	 * @param string $id
+	 */
 	function save($id) {
 		return PHPBuilder::build()
 			->l(
@@ -56,6 +59,9 @@ 
 			);
 	}
 
+	/**
+	 * @param string $id
+	 */
 	function restore( $id, $remove = FALSE ) {
 		$code = PHPBuilder::build()
 			->l(
@@ -71,6 +77,10 @@ 
 		return $code ;
 	}
 
+	/**
+	 * @param string $on
+	 * @param PHPBuilder $fail
+	 */
 	function match_fail_conditional( $on, $match = NULL, $fail = NULL ) {
 		return PHPBuilder::build()
 			->b( 'if (' . $on . ')',
@@ -83,6 +93,9 @@ 
 			);
 	}
 	
+	/**
+	 * @param PHPBuilder $code
+	 */
 	function match_fail_block( $code ) {
 		$id = $this->varid() ;
 
@@ -131,6 +144,9 @@ 
 	public $type ;
 	public $value ;
 
+	/**
+	 * @param string $type
+	 */
 	function __construct( $type, $value = NULL ) {
 		$this->type = $type ;
 		$this->value = $value ;
@@ -269,6 +285,10 @@ 
 }
 
 class TokenLiteral extends TokenExpressionable {
+
+	/**
+	 * @param string $value
+	 */
 	function __construct( $value ) {
 		parent::__construct( 'literal', "'" . substr($value,1,-1) . "'" );
 	}
@@ -294,6 +314,9 @@ 
 		return $rx ;
 	}
 	
+	/**
+	 * @param string $value
+	 */
 	function __construct( $value ) {
 		parent::__construct('rx', self::escape($value));
 	}
@@ -304,6 +327,10 @@ 
 }
 
 class TokenWhitespace extends TokenTerminal {
+
+	/**
+	 * @param boolean $optional
+	 */
 	function __construct( $optional ) {
 		parent::__construct( 'whitespace', $optional ) ;
 	}
@@ -316,6 +343,10 @@ 
 }
 
 class TokenRecurse extends Token {
+
+	/**
+	 * @param string $value
+	 */
 	function __construct( $value ) {
 		parent::__construct( 'recurse', $value ) ;
 	}
@@ -438,6 +469,9 @@ 
 		$this->what = NULL ;
 	}
 
+	/**
+	 * @param string $what
+	 */
 	function set( $what, $val = TRUE ) {
 		$this->what = $what ;
 		$this->val = $val ;
@@ -498,6 +532,9 @@ 
 	public $mode;
 	public $rule;
 	
+	/**
+	 * @param RuleSet $parser
+	 */
 	function __construct($parser, $lines) {
 		$this->parser = $parser;
 		$this->lines = $lines;
@@ -595,6 +632,9 @@ 
 		[^/]               # Anything except /
 	)*/}xu' ;
 
+	/**
+	 * @param string $str
+	 */
 	function tokenize( $str, &$tokens, $o = 0 ) {
 
 		$pending = new Pending() ;
@@ -848,6 +888,9 @@ 
 		return self::$parsers[$class]->compile($indent, $match[3]);
 	}
 
+	/**
+	 * @param string $string
+	 */
 	static function compile( $string ) {
 		static $rx = '@
 			^([\x20\t]*)/\*!\* (?:[\x20\t]*(!?\w*))?   # Start with some indent, a comment with the special marker, then an optional name

thirdparty/php-peg/Parser.php

Doc Comments +25 added lines

@@ -9,6 +9,10 @@ 
  *  the bracket if a failed match + restore has moved the current position backwards - so we have to check that too.
  */
 class ParserRegexp {
+
+	/**
+	 * @param Parser $parser
+	 */
 	function __construct( $parser, $rx ) {
 		$this->parser = $parser ;
 		$this->rx = $rx . 'Sx' ;
@@ -63,6 +67,9 @@ 
 		return FALSE ;
 	}
 
+ 	/**
+ 	 * @param string $token
+ 	 */
  	function literal( $token ) {
  		/* Debugging: * / print( "Looking for token '$token' @ '" . substr( $this->string, $this->pos ) . "'\n" ) ; /* */
  		$toklen = strlen( $token ) ;
@@ -74,11 +81,17 @@ 
 		return FALSE ;
 	}
 
+	/**
+	 * @param string $rx
+	 */
 	function rx( $rx ) {
 		if ( !isset( $this->regexps[$rx] ) ) $this->regexps[$rx] = new ParserRegexp( $this, $rx ) ;
 		return $this->regexps[$rx]->match() ;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	function expression( $result, $stack, $value ) {
 		$stack[] = $result; $rv = false;
 		
@@ -100,14 +113,23 @@ 
 		return is_array($rv) ? $rv['text'] : ($rv ? $rv : '');
 	}
 	
+	/**
+	 * @param string $key
+	 */
 	function packhas( $key, $pos ) {
 		return false ;
 	}
 
+	/**
+	 * @param string $key
+	 */
 	function packread( $key, $pos ) {
 		throw 'PackRead after PackHas=>false in Parser.php' ;
 	}
 
+	/**
+	 * @param string $key
+	 */
 	function packwrite( $key, $pos, $res ) {
 		return $res ;
 	}
@@ -144,6 +166,9 @@ 
 		return $result ;
 	}
 
+	/**
+	 * @param string $storetag
+	 */
 	function store ( &$result, $subres, $storetag = NULL ) {
 		$result['text'] .= $subres['text'] ;
 

thirdparty/simpletest/compatibility.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     
     /**
      *    Creates a copy whether in PHP5 or PHP4.
-     *    @param object $object     Thing to copy.
+     *    @param SimpleFormTag $object     Thing to copy.
      *    @return object            A copy.
      *    @access public
      *    @static

thirdparty/simpletest/cookies.php

Doc Comments -2 removed lines

@@ -239,8 +239,6 @@
     /**
      *    Removes expired and temporary cookies as if
      *    the browser was closed and re-opened.
-     *    @param string/integer $now   Time to test expiry against.
-     *    @access public
      */
     function restartSession($date = false) {
         $surviving_cookies = array();