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

NPE in mekhq.gui.handler.TOETransferHandler.canImport #1981

Closed
HammerGS opened this issue Sep 2, 2020 · 4 comments · Fixed by #1996
Closed

NPE in mekhq.gui.handler.TOETransferHandler.canImport #1981

HammerGS opened this issue Sep 2, 2020 · 4 comments · Fixed by #1996
Assignees
Labels

Comments

@HammerGS
Copy link
Member

HammerGS commented Sep 2, 2020

The TOE error was me bringing a unit out of refit. It appeared in the TOE and then I couldn't drag it to a force. Also I swear we used to be able to drag and drop to re-order the TOE.

Environment

What version of MekHQ does your issue apply to? 47.10 SNAPSHOT
What operating system are you using? Windows,

NPE

java.lang.NullPointerException
	at mekhq.gui.handler.TOETransferHandler.canImport(TOETransferHandler.java:109)
	at java.desktop/javax.swing.TransferHandler$DropHandler.handleDrag(TransferHandler.java:1467)
	at java.desktop/javax.swing.TransferHandler$DropHandler.dragOver(TransferHandler.java:1496)
	at java.desktop/java.awt.dnd.DropTarget.dragOver(DropTarget.java:379)
	at java.desktop/javax.swing.TransferHandler$SwingDropTarget.dragOver(TransferHandler.java:1250)
	at java.desktop/sun.awt.dnd.SunDropTargetContextPeer.processMotionMessage(SunDropTargetContextPeer.java:482)
	at java.desktop/sun.awt.dnd.SunDropTargetContextPeer$EventDispatcher.dispatchMotionEvent(SunDropTargetContextPeer.java:835)
	at java.desktop/sun.awt.dnd.SunDropTargetContextPeer$EventDispatcher.dispatchEvent(SunDropTargetContextPeer.java:781)

The Harriers30460613.cpnx.gz
megameklog (1).txt
mekhqlog (1).txt

@HammerGS HammerGS added the Bug label Sep 2, 2020
@Windchild292
Copy link
Contributor

This would be the file that handles drag and drop

@Windchild292
Copy link
Contributor

From a code look, did you drag and drop so it was a child of another unit?

@Windchild292 Windchild292 self-assigned this Sep 4, 2020
@HammerGS
Copy link
Member Author

HammerGS commented Sep 4, 2020

Not that I was aware of. I was trying to order the lances, kinda picky and like my Mech lances on the top of the list.

@Windchild292
Copy link
Contributor

I did fix two NPEs from code side, will test when I am able to.

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