How do you understand  visibility ?

of course $priv is visible, but **in the context of class C** as you defined it as "private" !!! (hint, hint)

$this in your code is(as expected) an instance of class D not of class C ;-)

if you make $priv "protected" it will work as expected (hint,hint)

Hi Judas,

Thanks for the comment. I think we may have the same understanding of visibility in this case: D extends C, so $priv (which is defined in C) should only be visible on instances of D if the current context is C.

If $priv is visible when using the object operator on $this, I think it is justified that it would be visible when using foreach on $this from the same context.

In case my use of $this was confusing the issue, here's the same problem presented without $this (same expected and actual output as above):

<?php
class C {

  private $priv = "A private variable";

  static function doLoop(D $myD) {
    echo "Proof that the private is visible: " . $myD->priv . "\n";
    foreach ($myD as $k=>$v) {
      echo "The private is visible so should be accessed in the loop:\n";
      echo "-> $k: $v";
    }
  }
  
}

class D extends C {}

$myD = new D;
C::doLoop($myD);
?>

If this is accepted as a bug and fixed, then test http://lxr.php.net/source/php-src/tests/classes/visibility_005.phpt will need to be updated.

The private $c should be visible from f() even when invoked on an instance of derived, so the expected output should be as follows (note the extra c=>3 under ===derived::function===):

--EXPECT--
===base::function===
a=>1
b=>2
c=>3
d=>4
===base,foreach===
a=>1
d=>4
===derived::function===
a=>1
b=>2
c=>3
d=>4
===derived,foreach===
a=>1
d=>4

This bug has been fixed in CVS.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.