[DomCrawler] each() iterates over items as Crawlers

[julien-c]

Q	A
Doc fix?	yes
New docs?	no
Applies to	2.3
Fixed tickets	#2568

[wouterj]

Could you please also fix the other things mentioned in #2568 ? And add the pull request format to the pr description?

[julien-c]

@wouterj PR format added. For the reduce function, there's nothing to do AFAICT, unless you want me to make the example more complex?

[wouterj]

you're right actually, I never did a good look into that example. I'm 👍 for this, thank you for working on it!

[wouterj]

One thing though: We can/should add a vesionadded directive mentioning the change. Something like:

.. versionadded:: 2.3
    The node argument is a ``Crawler`` instance as of Symfony 2.3.
    In prior versions, it was a ``DOMElement`` instance.

[julien-c]

Do you want me to do it?

[wouterj]

If you can do it: Yes! Otherwise I'll create a PR

[xabbuh]

Any news on this topic?

[weaverryan]

Hi Julien!

Thanks for the nice PR here! I've patched this into the 2.3 branch at sha: ed0920e. You'll notice that some of the changes are missing, only because they have already been done in other PR's. I also added the versionadded and a type-hint at sha: a15a9d5

Thanks!