No warning should be raised when defining an abstract model with no app_label
|Reported by:||charettes||Owned by:||aaugustin|
|Component:||Database layer (models, ORM)||Version:||master|
|Cc:||Triage Stage:||Ready for checkin|
|Has patch:||yes||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
Since those models are never registered and meant to be used by different apps it doesn't make much sense to raise a warning in this case.
I think this case was just overlooked when Aymeric refactored app-loading.
Attaching a patch fixing the issue and removing an extra space between in and INSTALLED_APPS in the warning message.
Change History (8)
Changed 7 weeks ago by charettes
comment:1 Changed 7 weeks ago by aaugustin
- Owner changed from nobody to aaugustin
- Status changed from new to assigned
Changed 7 weeks ago by aaugustin
comment:3 Changed 7 weeks ago by aaugustin
- Easy pickings unset
- Triage Stage changed from Unreviewed to Accepted
comment:5 Changed 7 weeks ago by Aymeric Augustin <aymeric.augustin@…>
- Resolution set to fixed
- Status changed from assigned to closed