Model.get_absolute_url lose method attributes

[Karataev Pavel]

class MyModel(models.Mode):
    @models.permalink
    @set_attr('x', 10) # set some attribute to MyModel.get_absolute_url method
    def get_absolute_url(self):
        return ('some', (self.attr, ))

>>>print MyModel().get_absolute_url.x
AttributeError: 'function' object has no attribute 'x'

[Russell Keith-Magee]

I am prompted to ask "why?", but using wraps on is good practice in general, so I'll accept.

[Karataev Pavel]

Replying to russellm:

I am prompted to ask "why?", but using wraps on is good practice in general, so I'll accept.

i am use my own cache decorator on get_absolute_url, and it store some object in function

[Karataev Pavel]

duplicate to #11891

[Carl Meyer]

This is not actually a duplicate of #11891. That ticket is about replacement of get_absolute_url in ModelBase, this one is about @models.permalink.

[Aymeric Augustin]

Change UI/UX from NULL to False.

[Aymeric Augustin]

Change Easy pickings from NULL to False.

[caleb.smithnc@…]

I added a pull request that adds the test here: ​https://github.com/django/django/pull/360
The fix.diff at the top was already applied, but the doc test in test.diff isn't. I simply rewrote it as a unit test.

[Caleb Smith <csmith@…>]

In 210fd7c658a44e51f4696555b3ce71498db55d96:

refs #12836 - Added a test to assure permalink wraps method attributes

[Stephen Burrows]

So apparently this was fixed as a side effect of fixing #15604. (See ​https://github.com/django/django/commit/72c57338). And the commit referenced by Caleb above adds tests for the specific use case mentioned here.