Skip to content
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

Core: Trade window glitches out if target does have AH tab opened #29308

Closed
CraftedRO opened this issue Sep 7, 2023 · 6 comments
Closed

Core: Trade window glitches out if target does have AH tab opened #29308

CraftedRO opened this issue Sep 7, 2023 · 6 comments

Comments

@CraftedRO
Copy link
Contributor

CraftedRO commented Sep 7, 2023

Description

When you try to open trade with someone while they have Auction House window opened, it glitches out and you won't be able to trade again (eachother) if you don't relog.

Expected behaviour

It shoud appear the following error instead: "(Character name) is busy right now."

WotLK classic PTR 3.4.3 (51126)

2023-09-07.17-10-12.mp4

Steps to reproduce the problem

  1. Have two accounts and with one character go to Auction House and open the window by click on any auctioneer.
  2. With other character go to the first character range and then press Trade.
  3. Notice.

Branch

3.3.5

TC rev. hash/commit

79e2f77

Operating system

Windows 10 x64

Custom changes

None

@CyberMist2
Copy link

As the chat says: Player is busy right now, this might be a good method to avoid annoying people trying to interrupt you from using auctionhouse.

@Ovahlord
Copy link
Contributor

Ovahlord commented Oct 4, 2023

Afaik this feature cannot be implemented on older clients because old clients do not track if a player is currently in an interaction. They only know when someone talks to an npc but not when the interaction ends. That has been implemented in later expansions only.

@AGandrup
Copy link

AGandrup commented Oct 4, 2023

As I understand it, the issue is that:
Player A: interacts with auction house.
Player B: Request trade with Player A. Gets chat message "Player A busy".
Everything is working as intended so far.
Then,
Player A: Closes interaction with auction house
Player B: Request trade with Player A, but still gets chat message "Player A busy", and still will not be able to trade untill relog.

@Aokromes
Copy link
Member

Aokromes commented Oct 4, 2023

iirc the bug is also present with other interactions. iirc 2 trades.

@funjoker
Copy link
Member

funjoker commented Oct 4, 2023

Missing CMSG?

Copy link

420cbd7

Ovahlord pushed a commit that referenced this issue Jan 31, 2024
)

Closes #29308

(cherry picked from commit 420cbd7)

# Conflicts:
#	src/server/game/Handlers/TradeHandler.cpp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants