Skip to content

Commit e89a873

Browse files
mordethvladolaru
andauthored
Update the progressive parameter in the KYC session creation API to use a boolean type (#10632)
Co-authored-by: Vlad Olaru <[email protected]>
1 parent 81f78d3 commit e89a873

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
Significance: patch
2+
Type: dev
3+
4+
Updated the progressive parameter in the KYC session creation API to use a boolean type.

includes/admin/class-wc-rest-payments-onboarding-controller.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,9 @@ public function register_routes() {
5959
'progressive' => [
6060
'required' => false,
6161
'description' => 'Whether the session is for progressive onboarding.',
62-
'type' => 'string',
62+
// phpcs:ignore Squiz.PHP.CommentedOutCode.Found
63+
// We expect a boolean (true, false, 0, 1, '0', '1', 'true', or 'false'), but will also accept `yes`/`no`.
64+
'type' => [ 'boolean', 'string' ],
6365
],
6466
'self_assessment' => [
6567
'required' => false,
@@ -228,7 +230,7 @@ public function register_routes() {
228230
*/
229231
public function get_embedded_kyc_session( WP_REST_Request $request ) {
230232
$self_assessment_data = ! empty( $request->get_param( 'self_assessment' ) ) ? wc_clean( wp_unslash( $request->get_param( 'self_assessment' ) ) ) : [];
231-
$progressive = ! empty( $request->get_param( 'progressive' ) ) && 'true' === $request->get_param( 'progressive' );
233+
$progressive = ! empty( $request->get_param( 'progressive' ) ) && filter_var( $request->get_param( 'progressive' ), FILTER_VALIDATE_BOOLEAN );
232234

233235
$account_session = $this->onboarding_service->create_embedded_kyc_session(
234236
$self_assessment_data,

0 commit comments

Comments
 (0)