[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 | frank.richter at hrz.tu-chemnitz.de
  Zusammenfassung  | Enhanced function of horde-active-sessions
  Warteschlange    | Horde Base
  Version          | 5.2.12
  Typ              | Enhancement
  Status           | Feedback
  Priorität        | 1. Low
  Milestone        |
  Patch            | 1
  Zuständige       |
+Neuer Anhang     | horde-active-sessions
------------------------------------------------------------------------------


frank.richter at hrz.tu-chemnitz.de (2016-11-01 15:18) hat geschrieben:

>>> - 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.
>>
>> Ok.
>>
>>> - You may want to switch to Horde_Argv because the parameter parsing
>>> is getting complicated:
>>> https://www.horde.org/libraries/Horde_Argv/docs/README
>>
>> In my first try it does'nt accept -ll
>> exception 'Horde_Argv_OptionException' with message 'invalid long
>> option string '-ll': must start with --, followed by non-dash' in
>> /usr/share/pear/Horde/Argv/Option.php:323
>
> Ah, okay. Then we probably have to stick with the current argument  
> parsing, to maintain backward compatibility.
>
>>> - You should check the return value of $sessionHandler->destroy()
>>
>> Is it boolean? Or shall I use a try ... catch?
>
> It's a boolean.
>
>>> - You should only output the number of sessions with the -l* flags.
>>
>> Ok.

Attached my next try.

Frank



frank.richter at hrz.tu-chemnitz.de (2016-11-01 15:18) hat hochgeladen:  
horde-active-sessions[1]

https://bugs.horde.org/horde/services/download/?app=whups&actionID=download_file&file=horde-active-sessions%5B1%5D&ticket=14478&fn=%2Fhorde-active-sessions%5B1%5D