silverstripe / silverstripe-framework

security/PasswordValidator.php

Doc Comments +4 added lines, -2 removed lines

@@ -28,6 +28,7 @@ 
 
 	/**
 	 * Minimum password length
+	 * @param integer $minLength
 	 */
 	public function minLength($minLength) {
 		$this->minLength = $minLength;
@@ -39,8 +40,8 @@ 
 	 *
 	 * Eg: $this->characterStrength(3, array("lowercase", "uppercase", "digits", "punctuation"))
 	 *
-	 * @param $minScore The minimum number of character tests that must pass
-	 * @param $testNames The names of the tests to perform
+	 * @param integer $minScore The minimum number of character tests that must pass
+	 * @param string[] $testNames The names of the tests to perform
 	 */
 	public function characterStrength($minScore, $testNames) {
 		$this->minScore = $minScore;
@@ -50,6 +51,7 @@ 
 
 	/**
 	 * Check a number of previous passwords that the user has used, and don't let them change to that.
+	 * @param integer $count
 	 */
 	public function checkHistoricalPasswords($count) {
 		$this->historicalPasswordCount = $count;

security/Permission.php

Doc Comments +2 added lines, -2 removed lines

@@ -652,7 +652,7 @@ 
 	 * Recursively traverse the nested list of declared permissions and create
 	 * a linear list.
 	 *
-	 * @param aeeay $declared Nested structure of permissions.
+	 * @param boolean $declared Nested structure of permissions.
 	 * @param $list List of permissions in the structure. The result will be
 	 *              written to this array.
 	 */
@@ -740,7 +740,7 @@ 
 	/**
 	 * Get permissions
 	 *
-	 * @return array Associative array of permissions in this permission
+	 * @return string Associative array of permissions in this permission
 	 *               group. The array indicies are the permission codes as
 	 *               used in {@link Permission::check()}. The value is
 	 *               suitable for using in an interface.

tasks/EncryptAllPasswordsTask.php

Doc Comments +1 added lines

@@ -66,6 +66,7 @@
 
 	/**
 	 * @todo This should really be taken care of by TestRunner
+	 * @param string $msg
 	 */
 	protected function debugMessage($msg) {
 		if(class_exists('SapphireTest', false) && !SapphireTest::is_running_test()) {

tests/api/RestfulServiceTest.php

Doc Comments +2 added lines

@@ -208,6 +208,8 @@
 	 * Simulate cached response file for testing error requests that are supposed to have cache files
 	 *
 	 * @todo Generate the cachepath without hardcoding the cache data
+	 * @param RestfulServiceTest_MockErrorService $connection
+	 * @param string $subUrl
 	 */
 	private function createFakeCachedResponse($connection, $subUrl) {
 		$fullUrl = $connection->getAbsoluteRequestURL($subUrl);

tests/core/manifest/ConfigManifestTest.php

Doc Comments +3 added lines, -2 removed lines

@@ -10,7 +10,7 @@ 
 
 	/**
 	 * This is a helper method for getting a new manifest
-	 * @param $name
+	 * @param string $name
 	 * @return any
 	 */
 	protected function getConfigFixtureValue($name) {
@@ -20,6 +20,7 @@ 
 
 	/**
 	 * This is a helper method for displaying a relevant message about a parsing failure
+	 * @param string $path
 	 */
 	protected function getParsedAsMessage($path) {
 		return sprintf('Reference path "%s" failed to parse correctly', $path);
@@ -41,7 +42,7 @@ 
 
 	/**
 	 * A helper method to return a mock of the manifest in order to test expectations and reduce dependency
-	 * @param $methods
+	 * @param string[] $methods
 	 * @return SS_ConfigManifest
 	 */
 	protected function getManifestMock($methods) {

tests/core/manifest/ManifestFileFinderTest.php

Doc Comments +4 added lines

@@ -14,6 +14,10 @@
 		parent::__construct();
 	}
 
+	/**
+	 * @param ManifestFileFinder $finder
+	 * @param string[] $expect
+	 */
 	public function assertFinderFinds($finder, $expect, $message = null) {
 		$found = $finder->find($this->base);
 

tests/core/manifest/TemplateLoaderTest.php

Doc Comments +6 added lines

@@ -174,12 +174,18 @@
 		$this->loader->pushManifest($this->manifest);
 	}
 
+	/**
+	 * @param string[] $templates
+	 */
 	protected function createTestTemplates($templates) {
 		foreach ($templates as $template) {
 			file_put_contents($template, '');
 		}
 	}
 
+	/**
+	 * @param string[] $templates
+	 */
 	protected function removeTestTemplates($templates) {
 		foreach ($templates as $template) {
 			unlink($template);

tests/core/startup/ParameterConfirmationTokenTest.php

Doc Comments +3 added lines

@@ -23,6 +23,9 @@
 
 class ParameterConfirmationTokenTest extends SapphireTest {
 
+	/**
+	 * @param string $part
+	 */
 	private function addPart($answer, $slash, $part) {
 		$bare = str_replace('/', '', $part);
 

tests/dev/CsvBulkLoaderTest.php

Doc Comments +3 added lines

@@ -222,6 +222,9 @@
 	}
 
 
+	/**
+	 * @param resource $file
+	 */
 	protected function getLineCount(&$file) {
 		$i = 0;
 		while(fgets($file) !== false) $i++;