Karaf
  1. Karaf
  2. KARAF-1499

InfoAction shell command should sort the properties from InfoProvider instances

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 2.2.4
    • Fix Version/s: 2.2.8, 2.3.0, 3.0.0
    • Component/s: karaf-shell
    • Labels:
      None

      Description

      The class org.apache.karaf.shell.commands.InfoAction accepts input from InfoProvider services. The items from those providers are printed via this code:

          for (String section : properties.keySet()) {
              System.out.println(section);
      
              for (Object key : properties.get(section).keySet()) {
                  printValue(String.valueOf(key), maxNameLen, String.valueOf(properties.get(section).get(key)));
              }
          }
      

      The .keySet() method returns keys in effectively random order, making the output hard to read. I propose instead the following presentation:

          List<String> sections = new ArrayList<String>(properties.keySet());
          Collections.sort(sections);
          for (String section : sections) {
              List<Object> keys = new ArrayList<Object>(properties.get(section).keySet());
              if (keys.size() > 0) {
                  System.out.println(section);
      
                  Collections.sort(keys, new Comparator<Object>() {
                      public int compare(Object o1, Object o2) {
                          return String.valueOf(o1).compareTo(String.valueOf(o2));
                      }
                  });
      
                  for (Object key : keys) {
                      printValue(String.valueOf(key), maxNameLen, String.valueOf(properties.get(section).get(key)));
                  }
              }
          }
      

        Activity

        No work has yet been logged on this issue.

          People

          • Assignee:
            Freeman Fang
            Reporter:
            Chris Dolan
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development