owncloud / core

lib/private/template.php

Doc Comments +1 added lines, -1 removed lines

@@ -136,7 +136,7 @@
 	 * Returns the list of members for a group-principal
 	 *
 	 * @param string $principal
-	 * @return array
+	 * @return string[]
 	 */
 	function getGroupMemberSet($principal) {
 		// TODO: for now the group principal has only one member, the user itself

lib/private/template/functions.php

Doc Comments +4 added lines, -3 removed lines

@@ -30,7 +30,7 @@ 
 
 /**
  * Prints a sanitized string
- * @param string|array $string the string which will be escaped and printed
+ * @param string $string the string which will be escaped and printed
  */
 function p($string) {
 	print(OC_Util::sanitizeHTML($string));
@@ -39,7 +39,7 @@ 
 /**
  * Prints an unsanitized string - usage of this function may result into XSS.
  * Consider using p() instead.
- * @param string|array $string the string which will be printed as it is
+ * @param string $string the string which will be printed as it is
  */
 function print_unescaped($string) {
 	print($string);
@@ -184,7 +184,7 @@ 
  * make preview_icon available as a simple function
  * Returns the path to the preview of the image.
  * @param string $path path of file
- * @return link to the preview
+ * @return string to the preview
  *
  * For further information have a look at OC_Helper::previewIcon
  */
@@ -194,6 +194,7 @@ 
 
 /**
  * @param string $path
+ * @param string $token
  */
 function publicPreview_icon ( $path, $token ) {
 	return OC_Helper::publicPreviewIcon( $path, $token );

tests/lib/appframework/middleware/security/CORSMiddlewareTest.php

Unused Use Statements -1 removed lines

@@ -27,7 +27,6 @@
 use OCP\IUser;
 use OCP\ILogger;
 use OCP\Files\Folder;
-
 use OC\Share20\Exception\ShareNotFound;
 
 /**

tests/lib/files/view.php

Doc Comments +13 added lines, -2 removed lines

@@ -398,6 +398,10 @@ 
 		$this->copyBetweenStorages($storage1, $storage2);
 	}
 
+	/**
+	 * @param \OC\Files\Storage\Storage $storage1
+	 * @param \OC\Files\Storage\Storage $storage2
+	 */
 	function copyBetweenStorages($storage1, $storage2) {
 		\OC\Files\Filesystem::mount($storage1, array(), '/');
 		\OC\Files\Filesystem::mount($storage2, array(), '/substorage');
@@ -444,6 +448,10 @@ 
 		$this->moveBetweenStorages($storage1, $storage2);
 	}
 
+	/**
+	 * @param \OC\Files\Storage\Storage $storage1
+	 * @param \OC\Files\Storage\Storage $storage2
+	 */
 	function moveBetweenStorages($storage1, $storage2) {
 		\OC\Files\Filesystem::mount($storage1, array(), '/');
 		\OC\Files\Filesystem::mount($storage2, array(), '/substorage');
@@ -1004,6 +1012,9 @@ 
 		$this->doTestCopyRenameFail('copy');
 	}
 
+	/**
+	 * @param string $operation
+	 */
 	private function doTestCopyRenameFail($operation) {
 		$storage1 = new Temporary(array());
 		/** @var \PHPUnit_Framework_MockObject_MockObject | \OC\Files\Storage\Temporary $storage2 */
@@ -1417,7 +1428,7 @@ 
 	/**
 	 * Create test movable mount points
 	 *
-	 * @param array $mountPoints array of mount point locations
+	 * @param string[] $mountPoints array of mount point locations
 	 * @return array array of MountPoint objects
 	 */
 	private function createTestMovableMountPoints($mountPoints) {
@@ -1641,7 +1652,7 @@ 
 	 * @param string $lockedPath path of the locked item to check
 	 * @param string $hookType hook type
 	 * @param int $expectedLockBefore expected lock during pre hooks
-	 * @param int $expectedLockduring expected lock during operation
+	 * @param int $expectedLockDuring expected lock during operation
 	 * @param int $expectedLockAfter expected lock during post hooks
 	 * @param int $expectedStrayLock expected lock after returning, should
 	 * be null (unlock) for most operations

tests/lib/repair/cleantags.php

Doc Comments +2 added lines, -2 removed lines

@@ -160,8 +160,8 @@
 	}
 
 	/**
-	 * @param $tableName
-	 * @param $idName
+	 * @param string $tableName
+	 * @param string $idName
 	 * @return int
 	 */
 	protected function getLastInsertID($tableName, $idName) {