Here is a tiny diff to change for clarity. The diff is self explaining.

This is both incorrect (the argument comes from a keyword argument to the view, which default to False, and your patch ignores that argument in favor of always forcing it to be True) and wrong (because the behavior should be configurable, and should default to raising an exception when the mail sending fails).