Skip to content

Fix ahbot bids no longer saved to the database issue#141 #142

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
Feb 26, 2025

Conversation

Jaytapp
Copy link

@Jaytapp Jaytapp commented Feb 26, 2025

Changes Proposed:

  • Properly commit database transaction after the bot bid on item

Issues Addressed:

SOURCE:

#141

Tests Performed:

  • Put item in AH, wait for bot to bid.
  • check UI and DB has bid
  • shutdown server
  • start server again, check AH bid was still there

How to Test the Changes:

  • Put item in AH, wait for bot to bid.
  • check UI and DB has bid
  • shutdown server
  • start server again, check AH bid was still there

image

Jaytapp and others added 15 commits January 15, 2025 19:33
…\mod-ah-bot

azerothcore#129

remove break in AuctionHouseBot::getNofAuctions to avoid always returning 1 auction when using ConsiderOnlyBotAuctions true
Fix Bug: wrong variable read from the config azerothcore#123 from azerothcore / mod-ah-bot

azerothcore#123

Fix copy paste typo when reading config for ElapsingTimeClass
Merge some fix from multiple contriburors.

Thanks to code from PixelWeaver, Icemansparks

Added some sanity checks in AHBConfig::DecItemCounts

made sure to remove auctions when item is not found

Co-Authored-By: Dennis <[email protected]>
Co-Authored-By: Antoine Cajot <[email protected]>
fix mistake withj unsigned int while hunting for the missing item auctions
revert back sql file to default values, git usage mistake
fix log error in AuctionHouseBot::Buy
Fixed a bunch of magic number and define values mismatch.

AuctionHouseBotyConfig was getting out of sync with the actual Server database until a server restart ( where on initialization it would query the server AH for real)
Changed SQL query of buyer to only get auctions from relevant house.

better logging on auction events in script manager
No need to stop selling item if above minimum and under maximum.
commented out unused param because of warning as errors
Fix issue azerothcore#141

ahbot bids no longer saved to the database

properly write in the database after updating the bid
@Jaytapp Jaytapp changed the title Fix https://github.com/azerothcore/mod-ah-bot/issues/141 Fix issue ahbot bids no longer saved to the database issue#141 Feb 26, 2025
@Jaytapp Jaytapp changed the title Fix issue ahbot bids no longer saved to the database issue#141 Fix ahbot bids no longer saved to the database issue#141 Feb 26, 2025
@Helias Helias merged commit db4165e into azerothcore:master Feb 26, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: ahbot bids no longer saved to the database
2 participants