Fixed #23395 -- Limited line lengths to 119 characters.

django/contrib/auth/admin.py

@@ -83,7 +83,11 @@ def get_form(self, request, obj=None, **kwargs):
 
     def get_urls(self):
         return [
-            url(r'^(.+)/password/$', self.admin_site.admin_view(self.user_change_password), name='auth_user_password_change'),
+            url(
+                r'^(.+)/password/$',
+                self.admin_site.admin_view(self.user_change_password),
+                name='auth_user_password_change',
+            ),
         ] + super(UserAdmin, self).get_urls()
 
     def lookup_allowed(self, lookup, value):

django/contrib/auth/migrations/0001_initial.py

@@ -58,16 +58,42 @@ class Migration(migrations.Migration):
                 ('id', models.AutoField(verbose_name='ID', serialize=False, auto_created=True, primary_key=True)),
                 ('password', models.CharField(max_length=128, verbose_name='password')),
                 ('last_login', models.DateTimeField(default=timezone.now, verbose_name='last login')),
-                ('is_superuser', models.BooleanField(default=False, help_text='Designates that this user has all permissions without explicitly assigning them.', verbose_name='superuser status')),
-                ('username', models.CharField(help_text='Required. 30 characters or fewer. Letters, digits and @/./+/-/_ only.', unique=True, max_length=30, verbose_name='username', validators=[validators.RegexValidator('^[\\w.@+-]+$', 'Enter a valid username.', 'invalid')])),
+                ('is_superuser', models.BooleanField(
+                    default=False,
+                    help_text='Designates that this user has all permissions without explicitly assigning them.',
+                    verbose_name='superuser status'
+                )),
+                ('username', models.CharField(
+                    help_text='Required. 30 characters or fewer. Letters, digits and @/./+/-/_ only.', unique=True,
+                    max_length=30, verbose_name='username',
+                    validators=[validators.RegexValidator('^[\\w.@+-]+$', 'Enter a valid username.', 'invalid')]
+                )),
                 ('first_name', models.CharField(max_length=30, verbose_name='first name', blank=True)),
                 ('last_name', models.CharField(max_length=30, verbose_name='last name', blank=True)),
                 ('email', models.EmailField(max_length=75, verbose_name='email address', blank=True)),
-                ('is_staff', models.BooleanField(default=False, help_text='Designates whether the user can log into this admin site.', verbose_name='staff status')),
-                ('is_active', models.BooleanField(default=True, help_text='Designates whether this user should be treated as active. Unselect this instead of deleting accounts.', verbose_name='active')),
+                ('is_staff', models.BooleanField(
+                    default=False, help_text='Designates whether the user can log into this admin site.',
+                    verbose_name='staff status'
+                )),
+                ('is_active', models.BooleanField(
+                    default=True, verbose_name='active', help_text=(
+                        'Designates whether this user should be treated as active. Unselect this instead of deleting '
+                        'accounts.'
+                    )
+                )),
                 ('date_joined', models.DateTimeField(default=timezone.now, verbose_name='date joined')),
-                ('groups', models.ManyToManyField(to='auth.Group', verbose_name='groups', blank=True, help_text='The groups this user belongs to. A user will get all permissions granted to each of their groups.', related_name='user_set', related_query_name='user')),
-                ('user_permissions', models.ManyToManyField(to='auth.Permission', verbose_name='user permissions', blank=True, help_text='Specific permissions for this user.', related_name='user_set', related_query_name='user')),
+                ('groups', models.ManyToManyField(
+                    to='auth.Group', verbose_name='groups', blank=True, related_name='user_set',
+                    related_query_name='user', help_text=(
+                        'The groups this user belongs to. A user will get all permissions granted to each of their '
+                        'groups.'
+                    )
+                )),
+                ('user_permissions', models.ManyToManyField(
+                    to='auth.Permission', verbose_name='user permissions', blank=True,
+                    help_text='Specific permissions for this user.', related_name='user_set',
+                    related_query_name='user')
+                 ),
             ],
             options={
                 'swappable': 'AUTH_USER_MODEL',

django/contrib/auth/migrations/0004_alter_user_username_opts.py

@@ -16,6 +16,15 @@ class Migration(migrations.Migration):
         migrations.AlterField(
             model_name='user',
             name='username',
-            field=models.CharField(error_messages={'unique': 'A user with that username already exists.'}, max_length=30, validators=[django.core.validators.RegexValidator('^[\\w.@+-]+$', 'Enter a valid username. This value may contain only letters, numbers and @/./+/-/_ characters.', 'invalid')], help_text='Required. 30 characters or fewer. Letters, digits and @/./+/-/_ only.', unique=True, verbose_name='username'),
+            field=models.CharField(
+                error_messages={'unique': 'A user with that username already exists.'}, max_length=30,
+                validators=[django.core.validators.RegexValidator(
+                    '^[\\w.@+-]+$',
+                    'Enter a valid username. This value may contain only letters, numbers and @/./+/-/_ characters.',
+                    'invalid'
+                )],
+                help_text='Required. 30 characters or fewer. Letters, digits and @/./+/-/_ only.',
+                unique=True, verbose_name='username'
+            ),
         ),
     ]

django/contrib/flatpages/migrations/0001_initial.py

@@ -19,8 +19,16 @@ class Migration(migrations.Migration):
                 ('title', models.CharField(max_length=200, verbose_name='title')),
                 ('content', models.TextField(verbose_name='content', blank=True)),
                 ('enable_comments', models.BooleanField(default=False, verbose_name='enable comments')),
-                ('template_name', models.CharField(help_text="Example: 'flatpages/contact_page.html'. If this isn't provided, the system will use 'flatpages/default.html'.", max_length=70, verbose_name='template name', blank=True)),
-                ('registration_required', models.BooleanField(default=False, help_text='If this is checked, only logged-in users will be able to view the page.', verbose_name='registration required')),
+                ('template_name', models.CharField(
+                    help_text=(
+                        "Example: 'flatpages/contact_page.html'. If this isn't provided, the system will use "
+                        "'flatpages/default.html'."
+                    ), max_length=70, verbose_name='template name', blank=True
+                )),
+                ('registration_required', models.BooleanField(
+                    default=False, help_text='If this is checked, only logged-in users will be able to view the page.',
+                    verbose_name='registration required'
+                )),
                 ('sites', models.ManyToManyField(to='sites.Site', verbose_name='sites')),
             ],
             options={

django/contrib/redirects/migrations/0001_initial.py

@@ -21,8 +21,15 @@ class Migration(migrations.Migration):
                     on_delete=models.CASCADE,
                     verbose_name='site',
                 )),
-                ('old_path', models.CharField(help_text="This should be an absolute path, excluding the domain name. Example: '/events/search/'.", max_length=200, verbose_name='redirect from', db_index=True)),
-                ('new_path', models.CharField(help_text="This can be either an absolute path (as above) or a full URL starting with 'http://'.", max_length=200, verbose_name='redirect to', blank=True)),
+                ('old_path', models.CharField(
+                    help_text=(
+                        "This should be an absolute path, excluding the domain name. Example: '/events/search/'."
+                    ), max_length=200, verbose_name='redirect from', db_index=True
+                )),
+                ('new_path', models.CharField(
+                    help_text="This can be either an absolute path (as above) or a full URL starting with 'http://'.",
+                    max_length=200, verbose_name='redirect to', blank=True
+                )),
             ],
             options={
                 'ordering': ('old_path',),

django/contrib/sessions/migrations/0001_initial.py

@@ -14,7 +14,9 @@ class Migration(migrations.Migration):
         migrations.CreateModel(
             name='Session',
             fields=[
-                ('session_key', models.CharField(max_length=40, serialize=False, verbose_name='session key', primary_key=True)),
+                ('session_key', models.CharField(
+                    max_length=40, serialize=False, verbose_name='session key', primary_key=True
+                )),
                 ('session_data', models.TextField(verbose_name='session data')),
                 ('expire_date', models.DateTimeField(verbose_name='expire date', db_index=True)),
             ],

django/contrib/sites/migrations/0001_initial.py

@@ -15,7 +15,9 @@ class Migration(migrations.Migration):
             name='Site',
             fields=[
                 ('id', models.AutoField(verbose_name='ID', serialize=False, auto_created=True, primary_key=True)),
-                ('domain', models.CharField(max_length=100, verbose_name='domain name', validators=[_simple_domain_name_validator])),
+                ('domain', models.CharField(
+                    max_length=100, verbose_name='domain name', validators=[_simple_domain_name_validator]
+                )),
                 ('name', models.CharField(max_length=50, verbose_name='display name')),
             ],
             options={

django/contrib/sites/migrations/0002_alter_domain_unique.py

@@ -15,6 +15,9 @@ class Migration(migrations.Migration):
         migrations.AlterField(
             model_name='site',
             name='domain',
-            field=models.CharField(max_length=100, unique=True, validators=[django.contrib.sites.models._simple_domain_name_validator], verbose_name='domain name'),
+            field=models.CharField(
+                max_length=100, unique=True, validators=[django.contrib.sites.models._simple_domain_name_validator],
+                verbose_name='domain name'
+            ),
         ),
     ]

django/core/management/commands/inspectdb.py

@@ -92,7 +92,10 @@ def handle_inspection(self, options):
                             extra_params['unique'] = True
 
                     if is_relation:
-                        rel_to = "self" if relations[column_name][1] == table_name else table2model(relations[column_name][1])
+                        rel_to = (
+                            "self" if relations[column_name][1] == table_name
+                            else table2model(relations[column_name][1])
+                        )
                         if rel_to in known_models:
                             field_type = 'ForeignKey(%s' % rel_to
                         else:

django/core/serializers/python.py

@@ -124,14 +124,15 @@ def Deserializer(object_list, **options):
 
             # Handle M2M relations
             if field.remote_field and isinstance(field.remote_field, models.ManyToManyRel):
-                if hasattr(field.remote_field.model._default_manager, 'get_by_natural_key'):
+                model = field.remote_field.model
+                if hasattr(model._default_manager, 'get_by_natural_key'):
                     def m2m_convert(value):
                         if hasattr(value, '__iter__') and not isinstance(value, six.text_type):
-                            return field.remote_field.model._default_manager.db_manager(db).get_by_natural_key(*value).pk
+                            return model._default_manager.db_manager(db).get_by_natural_key(*value).pk
                         else:
-                            return force_text(field.remote_field.model._meta.pk.to_python(value), strings_only=True)
+                            return force_text(model._meta.pk.to_python(value), strings_only=True)
                 else:
-                    m2m_convert = lambda v: force_text(field.remote_field.model._meta.pk.to_python(v), strings_only=True)
+                    m2m_convert = lambda v: force_text(model._meta.pk.to_python(v), strings_only=True)
 
                 try:
                     m2m_data[field.name] = []
@@ -142,21 +143,24 @@ def m2m_convert(value):
 
             # Handle FK fields
             elif field.remote_field and isinstance(field.remote_field, models.ManyToOneRel):
+                model = field.remote_field.model
                 if field_value is not None:
                     try:
-                        if hasattr(field.remote_field.model._default_manager, 'get_by_natural_key'):
+                        default_manager = model._default_manager
+                        field_name = field.remote_field.field_name
+                        if hasattr(default_manager, 'get_by_natural_key'):
                             if hasattr(field_value, '__iter__') and not isinstance(field_value, six.text_type):
-                                obj = field.remote_field.model._default_manager.db_manager(db).get_by_natural_key(*field_value)
+                                obj = default_manager.db_manager(db).get_by_natural_key(*field_value)
                                 value = getattr(obj, field.remote_field.field_name)
                                 # If this is a natural foreign key to an object that
                                 # has a FK/O2O as the foreign key, use the FK value
-                                if field.remote_field.model._meta.pk.remote_field:
+                                if model._meta.pk.remote_field:
                                     value = value.pk
                             else:
-                                value = field.remote_field.model._meta.get_field(field.remote_field.field_name).to_python(field_value)
+                                value = model._meta.get_field(field_name).to_python(field_value)
                             data[field.attname] = value
                         else:
-                            data[field.attname] = field.remote_field.model._meta.get_field(field.remote_field.field_name).to_python(field_value)
+                            data[field.attname] = model._meta.get_field(field_name).to_python(field_value)
                     except Exception as e:
                         raise base.DeserializationError.WithData(e, d['model'], d.get('pk'), field_value)
                 else:

django/core/serializers/xml_serializer.py

@@ -236,12 +236,13 @@ def _handle_fk_field_node(self, node, field):
         if node.getElementsByTagName('None'):
             return None
         else:
-            if hasattr(field.remote_field.model._default_manager, 'get_by_natural_key'):
+            model = field.remote_field.model
+            if hasattr(model._default_manager, 'get_by_natural_key'):
                 keys = node.getElementsByTagName('natural')
                 if keys:
                     # If there are 'natural' subelements, it must be a natural key
                     field_value = [getInnerText(k).strip() for k in keys]
-                    obj = field.remote_field.model._default_manager.db_manager(self.db).get_by_natural_key(*field_value)
+                    obj = model._default_manager.db_manager(self.db).get_by_natural_key(*field_value)
                     obj_pk = getattr(obj, field.remote_field.field_name)
                     # If this is a natural foreign key to an object that
                     # has a FK/O2O as the foreign key, use the FK value
@@ -250,29 +251,31 @@ def _handle_fk_field_node(self, node, field):
                 else:
                     # Otherwise, treat like a normal PK
                     field_value = getInnerText(node).strip()
-                    obj_pk = field.remote_field.model._meta.get_field(field.remote_field.field_name).to_python(field_value)
+                    obj_pk = model._meta.get_field(field.remote_field.field_name).to_python(field_value)
                 return obj_pk
             else:
                 field_value = getInnerText(node).strip()
-                return field.remote_field.model._meta.get_field(field.remote_field.field_name).to_python(field_value)
+                return model._meta.get_field(field.remote_field.field_name).to_python(field_value)
 
     def _handle_m2m_field_node(self, node, field):
         """
         Handle a <field> node for a ManyToManyField.
         """
-        if hasattr(field.remote_field.model._default_manager, 'get_by_natural_key'):
+        model = field.remote_field.model
+        default_manager = model._default_manager
+        if hasattr(default_manager, 'get_by_natural_key'):
             def m2m_convert(n):
                 keys = n.getElementsByTagName('natural')
                 if keys:
                     # If there are 'natural' subelements, it must be a natural key
                     field_value = [getInnerText(k).strip() for k in keys]
-                    obj_pk = field.remote_field.model._default_manager.db_manager(self.db).get_by_natural_key(*field_value).pk
+                    obj_pk = default_manager.db_manager(self.db).get_by_natural_key(*field_value).pk
                 else:
                     # Otherwise, treat like a normal PK value.
-                    obj_pk = field.remote_field.model._meta.pk.to_python(n.getAttribute('pk'))
+                    obj_pk = model._meta.pk.to_python(n.getAttribute('pk'))
                 return obj_pk
         else:
-            m2m_convert = lambda n: field.remote_field.model._meta.pk.to_python(n.getAttribute('pk'))
+            m2m_convert = lambda n: model._meta.pk.to_python(n.getAttribute('pk'))
         return [m2m_convert(c) for c in node.getElementsByTagName("object")]
 
     def _get_model_from_node(self, node, attr):

django/db/migrations/autodetector.py

@@ -468,7 +468,11 @@ def generate_renamed_models(self):
                                 )
                             )
                             self.renamed_models[app_label, model_name] = rem_model_name
-                            self.renamed_models_rel['%s.%s' % (rem_model_state.app_label, rem_model_state.name)] = '%s.%s' % (model_state.app_label, model_state.name)
+                            renamed_models_rel_key = '%s.%s' % (rem_model_state.app_label, rem_model_state.name)
+                            self.renamed_models_rel[renamed_models_rel_key] = '%s.%s' % (
+                                model_state.app_label,
+                                model_state.name,
+                            )
                             self.old_model_keys.remove((rem_app_label, rem_model_name))
                             self.old_model_keys.append((app_label, model_name))
                             break
@@ -505,7 +509,8 @@ def generate_created_models(self):
                             related_fields[field.name] = field
                     # through will be none on M2Ms on swapped-out models;
                     # we can treat lack of through as auto_created=True, though.
-                    if getattr(field.remote_field, "through", None) and not field.remote_field.through._meta.auto_created:
+                    if (getattr(field.remote_field, "through", None)
+                            and not field.remote_field.through._meta.auto_created):
                         related_fields[field.name] = field
             for field in model_opts.local_many_to_many:
                 if field.remote_field.model:
@@ -681,7 +686,8 @@ def generate_deleted_models(self):
                         related_fields[field.name] = field
                     # through will be none on M2Ms on swapped-out models;
                     # we can treat lack of through as auto_created=True, though.
-                    if getattr(field.remote_field, "through", None) and not field.remote_field.through._meta.auto_created:
+                    if (getattr(field.remote_field, "through", None)
+                            and not field.remote_field.through._meta.auto_created):
                         related_fields[field.name] = field
             for field in model._meta.local_many_to_many:
                 if field.remote_field.model: