[Feature]Always mark read when archiving email messages

[shamim-emon]

• Always mark read when archiving email messages.

• Fixed Always mark read when archiving email messages #8309

[cketti]

Please check this comment for additional requirements for this behavior change.

Unfortunately, this change doesn't achieve the desired effect.

1. MessagingController.moveOrCopyMessageSynchronous() is used for all move operations, not just for moving a message to the archive folder. However, we don't want all moved messages to automatically be marked as read.
2. Calling setFlag() on a LocalMessage instance will only mark the message as read locally, not on the server.

MessagingController always changes the local message state first, then enqueues a "command" that will perform the message changes on the server. By separating local changes from remote changes the app can be used offline. When connectivity returns the queue of remote operations will be processed.

There's already MoveOrCopyFlavor.MOVE_AND_MARK_AS_READ. Previously, there was no reason for MessagingController.moveOrCopyMessageSynchronous() to support this functionality. Now, adding support for MOVE_AND_MARK_AS_READ to this method would make it trivial to then change the behavior of only the archive action.

Please feel free to ask in the developer Matrix room if you have questions on any of this.