r78730 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r78729‎ | r78730 | r78731 >
Date:20:16, 21 December 2010
Author:awjrichards
Status:deferred
Tags:
Comment:
Fixed mis-named variable in wmf_premiums_get_shipping_info() and added watchdog debug statement to dump shipping values
Modified paths:
  • /civicrm/trunk/sites/all/modules/wmf_premiums/wmf_premiums.module (modified) (history)

Diff [purge]

Index: civicrm/trunk/sites/all/modules/wmf_premiums/wmf_premiums.module
@@ -91,7 +91,7 @@
9292 * @return array of shipping information
9393 */
9494 function wmf_premiums_get_shipping_address( $trxn_info ) {
95 - $shipping_info = array();
 95+ $shipping = array();
9696
9797 // if we have a secondary address, use that for shipping
9898 if ( strlen( trim( $trxn_info[ 'first_name_2' ] )) && strlen( trim( $trxn_info[ 'last_name_2' ]))) {
@@ -114,7 +114,9 @@
115115 $shipping[ 'country' ] = $trxn_info[ 'country' ];
116116 }
117117
118 - return $shipping_info;
 118+ watchdog( 'wmf_premiums', 'Setting shipping info for premiums info to: %shipping', array( '%shipping', print_r( $shipping, true )), WATCHDOG_DEBUG );
 119+
 120+ return $shipping;
119121 }
120122
121123 function wmf_premiums_test_insert() {

Status & tagging log