@@ -168,7 +168,7 @@ |
||
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) { |
@@ -190,7 +190,7 @@ discard block |
||
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 |
||
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) { |
@@ -189,7 +189,7 @@ discard block |
||
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 |
||
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 |
||
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() |
@@ -174,7 +174,7 @@ |
||
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]); |
@@ -166,6 +166,11 @@ |
||
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 | } |
@@ -40,6 +40,9 @@ discard block |
||
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 |
||
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(); |
@@ -55,6 +55,9 @@ |
||
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() |
@@ -41,7 +41,7 @@ discard block |
||
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 |
||
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; |
@@ -141,7 +141,7 @@ |
||
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) { |