@@ -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) { |