foreach on an object is allowed. You aren't seeing anything because the $obj is empty.
http://php.net/manual/en/control-structures.foreach.php
https://3v4l.org/CCrFo

But, the link clearly states (for the first form, which is what I used as example): "It [The first form] loops over the *array* given by *array_expression*. On each iteration, the value of the current element is assigned to $value..."

 - when not iterating an array, why are we not seeing at least a warning? Would there ever be a situation where you'd legitimately use foreach on a non-array object? If so please enlighten me. Not a single one of the examples in the documentation for foreach use a non-array object.

If you have ideas on how to make the documentation more clearly indicate that foreach does, in fact, work on objects then feel free to hit the Edit link in the top-right corner of the page and submit whatever changes you think are necessary.

I don't know how to edit the documentation to alleviate this problem as I don't know when or why you'd ever knowingly use foreach on a non-array object, hence the original bug report.

Consider the following:

class SomeClass {

    private $anArray;

    public function getArray() {
        return $this->anArray;
    }

}

I left out constructor etc, but assume the private variable contains a valid array, that our class has been instantiated and that it is referenced by a variable, let's call it $ourClass.

Now somewhere in our code we *accidentally* do this:

    foreach ($ourClass as $anObject) {
        *do something with $anObject*
        // Nothing happens
    }

Instead of the correct approach:

    foreach ($ourClass->getArray() as $anObject) {
        *do something with $anObject*
        // Something happens
    }

Now our code silently fails. I realize this is a programmer's error, but are warnings and such not meant to help us avoid errors that are *always* errors? That's why I wanted an example of when the first approach is ever sensible, as in; is it ever not an error? If not, why not warn the user that he is doing something senseless? If yes, please provide an example. Or does it have something to do with speed - as in having to check for this every time foreach is used?

Thanks :)

It's fine that you don't want to iterate over regular objects. I don't do it either; I'd probably use an array or implement Iterator/IteratorAggregate instead. But just because we don't do it doesn't mean other people do not.

foreach supports objects. That will not change anytime soon. If you want to change it then you need to use the RFC process. https://wiki.php.net/rfc/howto If instead you think of a way to clarify what's written in the documentation, I've already told you about the Edit link. Otherwise there is nothing to do here.

Okay, fair enough. I'll see if I can provide something to the documentation to make this more clear to everyone.