From 6f3308d84beb5ca85434b3a66fa2498c43e5a361 Mon Sep 17 00:00:00 2001 From: Joe Osborn Date: Fri, 6 Sep 2019 14:31:02 -0400 Subject: [PATCH] Changed command configuration function name Signed-off-by: Joe Osborn --- .../java/org/eclipse/ice/commands/FileHandler.java | 12 ++++++------ .../org/eclipse/ice/commands/LocalFileHandler.java | 4 ++-- .../eclipse/ice/commands/RemoteFileHandler.java | 14 +++++++++----- 3 files changed, 17 insertions(+), 13 deletions(-) diff --git a/org.eclipse.ice.commands/src/main/java/org/eclipse/ice/commands/FileHandler.java b/org.eclipse.ice.commands/src/main/java/org/eclipse/ice/commands/FileHandler.java index 2c237a608..a2890a6f6 100644 --- a/org.eclipse.ice.commands/src/main/java/org/eclipse/ice/commands/FileHandler.java +++ b/org.eclipse.ice.commands/src/main/java/org/eclipse/ice/commands/FileHandler.java @@ -68,7 +68,7 @@ public CommandStatus move(final String source, final String destination) throws checkExistence(source, destination); // Set the commands to have the appropriate properties - setMoveConfiguration(source, destination); + configureMoveCommand(source, destination); // Execute and process the file transfer transferStatus = executeTransfer(destination); @@ -90,7 +90,7 @@ public CommandStatus copy(final String source, final String destination) throws checkExistence(source, destination); // Set the commands to have the appropriate properties - setCopyConfiguration(source, destination); + configureCopyCommand(source, destination); // Execute and process the file transfer transferStatus = executeTransfer(destination); @@ -101,23 +101,23 @@ public CommandStatus copy(final String source, final String destination) throws /** * This function sets the command member variables to have the source and - * destination strings. It is delegated to the sublcasses so that the commands + * destination strings. It is delegated to the subclasses so that the commands * can be cast appropriately * * @param source * @param destination */ - protected abstract void setMoveConfiguration(final String source, final String destination); + protected abstract void configureMoveCommand(final String source, final String destination); /** * This function sets the command member variables to have the source and - * destination strings. It is delegated to the sublcasses so that the commands + * destination strings. It is delegated to the subclasses so that the commands * can be cast appropriately * * @param source * @param destination */ - protected abstract void setCopyConfiguration(final String source, final String destination); + protected abstract void configureCopyCommand(final String source, final String destination); /** diff --git a/org.eclipse.ice.commands/src/main/java/org/eclipse/ice/commands/LocalFileHandler.java b/org.eclipse.ice.commands/src/main/java/org/eclipse/ice/commands/LocalFileHandler.java index f48a2b478..e896a292a 100644 --- a/org.eclipse.ice.commands/src/main/java/org/eclipse/ice/commands/LocalFileHandler.java +++ b/org.eclipse.ice.commands/src/main/java/org/eclipse/ice/commands/LocalFileHandler.java @@ -35,7 +35,7 @@ public LocalFileHandler() { * See {@link org.eclipse.ice.commands.FileHandler#setConfiguration(String, String)} */ @Override - protected void setMoveConfiguration(final String source, final String destination) { + protected void configureMoveCommand(final String source, final String destination) { command = new LocalMoveFileCommand(); // Cast the Command as a LocalMoveFileCommand to set the source and destination paths ((LocalMoveFileCommand) command).setConfiguration(source, destination); @@ -45,7 +45,7 @@ protected void setMoveConfiguration(final String source, final String destinatio * See {@link org.eclipse.ice.commands.FileHandler#setConfiguration(String, String)} */ @Override - protected void setCopyConfiguration(final String source, final String destination) { + protected void configureCopyCommand(final String source, final String destination) { command = new LocalCopyFileCommand(); // Cast the Command as a LocalCopyFileCommand to set the source and destination paths ((LocalCopyFileCommand) command).setConfiguration(source, destination); diff --git a/org.eclipse.ice.commands/src/main/java/org/eclipse/ice/commands/RemoteFileHandler.java b/org.eclipse.ice.commands/src/main/java/org/eclipse/ice/commands/RemoteFileHandler.java index 3a55d9463..a78c7fdf6 100644 --- a/org.eclipse.ice.commands/src/main/java/org/eclipse/ice/commands/RemoteFileHandler.java +++ b/org.eclipse.ice.commands/src/main/java/org/eclipse/ice/commands/RemoteFileHandler.java @@ -45,19 +45,23 @@ public boolean exists(String file) throws IOException { public void checkExistence(String source, String destination) throws IOException { } + /** * See - * {@link org.eclipse.ice.commands.FileHandler#setMoveConfiguration(String, String)} + * {@link org.eclipse.ice.commands.FileHandler#configureMoveCommand(String, String)} */ @Override - protected void setMoveConfiguration(String source, String destination) { + protected void configureMoveCommand(String source, String destination) { + // TODO Auto-generated method stub + } - /** * See - * {@link org.eclipse.ice.commands.FileHandler#setCopyConfiguration(String, String)} + * {@link org.eclipse.ice.commands.FileHandler#configureCopyCommand(String, String)} */ @Override - protected void setCopyConfiguration(String source, String destination) { + protected void configureCopyCommand(String source, String destination) { + // TODO Auto-generated method stub + } }