Completed
Pull Request — stable7 (#24167)
by Lukas
07:55
created
apps/files_sharing/lib/cache.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -98,7 +98,7 @@
 block discarded – undo
98 98
 	/**
99 99
 	 * get the stored metadata of a file or folder
100 100
 	 *
101
-	 * @param string|int $file
101
+	 * @param string $file
102 102
 	 * @return array|false
103 103
 	 */
104 104
 	public function get($file) {
Please login to merge, or discard this patch.
apps/files_sharing/tests/share.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -218,6 +218,10 @@
 block discarded – undo
218 218
 
219 219
 	}
220 220
 
221
+	/**
222
+	 * @param OC\Files\FileInfo[] $content
223
+	 * @param string[] $expected
224
+	 */
221 225
 	public function verifyDirContent($content, $expected) {
222 226
 		foreach ($content as $c) {
223 227
 			if (!in_array($c['name'], $expected)) {
Please login to merge, or discard this patch.
apps/files_trashbin/tests/trashbin.php 1 patch
Doc Comments   +16 added lines, -3 removed lines patch added patch discarded remove patch
@@ -246,8 +246,8 @@  discard block
 block discarded – undo
246 246
 
247 247
 	/**
248 248
 	 * verify that the array contains the expected results
249
-	 * @param array $result
250
-	 * @param array $expected
249
+	 * @param OCP\Files\FileInfo[] $result
250
+	 * @param string[] $expected
251 251
 	 */
252 252
 	private function verifyArray($result, $expected) {
253 253
 		$this->assertSame(count($expected), count($result));
@@ -266,6 +266,11 @@  discard block
 block discarded – undo
266 266
 		}
267 267
 	}
268 268
 
269
+	/**
270
+	 * @param OCP\Files\FileInfo[] $files
271
+	 * @param string $trashRoot
272
+	 * @param integer $expireDate
273
+	 */
269 274
 	private function manipulateDeleteTime($files, $trashRoot, $expireDate) {
270 275
 		$counter = 0;
271 276
 		foreach ($files as &$file) {
@@ -625,7 +630,6 @@  discard block
 block discarded – undo
625 630
 	/**
626 631
 	 * @param string $user
627 632
 	 * @param bool $create
628
-	 * @param bool $password
629 633
 	 */
630 634
 	public static function loginHelper($user, $create = false) {
631 635
 		if ($create) {
@@ -648,11 +652,20 @@  discard block
 block discarded – undo
648 652
 
649 653
 // just a dummy class to make protected methods available for testing
650 654
 class TrashbinForTesting extends Files_Trashbin\Trashbin {
655
+
656
+	/**
657
+	 * @param OCP\Files\FileInfo[] $files
658
+	 * @param integer $limit
659
+	 */
651 660
 	public function dummyDeleteExpiredFiles($files, $limit) {
652 661
 		// dummy value for $retention_obligation because it is not needed here
653 662
 		return parent::deleteExpiredFiles($files, \Test_Trashbin::TEST_TRASHBIN_USER1, $limit, 0);
654 663
 	}
655 664
 
665
+	/**
666
+	 * @param OCP\Files\FileInfo[] $files
667
+	 * @param integer $availableSpace
668
+	 */
656 669
 	public function dummyDeleteFiles($files, $availableSpace) {
657 670
 		return parent::deleteFiles($files, \Test_Trashbin::TEST_TRASHBIN_USER1, $availableSpace);
658 671
 	}
Please login to merge, or discard this patch.
apps/user_ldap/command/setconfig.php 2 patches
Doc Comments   -2 removed lines patch added patch discarded remove patch
@@ -73,8 +73,6 @@
 block discarded – undo
73 73
 	/**
74 74
 	 * save the configuration value as provided
75 75
 	 * @param string $configID
76
-	 * @param string $configKey
77
-	 * @param string $configValue
78 76
 	 */
79 77
 	protected function setValue($configID, $key, $value) {
80 78
 		$configHolder = new Configuration($configID);
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,6 @@
 block discarded – undo
28 28
 use Symfony\Component\Console\Command\Command;
29 29
 use Symfony\Component\Console\Helper\Table;
30 30
 use Symfony\Component\Console\Input\InputInterface;
31
-use Symfony\Component\Console\Input\InputOption;
32 31
 use Symfony\Component\Console\Output\OutputInterface;
33 32
 
34 33
 class ListCertificates extends Base {
Please login to merge, or discard this patch.
apps/user_ldap/lib/jobs.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -68,7 +68,7 @@
 block discarded – undo
68 68
 	}
69 69
 
70 70
 	/**
71
-	 * @return int
71
+	 * @return string
72 72
 	 */
73 73
 	static private function getRefreshInterval() {
74 74
 		//defaults to every hour
Please login to merge, or discard this patch.
apps/user_ldap/lib/wizard.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -864,7 +864,7 @@  discard block
 block discarded – undo
864 864
 	 * @param integer $filterType int, for which use case the filter shall be created
865 865
 	 * can be any of self::LFILTER_USER_LIST, self::LFILTER_LOGIN or
866 866
 	 * self::LFILTER_GROUP_LIST
867
-	 * @return string|false string with the filter on success, false otherwise
867
+	 * @return string string with the filter on success, false otherwise
868 868
 	 * @throws \Exception
869 869
 	 */
870 870
 	private function composeLdapFilter($filterType) {
@@ -1107,7 +1107,7 @@  discard block
 block discarded – undo
1107 1107
 	}
1108 1108
 
1109 1109
 	/**
1110
-	 * @param array $reqs
1110
+	 * @param string[] $reqs
1111 1111
 	 * @return bool
1112 1112
 	 */
1113 1113
 	private function checkRequirements($reqs) {
Please login to merge, or discard this patch.
lib/private/files/stream/staticstream.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -144,6 +144,9 @@
 block discarded – undo
144 144
 		return true;
145 145
 	}
146 146
 
147
+	/**
148
+	 * @param string $path
149
+	 */
147 150
 	public function url_stat($path) {
148 151
 		if (isset(self::$data[$path])) {
149 152
 			$size = strlen(self::$data[$path]);
Please login to merge, or discard this patch.
lib/private/group/dummy.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -106,7 +106,7 @@
 block discarded – undo
106 106
 	 * Removes a user from a group
107 107
 	 * @param string $uid Name of the user to remove from group
108 108
 	 * @param string $gid Name of the group from which remove the user
109
-	 * @return bool
109
+	 * @return null|false
110 110
 	 *
111 111
 	 * removes the user from a group.
112 112
 	 */
Please login to merge, or discard this patch.
lib/private/group/manager.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -144,6 +144,9 @@
 block discarded – undo
144 144
 		return $this->getGroupObject($gid);
145 145
 	}
146 146
 
147
+	/**
148
+	 * @param string $gid
149
+	 */
147 150
 	protected function getGroupObject($gid) {
148 151
 		$backends = array();
149 152
 		foreach ($this->backends as $backend) {
Please login to merge, or discard this patch.