Skip to content

Commit 665a385

Browse files
committed
fix(IShareProvider): Add missing getChildren method to the interface
Signed-off-by: provokateurin <[email protected]>
1 parent 8bc5edf commit 665a385

File tree

6 files changed

+10
-24
lines changed

6 files changed

+10
-24
lines changed

apps/federatedfilesharing/lib/FederatedShareProvider.php

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -431,13 +431,7 @@ public function move(IShare $share, $recipient) {
431431
return $share;
432432
}
433433

434-
/**
435-
* Get all children of this share
436-
*
437-
* @param IShare $parent
438-
* @return IShare[]
439-
*/
440-
public function getChildren(IShare $parent) {
434+
public function getChildren(IShare $parent): array {
441435
$children = [];
442436

443437
$qb = $this->dbConnection->getQueryBuilder();

apps/sharebymail/lib/ShareByMailProvider.php

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -637,11 +637,6 @@ protected function generateToken(int $size = 15): string {
637637
return $token;
638638
}
639639

640-
/**
641-
* Get all children of this share
642-
*
643-
* @return IShare[]
644-
*/
645640
public function getChildren(IShare $parent): array {
646641
$children = [];
647642

build/psalm-baseline.xml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4261,9 +4261,6 @@
42614261
<UndefinedClass>
42624262
<code><![CDATA[\OCA\Circles\Api\v1\Circles]]></code>
42634263
</UndefinedClass>
4264-
<UndefinedInterfaceMethod>
4265-
<code><![CDATA[getChildren]]></code>
4266-
</UndefinedInterfaceMethod>
42674264
</file>
42684265
<file src="lib/private/Share20/ProviderFactory.php">
42694266
<InvalidReturnStatement>

lib/private/Share20/DefaultShareProvider.php

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -358,14 +358,7 @@ public function acceptShare(IShare $share, string $recipient): IShare {
358358
return $share;
359359
}
360360

361-
/**
362-
* Get all children of this share
363-
* FIXME: remove once https://github.com/owncloud/core/pull/21660 is in
364-
*
365-
* @param \OCP\Share\IShare $parent
366-
* @return \OCP\Share\IShare[]
367-
*/
368-
public function getChildren(\OCP\Share\IShare $parent) {
361+
public function getChildren(IShare $parent): array {
369362
$children = [];
370363

371364
$qb = $this->dbConn->getQueryBuilder();

lib/private/Share20/Manager.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1008,7 +1008,6 @@ private function setSharePasswordExpirationTime(IShare $share): void {
10081008

10091009
/**
10101010
* Delete all the children of this share
1011-
* FIXME: remove once https://github.com/owncloud/core/pull/21660 is in
10121011
*
10131012
* @param IShare $share
10141013
* @return IShare[] List of deleted shares

lib/public/Share/IShareProvider.php

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -208,4 +208,12 @@ public function getAccessList($nodes, $currentAccess);
208208
* @since 18.0.0
209209
*/
210210
public function getAllShares(): iterable;
211+
212+
/**
213+
* Get all children of this share
214+
*
215+
* @return IShare[]
216+
* @since 9.0.0
217+
*/
218+
public function getChildren(IShare $parent);
211219
}

0 commit comments

Comments
 (0)