cache_multi_on_arguments broken by @classmethod

[sqlalchemy-bot]

Migrated issue, originally created by Krishna Vudata (kvudata)

Specifically this is when using the default function_multi_key_generator.
At this line in region.py in the cache_multi_on_arguments decorator we zip up the args together with the generated keys but the default function_multi_key_generator will drop the first arg and return n-1 when the first arg is cls.

[sqlalchemy-bot]

Michael Bayer (zzzeek) wrote:

as far as I can recall, classmethods don't work at all, this is likely a dupe of #95. This is why function_multi_key_generator is pluggable. if we had more of a dedicated maintainer we could get someone to pick up on https://bitbucket.org/zzzeek/dogpile.cache/pull-requests/14/fix-cache_multi_on_arguments-when-used-on/diff

[sqlalchemy-bot]

Michael Bayer (zzzeek) wrote:

Duplicate of #95.

[sqlalchemy-bot]

Changes by Michael Bayer (zzzeek):

• added labels: duplicate
• changed status to closed

[sqlalchemy-bot]

Michael Bayer (zzzeek) wrote:

let's continue at #95. cc @jvanasco @morgan_fainberg