Details
-
Improvement
-
Resolution: Fixed
-
Major
-
COmanage Registry 3.3.0 (Magic Ring)
Description
The CoPetitionsController.php has in method execute_sendConfirmation code so that if debug is non-zero it sets a view variable and then lets the view be displayed. If debug is zero it instead redirects.
That works fine when there is no failure sending email.
If there is some failure with sending email, then the view is rendered regardless of the setting for debug and without the view variable the rendered HTML is unsightly.
This should be cleaned up so that if the email send fails the flash is displayed and the either better HTML is rendered or something else happens, like maybe a redirect to /registry. Either way, as long as the HTML in the view
app/View/CoPetitions/send_confirmation.ctp
is not rendered without the view variable since it looks unsightly.