mirror of https://github.com/django/django.git
Removed unused variables that are overwritten.
This commit is contained in:
parent
5890d6ab03
commit
ebf34c3cdc
|
@ -296,7 +296,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.
|
||||
|
|
|
@ -140,7 +140,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
|
||||
|
|
|
@ -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:
|
||||
|
|
|
@ -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:
|
||||
|
|
|
@ -823,7 +823,6 @@ class SQLCompiler:
|
|||
"""
|
||||
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]]
|
||||
|
|
|
@ -571,15 +571,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
|
||||
|
||||
|
|
|
@ -853,7 +853,7 @@ class Variable:
|
|||
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
|
||||
|
|
|
@ -324,7 +324,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()
|
||||
|
|
|
@ -227,7 +227,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
|
||||
|
|
|
@ -206,7 +206,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)
|
||||
|
@ -214,7 +213,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)
|
||||
|
|
|
@ -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
|
||||
|
|
|
@ -426,7 +426,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),
|
||||
|
|
Loading…
Reference in New Issue