directory-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From smckin...@apache.org
Subject directory-fortress-core git commit: FC-105 - [fortress-core] - command line interface
Date Mon, 18 May 2015 15:47:20 GMT
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 <smckinney@apache.org>
Authored: Mon May 18 10:47:05 2015 -0500
Committer: Shawn McKinney <smckinney@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)


Mime
View raw message