Context Navigation

← Previous Changeset
Next Changeset →

Changeset 6505

Timestamp:

02/01/2018 07:14:47 PM (8 years ago)

Author:

iandunn

Message:

WordCamp QBO Client: Handle case where VAT number is empty.

The VAT field is optional in the Sponsor post type, so assuming it exists in the meta data results in a PHP warning. Later parts of the code assume that the field exists in the payload, so set it to an empty value if a number was not given.

File:

: 1 edited

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-qbo-client/wordcamp-qbo-client.php (modified) (2 diffs)

Legend:

: Unmodified
: Added
: Removed

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-qbo-client/wordcamp-qbo-client.php

-              r5589
+              r6505
                                 'email-address' => is_email(            $sponsor_meta['_wcpt_sponsor_email_address'][0] ),
                                 'phone-number'  => sanitize_text_field( $sponsor_meta['_wcpt_sponsor_phone_number' ][0] ),
-                                'vat-number'    => sanitize_text_field( $sponsor_meta['_wcpt_sponsor_vat_number'   ][0] ),
                                 'address1' => sanitize_text_field( $sponsor_meta['_wcpt_sponsor_street_address1'][0] ),
 …
+                        )
                 );
+                $payload['sponsor']['vat-number'] = isset( $sponsor_meta['_wcpt_sponsor_vat_number'][0] )
+                        ? sanitize_text_field( $sponsor_meta['_wcpt_sponsor_vat_number'][0] )
+                        : '';
                 if ( isset( $sponsor_meta['_wcpt_sponsor_street_address2'][0] ) ) {

Note: See TracChangeset for help on using the changeset viewer.

Making WordPress.org

Context Navigation

Changeset 6505

Legend:

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-qbo-client/wordcamp-qbo-client.php

Download in other formats: