PHPcs file tidy up

diff --git a/CRM/Core/Page.php b/CRM/Core/Page.php
index 521cd824d9f21ac99e52c000cc6f03c40a1193ad..d35d792434ad44e9e3ba1aa40432a038bd2068a5 100644 (file)
--- 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 c2352b1870157f4d65c60e6789608950eb20cb28..fb97b7de0936048a810ee5207df52cb280c0d0b9 100644 (file)
--- 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();
   }
 }