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

User list empty chat #1925

Closed
1-alex98 opened this issue Sep 19, 2020 · 1 comment · Fixed by #2126
Closed

User list empty chat #1925

1-alex98 opened this issue Sep 19, 2020 · 1 comment · Fixed by #2126
Labels
Milestone

Comments

@1-alex98
Copy link
Member

Describe the bug
On startup chat user list stays empty

To Reproduce
Probably a race condition

Log or error message
This is my log file:

Exception in thread "JavaFX Application Thread" java.lang.NullPointerException
	at com.sun.javafx.collections.ObservableListWrapper.removeAll(ObservableListWrapper.java:173)
	at com.faforever.client.chat.ChannelTabController.lambda$onUserLeft$37(ChannelTabController.java:554)
	at com.sun.javafx.application.PlatformImpl.lambda$runLater$10(PlatformImpl.java:428)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:391)
	at com.sun.javafx.application.PlatformImpl.lambda$runLater$11(PlatformImpl.java:427)
	at com.sun.glass.ui.InvokeLaterDispatcher$Future.run$$$capture(InvokeLaterDispatcher.java:96)
	at com.sun.glass.ui.InvokeLaterDispatcher$Future.run(InvokeLaterDispatcher.java)
	at com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
	at com.sun.glass.ui.win.WinApplication.lambda$runLoop$3(WinApplication.java:174)
	at java.base/java.lang.Thread.run(Thread.java:832)

Expected behavior
The chat controllers are loaded before these kind of events.

OS
Linux or Windows

Wanna have the bug fixed quickly?
Visit Issue hunt...
Issue hunt

@1-alex98 1-alex98 added the bug label Sep 19, 2020
@Sheikah45
Copy link
Member

Have not seen this recently

@Sheikah45 Sheikah45 linked a pull request Feb 6, 2021 that will close this issue
@Sheikah45 Sheikah45 added this to the v1.4.1 milestone Feb 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants