Re: [telaen-cvs] telaen folders.php,1.11,1.12

[Jim J. <ji...@ja...>]

On Aug 30, 2006, at 2:16 PM, Andrea Volani wrote:

> Update of /cvsroot/telaen/telaen
> In directory sc8-pr-cvs10.sourceforge.net:/tmp/cvs-serv6817
>
> Modified Files:
> 	folders.php
> Log Message:
> Added a sort by name for the personal folders on the list, fixed a  
> bug where old non-empty array are used causing duplicates on folder  
> list
>
> Index: folders.php
> ===================================================================
> RCS file: /cvsroot/telaen/telaen/folders.php,v
> retrieving revision 1.11
> retrieving revision 1.12
> diff -C2 -d -r1.11 -r1.12
> *** folders.php	28 Jun 2006 07:41:38 -0000	1.11
> --- folders.php	30 Aug 2006 18:16:47 -0000	1.12
> ***************
> *** 102,105 ****
> --- 102,110 ----
>   $pcounter = 0;
>
> +
> + // Reset these arrays because are used on previuos included func!!
> + $system = Array();
> + $personal = Array();
> +
>
....
> --- 196,206 ----
>   $UM->mail_disconnect();
>   unset($SS,$UM);
> +
> + // Sort and merge the 2 folders arrays
>   array_qsort2ic ($system,"name");
> + array_qsort2ic ($personal,"name");
>
> ! if (!is_array($personal))
> ! 	$personal = Array();
>   $umFolderList = array_merge((array)$system, (array)$personal);

Hmmm. That last is_array is likely not needed now since
we reset it above and, in any case, the array_qsort2ic call
would complain :)