From 46116924dca1705dac10b89fb5f6d1df4eb9254b Mon Sep 17 00:00:00 2001
From: Eileen McNaughton <eileen@fuzion.co.nz>
Date: Thu, 15 Jan 2015 14:13:53 +1300
Subject: [PATCH] PHPcs file tidy up

---
 CRM/Core/Page.php        |  4 ----
 CRM/Mailing/Page/Tab.php | 21 ++++++---------------
 2 files changed, 6 insertions(+), 19 deletions(-)

diff --git a/CRM/Core/Page.php b/CRM/Core/Page.php
index 521cd824d9..d35d792434 100644
--- a/CRM/Core/Page.php
+++ b/CRM/Core/Page.php
@@ -164,9 +164,6 @@ class CRM_Core_Page {
    * This function takes care of all the things common to all
    * pages. This typically involves assigning the appropriate
    * smarty variable :)
-   *
-   * @return string
-   *   The content generated by running this page
    */
   public function run() {
     if ($this->_embedded) {
@@ -242,7 +239,6 @@ class CRM_Core_Page {
     CRM_Utils_Hook::alterContent($content, 'page', $pageTemplateFile, $this);
 
     echo CRM_Utils_System::theme($content, $this->_print);
-    return;
   }
 
   /**
diff --git a/CRM/Mailing/Page/Tab.php b/CRM/Mailing/Page/Tab.php
index c2352b1870..fb97b7de09 100644
--- a/CRM/Mailing/Page/Tab.php
+++ b/CRM/Mailing/Page/Tab.php
@@ -30,12 +30,10 @@
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2014
  * $Id$
- *
  */
 
 /**
  * This class handle mailing and contact related functions
- *
  */
 class CRM_Mailing_Page_Tab extends CRM_Contact_Page_View {
   /**
@@ -48,17 +46,13 @@ class CRM_Mailing_Page_Tab extends CRM_Contact_Page_View {
   public $_contactId = NULL;
 
   /**
-   * called when action is browse
-   *
-   * @return null
+   * Called when action is browse.
    */
   public function browse() {
   }
 
   /**
-   * Build all the data structures needed to build the form
-   *
-   * @return void
+   * Build all the data structures needed to build the form.
    */
   public function preProcess() {
     $this->_action = CRM_Utils_Request::retrieve('action', 'String', $this, FALSE, 'browse');
@@ -70,23 +64,20 @@ class CRM_Mailing_Page_Tab extends CRM_Contact_Page_View {
     $this->assign('contactId', $this->_contactId);
     $this->assign('displayName', $displayName);
 
-    // check logged in url permission
+    // Check logged in url permission.
     CRM_Contact_Page_View::checkUserPermission($this);
 
-    // set page title
     CRM_Utils_System::setTitle(ts('Mailings sent to %1', array(1 => $displayName)));
   }
 
   /**
-   * 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.
    *
-   * @return null
+   * It decides the which action has to be taken for the page.
    */
   public function run() {
     $this->preProcess();
     $this->browse();
-
-    return parent::run();
+    parent::run();
   }
 }
-- 
2.25.1