@@ -109,7 +109,7 @@ discard block |
||
| 109 | 109 | * |
| 110 | 110 | * @param string $newPath |
| 111 | 111 | * @param \OCP\Share\IShare $share |
| 112 | - * @return bool |
|
| 112 | + * @return boolean|null |
|
| 113 | 113 | */ |
| 114 | 114 | private function updateFileTarget($newPath, &$share) { |
| 115 | 115 | $share->setTarget($newPath); |
@@ -125,7 +125,7 @@ discard block |
||
| 125 | 125 | * @param string $path |
| 126 | 126 | * @param View $view |
| 127 | 127 | * @param SharedMount[] $mountpoints |
| 128 | - * @return mixed |
|
| 128 | + * @return string |
|
| 129 | 129 | */ |
| 130 | 130 | private function generateUniqueTarget($path, $view, array $mountpoints) { |
| 131 | 131 | $pathinfo = \pathinfo($path); |
@@ -121,6 +121,9 @@ |
||
| 121 | 121 | } |
| 122 | 122 | } |
| 123 | 123 | |
| 124 | + /** |
|
| 125 | + * @param integer $name |
|
| 126 | + */ |
|
| 124 | 127 | private function buildController($name) { |
| 125 | 128 | $parts = \explode('.', $name); |
| 126 | 129 | if (\count($parts) === 4 && $parts[0] === 'ocs') { |
@@ -132,7 +132,7 @@ discard block |
||
| 132 | 132 | /** |
| 133 | 133 | * List all installed apps |
| 134 | 134 | * |
| 135 | - * @return string[] |
|
| 135 | + * @return integer[] |
|
| 136 | 136 | */ |
| 137 | 137 | public function getInstalledApps() { |
| 138 | 138 | return \array_keys($this->getInstalledAppsValues()); |
@@ -241,7 +241,7 @@ discard block |
||
| 241 | 241 | /** |
| 242 | 242 | * Do not allow more than one active app-theme |
| 243 | 243 | * |
| 244 | - * @param $appId |
|
| 244 | + * @param string $appId |
|
| 245 | 245 | * @throws AppManagerException |
| 246 | 246 | */ |
| 247 | 247 | protected function canEnableTheme($appId) { |
@@ -400,6 +400,9 @@ discard block |
||
| 400 | 400 | return \in_array($appId, $this->shippedApps); |
| 401 | 401 | } |
| 402 | 402 | |
| 403 | + /** |
|
| 404 | + * @param string $appId |
|
| 405 | + */ |
|
| 403 | 406 | private function isAlwaysEnabled($appId) { |
| 404 | 407 | $alwaysEnabled = $this->getAlwaysEnabledApps(); |
| 405 | 408 | return \in_array($appId, $alwaysEnabled); |
@@ -428,7 +431,7 @@ discard block |
||
| 428 | 431 | /** |
| 429 | 432 | * @param string $package package path |
| 430 | 433 | * @param bool $skipMigrations whether to skip migrations, which would only install the code |
| 431 | - * @return string|false app id or false in case of error |
|
| 434 | + * @return integer app id or false in case of error |
|
| 432 | 435 | * @since 10.0 |
| 433 | 436 | */ |
| 434 | 437 | public function installApp($package, $skipMigrations = false) { |
@@ -441,7 +444,7 @@ discard block |
||
| 441 | 444 | |
| 442 | 445 | /** |
| 443 | 446 | * @param string $package |
| 444 | - * @return mixed |
|
| 447 | + * @return boolean |
|
| 445 | 448 | * @since 10.0 |
| 446 | 449 | */ |
| 447 | 450 | public function updateApp($package) { |
@@ -98,6 +98,9 @@ |
||
| 98 | 98 | } |
| 99 | 99 | } |
| 100 | 100 | |
| 101 | + /** |
|
| 102 | + * @param string $userId |
|
| 103 | + */ |
|
| 101 | 104 | private function buildAvatarPath($userId) { |
| 102 | 105 | $avatar = \substr_replace(\substr_replace(\md5($userId), '/', 4, 0), '/', 2, 0); |
| 103 | 106 | return \explode('/', $avatar); |
@@ -206,7 +206,7 @@ |
||
| 206 | 206 | /** |
| 207 | 207 | * removes an entry from the comments run time cache |
| 208 | 208 | * |
| 209 | - * @param mixed $id the comment's id |
|
| 209 | + * @param string $id the comment's id |
|
| 210 | 210 | */ |
| 211 | 211 | protected function uncache($id) { |
| 212 | 212 | $id = \strval($id); |
@@ -234,6 +234,10 @@ |
||
| 234 | 234 | return '/^' . \preg_quote($this->config->getSystemValue('dbtableprefix', 'oc_')) . '/'; |
| 235 | 235 | } |
| 236 | 236 | |
| 237 | + /** |
|
| 238 | + * @param integer $step |
|
| 239 | + * @param integer $max |
|
| 240 | + */ |
|
| 237 | 241 | protected function emit($sql, $step, $max) { |
| 238 | 242 | if ($this->noEmit) { |
| 239 | 243 | return; |
@@ -409,6 +409,9 @@ |
||
| 409 | 409 | } |
| 410 | 410 | } |
| 411 | 411 | |
| 412 | + /** |
|
| 413 | + * @param string $path |
|
| 414 | + */ |
|
| 412 | 415 | private function getParentPath($path) { |
| 413 | 416 | $parent = \dirname($path); |
| 414 | 417 | if ($parent === '.') { |
@@ -74,7 +74,7 @@ discard block |
||
| 74 | 74 | |
| 75 | 75 | /** |
| 76 | 76 | * @param array $entry |
| 77 | - * @return array |
|
| 77 | + * @return string |
|
| 78 | 78 | */ |
| 79 | 79 | protected function formatCacheEntry($entry) { |
| 80 | 80 | if (isset($entry['path'])) { |
@@ -193,6 +193,9 @@ discard block |
||
| 193 | 193 | return $this->cache->getStatus($this->getSourcePath($file)); |
| 194 | 194 | } |
| 195 | 195 | |
| 196 | + /** |
|
| 197 | + * @param \OCP\Files\Cache\ICacheEntry[] $results |
|
| 198 | + */ |
|
| 196 | 199 | private function formatSearchResults($results) { |
| 197 | 200 | $results = \array_filter($results, [$this, 'filterCacheEntry']); |
| 198 | 201 | $results = \array_values($results); |
@@ -145,6 +145,9 @@ |
||
| 145 | 145 | return true; |
| 146 | 146 | } |
| 147 | 147 | |
| 148 | + /** |
|
| 149 | + * @param string $path |
|
| 150 | + */ |
|
| 148 | 151 | public function url_stat($path) { |
| 149 | 152 | if (isset(self::$data[$path])) { |
| 150 | 153 | $size = \strlen(self::$data[$path]); |