Skip to content
This repository has been archived by the owner on Jun 29, 2023. It is now read-only.

NPE for Mount. #4

Closed
SilverCory opened this issue Aug 30, 2015 · 8 comments
Closed

NPE for Mount. #4

SilverCory opened this issue Aug 30, 2015 · 8 comments

Comments

@SilverCory
Copy link
Contributor

http://pastebin.com/ATgCBGHb

This occurs when you dismount something other than a mount.. The PR I made that added the try {} catch didn't work, it's getPlayer and Core#getCustomPlayer that make it weird.
@iSach
Copy link
Owner

iSach commented Aug 30, 2015

Mmh okay thanks for the report I'll fix that

@iSach iSach closed this as completed Aug 30, 2015
@iSach iSach reopened this Aug 30, 2015
@iSach
Copy link
Owner

iSach commented Aug 30, 2015

Fixed.

@iSach iSach closed this as completed Aug 30, 2015
@SilverCory
Copy link
Contributor Author

Nope.

http://pastebin.com/H0mPqWc0 Same issue.

@iSach
Copy link
Owner

iSach commented Aug 31, 2015

When does it happen?

@iSach iSach reopened this Aug 31, 2015
@SilverCory
Copy link
Contributor Author

Not sure I suppressed it like in my fork. Seems to work suppressed without any side effects.

@SilverCory SilverCory reopened this Aug 31, 2015
@SilverCory
Copy link
Contributor Author

The custom player could be null?

@iSach
Copy link
Owner

iSach commented Aug 31, 2015

No I don't think so, normally that's impossible they are generated on enable and join and also the error happens on line initStats

@iSach
Copy link
Owner

iSach commented Sep 4, 2015

Should be fixed for next version.

@iSach iSach closed this as completed Sep 4, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@SilverCory @iSach