Spring/spring-shell
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix history issue

Browse Source
This commit is contained in:
Jarred Li
2012-05-31 13:17:44 +08:00
parent cbe61e1d6f
commit befa5f12c5
2 changed files with 13 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
src/main/java/org/springframework/shell/JLineShell.java
View File

@@ -558,9 +558,7 @@ public abstract class JLineShell extends AbstractShell implements CommandMarker,
* *
* @return history file name * @return history file name
*/ */
protected String getHistoryFileName() { abstract protected String getHistoryFileName();
return Constant.HISTORY_FILE_NAME;
}
/** /**
* get prompt text from provider. The provider has highest order * get prompt text from provider. The provider has highest order
@@ -568,14 +566,20 @@ public abstract class JLineShell extends AbstractShell implements CommandMarker,
* *
* @return prompt text * @return prompt text
*/ */
protected String getPromptText() { abstract protected String getPromptText();
return Constant.COMMAND_LINE_PROMPT;
}
protected String getProductName() { /**
return Constant.PRODUCT_NAME; * get product name
} *
* @return Product Name
*/
abstract protected String getProductName();
/**
* get version information
*
* @return Version
*/
protected String getVersion() { protected String getVersion() {
return VersionUtils.versionInfo(); return VersionUtils.versionInfo();
} }

15
src/main/java/org/springframework/shell/JLineShellComponent.java
View File

@@ -43,7 +43,6 @@ public class JLineShellComponent extends JLineShell implements Lifecycle {
private String version; private String version;
private String welcomeMessage; private String welcomeMessage;
private int historySize;
// Fields // Fields
private ExecutionStrategy executionStrategy = new SimpleExecutionStrategy(); //ProcessManagerHostedExecutionStrategy is not what i think we need outside of Roo. private ExecutionStrategy executionStrategy = new SimpleExecutionStrategy(); //ProcessManagerHostedExecutionStrategy is not what i think we need outside of Roo.
@@ -207,20 +206,6 @@ public class JLineShellComponent extends JLineShell implements Lifecycle {
this.printBanner = printBanner; this.printBanner = printBanner;
} }
/**
* @return the historySize
*/
public int getHistorySize() {
return historySize;
}
/**
* @param historySize the historySize to set
*/
public void setHistorySize(int historySize) {
this.historySize = historySize;
}
protected String getProductName() { protected String getProductName() {
return productName; return productName;
} }