#20098: Django validate command fails to detect that multiple models declare the
same db_table
-------------------------------------+-------------------------------------
Reporter: carsten.klein@… | Owner: nobody
Type: | Status: new
Cleanup/optimization | Version: 1.5
Component: Database layer | Resolution:
(models, ORM) | Triage Stage: Accepted
Severity: Normal | Needs documentation: 0
Keywords: | Patch needs improvement: 0
Has patch: 1 | UI/UX: 0
Needs tests: 1 |
Easy pickings: 0 |
-------------------------------------+-------------------------------------
Changes (by bmispelon):
* cc: bmispelon@… (added)
* has_patch: 0 => 1
* needs_tests: 0 => 1
Comment:
This seems like a good idea but it needs to have tests in order for it to
be merged.
From what I can tell, the tests for the `validate` command go in
`tests/admin_scripts/tests.py`:
https://github.com/django/django/blob/master/tests/admin_scripts/tests.py#L1046
Also, consider adding a `.diff` extension to your patch so that trac can
use syntax highlighting on it.
Thanks.
--
Ticket URL: <https://code.djangoproject.com/ticket/20098#comment:2>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.
