minster586/OpenInv
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 36 Wiki Activity

Null pointer check #10

Merged
Fysac merged 2 commits from master into master 2014-08-03 17:58:31 -04:00
Conversation 0 Commits 2 Files Changed 1 +6 -3
Fysac commented 2014-08-01 13:39:15 -04:00 (Migrated from github.com)
Copy Link

Players who haven't joined your server since the UUID conversion will have a null name. As a result, if you try to /openinv a player who has joined your server since then, the loop can break due to a NPE while checking other players. This null check should fix that problem.

Players who haven't joined your server since the UUID conversion will have a null name. As a result, if you try to /openinv a player who _has_ joined your server since then, the loop can break due to a NPE while checking other players. This null check should fix that problem.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Bug
Cannot Reproduce
Enhancement
Fixed
Invalid
Low Priority
Need More Information
Pending
Question
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
minster586
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: minster586/OpenInv#10
No description provided.
Delete Branch "master"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?