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

Uncaught promise rejection - /house goto #438

Closed
ghost opened this issue Jan 11, 2017 · 0 comments
Closed

Uncaught promise rejection - /house goto #438

ghost opened this issue Jan 11, 2017 · 0 comments

Comments

@ghost
Copy link

ghost commented Jan 11, 2017

I was reviewing the logs and trying to track down some of the reported bugs. In the logs, there's a number of entries like this:

[11/01 14:09:52] =========================
[11/01 14:09:52] Uncaught promise rejection: TypeError: unable to execute pawnInvoke(): string overflow for argument 6
[11/01 14:09:52] in showDialog (entities/player.js:361)
[11/01 14:09:52] in Promise (components/dialogs/dialog_manager.js:36)
[11/01 14:09:52] in displayForPlayer (components/dialogs/dialog_manager.js:32)
[11/01 14:09:52] in displayMenu (components/dialogs/dialog.js:61)
[11/01 14:09:52] in resolve (components/menu/menu.js:55)
[11/01 14:09:52] in displayForPlayer (components/menu/menu.js:53)
[11/01 14:09:52] in onHouseGotoCommand (features/houses/house_commands.js:292)
[11/01 14:09:52] in anonymous function (components/command_manager/command_builder.js:303)
[11/01 14:09:52] in anonymous function (components/command_manager/command_builder.js:233)
[11/01 14:09:52] in onPlayerCommandText (components/command_manager/command_manager.js:78)
[11/01 14:09:52] in onEvent (base/scoped_callbacks.js:32)
[11/01 14:09:52] =========================

Reproducing this error can be done by using the /house goto command.

If an admin uses this command, there is no output and nothing happens, regardless of if they own a house or not.

If a player uses this command, they will be shown the dialog to choose which house they want to teleport to, assuming they own a house. If they don't own a house, the correct "sorry but you don't own any houses" error will show up.

This feature did previously work fine for admins - is it possible that there's too many houses and as such, the dialog isn't showing up now?

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

No branches or pull requests

0 participants