Repository: karaf
Updated Branches:
  refs/heads/master d413eef2e -> e6a21e20e


KARAF-4270: Don't NPE for commands without descriptions.


Project: http://git-wip-us.apache.org/repos/asf/karaf/repo
Commit: http://git-wip-us.apache.org/repos/asf/karaf/commit/5a00ab7e
Tree: http://git-wip-us.apache.org/repos/asf/karaf/tree/5a00ab7e
Diff: http://git-wip-us.apache.org/repos/asf/karaf/diff/5a00ab7e

Branch: refs/heads/master
Commit: 5a00ab7ee204c05df06925f38d196840bf7f0302
Parents: 3cb0a00
Author: Benson Margulies <[email protected]>
Authored: Fri Jan 15 09:23:56 2016 -0500
Committer: Benson Margulies <[email protected]>
Committed: Fri Jan 15 09:23:56 2016 -0500

----------------------------------------------------------------------
 .../java/org/apache/karaf/shell/compat/CommandTracker.java    | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/karaf/blob/5a00ab7e/shell/console/src/main/java/org/apache/karaf/shell/compat/CommandTracker.java
----------------------------------------------------------------------
diff --git 
a/shell/console/src/main/java/org/apache/karaf/shell/compat/CommandTracker.java 
b/shell/console/src/main/java/org/apache/karaf/shell/compat/CommandTracker.java
index 68e03ba..0e5c2e6 100644
--- 
a/shell/console/src/main/java/org/apache/karaf/shell/compat/CommandTracker.java
+++ 
b/shell/console/src/main/java/org/apache/karaf/shell/compat/CommandTracker.java
@@ -186,7 +186,12 @@ public class CommandTracker implements 
ServiceTrackerCustomizer<Object, Object>
 
                         @Override
                         public String getDescription() {
-                            return 
reference.getProperty("osgi.command.description").toString();
+                            Object property = 
reference.getProperty("osgi.command.description");
+                            if (property != null) {
+                                return property.toString();
+                            } else {
+                                return null;
+                            }
                         }
 
                         @Override

Reply via email to