@@ -131,7 +131,7 @@ |
||
131 | 131 | /** |
132 | 132 | * Create the federated share provider |
133 | 133 | * |
134 | - * @return FederatedShareProvider |
|
134 | + * @return null|ShareByMailProvider |
|
135 | 135 | */ |
136 | 136 | protected function getShareByMailProvider() { |
137 | 137 | if ($this->shareByMailProvider === null) { |
@@ -184,10 +184,10 @@ discard block |
||
184 | 184 | * publish activity if a file/folder was shared by mail |
185 | 185 | * |
186 | 186 | * @param $subject |
187 | - * @param $parameters |
|
188 | - * @param $affectedUser |
|
187 | + * @param string[] $parameters |
|
188 | + * @param string $affectedUser |
|
189 | 189 | * @param $fileId |
190 | - * @param $filePath |
|
190 | + * @param string $filePath |
|
191 | 191 | */ |
192 | 192 | protected function publishActivity($subject, $parameters, $affectedUser, $fileId, $filePath) { |
193 | 193 | $event = $this->activityManager->generateEvent(); |
@@ -239,6 +239,12 @@ discard block |
||
239 | 239 | |
240 | 240 | } |
241 | 241 | |
242 | + /** |
|
243 | + * @param string $link |
|
244 | + * @param string $owner |
|
245 | + * @param string $initiator |
|
246 | + * @param string $shareWith |
|
247 | + */ |
|
242 | 248 | protected function sendMailNotification($filename, $link, $owner, $initiator, $shareWith) { |
243 | 249 | if ($owner === $initiator) { |
244 | 250 | $subject = (string)$this->l->t('%s shared »%s« with you', array($owner, $filename)); |
@@ -264,6 +270,7 @@ discard block |
||
264 | 270 | * @param $link |
265 | 271 | * @param $owner |
266 | 272 | * @param $initiator |
273 | + * @param string $template |
|
267 | 274 | * @return string plain text mail |
268 | 275 | * @throws HintException |
269 | 276 | */ |
@@ -703,7 +710,7 @@ discard block |
||
703 | 710 | /** |
704 | 711 | * get database row of a give share |
705 | 712 | * |
706 | - * @param $id |
|
713 | + * @param integer $id |
|
707 | 714 | * @return array |
708 | 715 | * @throws ShareNotFound |
709 | 716 | */ |
@@ -94,7 +94,7 @@ |
||
94 | 94 | /** |
95 | 95 | * Performs a DELETE or UPDATE query to the database. |
96 | 96 | * @param \Doctrine\DBAL\Driver\Statement $query |
97 | - * @param array $parameters |
|
97 | + * @param string[] $parameters |
|
98 | 98 | * @return bool true if at least one row was modified, false otherwise |
99 | 99 | */ |
100 | 100 | protected function modify($query, $parameters) { |
@@ -36,7 +36,6 @@ |
||
36 | 36 | use OCA\Files_Sharing\ISharedStorage; |
37 | 37 | use OCP\Constants; |
38 | 38 | use OCP\Files\Cache\ICacheEntry; |
39 | -use OCP\Files\Storage\IStorage; |
|
40 | 39 | use OCP\Lock\ILockingProvider; |
41 | 40 | |
42 | 41 | /** |
@@ -28,7 +28,6 @@ |
||
28 | 28 | use OCP\IUserManager; |
29 | 29 | use OCP\Util; |
30 | 30 | use Symfony\Component\EventDispatcher\EventDispatcher; |
31 | -use Symfony\Component\EventDispatcher\GenericEvent; |
|
32 | 31 | |
33 | 32 | class HookManager { |
34 | 33 |
@@ -110,6 +110,9 @@ |
||
110 | 110 | return parent::moveFromCache($sourceCache, $sourcePath, $targetPath); |
111 | 111 | } |
112 | 112 | |
113 | + /** |
|
114 | + * @param ICacheEntry $entry |
|
115 | + */ |
|
113 | 116 | protected function formatCacheEntry($entry) { |
114 | 117 | $path = isset($entry['path']) ? $entry['path'] : ''; |
115 | 118 | $entry = parent::formatCacheEntry($entry); |
@@ -36,7 +36,6 @@ |
||
36 | 36 | use OCA\Files_Sharing\ISharedStorage; |
37 | 37 | use OCP\Constants; |
38 | 38 | use OCP\Files\Cache\ICacheEntry; |
39 | -use OCP\Files\Storage\IStorage; |
|
40 | 39 | use OCP\Lock\ILockingProvider; |
41 | 40 | |
42 | 41 | /** |
@@ -194,6 +194,9 @@ |
||
194 | 194 | return $this->cache->getStatus($this->getSourcePath($file)); |
195 | 195 | } |
196 | 196 | |
197 | + /** |
|
198 | + * @param ICacheEntry[] $results |
|
199 | + */ |
|
197 | 200 | private function formatSearchResults($results) { |
198 | 201 | $results = array_filter($results, array($this, 'filterCacheEntry')); |
199 | 202 | $results = array_values($results); |
@@ -32,7 +32,6 @@ |
||
32 | 32 | namespace OCA\Files_Sharing; |
33 | 33 | |
34 | 34 | use OC\Files\Filesystem; |
35 | -use OC\Files\Cache\FailedCache; |
|
36 | 35 | use OC\Files\Storage\FailedStorage; |
37 | 36 | use OCP\Constants; |
38 | 37 | use OCP\Files\Cache\ICacheEntry; |
@@ -350,7 +350,7 @@ discard block |
||
350 | 350 | * split user and remote from federated cloud id |
351 | 351 | * |
352 | 352 | * @param string $address federated share address |
353 | - * @return array [user, remoteURL] |
|
353 | + * @return string[] [user, remoteURL] |
|
354 | 354 | * @throws \Exception |
355 | 355 | */ |
356 | 356 | public function splitUserRemote($address) { |
@@ -625,6 +625,9 @@ discard block |
||
625 | 625 | return $result; |
626 | 626 | } |
627 | 627 | |
628 | + /** |
|
629 | + * @param string $search |
|
630 | + */ |
|
628 | 631 | protected function getLookup($search) { |
629 | 632 | $client = $this->clientService->newClient(); |
630 | 633 |