[11.x] Feat: factory generic in make:model command

[MrPunyapal]

Based on this comment by @taylorotwell, I have adjusted the make:model command to include a factory doc-block.

For example, if I run the following command:

php artisan make:model Post

I will get a file with the code below, as it was before:

<?php namespace App\Models; use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Model; class Post extends Model { use HasFactory; }

However, if I run the following command:

php artisan make:model Comment -f

or any other command that generates a factory along with it, the resulting file will contain the code below:

<?php namespace App\Models; use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Model; class Comment extends Model { /** @use HasFactory<\Database\Factories\CommentFactory> */ use HasFactory; }

[Jubeki]

Just as an idea: Could you not also replace the HasFactory use with an empty string if the flag --factory is not set?

(Change in behaviour, but acceptable in my opinion)

[MrPunyapal]

@Jubeki
I thought for the same

But yeah it's a change in behaviour!

I would like to see @taylorotwell's thoughts on the same.

[jasonmccreary]

@MrPunyapal, something seems off here... The factory option existed, but now it adds a DocBlock. However, from what I read, disabling the factory option doesn't add the DocBlock or create the factory, but the import still remains.

I appreciate some of this may predate this PR. But seems one of these paths may be buggy.

[MrPunyapal]

@jasonmccreary
Yes if we want to make it work with use Hasfactory then we also need to take care of an import statement too.