Fixed NPE when getting flairs #54

Closed
alisolarflare wants to merge 1 commit from Bugfix into master
alisolarflare commented 2016-12-17 06:53:54 +00:00 (Migrated from github.com)

Side effect: reduced Legendosh bug to happen every 10 seconds. I suspect
the author variable is null

Side effect: reduced Legendosh bug to happen every 10 seconds. I suspect the author variable is null
NorbiPeti commented 2016-12-17 10:29:52 +00:00 (Migrated from github.com)

You're 7 commits behind though... 😛 I'm fairly sure I fixed this yesterday.

You're 7 commits behind though... 😛 I'm fairly sure I fixed this yesterday.
NorbiPeti commented 2016-12-18 10:50:46 +00:00 (Migrated from github.com)

Please check if the latest version errors for you.

Please check if the latest version errors for you.
alisolarflare commented 2016-12-19 04:40:50 +00:00 (Migrated from github.com)

No, the master branch still errors like hell for me, check the post on #bot-room

No, the master branch still errors like hell for me, check the post on #bot-room
NorbiPeti commented 2016-12-19 11:04:21 +00:00 (Migrated from github.com)

It looks like getFlairState errors now... I can fix that later today.

The reason I don't really want to accept this PR because these aren't supposed to be null at all.

It looks like getFlairState errors now... I can fix that later today. The reason I don't really want to accept this PR because these aren't *supposed* to be null at all.

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: ChromaGaming/Chroma-Chat#54
No description provided.