cache_on_save is not working as expected

[bourivouh]

Hello!
I try to use cache_on_save feature.
In settings.py:

CACHEOPS = {
    'app.model': ('all', DEFAULT_CACHE_TIME, {"cache_on_save": "video"})
}

Model object cached after saving by key conj:app.model:video_id=1 - all right.
But in the next calling of get function (Model.objects.get(video=1)), Model is retrieved from database, not from cache.

It happens because _cache_key return diffrent keys.
That's because when Model is cached after saving we have in _cache_key function:

extra = "cacheops.query.<lambda>"
self.query.default_ordering == True
self.query.used_aliases == set(["app_model"])

But when Model retrieve:

extra = ""
self.query.default_ordering == False
self.query.used_aliases == set([])

[Suor]

Hello, thanks for your report. It would be very helpful if you provided a test for it.

You can read here an example how to do that: #29 (comment)

[Suor]

You can try if that helps. If not try using 'video_id' instead of 'video' in your settings.

[bourivouh]

Ok, I'll try to write test.
Using video_id doesn't help.

[Suor]

Thanks for test. The thing is it passes.

[bourivouh]

Yes, after your last commit.
Could you create new version of cacheops with this fix?

[Suor]

I usually let new code to cool down a couple of days before releasing it. Can you use git version (pip can handle this) or just tolerate cache_on_save not working for a while?

[bourivouh]

Ok, I can wait.
Many thanks for the prompt fix.