Skip to content
This repository has been archived by the owner on Dec 15, 2021. It is now read-only.

JS bug #5084: Transport doesn't support JEP-0085 (Chat State Notifications) #1

Closed
Jajcus opened this issue May 7, 2011 · 3 comments

Comments

@Jajcus
Copy link
Owner

Jajcus commented May 7, 2011

Test conditions:

  • client: Tkabber 0.9.8 (cvs 11. Oct 2005) with chat state notifications turned on
  • transport: gg.chrome.pl (v2.2.1)

Steps to reproduce:

  • send any message to transport (eg. 'help')
  • transport responds with help message
  • closing the window/panel triggers sending a chat state message in tkabber:
    <message to='gg.chrome.pl/registered'
- after closing window/panel new message appears from gg.chrome.pl with standard help: type='chat'>

Dostępne komendy (i skróty): ... and so on...
@Jajcus
Copy link
Owner Author

Jajcus commented May 7, 2011

This seems rather like Tkabber bug — it should not send events to an entity not supporting it. I see no need for chat state notification support in jGGtrans, but the transport should not answer to them anyway.

I think all what is needed to resolve this problem is to make jGGtrans ignore messages with no or empty body.

--jajcus

@Jajcus
Copy link
Owner Author

Jajcus commented May 7, 2011

just a test, sorry...

--jajcus

@Jajcus
Copy link
Owner Author

Jajcus commented May 7, 2011

(In [711]) - merged change from xiaoka: r73 'Fixed presence time stamping. Fixes #1'

--jajcus

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant