From af4c09bc2d31925bc5e5ca914c1ceeca4a3ab0ff Mon Sep 17 00:00:00 2001
From: Eileen McNaughton <eileen@fuzion.co.nz>
Date: Mon, 28 Apr 2014 09:43:08 -0700
Subject: [PATCH] fix function to be static, comment fixes

---
 CRM/Mailing/BAO/Mailing.php    |  5 ++++-
 CRM/Mailing/BAO/MailingJob.php | 11 ++++++++++-
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/CRM/Mailing/BAO/Mailing.php b/CRM/Mailing/BAO/Mailing.php
index 53b8dc5bf8..021a051216 100644
--- a/CRM/Mailing/BAO/Mailing.php
+++ b/CRM/Mailing/BAO/Mailing.php
@@ -1478,6 +1478,9 @@ ORDER BY   civicrm_email.is_bulkmail DESC
    *
    * @params array $params        Form values
    *
+   * @param $params
+   * @param array $ids
+   *
    * @return object $mailing      The new mailing object
    * @access public
    * @static
@@ -2848,4 +2851,4 @@ AND        m.id = %1
     $params['version'] = 3;
     return civicrm_api('MailingContact', 'getcount', $params);
   }
-}
\ No newline at end of file
+}
diff --git a/CRM/Mailing/BAO/MailingJob.php b/CRM/Mailing/BAO/MailingJob.php
index a5dc66c23f..276073673f 100644
--- a/CRM/Mailing/BAO/MailingJob.php
+++ b/CRM/Mailing/BAO/MailingJob.php
@@ -44,7 +44,12 @@ class CRM_Mailing_BAO_MailingJob extends CRM_Mailing_DAO_MailingJob {
     parent::__construct();
   }
 
-  function create ($params){
+  /**
+   * @param $params
+   *
+   * @return CRM_Mailing_BAO_MailingJob
+   */
+  static function create ($params){
     $job = new CRM_Mailing_BAO_MailingJob();
     $job->mailing_id = $params['mailing_id'];
     $job->status = $params['status'];
@@ -55,9 +60,13 @@ class CRM_Mailing_BAO_MailingJob extends CRM_Mailing_DAO_MailingJob {
     $eq = $mailing->getRecipients($job->id, $params['mailing_id'], NULL, NULL, true, false);
     return $job;
   }
+
   /**
    * Initiate all pending/ready jobs
    *
+   * @param null $testParams
+   * @param null $mode
+   *
    * @return void
    * @access public
    * @static
-- 
2.25.1