You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@karaf.apache.org by io...@apache.org on 2011/11/30 10:55:52 UTC

svn commit: r1208351 - /karaf/trunk/shell/config/src/main/java/org/apache/karaf/shell/config/completers/ConfigurationPropertyCompleter.java

Author: iocanel
Date: Wed Nov 30 09:55:51 2011
New Revision: 1208351

URL: http://svn.apache.org/viewvc?rev=1208351&view=rev
Log:
[KARAF-1029] Configuration property completer now cleans up candidates, even when no pid is found.

Modified:
    karaf/trunk/shell/config/src/main/java/org/apache/karaf/shell/config/completers/ConfigurationPropertyCompleter.java

Modified: karaf/trunk/shell/config/src/main/java/org/apache/karaf/shell/config/completers/ConfigurationPropertyCompleter.java
URL: http://svn.apache.org/viewvc/karaf/trunk/shell/config/src/main/java/org/apache/karaf/shell/config/completers/ConfigurationPropertyCompleter.java?rev=1208351&r1=1208350&r2=1208351&view=diff
==============================================================================
--- karaf/trunk/shell/config/src/main/java/org/apache/karaf/shell/config/completers/ConfigurationPropertyCompleter.java (original)
+++ karaf/trunk/shell/config/src/main/java/org/apache/karaf/shell/config/completers/ConfigurationPropertyCompleter.java Wed Nov 30 09:55:51 2011
@@ -55,8 +55,8 @@ public class ConfigurationPropertyComple
         if (session != null) {
             String pid = getPid(session);
             Set<String> propertyNames = getPropertyNames(pid);
+            delegate.getStrings().clear();
             if (propertyNames != null && !propertyNames.isEmpty()) {
-                delegate.getStrings().clear();
                 delegate.getStrings().addAll(propertyNames);
             }
         }