Skip to content

Commit

Permalink
Bug: Merge the two 'render_json' functions rucio#6598
Browse files Browse the repository at this point in the history
  • Loading branch information
Ban42 committed Apr 8, 2024
1 parent 7fedc87 commit ae27bf6
Showing 1 changed file with 7 additions and 10 deletions.
17 changes: 7 additions & 10 deletions lib/rucio/common/utils.py
Expand Up @@ -42,7 +42,7 @@
from functools import partial, wraps
from io import StringIO
from itertools import zip_longest
from typing import TYPE_CHECKING, Optional, Type, TypeVar
from typing import TYPE_CHECKING, Optional, Type, TypeVar, Any
from urllib.parse import parse_qsl, quote, urlencode, urlparse, urlunparse
from uuid import uuid4 as uuid
from xml.etree import ElementTree
Expand Down Expand Up @@ -611,16 +611,13 @@ def default(self, obj): # pylint: disable=E0202
return json.JSONEncoder.default(self, obj)


def render_json(**data):
""" JSON render function
def render_json(list_=None, **data: Optional[Any]):
""" JSON render function for list and dicts
"""
return json.dumps(data, cls=APIEncoder)


def render_json_list(list_):
""" JSON render function for list
"""
return json.dumps(list_, cls=APIEncoder)
if list_ is not None:
return json.dumps(list_, cls=APIEncoder)
else:
return json.dumps(data, cls=APIEncoder)


def datetime_parser(dct):
Expand Down

0 comments on commit ae27bf6

Please sign in to comment.