Merge pull request #5597 from pratikshad/CRM-16008
[civicrm-core.git] / CRM / Event / BAO / Event.php
index 48d98b49318bb5bc749d6fdf9cf063d15f3b5433..e87dd2d235cced9f64bd44cc351c101290496cd3 100644 (file)
@@ -3,7 +3,7 @@
  +--------------------------------------------------------------------+
  | CiviCRM version 4.6                                                |
  +--------------------------------------------------------------------+
- | Copyright CiviCRM LLC (c) 2004-2014                                |
+ | Copyright CiviCRM LLC (c) 2004-2015                                |
  +--------------------------------------------------------------------+
  | This file is a part of CiviCRM.                                    |
  |                                                                    |
  | GNU Affero General Public License or the licensing of CiviCRM,     |
  | see the CiviCRM license FAQ at http://civicrm.org/licensing        |
  +--------------------------------------------------------------------+
-*/
+ */
 
 /**
  *
  * @package CRM
- * @copyright CiviCRM LLC (c) 2004-2014
+ * @copyright CiviCRM LLC (c) 2004-2015
  * $Id$
  *
  */
 class CRM_Event_BAO_Event extends CRM_Event_DAO_Event {
 
   /**
-   * Class constructor
+   * Class constructor.
    */
   public function __construct() {
     parent::__construct();
   }
 
   /**
-   * Fetch object based on array of properties
+   * Fetch object based on array of properties.
    *
    * @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_Event_BAO_ManageEvent object
-   * @static
+   * @return CRM_Event_BAO_ManageEvent
    */
   public static function retrieve(&$params, &$defaults) {
     $event = new CRM_Event_DAO_Event();
@@ -63,7 +62,7 @@ class CRM_Event_BAO_Event extends CRM_Event_DAO_Event {
   }
 
   /**
-   * Update the is_active flag in the db
+   * Update the is_active flag in the db.
    *
    * @param int $id
    *   Id of the database record.
@@ -72,19 +71,17 @@ class CRM_Event_BAO_Event extends CRM_Event_DAO_Event {
    *
    * @return Object
    *   DAO object on sucess, null otherwise
-   * @static
    */
   public static function setIsActive($id, $is_active) {
     return CRM_Core_DAO::setFieldValue('CRM_Event_DAO_Event', $id, 'is_active', $is_active);
   }
 
   /**
-   * Add the event
+   * Add the event.
    *
    * @param array $params
    *   Reference array contains the values submitted by the form.
    *
-   * @static
    *
    * @return object
    */
@@ -119,13 +116,12 @@ class CRM_Event_BAO_Event extends CRM_Event_DAO_Event {
   }
 
   /**
-   * Create the event
+   * Create the event.
    *
    * @param array $params
    *   Reference array contains the values submitted by the form.
    *
    * @return object
-   * @static
    */
   public static function create(&$params) {
     $transaction = new CRM_Core_Transaction();
@@ -177,13 +173,12 @@ class CRM_Event_BAO_Event extends CRM_Event_DAO_Event {
   }
 
   /**
-   * Delete the event
+   * Delete the event.
    *
    * @param int $id
    *   Event id.
    *
    * @return mixed|null
-   * @static
    */
   public static function del($id) {
     if (!$id) {
@@ -235,7 +230,6 @@ class CRM_Event_BAO_Event extends CRM_Event_DAO_Event {
    * @param int $eventId
    *   Event with which loc block is associated.
    *
-   * @static
    */
   public static function deleteEventLocBlock($locBlockId, $eventId = NULL) {
     $query = "SELECT count(ce.id) FROM civicrm_event ce WHERE ce.loc_block_id = $locBlockId";
@@ -265,9 +259,8 @@ class CRM_Event_BAO_Event extends CRM_Event_DAO_Event {
    *   true if you need to check permission else false.
    *
    * @return array
-   * @static
    */
-  static function getEvents(
+  public static function getEvents(
     $all = 0,
     $id = FALSE,
     $isActive = TRUE,
@@ -318,9 +311,8 @@ WHERE  ( civicrm_event.is_template IS NULL OR civicrm_event.is_template = 0 )";
   }
 
   /**
-   * Get events Summary
+   * Get events Summary.
    *
-   * @static
    *
    * @return array
    *   Array of event summary values
@@ -584,16 +576,16 @@ $event_summary_limit
   }
 
   /**
-   * Get participant count
+   * Get participant count.
    *
    * @param int $eventId
-   * @param bool $considerStatusConsider status for participant count.
+   * @param bool $considerStatus consider status for participant count.
    *   Consider status for participant count.
-   * @param bool $statusConsider counted participant.
+   * @param bool $status counted participant.
    *   Consider counted participant.
-   * @param bool $considerRoleConsider role for participant count.
+   * @param bool $considerRole consider role for participant count.
    *   Consider role for participant count.
-   * @param bool $roleConsider counted( is filter role) participant.
+   * @param bool $role consider counted( is filter role) participant.
    *   Consider counted( is filter role) participant.
    *
    *
@@ -650,14 +642,13 @@ $event_summary_limit
   }
 
   /**
-   * Get the information to map a event
+   * Get the information to map a event.
    *
    * @param int $id
    *   For which we want map info.
    *
    * @return null|string
    *   title of the event
-   * @static
    */
   public static function &getMapInfo(&$id) {
 
@@ -718,24 +709,23 @@ WHERE civicrm_address.geo_code_1 IS NOT NULL
   }
 
   /**
-   * Get the complete information for one or more events
+   * Get the complete information for one or more events.
    *
    * @param date $start
    *   Get events with start date >= this date.
-   * @param int $typeGet events on the a specific event type (by event_type_id).
+   * @param int $type Get events on the a specific event type (by event_type_id).
    *   Get events on the a specific event type (by event_type_id).
-   * @param int $eventIdReturn a single event - by event id.
+   * @param int $eventId Return a single event - by event id.
    *   Return a single event - by event id.
    * @param date $end
    *   Also get events with end date >= this date.
-   * @param bool $onlyPublicInclude public events only, default TRUE.
+   * @param bool $onlyPublic Include public events only, default TRUE.
    *   Include public events only, default TRUE.
    *
    * @return array
    *   array of all the events that are searched
-   * @static
    */
-  static function &getCompleteInfo(
+  public static function &getCompleteInfo(
     $start = NULL,
     $type = NULL,
     $eventId = NULL,
@@ -915,16 +905,15 @@ WHERE civicrm_event.is_active = 1
    *
    * @param int $id
    *   The event id to copy.
-   *        obj     $newEvent    object of CRM_Event_DAO_Event
    *        boolean $afterCreate call to copy after the create function
    * @param null $newEvent
    * @param bool $afterCreate
    *
-   * @return void
+   * @return CRM_Event_DAO_Event
    */
   public static function copy($id, $newEvent = NULL, $afterCreate = FALSE) {
 
-    $defaults = $eventValues = array();
+    $eventValues = array();
 
     //get the require event values.
     $eventParams = array('id' => $id);
@@ -955,7 +944,7 @@ WHERE civicrm_event.is_active = 1
         array('id' => $id),
         array(
           'loc_block_id' =>
-            ($locBlockId) ? $locBlockId : NULL,
+          ($locBlockId) ? $locBlockId : NULL,
         ),
         $fieldsFix
       );
@@ -1180,7 +1169,7 @@ WHERE civicrm_event.is_active = 1
           $tplParams['participant_status'] = CRM_Event_PseudoConstant::participantStatus($statusId, NULL, 'label');
         }
         //CRM-15754 - if participant_status contains status ID
-        elseif (CRM_Utils_Rule::integer($tplParams['participant_status'])) {
+        elseif (!empty($tplParams['participant_status']) && CRM_Utils_Rule::integer($tplParams['participant_status'])) {
           $tplParams['participant_status'] = CRM_Event_PseudoConstant::participantStatus($tplParams['participant_status'], NULL, 'label');
         }
 
@@ -1271,7 +1260,7 @@ WHERE civicrm_event.is_active = 1
    *
    * @return void
    */
-  static function buildCustomDisplay(
+  public static function buildCustomDisplay(
     $id,
     $name,
     $cid,
@@ -1442,7 +1431,7 @@ WHERE civicrm_event.is_active = 1
   }
 
   /**
-   * Build the array for display the profile fields
+   * Build the array for display the profile fields.
    *
    * @param array $params
    *   Key value.
@@ -1456,7 +1445,6 @@ WHERE civicrm_event.is_active = 1
    * @param array $profileFields
    *
    * @return void
-   * @static
    */
   public static function displayProfile(&$params, $gid, &$groupTitle, &$values, &$profileFields = array()) {
     if ($gid) {
@@ -1736,7 +1724,7 @@ WHERE  id = $cfID
    * @return array
    *   array of Additional participant's info OR array of Ids.
    */
-  static function buildCustomProfile(
+  public static function buildCustomProfile(
     $participantId,
     $values,
     $contactId = NULL,
@@ -1893,11 +1881,11 @@ WHERE  ce.loc_block_id = $locBlockId";
   }
 
   /**
-   * Check if event registration is valid according to permissions AND Dates
+   * Check if event registration is valid according to permissions AND Dates.
    *
    * @param array $values
    * @param int $eventID
-   * @return boolean
+   * @return bool
    */
   public static function validRegistrationRequest($values, $eventID) {
     // check that the user has permission to register for this event
@@ -2004,14 +1992,13 @@ WHERE  ce.loc_block_id = $locBlockId";
   }
 
   /**
-   * Make sure that the user has permission to access this event
+   * Make sure that the user has permission to access this event.
    *
    * @param int $eventId
    * @param int $type
    *
    * @return string
    *   the permission that the user has (or null)
-   * @static
    */
   public static function checkPermission($eventId = NULL, $type = CRM_Core_Permission::VIEW) {
     static $permissions = NULL;
@@ -2076,7 +2063,6 @@ WHERE  ce.loc_block_id = $locBlockId";
    *
    * @return array
    *   an array of email ids
-   * @static
    */
   public static function getFromEmailIds($eventId = NULL) {
     $fromEmailValues['from_email_id'] = CRM_Core_BAO_Email::getFromEmail();
@@ -2113,7 +2099,6 @@ WHERE  ce.loc_block_id = $locBlockId";
    *
    * @return int
    *   event total seats w/ given criteria.
-   * @static
    */
   public static function eventTotalSeats($eventId, $extraWhereClause = NULL) {
     if (empty($eventId)) {
@@ -2152,14 +2137,14 @@ LEFT  JOIN  civicrm_price_field_value value ON ( value.id = lineItem.price_field
   }
 
   /**
-   * Retrieve event template default values to be set
+   * Retrieve event template default values to be set.
    *  as default values for current new event.
    *
    * @param int $templateId
    *   Event template id.
    *
    * @return array
-   *   of custom data defaults.
+   *   Array of custom data defaults.
    */
   public static function getTemplateDefaultValues($templateId) {
     $defaults = array();
@@ -2194,7 +2179,7 @@ LEFT  JOIN  civicrm_price_field_value value ON ( value.id = lineItem.price_field
   }
 
   /**
-   * Update the Campaign Id of all the participants of the given event
+   * Update the Campaign Id of all the participants of the given event.
    *
    * @param int $eventID
    *   Event id.
@@ -2233,7 +2218,7 @@ LEFT  JOIN  civicrm_price_field_value value ON ( value.id = lineItem.price_field
         // Fixme - this is going to ignore context, better to get conditions, add params, and call PseudoConstant::get
         return CRM_Financial_BAO_FinancialType::getIncomeFinancialType();
 
-        break;
+      break;
     }
     return CRM_Core_PseudoConstant::get(__CLASS__, $fieldName, $params, $context);
   }