Code formatting

Guify/src/code/Constants.java

@@ -7,13 +7,13 @@ public class Constants {
 	public static final String VERSION = "1.0.4";
 	public static final int VERSION_PROGRESSIVE = 4;
 
-	public static class Constants_FSOperations{
+	public static class Constants_FSOperations {
 		public static final int NONE = 0;
 		public static final int CUT = 1;
 		public static final int COPY = 2;
 	}
 
-	public static class GuifyColors{
+	public static class GuifyColors {
 		public static final Color BLUE = new Color(3, 169, 244);
 		public static final Color GRAY = new Color(240, 240, 240);
 		public static final Color GRAY_HOVER = new Color(220, 220, 220);

Guify/src/code/GuiAbstractions/Implementations/JFrameFactory.java

@@ -8,27 +8,28 @@ import views.*;
  */
 public class JFrameFactory implements IFrameFactory {
 
-	public static Object createJFrame(int frameType, Object controller) throws Exception{
+	public static Object createJFrame(int frameType, Object controller)
+			throws Exception {
 
-		switch(frameType) {
+		switch (frameType) {
 
-		case LOGIN:
+			case LOGIN :
-			return new Login(controller);
+				return new Login(controller);
 
-		case DESKTOP:
+			case DESKTOP :
-			return new Desktop(controller);
+				return new Desktop(controller);
 
-		case NOTEPAD:
+			case NOTEPAD :
-			return new Notepad(controller);
+				return new Notepad(controller);
 
-		case FIND_AND_REPLACE:
+			case FIND_AND_REPLACE :
-			return new FindAndReplace(controller);
+				return new FindAndReplace(controller);
 
-		case QUEUE:
+			case QUEUE :
-			return new Queue(controller);
+				return new Queue(controller);
 
-		default:
+			default :
-			throw new Exception("Invalid frame name");
+				throw new Exception("Invalid frame name");
 		}
 	}
 

Guify/src/code/GuiAbstractions/Implementations/JGenericTextArea.java

@@ -5,8 +5,8 @@ import code.GuiAbstractions.Interfaces.IGenericTextArea;
 
 /**
  * 
- * A class implementing an interface for a generic text
+ * A class implementing an interface for a generic text area. It is currently
- * area. It is currently using JTextArea (Java Swing)
+ * using JTextArea (Java Swing)
  *
  */
 public class JGenericTextArea implements IGenericTextArea {
@@ -19,15 +19,15 @@ public class JGenericTextArea implements IGenericTextArea {
 
 	@Override
 	public void selectText(int start, int end) {
-		if(textArea == null)
+		if (textArea == null)
 			return;
 
-	textArea.requestFocus(); // enforce focus or it will not be selected
+		textArea.requestFocus(); // enforce focus or it will not be selected
-	textArea.select(start, end);
+		textArea.select(start, end);
 	}
 
 	public String getText() {
-		if(textArea == null) {
+		if (textArea == null) {
 			throw new NullPointerException("TextArea is null");
 		}
 
@@ -35,7 +35,7 @@ public class JGenericTextArea implements IGenericTextArea {
 	}
 
 	public void setText(String text) {
-		if(textArea == null) {
+		if (textArea == null) {
 			throw new NullPointerException("TextArea is null");
 		}
 
@@ -43,7 +43,7 @@ public class JGenericTextArea implements IGenericTextArea {
 	}
 
 	public void replaceRange(String s, int start, int end) {
-		if(textArea == null) {
+		if (textArea == null) {
 			throw new NullPointerException("TextArea is null");
 		}
 
@@ -52,7 +52,7 @@ public class JGenericTextArea implements IGenericTextArea {
 
 	@Override
 	public void setCaretPosition(int position) {
-		if(textArea == null) {
+		if (textArea == null) {
 			throw new NullPointerException("TextArea is null");
 		}
 
@@ -61,15 +61,15 @@ public class JGenericTextArea implements IGenericTextArea {
 
 	@Override
 	public int getCaretPosition() {
-		if(textArea == null) {
+		if (textArea == null) {
 			throw new NullPointerException("TextArea is null");
 		}
 
 		return textArea.getCaretPosition();
 	}
 
-	public int getSelectionStart(){
+	public int getSelectionStart() {
-		if(textArea == null) {
+		if (textArea == null) {
 			throw new NullPointerException("TextArea is null");
 		}
 		return textArea.getSelectionStart();
@@ -77,7 +77,7 @@ public class JGenericTextArea implements IGenericTextArea {
 
 	@Override
 	public boolean hasHighlightedText() {
-		if(textArea == null) {
+		if (textArea == null) {
 			throw new NullPointerException("TextArea is null");
 		}
 		return textArea.getSelectionStart() != textArea.getCaretPosition();

Guify/src/code/GuiAbstractions/Interfaces/IGenericTextArea.java

@@ -3,17 +3,13 @@ package code.GuiAbstractions.Interfaces;
 /**
  * 
  * 
- * Interface for a generic TextArea.
+ * Interface for a generic TextArea. It is used to create an abstraction of a
- * It is used to create an abstraction of a
+ * TextArea, without using view-specific objects (such as JTextArea).
- * TextArea, without using view-specific objects
- * (such as JTextArea).
  * 
- * This increases modularity, flexibility and
+ * This increases modularity, flexibility and creates a separation of concerns.
- * creates a separation of concerns.
  * 
- * In case of change of the GUI library you
+ * In case of change of the GUI library you do not need to change neither the
- * do not need to change neither the Controllers nor
+ * Controllers nor these interfaces, but only the implementations.
- * these interfaces, but only the implementations.
  *
  */
 public interface IGenericTextArea {

Guify/src/code/GuifySftpProgressMonitor.java

@@ -10,8 +10,9 @@ public class GuifySftpProgressMonitor implements SftpProgressMonitor {
 	@Override
 	public boolean count(long bytes) {
 
-		if(transferProgress != null) {
+		if (transferProgress != null) {
-			transferProgress.setTransferredBytes(transferProgress.getTransferredBytes() + bytes);
+			transferProgress.setTransferredBytes(
+					transferProgress.getTransferredBytes() + bytes);
 			transferProgress.setTransferStatus(TransferProgress.UPDATING);
 			QueueEventManager.getInstance().notify(transferProgress);
 		}
@@ -23,7 +24,7 @@ public class GuifySftpProgressMonitor implements SftpProgressMonitor {
 
 	@Override
 	public void end() {
-		if(transferProgress != null) {
+		if (transferProgress != null) {
 			transferProgress.setTransferStatus(TransferProgress.END);
 			QueueEventManager.getInstance().notify(transferProgress);
 		}

Guify/src/code/Helper.java

@@ -9,7 +9,7 @@ public class Helper {
 	 */
 	public static String combinePath(String s1, String s2) {
 		StringBuilder result = new StringBuilder(s1);
-		if(!s1.endsWith("/")) {
+		if (!s1.endsWith("/")) {
 			result.append('/');
 		}
 		result.append(s2);
@@ -17,13 +17,12 @@ public class Helper {
 	}
 
 	public static String getParentPath(String path) {
-		if(path.equals("/")) {
+		if (path.equals("/")) {
 			return "/";
-		}
+		} else if (path.equals("~")) {
-		else if(path.equals("~")) {
+			return Path.of(SshEngine.executeCommand("pwd")).getParent()
-			return Path.of(SshEngine.executeCommand("pwd")).getParent().toString().replace('\\', '/');
+					.toString().replace('\\', '/');
-		}
+		} else {
-		else {
 			return Path.of(path).getParent().toString().replace('\\', '/');
 		}
 	}

Guify/src/code/IDirectoryNodeButton.java

@@ -4,9 +4,8 @@ import com.jcraft.jsch.ChannelSftp.LsEntry;
 
 /**
  * 
- * Interface describing a DirectoryNodeButton,
+ * Interface describing a DirectoryNodeButton, independently of how a concrete
- * independently of how a concrete DirectoryNodeButton
+ * DirectoryNodeButton will be (currently it is concretely a JButton)
- * will be (currently it is concretely a JButton)
  *
  */
 public interface IDirectoryNodeButton {

Guify/src/code/JDirectoryNodeButton.java

@@ -7,12 +7,13 @@ import com.jcraft.jsch.ChannelSftp.LsEntry;
 
 /**
  * 
- * A JButton representing a Directory Node.
+ * A JButton representing a Directory Node. Useful when a Directory Node is
- * Useful when a Directory Node is needed
+ * needed to be drew on screen.
- * to be drew on screen.
  *
  */
-public class JDirectoryNodeButton extends JButton implements IDirectoryNodeButton {
+public class JDirectoryNodeButton extends JButton
+		implements
+			IDirectoryNodeButton {
 
 	private static final long serialVersionUID = 1L;
 	public ChannelSftp.LsEntry node = null;

Guify/src/code/Main.java

@@ -21,3 +21,5 @@ public class Main {
 	}
 	
 }
+
+// In loving memory of L.B. and E.B.

Guify/src/code/QueueEventManager.java

@@ -7,35 +7,37 @@ public class QueueEventManager extends Observable {
 
 	private static QueueEventManager instance;
 
-    // Cannot instantiate from outside
+	// Cannot instantiate from outside
-    private QueueEventManager() {}
+	private QueueEventManager() {
+	}
 
-    // We need this object in order to retrieve old transfers which are not being transferred
+	// We need this object in order to retrieve old transfers which are not
-    ConcurrentLinkedQueue<TransferProgress> queue = new ConcurrentLinkedQueue<TransferProgress>();
+	// being transferred
+	ConcurrentLinkedQueue<TransferProgress> queue = new ConcurrentLinkedQueue<TransferProgress>();
 
-    public static synchronized QueueEventManager getInstance() {
+	public static synchronized QueueEventManager getInstance() {
-        if (instance == null) {
+		if (instance == null) {
-            instance = new QueueEventManager();
+			instance = new QueueEventManager();
-        }
+		}
-        return instance;
+		return instance;
-    }
+	}
 
-    public void notify(TransferProgress arg) {
+	public void notify(TransferProgress arg) {
-    	updateQueue(arg);
+		updateQueue(arg);
-        setChanged();
+		setChanged();
-        notifyObservers(arg);
+		notifyObservers(arg);
-    }
+	}
 
-    private void updateQueue(TransferProgress transferProgressObj) {
+	private void updateQueue(TransferProgress transferProgressObj) {
-    	if(transferProgressObj.getTransferStatus() == TransferProgress.INIT) {
+		if (transferProgressObj.getTransferStatus() == TransferProgress.INIT) {
-    		queue.add(transferProgressObj);
+			queue.add(transferProgressObj);
-    	}
+		} else if (transferProgressObj
-    	else if(transferProgressObj.getTransferStatus() == TransferProgress.END) {
+				.getTransferStatus() == TransferProgress.END) {
-    		queue.remove();
+			queue.remove();
-    	}
+		}
-    }
+	}
 
-    public TransferProgress[] getQueue() {
+	public TransferProgress[] getQueue() {
-    	return queue.toArray(new TransferProgress[queue.size()]);
+		return queue.toArray(new TransferProgress[queue.size()]);
-    }
+	}
 }

Guify/src/code/SshEngine.java

@@ -16,8 +16,7 @@ import java.nio.file.*;
 
 /**
  * 
- * Underlying SSH engine for
+ * Underlying SSH engine for the application
- * the application
  *
  */
 public class SshEngine {
@@ -38,7 +37,8 @@ public class SshEngine {
 		JSch jsch = new JSch();
 
 		try {
-			session = jsch.getSession(LoginController.LoginCredentials.username, LoginController.LoginCredentials.host,
+			session = jsch.getSession(LoginController.LoginCredentials.username,
+					LoginController.LoginCredentials.host,
 					LoginController.LoginCredentials.port);
 			session.setPassword(LoginController.LoginCredentials.password);
 			session.setConfig(config);
@@ -53,15 +53,15 @@ public class SshEngine {
 	}
 
 	private static void checkValidityOrCreateSession() throws JSchException {
-		if(session == null || !session.isConnected()) {
+		if (session == null || !session.isConnected()) {
-			if(!createSession()) {
+			if (!createSession()) {
 				throw new JSchException("Failed to create a session in Jsch");
 			}
 		}
 	}
 
 	public static void disconnectSession() {
-		if(session != null) {
+		if (session != null) {
 			session.disconnect();
 		}
 	}
@@ -70,14 +70,13 @@ public class SshEngine {
 	 * ========== END SSH Utilities ==========
 	 */
 
-
 	/*
 	 * ========== BEGIN SFTP get() and put() methods ==========
 	 */
 
 	/**
-	 * Downloads a file from remote host to local machine.
+	 * Downloads a file from remote host to local machine. Executed
-	 * Executed asynchronously.
+	 * asynchronously.
 	 */
 	public static void downloadFile(String source, String dest) {
 		// We execute the lengthy and time-consuming operation on a different
@@ -85,36 +84,38 @@ public class SshEngine {
 		// We use SwingWorker so any GUI changes requested by this thread will
 		// be correctly addressed to the Event Dispatch Thread
 		new Thread(new Runnable() {
-		    public void run() {
+			public void run() {
-		    	ChannelSftp channelSftp = null;
+				ChannelSftp channelSftp = null;
 				try {
 					checkValidityOrCreateSession();
 					channelSftp = (ChannelSftp) session.openChannel("sftp");
 					channelSftp.connect();
-					channelSftp.get(source, dest, new GuifySftpProgressMonitor());
+					channelSftp.get(source, dest,
-					System.out.println("File " + source + " downloaded in " + dest);
+							new GuifySftpProgressMonitor());
+					System.out.println(
+							"File " + source + " downloaded in " + dest);
 				} catch (Exception e) {
 					e.printStackTrace();
 				} finally {
 					if (channelSftp != null)
 						channelSftp.disconnect();
 				}
-		    }
+			}
 		}).start();
 	}
 
 	/**
-	 * Downloads a directory recursively.
+	 * Downloads a directory recursively. Executed asynchronously.
-	 * Executed asynchronously.
 	 */
-	public static void downloadDirectoryRecursively(String source, String dest) {
+	public static void downloadDirectoryRecursively(String source,
+			String dest) {
 		// We execute the lengthy and time-consuming operation on a different
 		// thread instead of the Event Dispatch Thread.
 		// We use SwingWorker so any GUI changes requested by this thread will
 		// be correctly addressed to the Event Dispatch Thread
 		new Thread(new Runnable() {
-		    public void run() {
+			public void run() {
-		    	ChannelSftp channelSftp = null;
+				ChannelSftp channelSftp = null;
 				try {
 					channelSftp = (ChannelSftp) session.openChannel("sftp");
 					channelSftp.connect();
@@ -125,139 +126,166 @@ public class SshEngine {
 					if (channelSftp != null)
 						channelSftp.disconnect();
 				}
-		    }
+			}
 		}).start();
 	}
 
 	/**
 	 * Private utility
-	 * @param channel_aux An auxiliary SFTP channel
+	 * 
+	 * @param channel_aux
+	 *            An auxiliary SFTP channel
 	 * @param remoteDirectory
 	 * @param localDirectory
 	 * @throws SftpException
 	 */
-    private static void downloadDirectoryRecursively_aux(ChannelSftp channel_aux, String remoteDirectory, String localDirectory) throws SftpException {
+	private static void downloadDirectoryRecursively_aux(
-    	channel_aux.cd(remoteDirectory);
+			ChannelSftp channel_aux, String remoteDirectory,
-        String newLocalDir = Helper.combinePath(localDirectory, Paths.get(remoteDirectory).getFileName().toString());
+			String localDirectory) throws SftpException {
-        new java.io.File(newLocalDir).mkdirs();
+		channel_aux.cd(remoteDirectory);
-        @SuppressWarnings("unchecked")
+		String newLocalDir = Helper.combinePath(localDirectory,
+				Paths.get(remoteDirectory).getFileName().toString());
+		new java.io.File(newLocalDir).mkdirs();
+		@SuppressWarnings("unchecked")
 		Vector<LsEntry> entries = channel_aux.ls("*");
 
-        for (ChannelSftp.LsEntry entry : entries) {
+		for (ChannelSftp.LsEntry entry : entries) {
-            if (!entry.getAttrs().isDir()) {
+			if (!entry.getAttrs().isDir()) {
-                // File - download it
+				// File - download it
-            	// Creates a thread for each file. If there are a lot of files
+				// Creates a thread for each file. If there are a lot of files
-            	// it may be resource-draining. Consider using a ThreadPool
+				// it may be resource-draining. Consider using a ThreadPool
-		    	downloadFile(Helper.combinePath(remoteDirectory,  entry.getFilename()), Helper.combinePath(newLocalDir, entry.getFilename()));
+				downloadFile(
-            } else if (!".".equals(entry.getFilename()) && !"..".equals(entry.getFilename())) {
+						Helper.combinePath(remoteDirectory,
-            	// Directory - download recursively
+								entry.getFilename()),
-                String newRemoteDir = Helper.combinePath(remoteDirectory, entry.getFilename());
+						Helper.combinePath(newLocalDir, entry.getFilename()));
-                downloadDirectoryRecursively_aux(channel_aux, newRemoteDir, newLocalDir);
+			} else if (!".".equals(entry.getFilename())
-            }
+					&& !"..".equals(entry.getFilename())) {
-        }
+				// Directory - download recursively
-    }
+				String newRemoteDir = Helper.combinePath(remoteDirectory,
+						entry.getFilename());
+				downloadDirectoryRecursively_aux(channel_aux, newRemoteDir,
+						newLocalDir);
+			}
+		}
+	}
 
 	/**
-	 * Uploads a file from the local machine to the remote host.
+	 * Uploads a file from the local machine to the remote host. Executed
-	 * Executed asynchronously.
+	 * asynchronously.
 	 */
-	public static void uploadFile(File fileToUpload, String remoteDirectory) throws SftpException {
+	public static void uploadFile(File fileToUpload, String remoteDirectory)
+			throws SftpException {
 		// We execute the lengthy and time-consuming operation on a different
 		// thread instead of the Event Dispatch Thread.
 		// We use SwingWorker so any GUI changes requested by this thread will
 		// be correctly addressed to the Event Dispatch Thread
 		new Thread(new Runnable() {
-		    public void run() {
+			public void run() {
-		    	ChannelSftp channelSftp = null;
+				ChannelSftp channelSftp = null;
 				String remotePath = null;
 				try {
 					checkValidityOrCreateSession();
 					channelSftp = (ChannelSftp) session.openChannel("sftp");
 					channelSftp.connect();
-					remotePath = Helper.combinePath(remoteDirectory, fileToUpload.getName());
+					remotePath = Helper.combinePath(remoteDirectory,
-					channelSftp.put(fileToUpload.getAbsolutePath(), remotePath, new GuifySftpProgressMonitor());
+							fileToUpload.getName());
-					System.out.println("File: " + fileToUpload.getAbsolutePath() + " uploaded to remote path: " + remotePath);
+					channelSftp.put(fileToUpload.getAbsolutePath(), remotePath,
-				}
+							new GuifySftpProgressMonitor());
-				catch(SftpException sftpex) {
+					System.out.println("File: " + fileToUpload.getAbsolutePath()
+							+ " uploaded to remote path: " + remotePath);
+				} catch (SftpException sftpex) {
 					// TODO maybe no permissions
-				}
+				} catch (Exception e) {
-				catch (Exception e) {
 					e.printStackTrace();
 				} finally {
 					if (channelSftp != null)
 						channelSftp.disconnect();
 				}
-		    }
+			}
 		}).start();
 	}
 
 	/**
-	 * Uploads directory recursively.
+	 * Uploads directory recursively. Executed asynchronously.
-	 * Executed asynchronously.
+	 * 
-	 * @param directory Full path of the local directory to upload
+	 * @param directory
-	 * @param remoteDirectory Full path of the remote directory which the local
+	 *            Full path of the local directory to upload
-	 * directory will be uploaded in
+	 * @param remoteDirectory
+	 *            Full path of the remote directory which the local directory
+	 *            will be uploaded in
 	 */
-	public static void uploadDirectoriesRecursively(File directory, String remoteDirectory) throws SftpException {
+	public static void uploadDirectoriesRecursively(File directory,
+			String remoteDirectory) throws SftpException {
 		// We execute the lengthy and time-consuming operation on a different
 		// thread instead of the Event Dispatch Thread.
 		// We use SwingWorker so any GUI changes requested by this thread will
 		// be correctly addressed to the Event Dispatch Thread
 		new Thread(new Runnable() {
-		    public void run() {
+			public void run() {
 				ChannelSftp channelSftp = null;
 				try {
 					checkValidityOrCreateSession();
 					channelSftp = (ChannelSftp) session.openChannel("sftp");
 					channelSftp.connect();
-					uploadDirectoriesRecursively_aux(channelSftp, directory, remoteDirectory);
+					uploadDirectoriesRecursively_aux(channelSftp, directory,
-				} 
+							remoteDirectory);
-				catch(SftpException sftpex) {
+				} catch (SftpException sftpex) {
-					//TODO maybe no permissions
+					// TODO maybe no permissions
-				}
+				} catch (Exception e) {
-				catch (Exception e) {
 					e.printStackTrace();
 				} finally {
 					if (channelSftp != null)
 						channelSftp.disconnect();
 				}
-		    }
+			}
 		}).start();
 	}
 
 	/**
 	 * Private utility
+	 * 
 	 * @param channel_aux
 	 * @param localPath
 	 * @param remoteDirectory
 	 * @throws SftpException
 	 */
-    private static void uploadDirectoriesRecursively_aux(ChannelSftp channel_aux, File localPath, String remoteDirectory) throws SftpException {
+	private static void uploadDirectoriesRecursively_aux(
-    	if(localPath != null) {
+			ChannelSftp channel_aux, File localPath, String remoteDirectory)
-    		String subDirectoryPath = Helper.combinePath(remoteDirectory, localPath.getName());
+			throws SftpException {
-    		channel_aux.mkdir(subDirectoryPath);
+		if (localPath != null) {
+			String subDirectoryPath = Helper.combinePath(remoteDirectory,
+					localPath.getName());
+			channel_aux.mkdir(subDirectoryPath);
 
-            File[] files = localPath.listFiles();
+			File[] files = localPath.listFiles();
-            if (files != null) {
+			if (files != null) {
-                for (File file : files) {
+				for (File file : files) {
-                    if (file.isFile()) {
+					if (file.isFile()) {
-                    	// Creates a thread for each file. If there are a lot of files
+						// Creates a thread for each file. If there are a lot of
-                    	// it may be resource-draining. Consider using a ThreadPool
+						// files
-                    	channel_aux.put(file.getAbsolutePath(), Helper.combinePath(subDirectoryPath, file.getName()), new GuifySftpProgressMonitor());
+						// it may be resource-draining. Consider using a
-                        System.out.println("File: " + file.getAbsolutePath() + " uploaded to remote path: " +  Helper.combinePath(subDirectoryPath, file.getName()));
+						// ThreadPool
-                    } else if (file.isDirectory()) {
+						channel_aux.put(file.getAbsolutePath(),
-                    	uploadDirectoriesRecursively_aux(channel_aux, file, subDirectoryPath);
+								Helper.combinePath(subDirectoryPath,
-                    }
+										file.getName()),
-                }
+								new GuifySftpProgressMonitor());
-            }
+						System.out.println("File: " + file.getAbsolutePath()
-    	}
+								+ " uploaded to remote path: "
-    }
+								+ Helper.combinePath(subDirectoryPath,
+										file.getName()));
+					} else if (file.isDirectory()) {
+						uploadDirectoriesRecursively_aux(channel_aux, file,
+								subDirectoryPath);
+					}
+				}
+			}
+		}
+	}
 
 	/*
 	 * ========== END SFTP get() and put() methods ==========
 	 */
 
-    
 	/*
 	 * ========== BEGIN File System operations ==========
 	 */
@@ -269,11 +297,9 @@ public class SshEngine {
 			channelSftp = (ChannelSftp) session.openChannel("sftp");
 			channelSftp.connect();
 			channelSftp.mkdir(path);
-		}
+		} catch (SftpException sftpex) {
-		catch(SftpException sftpex) {
+			throw sftpex;
-				throw sftpex;
+		} catch (Exception e) {
-		}
-		catch (Exception e) {
 			e.printStackTrace();
 		} finally {
 			if (channelSftp != null)
@@ -303,7 +329,8 @@ public class SshEngine {
 			checkValidityOrCreateSession();
 			channelSftp = (ChannelSftp) session.openChannel("sftp");
 			channelSftp.connect();
-			channelSftp.put(new ByteArrayInputStream(content.getBytes()), pathToFile);
+			channelSftp.put(new ByteArrayInputStream(content.getBytes()),
+					pathToFile);
 		} catch (Exception e) {
 			e.printStackTrace();
 		} finally {
@@ -312,18 +339,17 @@ public class SshEngine {
 		}
 	}
 
-	public static void rename(String oldPath, String newPath) throws SftpException {
+	public static void rename(String oldPath, String newPath)
+			throws SftpException {
 		ChannelSftp channelSftp = null;
 		try {
 			checkValidityOrCreateSession();
 			channelSftp = (ChannelSftp) session.openChannel("sftp");
 			channelSftp.connect();
 			channelSftp.rename(oldPath, newPath);
-		} 
+		} catch (SftpException sftpex) {
-		catch(SftpException sftpex) {
 			throw sftpex;
-		}
+		} catch (Exception e) {
-		catch (Exception e) {
 			e.printStackTrace();
 		} finally {
 			if (channelSftp != null)
@@ -333,6 +359,7 @@ public class SshEngine {
 
 	/**
 	 * Creates an empty file in the specified remote file path
+	 * 
 	 * @throws SftpException
 	 */
 	public static void touch(String remoteFilePath) throws SftpException {
@@ -341,12 +368,11 @@ public class SshEngine {
 			checkValidityOrCreateSession();
 			channelSftp = (ChannelSftp) session.openChannel("sftp");
 			channelSftp.connect();
-			channelSftp.put(new ByteArrayInputStream(new byte[0]), remoteFilePath);
+			channelSftp.put(new ByteArrayInputStream(new byte[0]),
-		} 
+					remoteFilePath);
-		catch(SftpException sftpex) {
+		} catch (SftpException sftpex) {
 			throw sftpex;
-		}
+		} catch (Exception e) {
-		catch (Exception e) {
 			e.printStackTrace();
 		} finally {
 			if (channelSftp != null)
@@ -361,11 +387,9 @@ public class SshEngine {
 			channelSftp = (ChannelSftp) session.openChannel("sftp");
 			channelSftp.connect();
 			channelSftp.rm(remoteFilePath);
-		} 
+		} catch (SftpException sftpex) {
-		catch(SftpException sftpex) {
 			throw sftpex;
-		}
+		} catch (Exception e) {
-		catch (Exception e) {
 			e.printStackTrace();
 		} finally {
 			if (channelSftp != null)
@@ -379,14 +403,12 @@ public class SshEngine {
 			checkValidityOrCreateSession();
 			channelSftp = (ChannelSftp) session.openChannel("sftp");
 			channelSftp.connect();
-			for(String remoteFilePath : remoteFilePaths) {
+			for (String remoteFilePath : remoteFilePaths) {
 				rm(remoteFilePath, channelSftp);
 			}
-		} 
+		} catch (SftpException sftpex) {
-		catch(SftpException sftpex) {
 			throw sftpex;
-		}
+		} catch (Exception e) {
-		catch (Exception e) {
 			e.printStackTrace();
 		} finally {
 			if (channelSftp != null)
@@ -394,7 +416,8 @@ public class SshEngine {
 		}
 	}
 
-	private static void rm(String remoteFilePath, ChannelSftp channelSftp) throws SftpException, JSchException {
+	private static void rm(String remoteFilePath, ChannelSftp channelSftp)
+			throws SftpException, JSchException {
 		channelSftp.rm(remoteFilePath);
 	}
 
@@ -405,11 +428,9 @@ public class SshEngine {
 			channelSftp = (ChannelSftp) session.openChannel("sftp");
 			channelSftp.connect();
 			channelSftp.rmdir(remoteFilePath);
-		} 
+		} catch (SftpException sftpex) {
-		catch(SftpException sftpex) {
 			throw sftpex;
-		}
+		} catch (Exception e) {
-		catch (Exception e) {
 			e.printStackTrace();
 		} finally {
 			if (channelSftp != null)
@@ -423,14 +444,12 @@ public class SshEngine {
 			checkValidityOrCreateSession();
 			channelSftp = (ChannelSftp) session.openChannel("sftp");
 			channelSftp.connect();
-			for(String remotePath : remotePaths) {
+			for (String remotePath : remotePaths) {
 				rmdir(remotePath, channelSftp);
 			}
-		} 
+		} catch (SftpException sftpex) {
-		catch(SftpException sftpex) {
 			throw sftpex;
-		}
+		} catch (Exception e) {
-		catch (Exception e) {
 			e.printStackTrace();
 		} finally {
 			if (channelSftp != null)
@@ -438,12 +457,14 @@ public class SshEngine {
 		}
 	}
 
-	private static void rmdir(String remotePath, ChannelSftp channelSftp) throws SftpException, JSchException {
+	private static void rmdir(String remotePath, ChannelSftp channelSftp)
+			throws SftpException, JSchException {
 		channelSftp.rmdir(remotePath);
 	}
 
 	@SuppressWarnings("unchecked")
-	public static Vector<ChannelSftp.LsEntry> ls(String path) throws SftpException{
+	public static Vector<ChannelSftp.LsEntry> ls(String path)
+			throws SftpException {
 		ChannelSftp channelSftp = null;
 		try {
 			checkValidityOrCreateSession();
@@ -451,13 +472,13 @@ public class SshEngine {
 			channelSftp.connect();
 			Vector<ChannelSftp.LsEntry> entries = channelSftp.ls(path);
 			// remove hidden directories (TODO create a setting for that)
-			entries.removeIf(entry -> entry.getFilename().equals(".") || entry.getFilename().equals("..") || entry.getFilename().startsWith("."));
+			entries.removeIf(entry -> entry.getFilename().equals(".")
+					|| entry.getFilename().equals("..")
+					|| entry.getFilename().startsWith("."));
 			return entries;
-		}
+		} catch (SftpException sftpex) {
-		catch(SftpException sftpex) {
 			throw sftpex;
-		}
+		} catch (Exception e) {
-		catch (Exception e) {
 			e.printStackTrace();
 			return null;
 		} finally {
@@ -470,7 +491,6 @@ public class SshEngine {
 	 * ========== END File System operations ==========
 	 */
 
-	
 	/*
 	 * ========== BEGIN Other ==========
 	 */
@@ -485,7 +505,8 @@ public class SshEngine {
 			((ChannelExec) channel).setCommand(command);
 			channel.connect();
 			in = channel.getInputStream();
-			String returnText = new String(in.readAllBytes(), StandardCharsets.UTF_8);
+			String returnText = new String(in.readAllBytes(),
+					StandardCharsets.UTF_8);
 			in.close();
 			// Remove possible \r\n at the end of the string
 			returnText = returnText.replaceAll("[\r\n]+$", "").trim();
@@ -507,7 +528,6 @@ public class SshEngine {
 	 * ========== END Other ==========
 	 */
 
-	
 }
 /*
  * When executing a command on a remote server using SSH with Jsch in Java, we

Guify/src/code/TransferProgress.java

@@ -2,8 +2,8 @@ package code;
 
 /**
  * 
- * An object representing the transfer progress of a
+ * An object representing the transfer progress of a file between the server and
- * file between the server and the host machine.
+ * the host machine.
  *
  */
 public class TransferProgress {

Guify/src/code/TreeNode.java

@@ -2,8 +2,7 @@ package code;
 
 /**
  * 
- * A class representing a
+ * A class representing a "tree -J" result
- * "tree -J" result
  *
  */
 public class TreeNode {

Guify/src/code/WrapLayout.java

@@ -5,188 +5,191 @@ import javax.swing.JScrollPane;
 import javax.swing.SwingUtilities;
 
 /**
- *  FlowLayout subclass that fully supports wrapping of components.
+ * FlowLayout subclass that fully supports wrapping of components.
  */
-public class WrapLayout extends FlowLayout
+public class WrapLayout extends FlowLayout {
-{
 	/**
 	 * 
 	 */
 	private static final long serialVersionUID = 1L;
 
 	/**
-	* Constructs a new <code>WrapLayout</code> with a left
+	 * Constructs a new <code>WrapLayout</code> with a left alignment and a
-	* alignment and a default 5-unit horizontal and vertical gap.
+	 * default 5-unit horizontal and vertical gap.
-	*/
+	 */
-	public WrapLayout()
+	public WrapLayout() {
-	{
 		super();
 	}
 
 	/**
-	* Constructs a new <code>FlowLayout</code> with the specified
+	 * Constructs a new <code>FlowLayout</code> with the specified alignment and
-	* alignment and a default 5-unit horizontal and vertical gap.
+	 * a default 5-unit horizontal and vertical gap. The value of the alignment
-	* The value of the alignment argument must be one of
+	 * argument must be one of <code>WrapLayout</code>, <code>WrapLayout</code>,
-	* <code>WrapLayout</code>, <code>WrapLayout</code>,
+	 * or <code>WrapLayout</code>.
-	* or <code>WrapLayout</code>.
+	 * 
-	* @param align the alignment value
+	 * @param align
-	*/
+	 *            the alignment value
-	public WrapLayout(int align)
+	 */
-	{
+	public WrapLayout(int align) {
 		super(align);
 	}
 
 	/**
-	* Creates a new flow layout manager with the indicated alignment
+	 * Creates a new flow layout manager with the indicated alignment and the
-	* and the indicated horizontal and vertical gaps.
+	 * indicated horizontal and vertical gaps.
-	* <p>
+	 * <p>
-	* The value of the alignment argument must be one of
+	 * The value of the alignment argument must be one of
-	* <code>WrapLayout</code>, <code>WrapLayout</code>,
+	 * <code>WrapLayout</code>, <code>WrapLayout</code>, or
-	* or <code>WrapLayout</code>.
+	 * <code>WrapLayout</code>.
-	* @param align the alignment value
+	 * 
-	* @param hgap the horizontal gap between components
+	 * @param align
-	* @param vgap the vertical gap between components
+	 *            the alignment value
-	*/
+	 * @param hgap
-	public WrapLayout(int align, int hgap, int vgap)
+	 *            the horizontal gap between components
-	{
+	 * @param vgap
+	 *            the vertical gap between components
+	 */
+	public WrapLayout(int align, int hgap, int vgap) {
 		super(align, hgap, vgap);
 	}
 
 	/**
-	* Returns the preferred dimensions for this layout given the
+	 * Returns the preferred dimensions for this layout given the <i>visible</i>
-	* <i>visible</i> components in the specified target container.
+	 * components in the specified target container.
-	* @param target the component which needs to be laid out
+	 * 
-	* @return the preferred dimensions to lay out the
+	 * @param target
-	* subcomponents of the specified container
+	 *            the component which needs to be laid out
-	*/
+	 * @return the preferred dimensions to lay out the subcomponents of the
+	 *         specified container
+	 */
 	@Override
-	public Dimension preferredLayoutSize(Container target)
+	public Dimension preferredLayoutSize(Container target) {
-	{
 		return layoutSize(target, true);
 	}
 
 	/**
-	* Returns the minimum dimensions needed to layout the <i>visible</i>
+	 * Returns the minimum dimensions needed to layout the <i>visible</i>
-	* components contained in the specified target container.
+	 * components contained in the specified target container.
-	* @param target the component which needs to be laid out
+	 * 
-	* @return the minimum dimensions to lay out the
+	 * @param target
-	* subcomponents of the specified container
+	 *            the component which needs to be laid out
-	*/
+	 * @return the minimum dimensions to lay out the subcomponents of the
+	 *         specified container
+	 */
 	@Override
-	public Dimension minimumLayoutSize(Container target)
+	public Dimension minimumLayoutSize(Container target) {
-	{
 		Dimension minimum = layoutSize(target, false);
 		minimum.width -= (getHgap() + 1);
 		return minimum;
 	}
 
 	/**
-	* Returns the minimum or preferred dimension needed to layout the target
+	 * Returns the minimum or preferred dimension needed to layout the target
-	* container.
+	 * container.
-	*
+	 *
-	* @param target target to get layout size for
+	 * @param target
-	* @param preferred should preferred size be calculated
+	 *            target to get layout size for
-	* @return the dimension to layout the target container
+	 * @param preferred
-	*/
+	 *            should preferred size be calculated
-	private Dimension layoutSize(Container target, boolean preferred)
+	 * @return the dimension to layout the target container
-	{
+	 */
-	synchronized (target.getTreeLock())
+	private Dimension layoutSize(Container target, boolean preferred) {
-	{
+		synchronized (target.getTreeLock()) {
-		//  Each row must fit with the width allocated to the containter.
+			// Each row must fit with the width allocated to the containter.
-		//  When the container width = 0, the preferred width of the container
+			// When the container width = 0, the preferred width of the
-		//  has not yet been calculated so lets ask for the maximum.
+			// container
+			// has not yet been calculated so lets ask for the maximum.
 
-		int targetWidth = target.getSize().width;
+			int targetWidth = target.getSize().width;
-		Container container = target;
+			Container container = target;
 
-		while (container.getSize().width == 0 && container.getParent() != null)
+			while (container.getSize().width == 0
-		{
+					&& container.getParent() != null) {
-			container = container.getParent();
+				container = container.getParent();
-		}
-
-		targetWidth = container.getSize().width;
-
-		if (targetWidth == 0)
-			targetWidth = Integer.MAX_VALUE;
-
-		int hgap = getHgap();
-		int vgap = getVgap();
-		Insets insets = target.getInsets();
-		int horizontalInsetsAndGap = insets.left + insets.right + (hgap * 2);
-		int maxWidth = targetWidth - horizontalInsetsAndGap;
-
-		//  Fit components into the allowed width
-
-		Dimension dim = new Dimension(0, 0);
-		int rowWidth = 0;
-		int rowHeight = 0;
-
-		int nmembers = target.getComponentCount();
-
-		for (int i = 0; i < nmembers; i++)
-		{
-			Component m = target.getComponent(i);
-
-			if (m.isVisible())
-			{
-				Dimension d = preferred ? m.getPreferredSize() : m.getMinimumSize();
-
-				//  Can't add the component to current row. Start a new row.
-
-				if (rowWidth + d.width > maxWidth)
-				{
-					addRow(dim, rowWidth, rowHeight);
-					rowWidth = 0;
-					rowHeight = 0;
-				}
-
-				//  Add a horizontal gap for all components after the first
-
-				if (rowWidth != 0)
-				{
-					rowWidth += hgap;
-				}
-
-				rowWidth += d.width;
-				rowHeight = Math.max(rowHeight, d.height);
 			}
+
+			targetWidth = container.getSize().width;
+
+			if (targetWidth == 0)
+				targetWidth = Integer.MAX_VALUE;
+
+			int hgap = getHgap();
+			int vgap = getVgap();
+			Insets insets = target.getInsets();
+			int horizontalInsetsAndGap = insets.left + insets.right
+					+ (hgap * 2);
+			int maxWidth = targetWidth - horizontalInsetsAndGap;
+
+			// Fit components into the allowed width
+
+			Dimension dim = new Dimension(0, 0);
+			int rowWidth = 0;
+			int rowHeight = 0;
+
+			int nmembers = target.getComponentCount();
+
+			for (int i = 0; i < nmembers; i++) {
+				Component m = target.getComponent(i);
+
+				if (m.isVisible()) {
+					Dimension d = preferred
+							? m.getPreferredSize()
+							: m.getMinimumSize();
+
+					// Can't add the component to current row. Start a new row.
+
+					if (rowWidth + d.width > maxWidth) {
+						addRow(dim, rowWidth, rowHeight);
+						rowWidth = 0;
+						rowHeight = 0;
+					}
+
+					// Add a horizontal gap for all components after the first
+
+					if (rowWidth != 0) {
+						rowWidth += hgap;
+					}
+
+					rowWidth += d.width;
+					rowHeight = Math.max(rowHeight, d.height);
+				}
+			}
+
+			addRow(dim, rowWidth, rowHeight);
+
+			dim.width += horizontalInsetsAndGap;
+			dim.height += insets.top + insets.bottom + vgap * 2;
+
+			// When using a scroll pane or the DecoratedLookAndFeel we need to
+			// make sure the preferred size is less than the size of the
+			// target containter so shrinking the container size works
+			// correctly. Removing the horizontal gap is an easy way to do this.
+
+			Container scrollPane = SwingUtilities
+					.getAncestorOfClass(JScrollPane.class, target);
+
+			if (scrollPane != null && target.isValid()) {
+				dim.width -= (hgap + 1);
+			}
+
+			return dim;
 		}
-
-		addRow(dim, rowWidth, rowHeight);
-
-		dim.width += horizontalInsetsAndGap;
-		dim.height += insets.top + insets.bottom + vgap * 2;
-
-		//	When using a scroll pane or the DecoratedLookAndFeel we need to
-		//  make sure the preferred size is less than the size of the
-		//  target containter so shrinking the container size works
-		//  correctly. Removing the horizontal gap is an easy way to do this.
-
-		Container scrollPane = SwingUtilities.getAncestorOfClass(JScrollPane.class, target);
-
-		if (scrollPane != null && target.isValid())
-		{
-			dim.width -= (hgap + 1);
-		}
-
-		return dim;
-	}
 	}
 
 	/*
-	 *  A new row has been completed. Use the dimensions of this row
+	 * A new row has been completed. Use the dimensions of this row to update
-	 *  to update the preferred size for the container.
+	 * the preferred size for the container.
 	 *
-	 *  @param dim update the width and height when appropriate
+	 * @param dim update the width and height when appropriate
-	 *  @param rowWidth the width of the row to add
+	 * 
-	 *  @param rowHeight the height of the row to add
+	 * @param rowWidth the width of the row to add
+	 * 
+	 * @param rowHeight the height of the row to add
 	 */
-	private void addRow(Dimension dim, int rowWidth, int rowHeight)
+	private void addRow(Dimension dim, int rowWidth, int rowHeight) {
-	{
 		dim.width = Math.max(dim.width, rowWidth);
 
-		if (dim.height > 0)
+		if (dim.height > 0) {
-		{
 			dim.height += getVgap();
 		}
 

Guify/src/controllers/DesktopController.java

@@ -38,10 +38,10 @@ public class DesktopController {
 
 	public DesktopController() {
 		try {
-			frame = (IDesktopFrame) JFrameFactory.createJFrame(JFrameFactory.DESKTOP, this);
+			frame = (IDesktopFrame) JFrameFactory
+					.createJFrame(JFrameFactory.DESKTOP, this);
 			frame.drawComponentsForDirectory("~");
-		} 
+		} catch (Exception e) {
-		catch (Exception e) {
 			e.printStackTrace();
 		}
 	}
@@ -50,7 +50,6 @@ public class DesktopController {
 	 * ========== END Constructors ==========
 	 */
 
-	
 	/*
 	 * ========== BEGIN Getters and Setters ==========
 	 */
@@ -60,10 +59,9 @@ public class DesktopController {
 	}
 
 	public void setCurrentWorkingDirectory(String directory) {
-		if(directory.equals("~")) {
+		if (directory.equals("~")) {
 			currentWorkingDirectory = SshEngine.executeCommand("pwd");
-		}
+		} else {
-		else {
 			currentWorkingDirectory = directory.trim();
 		}
 	}
@@ -74,15 +72,14 @@ public class DesktopController {
 
 	public void setLastSafeDirectory(String directory) {
 
-		if(directory == null) {
+		if (directory == null) {
 			lastSafeDirectory = null;
 			return;
 		}
 
-		if(directory.equals("~")) {
+		if (directory.equals("~")) {
 			lastSafeDirectory = SshEngine.executeCommand("pwd");
-		}
+		} else {
-		else {
 			lastSafeDirectory = directory.trim();
 		}
 	}
@@ -91,7 +88,6 @@ public class DesktopController {
 	 * ========== END Getters and Setters ==========
 	 */
 
-	
 	/*
 	 * ========== BEGIN Create desktop helper methods ==========
 	 */
@@ -109,19 +105,20 @@ public class DesktopController {
 			// Might throw Invalid JSON exception because of incorrect
 			// JSON output returned from tree:
 			// https://gitlab.com/OldManProgrammer/unix-tree/-/issues/11
-			// Fixed in tree 2.1.1 https://gitlab.com/OldManProgrammer/unix-tree/-/commit/84fa3ddff51b30835a0f9c4a9e4c9225970f9aff
+			// Fixed in tree 2.1.1
+			// https://gitlab.com/OldManProgrammer/unix-tree/-/commit/84fa3ddff51b30835a0f9c4a9e4c9225970f9aff
 			//
 			// For this reason, we temporarily explicitly avoid it to happen
 			jsonTree = jsonTree.replace("}{\"error\"", "},{\"error\"");
 			tree = new Gson().fromJson(jsonTree, TreeNode[].class);
 			return tree[0];
-		}
+		} catch (Exception ex) {
-		catch(Exception ex) {
 			return null;
 		}
 	}
 
-	public Vector<ChannelSftp.LsEntry> getDirectoryElements() throws SftpException {
+	public Vector<ChannelSftp.LsEntry> getDirectoryElements()
+			throws SftpException {
 		return SshEngine.ls(currentWorkingDirectory);
 	}
 
@@ -129,15 +126,17 @@ public class DesktopController {
 	 * ========== END Create desktop helper methods ==========
 	 */
 
-	
 	/*
 	 * ========== BEGIN Download and Upload section ==========
 	 */
 
 	/**
 	 * Downloads a file from the remote server to the local machine
-	 * @param sourcePath Remote file's full path
+	 * 
-	 * @param destinationPath Local file's  full path
+	 * @param sourcePath
+	 *            Remote file's full path
+	 * @param destinationPath
+	 *            Local file's full path
 	 */
 	public void downloadFile(String sourcePath, String destinationPath) {
 		SshEngine.downloadFile(sourcePath, destinationPath);
@@ -145,40 +144,40 @@ public class DesktopController {
 
 	/**
 	 * Uploads files and folders to the remote server
+	 * 
 	 * @param selectedNodes
 	 * @throws SftpException
 	 */
-	public void uploadToRemoteServer(File[] selectedNodes) throws SftpException {
+	public void uploadToRemoteServer(File[] selectedNodes)
-        if (selectedNodes.length > 0) {
+			throws SftpException {
-        	List<File> selectedFiles = new ArrayList<File>();
+		if (selectedNodes.length > 0) {
-        	List<File> selectedDirectories = new ArrayList<File>();
+			List<File> selectedFiles = new ArrayList<File>();
-            for (java.io.File file : selectedNodes) {
+			List<File> selectedDirectories = new ArrayList<File>();
-            	if(file.isFile()) {
+			for (java.io.File file : selectedNodes) {
-            		selectedFiles.add(file);
+				if (file.isFile()) {
-            	}
+					selectedFiles.add(file);
-            	else if(file.isDirectory()) {
+				} else if (file.isDirectory()) {
-            		selectedDirectories.add(file);
+					selectedDirectories.add(file);
-            	}
+				}
-            }
+			}
 
-            for(File file : selectedFiles) {
+			for (File file : selectedFiles) {
-            	SshEngine.uploadFile(file, this.getCurrentWorkingDirectory());
+				SshEngine.uploadFile(file, this.getCurrentWorkingDirectory());
-            }
+			}
 
-            
+			if (selectedDirectories.size() > 0) {
-            if(selectedDirectories.size() > 0) {
+				for (File directory : selectedDirectories) {
-            	for(File directory : selectedDirectories) {
+					SshEngine.uploadDirectoriesRecursively(directory,
-                	SshEngine.uploadDirectoriesRecursively(directory, this.getCurrentWorkingDirectory());
+							this.getCurrentWorkingDirectory());
-            	}
+				}
-            }
+			}
-        }
+		}
 	}
 
 	/*
 	 * ========== END Download and Upload section ==========
 	 */
 
-	
 	/*
 	 * ========== BEGIN Selected Nodes section ==========
 	 */
@@ -194,12 +193,12 @@ public class DesktopController {
 	}
 
 	public void clearSelectedNodes() {
-		if(selectedNodes != null) {
+		if (selectedNodes != null) {
 			Iterator<IDirectoryNodeButton> iterator = selectedNodes.iterator();
 			while (iterator.hasNext()) {
-			    IDirectoryNodeButton node = iterator.next();
+				IDirectoryNodeButton node = iterator.next();
-			    iterator.remove();
+				iterator.remove();
-			    node.setSelected(false);
+				node.setSelected(false);
 			}
 		}
 	}
@@ -217,12 +216,17 @@ public class DesktopController {
 		List<String> filesToDelete = new ArrayList<String>();
 		List<String> directoriesToDelete = new ArrayList<String>();
 
-		for(IDirectoryNodeButton node : selectedNodes) {
+		for (IDirectoryNodeButton node : selectedNodes) {
-			if(node.getNode().getAttrs().isDir()) {
+			if (node.getNode().getAttrs().isDir()) {
-				directoriesToDelete.add(Helper.combinePath(getCurrentWorkingDirectory(), node.getNode().getFilename()).replace("\"", "\\\""));
+				directoriesToDelete.add(Helper
-			}
+						.combinePath(getCurrentWorkingDirectory(),
-			else {
+								node.getNode().getFilename())
-				filesToDelete.add(Helper.combinePath(getCurrentWorkingDirectory(), node.getNode().getFilename()).replace("\"", "\\\""));
+						.replace("\"", "\\\""));
+			} else {
+				filesToDelete.add(Helper
+						.combinePath(getCurrentWorkingDirectory(),
+								node.getNode().getFilename())
+						.replace("\"", "\\\""));
 			}
 		}
 
@@ -236,21 +240,21 @@ public class DesktopController {
 		List<String> directories = new ArrayList<String>();
 		List<String> files = new ArrayList<String>();
 		String tmp;
-		for(IDirectoryNodeButton node : selectedNodes) {
+		for (IDirectoryNodeButton node : selectedNodes) {
-			tmp = Helper.combinePath(getCurrentWorkingDirectory(), node.getNode().getFilename());
+			tmp = Helper.combinePath(getCurrentWorkingDirectory(),
-			if(node.getNode().getAttrs().isDir()) {
+					node.getNode().getFilename());
+			if (node.getNode().getAttrs().isDir()) {
 				directories.add(tmp);
-			}
+			} else {
-			else {
 				files.add(tmp);
 			}
 		}
 
-		for(String dir : directories) {
+		for (String dir : directories) {
 			SshEngine.downloadDirectoryRecursively(dir, destinationPath);
 		}
 
-		for(String file : files) {
+		for (String file : files) {
 			SshEngine.downloadFile(file, destinationPath);
 		}
 	}
@@ -259,28 +263,31 @@ public class DesktopController {
 	 * ========== END Selected Nodes section ==========
 	 */
 
-	
 	/*
 	 * ========== BEGIN CutCopyPasteController controller ==========
 	 */
 
-	public class CutCopyPasteController{
+	public class CutCopyPasteController {
 		private List<String> sources = new ArrayList<String>();
 		private int selectedOperation = Constants.Constants_FSOperations.NONE;
 
-		public void startCopying(List<IDirectoryNodeButton> selectedNodes, String currentPath) {
+		public void startCopying(List<IDirectoryNodeButton> selectedNodes,
+				String currentPath) {
 			String fullPath = null;
-			for(IDirectoryNodeButton nodeBtn : selectedNodes) {
+			for (IDirectoryNodeButton nodeBtn : selectedNodes) {
-				fullPath = Helper.combinePath(currentPath, nodeBtn.getNode().getFilename());
+				fullPath = Helper.combinePath(currentPath,
+						nodeBtn.getNode().getFilename());
 				sources.add(fullPath);
 			}
 			selectedOperation = Constants.Constants_FSOperations.COPY;
 		}
 
-		public void startCuttying(List<IDirectoryNodeButton> selectedNodes, String currentPath) {
+		public void startCuttying(List<IDirectoryNodeButton> selectedNodes,
+				String currentPath) {
 			String fullPath = null;
-			for(IDirectoryNodeButton nodeBtn : selectedNodes) {
+			for (IDirectoryNodeButton nodeBtn : selectedNodes) {
-				fullPath = Helper.combinePath(currentPath, nodeBtn.getNode().getFilename());
+				fullPath = Helper.combinePath(currentPath,
+						nodeBtn.getNode().getFilename());
 				sources.add(fullPath);
 			}
 			selectedOperation = Constants.Constants_FSOperations.CUT;
@@ -290,7 +297,7 @@ public class DesktopController {
 			StringBuilder command = null;
 
 			// no source
-			if(sources.size() == 0) {
+			if (sources.size() == 0) {
 				return;
 			}
 
@@ -298,18 +305,18 @@ public class DesktopController {
 			// we keep using isWriteable as
 			// the executeCommand() does not fire
 			// an exception in case of fail
-			if(!isWriteable(destination)) {
+			if (!isWriteable(destination)) {
 				return;
 			}
 
 			// copy
-			if(selectedOperation == Constants.Constants_FSOperations.COPY) {
+			if (selectedOperation == Constants.Constants_FSOperations.COPY) {
 				command = new StringBuilder("cp -r");
 
 			}
 
 			// cut
-			else if(selectedOperation == Constants.Constants_FSOperations.CUT) {
+			else if (selectedOperation == Constants.Constants_FSOperations.CUT) {
 				command = new StringBuilder("mv");
 			}
 
@@ -319,7 +326,7 @@ public class DesktopController {
 			}
 
 			// execute
-			for(String path : sources) {
+			for (String path : sources) {
 				command.append(' ');
 				command.append('"');
 				command.append(path.replace("\"", "\\\""));
@@ -342,15 +349,15 @@ public class DesktopController {
 	 * ========== END CutCopyPasteController controller ==========
 	 */
 
-	
 	/*
 	 * ========== BEGIN File System Operations ==========
 	 */
 
-	
 	/**
 	 * Creates a new folder
-	 * @param newFolderPath Folder's path
+	 * 
+	 * @param newFolderPath
+	 *            Folder's path
 	 * @throws SftpException
 	 */
 	public void mkdir(String newFolderPath) throws SftpException {
@@ -359,7 +366,9 @@ public class DesktopController {
 
 	/**
 	 * Creates a file in the remote file path
-	 * @param remoteFilePath remote file path
+	 * 
+	 * @param remoteFilePath
+	 *            remote file path
 	 * @throws SftpException
 	 */
 	public void touch(String remoteFilePath) throws SftpException {
@@ -368,11 +377,15 @@ public class DesktopController {
 
 	/**
 	 * Renames a file
-	 * @param oldNamePath Path of the old name
+	 * 
-	 * @param newNamePath Path of the new name
+	 * @param oldNamePath
+	 *            Path of the old name
+	 * @param newNamePath
+	 *            Path of the new name
 	 * @throws SftpException
 	 */
-	public void rename(String oldNamePath, String newNamePath) throws SftpException {
+	public void rename(String oldNamePath, String newNamePath)
+			throws SftpException {
 		SshEngine.rename(oldNamePath, newNamePath);
 	}
 
@@ -386,43 +399,46 @@ public class DesktopController {
 
 	/**
 	 * Given a remote file path, opens a graphical notepad for it
-	 * @param filePath remote file path to display in the notepad
+	 * 
+	 * @param filePath
+	 *            remote file path to display in the notepad
 	 */
 	public void openNotepadForFile(String filePath) {
 		new NotepadController(filePath).show();
 	}
 
 	/**
-	 * Disposes resources which need to be freed before exiting
+	 * Disposes resources which need to be freed before exiting the application
-	 * the application
 	 */
 	public void disposeResources() {
 		SshEngine.disconnectSession();
 	}
 
 	/**
-	 * @deprecated This method is deprecated. 
+	 * @deprecated This method is deprecated. Catch SftpException and look for
-	 * Catch SftpException
+	 *             "Permission denied" instead. This prevents unnecessary
-	 * and look for "Permission denied" instead. This prevents
+	 *             overhead
-	 * unnecessary overhead
 	 */
 	public boolean isReadable(String path) {
 		StringBuilder command = new StringBuilder();
 		command.append("[ -r \"");
-		command.append(path.equals("~") ? SshEngine.executeCommand("pwd").replace("\"", "\\\"") : path.replace("\"", "\\\""));
+		command.append(path.equals("~")
+				? SshEngine.executeCommand("pwd").replace("\"", "\\\"")
+				: path.replace("\"", "\\\""));
 		command.append("\" ] && echo 1 || echo 0"); // short circuiting
 		return SshEngine.executeCommand(command.toString()).trim().equals("1");
 	}
 
 	/**
-	 * @deprecated This method is deprecated.
+	 * @deprecated This method is deprecated. Catch SftpException and look for
-	 * Catch SftpException
+	 *             "Permission denied" instead
-	 * and look for "Permission denied" instead
 	 */
 	public boolean isWriteable(String path) {
 		StringBuilder command = new StringBuilder();
 		command.append("[ -w \"");
-		command.append(path.equals("~") ? SshEngine.executeCommand("pwd").replace("\"", "\\\"") : path.replace("\"", "\\\""));
+		command.append(path.equals("~")
+				? SshEngine.executeCommand("pwd").replace("\"", "\\\"")
+				: path.replace("\"", "\\\""));
 		command.append("\" ] && echo 1 || echo 0"); // short circuiting
 		return SshEngine.executeCommand(command.toString()).trim().equals("1");
 	}

Guify/src/controllers/FindAndReplaceController.java

@@ -13,7 +13,8 @@ public class FindAndReplaceController {
 	public FindAndReplaceController(IGenericTextArea textArea) {
 		this.textArea = textArea;
 		try {
-			frame = (IFindAndReplaceFrame) JFrameFactory.createJFrame(IFrameFactory.FIND_AND_REPLACE, this);
+			frame = (IFindAndReplaceFrame) JFrameFactory
+					.createJFrame(IFrameFactory.FIND_AND_REPLACE, this);
 		} catch (Exception e) {
 			e.printStackTrace();
 		}
@@ -23,63 +24,66 @@ public class FindAndReplaceController {
 	 * Show frame centered to parent
 	 */
 	public void showAtTheCenterOfFrame(INotepadFrame notepadFrame) {
-        int childX = notepadFrame.getX() + (notepadFrame.getWidth() - frame.getWidth()) / 2;
+		int childX = notepadFrame.getX()
-        int childY = notepadFrame.getY() + (notepadFrame.getHeight() - frame.getHeight()) / 2;
+				+ (notepadFrame.getWidth() - frame.getWidth()) / 2;
-        frame.setLocation(childX, childY);
+		int childY = notepadFrame.getY()
+				+ (notepadFrame.getHeight() - frame.getHeight()) / 2;
+		frame.setLocation(childX, childY);
 		frame.setVisible(true);
 	}
 
 	public int findNext(String searchText) {
-	    String text = textArea.getText();
+		String text = textArea.getText();
-	    int currentIndex = textArea.getCaretPosition();
+		int currentIndex = textArea.getCaretPosition();
-	    int nextIndex = text.indexOf(searchText, currentIndex);
+		int nextIndex = text.indexOf(searchText, currentIndex);
 
-	    if (nextIndex != -1) {
+		if (nextIndex != -1) {
-	    	textArea.selectText(nextIndex, nextIndex + searchText.length());
+			textArea.selectText(nextIndex, nextIndex + searchText.length());
-	    	return nextIndex;
+			return nextIndex;
-	    }
+		}
-	    return -1;
+		return -1;
 	}
 
 	public int findPrevious(String searchText) {
-	    String text = textArea.getText();
+		String text = textArea.getText();
-	    int cutAt;
+		int cutAt;
 
-	    if(textArea.hasHighlightedText()) {
+		if (textArea.hasHighlightedText()) {
-	    	cutAt = textArea.getSelectionStart();
+			cutAt = textArea.getSelectionStart();
-	    }
+		} else {
-	    else {
+			cutAt = textArea.getCaretPosition();
-	    	cutAt = textArea.getCaretPosition();
+		}
-	    }
+		String firstPart = text.substring(0, cutAt);
-	    String firstPart = text.substring(0, cutAt);
+		int previousIndex = firstPart.lastIndexOf(searchText,
-	    int previousIndex = firstPart.lastIndexOf(searchText, firstPart.length() - 1);
+				firstPart.length() - 1);
-	    if (previousIndex != -1) {
+		if (previousIndex != -1) {
-	        textArea.selectText(previousIndex, previousIndex + searchText.length());
+			textArea.selectText(previousIndex,
-	        return previousIndex;
+					previousIndex + searchText.length());
-	    }
+			return previousIndex;
-	    else {
+		} else {
-	    	return -1;
+			return -1;
-	    }
+		}
 	}
 
 	public int replaceNext(String toReplace, String replaceWith) {
 		int index = findNext(toReplace);
 
-		if(index != -1) {
+		if (index != -1) {
-			textArea.replaceRange(replaceWith, index, index + toReplace.length());
+			textArea.replaceRange(replaceWith, index,
+					index + toReplace.length());
 		}
 
 		return index;
 	}
 
 	public void replaceAll(String searchText, String replacement) {
-	    String text = textArea.getText();
+		String text = textArea.getText();
-	    text = text.replaceAll(searchText, replacement);
+		text = text.replaceAll(searchText, replacement);
-	    textArea.setText(text);
+		textArea.setText(text);
 	}
 
 	public void disposeMyFrame() {
-		if(frame != null) {
+		if (frame != null) {
 			frame.dispose();
 		}
 	}

Guify/src/controllers/LoginController.java

@@ -15,14 +15,15 @@ public class LoginController {
 
 	public LoginController() {
 		try {
-			frame = (ILoginFrame) JFrameFactory.createJFrame(IFrameFactory.LOGIN, this);
+			frame = (ILoginFrame) JFrameFactory
-		} 
+					.createJFrame(IFrameFactory.LOGIN, this);
-		catch (Exception e) {
+		} catch (Exception e) {
 			e.printStackTrace();
 		}
 	}
 
-	public boolean Login(String host, String username, String password, String port) throws IllegalArgumentException {
+	public boolean Login(String host, String username, String password,
+			String port) throws IllegalArgumentException {
 		LoginCredentials.host = host;
 		LoginCredentials.username = username;
 		LoginCredentials.password = password;
@@ -32,13 +33,13 @@ public class LoginController {
 			frame.setVisible(false);
 			new DesktopController().showFrame(true);;
 			return true;
-		}
+		} else {
-		else {
 			return false;
 		}
 	}
 
-	public void ValidateInput(String host, String username, String password, String port) throws IllegalArgumentException {
+	public void ValidateInput(String host, String username, String password,
+			String port) throws IllegalArgumentException {
 
 		// Host Validation. Consider its necessity.
 		try {
@@ -50,8 +51,7 @@ public class LoginController {
 		// Port Validation
 		try {
 			Integer.parseInt(port);
-		}
+		} catch (NumberFormatException ex) {
-		catch(NumberFormatException ex) {
 			throw new IllegalArgumentException("Invalid port number", ex);
 		}
 	}
@@ -64,7 +64,7 @@ public class LoginController {
 		return Constants.APP_NAME + " " + Constants.VERSION;
 	}
 
-	public static class LoginCredentials{
+	public static class LoginCredentials {
 		public static String host;
 		public static String username;
 		public static String password;

Guify/src/controllers/NotepadController.java

@@ -33,7 +33,8 @@ public class NotepadController {
 		this.filePath = filePath;
 		initialText = SshEngine.readFile(filePath);
 		try {
-			notepadFrame = (INotepadFrame) JFrameFactory.createJFrame(IFrameFactory.NOTEPAD, this);
+			notepadFrame = (INotepadFrame) JFrameFactory
+					.createJFrame(IFrameFactory.NOTEPAD, this);
 		} catch (Exception e) {
 			e.printStackTrace();
 			return;
@@ -54,7 +55,7 @@ public class NotepadController {
 	}
 
 	public void disposeFindAndReplaceFrame() {
-		if(myFindAndReplaceController != null)
+		if (myFindAndReplaceController != null)
 			myFindAndReplaceController.disposeMyFrame();
 	}
 

Guify/src/controllers/QueueController.java

@@ -11,7 +11,7 @@ import views.interfaces.IQueueFrame;
 import java.util.HashMap;
 
 @SuppressWarnings("deprecation") // Observer - Observable are okay here
-public class QueueController implements Observer{
+public class QueueController implements Observer {
 
 	/*
 	 * ========== BEGIN Attributes ==========
@@ -30,47 +30,61 @@ public class QueueController implements Observer{
 	// Executed by the EDT
 	public QueueController() {
 		try {
-			frame = (IQueueFrame) JFrameFactory.createJFrame(IFrameFactory.QUEUE, this);
+			frame = (IQueueFrame) JFrameFactory
+					.createJFrame(IFrameFactory.QUEUE, this);
+		} catch (Exception e) {
 		}
-		catch (Exception e) {}
 
 		// Register observer of the changes
 		QueueEventManager.getInstance().addObserver(this);
 
-		// Get previous enqueued elements. Do not place before addObserver(this) or some
+		// Get previous enqueued elements. Do not place before addObserver(this)
+		// or some
 		// transfers could go lost.
 		//
-		// A necessary but not sufficient condition in order to claim that the Queue
+		// A necessary but not sufficient condition in order to claim that the
-		// works well, is that when it gets opened at time x, all the non-finished
+		// Queue
+		// works well, is that when it gets opened at time x, all the
+		// non-finished
 		// transfers initiated in a time t such that t < x must be shown
 		// in the table.
-		// More specifically, x represents the time of completion of the instruction
+		// More specifically, x represents the time of completion of the
+		// instruction
 		// QueueEventManager.getInstance().addObserver(this); as
-		// any subsequent transfer initiated after this point will be guaranteed to
+		// any subsequent transfer initiated after this point will be guaranteed
+		// to
 		// be shown in the table (handled by the "update()" method).
 		//
-		// Having understood this, we may now suppose t1 to be the time of completion of the
+		// Having understood this, we may now suppose t1 to be the time of
+		// completion of the
 		// instruction QueueEventManager.getInstance().getQueue() and t2 to be
 		// the time of completion of the instruction
 		// QueueEventManager.getInstance().addObserver(this) where t1 < t2.
 		// This would've meant that any transfer initiated in a time t such that
-		// t1 < t < t2 would not have been adequately processed as "getQueue()" was
+		// t1 < t < t2 would not have been adequately processed as "getQueue()"
-		// already executed, and "addObserver(this)" would not have been performed yet,
+		// was
-		// making the transfer not visible in the queue when the Queue frame would've
+		// already executed, and "addObserver(this)" would not have been
+		// performed yet,
+		// making the transfer not visible in the queue when the Queue frame
+		// would've
 		// been opened.
 		//
-		// One could argue that when any chunk of data is transferred at any time t
+		// One could argue that when any chunk of data is transferred at any
-		// where t > t2, the update() method will be called, showing the transfer in
+		// time t
-		// the queue. It's not guaranteed that this happens (as it may encounter an
+		// where t > t2, the update() method will be called, showing the
-		// error before t2 and after t1 or simply in may complete in this time frame)
+		// transfer in
+		// the queue. It's not guaranteed that this happens (as it may encounter
+		// an
+		// error before t2 and after t1 or simply in may complete in this time
+		// frame)
 
 		TransferProgress[] queued = QueueEventManager.getInstance().getQueue();
-		for(TransferProgress transferProgress : queued) {
+		for (TransferProgress transferProgress : queued) {
 			SwingUtilities.invokeLater(new Runnable() {
-			      @Override
+				@Override
-			      public void run() {
+				public void run() {
-			    	  frame.manageTransferProgress(transferProgress);
+					frame.manageTransferProgress(transferProgress);
-			      }
+				}
 			});
 		}
 	}
@@ -84,7 +98,8 @@ public class QueueController implements Observer{
 	// denoted as (update(o, arg)_1 , update(o, arg)_2) having the same
 	// object "arg", it holds that
 	// ((TransferProgress)arg).getTransferredBytes() evaluated during
-	// update(o, arg)_1 is less or equal than ((TransferProgress)arg).getTransferredBytes() 
+	// update(o, arg)_1 is less or equal than
+	// ((TransferProgress)arg).getTransferredBytes()
 	// evaluated during update(o, arg)_2.
 	// In simple words, if update() is called at time t1 and at time t2,
 	// where arg is the same TransferProgress object, then the second time
@@ -93,7 +108,8 @@ public class QueueController implements Observer{
 	// than the value of
 	// transferProgress.getTransferredBytes() evaluated the first time.
 	// This happens as the transfer of a single object is an operation executed
-	// on a single thread, hence any update will have a getTransferredBytes() greater
+	// on a single thread, hence any update will have a getTransferredBytes()
+	// greater
 	// or equal than the previous one.
 	//
 	// Keep also in mind that SwingUtilities.invokeLater() called at time t1
@@ -107,51 +123,61 @@ public class QueueController implements Observer{
 	// will go down (e.g. from 50% to 49%), but this would be true even if
 	// the concept described above would be false. This happens because
 	// the TransferProgress
-	// handled by manageTransferProgress will always have the latest updated values
+	// handled by manageTransferProgress will always have the latest updated
+	// values
 	// for that specific object, as it is shared with the thread
 	// which continuously updates it.
 	// We do not need any locks as no race conditions
 	// can happen because in the EDT thread we only read said object, nor any
 	// inconsistencies can arise.
-	// For more information you can view the comments at Queue@manageTransferProgress
+	// For more information you can view the comments at
+	// Queue@manageTransferProgress
 	@Override
 	public void update(Observable o, Object arg) {
-		TransferProgress transferProgress = (TransferProgress)arg;
+		TransferProgress transferProgress = (TransferProgress) arg;
 
 		// Do not invoke the Event Dispatch Thread too frequently as
 		// it may impact the performance of low-end computer systems.
-		if(System.currentTimeMillis() - lastGuiExecutionTime >= THROTTLE_TIME_MS || 
+		if (System.currentTimeMillis()
-				transferProgress.getTransferStatus() == TransferProgress.END) {
+				- lastGuiExecutionTime >= THROTTLE_TIME_MS
+				|| transferProgress
+						.getTransferStatus() == TransferProgress.END) {
 			SwingUtilities.invokeLater(new Runnable() {
-			      @Override
+				@Override
-			      public void run() {
+				public void run() {
-			    	  frame.manageTransferProgress(transferProgress);
+					frame.manageTransferProgress(transferProgress);
-			    	  lastGuiExecutionTime = System.currentTimeMillis();
+					lastGuiExecutionTime = System.currentTimeMillis();
-			      }
+				}
 			});
 		}
 	}
 
 	/**
 	 * Computes the transfer completion percentage
-	 * @param transferProgress A transferProgress object
+	 * 
-	 * @return A value in the range [0, 100] indicating
+	 * @param transferProgress
-	 * the transfer completion percentage
+	 *            A transferProgress object
+	 * @return A value in the range [0, 100] indicating the transfer completion
+	 *         percentage
 	 */
 	public int computePercentage(TransferProgress transferProgress) {
 		// Avoid division by zero
-		if(transferProgress.getTotalBytes() == 0) {
+		if (transferProgress.getTotalBytes() == 0) {
-			// The percentage is 100% if ∀ byte in the file, byte was transferred.
+			// The percentage is 100% if ∀ byte in the file, byte was
-			// If there are no bytes in the file, this logic proposition holds true (vacuous truth)
+			// transferred.
+			// If there are no bytes in the file, this logic proposition holds
+			// true (vacuous truth)
 			return 100;
-		}
+		} else {
-		else {
+			return (int) Math
-			return (int) Math.round( ((transferProgress.getTransferredBytes() * 100F) / transferProgress.getTotalBytes()) );
+					.round(((transferProgress.getTransferredBytes() * 100F)
+							/ transferProgress.getTotalBytes()));
 		}
 	}
 
 	/**
 	 * Given a TransferProgress object, retrieve its index
+	 * 
 	 * @param transferProgress
 	 * @return Its index or null if not present
 	 */
@@ -161,20 +187,26 @@ public class QueueController implements Observer{
 
 	/**
 	 * Put index in the HashMap
-	 * @param transferProgress TransferProgress object
+	 * 
-	 * @param index An integer value representing the index
+	 * @param transferProgress
+	 *            TransferProgress object
+	 * @param index
+	 *            An integer value representing the index
 	 */
-	public void putTableIndex(TransferProgress transferProgress, Integer index) {
+	public void putTableIndex(TransferProgress transferProgress,
+			Integer index) {
 		indexHashMap.put(transferProgress, index);
 	}
 
 	/**
-	 * Checks if a specific TransferProgress is contained in
+	 * Checks if a specific TransferProgress is contained in the HashMap
-	 * the HashMap
+	 * 
-	 * @param transferProgress A TransferProgress object
+	 * @param transferProgress
+	 *            A TransferProgress object
 	 * @return true if present, false otherwise
 	 */
-	public boolean isTransferProgressInHashMap(TransferProgress transferProgress) {
+	public boolean isTransferProgressInHashMap(
+			TransferProgress transferProgress) {
 		return getTableIndex(transferProgress) != null;
 	}
 

Guify/src/views/FindAndReplace.java

@@ -12,94 +12,98 @@ import views.interfaces.IFindAndReplaceFrame;
 
 public class FindAndReplace extends JFrame implements IFindAndReplaceFrame {
 
-    private static final long serialVersionUID = 1L;
+	private static final long serialVersionUID = 1L;
 
-    public FindAndReplace(Object controller) {
+	public FindAndReplace(Object controller) {
-			JLabel lab1 = new JLabel("Find:");
+		JLabel lab1 = new JLabel("Find:");
-	        JLabel lab2 = new JLabel("Replace:");
+		JLabel lab2 = new JLabel("Replace:");
-	        JTextField findTextField = new JTextField(30);
+		JTextField findTextField = new JTextField(30);
-	        JTextField replaceTextField = new JTextField(30);
+		JTextField replaceTextField = new JTextField(30);
-	        JButton findNextBtn = new JButton("Find Next");
+		JButton findNextBtn = new JButton("Find Next");
-	        findNextBtn.addMouseListener(new MouseAdapter() {
+		findNextBtn.addMouseListener(new MouseAdapter() {
-			    @Override
+			@Override
-			    public void mouseClicked(MouseEvent e){
+			public void mouseClicked(MouseEvent e) {
-			    	((FindAndReplaceController) controller).findNext(findTextField.getText());
+				((FindAndReplaceController) controller)
-			    }
+						.findNext(findTextField.getText());
-			});
+			}
+		});
 
-	        JButton findPreviousBtn = new JButton("Find Previous");
+		JButton findPreviousBtn = new JButton("Find Previous");
-	        findPreviousBtn.addMouseListener(new MouseAdapter() {
+		findPreviousBtn.addMouseListener(new MouseAdapter() {
-			    @Override
+			@Override
-			    public void mouseClicked(MouseEvent e){
+			public void mouseClicked(MouseEvent e) {
-			    	((FindAndReplaceController) controller).findPrevious(findTextField.getText());
+				((FindAndReplaceController) controller)
-			    }
+						.findPrevious(findTextField.getText());
-			});
+			}
+		});
 
-	        JButton replaceNextBtn = new JButton("Replace Next");
+		JButton replaceNextBtn = new JButton("Replace Next");
-	        replaceNextBtn.addMouseListener(new MouseAdapter() {
+		replaceNextBtn.addMouseListener(new MouseAdapter() {
-			    @Override
+			@Override
-			    public void mouseClicked(MouseEvent e){
+			public void mouseClicked(MouseEvent e) {
-			    	((FindAndReplaceController) controller).replaceNext(findTextField.getText(), replaceTextField.getText());
+				((FindAndReplaceController) controller).replaceNext(
-			    }
+						findTextField.getText(), replaceTextField.getText());
-			});
+			}
+		});
 
-	        JButton replaceAllBtn = new JButton("Replace All");
+		JButton replaceAllBtn = new JButton("Replace All");
-	        replaceAllBtn.addMouseListener(new MouseAdapter() {
+		replaceAllBtn.addMouseListener(new MouseAdapter() {
-			    @Override
+			@Override
-			    public void mouseClicked(MouseEvent e){
+			public void mouseClicked(MouseEvent e) {
-			    	((FindAndReplaceController) controller).replaceAll(findTextField.getText(), replaceTextField.getText());
+				((FindAndReplaceController) controller).replaceAll(
-			    }
+						findTextField.getText(), replaceTextField.getText());
-			});
+			}
+		});
 
-	        JButton cancel = new JButton("Cancel");
+		JButton cancel = new JButton("Cancel");
-	        cancel.addMouseListener(new MouseAdapter() {
+		cancel.addMouseListener(new MouseAdapter() {
-			    @Override
+			@Override
-			    public void mouseClicked(MouseEvent e){
+			public void mouseClicked(MouseEvent e) {
-			    	dispose();
+				dispose();
-			    }
+			}
-			});
+		});
 
-	        setLayout(null);
+		setLayout(null);
 
-	        // Set the width and height of the label
+		// Set the width and height of the label
-	        int labWidth = 80;
+		int labWidth = 80;
-	        int labHeight = 20;
+		int labHeight = 20;
 
-	        // Adding labels
+		// Adding labels
-	        lab1.setBounds(10,10, labWidth, labHeight);
+		lab1.setBounds(10, 10, labWidth, labHeight);
-	        add(lab1);
+		add(lab1);
-	        findTextField.setBounds(10+labWidth, 10, 120, 20);
+		findTextField.setBounds(10 + labWidth, 10, 120, 20);
-	        add(findTextField);
+		add(findTextField);
-	        lab2.setBounds(10, 10+labHeight+10, labWidth, labHeight);
+		lab2.setBounds(10, 10 + labHeight + 10, labWidth, labHeight);
-	        add(lab2);
+		add(lab2);
-	        replaceTextField.setBounds(10+labWidth, 10+labHeight+10, 120, 20);
+		replaceTextField.setBounds(10 + labWidth, 10 + labHeight + 10, 120, 20);
-	        add(replaceTextField);
+		add(replaceTextField);
 
-	        // Adding buttons
+		// Adding buttons
-	        findNextBtn.setBounds(225, 6, 115, 20);
+		findNextBtn.setBounds(225, 6, 115, 20);
-	        add(findNextBtn);
+		add(findNextBtn);
 
-	        findPreviousBtn.setBounds(225, 28, 115, 20);
+		findPreviousBtn.setBounds(225, 28, 115, 20);
-	        add(findPreviousBtn);
+		add(findPreviousBtn);
 
-	        replaceNextBtn.setBounds(225, 50, 115, 20);
+		replaceNextBtn.setBounds(225, 50, 115, 20);
-	        add(replaceNextBtn);
+		add(replaceNextBtn);
 
-	        replaceAllBtn.setBounds(225, 72, 115, 20);
+		replaceAllBtn.setBounds(225, 72, 115, 20);
-	        add(replaceAllBtn);
+		add(replaceAllBtn);
 
-	        cancel.setBounds(225, 94, 115, 20);
+		cancel.setBounds(225, 94, 115, 20);
-	        add(cancel);
+		add(cancel);
 
-	        // Set the width and height of the window
+		// Set the width and height of the window
-	        int width = 360;
+		int width = 360;
-	        int height = 160;
+		int height = 160;
 
-	        // Set size window
+		// Set size window
-	        setSize(width, height);
+		setSize(width, height);
-	        setResizable(false);
+		setResizable(false);
-	        setTitle("Find/Replace");
+		setTitle("Find/Replace");
-	        setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
+		setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
-	        setAlwaysOnTop(true);
+		setAlwaysOnTop(true);
 	}
 }

Guify/src/views/Login.java

@@ -91,7 +91,6 @@ public class Login extends JFrame implements ILoginFrame {
 		contentPane.add(portField);
 		portField.setColumns(10);
 
-		
 	}
 
 	/**
@@ -107,15 +106,16 @@ public class Login extends JFrame implements ILoginFrame {
 		// Perform validation
 		try {
 			controller.ValidateInput(host, username, password, port);
-		}
+		} catch (IllegalArgumentException ex) {
-		catch(IllegalArgumentException ex) {
+			JOptionPane.showMessageDialog(new JFrame(), ex.getMessage(),
-			JOptionPane.showMessageDialog(new JFrame(), ex.getMessage(), "Attention required", JOptionPane.ERROR_MESSAGE);
+					"Attention required", JOptionPane.ERROR_MESSAGE);
 			return;
 		}
 
 		// Perform login
-		if(!controller.Login(host, username, password, port)) {
+		if (!controller.Login(host, username, password, port)) {
-			JOptionPane.showMessageDialog(new JFrame(), "SSH Login failed", "SSH Login failed", JOptionPane.ERROR_MESSAGE);
+			JOptionPane.showMessageDialog(new JFrame(), "SSH Login failed",
+					"SSH Login failed", JOptionPane.ERROR_MESSAGE);
 		}
 	}
 

Guify/src/views/Notepad.java

@@ -41,73 +41,83 @@ public class Notepad extends JFrame implements INotepadFrame {
 		JPanel contentPanel = new JPanel(new BorderLayout());
 		JScrollPane scrollPane = new JScrollPane(textArea);
 		contentPanel.add(scrollPane, BorderLayout.CENTER);
-		contentPanel.setBorder(BorderFactory.createEmptyBorder(0, 10, 10, 10)); // We want to create a spaced JPanel
+		contentPanel.setBorder(BorderFactory.createEmptyBorder(0, 10, 10, 10)); // We
+																				// want
+																				// to
+																				// create
+																				// a
+																				// spaced
+																				// JPanel
 		getContentPane().add(contentPanel, BorderLayout.CENTER);
 		textArea = new JTextArea();
 		scrollPane.setViewportView(textArea);
 		textArea.setTabSize(4);
 		textArea.setFont(new Font("Monospaced", Font.PLAIN, 14));
 		textArea.setCaretPosition(0);
-		textArea.setText(((NotepadController)controller).getInitialText());
+		textArea.setText(((NotepadController) controller).getInitialText());
 		textArea.getDocument().addDocumentListener(new DocumentListener() {
-            @Override
+			@Override
-            public void insertUpdate(DocumentEvent e) {
+			public void insertUpdate(DocumentEvent e) {
-                // Called when text is inserted into the document
+				// Called when text is inserted into the document
-            	handleTextChange();
+				handleTextChange();
-            }
+			}
 
-            @Override
+			@Override
-            public void removeUpdate(DocumentEvent e) {
+			public void removeUpdate(DocumentEvent e) {
-                // Called when text is removed from the document
+				// Called when text is removed from the document
-            	handleTextChange();
+				handleTextChange();
-            }
+			}
 
-            @Override
+			@Override
-            public void changedUpdate(DocumentEvent e) {
+			public void changedUpdate(DocumentEvent e) {
-                // Called when attributes of the document change
+				// Called when attributes of the document change
-            	// NO OP
+				// NO OP
-            }
+			}
 
-            private void handleTextChange() {
+			private void handleTextChange() {
-            	if(!((NotepadController)controller).isUnsaved()) {
+				if (!((NotepadController) controller).isUnsaved()) {
-            		((NotepadController)controller).setUnsaved(true);
+					((NotepadController) controller).setUnsaved(true);
-            		setTitle(((NotepadController)controller).getTitle());
+					setTitle(((NotepadController) controller).getTitle());
-            	}
+				}
-            }
+			}
-        });
+		});
 
 		JButton saveBtn = new JButton();
 		saveBtn.setBorderPainted(false);
 		saveBtn.setBorder(new EmptyBorder(0, 0, 0, 0)); // Set empty border
 		saveBtn.setToolTipText("Save");
 		try {
-			saveBtn.setIcon(new ImageIcon(ImageIO.read(getClass().getClassLoader().getResource("save_icon.png")).getScaledInstance(25, 25, Image.SCALE_SMOOTH)));
+			saveBtn.setIcon(new ImageIcon(ImageIO
+					.read(getClass().getClassLoader()
+							.getResource("save_icon.png"))
+					.getScaledInstance(25, 25, Image.SCALE_SMOOTH)));
 			saveBtn.setBackground(new Color(240, 240, 240));
 			saveBtn.setBorderPainted(false);
 			saveBtn.setEnabled(true);
-		}catch(IOException ex) {
+		} catch (IOException ex) {
 			saveBtn.setText("Save");
 		}
 		saveBtn.addMouseListener(new MouseAdapter() {
 			@Override
 			public void mouseClicked(MouseEvent e) {
-				((NotepadController)controller).writeOnFile(textArea.getText());
+				((NotepadController) controller)
-				((NotepadController)controller).setUnsaved(false);
+						.writeOnFile(textArea.getText());
-				setTitle(((NotepadController)controller).getTitle());
+				((NotepadController) controller).setUnsaved(false);
+				setTitle(((NotepadController) controller).getTitle());
 			}
 
 			// Hover on
 			@Override
-            public void mouseEntered(MouseEvent e) {
+			public void mouseEntered(MouseEvent e) {
-				if(saveBtn.isEnabled())
+				if (saveBtn.isEnabled())
 					saveBtn.setBackground(new Color(220, 220, 220));
-            }
+			}
 
 			// Hover off
 			@Override
-            public void mouseExited(MouseEvent e) {
+			public void mouseExited(MouseEvent e) {
 				saveBtn.setBackground(new Color(240, 240, 240));
-            }
+			}
 		});
 
 		JButton searchBtn = new JButton();
@@ -115,31 +125,35 @@ public class Notepad extends JFrame implements INotepadFrame {
 		searchBtn.setBorder(new EmptyBorder(0, 0, 0, 0)); // Set empty border;
 		searchBtn.setToolTipText("Serch/Replace");
 		try {
-			searchBtn.setIcon(new ImageIcon(ImageIO.read(getClass().getClassLoader().getResource("search_icon.png")).getScaledInstance(25, 25, Image.SCALE_SMOOTH)));
+			searchBtn.setIcon(new ImageIcon(ImageIO
+					.read(getClass().getClassLoader()
+							.getResource("search_icon.png"))
+					.getScaledInstance(25, 25, Image.SCALE_SMOOTH)));
 			searchBtn.setBackground(new Color(240, 240, 240));
 			searchBtn.setBorderPainted(false);
 			searchBtn.setEnabled(true);
-		}catch(IOException ex) {
+		} catch (IOException ex) {
 			searchBtn.setText("Serch/Replace");
 		}
 		searchBtn.addMouseListener(new MouseAdapter() {
 			@Override
 			public void mouseClicked(MouseEvent e) {
-				((NotepadController)controller).showFindAndReplace(new JGenericTextArea(textArea));
+				((NotepadController) controller)
+						.showFindAndReplace(new JGenericTextArea(textArea));
 			}
 
 			// Hover on
 			@Override
-            public void mouseEntered(MouseEvent e) {
+			public void mouseEntered(MouseEvent e) {
-				if(searchBtn.isEnabled())
+				if (searchBtn.isEnabled())
 					searchBtn.setBackground(new Color(220, 220, 220));
-            }
+			}
 
 			// Hover off
 			@Override
-            public void mouseExited(MouseEvent e) {
+			public void mouseExited(MouseEvent e) {
 				searchBtn.setBackground(new Color(240, 240, 240));
-            }
+			}
 		});
 
 		JToolBar toolBar = new JToolBar();
@@ -155,14 +169,13 @@ public class Notepad extends JFrame implements INotepadFrame {
 		getContentPane().add(toolBarPanel, BorderLayout.NORTH);
 
 		/**
-		 * Close "Find and Replace" if this window
+		 * Close "Find and Replace" if this window gets closed
-		 * gets closed
 		 */
 		addWindowListener(new WindowAdapter() {
-	        @Override
+			@Override
-	        public void windowClosing(WindowEvent e) {
+			public void windowClosing(WindowEvent e) {
-	        	((NotepadController)controller).disposeFindAndReplaceFrame();
+				((NotepadController) controller).disposeFindAndReplaceFrame();
-	        }
+			}
-	    });	
+		});
 	}
 }

Guify/src/views/Queue.java

@@ -22,35 +22,38 @@ public class Queue extends JFrame implements IQueueFrame {
 
 	/**
 	 *
-	 * Custom cell renderer in order to be able to display
+	 * Custom cell renderer in order to be able to display a progress bar in the
-	 * a progress bar in the JTable
+	 * JTable
 	 *
 	 */
- 	public static class ProgressBarTableCellRenderer extends DefaultTableCellRenderer {
+	public static class ProgressBarTableCellRenderer
+			extends
+				DefaultTableCellRenderer {
 		private static final long serialVersionUID = 1L;
 		private JProgressBar progressBar;
 
-	    public ProgressBarTableCellRenderer() {
+		public ProgressBarTableCellRenderer() {
-	        super();
+			super();
-	        progressBar = new JProgressBar();
+			progressBar = new JProgressBar();
-	        progressBar.setStringPainted(true);
+			progressBar.setStringPainted(true);
-	    }
+		}
 
-	    @Override
+		@Override
-	    public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected,
+		public Component getTableCellRendererComponent(JTable table,
-	                                                   boolean hasFocus, int row, int column) {
+				Object value, boolean isSelected, boolean hasFocus, int row,
-	        if (value instanceof Integer) {
+				int column) {
-	            int progressValue = (Integer) value;
+			if (value instanceof Integer) {
-	            progressBar.setValue(progressValue);
+				int progressValue = (Integer) value;
-	            progressBar.setString(progressValue + "%");
+				progressBar.setValue(progressValue);
-	        }
+				progressBar.setString(progressValue + "%");
+			}
 
-	        progressBar.setStringPainted(true);
+			progressBar.setStringPainted(true);
-	        progressBar.setForeground(Constants.GuifyColors.BLUE);
+			progressBar.setForeground(Constants.GuifyColors.BLUE);
-            progressBar.setBackground(Color.WHITE);
+			progressBar.setBackground(Color.WHITE);
 
-	        return progressBar;
+			return progressBar;
-	    }
+		}
 	}
 
 	public DefaultTableModel tableModel;
@@ -58,66 +61,83 @@ public class Queue extends JFrame implements IQueueFrame {
 	public Queue(Object controller) {
 		this.controller = (QueueController) controller;
 		setTitle("Queue");
-		String[] columnNames = {"Source", "Destination", "Operation", "Percentage"};
+		String[] columnNames = {"Source", "Destination", "Operation",
-        tableModel = new DefaultTableModel(columnNames, 0);
+				"Percentage"};
-        JTable table = new JTable(tableModel);
+		tableModel = new DefaultTableModel(columnNames, 0);
-        table.setEnabled(false); // Prevents user editing
+		JTable table = new JTable(tableModel);
-        // Show percentage by using a custom cell renderer
+		table.setEnabled(false); // Prevents user editing
-        TableColumn percentageColumn = table.getColumnModel().getColumn(3);
+		// Show percentage by using a custom cell renderer
-        percentageColumn.setCellRenderer(new ProgressBarTableCellRenderer());
+		TableColumn percentageColumn = table.getColumnModel().getColumn(3);
-        JScrollPane scrollPane = new JScrollPane(table);
+		percentageColumn.setCellRenderer(new ProgressBarTableCellRenderer());
-        this.getContentPane().add(scrollPane, BorderLayout.CENTER);
+		JScrollPane scrollPane = new JScrollPane(table);
-        this.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
+		this.getContentPane().add(scrollPane, BorderLayout.CENTER);
-        this.pack();
+		this.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
+		this.pack();
 	}
 
 	/**
 	 * Adds a row in the JTable
+	 * 
 	 * @return The index of the inserted row
 	 */
-	public int addRow(String source, String destination, String operation, int percentage) {
+	public int addRow(String source, String destination, String operation,
-        tableModel.addRow(new Object[]{source, destination, operation, percentage});
+			int percentage) {
-        return tableModel.getRowCount() - 1;
+		tableModel.addRow(
+				new Object[]{source, destination, operation, percentage});
+		return tableModel.getRowCount() - 1;
 	}
 
 	/**
-	 * Given a value (representing the transfer completion percentage)
+	 * Given a value (representing the transfer completion percentage) and an
-	 * and an index representing the 0-based index of the row to update, 
+	 * index representing the 0-based index of the row to update, update that
-	 * update that row with that value
+	 * row with that value
-	 * @param rowIndex 0-base index of the row to update
+	 * 
-	 * @param percentage The transfer completion percentage to set
+	 * @param rowIndex
+	 *            0-base index of the row to update
+	 * @param percentage
+	 *            The transfer completion percentage to set
 	 */
 	public void updateRow(int rowIndex, int percentage) {
-		if(rowIndex < tableModel.getRowCount()) {
+		if (rowIndex < tableModel.getRowCount()) {
 			tableModel.setValueAt(percentage, rowIndex, 3);
 		}
 	}
 
-	// This method can receive a transferProgress whose status can be INIT, UPDATING
+	// This method can receive a transferProgress whose status can be INIT,
+	// UPDATING
 	// or END.
-	// In all of these cases it is possible that transferProgress is or is not in the HashMap,
+	// In all of these cases it is possible that transferProgress is or is not
+	// in the HashMap,
 	// and we'll prove its correctness in all the cases:
 	//
-	// 1. Init: This method can receive a transferProgress whose status is INIT and
+	// 1. Init: This method can receive a transferProgress whose status is INIT
-	// 		a. Not present in the HashMap: this can happen either when QueueController receives
+	// and
-	// 		an update() or when it iterates in the constructor over the previously enqueued
+	// a. Not present in the HashMap: this can happen either when
-	//		elements. In both cases, the transfer gets correctly put in the table.
+	// QueueController receives
-	//		b. Present in the HashMap: despite it's counterintuitive, this can happen when 
+	// an update() or when it iterates in the constructor over the previously
-	// 		a transfer gets initialized after QueueEventManager.getInstance().addObserver(this);
+	// enqueued
-	//		and before QueueEventManager.getInstance().getQueue().
+	// elements. In both cases, the transfer gets correctly put in the table.
-	// 		This would lead either update()
+	// b. Present in the HashMap: despite it's counterintuitive, this can happen
-	//		or the QueueController's constructor to call this method over a transferProgress
+	// when
-	//		already in the HashMap (because inserted by the other one)
+	// a transfer gets initialized after
-	//		but whose status is INIT. In this case, updateRow() will
+	// QueueEventManager.getInstance().addObserver(this);
-	//		be called, but without any side effects as the percentage would be zero regardless.
+	// and before QueueEventManager.getInstance().getQueue().
+	// This would lead either update()
+	// or the QueueController's constructor to call this method over a
+	// transferProgress
+	// already in the HashMap (because inserted by the other one)
+	// but whose status is INIT. In this case, updateRow() will
+	// be called, but without any side effects as the percentage would be zero
+	// regardless.
 	//
 	// 2. Updating:
-	//		a. Not present in the HashMap: This can happen when the Queue UI is opened
+	// a. Not present in the HashMap: This can happen when the Queue UI is
-	//		while an element is already being transferred. This happens because when
+	// opened
-	//		the transfer had a "INIT" status, this object did not exist yet.
+	// while an element is already being transferred. This happens because when
-	//		If it's not present in the HashMap
+	// the transfer had a "INIT" status, this object did not exist yet.
-	//		then it will be added.
+	// If it's not present in the HashMap
-	//		b. Present in the HashMap: then it will be correctly updated.
+	// then it will be added.
+	// b. Present in the HashMap: then it will be correctly updated.
 	//
 	// 3. End: Same case for Updating
 	//
@@ -137,20 +157,24 @@ public class Queue extends JFrame implements IQueueFrame {
 	@Override
 	public void manageTransferProgress(TransferProgress transferProgress) {
 
-		// Remember that when QueueController calls frame.manageTransferProgress(transferProgress),
+		// Remember that when QueueController calls
-		// here transferProgress might have different attributes than it was originally called on
+		// frame.manageTransferProgress(transferProgress),
+		// here transferProgress might have different attributes than it was
+		// originally called on
 		// (as it's updated by a different thread).
 		// We do not need a lock as we do not edit it, but just keep it in mind.
 
-		if(!controller.isTransferProgressInHashMap(transferProgress)) {
+		if (!controller.isTransferProgressInHashMap(transferProgress)) {
-			controller.putTableIndex(transferProgress, 
+			controller.putTableIndex(transferProgress, addRow(
-							addRow(transferProgress.getSource(), 
+					transferProgress.getSource(),
-									transferProgress.getDestination(), 
+					transferProgress.getDestination(),
-									transferProgress.getOperation() == SftpProgressMonitor.GET? "Download" : "Upload", 
+					transferProgress.getOperation() == SftpProgressMonitor.GET
-									controller.computePercentage(transferProgress)));
+							? "Download"
-		}
+							: "Upload",
-		else {
+					controller.computePercentage(transferProgress)));
-			updateRow(controller.getTableIndex(transferProgress), controller.computePercentage(transferProgress));
+		} else {
+			updateRow(controller.getTableIndex(transferProgress),
+					controller.computePercentage(transferProgress));
 		}
 	}
 }