← All changes
|
stripe/controllers/FrmTransLiteActionsController.php
+301
-39
6.5.2
→
trunk
View file →
| @@ -16,8 +16,9 @@ | ||
| 16 | 16 | /** |
| 17 | 17 | * Register payment action type. |
| 18 | 18 | * |
| 19 | 19 | * @param array $actions |
| 20 | + * | |
| 20 | 21 | * @return array |
| 21 | 22 | */ |
| 22 | 23 | public static function register_actions( $actions ) { |
| 23 | 24 | $actions['payment'] = 'FrmTransLiteAction'; |
| @@ -34,9 +35,9 @@ | ||
| 34 | 35 | public static function actions_js() { |
| 35 | 36 | wp_enqueue_script( |
| 36 | 37 | 'frmtrans_admin', |
| 37 | 38 | FrmTransLiteAppHelper::plugin_url() . '/js/frmtrans_admin.js', |
| 38 | - array( 'jquery' ), | |
| 39 | + array( 'jquery', 'wp-hooks' ), | |
| 39 | 40 | FrmAppHelper::plugin_version() |
| 40 | 41 | ); |
| 41 | 42 | wp_localize_script( |
| 42 | 43 | 'frmtrans_admin', |
| @@ -50,13 +51,15 @@ | ||
| 50 | 51 | /** |
| 51 | 52 | * Add event types for actions so an email can trigger on a successful payment. |
| 52 | 53 | * |
| 53 | 54 | * @param array $triggers |
| 55 | + * | |
| 54 | 56 | * @return array |
| 55 | 57 | */ |
| 56 | 58 | public static function add_payment_trigger( $triggers ) { |
| 57 | 59 | $triggers['payment-success'] = __( 'Successful Payment', 'formidable' ); |
| 58 | 60 | $triggers['payment-failed'] = __( 'Failed Payment', 'formidable' ); |
| 61 | + $triggers['payment-refunded'] = __( 'Refunded Payment', 'formidable' ); | |
| 59 | 62 | $triggers['payment-processing'] = __( 'Processing Payment', 'formidable' ); |
| 60 | 63 | $triggers['payment-future-cancel'] = __( 'Canceled Subscription', 'formidable' ); |
| 61 | 64 | $triggers['payment-canceled'] = __( 'Subscription Canceled and Expired', 'formidable' ); |
| 62 | 65 | return $triggers; |
| @@ -63,8 +66,9 @@ | ||
| 63 | 66 | } |
| 64 | 67 | |
| 65 | 68 | /** |
| 66 | 69 | * @param array $options |
| 70 | + * | |
| 67 | 71 | * @return array |
| 68 | 72 | */ |
| 69 | 73 | public static function add_trigger_to_action( $options ) { |
| 70 | 74 | $options['event'][] = 'payment-success'; |
| @@ -71,8 +75,9 @@ | ||
| 71 | 75 | $options['event'][] = 'payment-failed'; |
| 72 | 76 | $options['event'][] = 'payment-processing'; |
| 73 | 77 | $options['event'][] = 'payment-future-cancel'; |
| 74 | 78 | $options['event'][] = 'payment-canceled'; |
| 79 | + $options['event'][] = 'payment-refunded'; | |
| 75 | 80 | return $options; |
| 76 | 81 | } |
| 77 | 82 | |
| 78 | 83 | /** |
| @@ -78,19 +83,95 @@ | ||
| 78 | 83 | /** |
| 79 | 84 | * @param WP_Post $action |
| 80 | 85 | * @param stdClass $entry |
| 81 | 86 | * @param mixed $form |
| 87 | + * | |
| 82 | 88 | * @return void |
| 83 | 89 | */ |
| 84 | 90 | public static function trigger_action( $action, $entry, $form ) { |
| 85 | 91 | self::prepare_description( $action, compact( 'entry', 'form' ) ); |
| 86 | - FrmStrpLiteActionsController::trigger_gateway( $action, $entry, $form ); | |
| 92 | + | |
| 93 | + $gateway = self::get_gateway_for_action( $action ); | |
| 94 | + | |
| 95 | + if ( ! $gateway ) { | |
| 96 | + return; | |
| 97 | + } | |
| 98 | + | |
| 99 | + $class_name = FrmTransLiteAppHelper::get_setting_for_gateway( $gateway, 'class' ); | |
| 100 | + | |
| 101 | + if ( ! $class_name ) { | |
| 102 | + return; | |
| 103 | + } | |
| 104 | + | |
| 105 | + $class_name = 'Frm' . $class_name . 'ActionsController'; | |
| 106 | + $response = $class_name::trigger_gateway( $action, $entry, $form ); | |
| 107 | + | |
| 108 | + if ( ! $response['success'] && $response['show_errors'] ) { | |
| 109 | + // The payment failed | |
| 110 | + self::show_failed_message( compact( 'action', 'entry', 'form', 'response' ) ); | |
| 111 | + } | |
| 87 | 112 | } |
| 88 | 113 | |
| 89 | 114 | /** |
| 115 | + * @param WP_Post $action | |
| 116 | + * | |
| 117 | + * @return array|string | |
| 118 | + */ | |
| 119 | + private static function get_gateway_for_action( $action ) { | |
| 120 | + return $action->post_content['gateway'] ?? 'stripe'; | |
| 121 | + } | |
| 122 | + | |
| 123 | + /** | |
| 124 | + * @since 6.10 | |
| 125 | + * | |
| 126 | + * @param array $args | |
| 127 | + * | |
| 128 | + * @return void | |
| 129 | + */ | |
| 130 | + private static function show_failed_message( $args ) { | |
| 131 | + global $frm_vars; | |
| 132 | + $frm_vars['frm_trans'] = array( | |
| 133 | + 'pay_entry' => $args['entry'], | |
| 134 | + 'error' => $args['response']['error'] ?? '', | |
| 135 | + ); | |
| 136 | + | |
| 137 | + add_filter( 'frm_success_filter', 'FrmTransLiteActionsController::force_message_after_create' ); | |
| 138 | + add_filter( 'frm_pre_display_form', 'FrmTransLiteActionsController::include_form_with_success' ); | |
| 139 | + add_filter( 'frm_main_feedback', 'FrmTransLiteActionsController::replace_success_message', 5 ); | |
| 140 | + add_filter( 'frm_setup_new_fields_vars', 'FrmTransLiteActionsController::fill_entry_from_previous', 20, 2 ); | |
| 141 | + } | |
| 142 | + | |
| 143 | + /** | |
| 144 | + * @since 6.10 | |
| 145 | + * | |
| 146 | + * @param stdClass $form | |
| 147 | + * | |
| 148 | + * @return stdClass | |
| 149 | + */ | |
| 150 | + public static function include_form_with_success( $form ) { | |
| 151 | + $form->options['show_form'] = 1; | |
| 152 | + return $form; | |
| 153 | + } | |
| 154 | + | |
| 155 | + /** | |
| 156 | + * @return string | |
| 157 | + */ | |
| 158 | + public static function replace_success_message() { | |
| 159 | + global $frm_vars; | |
| 160 | + $message = $frm_vars['frm_trans']['error'] ?? ''; | |
| 161 | + | |
| 162 | + if ( ! $message ) { | |
| 163 | + $message = __( 'There was an error processing your payment.', 'formidable' ); | |
| 164 | + } | |
| 165 | + | |
| 166 | + return '<div class="frm_error_style">' . $message . '</div>'; | |
| 167 | + } | |
| 168 | + | |
| 169 | + /** | |
| 90 | 170 | * @param WP_Post $action |
| 91 | 171 | * @param stdClass $entry |
| 92 | 172 | * @param mixed $form |
| 173 | + * | |
| 93 | 174 | * @return array |
| 94 | 175 | */ |
| 95 | 176 | public static function trigger_gateway( $action, $entry, $form ) { |
| 96 | 177 | // This function must be overridden in a subclass. |
| @@ -111,8 +192,9 @@ | ||
| 111 | 192 | /** |
| 112 | 193 | * @since 6.5, introduced in v1.12 of the Payments submodule. |
| 113 | 194 | * |
| 114 | 195 | * @param object $sub |
| 196 | + * | |
| 115 | 197 | * @return void |
| 116 | 198 | */ |
| 117 | 199 | public static function trigger_subscription_status_change( $sub ) { |
| 118 | 200 | $frm_payment = new FrmTransLitePayment(); |
| @@ -129,14 +211,14 @@ | ||
| 129 | 211 | } |
| 130 | 212 | |
| 131 | 213 | /** |
| 132 | 214 | * @param array $atts |
| 215 | + * | |
| 133 | 216 | * @return void |
| 134 | 217 | */ |
| 135 | 218 | public static function trigger_payment_status_change( $atts ) { |
| 136 | - $action = isset( $atts['action'] ) ? $atts['action'] : $atts['payment']->action_id; | |
| 137 | 219 | $entry_id = isset( $atts['entry'] ) ? $atts['entry']->id : $atts['payment']->item_id; |
| 138 | - $atts = array( | |
| 220 | + $atts = array( | |
| 139 | 221 | 'trigger' => $atts['status'], |
| 140 | 222 | 'entry_id' => $entry_id, |
| 141 | 223 | ); |
| 142 | 224 | |
| @@ -151,8 +233,17 @@ | ||
| 151 | 233 | |
| 152 | 234 | // Set future-cancel as trigger when applicable. |
| 153 | 235 | $atts['trigger'] = str_replace( '_', '-', $atts['trigger'] ); |
| 154 | 236 | |
| 237 | + /** | |
| 238 | + * Trigger various hooks including frm_payment_status_complete. | |
| 239 | + * | |
| 240 | + * @since 6.25 This was included in the payments submodule, but not included in Lite until 6.25. | |
| 241 | + * | |
| 242 | + * @param array $atts | |
| 243 | + */ | |
| 244 | + do_action( 'frm_payment_status_' . $atts['trigger'], $atts ); | |
| 245 | + | |
| 155 | 246 | if ( $atts['payment'] ) { |
| 156 | 247 | self::trigger_actions_after_payment( $atts['payment'], $atts ); |
| 157 | 248 | } |
| 158 | 249 | } |
| @@ -161,8 +252,9 @@ | ||
| 161 | 252 | * Maybe trigger payment-success or payment-failed event after payment so actions (like emails) can run. |
| 162 | 253 | * |
| 163 | 254 | * @param object $payment |
| 164 | 255 | * @param array $atts |
| 256 | + * | |
| 165 | 257 | * @return void |
| 166 | 258 | */ |
| 167 | 259 | public static function trigger_actions_after_payment( $payment, $atts = array() ) { |
| 168 | 260 | if ( 'pending' === $payment->status ) { |
| @@ -169,20 +261,16 @@ | ||
| 169 | 261 | // 3D Secure has a delayed payment status, so avoid sending a payment failed email for a pending payment. |
| 170 | 262 | return; |
| 171 | 263 | } |
| 172 | 264 | |
| 173 | - $entry = FrmEntry::getOne( $payment->item_id ); | |
| 265 | + $entry = FrmEntry::getOne( $payment->item_id ); | |
| 266 | + $trigger_event = isset( $atts['trigger'] ) ? 'payment-' . $atts['trigger'] : 'payment-' . $payment->status; | |
| 267 | + $allowed_triggers = array_keys( self::add_payment_trigger( array() ) ); | |
| 174 | 268 | |
| 175 | - if ( isset( $atts['trigger'] ) ) { | |
| 176 | - $trigger_event = 'payment-' . $atts['trigger']; | |
| 177 | - } else { | |
| 178 | - $trigger_event = 'payment-' . $payment->status; | |
| 269 | + if ( ! in_array( $trigger_event, $allowed_triggers, true ) ) { | |
| 270 | + $trigger_event = $payment->status === 'complete' ? 'payment-success' : 'payment-failed'; | |
| 179 | 271 | } |
| 180 | 272 | |
| 181 | - $allowed_triggers = array_keys( self::add_payment_trigger( array() ) ); | |
| 182 | - if ( ! in_array( $trigger_event, $allowed_triggers, true ) ) { | |
| 183 | - $trigger_event = ( $payment->status === 'complete' ) ? 'payment-success' : 'payment-failed'; | |
| 184 | - } | |
| 185 | 273 | FrmFormActionsController::trigger_actions( $trigger_event, $entry->form_id, $entry->id ); |
| 186 | 274 | } |
| 187 | 275 | |
| 188 | 276 | /** |
| @@ -189,17 +277,21 @@ | ||
| 189 | 277 | * Filter fields in description. |
| 190 | 278 | * |
| 191 | 279 | * @param WP_Post $action |
| 192 | 280 | * @param array $atts |
| 281 | + * | |
| 193 | 282 | * @return void |
| 194 | 283 | */ |
| 195 | 284 | public static function prepare_description( &$action, $atts ) { |
| 196 | 285 | $description = $action->post_content['description']; |
| 197 | - if ( ! empty( $description ) ) { | |
| 198 | - $atts['value'] = $description; | |
| 199 | - $description = FrmTransLiteAppHelper::process_shortcodes( $atts ); | |
| 200 | - $action->post_content['description'] = $description; | |
| 286 | + | |
| 287 | + if ( ! $description ) { | |
| 288 | + return; | |
| 201 | 289 | } |
| 290 | + | |
| 291 | + $atts['value'] = $description; | |
| 292 | + $description = FrmTransLiteAppHelper::process_shortcodes( $atts ); | |
| 293 | + $action->post_content['description'] = $description; | |
| 202 | 294 | } |
| 203 | 295 | |
| 204 | 296 | /** |
| 205 | 297 | * Convert the amount into 10.00. |
| @@ -205,24 +297,25 @@ | ||
| 205 | 297 | * Convert the amount into 10.00. |
| 206 | 298 | * |
| 207 | 299 | * @param mixed $amount |
| 208 | 300 | * @param array $atts |
| 301 | + * | |
| 209 | 302 | * @return string |
| 210 | 303 | */ |
| 211 | 304 | public static function prepare_amount( $amount, $atts = array() ) { |
| 212 | 305 | if ( isset( $atts['form'] ) ) { |
| 213 | 306 | $atts['value'] = $amount; |
| 214 | - $amount = FrmTransLiteAppHelper::process_shortcodes( $atts ); | |
| 307 | + $amount = FrmTransLiteAppHelper::process_shortcodes( $atts ); | |
| 215 | 308 | } |
| 216 | 309 | |
| 217 | - if ( is_string( $amount ) && strlen( $amount ) >= 2 && $amount[0] == '[' && substr( $amount, -1 ) == ']' ) { | |
| 218 | - // make sure we don't use a field id as the amount | |
| 310 | + if ( is_string( $amount ) && strlen( $amount ) >= 2 && $amount[0] === '[' && str_ends_with( $amount, ']' ) ) { | |
| 311 | + // Make sure we don't use a field id as the amount. | |
| 219 | 312 | $amount = 0; |
| 220 | 313 | } |
| 221 | 314 | |
| 222 | 315 | $currency = self::get_currency_for_action( $atts ); |
| 316 | + $total = 0; | |
| 223 | 317 | |
| 224 | - $total = 0; | |
| 225 | 318 | foreach ( (array) $amount as $a ) { |
| 226 | 319 | $this_amount = self::get_amount_from_string( $a ); |
| 227 | 320 | self::maybe_use_decimal( $this_amount, $currency ); |
| 228 | 321 | self::normalize_number( $this_amount, $currency ); |
| @@ -237,12 +330,14 @@ | ||
| 237 | 330 | /** |
| 238 | 331 | * Get currency to use when preparing amount. |
| 239 | 332 | * |
| 240 | 333 | * @param array $atts |
| 334 | + * | |
| 241 | 335 | * @return array |
| 242 | 336 | */ |
| 243 | 337 | public static function get_currency_for_action( $atts ) { |
| 244 | 338 | $currency = 'usd'; |
| 339 | + | |
| 245 | 340 | if ( isset( $atts['form'] ) ) { |
| 246 | 341 | $currency = $atts['action']->post_content['currency']; |
| 247 | 342 | } elseif ( isset( $atts['currency'] ) ) { |
| 248 | 343 | $currency = $atts['currency']; |
| @@ -259,15 +354,15 @@ | ||
| 259 | 354 | private static function get_amount_from_string( $amount ) { |
| 260 | 355 | $amount = html_entity_decode( $amount ); |
| 261 | 356 | $amount = trim( $amount ); |
| 262 | 357 | preg_match_all( '/[0-9,.]*\.?\,?[0-9]+/', $amount, $matches ); |
| 263 | - $amount = $matches ? end( $matches[0] ) : 0; | |
| 264 | - return $amount; | |
| 358 | + return $matches ? end( $matches[0] ) : 0; | |
| 265 | 359 | } |
| 266 | 360 | |
| 267 | 361 | /** |
| 268 | 362 | * @param string $amount |
| 269 | 363 | * @param array $currency |
| 364 | + * | |
| 270 | 365 | * @return void |
| 271 | 366 | */ |
| 272 | 367 | private static function maybe_use_decimal( &$amount, $currency ) { |
| 273 | 368 | if ( $currency['thousand_separator'] !== '.' ) { |
| @@ -273,10 +368,17 @@ | ||
| 273 | 368 | if ( $currency['thousand_separator'] !== '.' ) { |
| 274 | 369 | return; |
| 275 | 370 | } |
| 276 | 371 | |
| 277 | - $amount_parts = explode( '.', $amount ); | |
| 278 | - $used_for_decimal = count( $amount_parts ) === 2 && strlen( $amount_parts[1] ) === 2; | |
| 372 | + $amount_parts = explode( '.', $amount ); | |
| 373 | + | |
| 374 | + if ( 2 !== count( $amount_parts ) ) { | |
| 375 | + return; | |
| 376 | + } | |
| 377 | + | |
| 378 | + $strlen = strlen( $amount_parts[1] ); | |
| 379 | + $used_for_decimal = $strlen === 1 || $strlen === 2; | |
| 380 | + | |
| 279 | 381 | if ( $used_for_decimal ) { |
| 280 | 382 | $amount = str_replace( '.', $currency['decimal_separator'], $amount ); |
| 281 | 383 | } |
| 282 | 384 | } |
| @@ -283,8 +385,9 @@ | ||
| 283 | 385 | |
| 284 | 386 | /** |
| 285 | 387 | * @param string $amount |
| 286 | 388 | * @param array $currency |
| 389 | + * | |
| 287 | 390 | * @return void |
| 288 | 391 | */ |
| 289 | 392 | private static function normalize_number( &$amount, $currency ) { |
| 290 | 393 | $amount = str_replace( $currency['thousand_separator'], '', $amount ); |
| @@ -295,22 +398,26 @@ | ||
| 295 | 398 | /** |
| 296 | 399 | * These settings are included in frm_stripe_vars.settings global JavaScript object on Stripe forms. |
| 297 | 400 | * |
| 298 | 401 | * @param int $form_id |
| 402 | + * | |
| 299 | 403 | * @return array |
| 300 | 404 | */ |
| 301 | 405 | public static function prepare_settings_for_js( $form_id ) { |
| 302 | 406 | $payment_actions = self::get_actions_for_form( $form_id ); |
| 303 | 407 | $action_settings = array(); |
| 408 | + | |
| 304 | 409 | foreach ( $payment_actions as $payment_action ) { |
| 305 | 410 | $settings_for_action = array( |
| 306 | 411 | 'id' => $payment_action->ID, |
| 307 | 412 | 'first_name' => $payment_action->post_content['billing_first_name'], |
| 308 | 413 | 'last_name' => $payment_action->post_content['billing_last_name'], |
| 414 | + 'address' => $payment_action->post_content['billing_address'] ?? '', | |
| 309 | 415 | 'gateways' => $payment_action->post_content['gateway'], |
| 310 | 416 | 'fields' => self::get_fields_for_price( $payment_action ), |
| 311 | 417 | 'one' => $payment_action->post_content['type'], |
| 312 | 418 | 'email' => $payment_action->post_content['email'], |
| 419 | + 'layout' => $payment_action->post_content['layout'] ?? '', | |
| 313 | 420 | ); |
| 314 | 421 | |
| 315 | 422 | /** |
| 316 | 423 | * @param array $settings_for_action |
| @@ -316,9 +423,9 @@ | ||
| 316 | 423 | * @param array $settings_for_action |
| 317 | 424 | * @param WP_Post $payment_action |
| 318 | 425 | */ |
| 319 | 426 | $settings_for_action = apply_filters( 'frm_trans_settings_for_js', $settings_for_action, $payment_action ); |
| 320 | - $action_settings[] = $settings_for_action; | |
| 427 | + $action_settings[] = $settings_for_action; | |
| 321 | 428 | } |
| 322 | 429 | |
| 323 | 430 | return $action_settings; |
| 324 | 431 | } |
| @@ -328,14 +435,15 @@ | ||
| 328 | 435 | * |
| 329 | 436 | * @since 6.5, introduced in v2.0 of the Payments submodule. |
| 330 | 437 | * |
| 331 | 438 | * @param WP_Post $action |
| 439 | + * | |
| 332 | 440 | * @return array|int |
| 333 | 441 | */ |
| 334 | 442 | private static function get_fields_for_price( $action ) { |
| 335 | 443 | $amount = $action->post_content['amount']; |
| 336 | 444 | $shortcodes = FrmFieldsHelper::get_shortcodes( $amount, $action->menu_order ); |
| 337 | - return isset( $shortcodes[2] ) ? $shortcodes[2] : -1; | |
| 445 | + return $shortcodes[2] ?? -1; | |
| 338 | 446 | } |
| 339 | 447 | |
| 340 | 448 | /** |
| 341 | 449 | * Get all published payment actions. |
| @@ -340,8 +448,9 @@ | ||
| 340 | 448 | /** |
| 341 | 449 | * Get all published payment actions. |
| 342 | 450 | * |
| 343 | 451 | * @param int|string $form_id |
| 452 | + * | |
| 344 | 453 | * @return array |
| 345 | 454 | */ |
| 346 | 455 | public static function get_actions_for_form( $form_id ) { |
| 347 | 456 | $action_status = array( |
| @@ -347,12 +456,10 @@ | ||
| 347 | 456 | $action_status = array( |
| 348 | 457 | 'post_status' => 'publish', |
| 349 | 458 | ); |
| 350 | 459 | $payment_actions = FrmFormAction::get_action_for_form( $form_id, 'payment', $action_status ); |
| 351 | - if ( ! $payment_actions ) { | |
| 352 | - $payment_actions = array(); | |
| 353 | - } | |
| 354 | - return $payment_actions; | |
| 460 | + | |
| 461 | + return $payment_actions ? $payment_actions : array(); | |
| 355 | 462 | } |
| 356 | 463 | |
| 357 | 464 | /** |
| 358 | 465 | * Make sure a gateway field is hidden on the front end. |
| @@ -358,8 +465,9 @@ | ||
| 358 | 465 | * Make sure a gateway field is hidden on the front end. |
| 359 | 466 | * |
| 360 | 467 | * @param array $values |
| 361 | 468 | * @param stdClass $field |
| 469 | + * | |
| 362 | 470 | * @return array |
| 363 | 471 | */ |
| 364 | 472 | public static function hide_gateway_field_on_front_end( $values, $field ) { |
| 365 | 473 | if ( $field->type !== 'gateway' ) { |
| @@ -365,12 +473,19 @@ | ||
| 365 | 473 | if ( $field->type !== 'gateway' ) { |
| 366 | 474 | return $values; |
| 367 | 475 | } |
| 368 | 476 | |
| 477 | + if ( FrmAppHelper::is_form_builder_page() ) { | |
| 478 | + // The hooks this uses can get called in the form builder and settings pages. | |
| 479 | + // But we do not need the script in this case. | |
| 480 | + return $values; | |
| 481 | + } | |
| 482 | + | |
| 369 | 483 | // This is also called from the frm_enqueue_form_scripts hook. |
| 370 | 484 | // With this here, the value of frm_stripe_vars.settings[0].fields is -1 |
| 371 | 485 | // This is because the amount value is processed and a shortcode is not found in '000'. |
| 372 | 486 | FrmStrpLiteActionsController::load_scripts( (int) $field->form_id ); |
| 487 | + FrmSquareLiteActionsController::load_scripts( (int) $field->form_id ); | |
| 373 | 488 | |
| 374 | 489 | $values['type'] = 'hidden'; |
| 375 | 490 | return $values; |
| 376 | 491 | } |
| @@ -381,14 +496,17 @@ | ||
| 381 | 496 | * @since 6.5.1 |
| 382 | 497 | * |
| 383 | 498 | * @param array $values |
| 384 | 499 | * @param stdClass $field |
| 500 | + * | |
| 385 | 501 | * @return array |
| 386 | 502 | */ |
| 387 | 503 | public static function fill_entry_from_previous( $values, $field ) { |
| 388 | 504 | global $frm_vars; |
| 389 | - $previous_entry = isset( $frm_vars['frm_trans']['pay_entry'] ) ? $frm_vars['frm_trans']['pay_entry'] : false; | |
| 390 | - if ( empty( $previous_entry ) || $previous_entry->form_id != $field->form_id ) { | |
| 505 | + $previous_entry = $frm_vars['frm_trans']['pay_entry'] ?? false; | |
| 506 | + | |
| 507 | + // phpcs:ignore Universal.Operators.StrictComparisons | |
| 508 | + if ( ! $previous_entry || $previous_entry->form_id != $field->form_id ) { | |
| 391 | 509 | return $values; |
| 392 | 510 | } |
| 393 | 511 | |
| 394 | 512 | if ( is_array( $previous_entry->metas ) && isset( $previous_entry->metas[ $field->id ] ) ) { |
| @@ -395,10 +513,9 @@ | ||
| 395 | 513 | $values['value'] = $previous_entry->metas[ $field->id ]; |
| 396 | 514 | } |
| 397 | 515 | |
| 398 | 516 | $frm_vars['trans_filled'] = true; |
| 399 | - | |
| 400 | - $previous_entry_id = $previous_entry->id; | |
| 517 | + $previous_entry_id = $previous_entry->id; | |
| 401 | 518 | self::destroy_entry_later( $previous_entry_id ); |
| 402 | 519 | |
| 403 | 520 | return $values; |
| 404 | 521 | } |
| @@ -409,9 +526,10 @@ | ||
| 409 | 526 | * See Formidable Stripe issue #136 for more information. |
| 410 | 527 | * |
| 411 | 528 | * @since 6.5.1 |
| 412 | 529 | * |
| 413 | - * @param string|int $entry_id | |
| 530 | + * @param int|string $entry_id | |
| 531 | + * | |
| 414 | 532 | * @return void |
| 415 | 533 | */ |
| 416 | 534 | private static function destroy_entry_later( $entry_id ) { |
| 417 | 535 | if ( in_array( (int) $entry_id, self::$entry_ids_to_destroy_later, true ) ) { |
| @@ -422,17 +540,161 @@ | ||
| 422 | 540 | $destroy_callback = |
| 423 | 541 | /** |
| 424 | 542 | * Destroy an entry and remove this action so it only tries to destroy the entry once. |
| 425 | 543 | * |
| 426 | - * @param string|int $entry_id | |
| 544 | + * @param int|string $entry_id | |
| 427 | 545 | * @param Closure $destroy_callback |
| 546 | + * | |
| 428 | 547 | * @return void |
| 429 | 548 | */ |
| 430 | - function() use ( $entry_id, &$destroy_callback ) { | |
| 549 | + function () use ( $entry_id, &$destroy_callback ) { | |
| 431 | 550 | FrmEntry::destroy( $entry_id ); |
| 432 | - remove_action( 'frm_entry_form', $destroy_callback ); // Only call this once. | |
| 551 | + // Only call this once. | |
| 552 | + remove_action( 'frm_entry_form', $destroy_callback ); | |
| 433 | 553 | }; |
| 434 | 554 | add_action( 'frm_entry_form', $destroy_callback ); |
| 435 | 555 | |
| 436 | 556 | self::$entry_ids_to_destroy_later[] = (int) $entry_id; |
| 557 | + } | |
| 558 | + | |
| 559 | + /** | |
| 560 | + * Filter payment action on save. | |
| 561 | + * | |
| 562 | + * @since 6.22 | |
| 563 | + * | |
| 564 | + * @param array $settings | |
| 565 | + * @param array $action | |
| 566 | + * | |
| 567 | + * @return array | |
| 568 | + */ | |
| 569 | + public static function before_save_settings( $settings, $action ) { | |
| 570 | + $settings['gateway'] = ! empty( $settings['gateway'] ) ? (array) $settings['gateway'] : array( 'stripe' ); | |
| 571 | + | |
| 572 | + if ( in_array( 'square', $settings['gateway'], true ) ) { | |
| 573 | + $currency = FrmSquareLiteConnectHelper::get_merchant_currency(); | |
| 574 | + $settings['currency'] = false !== $currency ? strtolower( $currency ) : 'usd'; | |
| 575 | + } else { | |
| 576 | + $settings['currency'] = strtolower( $settings['currency'] ); | |
| 577 | + } | |
| 578 | + | |
| 579 | + $form_id = absint( $action['menu_order'] ); | |
| 580 | + | |
| 581 | + if ( empty( $settings['credit_card'] ) ) { | |
| 582 | + $credit_card_field_id = FrmDb::get_var( | |
| 583 | + 'frm_fields', | |
| 584 | + array( | |
| 585 | + 'type' => 'credit_card', | |
| 586 | + 'form_id' => $form_id, | |
| 587 | + ) | |
| 588 | + ); | |
| 589 | + | |
| 590 | + if ( ! $credit_card_field_id ) { | |
| 591 | + $credit_card_field_id = self::add_a_credit_card_field( $form_id ); | |
| 592 | + } | |
| 593 | + | |
| 594 | + if ( $credit_card_field_id ) { | |
| 595 | + $settings['credit_card'] = $credit_card_field_id; | |
| 596 | + } | |
| 597 | + } | |
| 598 | + | |
| 599 | + if ( ! in_array( 'stripe', $settings['gateway'], true ) ) { | |
| 600 | + // We only need a gateway field for Stripe add-on compatibility, | |
| 601 | + // so unless Stripe is selected, we can return early. | |
| 602 | + return $settings; | |
| 603 | + } | |
| 604 | + | |
| 605 | + $gateway_field_id = FrmDb::get_var( | |
| 606 | + 'frm_fields', | |
| 607 | + array( | |
| 608 | + 'type' => 'gateway', | |
| 609 | + 'form_id' => $form_id, | |
| 610 | + ) | |
| 611 | + ); | |
| 612 | + | |
| 613 | + if ( ! $gateway_field_id ) { | |
| 614 | + self::add_a_gateway_field( $form_id ); | |
| 615 | + } | |
| 616 | + | |
| 617 | + return $settings; | |
| 618 | + } | |
| 619 | + | |
| 620 | + /** | |
| 621 | + * A credit card field is added automatically if missing before a Stripe action is updated. | |
| 622 | + * | |
| 623 | + * @param int $form_id | |
| 624 | + * | |
| 625 | + * @return false|int | |
| 626 | + */ | |
| 627 | + protected static function add_a_credit_card_field( $form_id ) { | |
| 628 | + return self::add_a_field( $form_id, 'credit_card', __( 'Payment', 'formidable' ) ); | |
| 629 | + } | |
| 630 | + | |
| 631 | + /** | |
| 632 | + * A gateway field is added automatically for compatibility with the Stripe add on. | |
| 633 | + * The gateway field is not important for the Stripe Lite implementation. | |
| 634 | + * | |
| 635 | + * @param int $form_id | |
| 636 | + * | |
| 637 | + * @return false|int | |
| 638 | + */ | |
| 639 | + protected static function add_a_gateway_field( $form_id ) { | |
| 640 | + return self::add_a_field( $form_id, 'gateway', __( 'Payment Method', 'formidable' ) ); | |
| 641 | + } | |
| 642 | + | |
| 643 | + /** | |
| 644 | + * @param int $form_id | |
| 645 | + * @param string $field_type | |
| 646 | + * @param string $field_name | |
| 647 | + * | |
| 648 | + * @return false|int | |
| 649 | + */ | |
| 650 | + protected static function add_a_field( $form_id, $field_type, $field_name ) { | |
| 651 | + $new_values = FrmFieldsHelper::setup_new_vars( $field_type, $form_id ); | |
| 652 | + $new_values['name'] = $field_name; | |
| 653 | + $new_values['field_order'] = self::get_field_order_before_submit( $form_id, $new_values['field_order'] ); | |
| 654 | + return FrmField::create( $new_values ); | |
| 655 | + } | |
| 656 | + | |
| 657 | + /** | |
| 658 | + * When auto-injecting a field, ensure it is placed before the submit button. | |
| 659 | + * | |
| 660 | + * @since 6.29 | |
| 661 | + * | |
| 662 | + * @param int $form_id | |
| 663 | + * @param int $field_order | |
| 664 | + * | |
| 665 | + * @return int | |
| 666 | + */ | |
| 667 | + private static function get_field_order_before_submit( $form_id, $field_order ) { | |
| 668 | + $submit_field = FrmSubmitHelper::get_submit_field( $form_id ); | |
| 669 | + | |
| 670 | + if ( ! $submit_field || $field_order < (int) $submit_field->field_order ) { | |
| 671 | + return $field_order; | |
| 672 | + } | |
| 673 | + | |
| 674 | + $submit_order = (int) $submit_field->field_order; | |
| 675 | + FrmField::update( $submit_field->id, array( 'field_order' => $submit_order + 1 ) ); | |
| 676 | + return $submit_order; | |
| 677 | + } | |
| 678 | + | |
| 679 | + /** | |
| 680 | + * Remove credit card validation errors. | |
| 681 | + * | |
| 682 | + * @param array $errors | |
| 683 | + * @param stdClass $field | |
| 684 | + * | |
| 685 | + * @return array | |
| 686 | + */ | |
| 687 | + public static function remove_cc_errors( $errors, $field ) { | |
| 688 | + $field_id = $field->temp_id ?? $field->id; | |
| 689 | + | |
| 690 | + if ( isset( $errors[ 'field' . $field_id . '-cc' ] ) ) { | |
| 691 | + unset( $errors[ 'field' . $field_id . '-cc' ] ); | |
| 692 | + } | |
| 693 | + | |
| 694 | + if ( isset( $errors[ 'field' . $field_id ] ) ) { | |
| 695 | + unset( $errors[ 'field' . $field_id ] ); | |
| 696 | + } | |
| 697 | + | |
| 698 | + return $errors; | |
| 437 | 699 | } |
| 438 | 700 | } |