Remove always true condition in EmailChangeConfirmView's form_valid

In a class-based view, form_valid is only called when the form has been
validated. Moreover, we only call the dispatch parent class if the reset
link has been validated. Checking it again in form_valid is not needed.

debexpo/accounts/views.py

@@ -308,7 +308,6 @@ class EmailChangeConfirmView(PasswordResetConfirmView):
         return self.render_to_response(self.get_context_data())
 
     def form_valid(self, form):
-        if self.validlink:
         self.user.email = self.email
         self.user.full_clean()
         self.user.save()