Skip to content

Commit 74fd14f

Browse files
committed
Autofix issues with Ruff
1 parent c447108 commit 74fd14f

File tree

11 files changed

+28
-27
lines changed

11 files changed

+28
-27
lines changed

drf_excel/fields.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,9 @@
11
import contextlib
22
import datetime
33
import json
4+
from collections.abc import Iterable
45
from decimal import Decimal
5-
from typing import Any, Callable, Iterable, Union
6+
from typing import Any, Callable, Union
67

78
from django.utils.dateparse import parse_date, parse_datetime, parse_time
89
from openpyxl.cell import Cell
@@ -29,7 +30,7 @@
2930
from drf_excel.utilities import XLSXStyle, get_setting, sanitize_value, set_cell_style
3031

3132

32-
class XLSXField(object):
33+
class XLSXField:
3334
sanitize = True
3435

3536
def __init__(

drf_excel/mixins.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
from rest_framework.response import Response
33

44

5-
class XLSXFileMixin(object):
5+
class XLSXFileMixin:
66
"""
77
Mixin which allows the override of the filename being
88
passed back to the user when the spreadsheet is downloaded.
@@ -26,7 +26,7 @@ def finalize_response(self, request, response, *args, **kwargs):
2626
isinstance(response, Response)
2727
and response.accepted_renderer.format == "xlsx"
2828
):
29-
response["content-disposition"] = "attachment; filename={}".format(
30-
escape_uri_path(self.get_filename(request=request, *args, **kwargs)),
29+
response["content-disposition"] = (
30+
f"attachment; filename={escape_uri_path(self.get_filename(request=request, *args, **kwargs))}"
3131
)
3232
return response

drf_excel/renderers.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ def render(self, data, accepted_media_type=None, renderer_context=None):
5757
Render `data` into XLSX workbook, returning a workbook.
5858
"""
5959
if data is None:
60-
return bytes()
60+
return b""
6161

6262
if not self._check_validation_data(data):
6363
return json.dumps(data)

drf_excel/utilities.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
ESCAPE_CHARS = ("=", "-", "+", "@", "\t", "\r", "\n")
66

77

8-
class XLSXStyle(object):
8+
class XLSXStyle:
99
# Class that holds all parts of a style, but without being an actual NamedStyle
1010

1111
def __init__(self, style_dict=None):

tests/conftest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import io
2-
from typing import Union, Callable
2+
from typing import Callable, Union
33

44
import pytest
55
from openpyxl.reader.excel import load_workbook

tests/test_fields.py

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,28 @@
1-
from decimal import Decimal
21
import datetime as dt
2+
from decimal import Decimal
33
from types import SimpleNamespace
44

55
import pytest
66
from openpyxl.cell import Cell
77
from openpyxl.worksheet.worksheet import Worksheet
88
from rest_framework.fields import (
9+
BooleanField,
910
CharField,
10-
IntegerField,
11-
FloatField,
12-
DecimalField,
13-
DateTimeField,
1411
DateField,
15-
TimeField,
12+
DateTimeField,
13+
DecimalField,
14+
FloatField,
15+
IntegerField,
1616
ListField,
17-
BooleanField,
17+
TimeField,
1818
)
1919

2020
from drf_excel.fields import (
21-
XLSXField,
22-
XLSXNumberField,
21+
XLSXBooleanField,
2322
XLSXDateField,
23+
XLSXField,
2424
XLSXListField,
25-
XLSXBooleanField,
25+
XLSXNumberField,
2626
)
2727
from drf_excel.utilities import XLSXStyle
2828

tests/test_utilities.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,15 @@
1+
import datetime as dt
12
from types import SimpleNamespace
23

34
import pytest
4-
import datetime as dt
5-
65
from openpyxl.cell import Cell
7-
from openpyxl.styles import Font, PatternFill, Alignment, Border, Color, Side
6+
from openpyxl.styles import Alignment, Border, Color, Font, PatternFill, Side
87
from openpyxl.worksheet.worksheet import Worksheet
98

109
from drf_excel.utilities import (
11-
get_setting,
1210
XLSXStyle,
1311
get_attribute,
12+
get_setting,
1413
sanitize_value,
1514
set_cell_style,
1615
)

tests/test_viewset_mixin.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
from rest_framework.test import APIClient
55
from time_machine import TimeMachineFixture
66

7-
from tests.testapp.models import ExampleModel, AllFieldsModel, Tag
7+
from tests.testapp.models import AllFieldsModel, ExampleModel, Tag
88

99
pytestmark = pytest.mark.django_db
1010

tests/testapp/serializers.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
from rest_framework import serializers
22

3-
from .models import ExampleModel, AllFieldsModel
3+
from .models import AllFieldsModel, ExampleModel
44

55

66
class ExampleSerializer(serializers.ModelSerializer):

tests/testapp/views.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
11
from rest_framework.viewsets import ReadOnlyModelViewSet
2+
23
from drf_excel.mixins import XLSXFileMixin
34
from drf_excel.renderers import XLSXRenderer
45

5-
from .models import ExampleModel, AllFieldsModel
6-
from .serializers import ExampleSerializer, AllFieldsSerializer
6+
from .models import AllFieldsModel, ExampleModel
7+
from .serializers import AllFieldsSerializer, ExampleSerializer
78

89

910
class ExampleViewSet(XLSXFileMixin, ReadOnlyModelViewSet):

0 commit comments

Comments
 (0)