Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding new form rows fails with multiple formsets #32

Open
elo80ka opened this issue Feb 27, 2014 · 3 comments
Open

Adding new form rows fails with multiple formsets #32

elo80ka opened this issue Feb 27, 2014 · 3 comments

Comments

@elo80ka
Copy link
Owner

elo80ka commented Feb 27, 2014

From hainea...@gmail.com on July 06, 2011 19:53:10

What steps will reproduce the problem? 1. Create a form which contains two different formsets
2. Click the add button of the first formset
3. The new form row is added to the second formset
4. Click the add button from the second formset and it will add a new row .. but using the first formset as template

I've provided a patch for this, however it changes slightly the usage because it requires the formsets to be wrapped into a block element that has a formset class (the element type doesn't matter).

Ex:

{# formset 1 goes here #}
{# formset 2 goes here #}

Attachment: formset.patch

Original issue: http://code.google.com/p/django-dynamic-formset/issues/detail?id=32

@elo80ka
Copy link
Owner Author

elo80ka commented Feb 27, 2014

From auzigog on December 13, 2011 18:05:33

Any timeline on integrating this into the code base?

@elo80ka
Copy link
Owner Author

elo80ka commented Feb 27, 2014

From stan.madueke on December 14, 2011 23:42:28

Hi auzigog. I'm guessing you followed the instructions in the documentation for working with multiple formsets ( http://code.google.com/p/django-dynamic-formset/source/browse/trunk/docs/usage.txt#137 ) -- what versions of the plugin and Django are you using?
I've reviewed your patch, but I need to be sure that this bug exists, still exists in trunk, and that applying the patch won't break anything else.

@elo80ka
Copy link
Owner Author

elo80ka commented Feb 27, 2014

From stan.madueke on December 14, 2011 23:43:51

My apologies...the comment above should've been addressed to haineault, as he was the one who reported the issue.

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

No branches or pull requests

1 participant