#17 unset gateway before storing session

eileenmcnaughton · Feb 5, 2017 · 9338069 · 9338069 · magnolia61 · Feb 5, 2017
1 parent 91bd74d
commit 9338069
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/CRM/Core/Payment/OmnipayMultiProcessor.php b/CRM/Core/Payment/OmnipayMultiProcessor.php
@@ -154,8 +154,12 @@ public function doDirectPayment(&$params, $component = 'contribute') {
         return $params;
       }
       elseif ($response->isRedirect()) {
+        $isTransparentRedirect = ($response->isTransparentRedirect() || !empty($this->gateway->transparentRedirect));
+        // Unset $this->gateway before storing session to cache due to risk of
+        // Serialization of 'Closure' is not allowed error - issue #17
+        $this->gateway = NULL;
         CRM_Core_Session::storeSessionObjects(FALSE);
-        if ($response->isTransparentRedirect() || !empty($this->gateway->transparentRedirect)) {
+        if ($$isTransparentRedirect) {
           $this->storeTransparentRedirectFormData($params['qfKey'], $response->getRedirectData() + array(
             'payment_processor_id' => $this->_paymentProcessor['id'],
             'post_submit_url' => $response->getRedirectURL(),