https://www.mediawiki.org/wiki/Special:Code/MediaWiki/103503

Revision: 103503
Author:   khorn
Date:     2011-11-17 20:23:15 +0000 (Thu, 17 Nov 2011)
Log Message:
-----------
GlobalCollect command-line orphan rectifier: Cleaning up some weird-o 
formatting. 

Modified Paths:
--------------
    
trunk/extensions/DonationInterface/globalcollect_gateway/scripts/orphan_adapter.php

Modified: 
trunk/extensions/DonationInterface/globalcollect_gateway/scripts/orphan_adapter.php
===================================================================
--- 
trunk/extensions/DonationInterface/globalcollect_gateway/scripts/orphan_adapter.php
 2011-11-17 20:21:54 UTC (rev 103502)
+++ 
trunk/extensions/DonationInterface/globalcollect_gateway/scripts/orphan_adapter.php
 2011-11-17 20:23:15 UTC (rev 103503)
@@ -1,18 +1,18 @@
 <?php
 
 class GlobalCollectOrphanAdapter extends GlobalCollectAdapter {
-       
+
        //Data we know to be good, that we always want to re-assert after a 
load or an addData. 
        //so far: order_id, i_order_id, and the utm data we pull from 
contribution tracking. 
-       protected $hard_data = array();
+       protected $hard_data = array( );
 
-       public function unstage_data( $data = array(), $final = true ){
-               $unstaged = array();
-               foreach ( $data as $key=>$val ){
-                       if (is_array($val)){
+       public function unstage_data( $data = array( ), $final = true ) {
+               $unstaged = array( );
+               foreach ( $data as $key => $val ) {
+                       if ( is_array( $val ) ) {
                                $unstaged += $this->unstage_data( $val, false );
                        } else {
-                               if (array_key_exists($key, $this->var_map)){
+                               if ( array_key_exists( $key, $this->var_map ) ) 
{
                                        //run the unstage data functions. 
                                        $unstaged[$this->var_map[$key]] = $val;
                                        //this would be EXTREMELY bad to put in 
the regular adapter. 
@@ -22,36 +22,35 @@
                                }
                        }
                }
-               if ($final){
-                       $this->stageData('response');
+               if ( $final ) {
+                       $this->stageData( 'response' );
                }
-               foreach ($unstaged as $key=>$val){
+               foreach ( $unstaged as $key => $val ) {
                        $unstaged[$key] = $this->staged_data[$key];
                }
                return $unstaged;
        }
-       
-       public function loadDataAndReInit( $data ){
+
+       public function loadDataAndReInit( $data ) {
                $this->batch = true; //or the hooks will accumulate badness. 
-               
                //re-init all these arrays, because this is a batch thing. 
-               $this->hard_data = array();
-               $this->transaction_results = array();
-               $this->raw_data = array();
-               $this->staged_data = array();
-               
+               $this->hard_data = array( );
+               $this->transaction_results = array( );
+               $this->raw_data = array( );
+               $this->staged_data = array( );
+
                $this->hard_data['order_id'] = $data['order_id'];
                $this->hard_data['i_order_id'] = $data['order_id'];
-               
+
                $this->dataObj = new DonationData( get_called_class(), false, 
$data );
 
                $this->raw_data = $this->dataObj->getData();
-               
+
                $this->hard_data = array_merge( $this->hard_data, 
$this->getUTMInfoFromDB() );
                $this->reAddHardData();
-               
+
                $this->staged_data = $this->raw_data;
-               
+
                $this->setPostDefaults();
                $this->defineTransactions();
                $this->defineErrorMap();
@@ -61,39 +60,39 @@
                $this->defineReturnValueMap();
 
                $this->stageData();
-               
+
                //have to do this again here. 
                $this->reAddHardData();
        }
-       
-       public function addData($dataArray){
-               parent::addData($dataArray);
+
+       public function addData( $dataArray ) {
+               parent::addData( $dataArray );
                $this->reAddHardData();
        }
-       
-       private function reAddHardData(){
+
+       private function reAddHardData() {
                //anywhere else, and this would constitute abuse of the system.
                //so don't do it. 
-               foreach ($this->hard_data as $key => $val){
+               foreach ( $this->hard_data as $key => $val ) {
                        $this->raw_data[$key] = $val;
                        $this->staged_data[$key] = $val;
                }
        }
-       
-       public function do_transaction($transaction){
-               switch ($transaction){
+
+       public function do_transaction( $transaction ) {
+               switch ( $transaction ) {
                        case 'SET_PAYMENT':
                        case 'CANCEL_PAYMENT':
-                               
self::log($this->getData_Raw('contribution_tracking_id') . ": CVV: " . 
$this->getData_Raw('cvv_result') . ": AVS: " . 
$this->getData_Raw('avs_result'));
-                               //and then go on, unless you're testing, in 
which case:
+                               self::log( $this->getData_Raw( 
'contribution_tracking_id' ) . ": CVV: " . $this->getData_Raw( 'cvv_result' ) . 
": AVS: " . $this->getData_Raw( 'avs_result' ) );
+                       //and then go on, unless you're testing, in which case:
 //                             return "NOPE";
 //                             break;
                        default:
-                               return parent::do_transaction($transaction);
+                               return parent::do_transaction( $transaction );
                                break;
                }
        }
-       
+
        public static function log( $msg, $log_level = LOG_INFO, $nothing = 
null ) {
                $identifier = 'orphans:' . self::getIdentifier() . 
"_gateway_trxn";
 
@@ -108,57 +107,57 @@
                syslog( $log_level, $msg );
                closelog();
        }
-       
-       public function getUTMInfoFromDB(){
 
+       public function getUTMInfoFromDB() {
+
                $db = 
ContributionTrackingProcessor::contributionTrackingConnection();
 
                if ( !$db ) {
-                       die("There is something terribly wrong with your 
Contribution Tracking database. fixit.");
+                       die( "There is something terribly wrong with your 
Contribution Tracking database. fixit." );
                        return null;
                }
-               
-               $ctid = $this->getData_Raw('contribution_tracking_id');
-               
-               $data = array();
 
+               $ctid = $this->getData_Raw( 'contribution_tracking_id' );
+
+               $data = array( );
+
                // if contrib tracking id is not already set, we need to insert 
the data, otherwise update                      
                if ( $ctid ) {
-                       $res = $db->select( 'contribution_tracking',
-             array(
-                 'utm_source',
-                                'utm_campaign',
-                                'utm_medium',
-                                'ts'
-             ),
-             array('id' => $ctid)
+                       $res = $db->select(
+                               'contribution_tracking', 
+                               array(
+                                       'utm_source',
+                                       'utm_campaign',
+                                       'utm_medium',
+                                       'ts'
+                               ), 
+                               array( 'id' => $ctid )
                        );
-                       foreach ($res as $thing){
+                       foreach ( $res as $thing ) {
                                $data['utm_source'] = $thing->utm_source;
                                $data['utm_campaign'] = $thing->utm_campaign;
                                $data['utm_medium'] = $thing->utm_medium;
                                $data['ts'] = $thing->ts;
                                $msg = '';
-                               foreach ($data as $key => $val){
+                               foreach ( $data as $key => $val ) {
                                        $msg .= "$key = $val ";
                                }
-                               $this->log("$ctid: Found UTM Data. $msg");
+                               $this->log( "$ctid: Found UTM Data. $msg" );
                                echo $msg;
                                return $data;
                        }
                }
-               
+
                //if we got here, we can't find anything else...
-               $this->log("$ctid: FAILED to find UTM Source value. Using 
default.");
+               $this->log( "$ctid: FAILED to find UTM Source value. Using 
default." );
                return $data;
        }
-       
-       
+
        /**
         * Copying this here because it's the fastest way to bring in an actual 
timestamp. 
         */
        protected function doStompTransaction() {
-               if ( !$this->getGlobal( 'EnableStomp' ) ){
+               if ( !$this->getGlobal( 'EnableStomp' ) ) {
                        return;
                }
                $this->debugarray[] = "Attempting Stomp Transaction!";
@@ -178,10 +177,10 @@
                        $this->debugarray[] = "No Stomp Hook Found for 
WMF_Status $status";
                        return;
                }
-               
-               
-               if (!is_null($this->getData_Raw('ts'))){
-                       $timestamp = strtotime($this->getData_Raw('ts')); //I 
hate that this works.
+
+
+               if ( !is_null( $this->getData_Raw( 'ts' ) ) ) {
+                       $timestamp = strtotime( $this->getData_Raw( 'ts' ) ); 
//I hate that this works.
                } else {
                        $timestamp = time();
                }
@@ -198,8 +197,8 @@
                try {
                        wfRunHooks( $hook, array( $transaction ) );
                } catch ( Exception $e ) {
-                       self::log( "STOMP ERROR. Could not add message. " . 
$e->getMessage() , LOG_CRIT );
+                       self::log( "STOMP ERROR. Could not add message. " . 
$e->getMessage(), LOG_CRIT );
                }
        }
-       
+
 }
\ No newline at end of file


_______________________________________________
MediaWiki-CVS mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to