added Sieve getMatchTypes() for easy valid match-types

dev/Sieve/Grammar.js

@@ -9,9 +9,7 @@ import {
 	MULTILINE_DOTSTART
 } from 'Sieve/RegEx';
 
-import {
+import { arrayToString, getMatchTypes } from 'Sieve/Utils';
-	arrayToString
-} from 'Sieve/Utils';
 
 /**
  * abstract
@@ -240,12 +238,10 @@ export class GrammarTest
 
 	toString()
 	{
-/*
 		// https://datatracker.ietf.org/doc/html/rfc6134#section-2.3
-		if (':list' == this.match_type && !capa.includes('extlists')) {
+		if (!getMatchTypes().includes(this.match_type)) {
 			throw 'Unsupported match-type ' + this.match_type;
 		}
-*/
 		return (this.identifier
 			+ (this.comparator ? ' :comparator ' + this.comparator : '')
 			+ (this.match_type ? ' ' + this.match_type : '')

dev/Sieve/Parser.js

@@ -2,7 +2,7 @@
  * https://tools.ietf.org/html/rfc5228#section-8
  */
 
-import { capa } from 'Sieve/Utils';
+import { capa, getMatchTypes } from 'Sieve/Utils';
 
 import {
 	BRACKET_COMMENT,
@@ -213,11 +213,7 @@ export const parseScript = (script, name = 'script.sieve') => {
 		pushArg = arg => {
 			command || error('Argument not part of command');
 			let prev_arg = args[args.length-1];
-/*
+			if (getMatchTypes(0).includes(arg)) {
-			// https://datatracker.ietf.org/doc/html/rfc6134#section-2.3
-			if (':is' === arg || ':contains' === arg || ':matches' === arg || (capa.includes('extlists') && ':list')) {
-*/
-			if (':is' === arg || ':contains' === arg || ':matches' === arg) {
 				command.match_type = arg;
 			} else if (':value' === prev_arg || ':count' === prev_arg) {
 				// Sieve relational [RFC5231] match types

dev/Sieve/Utils.js

@@ -56,4 +56,13 @@ export const
 	setError = text => {
 		serverError(true);
 		serverErrorDesc(text);
+	},
+
+	getMatchTypes = (validOnly = 1) => {
+		let result = [':is',':contains',':matches'];
+		// https://datatracker.ietf.org/doc/html/rfc6134#section-2.3
+		if (capa.includes('extlists') || !validOnly) {
+			result.push(':list');
+		}
+		return result;
 	};