Comments (4)
Just realized the issue arises since I extend the models, it picks the new class' name as table name.
I believe another config option should be added for tables to be added, otherwise
'qualified_tables' => false,
should allow third value e.g. NULL
therefore if qualified_tables
is NULL
, then tables are not added to models;
if FALSE
then they are added without schema,
if TRUE
they are added with schema.
Once again greatful for the good job you guys are doing with this library.
from laravel.
Thanks for the reply.
Looking at .../vendor/laravel/framework/src/Illuminate/Database/Eloquent/Model.php
on getTable()
(line 1097), this might not work as suggested. It will still pick the property from child class, and getting it as NULL, assume the plural of the child class' name.
Tested it out. and still persists.
Having lots of models extended, was avoiding going through each class adding the $table
property and wanted to rely on actual table models for such.
from laravel.
This is interesting. Right now, what you can do is add the property protected $table = null;
inside your newly created model, so it'll override the parent's vale. By default Laravel uses the plural form of your class name as the table name, I think.
from laravel.
I've merged your PR. I'm closing this issue. Thanks!
from laravel.
Related Issues (20)
- missing documentation / config example about template customisation
- primary key uuid => change $keyType HOT 1
- Dates property is deprecated since Laravel 8
- Hidden property not being generated in base Model files
- Seems to struggle with relationships where the source and target column have the same name HOT 2
- Typo in WhoDidIt referring to "udpated_by" instead of "updated_by" HOT 2
- Support for Enums HOT 1
- Model relationships not generated in Laravel 9
- Error when one of the MySQL table on the server have a "." in it's name HOT 1
- HasOneOrMany
- stancl/tenancy how use HOT 2
- Deprecated "dates" property HOT 1
- Datetime or Timestamp fields are mapped to classic date in casts
- Generating models with_property_constants doesn't work correctly
- error HOT 1
- Please delete this HOT 1
- Feature request for Base File to make this package more Advanced
- Add return types to generated relations HOT 1
- Development for sql server support
- Error creating models - Laravel 9.52.6
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from laravel.