From 9fff8cd80c9f8cb7d4b15d0df01e592fce41c2b8 Mon Sep 17 00:00:00 2001
From: Eileen McNaughton <emcnaughton@wikimedia.org>
Date: Sat, 15 Jul 2023 14:01:48 +1200
Subject: [PATCH] Fold preProcess into run

there are properties that appear to be being set solely to pass variables from preProcess to
run.

preProcess is called from run and that is the only way it is accessed.

There are some forms that use the pageRun hook in the universe but this has no impact on them
---
 CRM/Contact/Page/View/CustomData.php | 17 +++++------------
 1 file changed, 5 insertions(+), 12 deletions(-)

diff --git a/CRM/Contact/Page/View/CustomData.php b/CRM/Contact/Page/View/CustomData.php
index 8d8a5e9fef..dd82e852a1 100644
--- a/CRM/Contact/Page/View/CustomData.php
+++ b/CRM/Contact/Page/View/CustomData.php
@@ -28,9 +28,12 @@ class CRM_Contact_Page_View_CustomData extends CRM_Core_Page {
   public $_groupId;
 
   /**
-   * Add a few specific things to view contact.
+   * Run the page.
+   *
+   * This method is called after the page is created. It checks for the
+   * type of action and executes that action.
    */
-  public function preProcess() {
+  public function run() {
     $this->_groupId = CRM_Utils_Request::retrieve('gid', 'Positive', $this, TRUE);
     $this->assign('groupId', $this->_groupId);
 
@@ -58,16 +61,6 @@ class CRM_Contact_Page_View_CustomData extends CRM_Core_Page {
 
     $this->_multiRecordDisplay = CRM_Utils_Request::retrieve('multiRecordDisplay', 'String', $this, FALSE);
     $this->_cgcount = CRM_Utils_Request::retrieve('cgcount', 'Positive', $this, FALSE);
-  }
-
-  /**
-   * Run the page.
-   *
-   * This method is called after the page is created. It checks for the
-   * type of action and executes that action.
-   */
-  public function run() {
-    $this->preProcess();
 
     //set the userContext stack
     $doneURL = 'civicrm/contact/view';
-- 
2.25.1