Completed
Pull Request — master (#23063)
by Jan-Christoph
13:45
created
apps/dav/lib/caldav/caldavbackend.php 1 patch
Doc Comments   +8 added lines, -2 removed lines patch added patch discarded remove patch
@@ -286,6 +286,9 @@  discard block
 block discarded – undo
286 286
 		return $calendar;
287 287
 	}
288 288
 
289
+	/**
290
+	 * @param integer $calendarId
291
+	 */
289 292
 	public function getCalendarById($calendarId) {
290 293
 		$fields = array_values($this->propertyMap);
291 294
 		$fields[] = 'id';
@@ -594,7 +597,7 @@  discard block
 block discarded – undo
594 597
 	 * calendar-data. If the result of a subsequent GET to this object is not
595 598
 	 * the exact same as this request body, you should omit the ETag.
596 599
 	 *
597
-	 * @param mixed $calendarId
600
+	 * @param integer $calendarId
598 601
 	 * @param string $objectUri
599 602
 	 * @param string $calendarData
600 603
 	 * @return string
@@ -1033,7 +1036,7 @@  discard block
 block discarded – undo
1033 1036
 	 * @param string $principalUri
1034 1037
 	 * @param string $uri
1035 1038
 	 * @param array $properties
1036
-	 * @return mixed
1039
+	 * @return integer
1037 1040
 	 */
1038 1041
 	function createSubscription($principalUri, $uri, array $properties) {
1039 1042
 
@@ -1374,6 +1377,9 @@  discard block
 block discarded – undo
1374 1377
 		return $this->sharingBackend->applyShareAcl($resourceId, $acl);
1375 1378
 	}
1376 1379
 
1380
+	/**
1381
+	 * @param boolean $toV2
1382
+	 */
1377 1383
 	private function convertPrincipal($principalUri, $toV2) {
1378 1384
 		if ($this->principalBackend->getPrincipalPrefix() === 'principals') {
1379 1385
 			list(, $name) = URLUtil::splitPath($principalUri);
Please login to merge, or discard this patch.
apps/dav/lib/connector/legacydavacl.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -59,6 +59,9 @@
 block discarded – undo
59 59
 		);
60 60
 	}
61 61
 
62
+	/**
63
+	 * @param boolean $toV2
64
+	 */
62 65
 	private function convertPrincipal($principal, $toV2) {
63 66
 		list(, $name) = URLUtil::splitPath($principal);
64 67
 		if ($toV2) {
Please login to merge, or discard this patch.
apps/dav/lib/connector/sabre/file.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.
apps/dav/lib/connector/sabre/davaclplugin.php 1 patch
Unused Use Statements   -3 removed lines patch added patch discarded remove patch
@@ -22,14 +22,11 @@
 block discarded – undo
22 22
 namespace OCA\DAV\Connector\Sabre;
23 23
 
24 24
 use Sabre\DAV\Exception\NotFound;
25
-use Sabre\DAV\IFile;
26 25
 use Sabre\DAV\INode;
27 26
 use \Sabre\DAV\PropFind;
28 27
 use \Sabre\DAV\PropPatch;
29
-use Sabre\DAVACL\Exception\NeedPrivileges;
30 28
 use \Sabre\HTTP\RequestInterface;
31 29
 use \Sabre\HTTP\ResponseInterface;
32
-use Sabre\HTTP\URLUtil;
33 30
 
34 31
 /**
35 32
  * Class DavAclPlugin is a wrapper around \Sabre\DAVACL\Plugin that returns 404
Please login to merge, or discard this patch.
lib/private/Server.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -856,7 +856,7 @@
 block discarded – undo
856 856
 	 * Get the certificate manager for the user
857 857
 	 *
858 858
 	 * @param string $userId (optional) if not specified the current loggedin user is used
859
-	 * @return \OCP\ICertificateManager | null if $uid is null and no user is logged in
859
+	 * @return null|CertificateManager | null if $uid is null and no user is logged in
860 860
 	 */
861 861
 	public function getCertificateManager($userId = null) {
862 862
 		if (is_null($userId)) {
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 ShareTypeList
64 64
 	 */
65 65
 	static function xmlDeserialize(Reader $reader) {
66 66
 		$shareTypes = [];
Please login to merge, or discard this patch.
apps/files/controller/apicontroller.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -35,7 +35,6 @@
 block discarded – undo
35 35
 use OCA\Files\Service\TagService;
36 36
 use OCP\IPreview;
37 37
 use OCP\Share\IManager;
38
-use OCP\Files\FileInfo;
39 38
 use OCP\Files\Node;
40 39
 use OCP\IUserSession;
41 40
 
Please login to merge, or discard this patch.
apps/files/service/tagservice.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,6 @@
 block discarded – undo
24 24
 
25 25
 namespace OCA\Files\Service;
26 26
 
27
-use OC\Files\FileInfo;
28 27
 use OCP\Files\Node;
29 28
 
30 29
 /**
Please login to merge, or discard this patch.
apps/dav/lib/carddav/carddavbackend.php 1 patch
Doc Comments   +5 added lines, -1 removed lines patch added patch discarded remove patch
@@ -866,6 +866,7 @@  discard block
 block discarded – undo
866 866
 	 *   * readOnly - boolean
867 867
 	 *   * summary - Optional, a description for the share
868 868
 	 *
869
+	 * @param integer $addressBookId
869 870
 	 * @return array
870 871
 	 */
871 872
 	public function getShares($addressBookId) {
@@ -965,7 +966,7 @@  discard block
 block discarded – undo
965 966
 
966 967
 	/**
967 968
 	 * For shared address books the sharee is set in the ACL of the address book
968
-	 * @param $addressBookId
969
+	 * @param integer $addressBookId
969 970
 	 * @param $acl
970 971
 	 * @return array
971 972
 	 */
@@ -973,6 +974,9 @@  discard block
 block discarded – undo
973 974
 		return $this->sharingBackend->applyShareAcl($addressBookId, $acl);
974 975
 	}
975 976
 
977
+	/**
978
+	 * @param boolean $toV2
979
+	 */
976 980
 	private function convertPrincipal($principalUri, $toV2) {
977 981
 		if ($this->principalBackend->getPrincipalPrefix() === 'principals') {
978 982
 			list(, $name) = URLUtil::splitPath($principalUri);
Please login to merge, or discard this patch.