runekaagaard / django-admin-flexselect Goto Github PK

Add the flex select widget to an inline field does not work correctly. The trigger fields are not correctly identified by the widget.

It appears from the code that any logged in user could query for any object without read permission check.

I chained a fk select to another and then a third one to the second. When the "root" select is changed, the second select is adjusted accordingly but the third one isn't reset.

Hello,

This package does not currently appear to be thread safe. When running this package under gunicorn with multiple workers (I used 4 for my sample), flex select fails with the below traceback. For your reference my setup was django 1.5.5 on Python 2.7.3 with nginx proxying to gunicorn 18.0.

2013-11-18 08:57:51,967 django.request ERROR Internal Server Error: /flexselect/field_changed
Traceback (most recent call last):
  File "/home/xxx/xxx/local/lib/python2.7/site-packages/django/core/handlers/base.py", line 115, in get_response
    response = callback(request, *callback_args, **callback_kwargs)
  File "/home/xxx/xxx/local/lib/python2.7/site-packages/django/contrib/auth/decorators.py", line 25, in _wrapped_view
    return view_func(request, *args, **kwargs)
  File "/home/xxx/xxx/local/lib/python2.7/site-packages/flexselect/views.py", line 17, in field_changed
    widget = FlexSelectWidget.instances[hashed_name]
KeyError: u'_7f7204089cbe1187f19ebc6d47c87c273bb9bb52'

I think this is due to the way that the FlexWidgets are being stored in the module/memory. I am currently working to find a fix to the issue. I will raise a pull request when I have something that works.

Many thanks,
Rob

Hi,
thanks for flexselect! It's definitely the cleanest solution to django admin chained selects around.

If I give the triggering field an initial value then I currently have to do a Javascript call to change() manually.
Would be great if the trigger fields were checked for an existing value and triggered initially if there was any.

Cheers,
Danny

This doesn't work on Django 1.9 - is there another way to do this now?

error: ImportError: cannot import name execute_manager

If a parent change triggers new choices, the current choice will be blanked even if it is available in the new choices. I think that is undesirable (and makes the change() mentioned in issue #1 a little more complicated).

I am using this app in Django 1.7.8

I am getting the error: No module named defaults

Seems like the defaults module was removed in Django 1.6