@@ -318,6 +318,11 @@ |
||
318 | 318 | */ |
319 | 319 | // FIXME This method is only public, until OC_L10N does not need it anymore, |
320 | 320 | // FIXME This is also the reason, why it is not in the public interface |
321 | + |
|
322 | + /** |
|
323 | + * @param string $app |
|
324 | + * @param string $lang |
|
325 | + */ |
|
321 | 326 | public function getL10nFilesForApp($app, $lang) { |
322 | 327 | $languageFiles = []; |
323 | 328 |
@@ -45,7 +45,6 @@ |
||
45 | 45 | use OC\App\CodeChecker\CodeChecker; |
46 | 46 | use OC\App\CodeChecker\EmptyCheck; |
47 | 47 | use OC\App\CodeChecker\PrivateCheck; |
48 | -use OC\Archive\Archive; |
|
49 | 48 | use OC\Archive\TAR; |
50 | 49 | use OC_App; |
51 | 50 | use OC_DB; |
@@ -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 | /** |
@@ -185,6 +185,9 @@ discard block |
||
185 | 185 | return $event; |
186 | 186 | } |
187 | 187 | |
188 | + /** |
|
189 | + * @param string $subject |
|
190 | + */ |
|
188 | 191 | protected function getParsedParameters($subject, array $parameters) { |
189 | 192 | switch ($subject) { |
190 | 193 | case 'created_self': |
@@ -220,6 +223,9 @@ discard block |
||
220 | 223 | return []; |
221 | 224 | } |
222 | 225 | |
226 | + /** |
|
227 | + * @param string $subject |
|
228 | + */ |
|
223 | 229 | protected function getRichParameters($subject, array $parameters) { |
224 | 230 | switch ($subject) { |
225 | 231 | case 'created_self': |
@@ -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 | /** |