Merge pull request #36 from boxuk/PEAK-1246-registration-marketing-pr…

…eferences [J#PEAK-1246] Fixes user consent POST field on save_user_consent_on_registration
boxuk · May 23, 2019 · 2c95ff3 · 2c95ff3
2 parents fb9aa40 + e9c8e79
commit 2c95ff3
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/includes/class-gdpr.php b/includes/class-gdpr.php
@@ -332,15 +332,19 @@ public static function save_user_consent_on_registration( $user_id ) {
 		GDPR_Audit_Log::log( $user_id, esc_html__( 'User registered to the site.', 'gdpr' ) );
 
 		// phpcs:disable WordPressVIPMinimum.VIP.PHPFilterFunctions.RestrictedFilter
-		$user_consents = filter_input( INPUT_POST, 'shipping_method', FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
+		$user_consents = filter_input( INPUT_POST, 'user_consents', FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
 		// phpcs:enable WordPressVIPMinimum.VIP.PHPFilterFunctions.RestrictedFilter
 		if ( is_array( $user_consents ) ) {
 
 			$consents = array_map( 'sanitize_text_field', array_keys( wp_unslash( $user_consents ) ) );
 			foreach ( $consents as $consent ) {
 				/* translators: Name of consent */
 				GDPR_Audit_Log::log( $user_id, sprintf( esc_html__( 'User gave explicit consent to %s', 'gdpr' ), $consent ) );
-				add_user_meta( $user_id, 'gdpr_consents', $consent );
+				if ( defined( 'WPCOM_IS_VIP_ENV' ) && WPCOM_IS_VIP_ENV ) {
+					add_user_attribute( $user_id, 'gdpr_consents', $consent );
+				} else {
+					add_user_meta( $user_id, 'gdpr_consents', $consent );
+				}
 			}
 			// This only happens on a POST request and should have no impact on caching.
 			// phpcs:disable WordPressVIPMinimum.VIP.RestrictedFunctions.cookies_setcookie