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

 ID:                 62097
 User updated by:    kazuo at o-ishi dot jp
 Reported by:        kazuo at o-ishi dot jp
 Summary:            New behavior of string == has a compatibility
                     problem
 Status:             Assigned
 Type:               Bug
 Package:            Scripting Engine problem
 Operating System:   Gentoo Linux
 PHP Version:        5.4.4RC1
 Assigned To:        stas
 Block user comment: N
 Private report:     N

 New Comment:

In summary, this is my opinion:

Recent changes on string == have problems on compatibility.

Impact of the behavior change of == operator is heavy.
This change should be excluded from PHP 5.4.x series.

I'm sure that current (old) behavior also have problems, but
compatibility is better.


If it will be changed, it should be a natural extension of current
behavior; number-like string is compared as number.
(e.g. canonicalize string as number way before memcmp.)

Of course, such behavior should be described explicitly in PHP Manual.


Previous Comments:
------------------------------------------------------------------------
[2012-05-30 07:18:45] kazuo at o-ishi dot jp

Please think that it is money (or member ID).
But why?

------------------------------------------------------------------------
[2012-05-30 07:09:15] s...@php.net

What is in $row[0] in your example?

------------------------------------------------------------------------
[2012-05-30 06:17:28] s...@php.net

Automatic comment on behalf of cataphract
Revision: 
http://git.php.net/?p=php-src.git;a=commit;h=78ff9ebb6bb501dff995727512c38fdeff50021b
Log: Fixed bug #62097

------------------------------------------------------------------------
[2012-05-29 09:04:37] kazuo at o-ishi dot jp

Interface to user inputs or external format may rely on current
(released version of ) PHP behavior, ignore leading zeros or whitespaces.
I'm sorry that I cannot show you simple example.


Instead, this is example to show another incompatibility:

There is pgsql database table created by
  CREATE TABLE t (k INT, v NUMERIC(30,1));

This code print "true" on PHP 5.4.3 but "false" on latest.

-------------------------------------------------
<?php
$key = "1";
$num = "12345678901234567890";

$con = pg_connect("...");
pg_query($con, "DELETE FROM t");
pg_query($con, "INSERT INTO t (k, v) VALUES ($key, $num)");
$rs = pg_query($con, "SELECT v FROM t WHERE k = $key");
$row = pg_fetch_row($rs);
if ($row[0] == $num) {
    echo "true";
} else {
    echo "false";
}
-------------------------------------------------

------------------------------------------------------------------------
[2012-05-29 05:33:35] s...@php.net

I am sorry, but I still do not see the explanation why any code would rely on 
comparison such as "09223372036854775808" == "9223372036854775808" to return 
true. If you have such reason - please bring it forward. Repeating "HISTORICAL 
REASON" does not explain anything. If you have valid use case why it needs to 
be 
preserved - please provide it ASAP. Otherwise the only thing to conclude will 
be 
that no application has a valid reason to rely on that and the right thing will 
be to change it, as this code should behave this way from the start.

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


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=62097


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

Reply via email to