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. > > Attached my next try. > > Frank
Attachment
Watch this ticket