KnpLabs/KnpMenu — pull request #50 by stof (2012-05-03 23:43:33+00:00)
Title: Iterator

This adds the RecursiveIterator interface on the iterator of the MenuItem, allowing to iterate on the whole tree by using a RecursiveIteratorIterator (and potentially limiting the depth as it can do it).

My plan is to include the new iterator in the 1.1 release (as it can already be useful). However, I'm not sure about the inclusion of the [CurrentItemFilterIterator](https://github.com/KnpLabs/KnpMenu/compare/iterator...current_iterator) I wrote. My concern about it is that I will need to change the constructor for the 2.0 release to pass the Matcher too (see #49 for it), which would break code using it for people starting to use this iterator. On the other hand, they would probably already see some breakages. So what do you think about including it ?