search

TrangPham / django-admin-confirm

AdminConfirmMixin is a mixin for ModelAdmin that adds confirmations to changes, additions and actions.
Other
129 stars 16 forks source link

Redis Error: wrong number of arguments for 'del' command #46

Closed chrismaille closed 1 year ago

chrismaille commented 1 year ago

Describe the bug redis returning wrong number of arguments for 'del' command when called from django-admin-confirm and cache backend is configured to Redis. Works without issues when backend is Memcached.

To Reproduce Steps to reproduce the behavior:

  1. Install latest redis version pip install redis
  2. Configure Django settings cache backend to Redis
  3. Open a model configured for confirmation
  4. Confirm Save

Expected behavior The instance model needs to be updated.

Screenshots image

Versions:

Additional context image

NeilujD commented 1 year ago

A fix has been made in the coming Django 4.2: https://github.com/django/django/commit/608ab043f75f1f9c094de57d2fd678f522bb8243 But it would be great to have a workaround in the meantime !

TrangPham commented 1 year ago

Thanks for the detailed bug report @chrismaille. And thanks @NeilujD for linking to the resolution. I've made a change to fix this issue within our code base. I'll make a new minor release soon