silverstripe / silverstripe-framework

tests/security/SecurityTest.php

Doc Comments +5 added lines, -1 removed lines

@@ -132,7 +132,7 @@ 
 	 *
 	 * @param string $url
 	 * @param int $limit Max number of requests
-	 * @return SS_HTTPResponse
+	 * @return SilverStripe\Control\SS_HTTPResponse|null
 	 */
 	protected function getRecursive($url, $limit = 10) {
 		$this->cssParser = null;
@@ -589,6 +589,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'));
@@ -609,6 +611,8 @@ 
 
 	/**
 	 * Helper method to execute a change password form
+	 * @param string $oldPassword
+	 * @param string $newPassword
 	 */
 	public function doTestChangepasswordForm($oldPassword, $newPassword) {
 		return $this->submitForm(

view/Requirements.php

Doc Comments +5 added lines, -3 removed lines

@@ -68,6 +68,7 @@ 
 	 * while automatically busting this cache every time the file is changed.
 	 *
 	 * @param bool
+	 * @param boolean $var
 	 */
 	public static function set_suffix_requirements($var) {
 		self::backend()->setSuffixRequirements($var);
@@ -276,7 +277,7 @@ 
 	 * Attach requirements inclusion to X-Include-JS and X-Include-CSS headers on the given
 	 * HTTP Response
 	 *
-	 * @param SS_HTTPResponse $response
+	 * @param HTTPResponse $response
 	 */
 	public static function include_in_response(HTTPResponse $response) {
 		self::backend()->includeInResponse($response);
@@ -339,7 +340,7 @@ 
 	 * </code>
 	 *
 	 * @param string $combinedFileName Filename of the combined file relative to docroot
-	 * @param array  $files            Array of filenames relative to docroot
+	 * @param string[]  $files            Array of filenames relative to docroot
 	 * @param string $media
 	 *
 	 * @return bool|void
@@ -764,6 +765,7 @@ 
 	 * Forces the JavaScript requirements to the end of the body, right before the closing tag
 	 *
 	 * @param bool
+	 * @param boolean $var
 	 * @return $this
 	 */
 	public function setForceJSToBottom($var) {
@@ -1263,7 +1265,7 @@ 
 	 * Attach requirements inclusion to X-Include-JS and X-Include-CSS headers on the given
 	 * HTTP Response
 	 *
-	 * @param SS_HTTPResponse $response
+	 * @param HTTPResponse $response
 	 */
 	public function includeInResponse(HTTPResponse $response) {
 		$this->processCombinedFiles();