Edit report at https://bugs.php.net/bug.php?id=65208&edit=1

 ID:                 65208
 User updated by:    of dot olivier dot favre at gmail dot com
 Reported by:        of dot olivier dot favre at gmail dot com
-Summary:            array_unique with SORT_REGULAR inconsistently
                     compare 1 and "1"
+Summary:            array_unique SORT_STRICT flag for predictable
                     results with type-mixed values
 Status:             Open
 Type:               Feature/Change Request
 Package:            Arrays related
-Operating System:   
+Operating System:   Debian wheezy
-PHP Version:        5.4.17
+PHP Version:        5.4.4
 Block user comment: N
 Private report:     N

 New Comment:

Better patch summary.


Previous Comments:
------------------------------------------------------------------------
[2013-07-05 10:02:18] of dot olivier dot favre at gmail dot com

Description:
------------
The doc claims “SORT_REGULAR - compare items normally (don't change types)”
Hence (int)1 and (string)"1" should both be considered non duplicates.

Current results with SORT_REGULAR are too unpredictable for this function/flag 
to 
be useful.

See #47370:
“I think it's better for SORT_REGULAR to compare elements by using === 
instead of 
==.”

Request:
Add a SORT_STRICT flag using strict === comparison between elements. (prevents 
from changing SORT_REGULAR behavior).


Tested on:
Debian GNU/Linux 7.1 (wheezy)
$ php --version
PHP 5.4.4-10 (cli) (built: Nov 24 2012 11:21:26) 
Copyright (c) 1997-2012 The PHP Group
Zend Engine v2.4.0, Copyright (c) 1998-2012 Zend Technologies
    with Xdebug v2.2.0, Copyright (c) 2002-2012, by Derick Rethans

Test script:
---------------
var_export(array_unique(array( 1, 2, 2, 3, "1"), SORT_REGULAR)); // "1" is 
suppressed!
echo "\n";
var_export(array_unique(array( 1, 2, 2, 3, "1", "c"), SORT_REGULAR)); // "1" 
and "c" are kept (as it should be)
echo "\n";

// Weirder, with the same values in an other order
var_export(array_unique(array( 1, 2, 2, 3, "1", "c"), SORT_REGULAR)); // "1" 
and "c" are kept (as it should be)
echo "\n";
var_export(array_unique(array( "c", 1, 2, 2, 3, "1"), SORT_REGULAR)); // "1" is 
suppressed!
echo "\n";

Expected result:
----------------
array (
  0 => 1,
  1 => 2,
  3 => 3,
  4 => '1',
)
array (
  0 => 1,
  1 => 2,
  3 => 3,
  4 => '1',
  5 => 'c',
)
array (
  0 => 1,
  1 => 2,
  3 => 3,
  4 => '1',
  5 => 'c',
)
array (
  0 => 'c',
  1 => 1,
  2 => 2,
  4 => 3,
  5 => '1',
)


Actual result:
--------------
array (
  0 => 1,
  1 => 2,
  3 => 3,
)
array (
  0 => 1,
  1 => 2,
  3 => 3,
  4 => '1',
  5 => 'c',
)
array (
  0 => 1,
  1 => 2,
  3 => 3,
  4 => '1',
  5 => 'c',
)
array (
  0 => 'c',
  1 => 1,
  2 => 2,
  4 => 3,
)



------------------------------------------------------------------------



-- 
Edit this bug report at https://bugs.php.net/bug.php?id=65208&edit=1

Reply via email to