Skip to content

Commit

Permalink
Fix present user parameter emptiness check
Browse files Browse the repository at this point in the history
  • Loading branch information
Sipkab committed Apr 4, 2020
1 parent 26d48c7 commit 3fee1ff
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -46,11 +46,11 @@ public synchronized Set<UserParameterModification> contribute(ISakerPlugin plugi
JavaRuntime.addVMInstallChangedListener(this);
StringJoiner installlocations = new StringJoiner(";");
String present = parameters.get(INSTALL_LOCATIONS_ENV_PARAMETER_NAME);
if (present != null) {
if (present != null && !present.isEmpty()) {
installlocations.add(present);
}
//assign plugin later, because when we query the installs, the vmAdded events fire automatically
//therefor we assign the plugin later, so we don't invalidate it unnecessarily
//therefore we assign the plugin later, so we don't invalidate it unnecessarily
//it is acceptable, as concurrency errors won't happen due to synchronization of methods
boolean foundinstall = false;
for (IVMInstallType vmitype : JavaRuntime.getVMInstallTypes()) {
Expand Down

0 comments on commit 3fee1ff

Please sign in to comment.