NullPointerException when showing Source Selection Dialog when no sources are present

Description

Environment

None

Activity

Show:
Andrew Maitland
November 15, 2013, 5:35 PM

Hi Patrick,

To help the code team can you answer the following?
What version of PCGen?
What version of JAVA are you using 1.5, 1.6, 1.7 or other (And who is your source Sun or OpenJDK, something else?)
What Operating System?

Thanks

Patrick Bogen
November 15, 2013, 6:35 PM
  • SVN trunk

  • Oracle Java 7u45

  • Debian

Patrick Bogen
November 15, 2013, 6:36 PM

I expect it's just an edge case that isn't really expected to happen. I built from SVN using a very sparse checkout, which at first didn't include the data/ subdir.

James Dempsey
November 17, 2013, 5:55 PM

thanks for the patch! I've gone a slightly different way though and checked for nulls in advance.

I've also added an installation validation check at startup to ensure all of our essential folders are present. If they aren't then the user is shown an error message with the details and the program is stopped.

Patrick Bogen
November 17, 2013, 9:48 PM

Sounds good to me. p.s., I typically use a very high DPI laptop; the reason I discovered this was in the process of checking out the source to work on retrofitting it to be DPI-aware (at least with a user-configured DPI). Is there already any work going on in this vein?

Fixed

Assignee

James Dempsey

Reporter

Patrick Bogen

Labels

Theme

None

Epic/Theme

None

Pending User Input

None

Fix versions

Priority

Minor
Configure