TradeSkillMaster_Auctioning

438 - Problem posting items with random enchants.

What steps will reproduce the problem?
1.create a Category and set it up to Ignore Random Enchants
2.have one item with random enchant in your bag (for example: Bloodfist Helmet of the Eagle)
3.add item to a group
4.add group to category
5.first time you try to post in the AH, it will NOT take in consideration the Ignore Random Enchants flag from the category. You will notice that the name of the item on top of the window will be the full name (Bloodfist Helmet of the Eagle) AND the scan will not identify variants of the item (i.e. Bloodfist Helmet of Power, etc).
6.if you run the PostScan again, it will then take in consideration the Ignore Random Enchants flag from the category. You will notice that the name on top will show Bloodfist Helmet and all the different variants will be picked by the scanner.

If you /reload and try to post again the same item, it will go back to step 4 above.

What is the expected output? What do you see instead?
expected output: when you post an item with random enchant (of the...) and the flag is set to ignore random enchants, PostScan should find and list all the variants of the item.
what I see instead: the first time we run PostScan in a session it will not consider the ignore random enchants flag and will treat each variant of the item independently. If we run PostScan again during the same session, it will work as expected.

What version of the product are you using (put the actual number - never say "latest")?
v1.3

Do you have an error log of what happened?
No, the system does not output an error.

Please provide any additional information below.
I believe I found out the reason and solution:

/modules/manage.lua calls TSM:UpdateItemReverseLookup() and TSM:UpdateGroupReverseLookup() functions to build itemReverseTable and groupReverseTable, respectively.

The first function to run (TSM:UpdateItemReverseLookup()) calls TSM.Config:GetBoolConfigValue() which uses groupReverseTable. However groupReverseTable hasn't been built yet because it will be built by the second function (TSM:UpdateGroupReverseLookup()).

As a result, the first time you run the PostScan after logging in or /reload, the system will not have the groupReverseTable and it will not be able to pick the Ignore Random Enchant flags from the categories.

The second time you run the PostScan, the table will be there.

I believe a simple solution would be to swap the order of those functions in /modules/manage.lua Do TSM:UpdateGroupReverseLookup() before TSM:UpdateItemReverseLookup().

A workaround until this is fixed (in case this is actually a bug) would be to run PostScan two times. The first to build the table and the second to actually post the items.

User When Change
sapu94 Jul 28, 2013 at 22:42 UTC
j1curse Mar 27, 2013 at 19:39 UTC Create

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

  • 2 comments
  • Avatar of sapu94 sapu94 Jul 28, 2013 at 22:42 UTC - 0 likes

    Closing all 1.x tickets as 2.0 is now in public beta (via our forums).

    Lead Developer and Director of TradeSkillMaster | @Sapu94 on Twitter

    "A good programmer is someone who always looks both ways before crossing a one-way street." Doug Linder

  • Avatar of j1curse j1curse Mar 27, 2013 at 21:32 UTC - 0 likes

    two screenshots that will help explain the problem:

    The first time I try to post, the system do not recognize I set the flag to ignore random enchants in the category. It does not find any other auctions for the item:

    http://i.imgur.com/uod8jHp.jpg

    If I run another scan immediately after the first one it does recognize the flag and finds variants of the item:

    http://i.imgur.com/YgQ2NaB.jpg

    Thanks.

  • 2 comments

Facts

Last updated
Jul 28, 2013
Reported
Mar 27, 2013
Status
Declined - We decided not to take action on this ticket.
Type
Defect - A shortcoming, fault, or imperfection
Priority
Medium - Normal priority.
Votes
0

Reported by

Possible assignees