@@ -1100,6 +1100,9 @@ |
||
| 1100 | 1100 | return $share; |
| 1101 | 1101 | } |
| 1102 | 1102 | |
| 1103 | + /** |
|
| 1104 | + * @param \OCP\Share\IShare $share |
|
| 1105 | + */ |
|
| 1103 | 1106 | protected function checkExpireDate($share) { |
| 1104 | 1107 | if ($share->getExpirationDate() !== null && |
| 1105 | 1108 | $share->getExpirationDate() <= new \DateTime()) { |
@@ -163,7 +163,6 @@ discard block |
||
| 163 | 163 | /** |
| 164 | 164 | * @param string $url |
| 165 | 165 | * @param string $userName |
| 166 | - * @param string $addressBookUrl |
|
| 167 | 166 | * @param string $sharedSecret |
| 168 | 167 | * @return Client |
| 169 | 168 | */ |
@@ -301,7 +300,7 @@ discard block |
||
| 301 | 300 | } |
| 302 | 301 | |
| 303 | 302 | /** |
| 304 | - * @return array|null |
|
| 303 | + * @return string |
|
| 305 | 304 | */ |
| 306 | 305 | public function getLocalSystemAddressBook() { |
| 307 | 306 | if (is_null($this->localSystemAddressBook)) { |
@@ -37,7 +37,7 @@ |
||
| 37 | 37 | private $parameters = []; |
| 38 | 38 | |
| 39 | 39 | /** |
| 40 | - * @param object $object an object or classname |
|
| 40 | + * @param \OCP\AppFramework\Controller $object an object or classname |
|
| 41 | 41 | * @param string $method the method which we want to inspect |
| 42 | 42 | */ |
| 43 | 43 | public function reflect($object, $method){ |
@@ -276,10 +276,10 @@ discard block |
||
| 276 | 276 | * publish activity if a file/folder was shared by mail |
| 277 | 277 | * |
| 278 | 278 | * @param $subject |
| 279 | - * @param $parameters |
|
| 280 | - * @param $affectedUser |
|
| 279 | + * @param string[] $parameters |
|
| 280 | + * @param string $affectedUser |
|
| 281 | 281 | * @param $fileId |
| 282 | - * @param $filePath |
|
| 282 | + * @param string $filePath |
|
| 283 | 283 | */ |
| 284 | 284 | protected function publishActivity($subject, $parameters, $affectedUser, $fileId, $filePath) { |
| 285 | 285 | $event = $this->activityManager->generateEvent(); |
@@ -557,6 +557,7 @@ discard block |
||
| 557 | 557 | * @param string $uidOwner |
| 558 | 558 | * @param int $permissions |
| 559 | 559 | * @param string $token |
| 560 | + * @param string $password |
|
| 560 | 561 | * @return int |
| 561 | 562 | */ |
| 562 | 563 | protected function addShareToDB($itemSource, $itemType, $shareWith, $sharedBy, $uidOwner, $permissions, $token, $password) { |
@@ -950,7 +951,7 @@ discard block |
||
| 950 | 951 | /** |
| 951 | 952 | * get database row of a give share |
| 952 | 953 | * |
| 953 | - * @param $id |
|
| 954 | + * @param integer $id |
|
| 954 | 955 | * @return array |
| 955 | 956 | * @throws ShareNotFound |
| 956 | 957 | */ |
@@ -320,7 +320,7 @@ |
||
| 320 | 320 | |
| 321 | 321 | /** |
| 322 | 322 | * @param IJob $job |
| 323 | - * @param $timeTaken |
|
| 323 | + * @param integer $timeTaken |
|
| 324 | 324 | */ |
| 325 | 325 | public function setExecutionTime(IJob $job, $timeTaken) { |
| 326 | 326 | $query = $this->connection->getQueryBuilder(); |
@@ -431,7 +431,7 @@ |
||
| 431 | 431 | } |
| 432 | 432 | |
| 433 | 433 | /** |
| 434 | - * @param array $disabledApps |
|
| 434 | + * @param string[] $disabledApps |
|
| 435 | 435 | * @throws \Exception |
| 436 | 436 | */ |
| 437 | 437 | private function upgradeAppStoreApps(array $disabledApps) { |
@@ -48,6 +48,9 @@ discard block |
||
| 48 | 48 | $this->stack = $stack; |
| 49 | 49 | } |
| 50 | 50 | |
| 51 | + /** |
|
| 52 | + * @param double $time |
|
| 53 | + */ |
|
| 51 | 54 | public function end($time) { |
| 52 | 55 | $this->end = $time; |
| 53 | 56 | } |
@@ -67,7 +70,7 @@ discard block |
||
| 67 | 70 | } |
| 68 | 71 | |
| 69 | 72 | /** |
| 70 | - * @return float |
|
| 73 | + * @return integer |
|
| 71 | 74 | */ |
| 72 | 75 | public function getStart() { |
| 73 | 76 | return $this->start; |
@@ -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; |