Changeset 6106

Timestamp:

11/10/2017 02:17:53 AM (7 years ago)

Author:

iandunn

Message:

WordCamp Payments: Add warning/@todo about problematic $back_compat_statuses.

Location:

sites/trunk/wordcamp.org/public_html/wp-content/plugins

Files:

• wordcamp-payments-network/includes/payment-requests-dashboard.php (modified) (1 diff)
• wordcamp-payments-network/includes/reimbursement-requests-dashboard.php (modified) (1 diff)
• wordcamp-payments-network/includes/wordcamp-budgets-dashboard.php (modified) (1 diff)
• wordcamp-payments/includes/payment-request.php (modified) (3 diffs)
• wordcamp-payments/includes/reimbursement-request.php (modified) (2 diffs)

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

@@ -139 +139 @@
         }
 
+        // Warning/@todo: See note in WCP_Payment_Request::render_status_metabox()
         $back_compat_statuses = array(
             'unpaid' => 'draft',

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

@@ -203 +203 @@
 
     // Back-compat.
+    // Warning/@todo: See note in WCP_Payment_Request::render_status_metabox()
     $back_compat_statuses = array(
         'wcbrr_submitted' => 'pending-approval',

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

@@ -16 +16 @@
 add_action( 'admin_init', __NAMESPACE__ . '\process_action_approve', 11 );
 add_action( 'admin_init', __NAMESPACE__ . '\process_action_set_pending_payment', 11 );
-
 add_action( 'admin_init', __NAMESPACE__ . '\process_import_request', 11 );
 

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/includes/payment-request.php

@@ -207 +207 @@
         wp_nonce_field( 'status', 'status_nonce' );
 
+        /*
+         * @todo Remove $back_compat_statuses everywhere.
+         *
+         * They add a lot of fragility because they're not DRY and the data isn't consistent and
+         * predictable. They appear in many of the modules in both the wordcamp-payments and
+         * wordcamp-payments-network plugins.
+         *
+         * It seems like it'd be much better to just run a script over all sites and convert the
+         * data to the new statuses. Need to investigate why that wasn't done to begin with, in
+         * case there are some problems that make this more complicated than it appears.
+         */
         $back_compat_statuses = array(
             'unpaid' => 'draft',
@@ -526 +537 @@
 
         // Back-compat
+        // Warning/@todo: See note in render_status_metabox()
         $back_compat_statuses = array(
             'unpaid' => 'draft',
@@ -963 +975 @@
             $post = get_post( $entry->post_id );
 
+            // Warning/@todo: See note in render_status_metabox()
             $back_compat_statuses = array(
                 'unpaid' => 'draft',

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wordcamp-payments/includes/reimbursement-request.php

@@ -266 +266 @@
     wp_nonce_field( 'status', 'status_nonce' );
 
+    // Warning/@todo: See note in WCP_Payment_Request::render_status_metabox()
     $back_compat_statuses = array(
         'wcbrr_submitted'      => 'wcb-pending-approval',
@@ -867 +868 @@
         $post = get_post( $entry->request_id );
 
+        // Warning/@todo: See note in WCP_Payment_Request::render_status_metabox()
         $back_compat_statuses = array(
             'wcbrr_submitted'      => 'wcb-pending-approval',