Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Satisfied, but still room to improvement #7

Closed
TNChalise opened this issue Sep 7, 2016 · 1 comment
Closed

Satisfied, but still room to improvement #7

TNChalise opened this issue Sep 7, 2016 · 1 comment

Comments

@TNChalise
Copy link

TNChalise commented Sep 7, 2016 •
edited
Loading

Currently, I am satisfied with the approach. But we are using smarter IDEs for development, and I need still to add model properties to the doc block for the type hinting while assigning or accessing model attributes as:

     @property string    $created_at
     @property Date      $updated_at
     @property DateTime  $deleted_at

with in the model class. If it is possible and easy to go, please add this feature with in class doc block.
Still searching for fair model generation modules as Yii has.
PS: I am using PHPStrom.
daverogers pushed a commit to daverogers/generators that referenced this issue Oct 1, 2020
Use Config to Generate Better Data
79e3354 
- Establish a config file (modelfromtable.php) that devs can use to preset configurations (which can be overridden by command line) - downside is debug, all, etc have to be explicitly set to true in command line
- Use `app/Models` directory if Laravel 8+ (laracademy#40)
- New configuration to enable/disable timestamps (laracademy#13)
- Generating model docblocks (laracademy#7)
- New configuration to set delimiter (esp useful for tables with a lot of columns)
- New configuration that sets `overwrite` to `false` by default, much safer for models that a dev has created and modified (force them to overwrite explicitly)
- New configuration that sets `$primaryKey` - supports lambda to run database-specific query to obtain the primary key column
- Expanded on previous `$cast` types, tried to make it a little smarter with regexp
- New configurations for whitelist and blacklist ("migrations" table included in config's blacklist by default, so devs have the choice to include it or not)
- General stub cleanup
@daverogers daverogers mentioned this issue Oct 1, 2020
Merged
@michael-mcmullen
Copy link
Collaborator

Closed by PR #41

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@michael-mcmullen @TNChalise