Postal

58 - Auto-refresh upon 50+ mails

Submitted Via

Curse Client 4.0.1.104, Microsoft Windows NT 6.1.7601 Service Pack 1

Installed Version

v3.4.9

Description

Checked all settings, but the addon don't automatically refresh mail box, to claim other items that were 51st and further

Installed Addons

_NPCScan - _NPCScan 4.2.0.1 Auctioneer - AuctioneerSuite 5.11.5146 Aurora - Aurora-r75742-nolib Cellular - v4.2.001 KeyBinder - KeyBind Postal - v3.4.9 Tol Barad Master - Tol Barad Master v1.2

User When Change
Xinhuan Jul 02, 2011 at 17:01 UTC
Ferari Jul 01, 2011 at 20:08 UTC Create

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

  • 5 comments
  • Avatar of Xinhuan 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.

  • Avatar of KippG 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.

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

    Included the fix in 3.4.13.

  • Avatar of Xinhuan 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!

  • Avatar of paxi 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
  • 5 comments

Facts

Last updated
Jul 02, 2011
Reported
Jul 01, 2011
Status
Accepted - Problem reproduced / Need acknowledged.
Type
Task - Work item that doesn't change the code or docs
Priority
Medium - Normal priority.
Votes
0

Reported by

Possible assignees