nextcloud / server

lib/private/files/type/loader.php

Unused Use Statements -1 removed lines

@@ -23,7 +23,6 @@
 
 use OCP\Files\IMimeTypeLoader;
 use OCP\IDBConnection;
-
 use Doctrine\DBAL\Exception\UniqueConstraintViolationException;
 
 /**

lib/private/files/view.php

Doc Comments +3 added lines, -3 removed lines

@@ -200,7 +200,7 @@ 
 	 * and does not take the chroot into account )
 	 *
 	 * @param string $path
-	 * @return \OCP\Files\Mount\IMountPoint
+	 * @return Mount\MountPoint|null
 	 */
 	public function getMount($path) {
 		return Filesystem::getMountManager()->find($this->getAbsolutePath($path));
@@ -948,7 +948,7 @@ 
 
 	/**
 	 * @param string $path
-	 * @return bool|string
+	 * @return string|false
 	 * @throws \OCP\Files\InvalidPathException
 	 */
 	public function toTmpFile($path) {
@@ -2071,7 +2071,7 @@ 
 
 	/**
 	 * @param string $filename
-	 * @return array
+	 * @return string[]
 	 * @throws \OC\User\NoUserException
 	 * @throws NotFoundException
 	 */

lib/private/group/manager.php

Doc Comments +1 added lines, -1 removed lines

@@ -154,7 +154,7 @@
 
 	/**
 	 * @param string $gid
-	 * @return \OCP\IGroup
+	 * @return null|Group
 	 */
 	protected function getGroupObject($gid) {
 		$backends = array();

lib/private/helper.php

Doc Comments +4 added lines, -1 removed lines

@@ -77,6 +77,9 @@ 
 		return \OC::$server->getURLGenerator()->linkToRoute('core_ajax_preview', ['x' => 32, 'y' => 32, 'file' => $path]);
 	}
 
+	/**
+	 * @param string $path
+	 */
 	public static function publicPreviewIcon( $path, $token ) {
 		return \OC::$server->getURLGenerator()->linkToRoute('core_ajax_public_preview', ['x' => 32, 'y' => 32, 'file' => $path, 't' => $token]);
 	}
@@ -591,7 +594,7 @@ 
 	 *
 	 * @param string $path
 	 * @param \OCP\Files\FileInfo $rootInfo (optional)
-	 * @return array
+	 * @return string
 	 * @throws \OCP\Files\NotFoundException
 	 */
 	public static function getStorageInfo($path, $rootInfo = null) {

lib/private/integritycheck/helpers/fileaccesshelper.php

Doc Comments +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
 	 * Wrapper around file_get_contents($filename, $data)
 	 *
 	 * @param string $filename
-	 * @return string|false
+	 * @return string
 	 */
 	public function file_get_contents($filename) {
 		return file_get_contents($filename);
@@ -52,8 +52,8 @@ 
 	 * Wrapper around file_put_contents($filename, $data)
 	 *
 	 * @param string $filename
-	 * @param $data
-	 * @return int|false
+	 * @param string $data
+	 * @return integer
 	 */
 	public function file_put_contents($filename, $data) {
 		return file_put_contents($filename, $data);

lib/private/l10n/factory.php

Doc Comments +5 added lines

@@ -278,6 +278,11 @@
 	 */
 	// FIXME This method is only public, until OC_L10N does not need it anymore,
 	// FIXME This is also the reason, why it is not in the public interface
+
+	/**
+	 * @param string|boolean $app
+	 * @param string|null $lang
+	 */
 	public function getL10nFilesForApp($app, $lang) {
 		$languageFiles = [];
 

lib/private/l10n/l10n.php

Doc Comments +1 added lines, -1 removed lines

@@ -176,7 +176,7 @@
 	 * Returns an associative array with all translations
 	 *
 	 * Called by \OC_L10N_String
-	 * @return array
+	 * @return string[]
 	 */
 	public function getTranslations() {
 		return $this->translations;

lib/private/mail/message.php

Doc Comments +1 added lines, -1 removed lines

@@ -162,7 +162,7 @@
 	/**
 	 * Set the BCC recipients of this message.
 	 *
-	 * @param array $recipients Example: array('recipient@domain.org', 'other@domain.org' => 'A name')
+	 * @param string[] $recipients Example: array('recipient@domain.org', 'other@domain.org' => 'A name')
 	 * @return $this
 	 */
 	public function setBcc(array $recipients) {

lib/private/memcache/apc.php

Doc Comments +1 added lines, -1 removed lines

@@ -155,7 +155,7 @@
 	 * Set a value in the cache if it's not already stored
 	 *
 	 * @param string $key
-	 * @param mixed $value
+	 * @param integer $value
 	 * @param int $ttl Time To Live in seconds. Defaults to 60*60*24
 	 * @return bool
 	 * @throws \Exception