Codacy fixes and small interface changes

Core/src/org/sleuthkit/autopsy/report/GeneralReportModule.java

@@ -45,16 +45,16 @@ public interface GeneralReportModule extends ReportModule {
 
     /**
      * Called to generate the report. Method is responsible for saving the file
-     * at the path specified and updating progress via the progressPanel object.
+     * and updating progress via the progressPanel object. Configuration
-     * Configuration parameters can be passed in the settings class. Modules should
+     * parameters are passed in the settings class, most notably the directory
-     * try to respond to all configuration parameters.
+     * to save the report. Modules should try to respond to all configuration
+     * parameters.
      *
-     * @param baseReportDir Base directory that reports are being stored in.
+     * @param settings Configuration parameters to customize the report
-     * Report should go into baseReportDir + getRelativeFilePath().
+     * generation process
-     * @param settings Configuration parameters to customize the report generation process
      * @param progressPanel panel to update the report's progress with
      */
-    default void generateReport(String baseReportDir, GeneralReportSettings settings, ReportProgressPanel progressPanel) {
+    default void generateReport(GeneralReportSettings settings, ReportProgressPanel progressPanel) {
-        generateReport(baseReportDir, progressPanel);
+        generateReport(settings.getReportDirectoryPath(), progressPanel);
     }
 }

Core/src/org/sleuthkit/autopsy/report/GeneralReportSettings.java

@@ -29,6 +29,7 @@ public class GeneralReportSettings implements Serializable {
     private static final long serialVersionUID = 1L;
     
     private List<Long> dataSourcesToProcess;
+    private String reportDirectoryPath;
     
     public List<Long> getDataSourcesToProcess() {
         return dataSourcesToProcess;
@@ -37,4 +38,12 @@ public class GeneralReportSettings implements Serializable {
     public void setDataSourcesToProcess(List<Long> dataSourcesToProcess) {
         this.dataSourcesToProcess = new ArrayList<>(dataSourcesToProcess);
     }
+    
+    public String getReportDirectoryPath() {
+        return this.reportDirectoryPath;
+    } 
+    
+    public void setReportDirectoryPath(String reportDirectoryPath) {
+        this.reportDirectoryPath = reportDirectoryPath;
+    }
 }

Core/src/org/sleuthkit/autopsy/report/infrastructure/ReportGenerator.java

@@ -302,7 +302,8 @@ public class ReportGenerator {
         if (generalReportModule != null) {
             String reportDir = createReportDirectory(generalReportModule);
             setupProgressPanel(generalReportModule, reportDir);
-            generalReportModule.generateReport(reportDir, reportSettings, progressIndicator);
+            reportSettings.setReportDirectoryPath(reportDir);
+            generalReportModule.generateReport(reportSettings, progressIndicator);
         }
     }
 

Core/src/org/sleuthkit/autopsy/report/infrastructure/ReportWizardPanel1.java

@@ -130,7 +130,7 @@ class ReportWizardPanel1 implements WizardDescriptor.FinishablePanel<WizardDescr
         }
         
         // HTML reports in initial PR
-        if(tableModuleSelection != null && tableModuleSelection instanceof HTMLReport) {
+        if(tableModuleSelection instanceof HTMLReport) {
             prefs.putBoolean("showDataSourceSelectionPanel", true);
         }
     }

Core/src/org/sleuthkit/autopsy/report/modules/kml/KMLReport.java

@@ -165,13 +165,14 @@ public final class KMLReport implements GeneralReportModule {
     }
     
     @Override
-    public void generateReport(String baseReportDir, GeneralReportSettings settings, ReportProgressPanel progressPanel) {
+    public void generateReport(GeneralReportSettings settings, ReportProgressPanel progressPanel) {
         try {
             currentCase = Case.getCurrentCaseThrows();
         } catch (NoCurrentCaseException ex) {
             logger.log(Level.SEVERE, "Exception while getting open case.", ex); //NON-NLS
             return;
         }
+        String baseReportDir = settings.getReportDirectoryPath();
         this.settings = settings;
         // Start the progress bar and setup the report
         progressPanel.setIndeterminate(true);
@@ -239,15 +240,16 @@ public final class KMLReport implements GeneralReportModule {
     @Override
     public void generateReport(String baseReportDir, ReportProgressPanel progressPanel) {       
         try {        
-            GeneralReportSettings settings = new GeneralReportSettings();
+            GeneralReportSettings reportSettings = new GeneralReportSettings();
             currentCase = Case.getCurrentCaseThrows();
             // Process all data sources if none are provided.
             List<Long> selections = currentCase.getDataSources()
                     .stream()
                     .map(Content::getId)
                     .collect(Collectors.toList());
-            settings.setDataSourcesToProcess(selections);
+            reportSettings.setDataSourcesToProcess(selections);
-            generateReport(baseReportDir, settings, progressPanel);
+            reportSettings.setReportDirectoryPath(baseReportDir);
+            generateReport(reportSettings, progressPanel);
         } catch (NoCurrentCaseException | TskCoreException ex) {
             logger.log(Level.SEVERE, "Exception while accessing case resources.", ex); //NON-NLS
         }