Address minor comments

Signed-off-by: Julius Härtl <jus@bitgrid.net>
This commit is contained in:
Julius Härtl 2020-12-09 13:19:14 +01:00
parent 676d2b7a94
commit c4ea37b8a1
No known key found for this signature in database
GPG Key ID: 4C614C6ED2CDE6DF
6 changed files with 13 additions and 11 deletions

View File

@ -222,7 +222,7 @@ class ApiController extends Controller {
IShare::TYPE_REMOTE, IShare::TYPE_REMOTE,
IShare::TYPE_EMAIL, IShare::TYPE_EMAIL,
IShare::TYPE_ROOM, IShare::TYPE_ROOM,
IShare::TYPE_DECK IShare::TYPE_DECK,
]; ];
foreach ($requestedShareTypes as $requestedShareType) { foreach ($requestedShareTypes as $requestedShareType) {
// one of each type is enough to find out about the types // one of each type is enough to find out about the types

View File

@ -220,7 +220,7 @@ class DeletedShareAPIController extends OCSController {
throw new QueryException(); throw new QueryException();
} }
return $this->serverContainer->query('\OCA\Talk\Share\Helper\DeletedShareAPIController'); return $this->serverContainer->get('\OCA\Talk\Share\Helper\DeletedShareAPIController');
} }
/** /**
@ -237,6 +237,6 @@ class DeletedShareAPIController extends OCSController {
throw new QueryException(); throw new QueryException();
} }
return $this->serverContainer->query('\OCA\Deck\Sharing\ShareAPIHelper'); return $this->serverContainer->get('\OCA\Deck\Sharing\ShareAPIHelper');
} }
} }

View File

@ -1558,7 +1558,7 @@ class ShareAPIController extends OCSController {
throw new QueryException(); throw new QueryException();
} }
return $this->serverContainer->query('\OCA\Talk\Share\Helper\ShareAPIController'); return $this->serverContainer->get('\OCA\Talk\Share\Helper\ShareAPIController');
} }
/** /**
@ -1575,7 +1575,7 @@ class ShareAPIController extends OCSController {
throw new QueryException(); throw new QueryException();
} }
return $this->serverContainer->query('\OCA\Deck\Sharing\ShareAPIHelper'); return $this->serverContainer->get('\OCA\Deck\Sharing\ShareAPIHelper');
} }
/** /**

View File

@ -1558,7 +1558,7 @@ class ShareAPIControllerTest extends TestCase {
->with($share, $this->currentUser) ->with($share, $this->currentUser)
->willReturn($canAccessShareByHelper); ->willReturn($canAccessShareByHelper);
$this->serverContainer->method('query') $this->serverContainer->method('get')
->with('\OCA\Talk\Share\Helper\ShareAPIController') ->with('\OCA\Talk\Share\Helper\ShareAPIController')
->willReturn($helper); ->willReturn($helper);
} }
@ -2222,7 +2222,7 @@ class ShareAPIControllerTest extends TestCase {
} }
); );
$this->serverContainer->method('query') $this->serverContainer->method('get')
->with('\OCA\Talk\Share\Helper\ShareAPIController') ->with('\OCA\Talk\Share\Helper\ShareAPIController')
->willReturn($helper); ->willReturn($helper);
@ -2340,7 +2340,7 @@ class ShareAPIControllerTest extends TestCase {
} }
); );
$this->serverContainer->method('query') $this->serverContainer->method('get')
->with('\OCA\Talk\Share\Helper\ShareAPIController') ->with('\OCA\Talk\Share\Helper\ShareAPIController')
->willReturn($helper); ->willReturn($helper);
@ -4404,7 +4404,7 @@ class ShareAPIControllerTest extends TestCase {
->with($share) ->with($share)
->willReturn($formatShareByHelper); ->willReturn($formatShareByHelper);
$this->serverContainer->method('query') $this->serverContainer->method('get')
->with('\OCA\Talk\Share\Helper\ShareAPIController') ->with('\OCA\Talk\Share\Helper\ShareAPIController')
->willReturn($helper); ->willReturn($helper);
} }

View File

@ -71,9 +71,10 @@ class Constants {
public const SHARE_TYPE_ROOM = 10; public const SHARE_TYPE_ROOM = 10;
// const SHARE_TYPE_USERROOM = 11; // Internal type used by RoomShareProvider // const SHARE_TYPE_USERROOM = 11; // Internal type used by RoomShareProvider
/** /**
* @deprecated 21.0.0 - use IShare::TYPE_ROOM instead * @deprecated 21.0.0 - use IShare::TYPE_DECK instead
*/ */
public const SHARE_TYPE_DECK = 12; public const SHARE_TYPE_DECK = 12;
// const SHARE_TYPE_DECK_USER = 13; // Internal type used by DeckShareProvider
public const FORMAT_NONE = -1; public const FORMAT_NONE = -1;
public const FORMAT_STATUSES = -2; public const FORMAT_STATUSES = -2;

View File

@ -111,7 +111,8 @@ interface IShare {
public const TYPE_DECK = 12; public const TYPE_DECK = 12;
/** /**
* @internal 21.00 * @internal
* @since 21.0.0
*/ */
public const TYPE_DECK_USER = 13; public const TYPE_DECK_USER = 13;