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 > >> - You should check the return value of $sessionHandler->destroy() > > Is it boolean? Or shall I use a try ... catch? > >> - You should only output the number of sessions with the -l* flags. > > Ok. > > Frank
Attachment
Watch this ticket