various comment fixes
authorEileen McNaughton <eileen@fuzion.co.nz>
Mon, 2 Feb 2015 09:04:32 +0000 (22:04 +1300)
committerEileen McNaughton <eileen@fuzion.co.nz>
Tue, 3 Feb 2015 00:58:37 +0000 (13:58 +1300)
46 files changed:
CRM/Contribute/BAO/Contribution/Utils.php
CRM/Contribute/BAO/ContributionPage.php
CRM/Contribute/BAO/Query.php
CRM/Contribute/BAO/Widget.php
CRM/Contribute/Controller/Contribution.php
CRM/Contribute/Form/Contribution/Confirm.php
CRM/Contribute/Form/Contribution/Main.php
CRM/Contribute/Form/ContributionBase.php
CRM/Contribute/Form/Task/PDF.php
CRM/Contribute/Selector/Search.php
Civi/CCase/Analyzer.php
Civi/CCase/CaseChangeListener.php
Civi/CCase/Event/CaseChangeEvent.php
Civi/CCase/Events.php
Civi/CCase/SequenceListener.php
Civi/CiUtil/Arrays.php
Civi/CiUtil/Command/AntagonistCommand.php
Civi/CiUtil/Command/CompareCommand.php
Civi/CiUtil/Command/LsCommand.php
Civi/CiUtil/ComparisonPrinter.php
Civi/CiUtil/CsvPrinter.php
Civi/CiUtil/EnvTestRunner.php
Civi/Core/DAO/Event/PostDelete.php
Civi/Core/DAO/Event/PostUpdate.php
Civi/Core/Event/PostEvent.php
Civi/Core/Event/PreEvent.php
Civi/Core/Event/UnhandledExceptionEvent.php
Civi/Core/Transaction/Frame.php
tests/phpunit/CRM/Contribute/BAO/ContributionRecurTest.php
tests/phpunit/CRM/Core/InnoDBIndexerTest.php
tests/phpunit/CRM/Core/MenuTest.php
tests/phpunit/CRM/Core/Page/RedirectTest.php
tests/phpunit/CRM/Core/TransactionTest.php
tests/phpunit/CRM/Utils/HookTest.php
tests/phpunit/CRM/Utils/HtmlToTextTest.php
tests/phpunit/CRM/Utils/StringTest.php
tests/phpunit/CRM/Utils/TimeTest.php
tests/phpunit/CRM/Utils/versionCheckTest.php
tests/phpunit/Civi/API/Subscriber/DynamicFKAuthorizationTest.php
tests/phpunit/Civi/API/Subscriber/TransactionSubscriberTest.php
tests/phpunit/Civi/CCase/SequenceListenerTest.php
tests/phpunit/CiviTest/CiviSeleniumTestCase.php
tests/phpunit/WebTest/Activity/AddRecurringActivityTest.php
tests/phpunit/WebTest/Event/AddRecurringEventTest.php
tests/phpunit/WebTest/Pledge/StandaloneAddTest.php
tests/phpunit/api/v3/NoteTest.php

index 1abae585be55f93073b7a1113baa498b6e5e9054..d76dc8badd12058bbaff2c4206332eea35d2a9f1 100644 (file)
@@ -30,7 +30,6 @@
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2014
  * $Id$
- *
  */
 class CRM_Contribute_BAO_Contribution_Utils {
 
index c95cb01373866ccfd549fee3ddfb4c716b67497c..4d9647ecf610d02066f403320f3abf0e632dc740 100644 (file)
@@ -924,7 +924,8 @@ LEFT JOIN  civicrm_premiums            ON ( civicrm_premiums.entity_id = civicrm
   }
 
   /**
-   * Helper to determine if the page supports separate membership payments
+   * Helper to determine if the page supports separate membership payments.
+   *
    * @param int $id form id
    *
    * @return bool
index 6b915f2453ed69a5670cbbf5bb5323dc16fad74e..dd0693dd6b04cccc4ac3d378b4f643f1224f1640 100644 (file)
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2014
  * $Id$
- *
  */
 class CRM_Contribute_BAO_Query {
 
   /**
-   * Static field for all the export/import contribution fields
+   * Static field for all the export/import contribution fields.
    *
    * @var array
    */
@@ -63,7 +62,7 @@ class CRM_Contribute_BAO_Query {
   }
 
   /**
-   * If contributions are involved, add the specific contribute fields
+   * If contributions are involved, add the specific contribute fields.
    *
    * @param $query
    *
index c5dc7a984ebae0330b0a5e410859d415a138d1f2..d0f34918c4c92af74826472239da472b7fc10f64 100644 (file)
@@ -30,7 +30,6 @@
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2014
  * $Id$
- *
  */
 
 /**
index 6d302d84133de4a49fc33391f10c908eb7af6c2d..079e0af1bf78c81c24556829aa5e31b0036dd2b4 100644 (file)
  *
  * Typically the first form will display the search criteria and it's results
  *
- * The second form is used to process search results with the asscociated actions
+ * The second form is used to process search results with the associated actions
  *
  */
 class CRM_Contribute_Controller_Contribution extends CRM_Core_Controller {
 
   /**
-   * Class constructor
+   * Class constructor.
+   *
+   * @param string $title
+   * @param bool|int $action
+   * @param bool $modal
    */
   public function __construct($title = NULL, $action = CRM_Core_Action::NONE, $modal = TRUE) {
     parent::__construct($title, $modal);
index 31ca13b77efe23b1de3f5eb95787c75041a9ed6f..ffd6181f9d8b2d79fa6b8cf36f74e61f9d1c62a6 100644 (file)
@@ -1526,7 +1526,7 @@ class CRM_Contribute_Form_Contribution_Confirm extends CRM_Contribute_Form_Contr
   }
 
   /**
-   * Create the recurring contribution record
+   * Create the recurring contribution record.
    *
    * @param CRM_Core_Form $form
    * @param array $params
@@ -1614,7 +1614,7 @@ class CRM_Contribute_Form_Contribution_Confirm extends CRM_Contribute_Form_Contr
   }
 
   /**
-   * Add on behalf of organization and it's location
+   * Add on behalf of organization and it's location.
    *
    * @param array $behalfOrganization
    *   array of organization info.
@@ -1626,8 +1626,6 @@ class CRM_Contribute_Form_Contribution_Confirm extends CRM_Contribute_Form_Contr
    *   form values array.
    * @param array $params
    * @param null $fields
-   *
-   * @return void
    */
   public static function processOnBehalfOrganization(&$behalfOrganization, &$contactID, &$values, &$params, $fields = NULL) {
     $isCurrentEmployer = FALSE;
@@ -1726,16 +1724,16 @@ class CRM_Contribute_Form_Contribution_Confirm extends CRM_Contribute_Form_Contr
   }
 
   /**
-   * Function used to save pcp / soft credit entry
+   * Function used to save pcp / soft credit entry.
+   *
    * This is used by contribution and also event pcps
    *
    * @param array $params
    * @param object $contribution
    *   Contribution object.
-   *
    */
   public static function processPcpSoft(&$params, &$contribution) {
-    //add soft contribution due to pcp or Submit Credit / Debit Card Contribution by admin.
+    // Add soft contribution due to pcp or Submit Credit / Debit Card Contribution by admin.
     if (!empty($params['soft_credit_to'])) {
       $contributionSoftParams = array();
       foreach (array(
index 243daceabab84b8ecbdd3a7acf558a3fb44f58b6..ac4c167c606ab183f12c62d5d6a73ae3861568b6 100644 (file)
@@ -732,7 +732,6 @@ class CRM_Contribute_Form_Contribution_Main extends CRM_Contribute_Form_Contribu
    *   The uploaded files if any.
    * @param $self
    *
-   *
    * @return bool|array
    *   true if no errors, else array of errors
    */
@@ -1398,8 +1397,8 @@ class CRM_Contribute_Form_Contribution_Main extends CRM_Contribute_Form_Contribu
   }
 
   /**
-   * Handle Payment Processor switching
-   * For contribution and event registration forms
+   * Handle Payment Processor switching for contribution and event registration forms.
+   *
    * @param CRM_Contribute_Form_Contribution_Main|CRM_Event_Form_Registration_Register $form
    * @param bool $noFees
    */
index ef5d36563689eca61e16a71a2ee54be6f5cf4c91..af11f4cf41fcaf1b8fb918a5423b13d8ef1c1244 100644 (file)
@@ -200,8 +200,8 @@ class CRM_Contribute_Form_ContributionBase extends CRM_Core_Form {
   /**
    * Set variables up before form is built
    *
-   * @throws CRM_Contribution_Exception_InactiveContributionPageException
-   * @return void
+   * @throws \CRM_Contribute_Exception_InactiveContributionPageException
+   * @throws \Exception
    */
   public function preProcess() {
 
index 582db29cc36590e7767147591118c286eccbfb68..c9abc96453d7fdcebd60e5868c8776158983fe44 100644 (file)
@@ -29,7 +29,6 @@
  *
  * @package CRM
  * @copyright CiviCRM LLC (c) 2004-2014
- *
  */
 
 /**
index 1bd7aa97cc1da8730b1b7bec6de9e61eb8f197a2..ed22579c57b9bd8e4853903fbfae977ce286f82d 100644 (file)
@@ -280,7 +280,7 @@ class CRM_Contribute_Selector_Search extends CRM_Core_Selector_Base implements C
   /**
    * Returns total number of rows for the query.
    *
-   * @param
+   * @param string $action
    *
    * @return int
    *   Total number of rows
index b6ae7446891b0a96d6430d2a154f1ffdb3b8ec93..28b9b371ac2d8d37bc95a772d2a9260d24040d60 100644 (file)
  */
 namespace Civi\CCase;
 
+/**
+ * Class Analyzer
+ *
+ * @package Civi\CCase
+ */
 class Analyzer {
   /**
    * @var int
@@ -57,6 +62,9 @@ class Analyzer {
    */
   private $indices;
 
+  /**
+   * @param $caseId
+   */
   public function __construct($caseId) {
     $this->caseId = $caseId;
     $this->flush();
index d54bb60d0ce88c3b1c7ec6beed528148b980a4b5..70b12e0870eefa107be7911a32c4156816804c91 100644 (file)
  */
 namespace Civi\CCase;
 
+/**
+ * Interface CaseChangeListener
+ *
+ * @package Civi\CCase
+ */
 interface CaseChangeListener {
+  /**
+   * @param \Civi\CCase\Event\CaseChangeEvent $event
+   *
+   * @return mixed
+   */
   public function onCaseChange(\Civi\CCase\Event\CaseChangeEvent $event);
 
 }
index 2112d41c807ddb7a7c9457af290db5a1368a745f..270f2861b9e8849efe6c824330ec262acd217f6c 100644 (file)
@@ -37,6 +37,9 @@ class CaseChangeEvent extends \Symfony\Component\EventDispatcher\Event {
    */
   public $analyzer;
 
+  /**
+   * @param $analyzer
+   */
   public function __construct($analyzer) {
     $this->analyzer = $analyzer;
   }
index b31adcded567a7fe54563288aaf291c09a0f706b..db794dbd96b2763199ee67445ee19ab91e9028da 100644 (file)
  */
 namespace Civi\CCase;
 
+/**
+ * Class Events
+ *
+ * @package Civi\CCase
+ */
 class Events {
   /**
    * @var array (int $caseId => bool $active) list of cases for which we are actively firing case-change event
@@ -74,6 +79,9 @@ class Events {
     }
   }
 
+  /**
+   * @param $caseId
+   */
   public static function fireCaseChangeForRealz($caseId) {
     if (!isset(self::$isActive[$caseId])) {
       $tx = new \CRM_Core_Transaction();
index d2d9858035de4fed0db85813e4a99fe955e7a9b0..5d40b49628b609fd54d176343c1a69df06b42ca0 100644 (file)
@@ -25,10 +25,18 @@ class SequenceListener implements CaseChangeListener {
     return self::$singleton;
   }
 
+  /**
+   * @param \Civi\CCase\Event\CaseChangeEvent $event
+   */
   public static function onCaseChange_static(\Civi\CCase\Event\CaseChangeEvent $event) {
     self::singleton()->onCaseChange($event);
   }
 
+  /**
+   * @param \Civi\CCase\Event\CaseChangeEvent $event
+   *
+   * @throws \CiviCRM_API3_Exception
+   */
   public function onCaseChange(\Civi\CCase\Event\CaseChangeEvent $event) {
     /** @var \Civi\CCase\Analyzer $analyzer */
     $analyzer = $event->analyzer;
index 2c28c1383c74d5cb893e166037880940e7106efe..f34954c1d339b00df6d00571abf4766d72cf5128 100644 (file)
@@ -1,7 +1,18 @@
 <?php
 namespace Civi\CiUtil;
 
+/**
+ * Class Arrays
+ *
+ * @package Civi\CiUtil
+ */
 class Arrays {
+  /**
+   * @param $arr
+   * @param $col
+   *
+   * @return array
+   */
   public static function collect($arr, $col) {
     $r = array();
     foreach ($arr as $k => $item) {
index 46441305b35f2605e6872a48f4dac5c54b58382a..1f9e9335f669f49bfed049628262d4d86f639f47 100644 (file)
@@ -1,7 +1,15 @@
 <?php
 namespace Civi\CiUtil\Command;
 
+/**
+ * Class AntagonistCommand
+ *
+ * @package Civi\CiUtil\Command
+ */
 class AntagonistCommand {
+  /**
+   * @param $argv
+   */
   public static function main($argv) {
     if (count($argv) != 3) {
       print "usage: {$argv[0]} <TargetTest::testFunc> </path/to/suite>\n";
index 03c8b8d72a52514a38131af3fcfb419c787bdaed..ae8bd6d3f02647286d56636a3e6598591ba4a8ad 100644 (file)
@@ -1,7 +1,15 @@
 <?php
 namespace Civi\CiUtil\Command;
 
+/**
+ * Class CompareCommand
+ *
+ * @package Civi\CiUtil\Command
+ */
 class CompareCommand {
+  /**
+   * @param $argv
+   */
   public static function main($argv) {
     if (empty($argv[1])) {
       echo "summary: Compares the output of different test runs\n";
index 8cdb679b052f91a6be35693ef11247f3a0fe6671..88f0d84947e30db2755c4e40193e7a42bd656bf6 100644 (file)
@@ -7,6 +7,9 @@ namespace Civi\CiUtil\Command;
  * @package Civi\CiUtil\Command
  */
 class LsCommand {
+  /**
+   * @param $argv
+   */
   public static function main($argv) {
     $paths = $argv;
     array_shift($paths);
index bb75ad68ac841430af5dc21597126899b8f79970..a58a83eb6704af9b2d420099d1d623ea1dbc37e6 100644 (file)
@@ -1,10 +1,18 @@
 <?php
 namespace Civi\CiUtil;
 
+/**
+ * Class ComparisonPrinter
+ *
+ * @package Civi\CiUtil
+ */
 class ComparisonPrinter {
   var $headers;
   var $hasHeader = FALSE;
 
+  /**
+   * @param $headers
+   */
   public function __construct($headers) {
     $this->headers = $headers;
   }
@@ -35,6 +43,10 @@ class ComparisonPrinter {
     $this->hasHeader = TRUE;
   }
 
+  /**
+   * @param $test
+   * @param $values
+   */
   public function printRow($test, $values) {
     $this->printHeader();
     printf("%-90s ", $test);
index 9fe7d97fef50b7bfe538dd4f4eaacc2350d1e63f..4c4ec185a3efbd5d542725af792503a1549dbbbf 100644 (file)
@@ -1,11 +1,20 @@
 <?php
 namespace Civi\CiUtil;
 
+/**
+ * Class CsvPrinter
+ *
+ * @package Civi\CiUtil
+ */
 class CsvPrinter {
   var $file;
   var $headers;
   var $hasHeader = FALSE;
 
+  /**
+   * @param $file
+   * @param $headers
+   */
   public function __construct($file, $headers) {
     $this->file = fopen($file, "w");
     $this->headers = $headers;
@@ -23,6 +32,10 @@ class CsvPrinter {
     $this->hasHeader = TRUE;
   }
 
+  /**
+   * @param $test
+   * @param $values
+   */
   public function printRow($test, $values) {
     $this->printHeader();
     $row = $values;
index 6ab005fc489fbd6ed9166a36fd40e0607e78fc41..f5e0502758cf45b9df5fbf5e7b2ee623f575c058 100644 (file)
@@ -8,6 +8,10 @@ class EnvTestRunner {
   protected $phpunit;
   protected $envTestSuite;
 
+  /**
+   * @param string $phpunit
+   * @param string $envTestSuite
+   */
   public function __construct($phpunit = "phpunit", $envTestSuite = 'EnvTests') {
     $this->phpunit = $phpunit;
     $this->envTestSuite = $envTestSuite;
index 5ba051a7e2c3d8cfa00eb7eb8f689439fef9a4f6..b231d42c44da1919792d1e93c8528167d0adf024 100644 (file)
@@ -43,6 +43,10 @@ class PostDelete extends \Symfony\Component\EventDispatcher\Event {
    */
   public $result;
 
+  /**
+   * @param $object
+   * @param $result
+   */
   public function __construct($object, $result) {
     $this->object = $object;
     $this->result = $result;
index fcbd6081bbb4c2e469b5190e65dab56172212fb1..72dbeab2d9478b2dbf61e9c42cfb3f7102578aef 100644 (file)
@@ -38,6 +38,9 @@ class PostUpdate extends \Symfony\Component\EventDispatcher\Event {
    */
   public $object;
 
+  /**
+   * @param $object
+   */
   public function __construct($object) {
     $this->object = $object;
   }
index d71de10b68418d3fe50dc7b6681b7642a2c84aab..879f66c091d616cd641e7c5b2c0cc07589d3d037 100644 (file)
@@ -53,6 +53,12 @@ class PostEvent extends \Symfony\Component\EventDispatcher\Event {
    */
   public $object;
 
+  /**
+   * @param $action
+   * @param $entity
+   * @param $id
+   * @param $object
+   */
   public function __construct($action, $entity, $id, $object) {
     $this->action = $action;
     $this->entity = $entity;
index 8a59f8847a8215f3c22a00e672d63bbccddfa279..e8e15eb07ecdfa3c2155e27a0d5494db5366a9ea 100644 (file)
@@ -53,6 +53,12 @@ class PreEvent extends \Symfony\Component\EventDispatcher\Event {
    */
   public $params;
 
+  /**
+   * @param $action
+   * @param $entity
+   * @param $id
+   * @param $params
+   */
   public function __construct($action, $entity, $id, $params) {
     $this->action = $action;
     $this->entity = $entity;
index 547f882f7c2a48cfbce12fee9e6e08a0c6bb6735..42270723b0ad494e517673ba8d5bf74d04573376 100644 (file)
@@ -43,6 +43,10 @@ class UnhandledExceptionEvent extends \Symfony\Component\EventDispatcher\Event {
    */
   public $request;
 
+  /**
+   * @param $e
+   * @param $request
+   */
   public function __construct($e, $request) {
     $this->request = $request;
     $this->exception = $e;
index cb085c3ba40485514c2b295670bc367b3b2134f1..cf7fb75c92ed4028ea6d8aa872bdbc6fe070af1a 100644 (file)
@@ -99,10 +99,16 @@ class Frame {
     $this->refCount--;
   }
 
+  /**
+   * @return bool
+   */
   public function isEmpty() {
     return ($this->refCount == 0);
   }
 
+  /**
+   * @return bool
+   */
   public function isRollbackOnly() {
     return !$this->doCommit;
   }
index 08344100cb96e201fac5fe99a289fe16b5b793ae..5f36ceef306195fbdb8170f739ae9ec4950870c3 100644 (file)
@@ -27,6 +27,9 @@
 
 require_once 'CiviTest/CiviUnitTestCase.php';
 
+/**
+ * Class CRM_Contribute_BAO_ContributionRecurTest
+ */
 class CRM_Contribute_BAO_ContributionRecurTest extends CiviUnitTestCase {
   protected $_params = array();
 
index 24a507d3331bc1ce72abaa23540a730b7e55d9f4..8db28afb1bd8f074527ddcc858edd34fac7157c7 100644 (file)
@@ -78,6 +78,9 @@ class CRM_Core_InnoDBIndexerTest extends CiviUnitTestCase {
     CRM_Core_DAO::executeQuery('SELECT id FROM civicrm_contact WHERE MATCH(first_name,last_name) AGAINST ("joe")');
   }
 
+  /**
+   * @return mixed
+   */
   public function supportsFts() {
     return version_compare(CRM_Core_DAO::singleValueQuery('SELECT VERSION()'), '5.6.0', '>=');
   }
index cb30579421c78e519b648a7f914c227e3dfa7c8b..8be75f956ffb5f91bda4b0a363e6bf5a5409c810 100644 (file)
@@ -7,6 +7,9 @@ require_once 'CiviTest/CiviUnitTestCase.php';
  */
 class CRM_Core_MenuTest extends CiviUnitTestCase {
 
+  /**
+   * @return array
+   */
   public function pathArguments() {
     $cases = array(); // array(0 => string $input, 1 => array $expectedOutput)
     //$cases[] = array(NULL, array());
index 746a8eff77596112873118c0ba27dce2af46e0ac..a5f1866808aca9b09eefa3abe25e96f3b8161810 100644 (file)
@@ -6,6 +6,14 @@ require_once 'CiviTest/CiviUnitTestCase.php';
  * Class CRM_Core_Page_RedirectTest
  */
 class CRM_Core_Page_RedirectTest extends CiviUnitTestCase {
+  /**
+   * Get example data.
+   *
+   * @return array
+   */
+  /**
+   * @return array
+   */
   public function examples() {
     $cases = array();
     // $cases[] = array(string $requestPath, string $requestArgs, string $pageArgs, string $expectedUrl)
index f3be84e769bcbb2a8faa198efd2b3fe886f9bec4..24260ab9f66ec1f79857857456099e03f8b23358 100644 (file)
@@ -27,6 +27,9 @@ class CRM_Core_TransactionTest extends CiviUnitTestCase {
     $this->cids = array();
   }
 
+  /**
+   * @return array
+   */
   public function dataCreateStyle() {
     return array(
       array('sql-insert'),
@@ -34,6 +37,9 @@ class CRM_Core_TransactionTest extends CiviUnitTestCase {
     );
   }
 
+  /**
+   * @return array
+   */
   public function dataCreateAndCommitStyles() {
     return array(
       array('sql-insert', 'implicit-commit'),
@@ -423,18 +429,34 @@ class CRM_Core_TransactionTest extends CiviUnitTestCase {
     } // else: implicit-commit
   }
 
+  /**
+   * @param $arg1
+   * @param $arg2
+   */
   public function _preCommit($arg1, $arg2) {
     $this->callbackLog[] = array('_preCommit', $arg1, $arg2);
   }
 
+  /**
+   * @param $arg1
+   * @param $arg2
+   */
   public function _postCommit($arg1, $arg2) {
     $this->callbackLog[] = array('_postCommit', $arg1, $arg2);
   }
 
+  /**
+   * @param $arg1
+   * @param $arg2
+   */
   public function _preRollback($arg1, $arg2) {
     $this->callbackLog[] = array('_preRollback', $arg1, $arg2);
   }
 
+  /**
+   * @param $arg1
+   * @param $arg2
+   */
   public function _postRollback($arg1, $arg2) {
     $this->callbackLog[] = array('_postRollback', $arg1, $arg2);
   }
index 73dfa8aa46bf3501ccf9bb0a04352a87b69886cc..afe8ed133374215aa336f7dfe5ffbfa87118eafc 100644 (file)
@@ -100,6 +100,9 @@ function hooktestb_civicrm_testRunHooks_inner() {
   $test->log[] = 'b-inner';
 }
 
+/**
+ * @return array
+ */
 function hooktesta_civicrm_testRunHooks_merge() {
   return array('from-module-a1', 'from-module-a2');
 }
@@ -113,10 +116,16 @@ function hooktestc_civicrm_testRunHooks_merge() {
   return array();
 }
 
+/**
+ * @return null
+ */
 function hooktestd_civicrm_testRunHooks_merge() {
   return NULL;
 }
 
+/**
+ * @return array
+ */
 function hookteste_civicrm_testRunHooks_merge() {
   return array('from-module-e');
 }
index 390c6d320bc2627f1d454aa45b2378704d54ce73..2e47bd1fb31418dddb68f0ba64fa24ea64eda205 100644 (file)
@@ -11,6 +11,9 @@ class CRM_Utils_HtmlToTextTest extends CiviUnitTestCase {
     parent::setUp();
   }
 
+  /**
+   * @return array
+   */
   public function htmlToTextExamples() {
     $cases = array(); // array(0 => string $html, 1 => string $text)
 
index e92a8ad94fede3317eeb276a1faf4d2c795600d2..bd71876df004a873cc51c9f626a079043dabc71b 100644 (file)
@@ -123,6 +123,9 @@ class CRM_Utils_StringTest extends CiviUnitTestCase {
     $this->assertEquals($expected, $actual);
   }
 
+  /**
+   * @return array
+   */
   public function booleanDataProvider() {
     $cases = array(); // array(0 => $input, 1 => $expectedOutput)
     $cases[] = array(TRUE, TRUE);
index e2ef226c9796fe955c404be48c8141b4fcae576d..dd42564094b082ecf9ab3a07d36c0dfe5bfa8dc0 100644 (file)
@@ -2,7 +2,15 @@
 
 require_once 'CiviTest/CiviUnitTestCase.php';
 
+/**
+ * Class CRM_Utils_TimeTest
+ */
 class CRM_Utils_TimeTest extends CiviUnitTestCase {
+  /**
+   * Equal cases.
+   *
+   * @return array
+   */
   public function equalCases() {
     $cases = array(); // array(0 => $timeA, 1 => $timeB, 2 => $threshold, 3 => $expectedResult)
     $cases[] = array('2012-04-01 12:00:00', '2012-04-01 12:00:00', 0, 1);
@@ -16,10 +24,13 @@ class CRM_Utils_TimeTest extends CiviUnitTestCase {
   }
 
   /**
+   * Test equals.
+   *
    * @param string $timeA
    * @param string $timeB
    * @param int $threshold
    * @param bool $expectedResult
+   *
    * @dataProvider equalCases
    */
   public function testEquals($timeA, $timeB, $threshold, $expectedResult) {
index 2977d01a851ea4a3a3ee717eed47c6d5c6c8f8b9..367d98f4b0f469b5a096ba851c1b252f2a55e42e 100644 (file)
@@ -2,6 +2,9 @@
 
 require_once 'CiviTest/CiviUnitTestCase.php';
 
+/**
+ * Class CRM_Utils_versionCheckTest
+ */
 class CRM_Utils_versionCheckTest extends CiviUnitTestCase {
 
   /**
index e10579f5612407e00acad7c7ec33cee69213ec85..355193688c89d7d5a9cb5aceca86ed44d9ab9ffb 100644 (file)
@@ -103,6 +103,9 @@ class DynamicFKAuthorizationTest extends \CiviUnitTestCase {
     \CRM_Core_DAO_AllCoreTables::init(TRUE);
   }
 
+  /**
+   * @return array
+   */
   public function okDataProvider() {
     $cases = array();
 
@@ -121,6 +124,9 @@ class DynamicFKAuthorizationTest extends \CiviUnitTestCase {
     return $cases;
   }
 
+  /**
+   * @return array
+   */
   public function badDataProvider() {
     $cases = array();
 
index bf07cabdc72accbb38da0ac116dfe817b6d673c5..5e73953dabcb0f7019391d851d0cae88d3aa90b4 100644 (file)
@@ -7,6 +7,11 @@ require_once 'CiviTest/CiviUnitTestCase.php';
  */
 class TransactionSubscriberTest extends \CiviUnitTestCase {
 
+  /**
+   * Get transaction options.
+   *
+   * @return array
+   */
   public function transactionOptions() {
     $r = array();
     // $r[] = array(string $entity, string $action, array $params, bool $isTransactional, bool $isForceRollback, bool $isNested);
@@ -78,15 +83,18 @@ class TransactionSubscriberTest extends \CiviUnitTestCase {
   }
 
   /**
-   * Ensure that API parameters "is_transactional" and "force_rollback" are parsed correctly
+   * Ensure that API parameters "is_transactional" and "force_rollback" are parsed correctly.
+   *
    * @dataProvider transactionOptions
+   *
    * @param $version
    * @param $entity
    * @param $action
-   * @param $params
-   * @param $isTransactional
-   * @param $isForceRollback
-   * @param $isNested
+   * @param array $params
+   * @param bool $isTransactional
+   * @param bool $isForceRollback
+   * @param bool $isNested
+   *
    * @throws \API_Exception
    */
   public function testTransactionOptions($version, $entity, $action, $params, $isTransactional, $isForceRollback, $isNested) {
index 49bbc75c78ff7cc7afb7e3d2b4beb220e2d7cfd1..904f638e2716bd364ae53daace916ae6e69f4f82 100644 (file)
@@ -3,6 +3,11 @@ namespace Civi\CCase;
 
 require_once 'CiviTest/CiviCaseTestCase.php';
 
+/**
+ * Class SequenceListenerTest
+ *
+ * @package Civi\CCase
+ */
 class SequenceListenerTest extends \CiviCaseTestCase {
 
   public function setUp() {
@@ -101,6 +106,11 @@ class SequenceListenerTest extends \CiviCaseTestCase {
     );
   }
 
+  /**
+   * @param $expected
+   * @param $actual
+   * @param int $tolerance
+   */
   public function assertApproxTime($expected, $actual, $tolerance = 1) {
     $diff = abs(strtotime($expected) - strtotime($actual));
     $this->assertTrue($diff <= $tolerance, sprintf("Check approx time equality. expected=[%s] actual=[%s] tolerance=[%s]",
index 6c279a332bd33248b3bc315957fa39eca699e756..998eb914b83613739c3abbc4f463ac1e68ce37ec 100644 (file)
@@ -103,6 +103,9 @@ class CiviSeleniumTestCase extends PHPUnit_Extensions_SeleniumTestCase {
     }
   }
 
+  /**
+   * @return string
+   */
   protected function prepareTestSession() {
     $result = parent::prepareTestSession();
 
index cd4ab3f6d79c2833be329350a68d19be9d61183a..13fd85aaa73572c61ce1856750ff6d767fcd8b4d 100644 (file)
@@ -32,6 +32,9 @@
 
 require_once 'CiviTest/CiviSeleniumTestCase.php';
 
+/**
+ * Class WebTest_Activity_AddRecurringActivityTest
+ */
 class WebTest_Activity_AddRecurringActivityTest extends CiviSeleniumTestCase {
 
   protected function setUp() {
index e65a33f6e0ac5a6a5ec0222ad508b788fab457fd..a6448fd1c1dcf2f5eeb1bde83be950e97a149e0a 100644 (file)
@@ -32,6 +32,9 @@
 
 require_once 'CiviTest/CiviSeleniumTestCase.php';
 
+/**
+ * Class WebTest_Event_AddRecurringEventTest
+ */
 class WebTest_Event_AddRecurringEventTest extends CiviSeleniumTestCase {
 
   protected function setUp() {
index a835ab9c2d5155e7051f145ff5e1e71e4f51556d..0efbb6762795d21b13b1fcddc7323787125d4720 100644 (file)
@@ -35,6 +35,9 @@ class WebTest_Pledge_StandaloneAddTest extends CiviSeleniumTestCase {
     parent::setUp();
   }
 
+  /**
+   * @return array
+   */
   public function testStandalonePledgeAdd() {
     $this->webtestLogin();
 
index e4b9d0df570a03735150d53bdae0ad09890543ad..1f6aca9861d9097c5611b5e22a42eb5c92baba27 100644 (file)
@@ -29,7 +29,6 @@ require_once 'tests/phpunit/CiviTest/CiviUnitTestCase.php';
 
 /**
  * Class contains api test cases for "civicrm_note"
- *
  */
 class api_v3_NoteTest extends CiviUnitTestCase {
 
@@ -42,7 +41,7 @@ class api_v3_NoteTest extends CiviUnitTestCase {
   public function setUp() {
 
     $this->_apiversion = 3;
-    //  Connect to the database
+    // Connect to the database.
     parent::setUp();
     $this->useTransaction(TRUE);
 
@@ -63,7 +62,8 @@ class api_v3_NoteTest extends CiviUnitTestCase {
   ///////////////// civicrm_note_get methods
 
   /**
-   * Check retrieve note with empty parameter array
+   * Check retrieve note with empty parameter array.
+   *
    * Error expected
    */
   public function testGetWithEmptyParams() {
@@ -71,18 +71,19 @@ class api_v3_NoteTest extends CiviUnitTestCase {
   }
 
   /**
-   * Check retrieve note with missing patrameters
+   * Check retrieve note with missing parameters.
+   *
    * Error expected
    */
   public function testGetWithoutEntityId() {
     $params = array(
       'entity_table' => 'civicrm_contact',
     );
-    $note = $this->callAPISuccess('note', 'get', $params);
+    $this->callAPISuccess('note', 'get', $params);
   }
 
   /**
-   * Check civicrm_note_get
+   * Check civicrm_note get.
    */
   public function testGet() {
     $entityId = $this->_noteID;
@@ -90,44 +91,45 @@ class api_v3_NoteTest extends CiviUnitTestCase {
       'entity_table' => 'civicrm_contact',
       'entity_id' => $entityId,
     );
-    $result = $this->callAPIAndDocument('note', 'get', $params, __FUNCTION__, __FILE__);
+    $this->callAPIAndDocument('note', 'get', $params, __FUNCTION__, __FILE__);
   }
 
-
-  ///////////////// civicrm_note_create methods
-
   /**
-   * Check create with empty parameter array
+   * Check create with empty parameter array.
+   *
    * Error Expected
    */
   public function testCreateWithEmptyNoteField() {
     $this->_params['note'] = "";
-    $result = $this->callAPIFailure('note', 'create', $this->_params,
-      'Mandatory key(s) missing from params array: note');
+    $this->callAPIFailure('note', 'create', $this->_params,
+      'Mandatory key(s) missing from params array: note'
+    );
   }
 
   /**
-   * Check create with partial params
+   * Check create with partial params.
+   *
    * Error expected
    */
   public function testCreateWithoutEntityId() {
     unset($this->_params['entity_id']);
-    $result = $this->callAPIFailure('note', 'create', $this->_params,
+    $this->callAPIFailure('note', 'create', $this->_params,
       'Mandatory key(s) missing from params array: entity_id');
   }
 
   /**
-   * Check create with partially empty params
+   * Check create with partially empty params.
+   *
    * Error expected
    */
   public function testCreateWithEmptyEntityId() {
     $this->_params['entity_id'] = "";
-    $result = $this->callAPIFailure('note', 'create', $this->_params,
+    $this->callAPIFailure('note', 'create', $this->_params,
       'Mandatory key(s) missing from params array: entity_id');
   }
 
   /**
-   * Check civicrm_note_create
+   * Check civicrm note create.
    */
   public function testCreate() {
 
@@ -166,7 +168,7 @@ class api_v3_NoteTest extends CiviUnitTestCase {
   }
 
   /**
-   * Check civicrm_note_create - tests used of default set to now
+   * Check civicrm_note_create - tests used of default set to .
    */
   public function testCreateWithoutModifiedDate() {
     unset($this->_params['modified_date']);
@@ -178,20 +180,19 @@ class api_v3_NoteTest extends CiviUnitTestCase {
     ));
   }
 
-
-  ///////////////// civicrm_note_update methods
-
   /**
-   * Check update with empty parameter array
+   * Check update with empty parameter array.
+   *
    * Please don't copy & paste this - is of marginal value
    * better to put time into the function on Syntax Conformance class that tests this
    */
   public function testUpdateWithEmptyParams() {
-    $note = $this->callAPIFailure('note', 'create', array());
+    $this->callAPIFailure('note', 'create', array());
   }
 
   /**
-   * Check update with missing parameter (contact id)
+   * Check update with missing parameter (contact id).
+   *
    * Error expected
    */
   public function testUpdateWithoutContactId() {
@@ -199,13 +200,13 @@ class api_v3_NoteTest extends CiviUnitTestCase {
       'entity_id' => $this->_contactID,
       'entity_table' => 'civicrm_contact',
     );
-    $note = $this->callAPIFailure('note', 'create', $params,
+    $this->callAPIFailure('note', 'create', $params,
       'Mandatory key(s) missing from params array: note'
     );
   }
 
   /**
-   * Check civicrm_note_update
+   * Check civicrm_note update.
    */
   public function testUpdate() {
     $params = array(
@@ -215,7 +216,7 @@ class api_v3_NoteTest extends CiviUnitTestCase {
       'subject' => 'Hello World',
     );
 
-    //Update Note
+    // Update Note.
     $this->callAPISuccess('note', 'create', $params);
     $note = $this->callAPISuccess('Note', 'Get', array());
     $this->assertEquals($note['id'], $this->_noteID, 'in line ' . __LINE__);
@@ -225,30 +226,29 @@ class api_v3_NoteTest extends CiviUnitTestCase {
     $this->assertEquals('Note1', $note['values'][$this->_noteID]['note'], 'in line ' . __LINE__);
   }
 
-  ///////////////// civicrm_note_delete methods
-
-
   /**
-   * Check delete with empty parametes array
-   * Error expected
+   * Check delete with empty parameters array.
+   *
+   * Error expected.
    */
   public function testDeleteWithEmptyParams() {
-    $deleteNote = $this->callAPIFailure('note', 'delete', array(), 'Mandatory key(s) missing from params array: id');
+    $this->callAPIFailure('note', 'delete', array(), 'Mandatory key(s) missing from params array: id');
   }
 
   /**
-   * Check delete with wrong id
+   * Check delete with wrong id.
+   *
    * Error expected
    */
   public function testDeleteWithWrongID() {
     $params = array(
       'id' => 0,
     );
-    $deleteNote = $this->callAPIFailure('note', 'delete', $params, 'Mandatory key(s) missing from params array: id');
+    $this->callAPIFailure('note', 'delete', $params, 'Mandatory key(s) missing from params array: id');
   }
 
   /**
-   * Check civicrm_note_delete
+   * Check civicrm_note delete.
    */
   public function testDelete() {
     $additionalNote = $this->noteCreate($this->_contactID);
@@ -257,17 +257,17 @@ class api_v3_NoteTest extends CiviUnitTestCase {
       'id' => $additionalNote['id'],
     );
 
-    $result = $this->callAPIAndDocument('note', 'delete', $params, __FUNCTION__, __FILE__);
+    $this->callAPIAndDocument('note', 'delete', $params, __FUNCTION__, __FILE__);
   }
 
 }
 
 /**
- *  Test civicrm_activity_create() using example code
+ * Test civicrm note create() using example code.
  */
 function testNoteCreateExample() {
   require_once 'api/v3/examples/Note/Create.php';
-  $result = UF_match_get_example();
-  $expectedResult = UF_match_get_expectedresult();
+  $result = Note_get_example();
+  $expectedResult = Note_get_expectedresult();
   $this->assertEquals($result, $expectedResult);
 }