search

alarofrunetotem / MailCommander

5 stars 2 forks source link

"Send All" button may send items to wrong character #8

Closed B88000005 closed 1 year ago

B88000005 commented 3 years ago

"Send All" button may send items to wrong character.

B88000005 commented 3 years ago

I just debug it. The sender is set after the addon put items to the mail. However, the currentReceiver will change if we put the items to the mail. The scenario of this bug is:

  1. currentReceiver=alt1
  2. sender="working"
  3. put the items which should be sent to alt1 to the mail
  4. currentReceiver=alt2, because the addon already put items to the mail
  5. sender=currentReceiver (which is alt2)
  6. the addon send items to the wrong character.