Opened 11 years ago

Last modified 11 years ago

#22691 closed New feature

Allow cached_property to be used on other methods — at Version 1

Reported by: FunkyBob Owned by: nobody
Component: Utilities Version: dev
Severity: Normal Keywords:
Cc: loic84 Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by loic84)

This adds an optional name property to cached_property, overriding where it stores its cached value.

class Foo(object):
    def get_bar(self):
        ...heavy work...

    bar = cached_property(get_bar, name='bar')

Without the name, it would save the result in __dict__['get_bar'], thus masking the method.

Change History (1)

comment:1 by loic84, 11 years ago

Cc: loic84 added
Description: modified (diff)
Has patch: set
Needs documentation: set
Patch needs improvement: set
Triage Stage: Unreviewed → Accepted
Type: Uncategorized → New feature
Version: 1.6 → master

Sounds good.

PR: ​https://github.com/django/django/pull/2707

Since cached_property is documented, it'll need docs update and release notes.

Last edited 11 years ago by loic84 (previous) (diff)
Note: See TracTickets for help on using tickets.
Back to Top