ID: 41727
Updated by: [EMAIL PROTECTED]
Reported By: dan at yes dot lt
-Status: Open
+Status: Bogus
Bug Type: SPL related
Operating System: WinXP
PHP Version: 5.2.3
New Comment:
.
Previous Comments:
------------------------------------------------------------------------
[2007-06-22 07:46:23] dan at yes dot lt
ok, now think... if you have method to work with array or any object
with ArrayAccess implemented...
function some($list)
{
if (isset($list[0])) {
// do something with $list[0]
return true;
} else {
// no $list[0] defined, skipping
return false;
}
}
user, who implements this function must not know the real type of the
$list variable - but only that this variable can be accessed as
array...
now - if $list = array(0 => null), then this function works correctly,
but if $list = new ArrayObject(array(0 => null)), then it fails...
now, you say, i must to use offsetGet method... then, i must check
whether $list is instance of ArrayAccess, then add complex checkings
using offsetExists and offsetGet... was the ArrayAccess realy planned to
be such a silly structure to use it in such complex way?.. I don't think
so. I still think, isset() must automaticaly call offsetGet after
offsetExists returns true...
------------------------------------------------------------------------
[2007-06-22 07:34:01] [EMAIL PROTECTED]
Yes, I still think so.
------------------------------------------------------------------------
[2007-06-22 04:38:24] dan at yes dot lt
isset Determine whether a variable is set
[http://php.net/isset]
array_key_exists Checks if the given key or index exists in the
array
[http://php.net/array-key-exists]
<?php
$a = array(0 => 10, 1 => null);
// so...
var_dump(isset($a[0])); // bool(true)
var_dump(array_key_exists(0, $a)); // bool(true)
// but...
var_dump(isset($a[1])); // bool(false)
var_dump(array_key_exists(0, $a)); // bool(true)
?>
now...
ArrayObject::offsetExists Returns whether the requested $index
exists
[http://php.net/ArrayObject-offsetExists]
so, offsetExists must return the same as array_key_exists... but how
isset() must work with ArrayAccess?..
<?php
$a = new ArrayObject(array(0 => 10, 1 => null));
// so...
var_dump(isset($a[0])); // bool(true)
var_dump($a->offsetExists(0)); // bool(true)
var_dump(array_key_exists(0, $a)); // bool(true)
// but...
var_dump(isset($a[1])); // bool(true) | false expected
var_dump($a->offsetExists(1)); // bool(true)
var_dump(array_key_exists(1, $a)); // bool(true)
?>
in this case isset() returns true, but obviously must return false...
don't you think so?.. isn't this situation silly?..
and do you still think - "this is not a bug"?..
------------------------------------------------------------------------
[2007-06-21 21:55:36] [EMAIL PROTECTED]
Thank you for taking the time to write to us, but this is not
a bug. Please double-check the documentation available at
http://www.php.net/manual/ and the instructions on how to report
a bug at http://bugs.php.net/how-to-report.php
You can call offsetGet() yourself if you want.
------------------------------------------------------------------------
[2007-06-18 13:00:54] dan at yes dot lt
Description:
------------
method ArrayAccess actualy works wrong with isset()/empty()...
isset()/empty() now calls only ArrayAccess::offsetExist, but should
call both ArrayAccess::offsetExist and ArrayAccess::offsetGet to check
if value is realy set.
that's how arrays do...
Reproduce code:
---------------
class Test implements ArrayAccess
{
protected $_array = array();
protected $_count = 0;
function offsetExists($offset)
{
return $offset >= 0 && $offset < $this->_count;
}
function offsetGet($offset)
{
return $this->_array[$offset];
}
function offsetSet($offset, $value)
{
$this->_array[] = $value;
$this->_count++;
}
function offsetUnset($offset)
{
unset($this->_array[$offset]);
}
}
$a = new Test;
$a[] = 'A'; // 0
$a[] = 10; // 1
$a[] = null; // 2
echo isset($a[0]) ? 'set' : 'unset', "\n";
echo isset($a[1]) ? 'set' : 'unset', "\n";
echo isset($a[2]) ? 'set' : 'unset', "\n";
Expected result:
----------------
set
set
unset
Actual result:
--------------
set
set
set
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=41727&edit=1
