Skip to content
This repository has been archived by the owner on Mar 27, 2024. It is now read-only.

Commit

Permalink
Changed command configuration function name
Browse files Browse the repository at this point in the history
Signed-off-by: Joe Osborn <osbornjd@ornl.gov>
  • Loading branch information
Joe Osborn committed Sep 6, 2019
1 parent e09b2ec commit 6f3308d
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 13 deletions.
Expand Up @@ -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);
Expand All @@ -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);
Expand All @@ -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);


/**
Expand Down
Expand Up @@ -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);
Expand All @@ -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);
Expand Down
Expand Up @@ -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

}
}

0 comments on commit 6f3308d

Please sign in to comment.