Skip to content

Fix cache or default bugs#444

Open
FormJani wants to merge 5 commits into
django-commons:masterfrom
FormJani:fix-cache-or-default-bugs
Open

Fix cache or default bugs#444
FormJani wants to merge 5 commits into
django-commons:masterfrom
FormJani:fix-cache-or-default-bugs

Conversation

@FormJani

@FormJani FormJani commented Nov 14, 2024

Copy link
Copy Markdown
Contributor

Realised the same issue as fixed in #443 exist in the other cache backends.
This PR fixes all of them.

When a falsy value is returned from cache via memcached backend, it returns None (the default) instead, of the cached value.

This is the same issue as with the Redis backend, which was fixed here:
django-commons@1860549
@FormJani
FormJani force-pushed the fix-cache-or-default-bugs branch 3 times, most recently from 79af870 to c6e93d7 Compare November 14, 2024 13:40
@FormJani
FormJani force-pushed the fix-cache-or-default-bugs branch from c6e93d7 to 4730884 Compare November 14, 2024 13:48
@rvveber

rvveber commented Nov 25, 2025

Copy link
Copy Markdown

This is very important!
Same as #284

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants