arpad Mon, 29 Aug 2011 21:35:28 +0000 Revision: http://svn.php.net/viewvc?view=revision&revision=315746
Log: Fixed bug #55366: keys lost when using substr_replace an array Bug: https://bugs.php.net/55366 (Closed) keys lost when using substr_replace replace an array Changed paths: U php/php-src/branches/PHP_5_3/ext/standard/string.c A php/php-src/branches/PHP_5_3/ext/standard/tests/strings/substr_replace_array.phpt Modified: php/php-src/branches/PHP_5_3/ext/standard/string.c =================================================================== --- php/php-src/branches/PHP_5_3/ext/standard/string.c 2011-08-29 21:29:26 UTC (rev 315745) +++ php/php-src/branches/PHP_5_3/ext/standard/string.c 2011-08-29 21:35:28 UTC (rev 315746) @@ -2336,6 +2336,10 @@ RETURN_STRINGL(Z_STRVAL_PP(str), Z_STRLEN_PP(str), 1); } } else { /* str is array of strings */ + char *str_index = NULL; + uint str_index_len; + ulong num_index; + array_init(return_value); if (Z_TYPE_PP(from) == IS_ARRAY) { @@ -2471,7 +2475,13 @@ } result[result_len] = '\0'; - add_next_index_stringl(return_value, result, result_len, 0); + + if (zend_hash_get_current_key_ex(Z_ARRVAL_PP(str), &str_index, &str_index_len, &num_index, 0, &pos_str) == HASH_KEY_IS_STRING) { + add_assoc_stringl_ex(return_value, str_index, str_index_len, result, result_len, 0); + } else { + add_index_stringl(return_value, num_index, result, result_len, 0); + } + if(Z_TYPE_PP(tmp_str) != IS_STRING) { zval_dtor(orig_str); } Added: php/php-src/branches/PHP_5_3/ext/standard/tests/strings/substr_replace_array.phpt =================================================================== --- php/php-src/branches/PHP_5_3/ext/standard/tests/strings/substr_replace_array.phpt (rev 0) +++ php/php-src/branches/PHP_5_3/ext/standard/tests/strings/substr_replace_array.phpt 2011-08-29 21:35:28 UTC (rev 315746) @@ -0,0 +1,19 @@ +--TEST-- +substr_replace() function - array +--FILE-- +<?php + +$arr = array('abc' => 'llsskdkk','def' => 'llsskjkkdd', 4 => 'hello', 42 => 'world'); +$newarr = substr_replace($arr, 'zzz', 0, -2); + +print_r($newarr); + +?> +--EXPECTF-- +Array +( + [abc] => zzzkk + [def] => zzzdd + [4] => zzzlo + [42] => zzzld +)
-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
