Wrong UID dispatched on the EventBus for StoreMessageIdManager::setInMailboxes

Looking at the (internal) linagora bugtracker for the James server I found:

Given msg1 in mbx1 with uid 12 and in mbx2 with uid 58
When I set it in mbx3
Then two deletion events are dispatched, one in mbx1 with uid 12 ok, one in mbx2 and uid 12.

Which can lead to random message deletion in the search index (for instance) when a message is removed from more than one mailbox in JMAP.

I then took the few minutes to add a test case and fix the logic.

I should definitely audit our internal bug tracker and open relevant bugs here too, so that people can help and fix them.