| Index: trunk/extensions/DonationInterface/globalcollect_gateway/globalcollect_resultswitcher.body.php |
| — | — | @@ -72,7 +72,7 @@ |
| 73 | 73 | $oid = $wgRequest->getText( 'order_id' ); |
| 74 | 74 | |
| 75 | 75 | //this next block is for credit card coming back from GC. Only that. Nothing else, ever. |
| 76 | | - if ( $this->adapter->getData_Raw( 'payment_method') === 'cc' && $this->adapter->hasDonorDataInSession( 'order_id', $_GET['order_id'] ) ) { |
| | 76 | + if ( $this->adapter->getData_Raw( 'payment_method') === 'cc' ) { |
| 77 | 77 | if ( !array_key_exists( 'order_status', $_SESSION ) || !array_key_exists( $oid, $_SESSION['order_status'] ) ) { |
| 78 | 78 | $_SESSION['order_status'][$oid] = $this->adapter->do_transaction( 'Confirm_CreditCard' ); |
| 79 | 79 | $_SESSION['order_status'][$oid]['data']['count'] = 0; |
| — | — | @@ -98,10 +98,18 @@ |
| 99 | 99 | if ($go) { |
| 100 | 100 | $wgOut->addHTML( "<br>Redirecting to page $go" ); |
| 101 | 101 | $wgOut->redirect( $go ); |
| 102 | | - } //TODO: There really should be an else here. |
| | 102 | + } else { |
| | 103 | + $this->adapter->log("Resultswitcher: No redirect defined."); |
| | 104 | + } |
| | 105 | + } else { |
| | 106 | + $this->adapter->log("Resultswitcher: No TransactionWMFStatus."); |
| 103 | 107 | } |
| 104 | | - } |
| 105 | | - } |
| | 108 | + } else { |
| | 109 | + $this->adapter->log("Resultswitcher: Payment method is not cc."); |
| | 110 | + } |
| | 111 | + } else { |
| | 112 | + $this->adapter->log("Resultswitcher: Token Check Failed."); |
| | 113 | + } |
| 106 | 114 | } |
| 107 | 115 | |
| 108 | 116 | /** |