You must login to post a comment. Don't have an account? Register to get one!

• 

  • Reply
  • #5

  Xinhuan Dec 30, 2011 at 02:02 UTC - 0 likes

  I'm still getting some posts it wasn't fixed completely though. It's difficult to pin point how other unknown addons interact with the mailbox.

• 

  • Reply
  • #4

  KippG Dec 17, 2011 at 16:29 UTC - 0 likes

  I've been affected with the bug and I just wanted to let you know that first thing this morning after the update Postal refreshed and pullled the second page of mail without a problem. I have two addons that may have been looking at the mail and interfearing such as described here. "My Sales" and "LilSparky's Workshop".

  Thank you both for the fix.

• 

  • Reply
  • #3

  Xinhuan Dec 17, 2011 at 09:31 UTC - 0 likes

  Included the fix in 3.4.13.

• 

  • Reply
  • #2

  Xinhuan Dec 13, 2011 at 14:15 UTC - 0 likes

  I can certainly see no harm in adding that additional condition to check against 0. Thank you for your testing!

• 

  • Reply
  • #1

  paxi Nov 14, 2011 at 13:03 UTC - 0 likes

  I've been able to reproduce this using postal and Tradeskillmaster Mailing.

  1. Set up tradeskillmaster mailing to refresh the mailbox. /tsm go to mail options and check "Auto Recheck Mail"
  2. Load up the mailbox with 50+ items.
  3. Click Open All in postal

  What seems to happen is that while Postal is opening the mails Tradeskillmaster calls CheckInbox(). If this happens when postal has looted all mails there's a short "lag" period where there's new mail but they're not showing. This causes GetInboxHeaderInfo(1) to return 0 instead of the actual days left. This in turn causes Postal to think that a refresh has happened and it calls OpenAll. OpenAll tries to check the current number of mails but GetNumInboxItems() still returns 0 so OpenAll returns without re-enabling the timer.

  My solution for this is to change line 164 in OpenAll.lua from

  if currentFirstMailDaysLeft ~= firstMailDaysLeft then
  

  to

  if currentFirstMailDaysLeft ~= 0 and currentFirstMailDaysLeft ~= firstMailDaysLeft then
  

  this has solved it in my limited testing, I hope it's a decent solution and not just an ugly hack :)

  Last edited Dec 13, 2011 by paxi