Skip to content

Commit

Permalink
Feature Request: Allow args/kwargs to be passed to a callable with an…
Browse files Browse the repository at this point in the history
… Accessor jieter#939.  Additionally added the required unit test
  • Loading branch information
JordanHyatt committed Dec 8, 2023
1 parent 1c2bfc2 commit fb159ce
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 2 deletions.
9 changes: 7 additions & 2 deletions django_tables2/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -303,7 +303,12 @@ class Accessor(str):
"Failed lookup for key [{key}] in {context}, when resolving the accessor {accessor}"
)

def __new__(cls, value):
def __init__(self, object, *callable_args, **callable_kwargs):
self.callable_args = callable_args
self.callable_kwargs = callable_kwargs
super().__init__()

def __new__(cls, value, *args, **kwargs):
instance = super().__new__(cls, value)
if cls.LEGACY_SEPARATOR in value:
instance.SEPARATOR = cls.LEGACY_SEPARATOR
Expand Down Expand Up @@ -394,7 +399,7 @@ def resolve(self, context, safe=True, quiet=False):
if safe and getattr(current, "alters_data", False):
raise ValueError(self.ALTERS_DATA_ERROR_FMT.format(method=current.__name__))
if not getattr(current, "do_not_call_in_templates", False):
current = current()
current = current(*self.callable_args, **self.callable_kwargs)
# Important that we break in None case, or a relationship
# spanning across a null-key will raise an exception in the
# next iteration, instead of defaulting.
Expand Down
11 changes: 11 additions & 0 deletions tests/test_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -130,6 +130,17 @@ def test_short_circuit_dict(self):

self.assertEqual(Accessor("occupation__name").resolve(context), "Carpenter")

def test_callable_args_kwargs(self):
class MyClass:
def method(self, *args, **kwargs):
return args, kwargs

callable_args = ('arg1', 'arg2')
callable_kwargs = {'kwarg1': 'val1', 'kwarg2':'val2'}
obj = MyClass()
result = Accessor('method', *callable_args, **callable_kwargs).resolve(obj)
self.assertEqual(result, (callable_args, callable_kwargs))


class AccessorTestModel(models.Model):
foo = models.CharField(max_length=20)
Expand Down

0 comments on commit fb159ce

Please sign in to comment.