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

Command ".unstuck" does not work when stuck player is offline -__- #3177

Open
BarbzYHOOL opened this issue Jul 8, 2020 · 5 comments · May be fixed by #18913
Open

Command ".unstuck" does not work when stuck player is offline -__- #3177

BarbzYHOOL opened this issue Jul 8, 2020 · 5 comments · May be fixed by #18913

Comments

@BarbzYHOOL
Copy link
Member

BarbzYHOOL commented Jul 8, 2020

CURRENT BEHAVIOUR:

You can't use .unstuck barbz if character Barbz is offline. This is extremely inconvenient as a GM can receive a ticket "please i'm stuck, can you unstuck me" and the player logs out. Then you can't unstuck him easily until he logs back in. You have to go into the DB for that, or use other hacky techniques.

The command also doesn't return anything when you successfully unstuck someone. It should say something in chat AND console to warn the GM that it worked. It could also warn the player with a red notification "You have been unstuck by a GM.".

STEPS TO REPRODUCE THE PROBLEM:
  1. Take two accounts. Log in with both but stay at the character selection screen (before entering the game) for account 2
  2. With account 1 use .unstuck to unstuck (teleport to his $home inn) the other account's character
  3. Doesn't work when offline
EXTRA NOTES:

This might also affect other commands working similarly. Note that it looks like that these commands have not been touched for years, even on trinitycore, and are full of small bugs/inconvenience

Also, the command doesn't return useful output . It says "Player not found!", it should say player offline. That's the case for tons of commands, it's shit (can lead to a lot of time wasted).

AC HASH/COMMIT:

6d460c5

@natrist
Copy link
Contributor

natrist commented Nov 29, 2023

The GM tool I intend to develop soon™ will feature an option to teleport offline players via a SQL query.

Also if you're going to be implementing custom behaviour, that is behaviour that does not reflect the original behaviour of the game and its servers, you should be careful and put it under a module or Lua script. The codebase is already riddled with custom implementations and bad code.

@pangolp
Copy link
Contributor

pangolp commented Apr 30, 2024

I'll check this command later, however, to unlock a character, in case it gets caught and requires help, you can also use the .summon command. Anyway, I'll try to check what happens with this command.

@sudlud
Copy link
Member

sudlud commented May 2, 2024

I thought this already got fixed on the meantime, but thanks @pangolp for checking

@pangolp
Copy link
Contributor

pangolp commented May 2, 2024

Previously, I don't remember the year, we had assigned some issues. That's why I'm assigned this one and three more. But the truth is, I had completely forgotten about this.

@pangolp
Copy link
Contributor

pangolp commented May 2, 2024

Maybe it can be improved later, but maybe this way, for now it will be fine.

2024-05-02.04-32-06.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants