From 2ddc3873c2672f9e95135ec6cfdf402a099745ad Mon Sep 17 00:00:00 2001 From: mark_story Date: Sat, 18 Feb 2012 16:01:07 -0500 Subject: [PATCH] Update the Set::merge() documentation. --- lib/Cake/Utility/Set.php | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/lib/Cake/Utility/Set.php b/lib/Cake/Utility/Set.php index a62726708..53a5dce6b 100644 --- a/lib/Cake/Utility/Set.php +++ b/lib/Cake/Utility/Set.php @@ -30,9 +30,12 @@ class Set { * This function can be thought of as a hybrid between PHP's array_merge and array_merge_recursive. The difference * to the two is that if an array key contains another array then the function behaves recursive (unlike array_merge) * but does not do if for keys containing strings (unlike array_merge_recursive). - * See the unit test for more information. * - * Note: This function will work with an unlimited amount of arguments and typecasts non-array parameters into arrays. + * Since this method emulates `array_merge`, it will re-order numeric keys. When combined with out of + * order numeric keys containing arrays, results can be lossy. + * + * Note: This function will work with an unlimited amount of arguments and typecasts non-array + * parameters into arrays. * * @param array $arr1 Array to be merged * @param array $arr2 Array to merge with