BagSync

BagSync

3M Downloads

Seafarer's Doubloons count incorrect.

Sinaloit opened this issue ยท 8 comments

commented

Seafarer's Doubloons appear to only be listing the leftmost digit in the character tooltip. Example I have 296 but the tooltip indicates the character has 2. I also do not see Seafarer's Doubloons at all in the BagSync - Currency window.

commented

I'll take a look at this and see if I can reproduce the error. It may be an issue with how the itemid/itemlink is done with this particular item.

commented

I was under the impression that Doubloons don't go into your inventory but to your currency window instead. So the tooltip is only shown for them when you hover over it in the currency window. I'm assuming what you are saying is that the display is both in the official currency window and in the BagSync currency display? Could you provide a screenshot? I will try to replicate this when I get home.

commented

Yes, Doubloons do not go into inventory so you are correct the tooltip I am referring to is from the currency window.
image

The BagSync - Currency window that does not show them at all is this one:
image

commented

I'll provide a screenshot as well, sorry it's in German. Seafarer as well as Corrupted Memento and Brawlers Gold seem to be buggy or there is a problem with the database in general.
The screen shows my character having 523 dubloons, but the BagSync tooltip says 6 (for all chars by the way). For Corrupted Memento I have 398, tooltip says 0.
WoWScrnShot_012020_203843

commented

Oddly my corrupted mementos tooltip appears to be correct. I'm guessing brawlers gold is correct but I haven't participated in the current season yet so am unsure if anything would change.

commented

After looking at those screenshots, I suspect the issue is something related to localization and itemID storing.

So lets take a look at how things are being stored.

Please send me a PM (Private Message) on WOWInterface or wow.curse.com
In that message, if you could please attach or include in a zip file your BagSync.lua savedvariables.

You can find it in the following location C:\YOUR WOW INSTALL DIR_retail\WTF\YOUR ACCOUNT NAME\SavedVariables\BagSync.lua

I can use that file to determine how things are being stored and interpreted on your end. It may help me solve the issue.

commented

That GitGist helped tremendously.  I found the culprit!

Looks like multiple currency is sharing the same iconID but have a different currencyID.  Figures that Blizzard would do this.  Their code can be sloppy sometimes.

So doubloons are listed as 1604167.  But if you notice Seal of Broken Fate are also 1604167.  That is why it is only showing you such tiny numbers.  It's overwriting the doubloons.  This is a major issue.  

I'll need to code a way around this. I'll be working on a fix.

commented

Issue has been fixed in the latest commit above. Will be released with next version push.