Re: [Django] #27204: Clashing m2m intermediate table validation should ignore unmanaged models

2016-09-09 Thread Django 
#27204: Clashing m2m intermediate table validation should ignore unmanaged 
models
-+-
 Reporter:  timgraham|Owner:  nobody
 Type:  Bug  |   Status:  closed
Component:  Core (System |  Version:  1.10
  checks)|
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Tim Graham ):

 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"e7abb5ba8608f90ce97c6edb031ae877195616f5" e7abb5ba]:
 {{{
 #!CommitTicketReference repository=""
 revision="e7abb5ba8608f90ce97c6edb031ae877195616f5"
 Fixed #27204 -- Made clashing m2m intermediary table checks ignore
 unmanaged models.
 }}}

--
Ticket URL: 
Django 
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.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.30211268c1300b603c32d04256f3a437%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27204: Clashing m2m intermediate table validation should ignore unmanaged models

2016-09-09 Thread Django 
#27204: Clashing m2m intermediate table validation should ignore unmanaged 
models
-+-
 Reporter:  timgraham|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  Core (System |  Version:  1.10
  checks)|
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by berkerpeksag):

 * stage:  Accepted => Ready for checkin


Comment:

 [https://github.com/django/django/pull/7224/ PR #7224] looks good to me
 and tests passed on my system. Just left a comment about a potential typo
 on GitHub.

--
Ticket URL: 
Django 
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.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.a3a7ca7a0a9689828df461d0c7263ace%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27204: Clashing m2m intermediate table validation should ignore unmanaged models

2016-09-09 Thread Django 
#27204: Clashing m2m intermediate table validation should ignore unmanaged 
models
--+
 Reporter:  timgraham |Owner:  nobody
 Type:  Bug   |   Status:  new
Component:  Core (System checks)  |  Version:  1.10
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  1 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  0 |UI/UX:  0
--+
Changes (by timgraham):

 * needs_tests:  1 => 0


Comment:

 [https://github.com/django/django/pull/7224 PR]

--
Ticket URL: 
Django 
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.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.5b87149c17621cc07cc58a3c6784112b%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27204: Clashing m2m intermediate table validation should ignore unmanaged models

2016-09-09 Thread Django 
#27204: Clashing m2m intermediate table validation should ignore unmanaged 
models
--+
 Reporter:  timgraham |Owner:  nobody
 Type:  Bug   |   Status:  new
Component:  Core (System checks)  |  Version:  1.10
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  1 |  Needs documentation:  0
  Needs tests:  1 |  Patch needs improvement:  0
Easy pickings:  0 |UI/UX:  0
--+
Changes (by timgraham):

 * Attachment "27204.diff" added.

 patch without tests

--
Ticket URL: 
Django 
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.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.77d56f32a2f8a13c610f07bc36a4878b%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

[Django] #27204: Clashing m2m intermediate table validation should ignore unmanaged models

2016-09-09 Thread Django 
#27204: Clashing m2m intermediate table validation should ignore unmanaged 
models
+
   Reporter:  timgraham |  Owner:  nobody
   Type:  Bug   | Status:  new
  Component:  Core (System checks)  |Version:  1.10
   Severity:  Normal|   Keywords:
   Triage Stage:  Accepted  |  Has patch:  1
Needs documentation:  0 |Needs tests:  1
Patch needs improvement:  0 |  Easy pickings:  0
  UI/UX:  0 |
+
 The check added in #12810 currently triggers on some unmanaged models in
 Django's test suite that intentionally use clashing names. I think the
 correct solution is to ignore unmanaged models.

 ERRORS:
 model_options.Article.authors: (fields.E340) The field's intermediary
 table 'model_options_articleref_authors' clashes with the table name of
 'model_options.ArticleRef.authors'.
 model_options.Article.reviewers: (fields.E340) The field's intermediary
 table 'model_options_articleref_reviewers' clashes with the table name of
 'model_options.ArticleRef.reviewers'.
 model_options.ArticleRef.authors: (fields.E340) The field's intermediary
 table 'model_options_articleref_authors' clashes with the table name of
 'model_options.Article.authors'.
 model_options.ArticleRef.reviewers: (fields.E340) The field's intermediary
 table 'model_options_articleref_reviewers' clashes with the table name of
 'model_options.Article.reviewers'.
 unmanaged_models.C01.mm_a: (fields.E340) The field's intermediary table
 'd01' clashes with the table name of 'unmanaged_models.Intermediate'.
 unmanaged_models.C02.mm_a: (fields.E340) The field's intermediary table
 'd01' clashes with the table name of 'unmanaged_models.C01.mm_a'.

--
Ticket URL: 
Django 
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.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/052.4064f8cdb147fc05febd1bb4d8675af3%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.