Fix comment block for variable declarations
authoreileen <emcnaughton@wikimedia.org>
Thu, 23 May 2019 23:28:10 +0000 (11:28 +1200)
committereileen <emcnaughton@wikimedia.org>
Thu, 23 May 2019 23:41:26 +0000 (11:41 +1200)
Per https://github.com/civicrm/coder/pull/7

31 files changed:
CRM/Contact/BAO/Query.php
CRM/Contact/Form/Search.php
CRM/Contact/Form/Search/Custom/FullText.php
CRM/Contact/Form/Search/Custom/Group.php
CRM/Contact/Import/Parser.php
CRM/Contact/Page/ImageFile.php
CRM/Contact/Page/View/ContactSmartGroup.php
CRM/Contribute/Form/Contribution.php
CRM/Contribute/Form/ContributionBase.php
CRM/Core/Action.php
CRM/Custom/Import/Parser.php
CRM/Event/Form/ManageEvent.php
CRM/Event/Form/Registration/Register.php
CRM/Event/Import/Parser.php
CRM/Financial/Form/Payment.php
CRM/Mailing/Page/View.php
CRM/Member/Import/Parser.php
CRM/Report/Form.php
CRM/SMS/Message.php
CRM/UF/Form/Field.php
CRM/Upgrade/Incremental/Base.php
Civi/Angular/Page/Main.php
tests/phpunit/CRM/Batch/Form/EntryTest.php
tests/phpunit/CRM/Contact/BAO/QueryTestDataProvider.php
tests/phpunit/CRM/Contact/Form/Search/Custom/GroupTestDataProvider.php
tests/phpunit/CRM/Contact/Form/Search/Custom/SampleTestDataProvider.php
tests/phpunit/CRM/Group/Page/AjaxTest.php
tests/phpunit/CRM/Mailing/BAO/QueryTestDataProvider.php
tests/phpunit/CRM/Member/Import/Parser/MembershipTest.php
tests/phpunit/api/v3/RelationshipTest.php
tests/phpunit/api/v3/TagTest.php

index 82dab92b617580ca4fc418dfd0c4888f0777ab83..46840be319c430e970e5135e13ee3c89f411f551 100644 (file)
@@ -409,8 +409,9 @@ class CRM_Contact_BAO_Query {
   protected $_relationshipValuesAdded = FALSE;
 
   /**
-   * Set to the name of the temp table if one has been created
-   * @var String
+   * Set to the name of the temp table if one has been created.
+   *
+   * @var string
    */
   public static $_relationshipTempTable = NULL;
 
index 297b0f3face60e0724cf8fded3845c83675d72aa..9af7f55b17924d14a026b0e96d2b8efd3a46f52c 100644 (file)
@@ -120,7 +120,7 @@ class CRM_Contact_Form_Search extends CRM_Core_Form_Search {
   /**
    * The profile group id used for display.
    *
-   * @var integer
+   * @var int
    */
   protected $_ufGroupID;
 
@@ -132,13 +132,16 @@ class CRM_Contact_Form_Search extends CRM_Core_Form_Search {
   public static $csv = ['contact_type', 'group', 'tag'];
 
   /**
-   * @var string how to display the results. Should we display as
-   *             contributons, members, cases etc
+   * How to display the results. Should we display as contributons, members, cases etc.
+   *
+   * @var string
    */
   protected $_componentMode;
 
   /**
-   * @var string what operator should we use, AND or OR
+   * What operator should we use, AND or OR.
+   *
+   * @var string
    */
   protected $_operator;
 
index 28f61bbb1918a9316e2d00ff966cacc3502b809d..46050f87819f2b105271642f41eef99a8f601699 100644 (file)
@@ -35,7 +35,7 @@ class CRM_Contact_Form_Search_Custom_FullText extends CRM_Contact_Form_Search_Cu
   const LIMIT = 10;
 
   /**
-   * @var array CRM_Contact_Form_Search_Custom_FullText_AbstractPartialQuery
+   * @var CRM_Contact_Form_Search_Custom_FullText_AbstractPartialQuery[]
    */
   protected $_partialQueries = NULL;
 
@@ -54,23 +54,37 @@ class CRM_Contact_Form_Search_Custom_FullText extends CRM_Contact_Form_Search_Cu
   protected $_tableFields = NULL;
 
   /**
-   * @var array|null NULL if no limit; or array(0 => $limit, 1 => $offset)
+   * Limit clause.
+   *
+   * NULL if no limit; or array(0 => $limit, 1 => $offset).
+   *
+   * @var array|null
    */
   protected $_limitClause = NULL;
 
   /**
-   * @var array|null NULL if no limit; or array(0 => $limit, 1 => $offset)
+   * Limit row clause.
+   *
+   * NULL if no limit; or array(0 => $limit, 1 => $offset)
+   *
+   * @var array|null
    */
   protected $_limitRowClause = NULL;
 
   /**
-   * @var array|null NULL if no limit; or array(0 => $limit, 1 => $offset)
+   * Limit detail clause.
+   *
+   * NULL if no limit; or array(0 => $limit, 1 => $offset).
+   *
+   * @var array|null
    */
   protected $_limitDetailClause = NULL;
 
   protected $_limitNumber = 10;
+
   /**
-   * this should be one more than self::LIMIT
+   * This should be one more than self::LIMIT.
+   *
    * @var int
    */
   protected $_limitNumberPlus1 = 11;
index 76c7efbc369a031982803f89150eb8a1918d913d..92c4ddd71b9687e4057675e68f65d6a3eceac42f 100644 (file)
@@ -153,11 +153,12 @@ class CRM_Contact_Form_Search_Custom_Group extends CRM_Contact_Form_Search_Custo
   /**
    * @param int $offset
    * @param int $rowcount
-   * @param NULL $sort
+   * @param string $sort
    * @param bool $includeContactIDs
    * @param bool $justIDs
    *
    * @return string
+   * @throws \Exception
    */
   public function all(
     $offset = 0, $rowcount = 0, $sort = NULL,
@@ -587,10 +588,11 @@ WHERE  gcc.group_id = {$ssGroup->id}
   /**
    * @param int $offset
    * @param int $rowcount
-   * @param NULL $sort
+   * @param string $sort
    * @param bool $returnSQL
    *
    * @return string
+   * @throws \Exception
    */
   public function contactIDs($offset = 0, $rowcount = 0, $sort = NULL, $returnSQL = FALSE) {
     return $this->all($offset, $rowcount, $sort, FALSE, TRUE);
index c9d41c2cb1b101abacf3b7e412d3b8030329c089..827eeff9fa34ea4080df9dd4e6f0d456f02fbaf4 100644 (file)
@@ -37,18 +37,20 @@ abstract class CRM_Contact_Import_Parser extends CRM_Import_Parser {
   /**
    * Total number of lines in file
    *
-   * @var integer
+   * @var int
    */
   protected $_rowCount;
 
   /**
    * Running total number of un-matched Contacts.
+   *
    * @var int
    */
   protected $_unMatchCount;
 
   /**
-   * Array of unmatched lines
+   * Array of unmatched lines.
+   *
    * @var array
    */
   protected $_unMatch;
@@ -1354,7 +1356,7 @@ abstract class CRM_Contact_Import_Parser extends CRM_Import_Parser {
           'return' => 'location_type_id',
           'contact_id' => $contactID,
           'is_primary' => 1,
-          'sequential' => 1
+          'sequential' => 1,
         ]);
       }
       $defaultLocationType = CRM_Core_BAO_LocationType::getDefault();
index 9ea30c409b34f6be0084002a6c05e61788a366da..a567356a53cf35afa016939fd710e37a58f9be28 100644 (file)
@@ -32,7 +32,9 @@
  */
 class CRM_Contact_Page_ImageFile extends CRM_Core_Page {
   /**
-   * @var int Time to live (seconds).
+   * Time to live (seconds).
+   *
+   * @var int
    *
    * 12 hours: 12 * 60 * 60 = 43200
    */
index d018f62402ddce7defaed0a8f76fa3904c1ce007..0c323100a45e18da78aa279173bc653dd67587e3 100644 (file)
 class CRM_Contact_Page_View_ContactSmartGroup extends CRM_Core_Page {
 
   /**
-   * @var int contact id
+   * Contact id.
+   *
+   * @var int
    */
   public $_contactId;
 
   /**
-   * called when action is browse.
-   *
+   * Called when action is browse.
    */
   public function browse() {
     $in = CRM_Contact_BAO_GroupContact::getContactGroup($this->_contactId, 'Added');
index f426a800e82104eba3ec835a0c9a5e5072fbb0d1..081c9a031512d2a6a6420c5368f80ad00deefc52 100644 (file)
@@ -132,8 +132,9 @@ class CRM_Contribute_Form_Contribution extends CRM_Contribute_Form_AbstractEditP
   public $_fromEmails;
 
   /**
-   * ID of from email
-   * @var integer
+   * ID of from email.
+   *
+   * @var int
    */
   public $fromEmailId;
 
@@ -171,8 +172,9 @@ class CRM_Contribute_Form_Contribution extends CRM_Contribute_Form_AbstractEditP
   public $userEmail;
 
   /**
-   * Price set ID
-   * @var integer
+   * Price set ID.
+   *
+   * @var int
    */
   public $_priceSetId;
 
index 28ff5f2aaf90daba78155a6821eff13248d18d3f..18fcdc93be35905f83303c9fa8bcfb41641d7069 100644 (file)
@@ -118,7 +118,7 @@ class CRM_Contribute_Form_ContributionBase extends CRM_Core_Form {
   /**
    * Pcp id
    *
-   * @var integer
+   * @var int
    */
   public $_pcpId;
 
index f93583590f0fc5511fc88b3823b306afd2b44105..7ce505434d166204c961d239c5357a8f7e62d278 100644 (file)
@@ -41,7 +41,7 @@ class CRM_Core_Action {
    * Different possible actions are defined here. Keep in sync with the
    * constant from CRM_Core_Form for various modes.
    *
-   * @var integer const
+   * @var int
    */
   const
     NONE = 0,
index 7193515cf996fd9835d1b3970d887ac7e3ba693c..74e619b7fb5bc9270f6cddcd4dbf091ceb09c33a 100644 (file)
@@ -37,12 +37,8 @@ abstract class CRM_Custom_Import_Parser extends CRM_Contact_Import_Parser {
   protected $_fileName;
 
   /**
-   * #@+
-   * @var integer
-   */
-
-  /**
-   * Imported file size
+   * Imported file size.
+   *
    * @var int
    */
   protected $_fileSize;
index d432deadf431f97e2da7459964ce6062fe08b580..9e065f4cc16612d808b96c1912248e58feae6fd9 100644 (file)
@@ -65,8 +65,9 @@ class CRM_Event_Form_ManageEvent extends CRM_Core_Form {
   protected $_isTemplate = FALSE;
 
   /**
-   * Pre-populate fields based on this template event_id
-   * @var integer
+   * Pre-populate fields based on this template event_id.
+   *
+   * @var int
    */
   protected $_templateId;
 
index 2684b39fd805081193b2a4bcdbae9edbabd1a04e..080ecb98c4d561fde3adbe2e77b36937f5a4963b 100644 (file)
@@ -69,12 +69,14 @@ class CRM_Event_Form_Registration_Register extends CRM_Event_Form_Registration {
   /**
    * Show fee block or not.
    *
-   * @var bool determines if fee block should be shown or hidden
+   * @var bool
    */
   public $_noFees;
 
   /**
-   * @var array Fee Block
+   * Fee Block.
+   *
+   * @var array
    */
   public $_feeBlock;
 
index 5276b02c1a8f0b95aa3e15431469a13d73191c83..434f279614108370358d9b181c2b35dcfefcd1ed 100644 (file)
@@ -37,24 +37,22 @@ abstract class CRM_Event_Import_Parser extends CRM_Import_Parser {
   protected $_fileName;
 
   /**
-   * #@+
-   * @var integer
-   */
-
-  /**
-   * Imported file size
+   * Imported file size.
+   *
    * @var int
    */
   protected $_fileSize;
 
   /**
-   * Seperator being used
+   * Separator being used.
+   *
    * @var string
    */
   protected $_seperator;
 
   /**
-   * Total number of lines in file
+   * Total number of lines in file.
+   *
    * @var int
    */
   protected $_lineCount;
index 9deb8ea265545e01b3d6506995834fd3fc0db29c..f51cc0f2e25f81ce44547708aac0bc73b1982ac4 100644 (file)
@@ -51,7 +51,7 @@ class CRM_Financial_Form_Payment extends CRM_Core_Form {
   public $isBackOffice = FALSE;
 
   /**
-   * @var String
+   * @var string
    */
   public $_formName = '';
 
index b06d61cf34e3046f9005998a22e06025d6bd98e5..a6f03660f2be884beae482f9fa07989ccfb9215b 100644 (file)
@@ -37,7 +37,9 @@
 class CRM_Mailing_Page_View extends CRM_Core_Page {
 
   /**
-   * @var Signal to Flexmailer that this version of the class is usable.
+   * Signal to Flexmailer that this version of the class is usable.
+   *
+   * @var bool
    */
   const USES_MAILING_PREVIEW_API = 1;
 
index bf1cffec4dff53a5f6874249752bdcd90d7ebafc..3edd35bb547186ad97a64f5114aff3220d638e87 100644 (file)
@@ -36,11 +36,6 @@ abstract class CRM_Member_Import_Parser extends CRM_Import_Parser {
 
   protected $_fileName;
 
-  /**
-   * #@+
-   * @var integer
-   */
-
   /**
    * Imported file size
    * @var int
index 532bd0e022c6965b36189845e75943a5dfa1f5f9..f12c915cf7be624d5c9d49144aae7c5606e2809e 100644 (file)
@@ -49,14 +49,14 @@ class CRM_Report_Form extends CRM_Core_Form {
   /**
    * The id of the report instance
    *
-   * @var integer
+   * @var int
    */
   protected $_id;
 
   /**
    * The id of the report template
    *
-   * @var integer;
+   * @var int
    */
   protected $_templateID;
 
@@ -280,14 +280,15 @@ class CRM_Report_Form extends CRM_Core_Form {
    * when $_output mode is not 'html' or 'group' so as not to have to interpret / mess with that part
    * of the code (see limit() fn.
    *
-   * @var integer
+   * @var int
    */
   protected $_limitValue = NULL;
 
   /**
    * This can be set to specify row offset
    * See notes on _limitValue
-   * @var integer
+   *
+   * @var int
    */
   protected $_offsetValue = NULL;
   /**
@@ -422,33 +423,43 @@ class CRM_Report_Form extends CRM_Core_Form {
   public $_section;
 
   /**
-   * @var string Report description.
+   * Report description.
+   *
+   * @var string
    */
   public $_description;
 
   /**
-   * @var bool Is an address field selected.
+   * Is an address field selected.
+   *
+   * @var bool
    *   This was intended to determine if the address table should be joined in
    *   The isTableSelected function is now preferred for this purpose
    */
   protected $_addressField;
 
   /**
-   * @var bool Is an email field selected.
+   * Is an email field selected.
+   *
+   * @var bool
    *   This was intended to determine if the email table should be joined in
    *   The isTableSelected function is now preferred for this purpose
    */
   protected $_emailField;
 
   /**
-   * @var bool Is a phone field selected.
+   * Is a phone field selected.
+   *
+   * @var bool
    *   This was intended to determine if the phone table should be joined in
    *   The isTableSelected function is now preferred for this purpose
    */
   protected $_phoneField;
 
   /**
-   * @var bool Create new report instance? (or update existing) on save.
+   * Create new report instance? (or update existing) on save.
+   *
+   * @var bool
    */
   protected $_createNew;
 
@@ -456,12 +467,15 @@ class CRM_Report_Form extends CRM_Core_Form {
    *  When a grand total row has calculated the status we pop it off to here.
    *
    * This allows us to access it from the stats function and avoid recalculating.
+   *
    * @var array
    */
   protected $rollupRow = [];
 
   /**
-   * @var string Database attributes - character set and collation
+   * Database attributes - character set and collation.
+   *
+   * @var string
    */
   protected $_databaseAttributes = ' DEFAULT CHARACTER SET utf8 COLLATE utf8_unicode_ci';
 
index c6b1557ca1d4d769010633c78722b08bdcf80e77..d6188fc8f0e19114386bfded29b8d3cc16903e87 100644 (file)
 class CRM_SMS_Message {
 
   /**
-   * @var String
    * What address is this SMS message coming from.
+   *
+   * @var string
    */
   public $from = '';
 
 
   /**
-   * @var String
    * What address is this SMS message going to.
+   *
+   * @var string
    */
   public $to = '';
 
   /**
-   * @var Integer
-   * Contact ID that is matched to the From address
+   * Contact ID that is matched to the From address.
+   *
+   * @var int
    */
   public $fromContactID = NULL;
 
   /**
-   * @var Integer
-   * Contact ID that is matched to the To address
+   * Contact ID that is matched to the To address.
+   *
+   * @var int
    */
   public $toContactID = NULL;
 
   /**
-   * @var String
-   * Body content of the message
+   * Body content of the message.
+   *
+   * @var string
    */
   public $body = '';
 
   /**
-   * @var Integer
-   * Trackable ID in the system to match to
+   * Trackable ID in the system to match to.
+   *
+   * @var int
    */
   public $trackID = NULL;
 
index d8bd4c92324b95e4c9eec2245a7157fa4e7dae76..0f0eb5ecde4ca899bb752eb6bfea8cccaad2ad9e 100644 (file)
@@ -82,7 +82,7 @@ class CRM_UF_Form_Field extends CRM_Core_Form {
    * Is this profile has searchable field.
    * or is any field having in selector true.
    *
-   * @var bool.
+   * @var bool
    */
   protected $_hasSearchableORInSelector;
 
index 5145be10721100e7bb5c344a2ef6f1c3c78ca89b..48d8ed49ffbfea2b88c359512c3fceb15cf14d54 100644 (file)
@@ -149,7 +149,8 @@ class CRM_Upgrade_Incremental_Base {
    * @param string $column
    * @param string $properties
    * @param bool $localizable is this a field that should be localized
-   * @param string|NULL $version CiviCRM version to use if rebuilding multilingual schema
+   * @param string|null $version CiviCRM version to use if rebuilding multilingual schema
+   *
    * @return bool
    */
   public static function addColumn($ctx, $table, $column, $properties, $localizable = FALSE, $version = NULL) {
@@ -281,7 +282,8 @@ class CRM_Upgrade_Incremental_Base {
   /**
    * Rebuild Multilingual Schema.
    * @param CRM_Queue_TaskContext $ctx
-   * @param string|NULL $version CiviCRM version to use if rebuilding multilingual schema
+   * @param string|null $version CiviCRM version to use if rebuilding multilingual schema
+   *
    * @return bool
    */
   public static function rebuildMultilingalSchema($ctx, $version = NULL) {
index 1c2a447a58fe8592befebc8ad759a4f9fd31cc25..c844fd3cc04b2b9a31b7d13e5fd0233c0a0d6dbd 100644 (file)
@@ -37,7 +37,7 @@ class Main extends \CRM_Core_Page {
   /**
    * The region of the page into which JavaScript will be loaded.
    *
-   * @var String
+   * @var string
    * @deprecated
    */
   public $region;
index 726975dd695a159d2525447ab3514243a4cf3876..1bd225198d72bfe39da7405958f8f5fbd1328fb0 100644 (file)
@@ -34,36 +34,42 @@ class CRM_Batch_Form_EntryTest extends CiviUnitTestCase {
 
   /**
    * Membership type name used in test function.
-   * @var String
+   *
+   * @var string
    */
   protected $_membershipTypeName = NULL;
 
   /**
    * Membership type id used in test function.
-   * @var String
+   *
+   * @var string
    */
   protected $_membershipTypeID = NULL;
 
   /**
    * Contact id used in test function.
-   * @var String
+   *
+   * @var string
    */
   protected $_contactID = NULL;
   /**
    * Contact id used in test function.
-   * @var String
+   *
+   * @var string
    */
   protected $_contactID2 = NULL;
 
   /**
    * Contact id used in test function.
-   * @var String
+   *
+   * @var string
    */
   protected $_contactID3 = NULL;
 
   /**
    * Contact id used in test function.
-   * @var String
+   *
+   * @var string
    */
   protected $_contactID4 = NULL;
 
index 1d6104dd967377ac25697bc2efae0b673e67d9ec..a25441ffbaee1c0c141d2b9765467f705153b8e6 100644 (file)
@@ -38,7 +38,9 @@
 class CRM_Contact_BAO_QueryTestDataProvider implements Iterator {
 
   /**
-   * @var integer
+   * Current count.
+   *
+   * @var int
    */
   private $i = 0;
 
index 2c8234d22d671b87ac29708f7f99ff59e92e4189..8ce25c7a5cf56e4759b6006bda4c0178f4b3b35b 100644 (file)
@@ -37,7 +37,9 @@
 class CRM_Contact_Form_Search_Custom_GroupTestDataProvider implements Iterator {
 
   /**
-   * @var integer
+   * Current count.
+   *
+   * @var int
    */
   private $i = 0;
 
index d9249d5fe5a67b1d0ffe1080544983385905342d..d3fa7eeceedf5aa0b980fbe95f0ef907d3393f24 100644 (file)
@@ -33,7 +33,9 @@
 class CRM_Contact_Form_Search_Custom_SampleTestDataProvider implements Iterator {
 
   /**
-   * @var integer
+   * Current count.
+   *
+   * @var int
    */
   private $i = 0;
 
index fd69fd866e5155bf5deee4f5f902f905bd11d6dd..d05371a6ed498d24efcb59fd2895507750933a4d 100644 (file)
@@ -7,8 +7,9 @@
 class CRM_Group_Page_AjaxTest extends CiviUnitTestCase {
   /**
    * Permissioned group is used both as an active group the contact can see and as a group that allows
-   * logged in user to see contacts
-   * @var integer
+   * logged in user to see contacts.
+   *
+   * @var int
    */
   protected $_permissionedGroup;
   /**
index 37c47774e51de9e846a7daffa94335cc0fa61caa..5a24b3c4360c1140cc02711d63376345709895f4 100644 (file)
@@ -8,7 +8,7 @@
 class CRM_Mailing_BAO_QueryTestDataProvider implements Iterator {
 
   /**
-   * @var integer
+   * @var int
    */
   private $i = 0;
 
index c39b1618d561651433fb27b3a39341612e39c12f..17b6ec9ef9463fda4a6bd09ff798e49352cf04c5 100644 (file)
 class CRM_Member_Import_Parser_MembershipTest extends CiviUnitTestCase {
   /**
    * Membership type name used in test function.
-   * @var String
+   *
+   * @var string
    */
   protected $_membershipTypeName = NULL;
 
   /**
    * Membership type id used in test function.
-   * @var String
+   *
+   * @var string
    */
   protected $_membershipTypeID = NULL;
 
index 1fdc04b337ce74476a2a58ab0f22cd3d8c30c31e..db08d268cbc208d919685d715b724db0c99ba278 100644 (file)
@@ -37,7 +37,8 @@ class api_v3_RelationshipTest extends CiviUnitTestCase {
   protected $_cId_a;
   /**
    * Second individual.
-   * @var integer
+   *
+   * @var int
    */
   protected $_cId_a_2;
   protected $_cId_b;
index c85d7427199fafb8c2e439a030ec96397ce283aa..e5dd292073f7c513bbcdb11ee0efe3dab4ddecc0 100644 (file)
@@ -41,7 +41,8 @@ class api_v3_TagTest extends CiviUnitTestCase {
   protected $ids = array();
   /**
    * Tag id.
-   * @var integer
+   *
+   * @var int
    */
   protected $tag = array();