Error-proof HashLookupSettingsPanel.isLocalIngestJobEvent()

This commit is contained in:
rcordovano 2015-04-24 10:08:51 -04:00
parent 3cdfbf6934
commit de360793f9

View File

@ -76,7 +76,7 @@ public final class HashLookupSettingsPanel extends IngestModuleGlobalSettingsPan
public void run() { public void run() {
updateComponents(); updateComponents();
} }
}); });
} }
} }
}); });
@ -229,7 +229,16 @@ public final class HashLookupSettingsPanel extends IngestModuleGlobalSettingsPan
} }
private boolean isLocalIngestJobEvent(PropertyChangeEvent evt) { private boolean isLocalIngestJobEvent(PropertyChangeEvent evt) {
return evt instanceof AutopsyEvent && ((AutopsyEvent)evt).getSourceType() == AutopsyEvent.SourceType.LOCAL; if (evt instanceof AutopsyEvent) {
AutopsyEvent event = (AutopsyEvent) evt;
if (event.getSourceType() == AutopsyEvent.SourceType.LOCAL) {
String eventType = event.getPropertyName();
return (eventType.equals(IngestManager.IngestJobEvent.STARTED.toString())
|| eventType.equals(IngestManager.IngestJobEvent.CANCELLED.toString())
|| eventType.equals(IngestManager.IngestJobEvent.COMPLETED.toString()));
}
}
return false;
} }
@Override @Override
@ -276,7 +285,7 @@ public final class HashLookupSettingsPanel extends IngestModuleGlobalSettingsPan
HashDbManager.getInstance().loadLastSavedConfiguration(); HashDbManager.getInstance().loadLastSavedConfiguration();
} }
} }
void removeThese(List<HashDb> toRemove) { void removeThese(List<HashDb> toRemove) {
for (HashDb hashDb : toRemove) { for (HashDb hashDb : toRemove) {
hashSetManager.removeHashDatabaseInternal(hashDb); hashSetManager.removeHashDatabaseInternal(hashDb);
@ -305,7 +314,7 @@ public final class HashLookupSettingsPanel extends IngestModuleGlobalSettingsPan
} }
int res = JOptionPane.showConfirmDialog(this, message, int res = JOptionPane.showConfirmDialog(this, message,
NbBundle.getMessage(this.getClass(), NbBundle.getMessage(this.getClass(),
"HashDbConfigPanel.unindexedDbsMsg"), "HashDbConfigPanel.unindexedDbsMsg"),
JOptionPane.YES_NO_OPTION); JOptionPane.YES_NO_OPTION);
if (res == JOptionPane.YES_OPTION) { if (res == JOptionPane.YES_OPTION) {
ModalNoButtons indexingDialog = new ModalNoButtons(this, new Frame(), unindexed); ModalNoButtons indexingDialog = new ModalNoButtons(this, new Frame(), unindexed);
@ -753,7 +762,7 @@ public final class HashLookupSettingsPanel extends IngestModuleGlobalSettingsPan
private void deleteDatabaseButtonActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_deleteDatabaseButtonActionPerformed private void deleteDatabaseButtonActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_deleteDatabaseButtonActionPerformed
if (JOptionPane.showConfirmDialog(null, if (JOptionPane.showConfirmDialog(null,
NbBundle.getMessage(this.getClass(), NbBundle.getMessage(this.getClass(),
"HashDbConfigPanel.deleteDbActionConfirmMsg"), "HashDbConfigPanel.deleteDbActionConfirmMsg"),
NbBundle.getMessage(this.getClass(), "HashDbConfigPanel.deleteDbActionMsg"), NbBundle.getMessage(this.getClass(), "HashDbConfigPanel.deleteDbActionMsg"),
JOptionPane.YES_NO_OPTION, JOptionPane.YES_NO_OPTION,
JOptionPane.WARNING_MESSAGE) == JOptionPane.YES_OPTION) { JOptionPane.WARNING_MESSAGE) == JOptionPane.YES_OPTION) {
@ -828,4 +837,4 @@ public final class HashLookupSettingsPanel extends IngestModuleGlobalSettingsPan
private javax.swing.JCheckBox sendIngestMessagesCheckBox; private javax.swing.JCheckBox sendIngestMessagesCheckBox;
private javax.swing.JLabel typeLabel; private javax.swing.JLabel typeLabel;
// End of variables declaration//GEN-END:variables // End of variables declaration//GEN-END:variables
} }