You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by sm...@apache.org on 2015/05/18 17:47:20 UTC
directory-fortress-core git commit: FC-105 - [fortress-core] -
command line interface
Repository: directory-fortress-core
Updated Branches:
refs/heads/master f380e1e1c -> 0d7743766
FC-105 - [fortress-core] - command line interface
Project: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/repo
Commit: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/commit/0d774376
Tree: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/tree/0d774376
Diff: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/diff/0d774376
Branch: refs/heads/master
Commit: 0d774376644a5cb8a57a343f6b5e9d37518427c2
Parents: f380e1e
Author: Shawn McKinney <sm...@apache.org>
Authored: Mon May 18 10:47:05 2015 -0500
Committer: Shawn McKinney <sm...@apache.org>
Committed: Mon May 18 10:47:05 2015 -0500
----------------------------------------------------------------------
.../org/apache/directory/fortress/core/cli/CmdLineParser.java | 6 +++---
.../directory/fortress/core/cli/CommandLineInterpreter.java | 7 +++++--
.../java/org/apache/directory/fortress/core/cli/Options.java | 4 ++--
3 files changed, 10 insertions(+), 7 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0d774376/src/main/java/org/apache/directory/fortress/core/cli/CmdLineParser.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/cli/CmdLineParser.java b/src/main/java/org/apache/directory/fortress/core/cli/CmdLineParser.java
index 9bab2cd..48d39c4 100755
--- a/src/main/java/org/apache/directory/fortress/core/cli/CmdLineParser.java
+++ b/src/main/java/org/apache/directory/fortress/core/cli/CmdLineParser.java
@@ -53,7 +53,7 @@ public class CmdLineParser
/**
* Base class for exceptions that may be thrown when options are parsed
*/
- public static abstract class OptionException extends Exception
+ public abstract static class OptionException extends Exception
{
/** Default serialVersionUID */
private static final long serialVersionUID = 1L;
@@ -297,7 +297,7 @@ public class CmdLineParser
{
try
{
- return new Integer( arg );
+ return Integer.valueOf( arg );
}
catch ( NumberFormatException e )
{
@@ -325,7 +325,7 @@ public class CmdLineParser
{
try
{
- return new Long( arg );
+ return Long.valueOf( arg );
}
catch ( NumberFormatException e )
{
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0d774376/src/main/java/org/apache/directory/fortress/core/cli/CommandLineInterpreter.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/cli/CommandLineInterpreter.java b/src/main/java/org/apache/directory/fortress/core/cli/CommandLineInterpreter.java
index 9d3b3a2..d2b1aed 100755
--- a/src/main/java/org/apache/directory/fortress/core/cli/CommandLineInterpreter.java
+++ b/src/main/java/org/apache/directory/fortress/core/cli/CommandLineInterpreter.java
@@ -237,14 +237,17 @@ public class CommandLineInterpreter
{
processGroupCommand( commands, options );
}
+ /*
+ TODO: add these for delegated admin and review support:
else if ( commands.contains( DELEGATED_REVIEW ) )
{
- //processDelegatedReviewCommand(commands, options);
+ processDelegatedReviewCommand(commands, options);
}
else if ( commands.contains( DELEGATED_SYSTEM ) )
{
- //processDelegatedSystemCommand(commands, options);
+ processDelegatedSystemCommand(commands, options);
}
+ */
else
{
LOG.warn( "unknown admin operation detected" );
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/0d774376/src/main/java/org/apache/directory/fortress/core/cli/Options.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/cli/Options.java b/src/main/java/org/apache/directory/fortress/core/cli/Options.java
index 5a5e801..11127fd 100755
--- a/src/main/java/org/apache/directory/fortress/core/cli/Options.java
+++ b/src/main/java/org/apache/directory/fortress/core/cli/Options.java
@@ -154,7 +154,7 @@ public class Options implements java.io.Serializable
updateRoleAssigns(sdSet);
try
{
- Integer cardinality = new Integer(getCardinality());
+ Integer cardinality = Integer.valueOf( getCardinality() );
sdSet.setCardinality(cardinality);
}
catch (NumberFormatException ne)
@@ -287,7 +287,7 @@ public class Options implements java.io.Serializable
constraint.setName(getName());
try
{
- Integer to = new Integer(getTimeout());
+ Integer to = Integer.valueOf( getTimeout() );
constraint.setTimeout(to);
}
catch (NumberFormatException ne)