mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

CalDAV and CardDAV plugins need to be registered for the principals collection as well 

9f2e6431b8

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
This commit is contained in:
Roeland Jago Douma 2018-04-23 13:54:58 +02:00 committed by John Molakvoæ (skjnldsv)
parent 3a133b151f
commit 0f88f3f00e
No known key found for this signature in database
GPG Key ID: 60C25B8C072916CF
2 changed files with 29 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
apps/dav/lib/Server.php
View File

@ -73,7 +73,7 @@ class Server {
private $baseUri;
/** @var Connector\Sabre\Server */
private $server;
public $server;
public function __construct(IRequest $request, $baseUri) {
$this->request = $request;
@ -138,7 +138,7 @@ class Server {
$this->server->addPlugin($acl);
// calendar plugins
if ($this->requestIsForSubtree('calendars')) {
if ($this->requestIsForSubtree(['calendars', 'principals'])) {
$this->server->addPlugin(new \OCA\DAV\CalDAV\Plugin());
$this->server->addPlugin(new \Sabre\CalDAV\ICSExportPlugin());
$this->server->addPlugin(new \OCA\DAV\CalDAV\Schedule\Plugin());
@ -155,7 +155,8 @@ class Server {
}
// addressbook plugins
if ($this->requestIsForSubtree('addressbooks')) {
if ($this->requestIsForSubtree(['addressbooks', 'principals'])) {
$this->server->addPlugin(new DAV\Sharing\Plugin($authBackend, \OC::$server->getRequest()));
$this->server->addPlugin(new \OCA\DAV\CardDAV\Plugin());
$this->server->addPlugin(new VCFExportPlugin());
$this->server->addPlugin(new ImageExportPlugin(new PhotoCache(\OC::$server->getAppDataDir('dav-photocache'))));
@ -286,12 +287,13 @@ class Server {
$this->server->exec();
}
/**
* @param string $subTree
* @return bool
*/
private function requestIsForSubtree($subTree) {
$subTree = trim($subTree, " /");
return strpos($this->server->getRequestUri(), "$subTree/") === 0;
private function requestIsForSubtree(array $subTrees): bool {
foreach ($subTrees as $subTree) {
$subTree = trim($subTree, ' /');
if (strpos($this->server->getRequestUri(), $subTree.'/') === 0) {
return true;
}
}
return false;
}
}

20
apps/dav/tests/unit/ServerTest.php
View File

@ -38,10 +38,24 @@ use OCA\DAV\AppInfo\PluginManager;
*/
class ServerTest extends \Test\TestCase {
public function test() {
/** @var IRequest $r */
/**
* @dataProvider providesUris
*/
public function test($uri, array $plugins) {
/** @var IRequest | \PHPUnit_Framework_MockObject_MockObject $r */
$r = $this->createMock(IRequest::class);
$r->expects($this->any())->method('getRequestUri')->willReturn($uri);
$s = new Server($r, '/');
$this->assertInstanceOf('OCA\DAV\Server', $s);
$this->assertNotNull($s->server);
foreach ($plugins as $plugin) {
$this->assertNotNull($s->server->getPlugin($plugin));
}
}
public function providesUris() {
return [
'principals' => ['principals/users/admin', ['caldav', 'oc-resource-sharing', 'carddav']],
'calendars' => ['calendars/admin', ['caldav', 'oc-resource-sharing']],
'addressbooks' => ['addressbooks/admin', ['carddav', 'oc-resource-sharing']],
];
}
}