minor fix

civicrm · Jul 27, 2017 · ce8d1a4 · ce8d1a4
1 parent 4bd0e0a
commit ce8d1a4
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 10 deletions.
diff --git a/CRM/Contribute/Form/Contribution.php b/CRM/Contribute/Form/Contribution.php
@@ -701,6 +701,10 @@ public function buildQuickForm() {
     if ($this->_id) {
       $contributionStatus = CRM_Core_DAO::getFieldValue('CRM_Contribute_DAO_Contribution', $this->_id, 'contribution_status_id');
       $name = CRM_Utils_Array::value($contributionStatus, $statusName);
+
+      if ($name == 'Pending') {
+        $this->assign('isPending', TRUE);
+      }
       switch ($name) {
         case 'Completed':
           // [CRM-17498] Removing unsupported status change options.
@@ -1515,8 +1519,6 @@ protected function submit($submittedValues, $action, $pledgePaymentID) {
       foreach (array(
         'payment_instrument_id',
         'check_number',
-        // ****** @Monish - is this suppose to be trxn_date!?
-        'receipt_date',
       ) as $fieldName) {
         if (empty($submittedValues[$fieldName])) {
           $submittedValues[$fieldName] = CRM_Utils_Array::value($fieldName, $this->_values);

diff --git a/templates/CRM/Contribute/Form/Contribution.tpl b/templates/CRM/Contribute/Form/Contribution.tpl
@@ -184,12 +184,6 @@
         <td>{$form.contribution_status_id.html}
         {if $contribution_status_id eq 2}{if $is_pay_later }: {ts}Pay Later{/ts} {else}: {ts}Incomplete Transaction{/ts}{/if}{/if}
         </td>
-        <td>
-        {if $contactId && $contribID && $contributionMode EQ null && $contribution_status_id eq 2}
-          {capture assign=payNowLink}{crmURL p='civicrm/contact/view/contribution' q="reset=1&action=update&id=`$contribID`&cid=`$contactId`&context=`$context`&mode=live"}{/capture}
-          <a class="open-inline action-item crm-hover-button" href="{$payNowLink}">&raquo; {ts}Pay with Credit Card{/ts}</a>
-        {/if}
-      </td>
       </tr>
     {/if}
 
@@ -246,7 +240,7 @@
           {ts}Payment Details{/ts}
         </div>
         <div class="crm-accordion-body">
-        {if $contribID && $payments}
+        {if $contribID && ($payments || $isPending)}
           {include file="CRM/Contribute/Form/PaymentInfoBlock.tpl"}
         {else}
           <table class="form-layout-compressed" >

diff --git a/templates/CRM/Contribute/Form/PaymentInfoBlock.tpl b/templates/CRM/Contribute/Form/PaymentInfoBlock.tpl
@@ -53,6 +53,14 @@
    {else}
      {assign var='entity' value=$component}
    {/if}
-   {ts 1=$entity}No payments found for this %1 record{/ts}
+   <div class="odd-row">
+     {ts 1=$entity}&nbsp;&nbsp;&nbsp;No payments found for this %1 record.{/ts}
+     {if $isPending}
+       {capture assign=payNowLive}{crmURL p='civicrm/contact/view/contribution' q="reset=1&action=update&id=`$contribID`&cid=`$contactId`&context=`$context`&mode=live"}{/capture}
+       {capture assign=payNowOffline}{crmURL p='civicrm/payment' q="reset=1&id=`$contribID`&cid=`$contactId`&action=add&component=contribution"}{/capture}
+       <a class="open-inline action-item crm-hover-button" href="{$payNowLive}">&raquo; {ts}Pay with Credit Card{/ts}</a> OR
+       <a class="open-inline action-item crm-hover-button" href="{$payNowOffline}">&raquo; {ts}Pay Offline{/ts}</a>
+     {/if}
+   </div>
 {/if}
 {/crmRegion}