Changeset 8329

Timestamp:

02/22/2019 08:29:44 PM (7 years ago)

Author:

iandunn

Message:

WordCamp Budgets Dashboard: Set missing values to false to avoid notice.

The previous code would create an empty array, and then try to reference index 0, which wouldn't exist.

File:

• sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments-network/includes/sponsor-invoices-dashboard.php (modified) (1 diff)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments-network/includes/sponsor-invoices-dashboard.php

@@ -512 +512 @@
         $last_step_time      = $invoice_sent_at;
         $wordcamp_post       = get_wordcamp_post();
-        $wordcamp_start_date = ( $wordcamp_post->meta['Start Date (YYYY-mm-dd)'] ?? array() )[0];
-        $wordcamp_lead_email = ( $wordcamp_post->meta['Email Address'] ?? array() )[0];
-
-        if ( empty( $wordcamp_post ) ) {
+        $wordcamp_start_date = $wordcamp_post->meta['Start Date (YYYY-mm-dd)'][0] ?? false;
+        $wordcamp_lead_email = $wordcamp_post->meta['Email Address'][0]           ?? false;
+
+        if ( empty( $wordcamp_post ) || ! $wordcamp_lead_email ) {
             // Maybe this is a central.wordcamp.org test sponsor invoice.
             restore_current_blog();