Completed
Pull Request — master (#1279)
by Christoph
11:06
created
apps/dav/lib/HookManager.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,6 @@
 block discarded – undo
21 21
  */
22 22
 namespace OCA\DAV;
23 23
 
24
-use OCA\DAV\CalDAV\BirthdayService;
25 24
 use OCA\DAV\CalDAV\CalDavBackend;
26 25
 use OCA\DAV\CardDAV\CardDavBackend;
27 26
 use OCA\DAV\CardDAV\SyncService;
Please login to merge, or discard this patch.
apps/dav/lib/CalDAV/CalDavBackend.php 1 patch
Doc Comments   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
 	 *
138 138
 	 * By default this excludes the automatically generated birthday calendar
139 139
 	 *
140
-	 * @param $principalUri
140
+	 * @param string $principalUri
141 141
 	 * @param bool $excludeBirthday
142 142
 	 * @return int
143 143
 	 */
@@ -1114,7 +1114,7 @@  discard block
 block discarded – undo
1114 1114
 	 * @param string $principalUri
1115 1115
 	 * @param string $uri
1116 1116
 	 * @param array $properties
1117
-	 * @return mixed
1117
+	 * @return integer
1118 1118
 	 */
1119 1119
 	function createSubscription($principalUri, $uri, array $properties) {
1120 1120
 
@@ -1473,6 +1473,9 @@  discard block
 block discarded – undo
1473 1473
 		return $this->sharingBackend->applyShareAcl($resourceId, $acl);
1474 1474
 	}
1475 1475
 
1476
+	/**
1477
+	 * @param boolean $toV2
1478
+	 */
1476 1479
 	private function convertPrincipal($principalUri, $toV2) {
1477 1480
 		if ($this->principalBackend->getPrincipalPrefix() === 'principals') {
1478 1481
 			list(, $name) = URLUtil::splitPath($principalUri);
Please login to merge, or discard this patch.
apps/dav/lib/CardDAV/CardDavBackend.php 1 patch
Doc Comments   +8 added lines, -3 removed lines patch added patch discarded remove patch
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
 	/**
102 102
 	 * Return the number of address books for a principal
103 103
 	 *
104
-	 * @param $principalUri
104
+	 * @param string $principalUri
105 105
 	 * @return int
106 106
 	 */
107 107
 	public function getAddressBooksForUserCount($principalUri) {
@@ -234,7 +234,8 @@  discard block
 block discarded – undo
234 234
 	}
235 235
 
236 236
 	/**
237
-	 * @param $addressBookUri
237
+	 * @param string $addressBookUri
238
+	 * @param string $principal
238 239
 	 * @return array|null
239 240
 	 */
240 241
 	public function getAddressBooksByUri($principal, $addressBookUri) {
@@ -916,6 +917,7 @@  discard block
 block discarded – undo
916 917
 	 *   * readOnly - boolean
917 918
 	 *   * summary - Optional, a description for the share
918 919
 	 *
920
+	 * @param integer $addressBookId
919 921
 	 * @return array
920 922
 	 */
921 923
 	public function getShares($addressBookId) {
@@ -1015,7 +1017,7 @@  discard block
 block discarded – undo
1015 1017
 
1016 1018
 	/**
1017 1019
 	 * For shared address books the sharee is set in the ACL of the address book
1018
-	 * @param $addressBookId
1020
+	 * @param integer $addressBookId
1019 1021
 	 * @param $acl
1020 1022
 	 * @return array
1021 1023
 	 */
@@ -1023,6 +1025,9 @@  discard block
 block discarded – undo
1023 1025
 		return $this->sharingBackend->applyShareAcl($addressBookId, $acl);
1024 1026
 	}
1025 1027
 
1028
+	/**
1029
+	 * @param boolean $toV2
1030
+	 */
1026 1031
 	private function convertPrincipal($principalUri, $toV2) {
1027 1032
 		if ($this->principalBackend->getPrincipalPrefix() === 'principals') {
1028 1033
 			list(, $name) = URLUtil::splitPath($principalUri);
Please login to merge, or discard this patch.
lib/private/AppFramework/Http/Output.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@
 block discarded – undo
34 34
 	private $webRoot;
35 35
 
36 36
 	/**
37
-	 * @param $webRoot
37
+	 * @param string $webRoot
38 38
 	 */
39 39
 	public function __construct($webRoot) {
40 40
 		$this->webRoot = $webRoot;
Please login to merge, or discard this patch.
apps/federation/lib/AppInfo/Application.php 1 patch
Unused Use Statements   -3 removed lines patch added patch discarded remove patch
@@ -24,16 +24,13 @@
 block discarded – undo
24 24
 
25 25
 namespace OCA\Federation\AppInfo;
26 26
 
27
-use OCA\Federation\API\OCSAuthAPI;
28 27
 use OCA\Federation\Controller\SettingsController;
29 28
 use OCA\Federation\DAV\FedAuth;
30 29
 use OCA\Federation\DbHandler;
31 30
 use OCA\Federation\Hooks;
32 31
 use OCA\Federation\Middleware\AddServerMiddleware;
33 32
 use OCA\Federation\SyncFederationAddressBooks;
34
-use OCA\Federation\SyncJob;
35 33
 use OCA\Federation\TrustedServers;
36
-use OCP\API;
37 34
 use OCP\App;
38 35
 use OCP\AppFramework\IAppContainer;
39 36
 use OCP\SabrePluginEvent;
Please login to merge, or discard this patch.
lib/private/AppFramework/Middleware/OCSMiddleware.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,6 @@
 block discarded – undo
30 30
 use OCP\AppFramework\Http\Response;
31 31
 use OCP\AppFramework\OCS\OCSException;
32 32
 use OCP\AppFramework\OCS\OCSForbiddenException;
33
-use OCP\AppFramework\OCS\OCSNotFoundException;
34 33
 use OCP\AppFramework\OCSController;
35 34
 use OCP\IRequest;
36 35
 use OCP\AppFramework\Middleware;
Please login to merge, or discard this patch.