cakephp-fixture-factories icon indicating copy to clipboard operation
cakephp-fixture-factories copied to clipboard

Make it easier to set the connection used by the EventCollector

Open jamisonbryant opened this issue 5 months ago • 2 comments

Hi there, still using Fixture Factories and loving it after nearly 2 years. I have a suggestion for a quality-of-life improvement.

I believe this would resolve some very tricky edge cases such as described in #187 and #233.

I noticed in EventCollector that the $options array only contains the two entries for "listening to model events" and "listening to behaviors" but there are many other options you can pass to the TableLocator, including connection.

Would it make sense to add a method to EventCollector called setConnection() that could be called like so:

$article = ArticleFactory::make()->setConnection('foo')->listeningToBehaviors('Sluggable')->getEntity();

I'd be happy to draft a PR for this, if you think this solution is a good idea. If there's a concern about the EventCollector being the wrong place for this, I could think of some other ideas.

jamisonbryant avatar Aug 30 '24 14:08 jamisonbryant