Skip to content

Commit 13161bb

Browse files
committed
ignoring the list return vs the typical dictionary
1 parent 3639db7 commit 13161bb

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

atlassian/jira.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3401,17 +3401,17 @@ def get_all_available_screen_fields(self, screen_id: T_id) -> T_resp_json:
34013401
url = "{base_url}/{screen_id}/availableFields".format(base_url=base_url, screen_id=screen_id)
34023402
return self.get(url)
34033403

3404-
def get_screen_tabs(self, screen_id: T_id) -> list:
3404+
def get_screen_tabs(self, screen_id: T_id) -> Optional[list]:
34053405
"""
34063406
Get tabs for the screen id
34073407
:param screen_id:
34083408
:return:
34093409
"""
34103410
base_url = self.resource_url("screens")
34113411
url = "{base_url}/{screen_id}/tabs".format(base_url=base_url, screen_id=screen_id)
3412-
return self.get(url)
3412+
return self.get(url) # type: ignore[return-value]
34133413

3414-
def get_screen_tab_fields(self, screen_id: T_id, tab_id: T_id) -> list:
3414+
def get_screen_tab_fields(self, screen_id: T_id, tab_id: T_id) -> Optional[list]:
34153415
"""
34163416
Get fields by the tab id and the screen id
34173417
:param tab_id:
@@ -3422,20 +3422,20 @@ def get_screen_tab_fields(self, screen_id: T_id, tab_id: T_id) -> list:
34223422
url = "{base_url}/{screen_id}/tabs/{tab_id}/fields".format(
34233423
base_url=base_url, screen_id=screen_id, tab_id=tab_id
34243424
)
3425-
return self.get(url)
3425+
return self.get(url) # type: ignore[return-value]
34263426

34273427
def get_all_screen_fields(self, screen_id: T_id) -> list:
34283428
"""
34293429
Get all fields by screen id
34303430
:param screen_id:
34313431
:return:
34323432
"""
3433-
screen_tabs = self.get_screen_tabs(screen_id)
3433+
screen_tabs = self.get_screen_tabs(screen_id) or []
34343434
fields: list = []
34353435
for screen_tab in screen_tabs:
34363436
tab_id = screen_tab["id"]
34373437
if tab_id:
3438-
tab_fields = self.get_screen_tab_fields(screen_id=screen_id, tab_id=tab_id)
3438+
tab_fields = self.get_screen_tab_fields(screen_id=screen_id, tab_id=tab_id) or []
34393439
fields = fields + tab_fields
34403440
return fields
34413441

0 commit comments

Comments
 (0)