Add a button to force sync mailboxes

[miaulalala]

Currently, there is a two hour delay when creating a mailbox or folder in another mail program before it appears in Mail.

Add a button that force syncs mailboxes and folders from IMAP.

[miaulalala]

Fixes #4869, #5164

[kesselb]

I run into this "issue" today. I knew that a mailbox should be there so I tried to create the folder again to trigger a sync.

Unfortunately that does not work for now.

Index: lib/IMAP/FolderMapper.php
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/lib/IMAP/FolderMapper.php b/lib/IMAP/FolderMapper.php
--- a/lib/IMAP/FolderMapper.php	(revision 80b6634c52accb49c0bea94c22dedea51615ed75)
+++ b/lib/IMAP/FolderMapper.php	(date 1623745385082)
@@ -23,16 +23,16 @@
 
 namespace OCA\Mail\IMAP;
 
-use function array_filter;
-use function array_map;
-use function in_array;
-use function reset;
 use Horde_Imap_Client;
 use Horde_Imap_Client_Exception;
 use Horde_Imap_Client_Socket;
 use OCA\Mail\Account;
 use OCA\Mail\Exception\ServiceException;
 use OCA\Mail\Folder;
+use function array_filter;
+use function array_map;
+use function in_array;
+use function reset;
 
 class FolderMapper {
 
@@ -92,7 +92,14 @@
 	public function createFolder(Horde_Imap_Client_Socket $client,
 								 Account $account,
 								 string $name): Folder {
-		$client->createMailbox($name);
+
+		try {
+			$client->createMailbox($name);
+		} catch (Horde_Imap_Client_Exception $e) {
+			if ($e->getCode() !== 22) {
+				throw $e;
+			}
+		}
 
 		$list = $client->listMailboxes($name, Horde_Imap_Client::MBOX_ALL_SUBSCRIBED, [
 			'delimiter' => true,

If we catch the error "mailbox already exist" like above it seems to work. I'm not sure if there are side effect 🤷‍♂️

[StCyr]

I think it's there already in the master branch:

[ChristophWurst]

That button is to refresh the current mailbox. But what we want additionally is a way to refresh the list of mailboxes.

[mat-m]

@ChristophWurst Do you confirm it will also work for folder removal ?

[ChristophWurst]

It will