Changeset 7120

Timestamp:

04/17/2018 10:20:22 PM (8 years ago)

Author:

iandunn

Message:

WPF Stripe: Remove unnecessary finally block for readability.

File:

• sites/trunk/wordpressfoundation.org/public_html/content/plugins/wpf-stripe/wpf-stripe.php (modified) (2 diffs)

sites/trunk/wordpressfoundation.org/public_html/content/plugins/wpf-stripe/wpf-stripe.php

@@ -211 +211 @@
         }
 
-        $redirect_url = home_url( 'successful-donation/'  );
+        $redirect_url = home_url( 'successful-donation/' );
     } catch ( Exception $exception ) {
         log( $exception->getMessage(), $params );
-        $redirect_url = home_url( 'unsuccessful-donation/');
-    } finally {
-        wp_safe_redirect( $redirect_url );
-        exit;
+        $redirect_url = home_url( 'unsuccessful-donation/' );
     }
+
+    wp_safe_redirect( $redirect_url );
+    exit;
 }
 add_action( 'init', __NAMESPACE__ . '\process_payments' );
@@ -229 +229 @@
  */
 function log( $error_message, $data ) {
+    // Trigger instead of logging directly, so that it's conveniently displayed in dev environments.
     trigger_error( sprintf(
         '%s error: %s. Data: %s',