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

Chat tab is broken after running the client #2637

Closed
Marc-Spector opened this issue Apr 3, 2022 · 2 comments
Closed

Chat tab is broken after running the client #2637

Marc-Spector opened this issue Apr 3, 2022 · 2 comments
Labels

Comments

@Marc-Spector
Copy link
Collaborator

For start see the log.
It is unstable bug. After running the client, sometimes may call onUserJoinedChannel event before initialization default chat channel will be initialized.

I can't reproduce the bug manually.

2022-04-03 15:37:08.587 ERROR 8224 --- [JavaFX Application Thread] c.f.c.exception.GlobalExceptionHandler   : Uncaught exception on Thread[JavaFX Application Thread,5,main]: 

java.lang.IndexOutOfBoundsException: Index: -1, Size: 0
	at java.base/java.util.ArrayList.rangeCheckForAdd(ArrayList.java:756)
	at java.base/java.util.ArrayList.add(ArrayList.java:481)
	at com.sun.javafx.collections.ObservableListWrapper.doAdd(ObservableListWrapper.java:101)
	at javafx.collections.ModifiableObservableListBase.add(ModifiableObservableListBase.java:158)
	at com.faforever.client.chat.ChatController.addTab(ChatController.java:129)
	at com.faforever.client.chat.ChatController.getOrCreateChannelTab(ChatController.java:116)
	at com.faforever.client.chat.ChatController.lambda$onUserJoinedChannel$13(ChatController.java:261)
	at com.sun.javafx.application.PlatformImpl.lambda$runLater$10(PlatformImpl.java:457)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:399)
	at com.sun.javafx.application.PlatformImpl.lambda$runLater$11(PlatformImpl.java:456)
	at com.sun.glass.ui.InvokeLaterDispatcher$Future.run(InvokeLaterDispatcher.java:96)
	at com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
	at com.sun.glass.ui.win.WinApplication.lambda$runLoop$3(WinApplication.java:184)
	at java.base/java.lang.Thread.run(Thread.java:833)

Windows

2022.3.0

@Eternal-ll
Copy link

Eternal-ll commented Apr 3, 2022

Maybe related, but i found a bug in IRC notification system, when you join to the channel it wont notify you that "you" joined to the channel. But if you join the same channel secondly, after leaving from it, there will be notification that "you" joined to the channel and "you" will come with user list too. I noticed it when i had double "me" instance in chats xD

@Sheikah45
Copy link
Member

Fixed in recent chat rework

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

No branches or pull requests

3 participants