azerothcore / azerothcore-wotlk

Complete Open Source and Modular solution for MMO
http://www.azerothcore.org
GNU Affero General Public License v3.0
6.39k stars 2.55k forks source link

Feature: Auction House getAll scan #5068

Open eTzmNcbkrng opened 3 years ago

eTzmNcbkrng commented 3 years ago

Describe your feature request or suggestion in detail Add the Auction House feature to run a getAll scan which was added to WoW in 2.3

Describe a possible solution to your Feature or Suggestion in detail https://github.com/TrinityCore/TrinityCore/commit/3aaeb574050668e5a240078f6e40337c3975d110

Additional context

getAll

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/97541663-feature-auction-house-getall-scan?utm_campaign=plugin&utm_content=tracker%2F40032087&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F40032087&utm_medium=issues&utm_source=github).
necropola commented 3 years ago

I'd really like to have AuctionHouse Search with GetAll working on AC and the commit from TC would apply with very little modifications, if there weren't these weird pussywizard modifications: https://github.com/azerothcore/azerothcore-wotlk/blob/5906222fd6910beec4b5d6c07f4c69f69bb094a4/src/server/game/AuctionHouse/AuctionHouseMgr.cpp#L550-L585

It looks like the initial import into github already had these special modifications (which are spread over various parts of the code including worldserver/Master), but so far nobody touched it, not even to fix obvious typos.
https://github.com/azerothcore/azerothcore-wotlk/blob/5906222fd6910beec4b5d6c07f4c69f69bb094a4/src/server/worldserver/Master.cpp#L192-L194

Does any of the veteran devs know/remember, why this sepcial code is needed in AC?

tystu commented 3 years ago

Just wanted to chime in and add my support for this feature. Here's what I wrote on #Suggestions on discord:

AH API Suggestion: Have AzerothCore support the "GetAll" one-click AH scan. This is a type of all-in-one auction house scan / API dump (sorry if I'm not using the best terms for this) that blizzard implemented to allow faster scanning of the AH and lesser load on the servers.

Problem: Right now, ChromieCraft generally averages around 100 AH pages. AH addons like TSM, Auctioneer, etc can scan the Auction House page by page, but this currently takes about 5 minutes to complete a full scan. In addition, I'm sure it increases the server load as each page has to be pulled individually.

Benefits: My past experience with the "GetAll" scan on blizzard live servers is that it typically takes less than 30 seconds in one large information dump, also lessening server load.

Razorce commented 3 years ago

I have something to add, as stated in the Issues I posted... When I first started chromie, I did my first 3-4 scans no problem, then later it stopped working. But as I understand, it wasn't supposed to work at all? I started on this same bracket, so nothing really should have changed

Azcobu commented 3 years ago

A great deal more detailed information from @VFaleiro on this here: https://github.com/chromiecraft/chromiecraft/issues/1393

alchem1ster commented 2 years ago

@Azcobu any updates? Still doesn't fixed. Tested today on ChromieCraft.

Azcobu commented 2 years ago

I haven't worked on the project for some time, so have no other information regarding the status of this issue beyond what is linked here.