From ac83ff755f42daacdec57b687122218d5aa58679 Mon Sep 17 00:00:00 2001
From: SebastianKrupinski <krupinskis05@gmail.com>
Date: Thu, 6 Jun 2024 16:06:14 -0400
Subject: [PATCH] feature: mail provider backend

Signed-off-by: SebastianKrupinski <krupinskis05@gmail.com>
---
 lib/AppInfo/Application.php          |  5 +++-
 lib/Provider/Command/MessageSend.php | 34 +++++++++++-----------------
 2 files changed, 17 insertions(+), 22 deletions(-)

diff --git a/lib/AppInfo/Application.php b/lib/AppInfo/Application.php
index f4b99af101..371c5de5ff 100644
--- a/lib/AppInfo/Application.php
+++ b/lib/AppInfo/Application.php
@@ -177,7 +177,10 @@ public function register(IRegistrationContext $context): void {
 		}
 
 		// register mail provider
-		$context->registerMailProvider(MailProvider::class);
+		// TODO: drop condition if nextcloud < 30 is not supported anymore
+		if (interface_exists(MailProvider::class)) {
+			$context->registerMailProvider(MailProvider::class);
+		}
 
 		$context->registerNotifierService(Notifier::class);
 
diff --git a/lib/Provider/Command/MessageSend.php b/lib/Provider/Command/MessageSend.php
index 4e519ffeb5..234b117ff4 100644
--- a/lib/Provider/Command/MessageSend.php
+++ b/lib/Provider/Command/MessageSend.php
@@ -30,20 +30,18 @@
 use OCA\Mail\Service\Attachment\AttachmentService;
 use OCA\Mail\Service\OutboxService;
 use OCA\Mail\Service\SmimeService;
+use OCP\IConfig;
 use OCP\Mail\Provider\IMessage;
 
 class MessageSend {
 
 	public function __construct(
-		AccountService $accountService,
-		OutboxService $outboxService,
-		AttachmentService $attachmentService,
-		SmimeService $smimeService
+		protected IConfig $config,
+		protected AccountService $accountService,
+		protected OutboxService $outboxService,
+		protected AttachmentService $attachmentService,
+		protected SmimeService $smimeService
 	) {
-		$this->accountService = $accountService;
-		$this->outboxService = $outboxService;
-		$this->attachmentService = $attachmentService;
-		$this->smimeService = $smimeService;
 	}
 
 	public function perform(string $userId, string $serviceId, IMessage $message, array $option = []): void {
@@ -55,20 +53,9 @@ public function perform(string $userId, string $serviceId, IMessage $message, ar
 		$lm->setAccountId($account->getId());
 		$lm->setSubject($message->getSubject());
 		$lm->setBody($message->getBody());
-		//$lm->setEditorBody($editorBody);
 		$lm->setHtml(true);
-		//$lm->setInReplyToMessageId($inReplyToMessageId);
 		$lm->setSendAt(time());
-		//$lm->setSmimeSign($smimeSign);
-		//$lm->setSmimeEncrypt($smimeEncrypt);
-
-		/*
-		if (!empty($smimeCertificateId)) {
-			$smimeCertificate = $this->smimeService->findCertificate($smimeCertificateId, $this->userId);
-			$lm->setSmimeCertificateId($smimeCertificate->getId());
-		}
-		*/
-
+		
 		// convert all mail provider attachments to local attachments
 		$attachments = [];
 		if (count($message->getAttachments()) > 0) {
@@ -86,7 +73,7 @@ public function perform(string $userId, string $serviceId, IMessage $message, ar
 		$to = $this->convertAddressArray($message->getTo());
 		$cc = $this->convertAddressArray($message->getCc());
 		$bcc = $this->convertAddressArray($message->getBcc());
-		// save/send message
+		// save message for sending
 		$lm = $this->outboxService->saveMessage(
 			$account,
 			$lm,
@@ -96,6 +83,11 @@ public function perform(string $userId, string $serviceId, IMessage $message, ar
 			$attachments
 		);
 
+		// evaluate if job scheduler is NOT cron, send message right away otherwise let cron job handle it
+		if ($this->config->getAppValue('core', 'backgroundjobs_mode', 'ajax') !== 'cron') {
+			$lm = $this->outboxService->sendMessage($lm, $account);
+		}
+
 	}
 
 	protected function convertAddressArray(array|null $in) {