CiviCRM Community Forums (archive)

I wanted to prevent requiring users to enter their address info twice, once on our Contribution form, and then again once we send them off to Paypal Standard.

To achieve this, I made the modification included below to CRM/Core/Payment/PayPalImpl.php starting with the version from 2.0.1.

If you feel this is worth submitting in JIRA, please let me know if there's a standard on what diffs should look like--I didn't see one offhand.

Maarten.


Index: PayPalImpl.php
===================================================================
--- PayPalImpl.php (revision 189)
+++ PayPalImpl.php (working copy)
@@ -299,7 +299,33 @@
                    'return'             => $returnURL,
                    'rm'                 => 2,
                    'currency_code'      => $params['currencyID'],
-                   'invoice'            => $params['invoiceID'] );
+                   'invoice'            => $params['invoiceID'],
+            //!mvd mod to propagate contact info to Paypal starts here
+                   'first_name'         => $params['first_name'],
+                   'last_name'         => $params['last_name'] );
+
+        // CiviCRM can store multiple addresses, so we need to be agnostic as to what address
+        // for the contact was entered on the form. (field names are of the form street_address-Primary,
+        // street_address-Billing)
+        $field_names = array( 'street_address' => 'address1',
+        'city' => 'city',
+        'state_province' => 'state',
+        'postal_code' => 'zip',
+                                'email' => 'email' );
+       
+        foreach ( array_keys( $params ) as $p ) {
+          $parts = split( '-', $p );
+          if ( count( $parts ) > 1 && in_array( $parts[0], array_keys( $field_names ) ) ) {
+            // there was a hyphen, and the bit before it is one of the vars we're looking for
+           
+            if ( 'state_province' == $parts[0] ) {
+              $paypalParams[$field_names[$parts[0]]] =
+                CRM_Core_PseudoConstant::stateProvinceAbbreviation( $params[$p] );
+            } else {
+              $paypalParams[$field_names[$parts[0]]] = $params[$p];
+            }
+          }
+        }
 
         // if recurring donations, add a few more items
         if ( ! empty( $params['is_recur'] ) ) {


CRM-3130

I filed against 2.0 since that's what the patch applies to.

This is my first JIRA entry--let me know if I should have done things differently.