Merge pull request #4948 from pratikshad/code-cleanup
[civicrm-core.git] / CRM / Mailing / Page / Common.php
index 9b861563be53a2eac1ad316f5066f5b3c137eb33..419ada767c5ffa2ae260f49900fac8e416ba98ac 100644 (file)
@@ -39,10 +39,10 @@ class CRM_Mailing_Page_Common extends CRM_Core_Page {
    * @return string
    * @throws Exception
    */
-  function run() {
-    $job_id   = CRM_Utils_Request::retrieve('jid', 'Integer', CRM_Core_DAO::$_nullObject);
+  public function run() {
+    $job_id = CRM_Utils_Request::retrieve('jid', 'Integer', CRM_Core_DAO::$_nullObject);
     $queue_id = CRM_Utils_Request::retrieve('qid', 'Integer', CRM_Core_DAO::$_nullObject);
-    $hash     = CRM_Utils_Request::retrieve('h', 'String', CRM_Core_DAO::$_nullObject);
+    $hash = CRM_Utils_Request::retrieve('h', 'String', CRM_Core_DAO::$_nullObject);
 
     if (!$job_id ||
       !$queue_id ||
@@ -51,7 +51,6 @@ class CRM_Mailing_Page_Common extends CRM_Core_Page {
       CRM_Core_Error::fatal(ts("Missing input parameters"));
     }
 
-
     // verify that the three numbers above match
     $q = CRM_Mailing_Event_BAO_Queue::verify($job_id, $queue_id, $hash);
     if (!$q) {
@@ -126,4 +125,3 @@ class CRM_Mailing_Page_Common extends CRM_Core_Page {
     return parent::run();
   }
 }
-