Changeset 6718

Timestamp:

02/22/2018 08:48:12 PM (7 years ago)

Author:

coreymckrill

Message:

WordCamp Payments: Specify charges paid by remitter for all wire exports.

Wires in other currencies besides USD default to "shared" charges when they are
imported, so the payments team frequently has to go back and change it to
"remitter" after importing the transactions. This saves time by defaulting to
"remitter" in the first place.

Location:

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

Files:

• payment-request.php (modified) (2 diffs)
• reimbursement-request.php (modified) (2 diffs)

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

@@ -1554 +1554 @@
                 '112-priority' => '',
                 '113-blank' => '',
-                '114-charges' => '',
+                '114-charges' => 'OUR', // Charges should always be paid by the remitter, instead of shared.
                 '115-blank' => '',
                 '116-details' => '',
@@ -1582 +1582 @@
             }
 
-            if ( get_post_meta( $post->ID, '_camppayments_currency', true ) == 'CAD' ) {
-                $row['114-charges'] = 'OUR';
-            }
-
             // Use for debugging.
             // print_r( $row );

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

@@ -1506 +1506 @@
             '112-priority' => '',
             '113-blank' => '',
-            '114-charges' => '',
+            '114-charges' => 'OUR', // Charges should always be paid by the remitter, instead of shared.
             '115-blank' => '',
             '116-details' => '',
@@ -1534 +1534 @@
         }
 
-        if ( get_post_meta( $post->ID, '_wcbrr_currency', true ) == 'CAD' ) {
-            $row['114-charges'] = 'OUR';
-        }
-
         // Use for debugging.
         // print_r( $row );