On Mon, Nov 14, 2016 at 2:22 PM, Christoph M. Becker <[email protected]> wrote:
> On 13.11.2016 at 22:10, Craig Duncan wrote: > > >> How about just making those classes implement the interface instead? > > > > Christoph pointed out that there may be classes in extensions that use > > count_elements. > > Furthermore, letting those classes implement Countable could break BC. > How would this break BC? SXE even already has a count() method, and count_elements respects overrides of that method. Clearly SXE is supposed to implement Countable, it was simply forgotten. > Also it would make userland code simpler: > > is_countable($thing) > > vs > > is_array($thing) || $thing implements \Countable > > Good catch! > > Note that Craig has added is_countable() to PR #2185 (which implements > the RFC). In my opinion, that is okay, but if anybody has objections, > we probably need another RFC. > This should be done as a separate change IMHO. Nikita
