Merge lp:~godricglow/postorius/bug_1308219_fix into lp:postorius

[Pranjal Yadav]

Fixed 1308219 by avoiding else loop back and a default new.form() for both moderator and owner after submit click.

[lp:~godricglow/postorius/bug_1308219_fix]

195. By Pranjal Yadav <email address hidden> on 2015-01-30

fixed 1308219 and merged trunk

[Florian Fuchs (flo-fuchs)]

Hello Pranjal,

clearing both forms after submit makes a lot more sense than what we have now.
Approving the merge.

Thanks!

Florian

review: Approve

=== modified file 'src/postorius/views/list.py'
--- src/postorius/views/list.py	2015-01-19 15:03:09 +0000
+++ src/postorius/views/list.py	2015-01-29 19:57:44 +0000
@@ -66,8 +66,6 @@
                                    % request.POST['owner_email']))
                 except HTTPError as e:
                     messages.error(request, _(e.msg))
-        else:
-            owner_form = NewOwnerForm()
         if 'moderator_email' in request.POST:
             moderator_form = NewModeratorForm(request.POST)
             if moderator_form.is_valid():
@@ -79,8 +77,8 @@
                                    % request.POST['moderator_email']))
                 except HTTPError as e:
                     messages.error(request, _(e.msg))
-        else:
-            moderator_form = NewModeratorForm()
+        owner_form = NewOwnerForm()
+        moderator_form = NewModeratorForm()
         return render_to_response('postorius/lists/members.html',
                                   {'list': self.mailing_list,
                                    'owner_form': owner_form,