Completed
Pull Request — stable8.2 (#23025)
by Morris
12:11
created
lib/private/connector/sabre/principal.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -168,7 +168,7 @@
 block discarded – undo
168 168
 	 * The principals should be passed as a list of uri's.
169 169
 	 *
170 170
 	 * @param string $principal
171
-	 * @param array $members
171
+	 * @param string[] $members
172 172
 	 * @throws \Sabre\DAV\Exception
173 173
 	 */
174 174
 	public function setGroupMemberSet($principal, array $members) {
Please login to merge, or discard this patch.
lib/private/files/storage/wrapper/encryption.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -190,7 +190,7 @@  discard block
 block discarded – undo
190 190
 	 *
191 191
 	 * @param string $path
192 192
 	 * @param string $data
193
-	 * @return bool
193
+	 * @return integer
194 194
 	 */
195 195
 	public function file_put_contents($path, $data) {
196 196
 		// file put content will always be translated to a stream write
@@ -769,7 +769,7 @@  discard block
 block discarded – undo
769 769
 	/**
770 770
 	 * check if path points to a files version
771 771
 	 *
772
-	 * @param $path
772
+	 * @param string $path
773 773
 	 * @return bool
774 774
 	 */
775 775
 	protected function isVersion($path) {
Please login to merge, or discard this patch.
lib/private/files/view.php 1 patch
Doc Comments   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
 	 * and does not take the chroot into account )
190 190
 	 *
191 191
 	 * @param string $path
192
-	 * @return \OCP\Files\Mount\IMountPoint
192
+	 * @return Mount\MountPoint|null
193 193
 	 */
194 194
 	public function getMount($path) {
195 195
 		return Filesystem::getMountManager()->find($this->getAbsolutePath($path));
@@ -873,7 +873,7 @@  discard block
 block discarded – undo
873 873
 
874 874
 	/**
875 875
 	 * @param string $path
876
-	 * @return bool|string
876
+	 * @return string|false
877 877
 	 * @throws \OCP\Files\InvalidPathException
878 878
 	 */
879 879
 	public function toTmpFile($path) {
@@ -1593,6 +1593,9 @@  discard block
 block discarded – undo
1593 1593
 		return null;
1594 1594
 	}
1595 1595
 
1596
+	/**
1597
+	 * @param string $path
1598
+	 */
1596 1599
 	private function assertPathLength($path) {
1597 1600
 		$maxLen = min(PHP_MAXPATHLEN, 4000);
1598 1601
 		// Check for the string length - performed using isset() instead of strlen()
Please login to merge, or discard this patch.
lib/private/share/mailnotifications.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -174,7 +174,7 @@
 block discarded – undo
174 174
 	 * @param string $filename the shared file
175 175
 	 * @param string $link the public link
176 176
 	 * @param int $expiration expiration date (timestamp)
177
-	 * @return array $result of failed recipients
177
+	 * @return string[] $result of failed recipients
178 178
 	 */
179 179
 	public function sendLinkShareMail($recipient, $filename, $link, $expiration) {
180 180
 		$subject = (string)$this->l->t('%s shared »%s« with you', [$this->senderDisplayName, $filename]);
Please login to merge, or discard this patch.
lib/private/subadmin.php 1 patch
Doc Comments   +5 added lines patch added patch discarded remove patch
@@ -166,6 +166,11 @@
 block discarded – undo
166 166
 	/*
167 167
 	 * alias for self::isSubAdminofGroup()
168 168
 	 */
169
+
170
+	/**
171
+	 * @param string $subadmin
172
+	 * @param string $group
173
+	 */
169 174
 	public static function isGroupAccessible($subadmin, $group) {
170 175
 		return self::isSubAdminofGroup($subadmin, $group);
171 176
 	}
Please login to merge, or discard this patch.
tests/lib/connector/sabre/file.php 1 patch
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,6 +40,9 @@  discard block
 block discarded – undo
40 40
 		parent::tearDown();
41 41
 	}
42 42
 
43
+	/**
44
+	 * @return \OC\Files\Storage\Storage
45
+	 */
43 46
 	private function getMockStorage() {
44 47
 		$storage = $this->getMock('\OCP\Files\Storage');
45 48
 		$storage->expects($this->any())
@@ -243,7 +246,7 @@  discard block
 block discarded – undo
243 246
 	 * @param string $path path to put the file into
244 247
 	 * @param string $viewRoot root to use for the view
245 248
 	 *
246
-	 * @return result of the PUT operaiton which is usually the etag
249
+	 * @return string|null of the PUT operaiton which is usually the etag
247 250
 	 */
248 251
 	private function doPut($path, $viewRoot = null) {
249 252
 		$view = \OC\Files\Filesystem::getView();
Please login to merge, or discard this patch.
tests/lib/connector/sabre/filesplugin.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -55,6 +55,9 @@
 block discarded – undo
55 55
 		$this->plugin->initialize($this->server);
56 56
 	}
57 57
 
58
+	/**
59
+	 * @param string $class
60
+	 */
58 61
 	private function createTestNode($class) {
59 62
 		$node = $this->getMockBuilder($class)
60 63
 			->disableOriginalConstructor()
Please login to merge, or discard this patch.
tests/lib/connector/sabre/requesttest/auth.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
 	 *
42 42
 	 * @param \Sabre\DAV\Server $server
43 43
 	 * @param string $realm
44
-	 * @return bool
44
+	 * @return boolean|null
45 45
 	 */
46 46
 	function authenticate(\Sabre\DAV\Server $server, $realm) {
47 47
 		$userSession = \OC::$server->getUserSession();
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 	 *
62 62
 	 * If nobody is currently logged in, this method should return null.
63 63
 	 *
64
-	 * @return string|null
64
+	 * @return string
65 65
 	 */
66 66
 	function getCurrentUser() {
67 67
 		return $this->user;
Please login to merge, or discard this patch.
lib/private/largefilehelper.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -141,7 +141,7 @@
 block discarded – undo
141 141
 	*
142 142
 	* @param string $filename Path to the file.
143 143
 	*
144
-	* @return null|int|float Number of bytes as number (float or int) or
144
+	* @return integer|null Number of bytes as number (float or int) or
145 145
 	*                        null on failure.
146 146
 	*/
147 147
 	public function getFileSizeViaCOM($filename) {
Please login to merge, or discard this patch.