fix: Remove non-standard junk flag used by Thunderbird as it's moved to own PR

Signed-off-by: David Dreschner <[email protected]>

Author: DerDreschner

lib/Model/IMAPMessage.php

@@ -535,7 +535,7 @@ public function toDbMessage(int $mailboxId, MailAccount $account): Message {
 		$msg->setFlagJunk(
 			in_array(Horde_Imap_Client::FLAG_JUNK, $flags, true)
 			|| in_array('junk', $flags, true)
-		); // While this is not a standard IMAP flag, Thunderbird uses it to mark "junk"
+		);
 		$msg->setFlagNotjunk(in_array(Horde_Imap_Client::FLAG_NOTJUNK, $flags, true) || in_array('nonjunk', $flags, true));// While this is not a standard IMAP Flag, Thunderbird uses it to mark "not junk"
 		$msg->setFlagImportant(in_array('$important', $flags, true) || in_array('$labelimportant', $flags, true) || in_array(Tag::LABEL_IMPORTANT, $flags, true));
 		$msg->setFlagAttachments(false);
@@ -551,7 +551,6 @@ public function toDbMessage(int $mailboxId, MailAccount $account): Message {
 			Horde_Imap_Client::FLAG_DELETED,
 			Horde_Imap_Client::FLAG_DRAFT,
 			Horde_Imap_Client::FLAG_JUNK,
-			'junk', // While this is not a standard IMAP flag, Thunderbird uses it to mark "junk"
 			Horde_Imap_Client::FLAG_NOTJUNK,
 			'nonjunk', // While this is not a standard IMAP Flag, Thunderbird uses it to mark "not junk"
 			'$phishing', // Horde has no const for this flag yet