[Tickets #14478] Re: Enhanced function of horde-active-sessions

[noreply at bugs.horde.org]

BITTE NICHT AUF DIESE NACHRICHT ANTWORTEN. NACHRICHTEN AN DIESE  
E-MAIL-ADRESSE WERDEN NICHT GELESEN.

Ticket-URL: https://bugs.horde.org/ticket/14478
------------------------------------------------------------------------------
  Ticket           | 14478
  Aktualisiert Von | Jan Schneider <jan at horde.org>
  Zusammenfassung  | Enhanced function of horde-active-sessions
  Warteschlange    | Horde Base
  Version          | 5.2.12
  Typ              | Enhancement
-Status           | New
+Status           | Feedback
  Priorität        | 1. Low
  Milestone        |
  Patch            | 1
  Zuständige       |
------------------------------------------------------------------------------


Jan Schneider <jan at horde.org> (2016-10-12 14:38) hat geschrieben:

A few comments:

- The -ll output should probably stay as-is, if people already rely on  
the output, scripting it in some way
   - -lll should be used instead for the more detailed information
   - -ll could still receive the optional user parameter.
- You may want to switch to Horde_Argv because the parameter parsing  
is getting complicated:  
https://www.horde.org/libraries/Horde_Argv/docs/README
- You should check the return value of $sessionHandler->destroy()
- You should only output the number of sessions with the -l* flags.