naoray / laravel-factory-prefill Goto Github PK

I understand the support for 5.8+ but some of us right now are not able to update due to dependencies stuck on 5.7, any way to support 5.7 for the time being?

Hello.

First of all, great package!

I've noticed that relationships are generated using and old method. Have you considered using factory(App\REPLACE_THIS::class)->lazy(); instead?

It should not be a problem since the package is targeting illuminate 5.7 and lazy() is present since 5.5

Running the package command php artisan factory:prefill User on a laravel default installation results in

//...
'password' => $faker->bcrypt($faker->word(255)),

but should result in

//...
'password' => bcrypt($faker->word(255)),

• add test for failing scenario
• fix implementation

The current setting for Laravel 6 is too restrictive. Since Laravel now follows semver, you can use ^6.0 instead of ~6.0.0.

Improve TypeGuesser by checking the property name directly on faker instance.

• Check if Faker has a method name equal to the property name, If yes use it!

Through reflection it should be possible to identify all belongsTo() relationship and include them into the factory creation process.

Just wondering what the reason is for ignoring nullable fields.

Thanks

• Go through the upgrade guide and change everything as suggested
• Make sure all tests are passing afterwards

• Search & Find all models within the applications namespace App\ or App\Models
• Accept a custom namespace
• For each model call the factory:prefill command to create a factory