[jira] [Commented] (KARAF-4270) Shell-compat gets NPE trying to give help for combo of local and compat commands

["Markus Rathgeb (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/KARAF-4270?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15141760#comment-15141760 ] 

Markus Rathgeb commented on KARAF-4270:
---------------------------------------

I found this issue after I implemented the fix myself. ;) I should look at JIRA first...

The applied patch fixes the NPE but returns "null" in the getDescription() function.
The other branches in that file (that handles other command types) implement the getDescription() that it returns getName() if the description is null.
See
* https://github.com/bimargulies/karaf/blob/5a00ab7ee204c05df06925f38d196840bf7f0302/shell/console/src/main/java/org/apache/karaf/shell/compat/CommandTracker.java#L88
* https://github.com/bimargulies/karaf/blob/5a00ab7ee204c05df06925f38d196840bf7f0302/shell/console/src/main/java/org/apache/karaf/shell/compat/CommandTracker.java#L136

Shouldn't we do here the same to handle it the same way as the other ones?
I would prefer to change that line:
https://github.com/bimargulies/karaf/blob/5a00ab7ee204c05df06925f38d196840bf7f0302/shell/console/src/main/java/org/apache/karaf/shell/compat/CommandTracker.java#L193
to "return getName();"

What do you think?

> Shell-compat gets NPE trying to give help for combo of local and compat commands
> --------------------------------------------------------------------------------
>
>                 Key: KARAF-4270
>                 URL: https://issues.apache.org/jira/browse/KARAF-4270
>             Project: Karaf
>          Issue Type: Bug
>    Affects Versions: 4.0.4
>            Reporter: Benson Margulies
>            Assignee: Freeman Fang
>             Fix For: 4.1.0, 4.0.5
>
>
> {noformat}
> karaf@root>help scr
> pipe: java.lang.NullPointerException
> karaf@root>
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)