Merge pull request #17310 from agh1/autorenew-active-icon

[civicrm-core.git] / CRM / Member / Form / Search.php

diff --git a/CRM/Member/Form/Search.php b/CRM/Member/Form/Search.php
index 0ae19be96a7963a51a4d91ff342d9c60434b77f5..b8bb22b8be266074238cb9c78a92cda5163d8cb1 100644 (file)
--- a/CRM/Member/Form/Search.php
+++ b/CRM/Member/Form/Search.php
@@ -177,6 +177,22 @@ class CRM_Member_Form_Search extends CRM_Core_Form_Search {
     return ts('Member Contact Type');
   }
 
+  /**
+   * Set defaults.
+   *
+   * @return array
+   * @throws \Exception
+   */
+  public function setDefaultValues() {
+    $this->_defaults = parent::setDefaultValues();
+    //LCD also allow restrictions to membership owner via GET
+    $owner = CRM_Utils_Request::retrieve('owner', 'String');
+    if (in_array($owner, ['0', '1'])) {
+      $this->_defaults['member_is_primary'] = $owner;
+    }
+    return $this->_defaults;
+  }
+
   /**
    * The post processing of the form gets done here.
    *
@@ -248,8 +264,6 @@ class CRM_Member_Form_Search extends CRM_Core_Form_Search {
       $prefix
     );
     $controller->setEmbedded(TRUE);
-
-    $query = &$selector->getQuery();
     $controller->run();
   }
 
@@ -265,7 +279,8 @@ class CRM_Member_Form_Search extends CRM_Core_Form_Search {
       return;
     }
 
-    $status = CRM_Utils_Request::retrieve('status', 'String');
+    // @todo Most of the  below is likely no longer required.
+    $status = CRM_Utils_Request::retrieve('membership_status_id', 'String');
     if ($status) {
       $status = explode(',', $status);
       $this->_formValues['membership_status_id'] = $this->_defaults['membership_status_id'] = (array) $status;
@@ -319,12 +334,6 @@ class CRM_Member_Form_Search extends CRM_Core_Form_Search {
     $this->_limit = CRM_Utils_Request::retrieve('limit', 'Positive',
       $this
     );
-
-    //LCD also allow restrictions to membership owner via GET
-    $owner = CRM_Utils_Request::retrieve('owner', 'String');
-    if (in_array($owner, ['0', '1'])) {
-      $this->_formValues['member_is_primary'] = $this->_defaults['member_is_primary'] = $owner;
-    }
   }
 
   /**