@@ -71,7 +71,7 @@ discard block |
||
| 71 | 71 | /** |
| 72 | 72 | * get color for on-page elements: |
| 73 | 73 | * theme color by default, grey if theme color is to bright |
| 74 | - * @param $color |
|
| 74 | + * @param string $color |
|
| 75 | 75 | * @return string |
| 76 | 76 | */ |
| 77 | 77 | public function elementColor($color) { |
@@ -114,7 +114,7 @@ discard block |
||
| 114 | 114 | |
| 115 | 115 | /** |
| 116 | 116 | * @param $app string app name |
| 117 | - * @return string|ISimpleFile path to app icon / file of logo |
|
| 117 | + * @return string path to app icon / file of logo |
|
| 118 | 118 | */ |
| 119 | 119 | public function getAppIcon($app) { |
| 120 | 120 | $app = str_replace(array('\0', '/', '\\', '..'), '', $app); |
@@ -190,8 +190,8 @@ discard block |
||
| 190 | 190 | /** |
| 191 | 191 | * replace default color with a custom one |
| 192 | 192 | * |
| 193 | - * @param $svg string content of a svg file |
|
| 194 | - * @param $color string color to match |
|
| 193 | + * @param string $svg string content of a svg file |
|
| 194 | + * @param string $color string color to match |
|
| 195 | 195 | * @return string |
| 196 | 196 | */ |
| 197 | 197 | public function colorizeSvg($svg, $color) { |
@@ -265,6 +265,9 @@ discard block |
||
| 265 | 265 | return $this->mapper->has($table, $className); |
| 266 | 266 | } |
| 267 | 267 | |
| 268 | + /** |
|
| 269 | + * @param string $type |
|
| 270 | + */ |
|
| 268 | 271 | private function setupSettingsEntry($settingsClassName, $type) { |
| 269 | 272 | if (!class_exists($settingsClassName)) { |
| 270 | 273 | $this->log->debug('Could not find ' . $type . ' section class ' . $settingsClassName); |
@@ -294,6 +297,9 @@ discard block |
||
| 294 | 297 | } |
| 295 | 298 | } |
| 296 | 299 | |
| 300 | + /** |
|
| 301 | + * @param string $type |
|
| 302 | + */ |
|
| 297 | 303 | private function getSectionTableForType($type) { |
| 298 | 304 | if($type === 'admin') { |
| 299 | 305 | return Mapper::TABLE_ADMIN_SECTIONS; |
@@ -491,7 +491,7 @@ discard block |
||
| 491 | 491 | |
| 492 | 492 | /** |
| 493 | 493 | * returns the internal Nextcloud name for the given LDAP DN of the user, false on DN outside of search DN or failure |
| 494 | - * @param string $dn the dn of the user object |
|
| 494 | + * @param string $fdn the dn of the user object |
|
| 495 | 495 | * @param string $ldapName optional, the display name of the object |
| 496 | 496 | * @return string|false with with the name to use in Nextcloud |
| 497 | 497 | */ |
@@ -508,7 +508,7 @@ discard block |
||
| 508 | 508 | |
| 509 | 509 | /** |
| 510 | 510 | * returns an internal Nextcloud name for the given LDAP DN, false on DN outside of search DN |
| 511 | - * @param string $dn the dn of the user object |
|
| 511 | + * @param string $fdn the dn of the user object |
|
| 512 | 512 | * @param string $ldapName optional, the display name of the object |
| 513 | 513 | * @param bool $isUser optional, whether it is a user object (otherwise group assumed) |
| 514 | 514 | * @return string|false with with the name to use in Nextcloud |
@@ -771,7 +771,7 @@ discard block |
||
| 771 | 771 | * the login filter. |
| 772 | 772 | * |
| 773 | 773 | * @param string $loginName |
| 774 | - * @param array $attributes optional, list of attributes to read |
|
| 774 | + * @param string[] $attributes optional, list of attributes to read |
|
| 775 | 775 | * @return array |
| 776 | 776 | */ |
| 777 | 777 | public function fetchUsersByLoginName($loginName, $attributes = array('dn')) { |
@@ -844,7 +844,7 @@ discard block |
||
| 844 | 844 | |
| 845 | 845 | /** |
| 846 | 846 | * @param string $filter |
| 847 | - * @param string|string[] $attr |
|
| 847 | + * @param string[] $attr |
|
| 848 | 848 | * @param int $limit |
| 849 | 849 | * @param int $offset |
| 850 | 850 | * @return array |
@@ -892,7 +892,7 @@ discard block |
||
| 892 | 892 | |
| 893 | 893 | /** |
| 894 | 894 | * @param string $filter |
| 895 | - * @param string|string[] $attr |
|
| 895 | + * @param string[] $attr |
|
| 896 | 896 | * @param int $limit |
| 897 | 897 | * @param int $offset |
| 898 | 898 | * @return false|int |
@@ -989,6 +989,7 @@ discard block |
||
| 989 | 989 | * retrieved. Results will according to the order in the array. |
| 990 | 990 | * @param int $limit optional, maximum results to be counted |
| 991 | 991 | * @param int $offset optional, a starting point |
| 992 | + * @param string $filter |
|
| 992 | 993 | * @return array|false array with the search result as first value and pagedSearchOK as |
| 993 | 994 | * second | false if not successful |
| 994 | 995 | * @throws \OC\ServerNotAvailableException |
@@ -1240,7 +1241,7 @@ discard block |
||
| 1240 | 1241 | |
| 1241 | 1242 | /** |
| 1242 | 1243 | * @param string $name |
| 1243 | - * @return bool|mixed|string |
|
| 1244 | + * @return string |
|
| 1244 | 1245 | */ |
| 1245 | 1246 | public function sanitizeUsername($name) { |
| 1246 | 1247 | if($this->connection->ldapIgnoreNamingRules) { |
@@ -1264,6 +1265,7 @@ discard block |
||
| 1264 | 1265 | * escapes (user provided) parts for LDAP filter |
| 1265 | 1266 | * @param string $input, the provided value |
| 1266 | 1267 | * @param bool $allowAsterisk whether in * at the beginning should be preserved |
| 1268 | + * @param string $input |
|
| 1267 | 1269 | * @return string the escaped string |
| 1268 | 1270 | */ |
| 1269 | 1271 | public function escapeFilterPart($input, $allowAsterisk = false) { |
@@ -164,7 +164,7 @@ |
||
| 164 | 164 | * @param array $parameters |
| 165 | 165 | * @param bool $silence whether to suppress warnings |
| 166 | 166 | * @throws \ErrorException via trapError |
| 167 | - * @return mixed |
|
| 167 | + * @return string |
|
| 168 | 168 | */ |
| 169 | 169 | private function invoke($functionName, array $parameters = [], $silence = false) { |
| 170 | 170 | try { |
@@ -51,6 +51,9 @@ discard block |
||
| 51 | 51 | ]; |
| 52 | 52 | } |
| 53 | 53 | |
| 54 | + /** |
|
| 55 | + * @param string $type |
|
| 56 | + */ |
|
| 54 | 57 | protected function formatSections($sections, $currentSection, $type, $currentType) { |
| 55 | 58 | $templateParameters = []; |
| 56 | 59 | /** @var \OCP\Settings\ISection[] $prioritizedSections */ |
@@ -84,6 +87,9 @@ discard block |
||
| 84 | 87 | return $templateParameters; |
| 85 | 88 | } |
| 86 | 89 | |
| 90 | + /** |
|
| 91 | + * @param string $currentSections |
|
| 92 | + */ |
|
| 87 | 93 | protected function formatPersonalSections($currentType, $currentSections) { |
| 88 | 94 | $sections = $this->settingsManager->getPersonalSections(); |
| 89 | 95 | $templateParameters = $this->formatSections($sections, $currentSections, 'personal', $currentType); |
@@ -91,6 +97,9 @@ discard block |
||
| 91 | 97 | return $templateParameters; |
| 92 | 98 | } |
| 93 | 99 | |
| 100 | + /** |
|
| 101 | + * @param string $currentSections |
|
| 102 | + */ |
|
| 94 | 103 | protected function formatAdminSections($currentType, $currentSections) { |
| 95 | 104 | $sections = $this->settingsManager->getAdminSections(); |
| 96 | 105 | $templateParameters = $this->formatSections($sections, $currentSections, 'admin', $currentType); |
@@ -114,6 +123,10 @@ discard block |
||
| 114 | 123 | return ['content' => $html]; |
| 115 | 124 | } |
| 116 | 125 | |
| 126 | + /** |
|
| 127 | + * @param string $type |
|
| 128 | + * @param string $section |
|
| 129 | + */ |
|
| 117 | 130 | private function getIndexResponse($type, $section) { |
| 118 | 131 | $templateParams = []; |
| 119 | 132 | $templateParams = array_merge($templateParams, $this->getNavigationParameters($type, $section)); |
@@ -32,7 +32,7 @@ |
||
| 32 | 32 | |
| 33 | 33 | /** |
| 34 | 34 | * @brief states whether the filesystem was loaded |
| 35 | - * @return bool |
|
| 35 | + * @return boolean|null |
|
| 36 | 36 | */ |
| 37 | 37 | public function log($msg, $level) { |
| 38 | 38 | \OCP\Util::writeLog($this->app, $msg, $level); |
@@ -51,7 +51,7 @@ discard block |
||
| 51 | 51 | * |
| 52 | 52 | * @param IUser $user |
| 53 | 53 | * @param IStorageFactory $loader |
| 54 | - * @return \OCP\Files\Mount\IMountPoint |
|
| 54 | + * @return null|MountPoint |
|
| 55 | 55 | */ |
| 56 | 56 | public function getHomeMountForUser(IUser $user, IStorageFactory $loader) { |
| 57 | 57 | |
@@ -69,7 +69,7 @@ discard block |
||
| 69 | 69 | |
| 70 | 70 | /** |
| 71 | 71 | * @param IUser $user |
| 72 | - * @return array|null |
|
| 72 | + * @return IStorageFactory|null |
|
| 73 | 73 | */ |
| 74 | 74 | private function getSingleBucketObjectStoreConfig(IUser $user) { |
| 75 | 75 | $config = $this->config->getSystemValue('objectstore'); |
@@ -93,7 +93,7 @@ discard block |
||
| 93 | 93 | |
| 94 | 94 | /** |
| 95 | 95 | * @param IUser $user |
| 96 | - * @return array|null |
|
| 96 | + * @return IStorageFactory|null |
|
| 97 | 97 | */ |
| 98 | 98 | private function getMultiBucketObjectStoreConfig(IUser $user) { |
| 99 | 99 | $config = $this->config->getSystemValue('objectstore_multibucket'); |
@@ -69,7 +69,7 @@ |
||
| 69 | 69 | /** |
| 70 | 70 | * Generate a keypair |
| 71 | 71 | * |
| 72 | - * @param int $keyLenth |
|
| 72 | + * @param integer $keyLength |
|
| 73 | 73 | * @return array ['privatekey' => $privateKey, 'publickey' => $publicKey] |
| 74 | 74 | */ |
| 75 | 75 | public function createKey($keyLength) { |
@@ -126,7 +126,7 @@ |
||
| 126 | 126 | |
| 127 | 127 | /** |
| 128 | 128 | * @param string $data |
| 129 | - * @return int |
|
| 129 | + * @return boolean |
|
| 130 | 130 | */ |
| 131 | 131 | public function stream_write($data) { |
| 132 | 132 | return false; |