diff --git a/source/net/sourceforge/filebot/ui/rename/RenamePanel.java b/source/net/sourceforge/filebot/ui/rename/RenamePanel.java index 7f2822b3..0fc6b3ed 100644 --- a/source/net/sourceforge/filebot/ui/rename/RenamePanel.java +++ b/source/net/sourceforge/filebot/ui/rename/RenamePanel.java @@ -73,7 +73,6 @@ public class RenamePanel extends JComponent { protected final RenameAction renameAction = new RenameAction(renameModel); - private static final PreferencesEntry persistentOverrideExtension = Settings.forPackage(RenamePanel.class).entry("rename.extension.override").defaultValue("false"); private static final PreferencesEntry persistentEpisodeFormat = Settings.forPackage(RenamePanel.class).entry("rename.format.episode"); private static final PreferencesEntry persistentMovieFormat = Settings.forPackage(RenamePanel.class).entry("rename.format.movie"); private static final PreferencesEntry persistentPreferredLanguage = Settings.forPackage(RenamePanel.class).entry("rename.language").defaultValue("en"); @@ -87,9 +86,6 @@ public class RenamePanel extends JComponent { filesList.setTitle("Original Files"); filesList.setTransferablePolicy(new FilesListTransferablePolicy(renameModel.files())); - // restore state - renameModel.setPreserveExtension(!Boolean.parseBoolean(persistentOverrideExtension.getValue())); - // filename formatter renameModel.useFormatter(File.class, new FileNameFormatter(renameModel.preserveExtension())); @@ -158,8 +154,8 @@ public class RenamePanel extends JComponent { // create settings popup final Action settingsPopupAction = new ShowPopupAction("Options", ResourceManager.getIcon("action.report")); JButton settingsButton = createImageButton(settingsPopupAction); + settingsButton.setAction(openHistoryAction); ActionPopup settingsPopup = createSettingsPopup(); - settingsButton.setComponentPopupMenu(settingsPopup); renameButton.setComponentPopupMenu(settingsPopup); filesList.getButtonPanel().add(settingsButton, "gap 0"); @@ -287,31 +283,33 @@ public class RenamePanel extends JComponent { actionPopup.addSeparator(); actionPopup.addDescription(new JLabel("History:")); - - actionPopup.add(new AbstractAction("Open History", ResourceManager.getIcon("action.report")) { - - @Override - public void actionPerformed(ActionEvent e) { - History model = HistorySpooler.getInstance().getCompleteHistory(); - - HistoryDialog dialog = new HistoryDialog(getWindow(RenamePanel.this)); - dialog.setLocationRelativeTo(RenamePanel.this); - dialog.setModel(model); - - // show and block - dialog.setVisible(true); - - if (!model.equals(dialog.getModel())) { - // model was changed, switch to the new model - HistorySpooler.getInstance().commit(dialog.getModel()); - } - } - }); + actionPopup.add(openHistoryAction); return actionPopup; } + protected final Action openHistoryAction = new AbstractAction("Open History", ResourceManager.getIcon("action.report")) { + + @Override + public void actionPerformed(ActionEvent e) { + History model = HistorySpooler.getInstance().getCompleteHistory(); + + HistoryDialog dialog = new HistoryDialog(getWindow(RenamePanel.this)); + dialog.setLocationRelativeTo(RenamePanel.this); + dialog.setModel(model); + + // show and block + dialog.setVisible(true); + + if (!model.equals(dialog.getModel())) { + // model was changed, switch to the new model + HistorySpooler.getInstance().commit(dialog.getModel()); + } + } + } + + protected static class ShowPopupAction extends AbstractAction { public ShowPopupAction(String name, Icon icon) { @@ -348,9 +346,6 @@ public class RenamePanel extends JComponent { // display changed state filesList.repaint(); - - // save state - persistentOverrideExtension.setValue(Boolean.toString(activate)); } }