Factory::build should take times() into account

[vlakoff]

This generates, persists, and returns a collection of items:

Factory::times(42)->create(...);

Whereas this generates and returns only one item, ignoring the times() call:

Factory::times(42)->build(...);

For consistency and least astonishment, Factory::build should take times() into account. I can't think of any API/BC break, can you?

About implementation, beware: create iteratively calls persist, which in turn calls build, don't loop in another loop!

[vlakoff]

It appears to be a bit tricky to implement, as build is also called when persisting relations…

The problem is that we're using the same build method for both public and internal usage.

I think we should separate the concerns, by implementing a new internalBuild method. But it might break BC with people injecting some customized builder, though it should be a rare case.

Your opinion on this?

[vlakoff]

Just to point this out, Builder already has something similar: public attributesFor and protected getAttributes.