Completed
Pull Request — master (#28701)
by Sujith
17:57 queued 06:27
created
apps/dav/lib/Upload/ChunkingPlugin.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@
 block discarded – undo
90 90
 	 *
91 91
 	 * @param string $path source path
92 92
 	 * @param string $destination destination path
93
-	 * @return bool|void false to stop handling, void to skip this handler
93
+	 * @return null|false false to stop handling, void to skip this handler
94 94
 	 */
95 95
 	public function performMove($path, $destination) {
96 96
 		if (!$this->server->tree->nodeExists($destination)) {
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,6 @@
 block discarded – undo
23 23
 namespace OCA\DAV\Upload;
24 24
 
25 25
 
26
-use OCA\DAV\Connector\Sabre\File;
27 26
 use Sabre\DAV\Exception\BadRequest;
28 27
 use Sabre\DAV\Server;
29 28
 use Sabre\DAV\ServerPlugin;
Please login to merge, or discard this patch.
lib/private/L10N/L10N.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -171,7 +171,7 @@
 block discarded – undo
171 171
 	 * Returns an associative array with all translations
172 172
 	 *
173 173
 	 * Called by String
174
-	 * @return array
174
+	 * @return string[]
175 175
 	 */
176 176
 	public function getTranslations() {
177 177
 		return $this->translations;
Please login to merge, or discard this patch.
lib/public/Template.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -100,8 +100,8 @@
 block discarded – undo
100 100
 /**
101 101
  * Return the relative date in relation to today. Returns something like "last hour" or "two month ago"
102 102
  * @param int $timestamp unix timestamp
103
- * @param boolean $dateOnly
104
- * @return \OC\L10N\L10NString human readable interpretation of the timestamp
103
+ * @param integer $dateOnly
104
+ * @return string human readable interpretation of the timestamp
105 105
  *
106 106
  * @deprecated 8.0.0 Use \OCP\Template::relative_modified_date() instead
107 107
  */
Please login to merge, or discard this patch.
apps/dav/lib/Connector/Sabre/File.php 1 patch
Doc Comments   +7 added lines, -1 removed lines patch added patch discarded remove patch
@@ -248,6 +248,9 @@  discard block
 block discarded – undo
248 248
 		return '"' . $this->info->getEtag() . '"';
249 249
 	}
250 250
 
251
+	/**
252
+	 * @param string $path
253
+	 */
251 254
 	private function getPartFileBasePath($path) {
252 255
 		$partFileInStorage = \OC::$server->getConfig()->getSystemValue('part_file_in_storage', true);
253 256
 		if ($partFileInStorage) {
@@ -617,7 +620,7 @@  discard block
 block discarded – undo
617 620
 	/**
618 621
 	 * Set $algo to get a specific checksum, leave null to get all checksums
619 622
 	 * (space seperated)
620
-	 * @param null $algo
623
+	 * @param string $algo
621 624
 	 * @return string
622 625
 	 */
623 626
 	public function getChecksum($algo = null) {
@@ -640,6 +643,9 @@  discard block
 block discarded – undo
640 643
 		return '';
641 644
 	}
642 645
 
646
+	/**
647
+	 * @param string $string
648
+	 */
643 649
 	protected function header($string) {
644 650
 		\header($string);
645 651
 	}
Please login to merge, or discard this patch.
apps/files/lib/Command/TransferOwnership.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,6 @@
 block discarded – undo
28 28
 use OC\Encryption\Manager;
29 29
 use OC\Files\CustomView;
30 30
 use OC\Files\Filesystem;
31
-use OC\Files\Storage\Wrapper\Wrapper;
32 31
 use OC\Files\View;
33 32
 use OC\Memcache\ArrayCache;
34 33
 use OCP\Files\FileInfo;
Please login to merge, or discard this patch.
lib/private/Files/CustomView.php 2 patches
Doc Comments   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
 	 * and does not take the chroot into account )
140 140
 	 *
141 141
 	 * @param string $path
142
-	 * @return \OCP\Files\Mount\IMountPoint
142
+	 * @return Mount\MountPoint|null
143 143
 	 */
144 144
 	public function getMount($path) {
145 145
 		return Filesystem::getMountManager()->find($this->getAbsolutePath($path));
@@ -453,7 +453,7 @@  discard block
 block discarded – undo
453 453
 
454 454
 	/**
455 455
 	 * @param string $path
456
-	 * @return bool|mixed
456
+	 * @return string|boolean
457 457
 	 */
458 458
 	public function file_exists($path) {
459 459
 		if ($path == '/') {
@@ -910,7 +910,7 @@  discard block
 block discarded – undo
910 910
 
911 911
 	/**
912 912
 	 * @param string $path
913
-	 * @return bool|string
913
+	 * @return string|false
914 914
 	 * @throws \OCP\Files\InvalidPathException
915 915
 	 */
916 916
 	public function toTmpFile($path) {
@@ -1028,7 +1028,7 @@  discard block
 block discarded – undo
1028 1028
 	 * @param string $path
1029 1029
 	 * @param array $hooks (optional)
1030 1030
 	 * @param mixed $extraParam (optional)
1031
-	 * @return mixed
1031
+	 * @return string
1032 1032
 	 * @throws \Exception
1033 1033
 	 *
1034 1034
 	 * This method takes requests for basic filesystem functions (e.g. reading & writing
Please login to merge, or discard this patch.
Unused Use Statements   -3 removed lines patch added patch discarded remove patch
@@ -21,12 +21,9 @@
 block discarded – undo
21 21
 
22 22
 namespace  OC\Files;
23 23
 
24
-use OC\Encryption\CustomEncryptionWrapper;
25
-use OC\Encryption\Manager;
26 24
 use OC\Files\Cache;
27 25
 use OC\Files\Filesystem;
28 26
 use OC\Files\Storage\Storage;
29
-use OC\Memcache\ArrayCache;
30 27
 use OCP\Files\Mount\IMountPoint;
31 28
 use OCP\Lock\ILockingProvider;
32 29
 use OC\Files\Mount\MoveableMount;
Please login to merge, or discard this patch.
lib/private/Files/Storage/Wrapper/CustomEncryption.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -36,7 +36,6 @@
 block discarded – undo
36 36
 use OCP\Files\Mount\IMountPoint;
37 37
 use OCP\Files\Storage;
38 38
 use OCP\ILogger;
39
-use OCP\Files\Cache\ICacheEntry;
40 39
 
41 40
 class CustomEncryption extends Wrapper {
42 41
 
Please login to merge, or discard this patch.
Doc Comments   +7 added lines, -4 removed lines patch added patch discarded remove patch
@@ -98,7 +98,6 @@  discard block
 block discarded – undo
98 98
 	 * @param Update $update
99 99
 	 * @param Manager $mountManager
100 100
 	 * @param ArrayCache $arrayCache
101
-	 * @param Encryption $encryptionWrapper
102 101
 	 */
103 102
 	public function __construct(
104 103
 		$parameters,
@@ -385,7 +384,7 @@  discard block
 block discarded – undo
385 384
 	 * @param string $path
386 385
 	 * @param string $mode
387 386
 	 * @param string|null $sourceFileOfRename
388
-	 * @return resource|bool
387
+	 * @return resource
389 388
 	 * @throws GenericEncryptionException
390 389
 	 * @throws ModuleDoesNotExistsException
391 390
 	 */
@@ -514,6 +513,10 @@  discard block
 block discarded – undo
514 513
 		return $this->storage->fopen($path, $mode);
515 514
 	}
516 515
 
516
+	/**
517
+	 * @param string $path
518
+	 * @param string $mode
519
+	 */
517 520
 	public function fopenStorage($path, $mode) {
518 521
 		return fopen($this->storage->getSourcePath($path), $mode);
519 522
 	}
@@ -529,7 +532,7 @@  discard block
 block discarded – undo
529 532
 	 * @param string $uid
530 533
 	 * @param \OCP\Encryption\IEncryptionModule $encryptionModule
531 534
 	 * @param \OC\Files\Storage\Storage $storage
532
-	 * @param \OC\Files\Storage\Wrapper\Encryption $encStorage
535
+	 * @param CustomEncryption $encStorage
533 536
 	 * @param \OC\Encryption\Util $util
534 537
 	 * @param \OC\Encryption\File $file
535 538
 	 * @param string $mode
@@ -1139,7 +1142,7 @@  discard block
 block discarded – undo
1139 1142
 	/**
1140 1143
 	 * check if path points to a files version
1141 1144
 	 *
1142
-	 * @param $path
1145
+	 * @param string $path
1143 1146
 	 * @return bool
1144 1147
 	 */
1145 1148
 	protected function isVersion($path) {
Please login to merge, or discard this patch.
lib/private/Files/Storage/Wrapper/Encryption.php 1 patch
Doc Comments   +1 added lines, -3 removed lines patch added patch discarded remove patch
@@ -628,7 +628,6 @@  discard block
 block discarded – undo
628 628
 	 * @param string $sourceInternalPath
629 629
 	 * @param string $targetInternalPath
630 630
 	 * @param bool $preserveMtime
631
-	 * @param bool $isRename
632 631
 	 * @return bool
633 632
 	 */
634 633
 	public function copyFromStorage(Storage $sourceStorage, $sourceInternalPath, $targetInternalPath, $preserveMtime = false) {
@@ -692,7 +691,6 @@  discard block
 block discarded – undo
692 691
 	 * @param string $sourceInternalPath
693 692
 	 * @param string $targetInternalPath
694 693
 	 * @param bool $preserveMtime
695
-	 * @param bool $isRename
696 694
 	 * @return bool
697 695
 	 * @throws \Exception
698 696
 	 */
@@ -999,7 +997,7 @@  discard block
 block discarded – undo
999 997
 	/**
1000 998
 	 * check if path points to a files version
1001 999
 	 *
1002
-	 * @param $path
1000
+	 * @param string $path
1003 1001
 	 * @return bool
1004 1002
 	 */
1005 1003
 	protected function isVersion($path) {
Please login to merge, or discard this patch.