Fixes bad practice of naming variable a built-in type

This commit is contained in:
dlee 2023-02-14 11:38:35 -06:00
parent 19e58f19ca
commit b7cadfc53a
2 changed files with 11 additions and 10 deletions

1
newsfragments/3976.minor Normal file
View File

@ -0,0 +1 @@
Fixes variable name same as built-in type.

View File

@ -1616,30 +1616,30 @@ class StatisticsElement(Element):
@renderer @renderer
def uploads(self, req, tag): def uploads(self, req, tag):
files = self._stats["counters"].get("uploader.files_uploaded", 0) files = self._stats["counters"].get("uploader.files_uploaded", 0)
bytes = self._stats["counters"].get("uploader.bytes_uploaded", 0) bytes_uploaded = self._stats["counters"].get("uploader.bytes_uploaded", 0)
return tag(("%s files / %s bytes (%s)" % return tag(("%s files / %s bytes (%s)" %
(files, bytes, abbreviate_size(bytes)))) (files, bytes_uploaded, abbreviate_size(bytes_uploaded))))
@renderer @renderer
def downloads(self, req, tag): def downloads(self, req, tag):
files = self._stats["counters"].get("downloader.files_downloaded", 0) files = self._stats["counters"].get("downloader.files_downloaded", 0)
bytes = self._stats["counters"].get("downloader.bytes_downloaded", 0) bytes_uploaded = self._stats["counters"].get("downloader.bytes_downloaded", 0)
return tag("%s files / %s bytes (%s)" % return tag("%s files / %s bytes (%s)" %
(files, bytes, abbreviate_size(bytes))) (files, bytes_uploaded, abbreviate_size(bytes_uploaded)))
@renderer @renderer
def publishes(self, req, tag): def publishes(self, req, tag):
files = self._stats["counters"].get("mutable.files_published", 0) files = self._stats["counters"].get("mutable.files_published", 0)
bytes = self._stats["counters"].get("mutable.bytes_published", 0) bytes_uploaded = self._stats["counters"].get("mutable.bytes_published", 0)
return tag("%s files / %s bytes (%s)" % (files, bytes, return tag("%s files / %s bytes (%s)" % (files, bytes_uploaded,
abbreviate_size(bytes))) abbreviate_size(bytes_uploaded)))
@renderer @renderer
def retrieves(self, req, tag): def retrieves(self, req, tag):
files = self._stats["counters"].get("mutable.files_retrieved", 0) files = self._stats["counters"].get("mutable.files_retrieved", 0)
bytes = self._stats["counters"].get("mutable.bytes_retrieved", 0) bytes_uploaded = self._stats["counters"].get("mutable.bytes_retrieved", 0)
return tag("%s files / %s bytes (%s)" % (files, bytes, return tag("%s files / %s bytes (%s)" % (files, bytes_uploaded,
abbreviate_size(bytes))) abbreviate_size(bytes_uploaded)))
@renderer @renderer
def raw(self, req, tag): def raw(self, req, tag):