Skip to content

Commit 9d535f6

Browse files
skjnldsvskjnldsv
committed
feat: make systemtags public visible
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
1 parent 75f5cb7 commit 9d535f6

13 files changed

Lines changed: 132 additions & 47 deletions

File tree

‎apps/dav/appinfo/v2/publicremote.php‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
use OCA\DAV\Storage\PublicOwnerWrapper;
1313
use OCA\DAV\Storage\PublicShareWrapper;
1414
use OCA\FederatedFileSharing\FederatedShareProvider;
15+
use OCP\BeforeSabrePubliclyLoadedEvent;
1516
use OCP\EventDispatcher\IEventDispatcher;
1617
use OCP\Files\Mount\IMountManager;
1718
use OCP\IConfig;
@@ -35,6 +36,7 @@
3536

3637
$session = \OCP\Server::get(ISession::class);
3738
$request = \OCP\Server::get(IRequest::class);
39+
$eventDispatcher = \OCP\Server::get(IEventDispatcher::class);
3840

3941
$session->close();
4042
$requestUri = $request->getRequestUri();
@@ -59,7 +61,7 @@
5961
\OCP\Server::get(ITagManager::class),
6062
$request,
6163
\OCP\Server::get(IPreview::class),
62-
\OCP\Server::get(IEventDispatcher::class),
64+
$eventDispatcher,
6365
$l10nFactory->get('dav'),
6466
);
6567

@@ -131,5 +133,9 @@
131133
$server->addPlugin($linkCheckPlugin);
132134
$server->addPlugin($filesDropPlugin);
133135

136+
// allow setup of additional plugins
137+
$event = new BeforeSabrePubliclyLoadedEvent($server);
138+
$eventDispatcher->dispatchTyped($event);
139+
134140
// And off we go!
135141
$server->exec();

‎apps/dav/lib/Connector/Sabre/ServerFactory.php‎

Lines changed: 10 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -28,39 +28,19 @@
2828
use Sabre\DAV\Auth\Plugin;
2929

3030
class ServerFactory {
31-
private IConfig $config;
32-
private LoggerInterface $logger;
33-
private IDBConnection $databaseConnection;
34-
private IUserSession $userSession;
35-
private IMountManager $mountManager;
36-
private ITagManager $tagManager;
37-
private IRequest $request;
38-
private IPreview $previewManager;
39-
private IEventDispatcher $eventDispatcher;
40-
private IL10N $l10n;
4131

4232
public function __construct(
43-
IConfig $config,
44-
LoggerInterface $logger,
45-
IDBConnection $databaseConnection,
46-
IUserSession $userSession,
47-
IMountManager $mountManager,
48-
ITagManager $tagManager,
49-
IRequest $request,
50-
IPreview $previewManager,
51-
IEventDispatcher $eventDispatcher,
52-
IL10N $l10n,
33+
private IConfig $config,
34+
private LoggerInterface $logger,
35+
private IDBConnection $databaseConnection,
36+
private IUserSession $userSession,
37+
private IMountManager $mountManager,
38+
private ITagManager $tagManager,
39+
private IRequest $request,
40+
private IPreview $previewManager,
41+
private IEventDispatcher $eventDispatcher,
42+
private IL10N $l10n,
5343
) {
54-
$this->config = $config;
55-
$this->logger = $logger;
56-
$this->databaseConnection = $databaseConnection;
57-
$this->userSession = $userSession;
58-
$this->mountManager = $mountManager;
59-
$this->tagManager = $tagManager;
60-
$this->request = $request;
61-
$this->previewManager = $previewManager;
62-
$this->eventDispatcher = $eventDispatcher;
63-
$this->l10n = $l10n;
6444
}
6545

6646
/**

‎apps/dav/lib/Server.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,7 @@ public function __construct(IRequest $request, string $baseUri) {
197197
}
198198

199199
// system tags plugins
200-
$this->server->addPlugin(\OC::$server->get(SystemTagPlugin::class));
200+
// $this->server->addPlugin(\OCP\Server::get(SystemTagPlugin::class));
201201

202202
// comments plugin
203203
$this->server->addPlugin(new CommentsPlugin(

‎apps/dav/lib/SystemTag/SystemTagList.php‎

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,14 @@
2020
class SystemTagList implements Element {
2121
public const NS_NEXTCLOUD = 'http://nextcloud.org/ns';
2222

23-
/** @var ISystemTag[] */
24-
private array $tags;
25-
private ISystemTagManager $tagManager;
26-
private IUser $user;
27-
28-
public function __construct(array $tags, ISystemTagManager $tagManager, IUser $user) {
23+
/**
24+
* @param ISystemTag[] $tags
25+
*/
26+
public function __construct(
27+
private array $tags,
28+
private ISystemTagManager $tagManager,
29+
private ?IUser $user,
30+
) {
2931
$this->tags = $tags;
3032
$this->tagManager = $tagManager;
3133
$this->user = $user;

‎apps/dav/lib/SystemTag/SystemTagPlugin.php‎

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -305,9 +305,6 @@ private function propfindForFile(PropFind $propFind, Node $node): void {
305305

306306
$propFind->handle(self::SYSTEM_TAGS_PROPERTYNAME, function () use ($node) {
307307
$user = $this->userSession->getUser();
308-
if ($user === null) {
309-
return;
310-
}
311308

312309
$tags = $this->getTagsForFile($node->getId(), $user);
313310
usort($tags, function (ISystemTag $tagA, ISystemTag $tagB): int {
@@ -321,8 +318,7 @@ private function propfindForFile(PropFind $propFind, Node $node): void {
321318
* @param int $fileId
322319
* @return ISystemTag[]
323320
*/
324-
private function getTagsForFile(int $fileId, IUser $user): array {
325-
321+
private function getTagsForFile(int $fileId, ?IUser $user): array {
326322
if (isset($this->cachedTagMappings[$fileId])) {
327323
$tagIds = $this->cachedTagMappings[$fileId];
328324
} else {

‎apps/systemtags/composer/composer/autoload_classmap.php‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,9 @@
1313
'OCA\\SystemTags\\AppInfo\\Application' => $baseDir . '/../lib/AppInfo/Application.php',
1414
'OCA\\SystemTags\\Capabilities' => $baseDir . '/../lib/Capabilities.php',
1515
'OCA\\SystemTags\\Controller\\LastUsedController' => $baseDir . '/../lib/Controller/LastUsedController.php',
16+
'OCA\\SystemTags\\Listeners\\BeforeSabrePubliclyLoadedListener' => $baseDir . '/../lib/Listeners/BeforeSabrePubliclyLoadedListener.php',
17+
'OCA\\SystemTags\\Listeners\\BeforeTemplateRenderedListener' => $baseDir . '/../lib/Listeners/BeforeTemplateRenderedListener.php',
18+
'OCA\\SystemTags\\Listeners\\LoadAdditionalScriptsListener' => $baseDir . '/../lib/Listeners/LoadAdditionalScriptsListener.php',
1619
'OCA\\SystemTags\\Search\\TagSearchProvider' => $baseDir . '/../lib/Search/TagSearchProvider.php',
1720
'OCA\\SystemTags\\Settings\\Admin' => $baseDir . '/../lib/Settings/Admin.php',
1821
);

‎apps/systemtags/composer/composer/autoload_static.php‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,9 @@ class ComposerStaticInitSystemTags
2828
'OCA\\SystemTags\\AppInfo\\Application' => __DIR__ . '/..' . '/../lib/AppInfo/Application.php',
2929
'OCA\\SystemTags\\Capabilities' => __DIR__ . '/..' . '/../lib/Capabilities.php',
3030
'OCA\\SystemTags\\Controller\\LastUsedController' => __DIR__ . '/..' . '/../lib/Controller/LastUsedController.php',
31+
'OCA\\SystemTags\\Listeners\\BeforeSabrePubliclyLoadedListener' => __DIR__ . '/..' . '/../lib/Listeners/BeforeSabrePubliclyLoadedListener.php',
32+
'OCA\\SystemTags\\Listeners\\BeforeTemplateRenderedListener' => __DIR__ . '/..' . '/../lib/Listeners/BeforeTemplateRenderedListener.php',
33+
'OCA\\SystemTags\\Listeners\\LoadAdditionalScriptsListener' => __DIR__ . '/..' . '/../lib/Listeners/LoadAdditionalScriptsListener.php',
3134
'OCA\\SystemTags\\Search\\TagSearchProvider' => __DIR__ . '/..' . '/../lib/Search/TagSearchProvider.php',
3235
'OCA\\SystemTags\\Settings\\Admin' => __DIR__ . '/..' . '/../lib/Settings/Admin.php',
3336
);

‎apps/systemtags/lib/AppInfo/Application.php‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,18 @@
99
namespace OCA\SystemTags\AppInfo;
1010

1111
use OCA\Files\Event\LoadAdditionalScriptsEvent;
12+
use OCA\Files_Sharing\Event\BeforeTemplateRenderedEvent;
1213
use OCA\SystemTags\Activity\Listener;
1314
use OCA\SystemTags\Capabilities;
15+
use OCA\SystemTags\Listeners\BeforeSabrePubliclyLoadedListener;
16+
use OCA\SystemTags\Listeners\BeforeTemplateRenderedListener;
17+
use OCA\SystemTags\Listeners\LoadAdditionalScriptsListener;
1418
use OCA\SystemTags\Search\TagSearchProvider;
1519
use OCP\AppFramework\App;
1620
use OCP\AppFramework\Bootstrap\IBootContext;
1721
use OCP\AppFramework\Bootstrap\IBootstrap;
1822
use OCP\AppFramework\Bootstrap\IRegistrationContext;
23+
use OCP\BeforeSabrePubliclyLoadedEvent;
1924
use OCP\EventDispatcher\IEventDispatcher;
2025
use OCP\SystemTag\ManagerEvent;
2126
use OCP\SystemTag\MapperEvent;
@@ -30,6 +35,9 @@ public function __construct() {
3035
public function register(IRegistrationContext $context): void {
3136
$context->registerSearchProvider(TagSearchProvider::class);
3237
$context->registerCapability(Capabilities::class);
38+
$context->registerEventListener(LoadAdditionalScriptsEvent::class, LoadAdditionalScriptsListener::class);
39+
$context->registerEventListener(BeforeTemplateRenderedEvent::class, BeforeTemplateRenderedListener::class);
40+
$context->registerEventListener(BeforeSabrePubliclyLoadedEvent::class, BeforeSabrePubliclyLoadedListener::class);
3341
}
3442

3543
public function boot(IBootContext $context): void {

‎apps/systemtags/lib/Listeners/BeforeSabrePubliclyLoadedListener.php‎

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
/**
5+
* SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
6+
* SPDX-License-Identifier: AGPL-3.0-or-later
7+
*/
8+
namespace OCA\SystemTags\Listeners;
9+
10+
use OCA\DAV\SystemTag\SystemTagPlugin;
11+
use OCP\BeforeSabrePubliclyLoadedEvent;
12+
use OCP\EventDispatcher\Event;
13+
use OCP\EventDispatcher\IEventListener;
14+
use OCP\Server;
15+
16+
/**
17+
* @template-implements IEventListener<BeforeSabrePubliclyLoadedEvent>
18+
*/
19+
class BeforeSabrePubliclyLoadedListener implements IEventListener {
20+
public function handle(Event $event): void {
21+
if (!$event instanceof BeforeSabrePubliclyLoadedEvent) {
22+
return;
23+
}
24+
$event->getServer()->addPlugin(Server::get(SystemTagPlugin::class));
25+
}
26+
}

‎apps/systemtags/lib/Listeners/BeforeTemplateRenderedListener.php‎

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
/**
5+
* SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
6+
* SPDX-License-Identifier: AGPL-3.0-or-later
7+
*/
8+
namespace OCA\SystemTags\Listeners;
9+
10+
use OCA\Files_Sharing\Event\BeforeTemplateRenderedEvent;
11+
use OCA\SystemTags\AppInfo\Application;
12+
use OCP\EventDispatcher\Event;
13+
use OCP\EventDispatcher\IEventListener;
14+
use OCP\Util;
15+
16+
/**
17+
* @template-implements IEventListener<BeforeTemplateRenderedEvent>
18+
*/
19+
class BeforeTemplateRenderedListener implements IEventListener {
20+
public function handle(Event $event): void {
21+
if (!$event instanceof BeforeTemplateRenderedEvent) {
22+
return;
23+
}
24+
Util::addInitScript(Application::APP_ID, 'init');
25+
}
26+
}

0 commit comments

Comments
 (0)