From 3b2bf3b5627c6147f7ff97c9932dd00f31cb5b9d Mon Sep 17 00:00:00 2001
From: eileenmcnaugton <eileen@fuzion.co.nz>
Date: Sat, 6 Aug 2016 20:12:36 +1200
Subject: [PATCH] jenkins take that!

---
 CRM/Mailing/Page/View.php | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/CRM/Mailing/Page/View.php b/CRM/Mailing/Page/View.php
index 84805f2f57..7fb43dd984 100644
--- a/CRM/Mailing/Page/View.php
+++ b/CRM/Mailing/Page/View.php
@@ -78,6 +78,7 @@ class CRM_Mailing_Page_View extends CRM_Core_Page {
    * @param bool $allowID
    *
    * @return null|string
+   *   Not really sure if anything should be returned - parent doesn't
    */
   public function run($id = NULL, $contactID = NULL, $print = TRUE, $allowID = FALSE) {
     if (is_numeric($id)) {
@@ -115,7 +116,7 @@ class CRM_Mailing_Page_View extends CRM_Core_Page {
           !$allowID
         ) {
           CRM_Utils_System::permissionDenied();
-          return;
+          return NULL;
         }
       }
     }
@@ -128,7 +129,7 @@ class CRM_Mailing_Page_View extends CRM_Core_Page {
       !$this->checkPermission()
     ) {
       CRM_Utils_System::permissionDenied();
-      return;
+      return NULL;
     }
 
     CRM_Mailing_BAO_Mailing::tokenReplace($this->_mailing);
-- 
2.25.1