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

IndexOutOfBoundsException when advancing day #68

Closed
SirMegaV opened this issue May 6, 2016 · 8 comments
Closed

IndexOutOfBoundsException when advancing day #68

SirMegaV opened this issue May 6, 2016 · 8 comments

Comments

@SirMegaV
Copy link

SirMegaV commented May 6, 2016

MekHQ 0.3.27

Occasionally getting this error when advancing a day after combat, then selling units, then repairs, then removing prisoners or astechs, then assigning doctors. Error will disappear if restarting MekHQ right before the advance day.

Savegame and log attached.

IndexOutOfBoundsException.txt

IndexOutOfBoundsException.7z.txt

@Akjosch
Copy link
Contributor

Akjosch commented May 6, 2016

It looks like it's the doctors/patients part which makes problems. What are you doing on the day before the error hits, exactly?

@SirMegaV
Copy link
Author

SirMegaV commented May 6, 2016

It was after a battle. I sold unwanted units, then repair units, then GM
removed prisoners and astechs, then assign doctors, then hit next day.
Occasionally getting this after this routine.

On Friday, May 6, 2016, Akjosch notifications@github.com wrote:

It looks like it's the doctors/patients part which makes problems. What
are you doing on the day before the error hits, exactly?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#68 (comment)

@dericpage
Copy link
Collaborator

Added protection against PatientTableModel being given a bad index when attempting to retrieve an index. In such an instance, the model will return a NULL value. Also added null handling to the calling methods.

dericpage added a commit that referenced this issue May 20, 2016
Issue #68: IndexOutOfBoundsException when advancing day
@pheonixstorm
Copy link
Collaborator

If this issue is fixed please close.

pheonixstorm pushed a commit to pheonixstorm/mekhq that referenced this issue Jun 23, 2016
pheonixstorm pushed a commit to pheonixstorm/mekhq that referenced this issue Jun 23, 2016
Issue MegaMek#68: IndexOutOfBoundsException when advancing day
@SirMegaV
Copy link
Author

Not seeing it lately... might have been fixed. Pls close, I will post again if I receive the same error.

@pheonixstorm
Copy link
Collaborator

Its your ticket, you can close it yourself. Faster than waiting on the devs in most cases.

@SirMegaV
Copy link
Author

Sry, didn't know I can close myself.

@pheonixstorm
Copy link
Collaborator

No worries, your not the only one.

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

No branches or pull requests

4 participants