[1.11.x] Removed unused variables that are overwritten.

Backport of ebf34c3cdc from master

django/contrib/admindocs/views.py

@@ -302,7 +302,6 @@ class ModelDetailView(BaseAdminDocsView):
                     })
                 else:
                     arguments = get_func_full_args(func)
-                    print_arguments = arguments
                     # Join arguments with ', ' and in case of default value,
                     # join it with '='. Use repr() so that strings will be
                     # correctly displayed.

django/core/validators.py

@@ -146,7 +146,6 @@ class URLValidator(RegexValidator):
                     validate_ipv6_address(potential_ip)
                 except ValidationError:
                     raise ValidationError(self.message, code=self.code)
-            url = value
 
         # The maximum length of a full host name is 253 characters per RFC 1034
         # section 3.1. It's defined to be 255 bytes or less, but this includes

django/db/backends/mysql/creation.py

@@ -25,7 +25,7 @@ class DatabaseCreation(BaseDatabaseCreation):
         with self._nodb_connection.cursor() as cursor:
             try:
                 cursor.execute("CREATE DATABASE %s" % qn(target_database_name))
-            except Exception as e:
+            except Exception:
                 if keepdb:
                     return
                 try:

django/db/backends/postgresql/creation.py

@@ -41,7 +41,7 @@ class DatabaseCreation(BaseDatabaseCreation):
         with self._nodb_connection.cursor() as cursor:
             try:
                 cursor.execute(creation_sql)
-            except Exception as e:
+            except Exception:
                 if keepdb:
                     return
                 try:

django/db/models/sql/compiler.py

@@ -824,7 +824,6 @@ class SQLCompiler(object):
         """
         Returns an iterator over the results from executing this query.
         """
-        converters = None
         if results is None:
             results = self.execute_sql(MULTI)
         fields = [s[0] for s in self.select[0:self.col_count]]

django/http/multipartparser.py

@@ -578,15 +578,11 @@ def exhaust(stream_or_iterable):
 
     Raise a MultiPartParserError if the argument is not a stream or an iterable.
     """
-    iterator = None
     try:
         iterator = iter(stream_or_iterable)
     except TypeError:
         iterator = ChunkIter(stream_or_iterable, 16384)
 
-    if iterator is None:
-        raise MultiPartParserError('multipartparser.exhaust() was passed a non-iterable or stream parameter')
-
     for __ in iterator:
         pass
 

django/template/base.py

@@ -888,7 +888,7 @@ class Variable(object):
                         if isinstance(current, BaseContext) and getattr(type(current), bit):
                             raise AttributeError
                         current = getattr(current, bit)
-                    except (TypeError, AttributeError) as e:
+                    except (TypeError, AttributeError):
                         # Reraise if the exception was raised by a @property
                         if not isinstance(current, BaseContext) and bit in dir(current):
                             raise

django/utils/dateformat.py

@@ -327,7 +327,6 @@ class DateFormat(TimeFormat):
     def W(self):
         "ISO-8601 week number of year, weeks starting on Monday"
         # Algorithm from http://www.personal.ecu.edu/mccartyr/ISOwdALG.txt
-        week_number = None
         jan1_weekday = self.data.replace(month=1, day=1).weekday() + 1
         weekday = self.data.weekday() + 1
         day_of_year = self.z()

django/utils/ipv6.py

@@ -228,7 +228,6 @@ def _explode_shorthand_ip_string(ip_str):
         # We've already got a longhand ip_str.
         return ip_str
 
-    new_ip = []
     hextet = ip_str.split('::')
 
     # If there is a ::, we need to expand it with zeroes

tests/bulk_create/tests.py

@@ -208,7 +208,6 @@ class BulkCreateTests(TestCase):
 
     @skipUnlessDBFeature('can_return_ids_from_bulk_insert')
     def test_set_pk_and_insert_single_item(self):
-        countries = []
         with self.assertNumQueries(1):
             countries = Country.objects.bulk_create([self.data[0]])
         self.assertEqual(len(countries), 1)
@@ -216,7 +215,6 @@ class BulkCreateTests(TestCase):
 
     @skipUnlessDBFeature('can_return_ids_from_bulk_insert')
     def test_set_pk_and_query_efficiency(self):
-        countries = []
         with self.assertNumQueries(1):
             countries = Country.objects.bulk_create(self.data)
         self.assertEqual(len(countries), 4)

tests/migrations/test_base.py

@@ -77,7 +77,7 @@ class MigrationTestBase(TransactionTestCase):
                 ),
             )
 
-    def assertFKNotExists(self, table, columns, to, value=True):
+    def assertFKNotExists(self, table, columns, to):
         return self.assertFKExists(table, columns, to, False)
 
     @contextmanager

tests/urlpatterns_reverse/tests.py

@@ -422,7 +422,6 @@ class ResolverTests(SimpleTestCase):
         e = cm.exception
         # make sure we at least matched the root ('/') url resolver:
         self.assertIn('tried', e.args[0])
-        tried = e.args[0]['tried']
         self.assertEqual(
             len(e.args[0]['tried']),
             len(url_types_names),