Completed
Pull Request — master (#26361)
by Jörn Friedrich
57:44
created
core/Controller/OccController.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -131,8 +131,8 @@
 block discarded – undo
131 131
 
132 132
 	/**
133 133
 	 * Check if command is allowed and has a valid security token
134
-	 * @param $command
135
-	 * @param $token
134
+	 * @param string $command
135
+	 * @param string $token
136 136
 	 */
137 137
 	protected function validateRequest($command, $token){
138 138
 		$allowedHosts = ['::1', '127.0.0.1', 'localhost'];
Please login to merge, or discard this patch.
apps/dav/lib/Connector/Sabre/ShareTypeList.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@
 block discarded – undo
60 60
 	 * The deserialize method is called during xml parsing.
61 61
 	 *
62 62
 	 * @param Reader $reader
63
-	 * @return mixed
63
+	 * @return null|ShareTypeList
64 64
 	 */
65 65
 	static function xmlDeserialize(Reader $reader) {
66 66
 		$shareTypes = [];
Please login to merge, or discard this patch.
apps/dav/lib/Connector/Sabre/TagList.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@
 block discarded – undo
79 79
 	 * the next element.
80 80
 	 *
81 81
 	 * @param Reader $reader
82
-	 * @return mixed
82
+	 * @return null|TagList
83 83
 	 */
84 84
 	static function xmlDeserialize(Reader $reader) {
85 85
 		$tags = [];
Please login to merge, or discard this patch.
core/Application.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,6 @@
 block discarded – undo
32 32
 use OC\Core\Controller\AvatarController;
33 33
 use OC\Core\Controller\LoginController;
34 34
 use OC\Core\Controller\LostController;
35
-use OC\Core\Controller\OccController;
36 35
 use OC\Core\Controller\TokenController;
37 36
 use OC\Core\Controller\TwoFactorChallengeController;
38 37
 use OC\Core\Controller\UserController;
Please login to merge, or discard this patch.
apps/files_sharing/lib/Controller/ShareesController.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -327,7 +327,7 @@
 block discarded – undo
327 327
 	 * split user and remote from federated cloud id
328 328
 	 *
329 329
 	 * @param string $address federated share address
330
-	 * @return array [user, remoteURL]
330
+	 * @return string[] [user, remoteURL]
331 331
 	 * @throws \Exception
332 332
 	 */
333 333
 	public function splitUserRemote($address) {
Please login to merge, or discard this patch.
apps/federation/lib/Controller/OCSAuthAPIController.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -164,6 +164,10 @@
 block discarded – undo
164 164
 			'data' => ['sharedSecret' => $sharedSecret]];
165 165
 	}
166 166
 
167
+	/**
168
+	 * @param string $url
169
+	 * @param string $token
170
+	 */
167 171
 	protected function isValidToken($url, $token) {
168 172
 		$storedToken = $this->dbHandler->getToken($url);
169 173
 		return hash_equals($storedToken, $token);
Please login to merge, or discard this patch.
apps/dav/lib/CalDAV/CalDavBackend.php 1 patch
Doc Comments   +7 added lines, -1 removed lines patch added patch discarded remove patch
@@ -262,6 +262,9 @@  discard block
 block discarded – undo
262 262
 		return array_values($calendars);
263 263
 	}
264 264
 
265
+	/**
266
+	 * @param string $principalUri
267
+	 */
265 268
 	public function getUsersOwnCalendars($principalUri) {
266 269
 		$principalUri = $this->convertPrincipal($principalUri, true);
267 270
 		$fields = array_values($this->propertyMap);
@@ -1254,7 +1257,7 @@  discard block
 block discarded – undo
1254 1257
 	 * @param string $principalUri
1255 1258
 	 * @param string $uri
1256 1259
 	 * @param array $properties
1257
-	 * @return mixed
1260
+	 * @return integer
1258 1261
 	 */
1259 1262
 	function createSubscription($principalUri, $uri, array $properties) {
1260 1263
 
@@ -1657,6 +1660,9 @@  discard block
 block discarded – undo
1657 1660
 		return $this->sharingBackend->applyShareAcl($resourceId, $acl);
1658 1661
 	}
1659 1662
 
1663
+	/**
1664
+	 * @param boolean $toV2
1665
+	 */
1660 1666
 	private function convertPrincipal($principalUri, $toV2) {
1661 1667
 		if ($this->principalBackend->getPrincipalPrefix() === 'principals') {
1662 1668
 			list(, $name) = URLUtil::splitPath($principalUri);
Please login to merge, or discard this patch.
apps/dav/lib/Connector/Sabre/FilesReportPlugin.php 1 patch
Doc Comments   +4 added lines, -3 removed lines patch added patch discarded remove patch
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
 	 * @param ITagManager $fileTagger manager for private tags
105 105
 	 * @param IUserSession $userSession
106 106
 	 * @param IGroupManager $groupManager
107
-	 * @param Folder $userfolder
107
+	 * @param Folder $userFolder
108 108
 	 */
109 109
 	public function __construct(Tree $tree,
110 110
 								View $view,
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
 	 * This will be used in the {DAV:}supported-report-set property.
151 151
 	 *
152 152
 	 * @param string $uri
153
-	 * @return array
153
+	 * @return string[]
154 154
 	 */
155 155
 	public function getSupportedReportSet($uri) {
156 156
 		return [self::REPORT_NAME];
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
 	 * @param string $reportName
163 163
 	 * @param [] $report
164 164
 	 * @param string $uri
165
-	 * @return bool
165
+	 * @return null|false
166 166
 	 * @throws NotFound
167 167
 	 * @throws ReportNotSupported
168 168
 	 */
@@ -335,6 +335,7 @@  discard block
 block discarded – undo
335 335
 	 * with a leading slash but no trailing slash
336 336
 	 * @param string[] $requestedProps requested properties
337 337
 	 * @param Node[] nodes nodes for which to fetch and prepare responses
338
+	 * @param Node[] $nodes
338 339
 	 * @return Response[]
339 340
 	 */
340 341
 	public function prepareResponses($filesUri, $requestedProps, $nodes) {
Please login to merge, or discard this patch.
lib/private/AvatarManager.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -104,7 +104,7 @@
 block discarded – undo
104 104
 	/**
105 105
 	 * Returns the avatar folder for the given user
106 106
 	 *
107
-	 * @param $userId user id
107
+	 * @param string $userId user id
108 108
 	 * @return Folder|\OCP\Files\Node
109 109
 	 *
110 110
 	 * @internal
Please login to merge, or discard this patch.