From 63abe9bc8a40931cc599581d6f31e79abec20f21 Mon Sep 17 00:00:00 2001 From: Ferdinand Thiessen Date: Mon, 13 Apr 2026 16:06:35 +0200 Subject: [PATCH 1/2] feat(files_versions): allow to block version creation using WFE This allows users to create workflows to block the versions creation for some files, based on tags or other conditions using the workflow engine. The usecase would be compliance to allow configure some rules to prevent versioning. Signed-off-by: Ferdinand Thiessen --- .../composer/composer/autoload_classmap.php | 3 + .../composer/composer/autoload_static.php | 3 + .../lib/AppInfo/Application.php | 11 ++- .../lib/BlockVersioningOperation.php | 95 +++++++++++++++++++ .../CreateVersionListenerForWorkflow.php | 37 ++++++++ .../RegisterWorkflowIntegrationListener.php | 33 +++++++ apps/files_versions/src/workflow.ts | 12 +++ build/frontend/vite.config.ts | 1 + 8 files changed, 194 insertions(+), 1 deletion(-) create mode 100644 apps/files_versions/lib/BlockVersioningOperation.php create mode 100644 apps/files_versions/lib/Listener/CreateVersionListenerForWorkflow.php create mode 100644 apps/files_versions/lib/Listener/RegisterWorkflowIntegrationListener.php create mode 100644 apps/files_versions/src/workflow.ts diff --git a/apps/files_versions/composer/composer/autoload_classmap.php b/apps/files_versions/composer/composer/autoload_classmap.php index 27e68decdcc52..22f7174395ff0 100644 --- a/apps/files_versions/composer/composer/autoload_classmap.php +++ b/apps/files_versions/composer/composer/autoload_classmap.php @@ -20,13 +20,16 @@ 'OCA\\Files_Versions\\Events\\VersionCreatedEvent' => $baseDir . '/../lib/Events/VersionCreatedEvent.php', 'OCA\\Files_Versions\\Events\\VersionRestoredEvent' => $baseDir . '/../lib/Events/VersionRestoredEvent.php', 'OCA\\Files_Versions\\Expiration' => $baseDir . '/../lib/Expiration.php', + 'OCA\\Files_Versions\\Listener\\CreateVersionListenerForWorkflow' => $baseDir . '/../lib/Listener/CreateVersionListenerForWorkflow.php', 'OCA\\Files_Versions\\Listener\\FileEventsListener' => $baseDir . '/../lib/Listener/FileEventsListener.php', 'OCA\\Files_Versions\\Listener\\LegacyRollbackListener' => $baseDir . '/../lib/Listener/LegacyRollbackListener.php', 'OCA\\Files_Versions\\Listener\\LoadAdditionalListener' => $baseDir . '/../lib/Listener/LoadAdditionalListener.php', 'OCA\\Files_Versions\\Listener\\LoadSidebarListener' => $baseDir . '/../lib/Listener/LoadSidebarListener.php', + 'OCA\\Files_Versions\\Listener\\RegisterWorkflowIntegrationListener' => $baseDir . '/../lib/Listener/RegisterWorkflowIntegrationListener.php', 'OCA\\Files_Versions\\Listener\\VersionAuthorListener' => $baseDir . '/../lib/Listener/VersionAuthorListener.php', 'OCA\\Files_Versions\\Listener\\VersionStorageMoveListener' => $baseDir . '/../lib/Listener/VersionStorageMoveListener.php', 'OCA\\Files_Versions\\Migration\\Version1020Date20221114144058' => $baseDir . '/../lib/Migration/Version1020Date20221114144058.php', + 'OCA\\Files_Versions\\Operation' => $baseDir . '/../lib/Operation.php', 'OCA\\Files_Versions\\Sabre\\Plugin' => $baseDir . '/../lib/Sabre/Plugin.php', 'OCA\\Files_Versions\\Sabre\\RestoreFolder' => $baseDir . '/../lib/Sabre/RestoreFolder.php', 'OCA\\Files_Versions\\Sabre\\RootCollection' => $baseDir . '/../lib/Sabre/RootCollection.php', diff --git a/apps/files_versions/composer/composer/autoload_static.php b/apps/files_versions/composer/composer/autoload_static.php index 060c062c9f6cc..5c9867c42aec5 100644 --- a/apps/files_versions/composer/composer/autoload_static.php +++ b/apps/files_versions/composer/composer/autoload_static.php @@ -35,13 +35,16 @@ class ComposerStaticInitFiles_Versions 'OCA\\Files_Versions\\Events\\VersionCreatedEvent' => __DIR__ . '/..' . '/../lib/Events/VersionCreatedEvent.php', 'OCA\\Files_Versions\\Events\\VersionRestoredEvent' => __DIR__ . '/..' . '/../lib/Events/VersionRestoredEvent.php', 'OCA\\Files_Versions\\Expiration' => __DIR__ . '/..' . '/../lib/Expiration.php', + 'OCA\\Files_Versions\\Listener\\CreateVersionListenerForWorkflow' => __DIR__ . '/..' . '/../lib/Listener/CreateVersionListenerForWorkflow.php', 'OCA\\Files_Versions\\Listener\\FileEventsListener' => __DIR__ . '/..' . '/../lib/Listener/FileEventsListener.php', 'OCA\\Files_Versions\\Listener\\LegacyRollbackListener' => __DIR__ . '/..' . '/../lib/Listener/LegacyRollbackListener.php', 'OCA\\Files_Versions\\Listener\\LoadAdditionalListener' => __DIR__ . '/..' . '/../lib/Listener/LoadAdditionalListener.php', 'OCA\\Files_Versions\\Listener\\LoadSidebarListener' => __DIR__ . '/..' . '/../lib/Listener/LoadSidebarListener.php', + 'OCA\\Files_Versions\\Listener\\RegisterWorkflowIntegrationListener' => __DIR__ . '/..' . '/../lib/Listener/RegisterWorkflowIntegrationListener.php', 'OCA\\Files_Versions\\Listener\\VersionAuthorListener' => __DIR__ . '/..' . '/../lib/Listener/VersionAuthorListener.php', 'OCA\\Files_Versions\\Listener\\VersionStorageMoveListener' => __DIR__ . '/..' . '/../lib/Listener/VersionStorageMoveListener.php', 'OCA\\Files_Versions\\Migration\\Version1020Date20221114144058' => __DIR__ . '/..' . '/../lib/Migration/Version1020Date20221114144058.php', + 'OCA\\Files_Versions\\Operation' => __DIR__ . '/..' . '/../lib/Operation.php', 'OCA\\Files_Versions\\Sabre\\Plugin' => __DIR__ . '/..' . '/../lib/Sabre/Plugin.php', 'OCA\\Files_Versions\\Sabre\\RestoreFolder' => __DIR__ . '/..' . '/../lib/Sabre/RestoreFolder.php', 'OCA\\Files_Versions\\Sabre\\RootCollection' => __DIR__ . '/..' . '/../lib/Sabre/RootCollection.php', diff --git a/apps/files_versions/lib/AppInfo/Application.php b/apps/files_versions/lib/AppInfo/Application.php index 2915827641500..bf8c96abf2998 100644 --- a/apps/files_versions/lib/AppInfo/Application.php +++ b/apps/files_versions/lib/AppInfo/Application.php @@ -13,11 +13,14 @@ use OCA\Files\Event\LoadAdditionalScriptsEvent; use OCA\Files\Event\LoadSidebar; use OCA\Files_Versions\Capabilities; +use OCA\Files_Versions\Events\CreateVersionEvent; use OCA\Files_Versions\Events\VersionRestoredEvent; +use OCA\Files_Versions\Listener\CreateVersionListenerForWorkflow; use OCA\Files_Versions\Listener\FileEventsListener; use OCA\Files_Versions\Listener\LegacyRollbackListener; use OCA\Files_Versions\Listener\LoadAdditionalListener; use OCA\Files_Versions\Listener\LoadSidebarListener; +use OCA\Files_Versions\Listener\RegisterWorkflowIntegrationListener; use OCA\Files_Versions\Listener\VersionAuthorListener; use OCA\Files_Versions\Listener\VersionStorageMoveListener; use OCA\Files_Versions\Versions\IVersionManager; @@ -47,6 +50,8 @@ use OCP\L10N\IFactory; use OCP\Server; use OCP\Share\IManager as IShareManager; +use OCP\WorkflowEngine\Events\LoadSettingsScriptsEvent; +use OCP\WorkflowEngine\Events\RegisterOperationsEvent; use Psr\Container\ContainerInterface; use Psr\Log\LoggerInterface; @@ -110,8 +115,12 @@ public function register(IRegistrationContext $context): void { // we add the version author listener with lower priority to make sure new versions already are created by FileEventsListener $context->registerEventListener(NodeWrittenEvent::class, VersionAuthorListener::class, -1); - $context->registerEventListener(VersionRestoredEvent::class, LegacyRollbackListener::class); + + // WFE integration + $context->registerEventListener(RegisterOperationsEvent::class, RegisterWorkflowIntegrationListener::class); + $context->registerEventListener(LoadSettingsScriptsEvent::class, RegisterWorkflowIntegrationListener::class); + $context->registerEventListener(CreateVersionEvent::class, CreateVersionListenerForWorkflow::class); } public function boot(IBootContext $context): void { diff --git a/apps/files_versions/lib/BlockVersioningOperation.php b/apps/files_versions/lib/BlockVersioningOperation.php new file mode 100644 index 0000000000000..42016403eb2ea --- /dev/null +++ b/apps/files_versions/lib/BlockVersioningOperation.php @@ -0,0 +1,95 @@ +l10n->t('Block versioning'); + } + + #[\Override] + public function getDescription(): string { + return $this->l10n->t('Automatic tag based blocking of files version creation.'); + } + + #[\Override] + public function getIcon(): string { + return $this->urlGenerator->imagePath('files_versions', 'app.svg'); + } + + #[\Override] + public function isAvailableForScope(int $scope): bool { + return $scope === IManager::SCOPE_ADMIN; + } + + #[\Override] + public function validateOperation(string $name, array $checks, string $operation): void { + if (empty($checks)) { + throw new \UnexpectedValueException($this->l10n->t('No rule given')); + } + } + + #[\Override] + public function onEvent(string $eventName, Event $event, IRuleMatcher $ruleMatcher): void { + if ($eventName !== CreateVersionEvent::class || !($event instanceof CreateVersionEvent)) { + return; + } + + $node = $event->getNode(); + $path = $node->getInternalPath(); + + $ruleMatcher->setFileInfo( + $node->getStorage(), + $path, + $node instanceof Folder, + ); + $ruleMatcher->setEntitySubject($this->fileEntity, $node); + $ruleMatcher->setOperation($this); + $flows = $ruleMatcher->getFlows(); + + if ($flows !== []) { + $this->logger->debug('Blocking version creation due to matching workflow rules', [ + 'path' => $path, + ]); + $event->disableVersions(); + } + } + + #[\Override] + public function getTriggerHint(): string { + return $this->l10n->t('A new version is created'); // TRANSLATORS: This will be shown as "When: " "A new version is created" + } +} diff --git a/apps/files_versions/lib/Listener/CreateVersionListenerForWorkflow.php b/apps/files_versions/lib/Listener/CreateVersionListenerForWorkflow.php new file mode 100644 index 0000000000000..66a4105ecac5a --- /dev/null +++ b/apps/files_versions/lib/Listener/CreateVersionListenerForWorkflow.php @@ -0,0 +1,37 @@ + */ +class CreateVersionListenerForWorkflow implements IEventListener { + + public function __construct( + private IManager $manager, + private BlockVersioningOperation $operation, + ) { + } + + public function handle(Event $event): void { + if (!($event instanceof CreateVersionEvent)) { + return; + } + + $this->operation->onEvent( + $event::class, + $event, + $this->manager->getRuleMatcher(), + ); + } +} diff --git a/apps/files_versions/lib/Listener/RegisterWorkflowIntegrationListener.php b/apps/files_versions/lib/Listener/RegisterWorkflowIntegrationListener.php new file mode 100644 index 0000000000000..8da2b2e05c228 --- /dev/null +++ b/apps/files_versions/lib/Listener/RegisterWorkflowIntegrationListener.php @@ -0,0 +1,33 @@ + */ +class RegisterWorkflowIntegrationListener implements IEventListener { + + public function __construct( + private readonly BlockVersioningOperation $operation, + ) { + } + + public function handle(Event $event): void { + if ($event instanceof RegisterOperationsEvent) { + $event->registerOperation($this->operation); + } elseif ($event instanceof LoadSettingsScriptsEvent) { + Util::addScript('files_versions', 'workflow', 'workflowengine'); + } + } +} diff --git a/apps/files_versions/src/workflow.ts b/apps/files_versions/src/workflow.ts new file mode 100644 index 0000000000000..f3ffe6b6c8c08 --- /dev/null +++ b/apps/files_versions/src/workflow.ts @@ -0,0 +1,12 @@ +/*! + * SPDX-FileCopyrightText: 2026 Nextcloud GmbH and Nextcloud contributors + * SPDX-License-Identifier: AGPL-3.0-or-later + */ + +window.addEventListener('DOMContentLoaded', () => { + globalThis.OCA.WorkflowEngine.registerOperator({ + id: 'OCA\\Files_Versions\\BlockVersioningOperation', + color: '#ff5900', + operation: 'deny', + }) +}) diff --git a/build/frontend/vite.config.ts b/build/frontend/vite.config.ts index d8686121291d7..f0638262d4933 100644 --- a/build/frontend/vite.config.ts +++ b/build/frontend/vite.config.ts @@ -45,6 +45,7 @@ const modules = { }, files_versions: { 'sidebar-tab': resolve(import.meta.dirname, 'apps/files_versions/src', 'sidebar_tab.ts'), + workflow: resolve(import.meta.dirname, 'apps/files_versions/src', 'workflow.ts'), }, oauth2: { 'settings-admin': resolve(import.meta.dirname, 'apps/oauth2/src', 'settings-admin.ts'), From 88d7fc9162565afd4bab6e6b73937e49cc3d6f9b Mon Sep 17 00:00:00 2001 From: Ferdinand Thiessen Date: Mon, 13 Apr 2026 16:11:31 +0200 Subject: [PATCH 2/2] fix(files_version): dispatch `CreateVersionEvent` as typed event Signed-off-by: Ferdinand Thiessen --- apps/files_versions/lib/Storage.php | 1 + 1 file changed, 1 insertion(+) diff --git a/apps/files_versions/lib/Storage.php b/apps/files_versions/lib/Storage.php index fe9f91534fa01..eaa242e98e138 100644 --- a/apps/files_versions/lib/Storage.php +++ b/apps/files_versions/lib/Storage.php @@ -205,6 +205,7 @@ public static function store($filename) { } $event = new CreateVersionEvent($file); + $eventDispatcher->dispatchTyped($event); $eventDispatcher->dispatch('OCA\Files_Versions::createVersion', $event); if ($event->shouldCreateVersion() === false) { return false;