Completed
Pull Request — master (#26782)
by Victor
10:16
created
lib/private/legacy/app.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
 
165 165
 	/**
166 166
 	 * Enables the app as a theme if it has the type "theme"
167
-	 * @param $app
167
+	 * @param string $app
168 168
 	 */
169 169
 	private static function enableTheme($app) {
170 170
 		if (self::isType($app, 'theme')) {
@@ -1365,8 +1365,8 @@  discard block
 block discarded – undo
1365 1365
 	}
1366 1366
 
1367 1367
 	/**
1368
-	 * @param $config
1369
-	 * @param $l
1368
+	 * @param OCP\IConfig $config
1369
+	 * @param OCP\IL10N $l
1370 1370
 	 * @param $info
1371 1371
 	 * @throws Exception
1372 1372
 	 */
Please login to merge, or discard this patch.
apps/dav/lib/CardDAV/ImageExportPlugin.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -136,6 +136,9 @@
 block discarded – undo
136 136
 		return false;
137 137
 	}
138 138
 
139
+	/**
140
+	 * @param string $cardData
141
+	 */
139 142
 	private function readCard($cardData) {
140 143
 		return Reader::read($cardData);
141 144
 	}
Please login to merge, or discard this patch.
lib/private/L10N/Factory.php 1 patch
Doc Comments   +5 added lines patch added patch discarded remove patch
@@ -278,6 +278,11 @@
 block discarded – undo
278 278
 	 */
279 279
 	// FIXME This method is only public, until \OCP\IL10N does not need it anymore,
280 280
 	// FIXME This is also the reason, why it is not in the public interface
281
+
282
+	/**
283
+	 * @param string $app
284
+	 * @param string $lang
285
+	 */
281 286
 	public function getL10nFilesForApp($app, $lang) {
282 287
 		$languageFiles = [];
283 288
 
Please login to merge, or discard this patch.
apps/dav/lib/Connector/Sabre/FilesReportPlugin.php 1 patch
Doc Comments   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
 	 * This will be used in the {DAV:}supported-report-set property.
149 149
 	 *
150 150
 	 * @param string $uri
151
-	 * @return array
151
+	 * @return string[]
152 152
 	 */
153 153
 	public function getSupportedReportSet($uri) {
154 154
 		return [self::REPORT_NAME];
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
 	 * @param string $reportName
161 161
 	 * @param $report
162 162
 	 * @param string $uri
163
-	 * @return bool
163
+	 * @return null|false
164 164
 	 * @throws BadRequest
165 165
 	 * @throws PreconditionFailed
166 166
 	 * @internal param $ [] $report
@@ -333,6 +333,7 @@  discard block
 block discarded – undo
333 333
 	 * with a leading slash but no trailing slash
334 334
 	 * @param string[] $requestedProps requested properties
335 335
 	 * @param Node[] nodes nodes for which to fetch and prepare responses
336
+	 * @param Node[] $nodes
336 337
 	 * @return Response[]
337 338
 	 */
338 339
 	public function prepareResponses($filesUri, $requestedProps, $nodes) {
Please login to merge, or discard this patch.
apps/files_external/lib/Lib/Backend/Google.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -22,7 +22,6 @@
 block discarded – undo
22 22
 namespace OCA\Files_External\Lib\Backend;
23 23
 
24 24
 use OCP\IL10N;
25
-use OCP\Files\External\DefinitionParameter;
26 25
 use OCP\Files\External\Auth\AuthMechanism;
27 26
 use OCP\Files\External\Backend\Backend;
28 27
 use OCA\Files_External\Lib\LegacyDependencyCheckPolyfill;
Please login to merge, or discard this patch.
apps/files_external/lib/Lib/Backend/Dropbox.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -22,7 +22,6 @@
 block discarded – undo
22 22
 namespace OCA\Files_External\Lib\Backend;
23 23
 
24 24
 use OCP\IL10N;
25
-use OCP\Files\External\DefinitionParameter;
26 25
 use OCP\Files\External\Auth\AuthMechanism;
27 26
 use OCP\Files\External\Backend\Backend;
28 27
 use OCA\Files_External\Lib\LegacyDependencyCheckPolyfill;
Please login to merge, or discard this patch.
lib/private/App/AppManager.php 1 patch
Doc Comments   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 	/**
121 121
 	 * List all installed apps
122 122
 	 *
123
-	 * @return string[]
123
+	 * @return integer[]
124 124
 	 */
125 125
 	public function getInstalledApps() {
126 126
 		return array_keys($this->getInstalledAppsValues());
@@ -347,6 +347,9 @@  discard block
 block discarded – undo
347 347
 		return in_array($appId, $this->shippedApps);
348 348
 	}
349 349
 
350
+	/**
351
+	 * @param string $appId
352
+	 */
350 353
 	private function isAlwaysEnabled($appId) {
351 354
 		$alwaysEnabled = $this->getAlwaysEnabledApps();
352 355
 		return in_array($appId, $alwaysEnabled);
@@ -374,7 +377,7 @@  discard block
 block discarded – undo
374 377
 
375 378
 	/**
376 379
 	 * @param string $package
377
-	 * @return mixed
380
+	 * @return integer
378 381
 	 * @since 9.2.0
379 382
 	 */
380 383
 	public function installApp($package) {
@@ -386,7 +389,7 @@  discard block
 block discarded – undo
386 389
 
387 390
 	/**
388 391
 	 * @param string $package
389
-	 * @return mixed
392
+	 * @return boolean
390 393
 	 * @since 9.2.0
391 394
 	 */
392 395
 	public function updateApp($package) {
Please login to merge, or discard this patch.
apps/files_sharing/lib/SharedMount.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 	 *
111 111
 	 * @param string $newPath
112 112
 	 * @param \OCP\Share\IShare $share
113
-	 * @return bool
113
+	 * @return boolean|null
114 114
 	 */
115 115
 	private function updateFileTarget($newPath, &$share) {
116 116
 		$share->setTarget($newPath);
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
 	 * @param string $path
127 127
 	 * @param View $view
128 128
 	 * @param SharedMount[] $mountpoints
129
-	 * @return mixed
129
+	 * @return string
130 130
 	 */
131 131
 	private function generateUniqueTarget($path, $view, array $mountpoints) {
132 132
 		$pathinfo = pathinfo($path);
Please login to merge, or discard this patch.
lib/private/Files/Node/File.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
32 32
 	 * Creates a Folder that represents a non-existing path
33 33
 	 *
34 34
 	 * @param string $path path
35
-	 * @return string non-existing node class
35
+	 * @return NonExistingFile non-existing node class
36 36
 	 */
37 37
 	protected function createNonExistingNode($path) {
38 38
 		return new NonExistingFile($this->root, $this->view, $path);
Please login to merge, or discard this patch.