diff --git a/Core/src/org/sleuthkit/autopsy/directorytree/ExternalViewerAction.java b/Core/src/org/sleuthkit/autopsy/directorytree/ExternalViewerAction.java index 3a177d9f88..4a904042c7 100644 --- a/Core/src/org/sleuthkit/autopsy/directorytree/ExternalViewerAction.java +++ b/Core/src/org/sleuthkit/autopsy/directorytree/ExternalViewerAction.java @@ -24,7 +24,9 @@ import java.io.File; import java.io.IOException; import java.util.logging.Level; import javax.swing.AbstractAction; +import javax.swing.JOptionPane; import org.openide.nodes.Node; +import org.openide.util.NbBundle.Messages; import org.sleuthkit.autopsy.casemodule.Case; import org.sleuthkit.autopsy.coreutils.Logger; import org.sleuthkit.autopsy.datamodel.ContentUtils; @@ -67,6 +69,8 @@ public class ExternalViewerAction extends AbstractAction { } } + @Messages({"ExternalViewerAction.actionPerformed.failure.message=Could not find a viewer for the given file.", + "ExternalViewerAction.actionPerformed.failure.title=Open Failure"}) @Override public void actionPerformed(ActionEvent e) { // Get the temp folder path of the case @@ -82,15 +86,14 @@ public class ExternalViewerAction extends AbstractAction { tempFile.createNewFile(); ContentUtils.writeToFile(fileObject, tempFile); } catch (IOException ex) { - // throw an error here logger.log(Level.WARNING, "Can't save to temporary file.", ex); //NON-NLS } try { Desktop.getDesktop().open(tempFile); } catch (IOException ex) { - // if can't open the file, throw the error saying: "File type not supported." - logger.log(Level.WARNING, "File type not supported.", ex); //NON-NLS + logger.log(Level.WARNING, "Could not find a viewer for the given file: " + tempFile.getName(), ex); //NON-NLS + JOptionPane.showMessageDialog(null, Bundle.ExternalViewerAction_actionPerformed_failure_message(), Bundle.ExternalViewerAction_actionPerformed_failure_title(), JOptionPane.ERROR_MESSAGE); } // delete the file on exit