@@ -102,7 +102,7 @@ discard block |
||
102 | 102 | * @param string $wrongPath wrong path of the entry to fix |
103 | 103 | * @param int $correctStorageNumericId numeric idea of the correct storage |
104 | 104 | * @param string $correctPath value to which to set the path of the entry |
105 | - * @return bool true for success |
|
105 | + * @return boolean|null true for success |
|
106 | 106 | */ |
107 | 107 | private function fixEntryPath(IOutput $out, $fileId, $wrongPath, $correctStorageNumericId, $correctPath) { |
108 | 108 | // delete target if exists |
@@ -132,6 +132,9 @@ discard block |
||
132 | 132 | $out->advance(1, $text); |
133 | 133 | } |
134 | 134 | |
135 | + /** |
|
136 | + * @param \OCP\DB\QueryBuilder\IQueryBuilder $qb |
|
137 | + */ |
|
135 | 138 | private function addQueryConditionsParentIdWrongPath($qb) { |
136 | 139 | // thanks, VicDeo! |
137 | 140 | if ($this->connection->getDatabasePlatform() instanceof MySqlPlatform) { |
@@ -171,6 +174,9 @@ discard block |
||
171 | 174 | } |
172 | 175 | } |
173 | 176 | |
177 | + /** |
|
178 | + * @param \OCP\DB\QueryBuilder\IQueryBuilder $qb |
|
179 | + */ |
|
174 | 180 | private function addQueryConditionsNonExistingParentIdEntry($qb, $storageNumericId = null) { |
175 | 181 | // Subquery for parent existence |
176 | 182 | $qbe = $this->connection->getQueryBuilder(); |
@@ -200,6 +206,9 @@ discard block |
||
200 | 206 | } |
201 | 207 | } |
202 | 208 | |
209 | + /** |
|
210 | + * @param integer $storageNumericId |
|
211 | + */ |
|
203 | 212 | private function countResultsToProcessParentIdWrongPath($storageNumericId = null) { |
204 | 213 | $qb = $this->connection->getQueryBuilder(); |
205 | 214 | $qb->select($qb->createFunction('COUNT(*)')); |
@@ -210,6 +219,9 @@ discard block |
||
210 | 219 | return $count; |
211 | 220 | } |
212 | 221 | |
222 | + /** |
|
223 | + * @param integer $storageNumericId |
|
224 | + */ |
|
213 | 225 | private function countResultsToProcessNonExistingParentIdEntry($storageNumericId = null) { |
214 | 226 | $qb = $this->connection->getQueryBuilder(); |
215 | 227 | $qb->select($qb->createFunction('COUNT(*)')); |
@@ -359,6 +371,7 @@ discard block |
||
359 | 371 | * |
360 | 372 | * @param int $storageId storage id |
361 | 373 | * @param string $path path for which to create the parent entry |
374 | + * @param integer $reuseFileId |
|
362 | 375 | * @return int file id of the newly created parent |
363 | 376 | */ |
364 | 377 | private function getOrCreateEntry($storageId, $path, $reuseFileId = null) { |
@@ -741,11 +741,19 @@ discard block |
||
741 | 741 | } |
742 | 742 | |
743 | 743 | // case-insensitive array_search |
744 | + |
|
745 | + /** |
|
746 | + * @param string $needle |
|
747 | + */ |
|
744 | 748 | protected function array_searchi($needle, $haystack, $mem='getName') { |
745 | 749 | if (!\is_array($haystack)) { |
746 | 750 | return false; |
747 | 751 | } |
748 | 752 | return \array_search(\strtolower($needle), \array_map( |
753 | + |
|
754 | + /** |
|
755 | + * @param string $tag |
|
756 | + */ |
|
749 | 757 | function ($tag) use ($mem) { |
750 | 758 | return \strtolower(\call_user_func([$tag, $mem])); |
751 | 759 | }, $haystack) |
@@ -770,7 +778,7 @@ discard block |
||
770 | 778 | * Get a tag by its name. |
771 | 779 | * |
772 | 780 | * @param string $name The tag name. |
773 | - * @return integer|bool The tag object's offset within the $this->tags |
|
781 | + * @return \OCP\AppFramework\Db\Entity The tag object's offset within the $this->tags |
|
774 | 782 | * array or false if it doesn't exist. |
775 | 783 | */ |
776 | 784 | private function getTagByName($name) { |
@@ -781,7 +789,7 @@ discard block |
||
781 | 789 | * Get a tag by its ID. |
782 | 790 | * |
783 | 791 | * @param string $id The tag ID to look for. |
784 | - * @return integer|bool The tag object's offset within the $this->tags |
|
792 | + * @return \OCP\AppFramework\Db\Entity The tag object's offset within the $this->tags |
|
785 | 793 | * array or false if it doesn't exist. |
786 | 794 | */ |
787 | 795 | private function getTagById($id) { |
@@ -67,7 +67,7 @@ |
||
67 | 67 | /** |
68 | 68 | * Search for files by mimetype |
69 | 69 | * @param string $mimetype |
70 | - * @return array |
|
70 | + * @return \OC\Files\FileInfo[] |
|
71 | 71 | * @since 6.0.0 |
72 | 72 | */ |
73 | 73 | public static function searchByMime($mimetype) { |
@@ -32,18 +32,21 @@ discard block |
||
32 | 32 | /** |
33 | 33 | * @param string $message |
34 | 34 | * @since 9.1.0 |
35 | + * @return void |
|
35 | 36 | */ |
36 | 37 | public function info($message); |
37 | 38 | |
38 | 39 | /** |
39 | 40 | * @param string $message |
40 | 41 | * @since 9.1.0 |
42 | + * @return void |
|
41 | 43 | */ |
42 | 44 | public function warning($message); |
43 | 45 | |
44 | 46 | /** |
45 | 47 | * @param int $max |
46 | 48 | * @since 9.1.0 |
49 | + * @return void |
|
47 | 50 | */ |
48 | 51 | public function startProgress($max = 0); |
49 | 52 | |
@@ -51,12 +54,13 @@ discard block |
||
51 | 54 | * @param int $step |
52 | 55 | * @param string $description |
53 | 56 | * @since 9.1.0 |
57 | + * @return void |
|
54 | 58 | */ |
55 | 59 | public function advance($step = 1, $description = ''); |
56 | 60 | |
57 | 61 | /** |
58 | - * @param int $max |
|
59 | 62 | * @since 9.1.0 |
63 | + * @return void |
|
60 | 64 | */ |
61 | 65 | public function finishProgress(); |
62 | 66 | } |
@@ -67,6 +67,9 @@ |
||
67 | 67 | } |
68 | 68 | } |
69 | 69 | |
70 | + /** |
|
71 | + * @param string|boolean $username |
|
72 | + */ |
|
70 | 73 | private static function sendNotificationMail($username) { |
71 | 74 | $userObject = \OC::$server->getUserManager()->get($username); |
72 | 75 | $email = $userObject->getEMailAddress(); |
@@ -245,7 +245,7 @@ |
||
245 | 245 | } |
246 | 246 | |
247 | 247 | /** |
248 | - * @return array|null |
|
248 | + * @return string |
|
249 | 249 | */ |
250 | 250 | public function getLocalSystemAddressBook() { |
251 | 251 | if ($this->localSystemAddressBook === null) { |
@@ -33,7 +33,6 @@ |
||
33 | 33 | use OCP\Share\IShare; |
34 | 34 | use Symfony\Component\EventDispatcher\GenericEvent; |
35 | 35 | use OCA\Files_Sharing\Service\NotificationPublisher; |
36 | -use OCP\Share\Exceptions\ShareNotFound; |
|
37 | 36 | |
38 | 37 | class Hooks { |
39 | 38 | /** |
@@ -138,6 +138,10 @@ |
||
138 | 138 | ); |
139 | 139 | } |
140 | 140 | |
141 | + /** |
|
142 | + * @param IShare[] $shares |
|
143 | + * @param string $fileId |
|
144 | + */ |
|
141 | 145 | private function filterSharesByFileId($shares, $fileId) { |
142 | 146 | return \array_filter($shares, function (IShare $share) use ($fileId) { |
143 | 147 | return \strval($share->getNodeId()) === \strval($fileId); |
@@ -310,7 +310,7 @@ |
||
310 | 310 | * @param string[] $removedUsers |
311 | 311 | * @param InputInterface $input |
312 | 312 | * @param OutputInterface $output |
313 | - * @param $missingAccountsAction |
|
313 | + * @param string $missingAccountsAction |
|
314 | 314 | */ |
315 | 315 | private function handleRemovedUsers(array $removedUsers, InputInterface $input, OutputInterface $output, $missingAccountsAction) { |
316 | 316 | if (empty($removedUsers)) { |
@@ -26,7 +26,6 @@ |
||
26 | 26 | namespace OC\Files\External\Service; |
27 | 27 | |
28 | 28 | use OC\Files\Filesystem; |
29 | - |
|
30 | 29 | use OCP\Files\External\IStorageConfig; |
31 | 30 | use OCP\Files\External\IStoragesBackendService; |
32 | 31 | use OCP\Files\External\Service\IGlobalStoragesService; |