Unable to create a workflow based on mimetype and group membership

[kyteinsky]

How to use GitHub

• Please use the 👍 reaction to show that you are affected by the same issue.
• Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
• Subscribe to receive notifications on status change and new comments.

---

Steps to reproduce

1. Create a tag "deltag"
2. Create a group "autotag" and assign a few users to it
3. Create an external mount that is accessible to the group "autotag" (although this might not be relate)
4. Create a flow as shown below

Expected behaviour

Workflow is created.

Actual behaviour

Regex error from the "workflowengine" app. Maybe this issue belongs in the server?

{"reqId":"ifxsiY2XYeIasD9xp9PR","level":2,"time":"2025-12-17T10:22:11+00:00","remoteAddr":"192.168.21.8","user":"admin","app":"PHP","method":"POST","url":"/ocs/v2.php/apps/workflowengine/api/v1/workflows/global?format=json","message":"preg_match(): Delimiter must not be alphanumeric, backslash, or NUL at /var/www/html/apps/workflowengine/lib/Check/AbstractStringCheck.php#73","userAgent":"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/137.0.0.0 Safari/537.36","version":"32.0.3.2","data":{"app":"PHP"}}

Server configuration

Web server: Nginx

Database: Maria

PHP version: 8.3

Nextcloud version: 32.0.3

List of activated apps

If you have access to your command line run e.g.:
sudo -u www-data php occ app:list
from within your Nextcloud installation folder

Nextcloud configuration

If you have access to your command line run e.g.:
sudo -u www-data php occ config:list system
from within your Nextcloud installation folder

Browser

Browser name: Firefox/Chrome/Safari/…

Browser version: 124/125/…

Operating system: Windows/Ubuntu/Mac/…

Browser log

Insert your browser log here, this could for example include:
a) The javascript console log
b) The network log
c) ...

[kyteinsky]

the mimetype comparison should be "is" instead of "matches". It works then.
not sure if any validation can be done here to show an error message, or if the item "matches" can be removed from the list.
this issue can be closed, although a hint would be quite nice.

[nickvergessen]

Yeah that should be somehow added to Folder and PDF that it requires is/is-not
And for the others that they require match/match-not

server/apps/workflowengine/src/components/Checks/FileMimeType.vue

Lines 71 to 75 in 1588abe

	{
	icon: 'icon-folder',
	label: t('workflowengine', 'Folder'),
	id: 'httpd/unix-directory',
	},

But I have too little vue understanding to pass it up to the Check level.

I can only get it to show no more invalid operators:

But it doesn't change the currently selected one.

Diff would be something like:

diff --git a/apps/workflowengine/src/components/Checks/file.js b/apps/workflowengine/src/components/Checks/file.js
index ddfb41c59b8..f5ee4752d03 100644
--- a/apps/workflowengine/src/components/Checks/file.js
+++ b/apps/workflowengine/src/components/Checks/file.js
@@ -37,7 +37,21 @@ const FileChecks = [
        {
                class: 'OCA\\WorkflowEngine\\Check\\FileMimeType',
                name: t('workflowengine', 'File MIME type'),
-               operators: stringOrRegexOperators,
+               operators: (check) => {
+                       if (check.value === 'httpd/unix-directory' || check.value === 'application/pdf') {
+                               return [
+                                       { operator: 'is', name: t('workflowengine', 'is') },
+                                       { operator: '!is', name: t('workflowengine', 'is not') },
+                               ]
+                       }
+                       if (check.value[0] === '/') {
+                               return [
+                                       { operator: 'matches', name: t('workflowengine', 'matches') },
+                                       { operator: '!matches', name: t('workflowengine', 'does not match') },
+                               ]
+                       }
+                       return stringOrRegexOperators
+               },
                element: registerCustomElement(FileMimeType, 'oca-workflowengine-checks-file_mime_type'),
        },
 

[kyteinsky]

noice, that is already quite better!

the default selection "is" is loaded when the Check component is mounted it seems:

server/apps/workflowengine/src/components/Check.vue

Line 157 in 154f359

this.currentOperator = this.operators.find((operator) => operator.operator === this.check.operator)

it should be updated when the mimetype is selected as folder/pdf.
but since the flow is kind of backward now, selecting "File MIME Type" then "matches" and then the right most "Folder" or "PDF documents". Now the selection in the opposite direction (flow, if you will) has changed which is not very intuitive.

maybe in addition to/instead of changing the options, we can add a validate key in file.js, in the same object in the diff above that explains the issue, not sure. Feedback from a designer may be required it seems 😅

[come-nc]

Related: #23666

[come-nc]

Would it be hard to only allow matches (not) and turn all options into regular expressions?
But then we would still have the issue with custom, that it needs to be a regex and people are not familiar with them.