From cc3d21c3ca26e65b07e863db62a6171e5ca8dd4b Mon Sep 17 00:00:00 2001 From: Nick Davis Date: Thu, 15 Jun 2017 18:03:52 -0400 Subject: [PATCH] started on issue #174. Changed Save button to OK button. Updated default db path. Use default db path in file chooser. Use default db dir in file chooser. --- .../datamodel/SqliteEamDbSettings.java | 2 +- .../optionspanel/Bundle.properties | 2 +- .../optionspanel/EamSqliteSettingsDialog.form | 10 +++---- .../optionspanel/EamSqliteSettingsDialog.java | 29 +++++++++++-------- 4 files changed, 24 insertions(+), 19 deletions(-) diff --git a/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/datamodel/SqliteEamDbSettings.java b/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/datamodel/SqliteEamDbSettings.java index ede8e04d51..787de9c4f0 100644 --- a/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/datamodel/SqliteEamDbSettings.java +++ b/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/datamodel/SqliteEamDbSettings.java @@ -41,7 +41,7 @@ public final class SqliteEamDbSettings { private final static Logger LOGGER = Logger.getLogger(SqliteEamDbSettings.class.getName()); private final String DEFAULT_DBNAME = "EnterpriseArtifacts.db"; // NON-NLS - private final String DEFAULT_DBDIRECTORY = PlatformUtil.getUserDirectory() + File.separator + "Autopsy" + File.separator + "eamdb"; // NON-NLS + private final String DEFAULT_DBDIRECTORY = PlatformUtil.getUserDirectory() + File.separator + "enterprise_artifacts_manager"; // NON-NLS private final int DEFAULT_BULK_THRESHHOLD = 1000; private final String DEFAULT_BAD_TAGS = "Evidence"; // NON-NLS private final String JDBC_DRIVER = "org.sqlite.JDBC"; // NON-NLS diff --git a/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/optionspanel/Bundle.properties b/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/optionspanel/Bundle.properties index ccb83b2e46..1b1cb090cf 100644 --- a/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/optionspanel/Bundle.properties +++ b/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/optionspanel/Bundle.properties @@ -17,7 +17,6 @@ EamPostgresSettingsDialog.bnSave.text=Save EamPostgresSettingsDialog.lbHostName.text=Host Name / IP : EamPostgresSettingsDialog.bnTestConnection.text=Test Connection EamPostgresSettingsDialog.lbDatabaseName.text=Database name : -EamSqliteSettingsDialog.bnSave.text=Save EamSqliteSettingsDialog.bnCancel.text=Cancel EamSqliteSettingsDialog.lbTestDatabase.text= EamSqliteSettingsDialog.bnTestDatabase.text=Test Connection @@ -68,3 +67,4 @@ ManageArtifactTypesDialog.cancelButton.text=Cancel ManageArtifactTypesDialog.okButton.text=OK ManageArtifactTypesDialog.lbWarningMsg.text=Warning Message ManageArtifactTypesDialog.taInstructionsMsg.text=Select one or more artifact types to store in the database and use for correlation during Ingest. +EamSqliteSettingsDialog.bnOk.text=OK diff --git a/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/optionspanel/EamSqliteSettingsDialog.form b/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/optionspanel/EamSqliteSettingsDialog.form index 9ab466696d..de9cad5ca8 100644 --- a/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/optionspanel/EamSqliteSettingsDialog.form +++ b/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/optionspanel/EamSqliteSettingsDialog.form @@ -91,7 +91,7 @@ - + @@ -116,7 +116,7 @@ - + @@ -198,14 +198,14 @@ - + - + - + diff --git a/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/optionspanel/EamSqliteSettingsDialog.java b/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/optionspanel/EamSqliteSettingsDialog.java index e46288dd21..60ff1f8f81 100644 --- a/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/optionspanel/EamSqliteSettingsDialog.java +++ b/Experimental/src/org/sleuthkit/autopsy/experimental/enterpriseartifactsmanager/optionspanel/EamSqliteSettingsDialog.java @@ -87,7 +87,7 @@ public class EamSqliteSettingsDialog extends javax.swing.JDialog { lbTestDatabaseWarning.setText(""); hasChanged = false; tfDatabasePath.getDocument().addDocumentListener(textBoxChangedListener); - bnSave.setEnabled(false); + bnOk.setEnabled(false); bnTestDatabase.setEnabled(false); } @@ -135,11 +135,11 @@ public class EamSqliteSettingsDialog extends javax.swing.JDialog { private boolean valid() { boolean result = false; if (tfDatabasePath.getText().trim().isEmpty()) { - bnSave.setEnabled(false); + bnOk.setEnabled(false); bnTestDatabase.setEnabled(false); } else { storeDbNameAndDirectory(); - bnSave.setEnabled(true); + bnOk.setEnabled(true); bnTestDatabase.setEnabled(true); result = true; } @@ -182,7 +182,7 @@ public class EamSqliteSettingsDialog extends javax.swing.JDialog { bnTestDatabase = new javax.swing.JButton(); lbTestDatabase = new javax.swing.JLabel(); bnCancel = new javax.swing.JButton(); - bnSave = new javax.swing.JButton(); + bnOk = new javax.swing.JButton(); setDefaultCloseOperation(javax.swing.WindowConstants.DISPOSE_ON_CLOSE); @@ -218,10 +218,10 @@ public class EamSqliteSettingsDialog extends javax.swing.JDialog { } }); - org.openide.awt.Mnemonics.setLocalizedText(bnSave, org.openide.util.NbBundle.getMessage(EamSqliteSettingsDialog.class, "EamSqliteSettingsDialog.bnSave.text")); // NOI18N - bnSave.addActionListener(new java.awt.event.ActionListener() { + org.openide.awt.Mnemonics.setLocalizedText(bnOk, org.openide.util.NbBundle.getMessage(EamSqliteSettingsDialog.class, "EamSqliteSettingsDialog.bnOk.text")); // NOI18N + bnOk.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(java.awt.event.ActionEvent evt) { - bnSaveActionPerformed(evt); + bnOkActionPerformed(evt); } }); @@ -247,7 +247,7 @@ public class EamSqliteSettingsDialog extends javax.swing.JDialog { .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED) .addComponent(lbTestDatabase, javax.swing.GroupLayout.PREFERRED_SIZE, 30, javax.swing.GroupLayout.PREFERRED_SIZE) .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE) - .addComponent(bnSave) + .addComponent(bnOk) .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.UNRELATED) .addComponent(bnCancel) .addContainerGap()))) @@ -267,7 +267,7 @@ public class EamSqliteSettingsDialog extends javax.swing.JDialog { .addGap(19, 19, 19) .addGroup(pnContentLayout.createParallelGroup(javax.swing.GroupLayout.Alignment.BASELINE) .addComponent(bnCancel) - .addComponent(bnSave))) + .addComponent(bnOk))) .addGroup(pnContentLayout.createSequentialGroup() .addGap(18, 18, 18) .addGroup(pnContentLayout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING) @@ -311,11 +311,15 @@ public class EamSqliteSettingsDialog extends javax.swing.JDialog { @Messages({"EnterpriseArtifactsManagerSQLiteSettingsDialog.chooserPath.failedToGetDbPathMsg=Selected database path is invalid. Try again."}) private void bnDatabasePathFileOpenActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_bnDatabasePathFileOpenActionPerformed + fcDatabasePath.setCurrentDirectory(new File(dbSettings.getDbDirectory())); + fcDatabasePath.setSelectedFile(new File(dbSettings.getFileNameWithPath())); if (fcDatabasePath.showOpenDialog(this) == JFileChooser.APPROVE_OPTION) { File databaseFile = fcDatabasePath.getSelectedFile(); try { tfDatabasePath.setText(databaseFile.getCanonicalPath()); valid(); + // TODO: create the db/schema if it doesn't exist. + // TODO: set variable noting that we created a new db, so it can be removed if Cancel button is clicked. } catch (IOException ex) { LOGGER.log(Level.SEVERE, "Failed to get path of selected database file", ex); // NON-NLS @@ -339,20 +343,21 @@ public class EamSqliteSettingsDialog extends javax.swing.JDialog { }//GEN-LAST:event_bnTestDatabaseActionPerformed private void bnCancelActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_bnCancelActionPerformed + // TODO: if a new db was created, then delete it. update settings to disable this platform dispose(); }//GEN-LAST:event_bnCancelActionPerformed - private void bnSaveActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_bnSaveActionPerformed + private void bnOkActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_bnOkActionPerformed hasChanged = true; dbSettings.setEnabled(true); dbSettings.saveSettings(); dispose(); - }//GEN-LAST:event_bnSaveActionPerformed + }//GEN-LAST:event_bnOkActionPerformed // Variables declaration - do not modify//GEN-BEGIN:variables private javax.swing.JButton bnCancel; private javax.swing.JButton bnDatabasePathFileOpen; - private javax.swing.JButton bnSave; + private javax.swing.JButton bnOk; private javax.swing.JButton bnTestDatabase; private javax.swing.JFileChooser fcDatabasePath; private javax.swing.JScrollPane jScrollPane;