From f4a930cd195cfd54f930470958b54222b2089e2c Mon Sep 17 00:00:00 2001
From: "Dirk-Jan C. Binnema" <djcb@djcbsoftware.nl>
Date: Mon, 9 Oct 2023 23:47:06 +0300
Subject: [PATCH] lib: query-processor: fix handling unknown flags

Fixes #2567.
---
 lib/mu-query-processor.cc | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/lib/mu-query-processor.cc b/lib/mu-query-processor.cc
index 80aa3402..592beb41 100644
--- a/lib/mu-query-processor.cc
+++ b/lib/mu-query-processor.cc
@@ -311,10 +311,10 @@ basify(Element&& element)
 		auto val{str->substr(pos + 1)};
 		if (field == Field::Id::Flags) {
 			if (auto&& finfo{flag_info(val)}; finfo)
-				element.value =  Element::Basic{field->name,
+				element.value = Element::Basic{field->name,
 					std::string{finfo->name}};
 			else
-				Element::Basic{*str};
+				element.value = Element::Basic{*str};
 		} else if (field == Field::Id::Priority) {
 			if (auto&& prio{priority_from_name(val)}; prio)
 				element.value = Element::Basic{field->name,
@@ -444,7 +444,6 @@ process(const std::string& expr)
 			.and_then(opify)
 			.and_then(basify)
 			.and_then(regexpify)
-			//.and_then(phrasify)
 			.and_then(wildcardify)
 			.and_then(rangify);
 		if (element)
@@ -490,6 +489,7 @@ main (int argc, char *argv[])
 
 #if BUILD_TESTS
 /*
+ *
  * Tests.
  *
  */
@@ -505,6 +505,7 @@ test_processor()
 		// basics
 		TestCase{R"(hello world)", R"(((_ "hello") (_ "world")))"},
 		TestCase{R"(maildir:/"hello world")", R"(((maildir "/hello world")))"},
+		TestCase{R"(flag:deleted)", R"(((_ "flag:deleted")))"} // non-existing flags
 	};
 
 	for (auto&& test: cases) {