Add unique_together support to inspectdb

[anonymous]

django/db/backends/mysql/introspection.py has a get_constraints() method.

This is what get_indexes() would return

{u'commit': {'primary_key': False, 'unique': False},
 u'repository': {'primary_key': False, 'unique': False}}

This is what get_constraints() would return

{u'commit_index': {'check': False,
                   'columns': [u'commit'],
                   'foreign_key': None,
                   'index': True,
                   'primary_key': False,
                   'unique': False},
 u'repo_index': {'check': False,
                 'columns': [u'repository'],
                 'foreign_key': None,
                 'index': True,
                 'primary_key': False,
                 'unique': False},
 u'unique_build': {'check': False,
                   'columns': [u'number', u'jobname', u'repository'],
                   'foreign_key': None,
                   'index': True,
                   'primary_key': False,
                   'unique': True}}

This is what manage.py inspectdb currently returns

class Builds(models.Model):
    number = models.IntegerField(blank=True, null=True)
    jobname = models.CharField(max_length=64)
    repository = models.CharField(max_length=64)
    commit = models.CharField(max_length=40)
    class Meta:
        managed = False
        db_table = 'builds'

it could be smarter and invoke get_constaints() to add the correct unique_together clause in Meta.

unique_together = ((u'number', u'jobname', u'repository'),)

[Damien Nozay]

reported by dnozay.

patch is at ​https://github.com/django/django/pull/2920.patch

[Tim Graham]

Added some comments for improvement on the PR. Please uncheck "Patch needs improvement" when you update it, thanks.

[Tim Graham <timograham@…>]

In 70c54a3694975c43a2e0e22c4a90c3cbe1eb54e4:

Fixed #23028: Added unique_togther support to inspectdb.