Merge pull request #4917 from eileenmcnaughton/minor-tidies

[civicrm-core.git] / CRM / Event / Page / Tab.php

diff --git a/CRM/Event/Page/Tab.php b/CRM/Event/Page/Tab.php
index cdec1928521da6f80751e7f3c87fdf06881b8f6c..1cc1f3f996248fffe6b3550efc5362f24ebb78e5 100644 (file)
--- a/CRM/Event/Page/Tab.php
+++ b/CRM/Event/Page/Tab.php
@@ -38,12 +38,11 @@ class CRM_Event_Page_Tab extends CRM_Core_Page {
   public $_contactId = NULL;
 
   /**
-   * This function is called when action is browse
+   * called when action is browse
    *
-   * return null
-   * @access public
+   * @return null
    */
-  function browse() {
+  public function browse() {
     $controller = new CRM_Core_Controller_Simple(
       'CRM_Event_Form_Search',
       ts('Events'),
@@ -71,12 +70,11 @@ class CRM_Event_Page_Tab extends CRM_Core_Page {
   }
 
   /**
-   * This function is called when action is view
+   * called when action is view
    *
-   * return null
-   * @access public
+   * @return null
    */
-  function view() {
+  public function view() {
     // build associated contributions
     $this->associatedContribution();
 
@@ -93,12 +91,11 @@ class CRM_Event_Page_Tab extends CRM_Core_Page {
   }
 
   /**
-   * This function is called when action is update or new
+   * called when action is update or new
    *
-   * return null
-   * @access public
+   * @return null
    */
-  function edit() {
+  public function edit() {
     // set https for offline cc transaction
     $mode = CRM_Utils_Request::retrieve('mode', 'String', $this);
     if ($mode == 'test' || $mode == 'live') {
@@ -123,10 +120,10 @@ class CRM_Event_Page_Tab extends CRM_Core_Page {
     return $controller->run();
   }
 
-  function preProcess() {
-    $context       = CRM_Utils_Request::retrieve('context', 'String', $this);
+  public function preProcess() {
+    $context = CRM_Utils_Request::retrieve('context', 'String', $this);
     $this->_action = CRM_Utils_Request::retrieve('action', 'String', $this, FALSE, 'browse');
-    $this->_id     = CRM_Utils_Request::retrieve('id', 'Positive', $this);
+    $this->_id = CRM_Utils_Request::retrieve('id', 'Positive', $this);
 
     if ($context == 'standalone') {
       $this->_action = CRM_Core_Action::ADD;
@@ -149,12 +146,11 @@ class CRM_Event_Page_Tab extends CRM_Core_Page {
   }
 
   /**
-   * This function is the main function that is called when the page loads, it decides the which action has to be taken for the page.
+   * the main function that is called when the page loads, it decides the which action has to be taken for the page.
    *
-   * return null
-   * @access public
+   * @return null
    */
-  function run() {
+  public function run() {
     $this->preProcess();
 
     // check if we can process credit card registration
@@ -176,7 +172,8 @@ class CRM_Event_Page_Tab extends CRM_Core_Page {
     elseif ($this->_action & (CRM_Core_Action::UPDATE |
         CRM_Core_Action::ADD |
         CRM_Core_Action::DELETE
-      )) {
+      )
+    ) {
       $this->edit();
     }
     else {
@@ -186,7 +183,7 @@ class CRM_Event_Page_Tab extends CRM_Core_Page {
     return parent::run();
   }
 
-  function setContext() {
+  public function setContext() {
     $context = CRM_Utils_Request::retrieve('context',
       'String', $this, FALSE, 'search'
     );
@@ -218,7 +215,7 @@ class CRM_Event_Page_Tab extends CRM_Core_Page {
         if ($compContext == 'advanced') {
           $url = CRM_Utils_System::url('civicrm/contact/search/advanced', $urlParams);
         }
-        else if ($searchContext) {
+        elseif ($searchContext) {
           $url = CRM_Utils_System::url("civicrm/$searchContext/search", $urlParams);
         }
         else {
@@ -251,7 +248,7 @@ class CRM_Event_Page_Tab extends CRM_Core_Page {
         break;
 
       case 'fulltext':
-        $keyName   = '&qfKey';
+        $keyName = '&qfKey';
         $urlParams = 'force=1';
         $urlString = 'civicrm/contact/search/custom';
         if ($this->_action == CRM_Core_Action::UPDATE) {
@@ -284,13 +281,12 @@ class CRM_Event_Page_Tab extends CRM_Core_Page {
   }
 
   /**
-   * This function is used for the to show the associated
+   * used for the to show the associated
    * contribution for the participant
    *
-   * return null
-   * @access public
+   * @return null
    */
-  function associatedContribution() {
+  public function associatedContribution() {
     if (CRM_Core_Permission::access('CiviContribute')) {
       $this->assign('accessContribution', TRUE);
       $controller = new CRM_Core_Controller_Simple(
@@ -312,4 +308,3 @@ class CRM_Event_Page_Tab extends CRM_Core_Page {
     }
   }
 }
-