ID: 36675
Updated by: tony2001@php.net
Reported By: iain at iaindooley dot com
-Status: Open
+Status: Bogus
Bug Type: Class/Object related
Operating System: FreeBSD 6.0
PHP Version: 5.1.3RC3-dev
New Comment:
>this same notice is also caused by:
>if($obj1 == $obj2)
Did you try it? Please try and see.
>please do not dismiss a bug without properly understanding
> what is going on.
Please don't claim anything if you didn't check it out yourself.
>if this is sensless then why doesn't it fail with the
>simple test case i provided?
Because you're comparing objects to '', which is autoconverted to
FALSE.
Try to compare objects to non-empty strings and you'll the expected
warning.
It MAY change in the future, but this is not a bug, because comparing
circles to squares doesn't make any sense.
Previous Comments:
------------------------------------------------------------------------
[2006-04-07 08:48:57] iain at iaindooley dot com
haha i just notice that you (tony2001) were the one who stated:
Dmitry, I guess we need to return at least objects index in this
case.
Or maybe use the same way we use when compatibility_mode is on.
What do you think?
which is clearly what my patch does, which has somehow been undone in
later versions.
why would you dismiss such a thourough bug report as being senseless??
you could at least have the decency to forward it on to someone else.
------------------------------------------------------------------------
[2006-04-07 08:44:45] iain at iaindooley dot com
Wrong, this is not expected, as http://bugs.php.net/bug.php?id=33999
states, and this same notice is also caused by:
if($obj1 == $obj2)
please do not dismiss a bug without properly understanding what is
going on. if this is sensless then why doesn't it fail with the simple
test case i provided?
------------------------------------------------------------------------
[2006-04-07 08:37:04] tony2001@php.net
Comparing objects or resources to strings, booleans and integers is
obviously senseless.
This is expected behaviour.
------------------------------------------------------------------------
[2006-04-07 08:24:12] iain at iaindooley dot com
By the way, enabling compatbility mode causes problems because then you
cannot pass an object as the first argument of array_key_exists() which
is unintuitive
------------------------------------------------------------------------
[2006-04-07 07:29:17] iain at iaindooley dot com
It's back!! Sorry, I previously wrote that this problem seemed to have
disappeared when i updated to 5.1.3RC3-dev, but it is now back. The
error I'm getting is:
Notice: Object of class Clause could not be converted to int in
/usr/home/iain/public_html/vpc_test/packages/nozzle/db_object.class.php
on line 2151
the line that is causing it is:
if($this->unique_clause == '')
This does not happen consistently everytime an object is used in a
boolean expression and I am unable to create a simple test case that
replicates the bug. ie. if you execute the following with php -f
test.php:
<?
class ClassOne
{
private $var;
function ClassOne()
{
$this->var = 'some value';
}
}
$var = '';
if($var == '')
{
echo('there is nothing there
');
}
$var = new ClassOne();
if($var == '')
{
echo('there is still nothing there!
');
}
else
{
echo('houston, we have no problem
');
}
die('done testing
');
?>
then you get:
> php -f test.php
there is nothing there
houston, we have no problem
done testing
as expected. It's difficult to know where to go from here because this
is obviously a bug, because it is perfectly okay to use an object in a
boolean expression, and this behaviour is _not_ exhibited in PHP 5.0.4,
but I am unable to provide you with a simple test case that reproduces
the bug.
A similar problem occurred in http://bugs.php.net/bug.php?id=33999, and
it is supposedly fixed in CVS but it has apparently crept back in.
I have been able to hack in a fix for this in the code for 5.1.3RC3-dev
with the following patch, which apparently just bypasses something
called compatibility mode:
Index: zend_operators.c
===================================================================
RCS file: /repository/ZendEngine2/zend_operators.c,v
retrieving revision 1.208.2.4
diff -u -r1.208.2.4 zend_operators.c
--- zend_operators.c 5 Feb 2006 17:07:40 -0000 1.208.2.4
+++ zend_operators.c 7 Apr 2006 07:28:18 -0000
@@ -333,14 +333,15 @@
return;
}
- if (EG(ze1_compatibility_mode)) {
+ /*if (EG(ze1_compatibility_mode)) {*/
+ else {
HashTable *ht =
Z_OBJPROP_P(op);
if (ht) {
retval =
(zend_hash_num_elements(ht)?1:0);
}
- } else {
+ }/* else {
zend_error(E_NOTICE, "Object of
class %s could not be converted to int", Z_OBJCE_P(op)->name);
- }
+ }*/
zval_dtor(op);
ZVAL_LONG(op, retval);
return;
------------------------------------------------------------------------
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
http://bugs.php.net/36675
--
Edit this bug report at http://bugs.php.net/?id=36675&edit=1