Path: csiph.com!fu-berlin.de!uni-berlin.de!not-for-mail
From: Ian Kelly <ian.g.kelly@gmail.com>
Newsgroups: comp.lang.python
Subject: Re: monkey patching __code__
Date: Fri, 18 Mar 2016 07:47:28 -0600
Lines: 43
Message-ID: <mailman.317.1458308891.12893.python-list@python.org>
References: <56EBEB5A.7050207@mail.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
In-Reply-To: <56EBEB5A.7050207@mail.de>
Precedence: list
Xref: csiph.com comp.lang.python:105212

On Fri, Mar 18, 2016 at 5:49 AM, Sven R. Kunze <srkunze@mail.de> wrote:
> Hi,
>
> we got an interesting problem. We need to monkeypatch Django's reverse
> function:
>
>
> First approach:
>
> urlresolvers.reverse = patched_reverse
>
>
> Problem: some of Django's internal modules import urlresolvers.reverse
> before we can patch it for some reasons.
>
>
> Second approach:
>
> urlresolvers.reverse.__code__ = patched_reverse.__code__
>
>
> Unfortunately, we got this error:
>
>>>> reverse('login')
>
> patched_reverse() takes at least 3 arguments (1 given)
>
>
> These are the functions' signatures:
>
> def patched_reverse(viewname, urlconf=None, args=None, kwargs=None,
> prefix=None, current_app=None, get=None, fragment=None):
> def reverse(viewname, urlconf=None, args=None, kwargs=None, prefix=None,
> current_app=None):
>
>
> Some ideas?

Your patched version takes two extra arguments. Did you add the
defaults for those to the function's __defaults__ attribute?

This sounds like a pretty hairy thing that you're trying to do. Surely
there must be some better way to accomplish the same goal.