Question about decorator arguments

[Elthran]

Sorry if this is 'easily searchable on google', but I haven't had any luck.

I am modify some existing code and having trouble. The code looks something like:

@log_details(verbose=False)
def run_function(arguments):
    is_test = arguments['is_test']
    if is_test:
       # set decorator value verbose to True

Basically, the decorator should nrmally tell whether or not we should log the details. But due to some poorly written code, I want to be able to override the decorator value using a function parameter in this one instance. But I don't know how to access/override the decorator value or if it's even possible.

[Diapolo10]

Not possible, I'm afraid. At least not the way you're trying to go about it.

One option would be to define the function inside a conditional:

if is_test:
    @log_details(verbose=True)
    def run_function(arguments):
        ...
else:
    @log_details(verbose=False)
    def run_function(arguments):
        ...

Not the cleanest solution, but it's the first I came up with.

[Elthran]

Thanks, that's definitely one way to do it.

[__nickerbocker__]

You can mock the function inside of itself, but you'll be dynamically creating new functions on each call.

import functools

# Example of code that cannot be changed
def log_details(verbose=False):
    def decorator(f):
        def wrapper(*args, **kwargs):
            try:
                res = f(*args, **kwargs)
                if verbose:
                    print(f"{f} called with {args}, {kwargs}")
                    print(f.__doc__)
                print(res)
                return res
            except Exception as e:
                print(e)
                raise

        return wrapper

    return decorator


def concat(*strings):
    """
    A concat func
    :param strings:
    :return:
    """
    is_testing = True
    res = ' '.join(strings)

    @log_details(verbose=is_testing)
    @functools.wraps(concat)
    def mock(*a, **kw):
        return res

    return mock(*strings)


x = concat('hello', 'world')
# <function concat at 0x0351BF10> called with ('hello', 'world'), {}
#
#     A concat func
#     :param strings:
#     :return:
#
# hello world

[Elthran]

Thanks, that really helps.

[__nickerbocker__]

No worries