From ea9cc3bca720411ad0bd827d79bc4ca2c794bd04 Mon Sep 17 00:00:00 2001
From: Reinhard Pointner <rednoah@users.noreply.github.com>
Date: Thu, 22 Nov 2012 14:11:55 +0000
Subject: [PATCH] * Added Revert button for people that can't figure out the
 Revert ContextMenu

---
 .../filebot/resources/action.revert.png       | Bin 0 -> 1240 bytes
 .../filebot/ui/rename/HistoryDialog.java      |  78 ++++++++++++++----
 2 files changed, 62 insertions(+), 16 deletions(-)
 create mode 100644 source/net/sourceforge/filebot/resources/action.revert.png

diff --git a/source/net/sourceforge/filebot/resources/action.revert.png b/source/net/sourceforge/filebot/resources/action.revert.png
new file mode 100644
index 0000000000000000000000000000000000000000..4ef7981af1f07bbd6a3387c5de6eaaddd84a124d
GIT binary patch
literal 1240
zcmeAS@N?(olHy`uVBq!ia0vp^0wB!61|;P_|4#%`Y)RhkE<hRxemd{22U45`9+AaB
z+5?Q;PG;MI1j`B>(}6TtKSRv1wTBrPm<2pt978NlFP-e`6O$^^Hve~RdEHETwi$}X
z!3re}2l;&4W@Wr`>x}O8lrFt8C8^VJsvFa`t3I<r?p{qWVH1xv$&!v<-Px&K7L~Lp
zHbgVZYgs_9)ilW&OrMY6sXl!tTD(_7?@mQndf}&ebLKn`{1JYZ=TpzCqB?~TK~WCQ
zgv6O)cXtI}|FZGl=9<svE1#}UdlzX_8E^1;u}@pQF_V|8`9!COAN3}Oe_N9@CHu*<
zy6syYxrMEHUE8^SMILL-cNRZWu^qo#YgRq{@WN1NcUgX}gk4F4a_?oA+c74kTY}E5
zpS*ilfToVb!ba~GObR09Xa9**tVv9HV=*y9<no%&KU<5J#MSgg_?uswtbQ^)YRA{Z
zrBbu@y2!oz*!E~$B*WKJ3e4}s;&bfYCob7Ny><3ylV8j}zP^TrhK*B%e6MS4S6}(J
zX4;=KtYtR0yjLC--)*#lcg6CazxOwB)a_^2R9Tj~Xo14gl$siuhLa~fbIhbySF_!W
znVFib>2GjPZFRYKddT+MXP>ysKmJ&8a=i{?Kt$}b&Hm>C*M?bq`o*y->+mL>J#qS5
z_lewkI(dsrbYQ5Upyux_m49w%ygq80V`k0Z<Z-P#yGLHu*@J!hvi<3cqny1&H(R!?
zS!-9ZP#~z}*7bM$ugzEedriVg@XJ><!7hbbrL${u`fk^!Sbn|t`%U#fRlT~OrcPQn
zuI#rI2)nyUj_YpnijPJwzsx=pyWj1X%dCaX&I>qf<mBD&OFVt_)BooF_nC8!&R+2^
zjp1GNG)<q=yB3FMsg!HU<m^9d{eItB?v*04vTt?t*B88a%KYZ;Zw|ipxo3Z=`Of@&
zV;0kksZw8eZB^m%mDfwzv`mU;<7`PgpHiWlf{x}U3_`mfNwoDPzNo0PY+h~C&(o#>
zRI~8qVXgyrR<HBfv48dP+fN?ed&FB(umAK(;;DHOe#@D+e$f7R^Rk7vq)e=B$hB2Z
z(!}yr{IWmlPHDc|H??5X?brP52HdA}#IysnZFgLKdRXp*V)NQ5MYjX*sZACwh$}WV
z+tMmL_t3-Q{LNqNQ-9f8{pZYz_J5$j*#OEgswJ)wCCNppi6yDJDL{h3$iTo-*U(Vc
z&@{xz*vi1f%D`0Dz}(8fK>y}mf0zc4N-Kyipaw$&T?3#7!w>^YD+3EFV+)9eXFp>r
zQ8eV{r(~v8;?^*I$<w1i4JIHb`D7-i<`t(hWR#Q?Sn2D7?aC}mP0r8F&Ce^=%P&gT
z&w(f@)^`u^(KpgF(9g+YTLH5WViy4~6=a;Q25JxoSs0vOT9lm1;O^(|>#E?HoS&x<
yl98I5dhn-OCQyk2LP>mRkPpmRiA5!P>3R9NsbB{gJ(&pdFoUP7pUXO@geCw~n*H<u

literal 0
HcmV?d00001

diff --git a/source/net/sourceforge/filebot/ui/rename/HistoryDialog.java b/source/net/sourceforge/filebot/ui/rename/HistoryDialog.java
index 5568282c..3f772666 100644
--- a/source/net/sourceforge/filebot/ui/rename/HistoryDialog.java
+++ b/source/net/sourceforge/filebot/ui/rename/HistoryDialog.java
@@ -132,7 +132,8 @@ class HistoryDialog extends JDialog {
 		
 		content.add(new JButton(importAction), "wmin button, hmin 25px, gap indent, sg button");
 		content.add(new JButton(new SaveAction("Export", null, exportHandler)), "gap rel, sg button");
-		content.add(new JButton(closeAction), "gap left unrel:push, gap right indent, sg button");
+		content.add(new JButton(new RevertCurrentSelectionAction()), "gap left unrel:push, sgy button");
+		content.add(new JButton(closeAction), "gap left unrel, gap right indent, sg button");
 		
 		JComponent pane = (JComponent) getContentPane();
 		pane.setLayout(new MigLayout("fill, insets 0, nogrid"));
@@ -315,7 +316,6 @@ class HistoryDialog extends JDialog {
 		return table;
 	}
 	
-	
 	private final Action closeAction = new AbstractAction("Close") {
 		
 		@Override
@@ -377,7 +377,7 @@ class HistoryDialog extends JDialog {
 				
 				if (selection.size() > 0) {
 					JPopupMenu menu = new JPopupMenu();
-					menu.add(new RevertAction(selection, HistoryDialog.this));
+					menu.add(new RevertSelectionAction(selection));
 					
 					// display popup
 					menu.show(table, e.getX(), e.getY());
@@ -387,22 +387,65 @@ class HistoryDialog extends JDialog {
 	};
 	
 	
-	private static class RevertAction extends AbstractAction {
+	private class RevertCurrentSelectionAction extends RevertAction {
+		
+		public RevertCurrentSelectionAction() {
+			super("Revert Selection", HistoryDialog.this);
+		}
+		
+		
+		@Override
+		public List<Element> elements() {
+			List<Element> selection = new ArrayList<Element>();
+			
+			if (sequenceTable.getSelectedRow() >= 0) {
+				for (int i : sequenceTable.getSelectedRows()) {
+					int index = sequenceTable.convertRowIndexToModel(i);
+					selection.addAll(sequenceModel.getRow(index).elements());
+				}
+			} else if (elementTable.getSelectedRow() >= 0) {
+				for (int i : elementTable.getSelectedRows()) {
+					int index = elementTable.convertRowIndexToModel(i);
+					selection.add(elementModel.getRow(index));
+				}
+			}
+			
+			return selection;
+		}
+	}
+	
+	
+	private class RevertSelectionAction extends RevertAction {
 		
 		public static final String ELEMENTS = "elements";
+		
+		
+		public RevertSelectionAction(Collection<Element> elements) {
+			super("Revert...", HistoryDialog.this);
+			putValue(ELEMENTS, elements.toArray(new Element[0]));
+		}
+		
+		
+		@Override
+		public List<Element> elements() {
+			return Arrays.asList((Element[]) getValue(ELEMENTS));
+		}
+	}
+	
+	
+	private static abstract class RevertAction extends AbstractAction {
+		
 		public static final String PARENT = "parent";
 		
 		
-		public RevertAction(Collection<Element> elements, HistoryDialog parent) {
-			putValue(NAME, "Revert...");
-			putValue(ELEMENTS, elements.toArray(new Element[0]));
+		public RevertAction(String name, HistoryDialog parent) {
+			putValue(NAME, name);
+			putValue(SMALL_ICON, ResourceManager.getIcon("action.revert"));
 			putValue(PARENT, parent);
 		}
 		
 		
-		public Element[] elements() {
-			return (Element[]) getValue(ELEMENTS);
-		}
+		public abstract List<Element> elements();
 		
 		
 		public HistoryDialog parent() {
@@ -437,6 +480,10 @@ class HistoryDialog extends JDialog {
 		
 		@Override
 		public void actionPerformed(ActionEvent e) {
+			List<Element> elements = elements();
+			if (elements.isEmpty())
+				return;
+			
 			// use default directory
 			File directory = null;
 			
@@ -451,7 +498,7 @@ class HistoryDialog extends JDialog {
 				Set<Option> options;
 				
 				if (missingFiles.isEmpty()) {
-					message = String.format("Are you sure you want to rename %d file(s)?", elements().length);
+					message = String.format("Are you sure you want to rename %d file(s)?", elements.size());
 					type = QUESTION_MESSAGE;
 					options = EnumSet.of(Option.Rename, Option.ChangeDirectory, Option.Cancel);
 				} else {
@@ -500,12 +547,12 @@ class HistoryDialog extends JDialog {
 			
 			// rename files
 			if (selectedOption == Option.Rename) {
-				rename(directory);
+				rename(directory, elements);
 			}
 		}
 		
 		
-		private void rename(File directory) {
+		private void rename(File directory, List<Element> elements) {
 			int count = 0;
 			
 			for (Entry<File, File> entry : getRenameMap(directory).entrySet()) {
@@ -519,11 +566,11 @@ class HistoryDialog extends JDialog {
 			
 			JLabel status = parent().getInfoLabel();
 			
-			if (count == elements().length) {
+			if (count == elements.size()) {
 				status.setText(String.format("%d file(s) have been renamed.", count));
 				status.setIcon(ResourceManager.getIcon("status.ok"));
 			} else {
-				status.setText(String.format("Failed to revert %d file(s).", elements().length - count, elements().length));
+				status.setText(String.format("Failed to revert %d file(s).", elements.size() - count, elements.size()));
 				status.setIcon(ResourceManager.getIcon("status.error"));
 			}
 			
@@ -568,7 +615,6 @@ class HistoryDialog extends JDialog {
 		}
 	}
 	
-	
 	private final FileTransferablePolicy importHandler = new FileTransferablePolicy() {
 		
 		@Override