Skip to content
This repository has been archived by the owner on Oct 29, 2019. It is now read-only.

Cannot add a new group properly when there are none to start with #71

Open
mkoistinen opened this issue Jul 28, 2015 · 5 comments
Open

Comments

@mkoistinen
Copy link
Contributor

When there are no, existing groups, attempting to create a group with the ➕ in the Person change form doesn't work. Well, technically, it works (the group is created), but the pop-up window turns blank and doesn't refresh the Person change form.

On investigation, there is a JS error in the popup as shown here:

image

It is not clear why, but there seems to be an interaction between sortedm2m's and Django's own JS that only happens when there are no existing objects in the m2m widget.

@mkoistinen mkoistinen added the bug label Jul 28, 2015
@mkoistinen mkoistinen added this to the 1.0.0 milestone Jul 28, 2015
@mkoistinen
Copy link
Contributor Author

For the record this issue occurs even without using aldryn_common.admin_fields.sortedm2m.SortedM2MModelField.

@mkoistinen
Copy link
Contributor Author

If the field is changed to a normal, models.ManyToManyField, it works perfectly (albeit without support for sorting, obviously).

@mkoistinen
Copy link
Contributor Author

I have now proven this to be purely a sortedm2m issue and have opened this ticket at that project's repo: jazzband/django-sortedm2m#57.

@mkoistinen mkoistinen modified the milestones: Future, 1.0.0 Jul 28, 2015
@mkoistinen
Copy link
Contributor Author

NOTE: once this is resolved, please remove the notice in docs/user/index.rst

@svleeuwen
Copy link

The issue in sortedm2m is resolved right

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants