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

* Your Email Address
* Spam protection	Enter the letters below: .__ .__ .___.__ .__. [__)[__)[__ \| \\| \| \| \| \| \|__/\|__\|
Comment	>>> - 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. >> >> Frank >
Attachment
Watch this ticket