lineitem.diff

diff --git a/CRM/Contribute/Form/Contribution/Confirm.php b/CRM/Contribute/Form/Contribution/Confirm.php
index 85574cca95..e8ca131364 100644
--- a/CRM/Contribute/Form/Contribution/Confirm.php
+++ b/CRM/Contribute/Form/Contribution/Confirm.php
@@ -1778,6 +1778,12 @@ class CRM_Contribute_Form_Contribution_Confirm extends CRM_Contribute_Form_Contr
     );
 
     $result = array();
+    if ($tempParams['skipLineItem']) {
+      // We are not processing the line item here because we are processing a membership.
+      // Do not continue with contribution processing in this function.
+      return array($membershipContribution, $result);
+    }
+
     if ($form->_values['is_monetary'] && !$form->_params['is_pay_later'] && $minimumFee > 0.0) {
       // At the moment our tests are calling this form in a way that leaves 'object' empty. For
       // now we compensate here.

Hi Jitendra. FYI, in the case described in https://issues.civicrm.org/jira/browse/CRM-20172, this patch has the effect of sending only one payment request (in the amount of the non-membership contribution) to Authorize.net, and leaving the membership portion of the contribution unsent.