comment fixes
authoreileenmcnaughton <eileen@fuzion.co.nz>
Thu, 22 Oct 2015 01:51:56 +0000 (01:51 +0000)
committereileenmcnaughton <eileen@fuzion.co.nz>
Thu, 22 Oct 2015 03:44:32 +0000 (03:44 +0000)
wq!
comment fixes

f

45 files changed:
CRM/Admin/Form/PaymentProcessor.php
CRM/Case/ManagedEntities.php
CRM/Contact/BAO/Query.php
CRM/Contact/Form/Search/Interface.php
CRM/Core/BAO/Setting.php
CRM/Core/Form.php
CRM/Core/Module.php
CRM/Core/Selector/API.php
CRM/Event/BAO/Event.php
CRM/Event/BAO/Participant.php
CRM/Event/BAO/Query.php
CRM/Event/Controller/Search.php
CRM/Event/Form/EventFees.php
CRM/Event/Form/ManageEvent.php
CRM/Event/Form/ManageEvent/Conference.php
CRM/Event/Form/ManageEvent/Delete.php
CRM/Event/Form/ManageEvent/EventInfo.php
CRM/Event/Form/ManageEvent/Fee.php
CRM/Event/Form/ManageEvent/Location.php
CRM/Event/Form/ManageEvent/Registration.php
CRM/Event/Form/Registration.php
CRM/Event/Form/Registration/Confirm.php
CRM/Event/Form/Registration/Register.php
CRM/Import/DataSource/CSV.php
CRM/Pledge/Controller/Search.php
CRM/Report/Form/Contact/Summary.php
CRM/SMS/Provider.php
CRM/UF/Page/Group.php
CRM/Upgrade/Incremental/Base.php
CRM/Upgrade/Incremental/php/FourFive.php
CRM/Upgrade/Incremental/php/FourFour.php
CRM/Upgrade/Incremental/php/FourOne.php
CRM/Upgrade/Incremental/php/FourSeven.php
CRM/Upgrade/Incremental/php/FourSix.php
CRM/Upgrade/Incremental/php/FourThree.php
CRM/Upgrade/Incremental/php/FourTwo.php
CRM/Upgrade/Snapshot/V4p2/Price/BAO/LineItem.php
CRM/Upgrade/Snapshot/V4p2/Price/BAO/Set.php
CRM/Upgrade/Snapshot/V4p2/Price/DAO/Field.php
CRM/Upgrade/Snapshot/V4p2/Price/DAO/FieldValue.php
CRM/Utils/Network.php
Civi/API/Event/Event.php
api/v3/Generic.php
api/v3/Generic/Getlist.php
tests/phpunit/WebTest/Utils/RedirectTest.php

index 34d5f5d8351f66cb2dca1404c51bf3400171bdb3..2884833fb183390488273aa2bc771224f7253f95 100644 (file)
@@ -370,6 +370,7 @@ class CRM_Admin_Form_PaymentProcessor extends CRM_Admin_Form {
    * @param bool $test
    */
   public function updatePaymentProcessor(&$values, $domainID, $test) {
+    // @todo remove this function (some or all) in favour or CRM_Financial_BAO_PaymentProcessor::create.
     $dao = new CRM_Financial_DAO_PaymentProcessor();
 
     $dao->id = $test ? $this->_testID : $this->_id;
index 4cef93581c62adb1df814a3764b229d234d75682..e89ab51bfab63cff9464fe0c666090d03da76730 100644 (file)
@@ -55,11 +55,13 @@ class CRM_Case_ManagedEntities {
   }
 
   /**
-   * Get a list of managed activity-types by searching CiviCase XML files
+   * Get a list of managed activity-types by searching CiviCase XML files.
+   *
+   * @param \CRM_Case_XMLRepository $xmlRepo
+   * @param \CRM_Core_ManagedEntities $me
    *
    * @return array
    * @see CRM_Utils_Hook::managed
-   * @throws CRM_Core_Exception
    */
   public static function createManagedActivityTypes(CRM_Case_XMLRepository $xmlRepo, CRM_Core_ManagedEntities $me) {
     $result = array();
index 5659fcd932381e12b07bd373c2ac4c6e59f04222..cc0c32fb0645fe456988233a2acb784e68f6e270 100644 (file)
@@ -5557,8 +5557,8 @@ AND   displayRelType.is_active = 1
    *   depends on the operator and who's calling the query builder.
    * @param int $grouping
    *   the index where to place the where clause.
-   * @param $selectValues
-   *   The key value pairs for this element. This allows us to use this function for things besides option-value pairs.
+   * @param string $daoName
+   *   DAO Name.
    * @param array $field
    *   an array that contains various properties of the field identified by $name.
    * @param string $label
index f2a2fba398b8922708b037efce67a74a2de8329d..4899a4c0e44ea080df90ad6a0f3e40d59d574475 100644 (file)
@@ -48,7 +48,9 @@ interface CRM_Contact_Form_Search_Interface {
   public function buildTaskList(CRM_Core_Form_Search $form);
 
   /**
-   * Builds the quickform for this search
+   * Builds the quickform for this search.
+   *
+   * @param CRM_Core_Form $form
    */
   public function buildForm(&$form);
 
index b26de01920d9c5ea2a9ab0389d348a1d3673f12c..9fe452586057e994b1236dc18728757e509b23df 100644 (file)
@@ -279,10 +279,15 @@ class CRM_Core_BAO_Setting extends CRM_Core_DAO_Setting {
   }
 
   /**
-   * Validate & convert settings input
+   * Validate & convert settings input.
    *
-   * @value mixed value of the setting to be set
-   * @fieldSpec array Metadata for given field (drawn from the xml)
+   * @param mixed $value
+   *   value of the setting to be set
+   * @param array $fieldSpec
+   *   Metadata for given field (drawn from the xml)
+   *
+   * @return bool
+   * @throws \api_Exception
    */
   public static function validateSetting(&$value, $fieldSpec) {
     if ($fieldSpec['type'] == 'String' && is_array($value)) {
index 63ab7fb4048f0647e8be3f59aaba4f7d4c7cb07a..e85368b4c9e2896dafa3e4ef9dbdc6cfb1c9db4f 100644 (file)
@@ -468,8 +468,9 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
   }
 
   /**
-   * Core function that builds the form. We redefine this function
-   * here and expect all CRM forms to build their form in the function
+   * Core function that builds the form.
+   *
+   * We redefine this function here and expect all CRM forms to build their form in the function
    * buildQuickForm.
    */
   public function buildForm() {
@@ -523,7 +524,7 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
   }
 
   /**
-   * Add default Next / Back buttons
+   * Add default Next / Back buttons.
    *
    * @param array $params
    *   Array of associative arrays in the order in which the buttons should be
@@ -623,7 +624,9 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
   }
 
   /**
-   * Getter function for title. Should be over-ridden by derived class
+   * Getter function for title.
+   *
+   * Should be over-ridden by derived class.
    *
    * @return string
    */
@@ -872,8 +875,9 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
   }
 
   /**
-   * Getter function for renderer. If renderer is not set
-   * create one and initialize it
+   * Getter function for renderer.
+   *
+   * If renderer is not set create one and initialize it.
    *
    * @return object
    */
@@ -908,8 +912,9 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
   }
 
   /**
-   * A wrapper for getTemplateFileName that includes calling the hook to
-   * prevent us from having to copy & paste the logic of calling the hook
+   * A wrapper for getTemplateFileName.
+   *
+   * This includes calling the hook to prevent us from having to copy & paste the logic of calling the hook.
    */
   public function getHookedTemplateFileName() {
     $pageTemplateFile = $this->getTemplateFileName();
@@ -918,8 +923,9 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
   }
 
   /**
-   * Default extra tpl file basically just replaces .tpl with .extra.tpl
-   * i.e. we dont override
+   * Default extra tpl file basically just replaces .tpl with .extra.tpl.
+   *
+   * i.e. we do not override.
    *
    * @return string
    */
@@ -1759,6 +1765,17 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
 
   /**
    * Add a currency and money element to the form.
+   *
+   * @param string $name
+   * @param string $label
+   * @param bool $required
+   * @param null $attributes
+   * @param bool $addCurrency
+   * @param string $currencyName
+   * @param null $defaultCurrency
+   * @param bool $freezeCurrency
+   *
+   * @return \HTML_QuickForm_Element
    */
   public function addMoney(
     $name,
@@ -1920,7 +1937,12 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
 
   /**
    * Helper function to verify that required fields have been filled.
+   *
    * Typically called within the scope of a FormRule function
+   *
+   * @param array $fields
+   * @param array $values
+   * @param array $errors
    */
   public static function validateMandatoryFields($fields, $values, &$errors) {
     foreach ($fields as $name => $fld) {
@@ -1991,6 +2013,11 @@ class CRM_Core_Form extends HTML_QuickForm_Page {
     return is_numeric($userID) ? $userID : NULL;
   }
 
+  /**
+   * Get the contact id that the form is being submitted for.
+   *
+   * @return int|NULL
+   */
   public function getContactID() {
     return $this->setContactID();
   }
index 634c300013d963d2598d5334f8747a0fe1548096..861c41f7f334a794f7f986baebdb8e6a44ca5faa 100644 (file)
@@ -58,6 +58,11 @@ class CRM_Core_Module {
 
   /**
    * Get a list of all known modules.
+   *
+   * @param bool $fresh
+   *   Force new results?
+   *
+   * @return array
    */
   public static function getAll($fresh = FALSE) {
     static $result;
index cbdd6ed2e9f81180a98e1cc204aa8a471c0f9ce5..382236e82ac124a580485f08df3f9f5fb6561470 100644 (file)
  * list display, pagination, sorting and export in multiple formats (CSV is
  * supported right now, XML support will be added as and when needed
  *
- *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 interface CRM_Core_Selector_API {
 
   /**
+   * Get pager parameters.
+   *
    * Based on the action, the GET variables and the session state
    * it adds various key => value pairs to the params array including
    *
@@ -55,8 +54,6 @@ interface CRM_Core_Selector_API {
    *   The action being performed.
    * @param array $params
    *   The array that the pagerParams will be inserted into.
-   *
-   * @return void
    */
   public function getPagerParams($action, &$params);
 
@@ -72,7 +69,8 @@ interface CRM_Core_Selector_API {
   public function &getSortOrder($action);
 
   /**
-   * Returns the column headers as an array of tuples:
+   * Returns the column headers as an array of tuples.
+   *
    * (name, sortName (key to the sort array))
    *
    * @param string $action
@@ -116,7 +114,7 @@ interface CRM_Core_Selector_API {
   public function &getRows($action, $offset, $rowCount, $sort, $type = NULL);
 
   /**
-   * Return the template (.tpl) filename
+   * Return the template (.tpl) filename.
    *
    * @param string $action
    *   The action being performed.
index ce6cf4646de843cb893141d6ed4e5d2c0673df20..0e1feb7ff27423a38cb1bd37f78cc9ed3d1e4959 100644 (file)
@@ -29,8 +29,6 @@
  *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 class CRM_Event_BAO_Event extends CRM_Event_DAO_Event {
 
@@ -222,10 +220,11 @@ class CRM_Event_BAO_Event extends CRM_Event_DAO_Event {
   }
 
   /**
-   * Delete the location block associated with an event,
-   * if not being used by any other event.
+   * Delete the location block associated with an event.
+   *
+   * Function checks that it is not being used by any other event.
    *
-   * @param $locBlockId
+   * @param int $locBlockId
    *   Location block id to be deleted.
    * @param int $eventId
    *   Event with which loc block is associated.
@@ -246,7 +245,7 @@ class CRM_Event_BAO_Event extends CRM_Event_DAO_Event {
   }
 
   /**
-   * Get current/future Events
+   * Get current/future Events.
    *
    * @param int $all
    *   0 returns current and future events.
@@ -321,7 +320,6 @@ WHERE  ( civicrm_event.is_template IS NULL OR civicrm_event.is_template = 0 )";
   /**
    * Get events Summary.
    *
-   *
    * @return array
    *   Array of event summary values
    */
@@ -911,8 +909,9 @@ WHERE civicrm_event.is_active = 1
   }
 
   /**
-   * make a copy of a Event, including
-   * all the fields in the event Wizard
+   * Make a copy of a Event.
+   *
+   * Include all the fields in the event Wizard.
    *
    * @param int $id
    *   The event id to copy.
@@ -1030,8 +1029,9 @@ WHERE civicrm_event.is_active = 1
   }
 
   /**
-   * This is sometimes called in a loop (during event search)
-   * hence we cache the values to prevent repeated calls to the db
+   * This is sometimes called in a loop (during event search).
+   *
+   * We cache the values to prevent repeated calls to the db.
    */
   public static function isMonetary($id) {
     static $isMonetary = array();
@@ -1045,8 +1045,9 @@ WHERE civicrm_event.is_active = 1
   }
 
   /**
-   * This is sometimes called in a loop (during event search)
-   * hence we cache the values to prevent repeated calls to the db
+   * This is sometimes called in a loop (during event search).
+   *
+   * We cache the values to prevent repeated calls to the db.
    */
   public static function usesPriceSet($id) {
     static $usesPriceSet = array();
@@ -1057,15 +1058,13 @@ WHERE civicrm_event.is_active = 1
   }
 
   /**
-   * Process that send e-mails
+   * Send e-mails.
    *
    * @param int $contactID
-   * @param $values
+   * @param array $values
    * @param int $participantId
    * @param bool $isTest
    * @param bool $returnMessageText
-   *
-   * @return void
    */
   public static function sendMail($contactID, &$values, $participantId, $isTest = FALSE, $returnMessageText = FALSE) {
 
@@ -1260,8 +1259,6 @@ WHERE civicrm_event.is_active = 1
    * @param $isTest
    * @param bool $isCustomProfile
    * @param array $participantParams
-   *
-   * @return void
    */
   public static function buildCustomDisplay(
     $id,
@@ -1447,8 +1444,6 @@ WHERE civicrm_event.is_active = 1
    *   Formatted array of key value.
    *
    * @param array $profileFields
-   *
-   * @return void
    */
   public static function displayProfile(&$params, $gid, &$groupTitle, &$values, &$profileFields = array()) {
     if ($gid) {
index 941aeb6766773ce0de30e1c18999b8f9286720f6..ebc1ff95a82392b874a00d161a73301694aa389e 100644 (file)
  */
 
 /**
- *
- *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 class CRM_Event_BAO_Participant extends CRM_Event_DAO_Participant {
 
@@ -817,8 +813,6 @@ WHERE  civicrm_participant.id = {$participantId}
    *   (reference) the default values, some of which need to be resolved.
    * @param bool $reverse
    *   True if we want to resolve the values in the reverse direction (value -> name).
-   *
-   * @return void
    */
   public static function resolveDefaults(&$defaults, $reverse = FALSE) {
     self::lookupValue($defaults, 'event', CRM_Event_PseudoConstant::event(), $reverse);
@@ -827,8 +821,7 @@ WHERE  civicrm_participant.id = {$participantId}
   }
 
   /**
-   * convert associative array names to values
-   * and vice-versa.
+   * Convert associative array names to values and vice-versa.
    *
    * This function is used by both the web form layer and the api. Note that
    * the api needs the name => value conversion, also the view layer typically
@@ -856,12 +849,12 @@ WHERE  civicrm_participant.id = {$participantId}
   }
 
   /**
-   * Delete the record that are associated with this participation.
+   * Delete the records that are associated with this participation.
    *
    * @param int $id
    *   Id of the participation to delete.
    *
-   * @return void
+   * @return \CRM_Event_DAO_Participant
    */
   public static function deleteParticipant($id) {
     CRM_Utils_Hook::pre('delete', 'Participant', $id, CRM_Core_DAO::$_nullArray);
@@ -967,10 +960,7 @@ WHERE  civicrm_participant.id = {$participantId}
    * separated string before using fee_level in view mode.
    *
    * @param string $eventLevel
-   *   Event_leval string from db.
-   *
-   *
-   * @return void
+   *   Event_level string from db.
    */
   public static function fixEventLevel(&$eventLevel) {
     if ((substr($eventLevel, 0, 1) == CRM_Core_DAO::VALUE_SEPARATOR) &&
@@ -1185,9 +1175,6 @@ INNER JOIN civicrm_price_field_value value ON ( value.id = lineItem.price_field_
    * @param int $statusId
    *   Status id for participant.
    * @param bool $updateRegisterDate
-   *
-   * @return void
-   *
    */
   public static function updateStatus($participantIds, $statusId, $updateRegisterDate = FALSE) {
     if (!is_array($participantIds) || empty($participantIds) || !$statusId) {
index 05328b5c337da0a72fc00bfe0d584eb43d349bfa..29188e95bcaee8ff5d8f6faafd844a8e52049889 100644 (file)
@@ -58,8 +58,6 @@ class CRM_Event_BAO_Query {
    * Build select for CiviEvent.
    *
    * @param $query
-   *
-   * @return void
    */
   public static function select(&$query) {
     if (($query->_mode & CRM_Contact_BAO_Query::MODE_EVENT) ||
index 7bbcf6da98b5c21faf1d13e16db03d83b4ce90e6..bd2c5e0c060a5de9d1cd190f0fbd539b2847cdab 100644 (file)
@@ -48,6 +48,10 @@ class CRM_Event_Controller_Search extends CRM_Core_Controller {
 
   /**
    * Class constructor.
+   *
+   * @param string $title
+   * @param bool|int $action
+   * @param bool $modal
    */
   public function __construct($title = NULL, $action = CRM_Core_Action::NONE, $modal = TRUE) {
 
index 7c9ed4aa1beb9c4929ea2e88762a32efaf0fe111..92dc9a4b161f055c46e0d9fce42fdbb123341c62 100644 (file)
@@ -43,8 +43,6 @@ class CRM_Event_Form_EventFees {
    * Set variables up before form is built.
    *
    * @param CRM_Core_Form $form
-   *
-   * @return void
    */
   public static function preProcess(&$form) {
     //as when call come from register.php
@@ -69,10 +67,7 @@ class CRM_Event_Form_EventFees {
    * This function sets the default values for the form in edit/view mode
    * the default values are retrieved from the database
    *
-   *
    * @param CRM_Core_Form $form
-   *
-   * @return void
    */
   public static function setDefaultValues(&$form) {
     $defaults = array();
@@ -259,7 +254,7 @@ class CRM_Event_Form_EventFees {
    * @param int $eventID
    * @param bool $includeQtyZero
    *
-   * @return void
+   * @return array
    */
   public static function setDefaultPriceSet($participantID, $eventID = NULL, $includeQtyZero = TRUE) {
     $defaults = array();
@@ -342,8 +337,6 @@ SELECT  id, html_type
    * Build the form object.
    *
    * @param CRM_Core_Form $form
-   *
-   * @return void
    */
   public static function buildQuickForm(&$form) {
     if ($form->_eventId) {
index a1a1cc2981706906b92fbf90ec0ea895f32da80a..d759c0f4a13a8a87493f210d17dc2260540a45f1 100644 (file)
  */
 
 /**
- *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
- * This class generates form components for processing Event
- *
+ * This class generates form components for processing Event.
  */
 class CRM_Event_Form_ManageEvent extends CRM_Core_Form {
 
   /**
-   * The id of the event we are proceessing.
+   * The id of the event we are processing.
    *
    * @var int
    */
@@ -103,8 +99,6 @@ class CRM_Event_Form_ManageEvent extends CRM_Core_Form {
 
   /**
    * Set variables up before form is built.
-   *
-   * @return void
    */
   public function preProcess() {
     $config = CRM_Core_Config::singleton();
@@ -256,11 +250,9 @@ class CRM_Event_Form_ManageEvent extends CRM_Core_Form {
   }
 
   /**
-   * Set default values for the form. For edit/view mode
-   * the default values are retrieved from the database
-   *
+   * Set default values for the form.
    *
-   * @return void
+   * For edit/view mode the default values are retrieved from the database.
    */
   public function setDefaultValues() {
     $defaults = array();
@@ -287,8 +279,6 @@ class CRM_Event_Form_ManageEvent extends CRM_Core_Form {
 
   /**
    * Build the form object.
-   *
-   * @return void
    */
   public function buildQuickForm() {
     $session = CRM_Core_Session::singleton();
index f45d19630f66ab3f9c971f053fb29befa5d8ea8e..3547739399006f9975cf858006f8bbd240062f25 100644 (file)
  *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
- * This class generates form components for Conference Slots
- *
+ * This class generates form components for Conference Slots.
  */
 class CRM_Event_Form_ManageEvent_Conference extends CRM_Event_Form_ManageEvent {
 
@@ -45,8 +42,7 @@ class CRM_Event_Form_ManageEvent_Conference extends CRM_Event_Form_ManageEvent {
   public $_action;
 
   /**
-   *
-   * @return void
+   * Build quick form.
    */
   public function buildQuickForm() {
     $slots = CRM_Core_OptionGroup::values('conference_slot');
@@ -70,6 +66,9 @@ class CRM_Event_Form_ManageEvent_Conference extends CRM_Event_Form_ManageEvent {
     parent::buildQuickForm();
   }
 
+  /**
+   * Post process form.
+   */
   public function postProcess() {
     $params = $this->exportValues();
 
@@ -80,7 +79,7 @@ class CRM_Event_Form_ManageEvent_Conference extends CRM_Event_Form_ManageEvent {
   }
 
   /**
-   * Return a descriptive name for the page, used in wizard header
+   * Return a descriptive name for the page, used in wizard header.
    *
    * @return string
    */
index 3be445512d98dd9dbffdc62c03c02dafa41eb3e2..bfd8d416c7bfab453f61fdd41dce23790e4a2bd2 100644 (file)
  *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
- * This class is to build the form for Deleting Group
+ * This class is to build the form for Deleting Group.
  */
 class CRM_Event_Form_ManageEvent_Delete extends CRM_Event_Form_ManageEvent {
 
@@ -47,8 +45,6 @@ class CRM_Event_Form_ManageEvent_Delete extends CRM_Event_Form_ManageEvent {
 
   /**
    * Set variables up before form is built.
-   *
-   * @return void
    */
   public function preProcess() {
     parent::preProcess();
@@ -67,8 +63,6 @@ class CRM_Event_Form_ManageEvent_Delete extends CRM_Event_Form_ManageEvent {
 
   /**
    * Build the form object.
-   *
-   * @return void
    */
   public function buildQuickForm() {
     $this->assign('title', $this->_title);
@@ -89,8 +83,6 @@ class CRM_Event_Form_ManageEvent_Delete extends CRM_Event_Form_ManageEvent {
 
   /**
    * Process the form when submitted.
-   *
-   * @return void
    */
   public function postProcess() {
     $participant = new CRM_Event_DAO_Participant();
index 98f9faa27c07d8e4801bbabb419d2aab1fe166a1..1e210b94cefba0c0e84d78b578a6b0352db06419 100644 (file)
  */
 
 /**
- *
- *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
- * This class generates form components for processing Event
- *
+ * This class generates form components for processing Event.
  */
 class CRM_Event_Form_ManageEvent_EventInfo extends CRM_Event_Form_ManageEvent {
 
@@ -47,8 +42,6 @@ class CRM_Event_Form_ManageEvent_EventInfo extends CRM_Event_Form_ManageEvent {
 
   /**
    * Set variables up before form is built.
-   *
-   * @return void
    */
   public function preProcess() {
     parent::preProcess();
@@ -78,11 +71,9 @@ class CRM_Event_Form_ManageEvent_EventInfo extends CRM_Event_Form_ManageEvent {
   }
 
   /**
-   * Set default values for the form. For edit/view mode
-   * the default values are retrieved from the database
-   *
+   * Set default values for the form.
    *
-   * @return void
+   * For edit/view mode he default values are retrieved from the database.
    */
   public function setDefaultValues() {
     $defaults = parent::setDefaultValues();
@@ -130,8 +121,6 @@ class CRM_Event_Form_ManageEvent_EventInfo extends CRM_Event_Form_ManageEvent {
 
   /**
    * Build the form object.
-   *
-   * @return void
    */
   public function buildQuickForm() {
     //need to assign custom data type and subtype to the template
@@ -247,9 +236,6 @@ class CRM_Event_Form_ManageEvent_EventInfo extends CRM_Event_Form_ManageEvent {
 
   /**
    * Process the form submission.
-   *
-   *
-   * @return void
    */
   public function postProcess() {
     $params = $this->controller->exportValues($this->_name);
@@ -311,7 +297,7 @@ class CRM_Event_Form_ManageEvent_EventInfo extends CRM_Event_Form_ManageEvent {
   }
 
   /**
-   * Return a descriptive name for the page, used in wizard header
+   * Return a descriptive name for the page, used in wizard header.
    *
    * @return string
    */
@@ -320,8 +306,7 @@ class CRM_Event_Form_ManageEvent_EventInfo extends CRM_Event_Form_ManageEvent {
   }
 
   /**
-   * Retrieve event template custom data values.
-   * and set as default values for current new event.
+   * Retrieve event template custom data values and set as default values for current new event.
    *
    * @param int $templateId
    *   Event template id.
index f0c53a30c464e0e9695948d526445f3f845c2423..55123f9f81aefd239395ef7a5276013702a4d307 100644 (file)
  */
 
 /**
- *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
- * This class generates form components for Event Fees
- *
+ * This class generates form components for Event Fees.
  */
 class CRM_Event_Form_ManageEvent_Fee extends CRM_Event_Form_ManageEvent {
 
@@ -61,19 +57,15 @@ class CRM_Event_Form_ManageEvent_Fee extends CRM_Event_Form_ManageEvent {
 
   /**
    * Set variables up before form is built.
-   *
-   * @return void
    */
   public function preProcess() {
     parent::preProcess();
   }
 
   /**
-   * Set default values for the form. For edit/view mode
-   * the default values are retrieved from the database
-   *
+   * Set default values for the form.
    *
-   * @return void
+   * For edit/view mode the default values are retrieved from the database.
    */
   public function setDefaultValues() {
     $parentDefaults = parent::setDefaultValues();
@@ -247,8 +239,6 @@ class CRM_Event_Form_ManageEvent_Fee extends CRM_Event_Form_ManageEvent {
 
   /**
    * Build the form object.
-   *
-   * @return void
    */
   public function buildQuickForm() {
 
@@ -400,9 +390,6 @@ class CRM_Event_Form_ManageEvent_Fee extends CRM_Event_Form_ManageEvent {
 
   /**
    * Add local and global form rules.
-   *
-   *
-   * @return void
    */
   public function addRules() {
     $this->addFormRule(array('CRM_Event_Form_ManageEvent_Fee', 'formRule'));
@@ -557,8 +544,6 @@ class CRM_Event_Form_ManageEvent_Fee extends CRM_Event_Form_ManageEvent {
 
   /**
    * Process the form.
-   *
-   * @return void
    */
   public function postProcess() {
     $eventTitle = '';
index 75d08a2a7307d8d1e02f5987160c589c282361c3..04cc5b0945b961d96b2e303557816523c80e94e6 100644 (file)
@@ -69,8 +69,6 @@ class CRM_Event_Form_ManageEvent_Location extends CRM_Event_Form_ManageEvent {
 
   /**
    * Set variables up before form is built.
-   *
-   * @return void
    */
   public function preProcess() {
     parent::preProcess();
@@ -95,11 +93,9 @@ class CRM_Event_Form_ManageEvent_Location extends CRM_Event_Form_ManageEvent {
   }
 
   /**
-   * Set default values for the form. Note that in edit/view mode
-   * the default values are retrieved from the database
-   *
+   * Set default values for the form.
    *
-   * @return void
+   * Note that in edit/view mode the default values are retrieved from the database.
    */
   public function setDefaultValues() {
     $defaults = $this->_values;
@@ -126,9 +122,6 @@ class CRM_Event_Form_ManageEvent_Location extends CRM_Event_Form_ManageEvent {
 
   /**
    * Add local and global form rules.
-   *
-   *
-   * @return void
    */
   public function addRules() {
     $this->addFormRule(array('CRM_Event_Form_ManageEvent_Location', 'formRule'));
@@ -151,9 +144,7 @@ class CRM_Event_Form_ManageEvent_Location extends CRM_Event_Form_ManageEvent {
   }
 
   /**
-   *  function to build location block.
-   *
-   * @return void
+   * Function to build location block.
    */
   public function buildQuickForm() {
     //load form for child blocks
@@ -214,9 +205,6 @@ class CRM_Event_Form_ManageEvent_Location extends CRM_Event_Form_ManageEvent {
 
   /**
    * Process the form submission.
-   *
-   *
-   * @return void
    */
   public function postProcess() {
     $params = $this->exportValues();
index 0c8ccbe3e19782c7588fb4a51b7669c5497cb98d..f4c2d941b45320d2a0a8fd4e98cd6d1a9a9ad923 100644 (file)
  */
 
 /**
- *
- *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
- * This class generates form components for processing Event
- *
+ * This class generates form components for processing Event.
  */
 class CRM_Event_Form_ManageEvent_Registration extends CRM_Event_Form_ManageEvent {
 
@@ -52,8 +47,6 @@ class CRM_Event_Form_ManageEvent_Registration extends CRM_Event_Form_ManageEvent
 
   /**
    * Set variables up before form is built.
-   *
-   * @return void
    */
   public function preProcess() {
     $this->_addProfileBottom = CRM_Utils_Array::value('addProfileBottom', $_GET, FALSE);
@@ -90,10 +83,8 @@ class CRM_Event_Form_ManageEvent_Registration extends CRM_Event_Form_ManageEvent
 
   /**
    * Set default values for the form.
-   * the default values are retrieved from the database
    *
-   *
-   * @return void
+   * The default values are retrieved from the database.
    */
   public function setDefaultValues() {
     if ($this->_addProfileBottom || $this->_addProfileBottomAdd) {
index f600b1a1817c91a5b5123caeba02f0ece1a7192d..a3f754bc9f1056c1021f827ef267ef62f88ccdfa 100644 (file)
  */
 
 /**
- *
- *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
- * This class generates form components for processing Event
- *
+ * This class generates form components for processing Event.
  */
 class CRM_Event_Form_Registration extends CRM_Core_Form {
 
@@ -427,8 +422,6 @@ class CRM_Event_Form_Registration extends CRM_Core_Form {
 
   /**
    * Assign the minimal set of variables to the template.
-   *
-   * @return void
    */
   public function assignToTemplate() {
     //process only primary participant params
@@ -531,8 +524,6 @@ class CRM_Event_Form_Registration extends CRM_Core_Form {
    * @param int $id
    * @param string $name
    * @param bool $viewOnly
-   *
-   * @return void
    */
   public function buildCustom($id, $name, $viewOnly = FALSE) {
     if ($id) {
@@ -629,6 +620,8 @@ class CRM_Event_Form_Registration extends CRM_Core_Form {
   }
 
   /**
+   * Initiate event fee.
+   *
    * @param CRM_Core_Form $form
    * @param int $eventID
    *
@@ -712,8 +705,6 @@ class CRM_Event_Form_Registration extends CRM_Core_Form {
    * @param int $contactID
    * @param null $contribution
    * @param null $payment
-   *
-   * @return void
    */
   public function confirmPostProcess($contactID = NULL, $contribution = NULL, $payment = NULL) {
     // add/update contact information
@@ -815,6 +806,7 @@ class CRM_Event_Form_Registration extends CRM_Core_Form {
    *
    * @param CRM_Core_Form $form
    * @param int $contactID
+   *
    * @return \CRM_Event_BAO_Participant
    */
   public static function addParticipant(&$form, $contactID) {
@@ -915,19 +907,15 @@ WHERE  v.option_group_id = g.id
     return $participant;
   }
 
-  /* Calculate the total participant count as per params.
-   *
-   * @param array $params
-   *   User params.
-   *
-   * @return $totalCount total participant count.
-   */
   /**
+   * Calculate the total participant count as per params.
+   *
    * @param CRM_Core_Form $form
    * @param array $params
+   *   User params.
    * @param bool $skipCurrent
    *
-   * @return int|string
+   * @return int
    */
   public static function getParticipantCount(&$form, $params, $skipCurrent = FALSE) {
     $totalCount = 0;
@@ -1012,21 +1000,17 @@ WHERE  v.option_group_id = g.id
     return $totalCount;
   }
 
-  /* Format user submitted price set params.
+  /**
+   * Format user submitted price set params.
+   *
    * Convert price set each param as an array.
    *
+   * @param CRM_Core_Form $form
    * @param array $params
    *   An array of user submitted params.
    *
-   *
    * @return array
-   *   , formatted price set params.
-   */
-  /**
-   * @param CRM_Core_Form $form
-   * @param array $params
-   *
-   * @return mixed
+   *   Formatted price set params.
    */
   public static function formatPriceSetParams(&$form, $params) {
     if (!is_array($params) || empty($params)) {
@@ -1063,20 +1047,16 @@ WHERE  v.option_group_id = g.id
     return $params;
   }
 
-  /* Calculate total count for each price set options.
-   * those are currently selected by user.
-   *
-   * @param $form
-   *   Form object.
+  /**
+   * Calculate total count for each price set options.
    *
+   * - currently selected by user.
    *
-   * @return array
-   *   , array of each option w/ count total.
-   */
-  /**
-   * @param $form
+   * @param CRM_Core_Form $form
+   *   Form object.
    *
    * @return array
+   *   array of each option w/ count total.
    */
   public static function getPriceSetOptionCount(&$form) {
     $params = $form->get('params');
@@ -1144,6 +1124,8 @@ WHERE  v.option_group_id = g.id
   }
 
   /**
+   * Check if template file exists.
+   *
    * @param string $suffix
    *
    * @return null|string
@@ -1165,6 +1147,8 @@ WHERE  v.option_group_id = g.id
   }
 
   /**
+   * Get template file name.
+   *
    * @return null|string
    */
   public function getTemplateFileName() {
@@ -1173,6 +1157,8 @@ WHERE  v.option_group_id = g.id
   }
 
   /**
+   * Override extra template name.
+   *
    * @return null|string
    */
   public function overrideExtraTemplateFileName() {
@@ -1184,7 +1170,7 @@ WHERE  v.option_group_id = g.id
    * Reset values for all options those are full.
    *
    * @param array $optionFullIds
-   * @param $form
+   * @param CRM_Core_Form $form
    */
   public static function resetElementValue($optionFullIds = array(), &$form) {
     if (!is_array($optionFullIds) ||
@@ -1253,6 +1239,8 @@ WHERE  v.option_group_id = g.id
   }
 
   /**
+   * Reset submitted value.
+   *
    * @param string $elementName
    * @param array $optionIds
    * @param CRM_Core_form $form
@@ -1298,8 +1286,9 @@ WHERE  v.option_group_id = g.id
   }
 
   /**
-   * Validate price set submitted params for price option limit,
-   * as well as user should select at least one price field option.
+   * Validate price set submitted params for price option limit.
+   *
+   * User should select at least one price field option.
    *
    * @param CRM_Core_Form $form
    * @param array $params
@@ -1433,7 +1422,9 @@ WHERE  v.option_group_id = g.id
   }
 
   /**
-   * set the first participant ID if not set, CRM-10032
+   * Set the first participant ID if not set.
+   *
+   * CRM-10032.
    *
    * @param int $participantID
    */
@@ -1458,6 +1449,8 @@ WHERE  v.option_group_id = g.id
   }
 
   /**
+   * Check if event is valid.
+   *
    * @todo - combine this with CRM_Event_BAO_Event::validRegistrationRequest
    * (probably extract relevant values here & call that with them & handle bounces & redirects here -as
    * those belong in the form layer)
index 052533502f2afec0dc1b84b2318b7d977df418e1..f32a03cc70e727f81b02d72c0869369fd3e1a007 100644 (file)
  */
 
 /**
- *
- *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
- * $Id$
- *
  */
 
 /**
- * This class generates form components for processing Event
- *
+ * This class generates form components for processing Event.
  */
 class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
 
@@ -58,7 +53,6 @@ class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
    * Set variables up before form is built.
    */
   public function preProcess() {
-
     parent::preProcess();
 
     // lineItem isn't set until Register postProcess
@@ -187,8 +181,7 @@ class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
   }
 
   /**
-   * Overwrite action, since we are only showing elements in frozen mode
-   * no help display needed
+   * Overwrite action, since we are only showing elements in frozen mode no help display needed.
    *
    * @return int
    */
@@ -388,9 +381,11 @@ class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
   }
 
   /**
-   * @param $fields
-   * @param $files
-   * @param $self
+   * Apply form rule.
+   *
+   * @param array $fields
+   * @param array $files
+   * @param CRM_Core_Form $self
    *
    * @return array|bool
    */
@@ -805,7 +800,8 @@ class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
             $participantNum = NULL;
             $participantNum = $participantID;
             if ($participantID == $registerByID) {
-              $participantNum = 0;  // is primary particpant
+              // This is the is primary participant.
+              $participantNum = 0;
             }
             else {
               if ($participantNum = array_search('participant', $copyParticipantCountLines)) {
@@ -955,12 +951,12 @@ class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
    *
    * @param CRM_Core_Form $form
    * @param array $params
-   * @param $result
+   * @param array $result
    * @param int $contactID
    * @param bool $pending
    * @param bool $isAdditionalAmount
    *
-   * @return void
+   * @return \CRM_Contribute_BAO_Contribution
    */
   public static function processContribution(
     &$form, $params, $result, $contactID,
@@ -1061,10 +1057,8 @@ class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
    * Fix the Location Fields.
    *
    * @param array $params
-   * @param $fields
+   * @param array $fields
    * @param CRM_Core_Form $form
-   *
-   * @return void
    */
   public static function fixLocationFields(&$params, &$fields, &$form) {
     if (!empty($form->_fields)) {
@@ -1110,10 +1104,10 @@ class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
    *
    * @param int $contactID
    * @param array $params
-   * @param $fields
+   * @param array $fields
    * @param CRM_Core_Form $form
    *
-   * @return void
+   * @return int
    */
   public static function updateContactFields($contactID, $params, $fields, &$form) {
     //add the contact to group, if add to group is selected for a
@@ -1205,7 +1199,9 @@ class CRM_Event_Form_Registration_Confirm extends CRM_Event_Form_Registration {
   }
 
   /**
-   * @param $form
+   * Assign Profiles.
+   *
+   * @param CRM_Core_Form $form
    */
   public static function assignProfiles(&$form) {
     $participantParams = $form->_params;
index 5c0f9055d9e82414d715eab1b49204de869d6b02..988190f01c4f13d194ba616008cf42fa1c5921bb 100644 (file)
  */
 
 /**
- *
- *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2015
  */
 
 /**
- * This class generates form components for processing Event
+ * This class generates form components for processing Event.
  */
 class CRM_Event_Form_Registration_Register extends CRM_Event_Form_Registration {
 
@@ -48,11 +46,17 @@ class CRM_Event_Form_Registration_Register extends CRM_Event_Form_Registration {
   protected $_waitlistMsg = NULL;
   protected $_requireApprovalMsg = NULL;
 
+  /**
+   * Deprecated parameter that we hope to remove.
+   *
+   * @var bool
+   */
   public $_quickConfig = NULL;
 
   /**
-   * Allow developer to use hook_civicrm_buildForm()
-   * to override the registration dupe check
+   * Skip duplicate check.
+   *
+   * This can be set using hook_civicrm_buildForm() to override the registration dupe check.
    * CRM-7604
    */
   public $_skipDupeRegistrationCheck = FALSE;
@@ -60,12 +64,17 @@ class CRM_Event_Form_Registration_Register extends CRM_Event_Form_Registration {
   public $_paymentProcessorID;
 
   /**
+   * Show fee block or not.
+   *
    * @var boolean determines if fee block should be shown or hidden
    */
   public $_noFees;
 
   /**
-   * Array of payment related fields to potentially display on this form (generally credit card or debit card fields). This is rendered via billingBlock.tpl
+   * Array of payment related fields to potentially display on this form (generally credit card or debit card fields).
+   *
+   * This is rendered via billingBlock.tpl.
+   *
    * @var array
    */
   public $_paymentFields = array();
@@ -266,6 +275,8 @@ class CRM_Event_Form_Registration_Register extends CRM_Event_Form_Registration {
 
   /**
    * Build the form object.
+   *
+   * @return void
    */
   public function buildQuickForm() {
     // build profiles first so that we can determine address fields etc
index 54218d7fa11a863dc89a2fa9674e1dec9273d33e..3960d1fc6343e6b5af6c908cfac7cf2a2b1ecc95 100644 (file)
@@ -46,6 +46,8 @@ class CRM_Import_DataSource_CSV extends CRM_Import_DataSource {
 
   /**
    * Set variables up before form is built.
+   *
+   * @param CRM_Core_Form $form
    */
   public function preProcess(&$form) {
   }
index 2bfd9e2f88912d1a45f1ff5053d418c9d39985e2..01342697b41c6319c05e7084aaef89f174118cf5 100644 (file)
@@ -45,6 +45,10 @@ class CRM_Pledge_Controller_Search extends CRM_Core_Controller {
 
   /**
    * Class constructor.
+   *
+   * @param string $title
+   * @param bool|int $action
+   * @param bool $modal
    */
   public function __construct($title = NULL, $action = CRM_Core_Action::NONE, $modal = TRUE) {
 
index 8e8eff46540aa6ff2bff592b89266c174b5fd8c3..d496b526f494f6961cf2c30cfcd583ad819915da 100644 (file)
@@ -273,7 +273,15 @@ class CRM_Report_Form_Contact_Summary extends CRM_Report_Form {
   }
 
   /**
-   * @param $rows
+   * Initialise basic row.
+   *
+   * @param array $rows
+   *
+   * @param bool $entryFound
+   * @param array $row
+   * @param int $rowId
+   * @param int $rowNum
+   * @param array $types
    *
    * @return bool
    */
index 1b73fc7f09d2f7ace696912cf128d750659f6d55..ce9d2788fc41132e845aff308bcc9a3bda2e039b 100644 (file)
@@ -92,7 +92,15 @@ abstract class CRM_SMS_Provider {
   abstract public function send($recipients, $header, $message, $dncID = NULL);
 
   /**
-   * Return message text. Child class could override this function to have better control over the message being sent.
+   * Return message text.
+   *
+   * Child class could override this function to have better control over the message being sent.
+   *
+   * @param string $message
+   * @param int $contactID
+   * @param array $contactDetails
+   *
+   * @return string
    */
   public function getMessage($message, $contactID, $contactDetails) {
     $html = $message->getHTMLBody();
@@ -102,8 +110,10 @@ abstract class CRM_SMS_Provider {
   }
 
   /**
-   * @param $fields
-   * @param $additionalDetails
+   * Get recipient details.
+   *
+   * @param array $fields
+   * @param array $additionalDetails
    *
    * @return mixed
    */
index 55a4823e23c112f5dbc69a74c1e27b74b29e48cd..d3ab1a54ca1dc9f46ac40749e955e9113f09f195 100644 (file)
@@ -379,9 +379,7 @@ class CRM_UF_Page_Group extends CRM_Core_Page {
    * @param int $id
    *   Uf group id.
    *
-   * @param $action
-   *
-   * @return void
+   * @param int $action
    */
   public function preview($id, $action) {
     $controller = new CRM_Core_Controller_Simple('CRM_UF_Form_Preview', ts('CiviCRM Profile Group Preview'), NULL);
index d4eb3113e76b6411cf732489d3e3e8a20a689412..5b19b3b20c6c4368dceb93897c82e3f73714daba 100644 (file)
@@ -63,7 +63,6 @@ class CRM_Upgrade_Incremental_Base {
    *   alterable.
    * @param string $rev
    *   an intermediate version; note that setPostUpgradeMessage is called repeatedly with different $revs.
-   * @return void
    */
   public function setPostUpgradeMessage(&$postUpgradeMessage, $rev) {
   }
index 44bdf3bdf17c85c5f07e54e671c12ded954995f2..d9fef968e1f7008fcb39ac39ea4d85bd194e0d17 100755 (executable)
@@ -37,7 +37,6 @@ class CRM_Upgrade_Incremental_php_FourFive extends CRM_Upgrade_Incremental_Base
    *   alterable.
    * @param string $rev
    *   an intermediate version; note that setPostUpgradeMessage is called repeatedly with different $revs.
-   * @return void
    */
   public function setPostUpgradeMessage(&$postUpgradeMessage, $rev) {
     if ($rev == '4.5.alpha1') {
index 3b68ed460e4c72d48c298ed930cf62590093d25e..080a15ece465f4b92aab21ccfd9f100c9949f905 100644 (file)
@@ -40,9 +40,7 @@ class CRM_Upgrade_Incremental_php_FourFour extends CRM_Upgrade_Incremental_Base
    * @param $preUpgradeMessage
    * @param string $rev
    *   a version number, e.g. '4.4.alpha1', '4.4.beta3', '4.4.0'.
-   * @param null $currentVer
-   *
-   * @return void
+   * @param string $currentVer
    */
   public function setPreUpgradeMessage(&$preUpgradeMessage, $rev, $currentVer = NULL) {
     if ($rev == '4.4.beta1') {
@@ -68,7 +66,6 @@ class CRM_Upgrade_Incremental_php_FourFour extends CRM_Upgrade_Incremental_Base
    *   alterable.
    * @param string $rev
    *   an intermediate version; note that setPostUpgradeMessage is called repeatedly with different $revs.
-   * @return void
    */
   public function setPostUpgradeMessage(&$postUpgradeMessage, $rev) {
     if ($rev == '4.4.1') {
@@ -352,11 +349,11 @@ ALTER TABLE civicrm_dashboard
   }
 
   /**
-   * @param $rev
-   * @param $originalVer
-   * @param $latestVer
+   * Upgrade script for 4.4.7.
    *
-   * @return void
+   * @param string $rev
+   * @param string $originalVer
+   * @param string $latestVer
    */
   public function upgrade_4_4_7($rev, $originalVer, $latestVer) {
     // For WordPress/Joomla(?), cleanup broken image_URL from 4.4.6 upgrades - https://issues.civicrm.org/jira/browse/CRM-14971
index ba9ede212e1ea9eeb05425986165c130331c164a..d5bbd49ebbe317e225466819a3552ddabaa52f8f 100644 (file)
@@ -60,8 +60,6 @@ class CRM_Upgrade_Incremental_php_FourOne {
    *   alterable.
    * @param string $rev
    *   an intermediate version; note that setPostUpgradeMessage is called repeatedly with different $revs.
-   *
-   * @return void
    */
   public function setPostUpgradeMessage(&$postUpgradeMessage, $rev) {
     if ($rev == '4.1.alpha1') {
index 1b986b1b577d368477ac3e54bca7a75a34ce7274..06734d693f74a22f8f7b69da57bd7adcc9773cff 100644 (file)
@@ -35,12 +35,10 @@ class CRM_Upgrade_Incremental_php_FourSeven extends CRM_Upgrade_Incremental_Base
    * Note: This function is called iteratively for each upcoming
    * revision to the database.
    *
-   * @param $preUpgradeMessage
+   * @param string $preUpgradeMessage
    * @param string $rev
    *   a version number, e.g. '4.4.alpha1', '4.4.beta3', '4.4.0'.
    * @param null $currentVer
-   *
-   * @return void
    */
   public function setPreUpgradeMessage(&$preUpgradeMessage, $rev, $currentVer = NULL) {
     if ($rev == '4.7.alpha1') {
@@ -70,7 +68,6 @@ class CRM_Upgrade_Incremental_php_FourSeven extends CRM_Upgrade_Incremental_Base
    *   alterable.
    * @param string $rev
    *   an intermediate version; note that setPostUpgradeMessage is called repeatedly with different $revs.
-   * @return void
    */
   public function setPostUpgradeMessage(&$postUpgradeMessage, $rev) {
     if ($rev == '4.7.alpha1') {
index 6d88270366e05c675c4012144029eb4f8d0d4713..2c1f3113b2e539f4923107dcf6e13f3a281f280a 100644 (file)
@@ -37,7 +37,6 @@ class CRM_Upgrade_Incremental_php_FourSix extends CRM_Upgrade_Incremental_Base {
    *   alterable.
    * @param string $rev
    *   an intermediate version; note that setPostUpgradeMessage is called repeatedly with different $revs.
-   * @return void
    */
   public function setPostUpgradeMessage(&$postUpgradeMessage, $rev) {
     if ($rev == '4.6.alpha1') {
index ffcbb9ba5cbfc66fa0331c15132e751492494794..30e5379f49179c2b5705c62a06593359cc2d88f9 100644 (file)
@@ -36,12 +36,12 @@ class CRM_Upgrade_Incremental_php_FourThree extends CRM_Upgrade_Incremental_Base
    * Note: This function is called iteratively for each upcoming
    * revision to the database.
    *
-   * @param $preUpgradeMessage
+   * @param string $preUpgradeMessage
    * @param string $rev
    *   a version number, e.g. '4.3.alpha1', '4.3.beta3', '4.3.0'.
    * @param null $currentVer
    *
-   * @return void|bool
+   * @return bool
    */
   public function setPreUpgradeMessage(&$preUpgradeMessage, $rev, $currentVer = NULL) {
     if ($rev == '4.3.beta3') {
index 2954d3fb8081ed05ea2be85195aefc345c955c93..53769370e5c01b99fae6a5e7bd25a4ea7f36e58d 100644 (file)
@@ -38,12 +38,12 @@ class CRM_Upgrade_Incremental_php_FourTwo extends CRM_Upgrade_Incremental_Base {
    * Note: This function is called iteratively for each upcoming
    * revision to the database.
    *
-   * @param $preUpgradeMessage
+   * @param string $preUpgradeMessage
    * @param string $rev
    *   a version number, e.g. '4.2.alpha1', '4.2.beta3', '4.2.0'.
    * @param null $currentVer
    *
-   * @return void
+   * @return bool
    */
   public function setPreUpgradeMessage(&$preUpgradeMessage, $rev, $currentVer = NULL) {
     if ($rev == '4.2.alpha1') {
@@ -139,7 +139,6 @@ INNER JOIN civicrm_price_set cps ON cps.id = cpf.price_set_id AND cps.name <>'de
    *   alterable.
    * @param string $rev
    *   an intermediate version; note that setPostUpgradeMessage is called repeatedly with different $revs.
-   * @return void
    */
   public function setPostUpgradeMessage(&$postUpgradeMessage, $rev) {
     if ($rev == '4.2.beta5') {
index 3d24ceb988d07a76a61ef50c748d0cc8cd1c8ffa..66b8d444a7e3635ed80f75685306ff7604634b25 100644 (file)
@@ -180,10 +180,7 @@ WHERE     %2.id = %1";
    *                          to the price set used for particular event
    * @param array $values
    *   Reference to the values array(.
-   *   this is
-   *                          lineItem array)
-   *
-   * @return void
+   *   this is lineItem array)
    */
   public static function format($fid, &$params, &$fields, &$values) {
     if (empty($params["price_{$fid}"])) {
index 653f0b5eec62f7097480de94873f5f7852690738..b053763d84a10424187842829297dce884ad898f 100644 (file)
@@ -764,8 +764,6 @@ WHERE  id = %1";
    * Build the price set form.
    *
    * @param CRM_Core_Form $form
-   *
-   * @return void
    */
   public static function buildPriceSet(&$form) {
     $priceSetId = $form->get('priceSetId');
index 36febeb338998392c2e0e120b2265051a93ea5b5..ffcd7cdcfe6b291e7cbe7dc344d76f0433b26309 100644 (file)
@@ -427,7 +427,6 @@ class CRM_Upgrade_Snapshot_V4p2_Price_DAO_Field extends CRM_Core_DAO {
    *
    * @param array $values
    *   (reference) the array up for enhancing.
-   * @return void
    */
   public static function addDisplayEnums(&$values) {
     $enumFields = &Snapshot_v4p2_Price_DAO_Field::getEnums();
index f8692396292c24ef9ac042bd7be0fc3ef3eec273..df113703401f59b4d2151ab1a02e2c613f60fde1 100644 (file)
@@ -281,7 +281,9 @@ class CRM_Upgrade_Snapshot_V4p2_Price_DAO_FieldValue extends CRM_Core_DAO {
   }
 
   /**
-   * returns the list of fields that can be imported.
+   * Returns the list of fields that can be imported.
+   *
+   * @param bool $prefix
    *
    * @return array
    */
index a2c7b0f7ea225f652ce447234ea1fa0648b18fcc..2027df7e68a8b2990b67d8bf3e14af9d1da0a281 100644 (file)
@@ -64,7 +64,13 @@ class CRM_Utils_Network {
   }
 
   /**
-   * Check whether a TCP service is available on $host and $port
+   * Check whether a TCP service is available on $host and $port.
+   *
+   * @param string $host
+   * @param string $port
+   * @param string $serverConnectionTimeOut
+   *
+   * @return bool
    */
   public static function checkService($host, $port, $serverConnectionTimeOut) {
     $old_error_reporting = error_reporting();
index ca8adbd55702feef19729b7602a0b44dd1f07b17..36c09215b574d4596df42da2d9026c37bab6da32 100644 (file)
@@ -57,6 +57,7 @@ class Event extends \Symfony\Component\EventDispatcher\Event {
    *   The API responsible for executing the request.
    * @param array $apiRequest
    *   The full description of the API request.
+   * @param \Civi\API\Kernel $apiKernel
    */
   public function __construct($apiProvider, $apiRequest, $apiKernel) {
     $this->apiKernel = $apiKernel;
@@ -65,6 +66,8 @@ class Event extends \Symfony\Component\EventDispatcher\Event {
   }
 
   /**
+   * Get api kernel.
+   *
    * @return \Civi\API\Kernel
    */
   public function getApiKernel() {
index 090970a584abdcf3b889e1f6383caadd4a5b344d..d8f35dd467c95458fcbf916792c59664093379aa 100644 (file)
@@ -350,6 +350,7 @@ function civicrm_api3_generic_getvalue($apiRequest) {
  * Get count of contact references.
  *
  * @param array $params
+ * @param array $apiRequest
  */
 function _civicrm_api3_generic_getrefcount_spec(&$params, $apiRequest) {
   $params['id']['api.required'] = 1;
index 20134c1578d7ed58f61cdfd510c01e9edea8b559..683fb44c24f10b07a0ca931b2edcc5a778cdc810 100644 (file)
@@ -80,6 +80,7 @@ function civicrm_api3_generic_getList($apiRequest) {
  * @param string $entity
  * @param array $request
  * @param array $apiDefaults
+ * @param array $fields
  */
 function _civicrm_api3_generic_getList_defaults($entity, &$request, $apiDefaults, $fields) {
   $config = CRM_Core_Config::singleton();
index 003a2d2281a23c80245941d2c808ae887995e4a1..39d5df94d0178adf0336da5de48476b2b74cfc0f 100644 (file)
@@ -80,7 +80,10 @@ class WebTest_Utils_RedirectTest extends CiviUnitTestCase {
   }
 
   /**
+   * Try redirect.
    *
+   * @param string $input_url
+   * @param string $expected_url
    */
   private function tryRedirect($input_url, $expected_url) {
     // file_put_contents('php://stderr', $input_url . "\n", FILE_APPEND);