Remove isset from batch form

Affects url civicrm/financial/financialbatches?reset=1&batchStatus=1

diff --git a/CRM/Financial/Form/BatchTransaction.php b/CRM/Financial/Form/BatchTransaction.php
index 5a52547bd6f4512e37a504e0a0aaa41b3c176b6d..e93585cb5669a5273981db82a0d105329c2980af 100644 (file)
--- a/CRM/Financial/Form/BatchTransaction.php
+++ b/CRM/Financial/Form/BatchTransaction.php
@@ -30,14 +30,14 @@ class CRM_Financial_Form_BatchTransaction extends CRM_Contribute_Form_Search {
 
   /**
    * Batch status name.
+   *
    * @var string
    */
-  protected $_batchStatus;
+  protected $_batchStatus = 'open';
 
   public function preProcess() {
     // This reuses some styles from search forms
     CRM_Core_Resources::singleton()->addStyleFile('civicrm', 'css/searchForm.css', 1, 'html-header');
-
     self::$_entityID = CRM_Utils_Request::retrieve('bid', 'Positive') ? CRM_Utils_Request::retrieve('bid', 'Positive') : CRM_Utils_Array::value('batch_id', $_POST);
     $this->assign('entityID', self::$_entityID);
     if (isset(self::$_entityID)) {
@@ -45,7 +45,6 @@ class CRM_Financial_Form_BatchTransaction extends CRM_Contribute_Form_Search {
       $batchStatuses = CRM_Core_PseudoConstant::get('CRM_Batch_DAO_Batch', 'status_id', ['labelColumn' => 'name', 'condition' => " v.value={$this->_batchStatusId}"]);
       $this->_batchStatus = $batchStatuses[$this->_batchStatusId];
       $this->assign('statusID', $this->_batchStatusId);
-      $this->assign('batchStatus', $this->_batchStatus);
       $validStatus = FALSE;
       if (in_array($this->_batchStatus, ['Open', 'Reopened'])) {
         $validStatus = TRUE;
@@ -68,6 +67,7 @@ class CRM_Financial_Form_BatchTransaction extends CRM_Contribute_Form_Search {
       ];
       $this->assign('columnHeaders', $columnHeaders);
     }
+    $this->assign('batchStatus', $this->_batchStatus);
   }
 
   /**

diff --git a/templates/CRM/Financial/Form/Search.tpl b/templates/CRM/Financial/Form/Search.tpl
index 5eb3c91a629977abd16ffbd88d31a1f0ccc55b3c..53301668cf952f5168fd55c40180d7ce79e5f2d5 100644 (file)
--- a/templates/CRM/Financial/Form/Search.tpl
+++ b/templates/CRM/Financial/Form/Search.tpl
@@ -9,9 +9,6 @@
 *}
 
 {* Financial search component. *}
-{if !isset($batchStatus)}
-  {assign var="batchStatus" value="open"}
-{/if}
 <div id="enableDisableStatusMsg" class="crm-container" style="display:none"></div>
 <div class="action-link">
   <a accesskey="N" href="{crmURL p='civicrm/financial/batch' q="reset=1&action=add&context=$batchStatus"}" id="newBatch" class="button"><span><i class="crm-i fa-plus-circle" aria-hidden="true"></i> {ts}New Accounting Batch{/ts}</span></a>