#785: many2many table and legacy databases.
-------------------------------+--------------------------------------------
Reporter: [EMAIL PROTECTED] | Owner: mtredinnick
Status: new | Component: Database wrapper
Version: SVN | Resolution:
Keywords: | Stage: Accepted
Has_patch: 1 | Needs_docs: 1
Needs_tests: 1 | Needs_better_patch: 1
-------------------------------+--------------------------------------------
Changes (by russellm):
* needs_better_patch: 0 => 1
* needs_tests: 0 => 1
* needs_docs: 0 => 1
Comment:
Three comments:
1) This should be attached as a file to the ticket, rather than copied
into a comment
2) The m2m test at the end is currently testing db_table. By changing that
option, you are removing the regression test for the db_table setting.
Don't remove an existing test, just add a new one (e.g., another m2m
relation in that model, with some logic taking the relation for a walk).
3) Documentation! There needs to be an entry on the db-api page on these
settings. This documentation should be part of your patch.
--
Ticket URL: <http://code.djangoproject.com/ticket/785#comment:8>
Django Code <http://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 post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
