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

ANSI Music Data Error #11

Closed
ghost opened this issue May 11, 2018 · 1 comment
Closed

ANSI Music Data Error #11

ghost opened this issue May 11, 2018 · 1 comment

Comments

@ghost
Copy link

ghost commented May 11, 2018

Running "The Pit" first time player will be asked if they see color (ANSI) if you say yes, you should be asked if you heard music.

In SyncTERM this question works and it is yes/no.
In fTelnet the line never prints and in the browser console the following error is displayed:

Uncaught TypeError: Cannot read property '0' of undefined
    at Ansi.AnsiCommand (ftelnet.norip.xfer.min.js:105)
    at Ansi.Write (ftelnet.norip.xfer.min.js:138)
    at fTelnetClient.OnConnectionData (ftelnet.norip.xfer.min.js:624)
    at Object.<anonymous> (ftelnet.norip.xfer.min.js:585)
    at TypedEvent.trigger (ftelnet.norip.xfer.min.js:91)
    at TelnetConnection.WebSocketConnection.OnWebSocketMessage (ftelnet.norip.xfer.min.js:406)
    at WebSocket._WebSocket.onmessage (ftelnet.norip.xfer.min.js:393)
Ansi.AnsiCommand @ ftelnet.norip.xfer.min.js:105
Ansi.Write @ ftelnet.norip.xfer.min.js:138
fTelnetClient.OnConnectionData @ ftelnet.norip.xfer.min.js:624
(anonymous) @ ftelnet.norip.xfer.min.js:585
TypedEvent.trigger @ ftelnet.norip.xfer.min.js:91
WebSocketConnection.OnWebSocketMessage @ ftelnet.norip.xfer.min.js:406
_WebSocket.onmessage @ ftelnet.norip.xfer.min.js:393

Normally not an issue, but since the game is asking for a Y or N, the line never printed and people force disconnect because they don't know whats going on. If you hit Y or N the game continues and all is good.

Anyway to bypass that error? Print the line and ignore ANSI music codes?

If you wish to test on my system, I can send more information via email.

Thanks.

@rickparrish
Copy link
Owner

Fairly sure this is related to my handling of the ESC[M sequence -- I was looking to see if the sequence contained an '=' in a way that would have broken for the parameter-less ESC[M sequence.

I've now pushed a change to both the fTelnet repository and the embed-v2 wizard, so hopefully this fixes things for you. If not I'll have to actually install The Pit to do some proper testing.

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

1 participant