Add support for formset with can_delete=False

nyergler / nested-formset

Nest Django formsets for multi-level editing.

BSD 3-Clause "New" or "Revised" License

87 stars 21 forks source link

Add support for formset with can_delete=False #14

Closed sophalch closed 10 years ago

sophalch commented 10 years ago

Current instance of nested formset doesn't support can_delete=False, causing KeyError 'DELETE' exception.

nyergler commented 10 years ago

Thanks for the pull request, @schiv! Can you write unit tests for this behavior?

sophalch commented 10 years ago

Apologize Nathan, I just notice I'm using Django 1.4. The _should_delete_form method in later version of Django has been improved and doesn't cause this issue. Please ignore this pull request.

nyergler commented 10 years ago

No problem, @schiv! Thanks for letting me know.