diff options
author | Tim Culverhouse <tim@timculverhouse.com> | 2022-08-16 16:23:40 -0500 |
---|---|---|
committer | Robin Jarry <robin@jarry.cc> | 2022-08-22 15:46:54 +0200 |
commit | af72ca36072615f3afc79d7a3f28f82e381b618c (patch) | |
tree | 1c5f4353568cc683b8fb72d323797bc9b591ea3c /worker/maildir/container.go | |
parent | 64e1a7ca933b404d3556e776d0373069c1a0b763 (diff) | |
download | aerc-af72ca36072615f3afc79d7a3f28f82e381b618c.tar.gz |
maildir: implement MoveMessages handling
Implement MoveMessages in the maildir worker. go-maildir supports Move
operations by default, and is functionally equivalent to a OS-level
rename. Creation date of the file is preserved by using Move, which is
used by at least one maildir-to-IMAP synchronizer (isync/mbsync). The
previous move method of copy-and-delete would reset the creation date of
the message, and potentially cause sorting issues in other email
clients.
Signed-off-by: Tim Culverhouse <tim@timculverhouse.com>
Acked-by: Robin Jarry <robin@jarry.cc>
Diffstat (limited to 'worker/maildir/container.go')
-rw-r--r-- | worker/maildir/container.go | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/worker/maildir/container.go b/worker/maildir/container.go index a8ac1aca..c5ac04ac 100644 --- a/worker/maildir/container.go +++ b/worker/maildir/container.go @@ -205,3 +205,23 @@ func (c *Container) copyMessage( _, err := src.Copy(dest, key) return err } + +func (c *Container) MoveAll(dest maildir.Dir, src maildir.Dir, uids []uint32) ([]uint32, error) { + var success []uint32 + for _, uid := range uids { + if err := c.moveMessage(dest, src, uid); err != nil { + return success, fmt.Errorf("could not move message %d: %w", uid, err) + } + success = append(success, uid) + } + return success, nil +} + +func (c *Container) moveMessage(dest maildir.Dir, src maildir.Dir, uid uint32) error { + key, ok := c.uids.GetKey(uid) + if !ok { + return fmt.Errorf("could not find key for message id %d", uid) + } + err := src.Move(dest, key) + return err +} |