#31286: Database specific fields checks should be databases aware.
-------------------------------------+-------------------------------------
     Reporter:  Hongtao Ma           |                    Owner:  Hongtao
                                     |  Ma
         Type:  Bug                  |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  1                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Hongtao Ma):

 * owner:  nobody => Hongtao Ma
 * status:  new => assigned


Comment:

 Applied the patch, two testcases failed, however:

 * FAIL: test_checks_called_on_the_default_database
 (check_framework.test_multi_db.TestMultiDBChecks)
 * FAIL: test_checks_called_on_the_other_database
 (check_framework.test_multi_db.TestMultiDBChecks)

 {{{
         for alias in databases:
             if router.allow_migrate(alias, app_label,
 model_name=self.model._meta.model_name):
                 return connections[alias].validation.check_field(self,
 **kwargs)
         return []
 }}}
 The early return will cause only a single backend be checked, in case of
 multiple backends provided:
 {{{
 (djangodev) ➜  my_test_proj git:(iss_check) ✗ python manage.py check
 --database sqlite3 --database mysql
 System check identified no issues (0 silenced).
 (djangodev) ➜  my_test_proj git:(iss_check) ✗ python manage.py check
 --database mysql
 SystemCheckError: System check identified some issues:

 ERRORS:
 issue.AnotherIndex.name: (mysql.E001) MySQL does not allow unique
 CharFields to have a max_length > 255.
 issue.Networkservergroup.groupname: (mysql.E001) MySQL does not allow
 unique CharFields to have a max_length > 255.

 System check identified 2 issues (0 silenced).
 (djangodev) ➜  my_test_proj git:(iss_check) ✗
 }}}

-- 
Ticket URL: <https://code.djangoproject.com/ticket/31286#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 [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/063.52158b16cde1c4d5a6bbe187b115d654%40djangoproject.com.

Reply via email to