Skip to content

Commit 35643e5

Browse files
committed
chore: add self type hints
1 parent 3ae6de8 commit 35643e5

1 file changed

Lines changed: 13 additions & 13 deletions

File tree

tableauserverclient/server/query.py

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ def __init__(self, model: "QuerysetEndpoint[T]") -> None:
3939
self._result_cache: List[T] = []
4040
self._pagination_item = PaginationItem()
4141

42-
def __iter__(self) -> Iterator[T]:
42+
def __iter__(self: Self) -> Iterator[T]:
4343
# Not built to be re-entrant. Starts back at page 1, and empties
4444
# the result cache. Ensure the result_cache is empty to not yield
4545
# items from prior usage.
@@ -55,11 +55,11 @@ def __iter__(self) -> Iterator[T]:
5555
return
5656

5757
@overload
58-
def __getitem__(self, k: Slice) -> List[T]:
58+
def __getitem__(self: Self, k: Slice) -> List[T]:
5959
...
6060

6161
@overload
62-
def __getitem__(self, k: int) -> T:
62+
def __getitem__(self: Self, k: int) -> T:
6363
...
6464

6565
def __getitem__(self, k):
@@ -109,53 +109,53 @@ def __getitem__(self, k):
109109
# If k is unreasonable, raise an IndexError.
110110
raise IndexError
111111

112-
def _fetch_all(self) -> None:
112+
def _fetch_all(self: Self) -> None:
113113
"""
114114
Retrieve the data and store result and pagination item in cache
115115
"""
116116
if not self._result_cache:
117117
self._result_cache, self._pagination_item = self.model.get(self.request_options)
118118

119-
def __len__(self) -> int:
119+
def __len__(self: Self) -> int:
120120
return self.total_available
121121

122122
@property
123-
def total_available(self) -> int:
123+
def total_available(self: Self) -> int:
124124
self._fetch_all()
125125
return self._pagination_item.total_available
126126

127127
@property
128-
def page_number(self) -> int:
128+
def page_number(self: Self) -> int:
129129
self._fetch_all()
130130
return self._pagination_item.page_number
131131

132132
@property
133-
def page_size(self) -> int:
133+
def page_size(self: Self) -> int:
134134
self._fetch_all()
135135
return self._pagination_item.page_size
136136

137-
def filter(self, *invalid, **kwargs) -> Self:
137+
def filter(self: Self, *invalid, **kwargs) -> Self:
138138
if invalid:
139139
raise RuntimeError("Only accepts keyword arguments.")
140140
for kwarg_key, value in kwargs.items():
141141
field_name, operator = self._parse_shorthand_filter(kwarg_key)
142142
self.request_options.filter.add(Filter(field_name, operator, value))
143143
return self
144144

145-
def order_by(self, *args) -> Self:
145+
def order_by(self: Self, *args) -> Self:
146146
for arg in args:
147147
field_name, direction = self._parse_shorthand_sort(arg)
148148
self.request_options.sort.add(Sort(field_name, direction))
149149
return self
150150

151-
def paginate(self, **kwargs) -> Self:
151+
def paginate(self: Self, **kwargs) -> Self:
152152
if "page_number" in kwargs:
153153
self.request_options.pagenumber = kwargs["page_number"]
154154
if "page_size" in kwargs:
155155
self.request_options.pagesize = kwargs["page_size"]
156156
return self
157157

158-
def _parse_shorthand_filter(self, key: str) -> Tuple[str, str]:
158+
def _parse_shorthand_filter(self: Self, key: str) -> Tuple[str, str]:
159159
tokens = key.split("__", 1)
160160
if len(tokens) == 1:
161161
operator = RequestOptions.Operator.Equals
@@ -169,7 +169,7 @@ def _parse_shorthand_filter(self, key: str) -> Tuple[str, str]:
169169
raise ValueError("Field name `{}` is not valid.".format(field))
170170
return (field, operator)
171171

172-
def _parse_shorthand_sort(self, key: str) -> Tuple[str, str]:
172+
def _parse_shorthand_sort(self: Self, key: str) -> Tuple[str, str]:
173173
direction = RequestOptions.Direction.Asc
174174
if key.startswith("-"):
175175
direction = RequestOptions.Direction.Desc

0 commit comments

Comments
 (0)