Merge pull request #5145 from agh1/case-insensitive-headers

[civicrm-core.git] / CRM / Core / BAO / Job.php

diff --git a/CRM/Core/BAO/Job.php b/CRM/Core/BAO/Job.php
index 016139474612fe740be6bb4ecf9c7497657c2691..db5e6d7278ee5fd1207cbef03778a9b6cbdfaf09 100644 (file)
--- a/CRM/Core/BAO/Job.php
+++ b/CRM/Core/BAO/Job.php
@@ -23,7 +23,7 @@
  | GNU Affero General Public License or the licensing of CiviCRM,     |
  | see the CiviCRM license FAQ at http://civicrm.org/licensing        |
  +--------------------------------------------------------------------+
-*/
+ */
 
 /**
  *
@@ -39,7 +39,7 @@
 class CRM_Core_BAO_Job extends CRM_Core_DAO_Job {
 
   /**
-   * Class constructor
+   * Class constructor.
    */
   public function __construct() {
     parent::__construct();
@@ -48,10 +48,10 @@ class CRM_Core_BAO_Job extends CRM_Core_DAO_Job {
   /**
    * Add the payment-processor type in the db
    *
-   * @param array $params an assoc array of name/value pairs
+   * @param array $params
+   *   An assoc array of name/value pairs.
    *
    * @return CRM_Financial_DAO_PaymentProcessorType
-   * @static
    */
   public static function create($params) {
     $job = new CRM_Core_DAO_Job();
@@ -60,15 +60,17 @@ class CRM_Core_BAO_Job extends CRM_Core_DAO_Job {
   }
 
   /**
-   * Takes a bunch of params that are needed to match certain criteria and
-   * retrieves the relevant objects. It also stores all the retrieved
-   * values in the default array
+   * Retrieve DB object based on input parameters.
+   *
+   * It also stores all the retrieved values in the default array.
    *
-   * @param array $params   (reference ) an assoc array of name/value pairs
-   * @param array $defaults (reference ) an assoc array to hold the flattened values
+   * @param array $params
+   *   (reference ) an assoc array of name/value pairs.
+   * @param array $defaults
+   *   (reference ) an assoc array to hold the flattened values.
    *
-   * @return CRM_Core_DAO_Job object on success, null otherwise
-   * @static
+   * @return CRM_Core_DAO_Job|null
+   *   object on success, null otherwise
    */
   public static function retrieve(&$params, &$defaults) {
     $job = new CRM_Core_DAO_Job();
@@ -81,26 +83,28 @@ class CRM_Core_BAO_Job extends CRM_Core_DAO_Job {
   }
 
   /**
-   * Update the is_active flag in the db
+   * Update the is_active flag in the db.
    *
-   * @param int      $id        id of the database record
-   * @param boolean  $is_active value we want to set the is_active field
+   * @param int $id
+   *   Id of the database record.
+   * @param bool $is_active
+   *   Value we want to set the is_active field.
    *
-   * @return Object             DAO object on sucess, null otherwise
+   * @return Object
+   *   DAO object on sucess, null otherwise
    *
-   * @static
    */
   public static function setIsActive($id, $is_active) {
     return CRM_Core_DAO::setFieldValue('CRM_Core_DAO_Job', $id, 'is_active', $is_active);
   }
 
   /**
-   * Function  to delete scheduled job
+   * Function  to delete scheduled job.
    *
-   * @param $jobID ID of the job to be deleted.
+   * @param $jobID
+   *   ID of the job to be deleted.
    *
    * @return bool|null
-   * @static
    */
   public static function del($jobID) {
     if (!$jobID) {
@@ -119,7 +123,7 @@ class CRM_Core_BAO_Job extends CRM_Core_DAO_Job {
   }
 
   /**
-   * Trim job table on a regular basis to keep it at a good size
+   * Trim job table on a regular basis to keep it at a good size.
    *
    * CRM-10513
    */
@@ -129,7 +133,7 @@ class CRM_Core_BAO_Job extends CRM_Core_DAO_Job {
     $query = 'SELECT COUNT(*) FROM civicrm_job_log';
     $count = CRM_Core_DAO::singleValueQuery($query);
 
-    if ( $count <= $maxEntriesToKeep) {
+    if ($count <= $maxEntriesToKeep) {
       return;
     }