code cleanup
authorkurund <kurund@civicrm.org>
Wed, 14 Jan 2015 13:45:55 +0000 (19:15 +0530)
committerkurund <kurund@civicrm.org>
Wed, 14 Jan 2015 13:45:55 +0000 (19:15 +0530)
34 files changed:
CRM/ACL/Form/WordPress/Permissions.php
CRM/Admin/Form/WordReplacements.php
CRM/Campaign/Page/AJAX.php
CRM/Case/Form/Task/Restore.php
CRM/Contact/BAO/GroupContact.php
CRM/Contact/Form/Domain.php
CRM/Contact/Page/View/Sunlight.php
CRM/Contact/Page/View/Tag.php
CRM/Contact/StateMachine/Search.php
CRM/Core/BAO/ConfigSetting.php
CRM/Core/BAO/MessageTemplate.php
CRM/Core/BAO/UFField.php
CRM/Core/Form/Tag.php
CRM/Core/Permission/UnitTests.php
CRM/Dashlet/Page/Blog.php
CRM/Event/Form/Registration/AdditionalParticipant.php
CRM/Event/Form/Registration/Register.php
CRM/Event/Page/ManageEvent.php
CRM/Grant/Form/Task.php
CRM/Member/BAO/Membership.php
CRM/Member/BAO/MembershipLog.php
CRM/PCP/Page/PCPInfo.php
CRM/Utils/Geocode/Yahoo.php
CRM/Utils/Recent.php
CRM/Utils/String.php
Civi/CCase/Analyzer.php
api/v3/Job.php
api/v3/UFGroup.php
bin/cleanup42.php
tests/extensions/test.extension.manager.searchtest/main.php
tests/phpunit/CRM/Contact/AllTests.php
tests/phpunit/CRM/Core/TransactionTest.php
tests/phpunit/CRM/Mailing/BAO/QueryTest.php
tests/phpunit/Civi/AllTests.php

index 489bd60757b1cc386649379e1474bedbd33e3a6d..acd6254449b6acc7c4062f89cbdc3875899363c5 100644 (file)
@@ -90,7 +90,7 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form {
           'type' => 'next',
           'name' => ts('Save'),
           'spacing' => '',
-          'isDefault' => FALSE
+          'isDefault' => FALSE,
         ),
       )
     );
@@ -143,7 +143,7 @@ class CRM_ACL_Form_WordPress_Permissions extends CRM_Core_Form {
           CRM_Core_Session::setStatus(
             ts('The %1 role was assigned one or more permissions that may prove dangerous for users of that role to have. Please reconsider assigning %2 to them.', array(
                 1 => $wp_roles->role_names[$role],
-                2 => implode(', ', $warningPermissionNames)
+                2 => implode(', ', $warningPermissionNames),
               )),
             ts('Unsafe Permission Settings')
           );
index 4394353ba637ebda32e357bed799da1ab265ab3a..b0d0db63a226679436d6c36b2a33b50fd1958745 100644 (file)
@@ -55,7 +55,7 @@ class CRM_Admin_Form_WordReplacements extends CRM_Core_Form {
       array(
         'title' => ts('Word Replacements'),
         'url' => $breadCrumbUrl,
-      )
+      ),
     );
     CRM_Utils_System::appendBreadCrumb($breadCrumb);
   }
@@ -219,7 +219,7 @@ class CRM_Admin_Form_WordReplacements extends CRM_Core_Form {
    */
   public function postProcess() {
     $params = $this->controller->exportValues($this->_name);
-    $this->_numStrings = sizeof($params['old']);
+    $this->_numStrings = count($params['old']);
 
     $enabled['exactMatch'] = $enabled['wildcardMatch'] = $disabled['exactMatch'] = $disabled['wildcardMatch'] = array();
     for ($i = 1; $i <= $this->_numStrings; $i++) {
index b8cf42de6d5eefeccd5809a8fa6629b2773cc4dc..b6aeb8ffc5253b85ceb18561fa15952ccccb7fe4 100644 (file)
@@ -158,7 +158,7 @@ class CRM_Campaign_Page_AJAX {
     //format multi-select group and contact types.
     foreach (array(
                'group',
-               'contact_type'
+               'contact_type',
              ) as $param) {
       $paramValue = CRM_Utils_Array::value($param, $params);
       if ($paramValue) {
@@ -174,7 +174,7 @@ class CRM_Campaign_Page_AJAX {
     foreach (array(
                'campaign_survey_id',
                'survey_interviewer_id',
-               'campaign_search_voter_for'
+               'campaign_search_voter_for',
              ) as $fld) {
       $voterClauseParams[$fld] = CRM_Utils_Array::value($fld, $params);
     }
@@ -499,7 +499,7 @@ class CRM_Campaign_Page_AJAX {
       array(
         'value' => '',
         'title' => ts('- select -'),
-      )
+      ),
     );
     foreach ($campaigns as $value => $title) {
       $class = NULL;
@@ -546,7 +546,7 @@ class CRM_Campaign_Page_AJAX {
       array(
         'value' => '',
         'title' => ts('- select -'),
-      )
+      ),
     );
     foreach ($campGroups as $grpId => $title) {
       $groups[] = array(
@@ -565,7 +565,7 @@ class CRM_Campaign_Page_AJAX {
   /**
    * Retrieve campaigns as for campaign dashboard.
    *
-   **/
+   */
   public function campaignList() {
     //get the search criteria params.
     $searchParams = explode(',', CRM_Utils_Array::value('searchCriteria', $_POST));
@@ -635,7 +635,7 @@ class CRM_Campaign_Page_AJAX {
                'sort',
                'offset',
                'rowCount',
-               'sortOrder'
+               'sortOrder',
              ) as $sortParam) {
       $params[$sortParam] = $$sortParam;
     }
@@ -666,7 +666,7 @@ class CRM_Campaign_Page_AJAX {
   /**
    * Retrieve survey for survey dashboard.
    *
-   **/
+   */
   public function surveyList() {
     //get the search criteria params.
     $searchParams = explode(',', CRM_Utils_Array::value('searchCriteria', $_POST));
@@ -738,7 +738,7 @@ class CRM_Campaign_Page_AJAX {
                'sort',
                'offset',
                'rowCount',
-               'sortOrder'
+               'sortOrder',
              ) as $sortParam) {
       $params[$sortParam] = $$sortParam;
     }
@@ -769,7 +769,7 @@ class CRM_Campaign_Page_AJAX {
   /**
    * Retrieve petitions for petition dashboard.
    *
-   **/
+   */
   public function petitionList() {
     //get the search criteria params.
     $searchParams = explode(',', CRM_Utils_Array::value('searchCriteria', $_POST));
@@ -836,7 +836,7 @@ class CRM_Campaign_Page_AJAX {
                'sort',
                'offset',
                'rowCount',
-               'sortOrder'
+               'sortOrder',
              ) as $sortParam) {
       $params[$sortParam] = $$sortParam;
     }
index 2709c8171d4f9584e497a30975b7cae67a37127e..93bbaa0f9068efd6b96ad90b4fcfc4891a4f9f2e 100644 (file)
@@ -53,7 +53,7 @@ class CRM_Case_Form_Task_Restore extends CRM_Case_Form_Task {
    *
    * @return void
    */
-  function preProcess() {
+  public function preProcess() {
     parent::preProcess();
   }
 
index bfadfc54b8d8d70a4b6f3a3bf6b0c8fe1e24363f..72be26a332fbcdc810c0cfaf39da25b404e2e8ad 100644 (file)
@@ -73,7 +73,7 @@ class CRM_Contact_BAO_GroupContact extends CRM_Contact_DAO_GroupContact {
    * @param array $params
    *   (reference ) an assoc array of name/value pairs.
    *
-   * @return boolean
+   * @return bool
    */
   public static function dataExists(&$params) {
     // return if no data present
@@ -129,7 +129,7 @@ class CRM_Contact_BAO_GroupContact extends CRM_Contact_DAO_GroupContact {
    * @return array
    *   (total, added, notAdded) count of contacts added to group
    */
-  static function addContactsToGroup(
+  public static function addContactsToGroup(
     $contactIds,
     $groupId,
     $method = 'Admin',
@@ -137,7 +137,6 @@ class CRM_Contact_BAO_GroupContact extends CRM_Contact_DAO_GroupContact {
     $tracking = NULL
   ) {
 
-
     CRM_Utils_Hook::pre('create', 'GroupContact', $groupId, $contactIds);
 
     list($numContactsAdded,
@@ -179,7 +178,7 @@ class CRM_Contact_BAO_GroupContact extends CRM_Contact_DAO_GroupContact {
    * @return array
    *   (total, removed, notRemoved) count of contacts removed to group
    */
-  static function removeContactsFromGroup(
+  public static function removeContactsFromGroup(
     &$contactIds,
     $groupId,
     $method = 'Admin',
@@ -329,13 +328,11 @@ class CRM_Contact_BAO_GroupContact extends CRM_Contact_DAO_GroupContact {
    *
    * @param bool $excludeHidden
    *
-   * @return array
-   *   (reference )|int $values the relevant data object values for the contact or
-   *                                 the total count when $count is TRUE
-   *
-   * $access public
+   * @return array (reference)|int $values
+   *   the relevant data object values for the contact or
+   *   the total count when $count is TRUE
    */
-  static function &getContactGroup(
+  public static function &getContactGroup(
     $contactId,
     $status = NULL,
     $numGroupContact = NULL,
@@ -720,7 +717,7 @@ AND       group_id IN ( $groupIDString )
    * @return array
    *   (total, added, notAdded) count of contacts added to group
    */
-  static function bulkAddContactsToGroup(
+  public static function bulkAddContactsToGroup(
     $contactIDs,
     $groupID,
     $method = 'Admin',
@@ -797,11 +794,11 @@ AND    contact_id IN ( $contactStr )
    *
    * @param string $fieldName
    * @param string $context
-   *   @see CRM_Core_DAO::buildOptionsContext.
+   * @see CRM_Core_DAO::buildOptionsContext
    * @param array $props
    *   whatever is known about this dao object.
    *
-   * @return Array|bool
+   * @return array|bool
    */
   public static function buildOptions($fieldName, $context = NULL, $props = array()) {
     $params = array();
index 22328fb9db9a79110d1d061ab5a43d7043805655..d749ee8eaf4e0d2324f3fe7f3012509de98acad4 100644 (file)
@@ -298,7 +298,6 @@ class CRM_Contact_Form_Domain extends CRM_Core_Form {
       $emailParams['weight'] = CRM_Utils_Weight::getDefaultWeight('CRM_Core_DAO_OptionValue', $fieldValues);
     }
 
-
     //reset default within domain.
     $emailParams['reset_default_for'] = array('domain_id' => CRM_Core_Config::domainID());
 
index 09b2d1148f0a5fd2b00dd19245b1b07525b91163..9bd21723e94d587e4f1bdfd0fc223ef30ec58d6f 100644 (file)
@@ -37,7 +37,7 @@ class CRM_Contact_Page_View_Sunlight extends CRM_Contact_Page_View {
   /**
    * called when action is browse
    *
-   * @return null
+   * @return void
    */
   public function browse() {
     // get the primary city, state and zip for the contact
index dd9e83f736537d9f60d7fc13d81abed7a4b8f632..7e902626c5debd3701d27046310ed435b3c5b0bf 100644 (file)
@@ -37,7 +37,7 @@ class CRM_Contact_Page_View_Tag extends CRM_Core_Page {
   /**
    * called when action is browse
    *
-   * @return null
+   * @return void
    */
   public function browse() {
     $controller = new CRM_Core_Controller_Simple('CRM_Tag_Form_Tag', ts('Contact Tags'), $this->_action);
index 2b35e17f34588f87f1e2b8ce6fd5c3035899fd59..9dc31c61a8c5b722257df0c6316e2b417dc29ae3 100644 (file)
@@ -44,7 +44,7 @@ class CRM_Contact_StateMachine_Search extends CRM_Core_StateMachine {
   /**
    * Class constructor
    */
-  function __construct($controller, $action = CRM_Core_Action::NONE) {
+  public function __construct($controller, $action = CRM_Core_Action::NONE) {
     parent::__construct($controller, $action);
 
     $this->_pages = array();
index 546b145ac7531b0e9e03195284ec515faf29f5d1..4b2a5f2709ce03b59434b6e0923deeaabf8c5fd0 100644 (file)
@@ -47,7 +47,7 @@ class CRM_Core_BAO_ConfigSetting {
    * @param array $params
    *   Associated array of civicrm variables.
    *
-   * @return null
+   * @return void
    */
   public static function create($params) {
     self::add($params);
@@ -63,7 +63,7 @@ class CRM_Core_BAO_ConfigSetting {
    * @param array $params
    *   Associated array of civicrm variables.
    *
-   * @return null
+   * @return void
    */
   public static function add(&$params) {
     self::fixParams($params);
@@ -71,8 +71,8 @@ class CRM_Core_BAO_ConfigSetting {
     // also set a template url so js files can use this
     // CRM-6194
     $params['civiRelativeURL'] = CRM_Utils_System::url('CIVI_BASE_TEMPLATE');
-    $params['civiRelativeURL'] =
-      str_replace(
+    $params['civiRelativeURL']
+      str_replace(
         'CIVI_BASE_TEMPLATE',
         '',
         $params['civiRelativeURL']
@@ -144,7 +144,7 @@ class CRM_Core_BAO_ConfigSetting {
    * @param array $params
    *   Associated array of civicrm variables.
    *
-   * @return null
+   * @return void
    */
   public static function fixParams(&$params) {
     // in our old civicrm.settings.php we were using ISO code for country and
@@ -178,7 +178,7 @@ class CRM_Core_BAO_ConfigSetting {
    * @param array $values
    *   Associated array of civicrm variables stored in db.
    *
-   * @return null
+   * @return void
    */
   public static function formatParams(&$params, &$values) {
     if (empty($params) ||
@@ -228,7 +228,7 @@ class CRM_Core_BAO_ConfigSetting {
       $defaults = unserialize($domain->config_backend);
       if ($defaults === FALSE || !is_array($defaults)) {
         $defaults = array();
-        return;
+        return FALSE;
       }
 
       $skipVars = self::skipVars();
@@ -519,7 +519,6 @@ class CRM_Core_BAO_ConfigSetting {
     list($oldURL, $oldDir, $oldSiteName, $oldSiteRoot) = self::getConfigSettings();
     list($newURL, $newDir, $newSiteName, $newSiteRoot) = self::getBestGuessSettings();
 
-
     // retrieve these values from the argument list
     $variables = array('URL', 'Dir', 'SiteName', 'SiteRoot', 'Val_1', 'Val_2', 'Val_3');
     $states = array('old', 'new');
@@ -656,7 +655,7 @@ WHERE  option_group_id = (
    * @param string $componentName
    *   Name of the component to be enabled, needs to be valid.
    *
-   * @return boolean
+   * @return bool
    *   true if valid component name and enabling succeeds, else false
    */
   public static function enableComponent($componentName) {
@@ -672,8 +671,8 @@ WHERE  option_group_id = (
     }
 
     // get enabled-components from DB and add to the list
-    $enabledComponents =
-      CRM_Core_BAO_Setting::getItem(CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME, 'enable_components', NULL, array());
+    $enabledComponents
+      CRM_Core_BAO_Setting::getItem(CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME, 'enable_components', NULL, array());
     $enabledComponents[] = $componentName;
 
     self::setEnabledComponents($enabledComponents);
@@ -691,8 +690,8 @@ WHERE  option_group_id = (
     }
 
     // get enabled-components from DB and add to the list
-    $enabledComponents =
-      CRM_Core_BAO_Setting::getItem(CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME, 'enable_components', NULL, array());
+    $enabledComponents
+      CRM_Core_BAO_Setting::getItem(CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME, 'enable_components', NULL, array());
     $enabledComponents = array_diff($enabledComponents, array($componentName));
 
     self::setEnabledComponents($enabledComponents);
index eeb4f5448fdcd833b01f227f06827f4c1e8d3bfe..591fcd3320e76513635b7fd3ff577c251f14bcfb 100644 (file)
@@ -99,10 +99,8 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
   /**
    * Delete the Message Templates
    *
-   *
    * @param int $messageTemplatesID
-   *
-   * @return object
+   * @return void
    */
   public static function del($messageTemplatesID) {
     // make sure messageTemplatesID is an integer
@@ -229,7 +227,7 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
       $smarty = CRM_Core_Smarty::singleton();
       foreach (array(
                  'text',
-                 'html'
+                 'html',
                ) as $elem) {
         $$elem = $smarty->fetch("string:{$$elem}");
       }
@@ -274,12 +272,12 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
   /**
    * Revert a message template to its default subject+text+HTML state
    *
-   * @param int id id of the template
+   * @param int $id id of the template
    *
    * @return void
    */
   public static function revert($id) {
-    $diverted = new self;
+    $diverted = new CRM_Core_BAO_MessageTemplate();
     $diverted->id = (int) $id;
     $diverted->find(1);
 
@@ -287,7 +285,7 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
       CRM_Core_Error::fatal(ts('Did not find a message template with id of %1.', array(1 => $id)));
     }
 
-    $orig = new self;
+    $orig = new CRM_Core_BAO_MessageTemplate();
     $orig->workflow_id = $diverted->workflow_id;
     $orig->is_reserved = 1;
     $orig->find(1);
@@ -379,7 +377,7 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
       else {
         CRM_Core_Error::fatal(ts('No such message template: option group %1, option value %2.', array(
               1 => $params['groupName'],
-              2 => $params['valueName']
+              2 => $params['valueName'],
             )));
       }
     }
@@ -409,7 +407,7 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
     // replace tokens in the three elements (in subject as if it was the text body)
     $domain = CRM_Core_BAO_Domain::getDomain();
     $hookTokens = array();
-    $mailing = new CRM_Mailing_BAO_Mailing;
+    $mailing = new CRM_Mailing_BAO_Mailing();
     $mailing->body_text = $text;
     $mailing->body_html = $html;
     $tokens = $mailing->getTokens();
@@ -452,7 +450,6 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
       $text = CRM_Utils_Token::replaceContactTokens($text, $contact, FALSE, $tokens['text'], FALSE, TRUE);
       $html = CRM_Utils_Token::replaceContactTokens($html, $contact, FALSE, $tokens['html'], FALSE, TRUE);
 
-
       $contactArray = array($contactID => $contact);
       CRM_Utils_Hook::tokenValues($contactArray,
         array($contactID),
@@ -472,8 +469,6 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
     $subject = "{strip}$subject{/strip}";
 
     // parse the three elements with Smarty
-
-
     $smarty = CRM_Core_Smarty::singleton();
     foreach ($params['tplParams'] as $name => $value) {
       $smarty->assign($name, $value);
@@ -481,7 +476,7 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
     foreach (array(
                'subject',
                'text',
-               'html'
+               'html',
              ) as $elem) {
       $$elem = $smarty->fetch("string:{$$elem}");
     }
index 0781e9e5549e98d4704f3e6c52f2f3330caad144..6abb7e34c6160bcdce07bcc582aff05bea0425f8 100644 (file)
@@ -106,7 +106,7 @@ class CRM_Core_BAO_UFField extends CRM_Core_DAO_UFField {
    * @param int $id
    *   Field Id.
    *
-   * @return boolean
+   * @return bool
    *
    */
   public static function del($id) {
@@ -360,7 +360,7 @@ WHERE cf.id IN (" . $customFieldIds . ") AND is_multiple = 1 LIMIT 0,1";
    *
    * @param int $UFFieldId
    *
-   * @return boolean
+   * @return bool
    *   false if custom field are disabled else true
    */
   public static function checkUFStatus($UFFieldId) {
@@ -450,11 +450,11 @@ WHERE cf.id IN (" . $customFieldIds . ") AND is_multiple = 1 LIMIT 0,1";
    * @param array $optional
    *   Array of types those are optional.
    *
-   * @return boolean
+   * @return bool
    */
   public static function checkValidProfileType($ufGroupId, $required, $optional = NULL) {
     if (!is_array($required) || empty($required)) {
-      return;
+      return FALSE;
     }
 
     $ufGroup = new CRM_Core_DAO_UFGroup();
@@ -569,7 +569,6 @@ WHERE cf.id IN (" . $customFieldIds . ") AND is_multiple = 1 LIMIT 0,1";
    * @return string
    *   profile group_type
    *
-   * TODO Why is this function in this class? It seems to be about the UFGroup.
    */
   public static function getProfileType($ufGroupId, $returnMixType = TRUE, $onlyPure = FALSE, $skipComponentType = FALSE) {
     $ufGroup = new CRM_Core_DAO_UFGroup();
@@ -590,10 +589,8 @@ WHERE cf.id IN (" . $customFieldIds . ") AND is_multiple = 1 LIMIT 0,1";
    *   True if only pure profiles are required.
    * @param bool $skipComponentType
    *
-   * @return string
-   *   profile group_type
+   * @return string  profile group_type
    *
-   * TODO Why is this function in this class? It seems to be about the UFGroup.
    */
   public static function calculateProfileType($ufGroupType, $returnMixType = TRUE, $onlyPure = FALSE, $skipComponentType = FALSE) {
     // profile types
@@ -728,7 +725,7 @@ SELECT ufg.id as id
    *
    * @param int $profileID
    *
-   * @return boolean
+   * @return bool
    */
   public static function checkSearchableORInSelector($profileID) {
     $result = FALSE;
@@ -802,7 +799,7 @@ SELECT  id
         'return' => 'field_name, is_required',
         'options' => array(
           'limit' => 0,
-        )
+        ),
       )
     ));
     //check for valid fields ( fields that are present in billing block )
@@ -831,7 +828,7 @@ SELECT  id
     }
 
     if (!in_array($prefixName, $validProfileFields)) {
-      return;
+      return FALSE;
     }
 
     if (!empty($index) && (
@@ -1029,7 +1026,7 @@ SELECT  id
     ));
     //unset selected fields
     foreach ($groupFieldList as $key => $value) {
-      if (is_integer($key)) {
+      if (is_int($key)) {
         unset($fields['Individual'][$value], $fields['Household'][$value], $fields['Organization'][$value]);
         continue;
       }
index 50c182feea484a08f1894729b219e613389bad8f..2a004296768fc1bca705dc4333f15efd88b24fd3 100644 (file)
@@ -60,7 +60,7 @@ class CRM_Core_Form_Tag {
    *
    * @return void
    */
-  static function buildQuickForm(
+  public static function buildQuickForm(
     &$form, $parentNames, $entityTable, $entityId = NULL, $skipTagCreate = FALSE,
     $skipEntityAction = FALSE, $tagsetElementName = NULL) {
     $tagset = $form->_entityTagValues = array();
index 904703fc7f710213aed19d67d74f499aecd83b48..2e5043f1e3c59aab6808a218edd2911506d0b3e9 100644 (file)
@@ -47,7 +47,7 @@ class CRM_Core_Permission_UnitTests extends CRM_Core_Permission_Base {
    * @param string $str
    *   The permission to check.
    *
-   * @return boolean
+   * @return bool
    *   true if yes, else false
    */
   public function check($str) {
index 60069a94fd554c9174c86f9537c44587d37c9857..c76d86b6782ed6832601fe8be656a802a03859b2 100644 (file)
@@ -117,7 +117,7 @@ class CRM_Dashlet_Page_Blog extends CRM_Core_Page {
         $item = (array) $item;
         // Clean up description - remove tags that would break dashboard layout
         $description = preg_replace('#<h[1-3][^>]*>(.+?)</h[1-3][^>]*>#s', '<h4>$1</h4>', $item['description']);
-        $item['description'] = strip_tags($description, "<a><p><h4><h5><h6><b><i><em><strong><ol><ul><li><dd><dt><code><pre><br>");
+        $item['description'] = strip_tags($description, "<a><p><h4><h5><h6><b><i><em><strong><ol><ul><li><dd><dt><code><pre><br/>");
         $blog[] = $item;
       }
       if ($blog) {
index 35c11170be8dfe801ded20ccb4477bac5893d6e3..3ad922376bee0869a98aeafeee853d3d65dfc2ee 100644 (file)
@@ -178,7 +178,7 @@ class CRM_Event_Form_Registration_AdditionalParticipant extends CRM_Event_Form_R
     $pre = $post = array();
     foreach (array(
                'pre',
-               'post'
+               'post',
              ) as $keys) {
       if (isset($this->_values['additional_custom_' . $keys . '_id'])) {
         $this->buildCustom($this->_values['additional_custom_' . $keys . '_id'], 'additionalCustom' . ucfirst($keys), TRUE);
@@ -186,7 +186,7 @@ class CRM_Event_Form_Registration_AdditionalParticipant extends CRM_Event_Form_R
       }
       foreach (array(
                  'first_name',
-                 'last_name'
+                 'last_name',
                ) as $name) {
         if (array_key_exists($name, $$keys) &&
           CRM_Utils_Array::value('is_required', CRM_Utils_Array::value($name, $$keys))
@@ -258,20 +258,20 @@ class CRM_Event_Form_Registration_AdditionalParticipant extends CRM_Event_Form_R
           if ($this->_requireApproval) {
             $statusMessage = ts("It looks like you are now registering a group of %1 participants. The event has %2 available spaces (you will not be wait listed). Registration for this event requires approval. You will receive an email once your registration has been reviewed.", array(
                 1 => ++$processedCnt,
-                2 => $spaces
+                2 => $spaces,
               ));
           }
           else {
             $statusMessage = ts("It looks like you are now registering a group of %1 participants. The event has %2 available spaces (you will not be wait listed).", array(
                 1 => ++$processedCnt,
-                2 => $spaces
+                2 => $spaces,
               ));
           }
         }
         else {
           $statusMessage = ts("It looks like you are now registering a group of %1 participants. The event has %2 available spaces (you will not be wait listed). Please go back to the main registration page and reduce the number of additional people. You will also need to complete payment information.", array(
               1 => ++$processedCnt,
-              2 => $spaces
+              2 => $spaces,
             ));
           $allowToProceed = FALSE;
         }
@@ -365,7 +365,7 @@ class CRM_Event_Form_Registration_AdditionalParticipant extends CRM_Event_Form_R
               'name' => ts('Skip Participant'),
               'subName' => 'skip',
               'icon' => 'seek-next',
-            )
+            ),
           )
         );
       }
@@ -487,7 +487,7 @@ class CRM_Event_Form_Registration_AdditionalParticipant extends CRM_Event_Form_R
           ) {
             $errors['_qf_default'] = ts('It looks like event has only %2 seats available and you are trying to register %1 participants, so could you please select price options accordingly.', array(
                 1 => $totalParticipants,
-                2 => $self->_availableRegistrations
+                2 => $self->_availableRegistrations,
               ));
           }
         }
@@ -763,7 +763,7 @@ class CRM_Event_Form_Registration_AdditionalParticipant extends CRM_Event_Form_R
    *
    * @param bool $isButtonJs
    *
-   * @return boolean
+   * @return bool
    *   ture on success.
    */
   public function isLastParticipant($isButtonJs = FALSE) {
@@ -777,7 +777,7 @@ class CRM_Event_Form_Registration_AdditionalParticipant extends CRM_Event_Form_R
   /**
    * Reset values for all options those are full.
    *
-   **/
+   */
   public function resetElementValue($optionFullIds = array()) {
     if (!is_array($optionFullIds) ||
       empty($optionFullIds) ||
@@ -847,7 +847,7 @@ class CRM_Event_Form_Registration_AdditionalParticipant extends CRM_Event_Form_R
     foreach (array(
                'constantValues',
                'submitValues',
-               'defaultValues'
+               'defaultValues',
              ) as $val) {
       $values = &$this->{"_$val"};
       if (!is_array($values) || empty($values)) {
index f7cf8a7180ed6c07c575bfcc5acd48834afbef03..cfb1aa15d028f80cfe3127f1e2528e6792332550 100644 (file)
@@ -255,8 +255,8 @@ class CRM_Event_Form_Registration_Register extends CRM_Event_Form_Registration {
     // add this event's default participant role to defaults array
     // (for cases where participant_role field is included in form via profile)
     if ($this->_values['event']['default_role_id']) {
-      $this->_defaults['participant_role'] =
-      $this->_defaults['participant_role_id'] = $this->_values['event']['default_role_id'];
+      $this->_defaults['participant_role']
+        = $this->_defaults['participant_role_id'] = $this->_values['event']['default_role_id'];
     }
     if ($this->_priceSetId && !empty($this->_feeBlock)) {
       foreach ($this->_feeBlock as $key => $val) {
@@ -751,7 +751,6 @@ class CRM_Event_Form_Registration_Register extends CRM_Event_Form_Registration {
         $dbTotalCount = CRM_Utils_Array::value($optId, $recordedOptionsCount, 0);
         $currentTotalCount = CRM_Utils_Array::value($optId, $currentOptionsCount, 0);
 
-
         $totalCount = $currentTotalCount + $dbTotalCount;
         $isFull = FALSE;
         if ($maxValue &&
index 6cde6e5f15934f062953fb5a1d1dfcff3a71ff2e..320c328b9dc97ce2b7e5f140b3caf84b9d3016ea 100644 (file)
@@ -111,64 +111,64 @@ class CRM_Event_Page_ManageEvent extends CRM_Core_Page {
       self::$_tabLinks = array();
     }
     if (!isset(self::$_tabLinks[$cacheKey])) {
-      self::$_tabLinks[$cacheKey]['settings'] =
-        array(
+      self::$_tabLinks[$cacheKey]['settings']
+        array(
           'title' => ts('Info and Settings'),
           'url' => 'civicrm/event/manage/settings',
           'field' => 'id',
         );
-      self::$_tabLinks[$cacheKey]['location'] =
-        array(
+      self::$_tabLinks[$cacheKey]['location']
+        array(
           'title' => ts('Location'),
           'url' => 'civicrm/event/manage/location',
           'field' => 'loc_block_id',
         );
 
-      self::$_tabLinks[$cacheKey]['fee'] =
-        array(
+      self::$_tabLinks[$cacheKey]['fee']
+        array(
           'title' => ts('Fees'),
           'url' => 'civicrm/event/manage/fee',
           'field' => 'is_monetary',
         );
-      self::$_tabLinks[$cacheKey]['registration'] =
-        array(
+      self::$_tabLinks[$cacheKey]['registration']
+        array(
           'title' => ts('Online Registration'),
           'url' => 'civicrm/event/manage/registration',
           'field' => 'is_online_registration',
         );
+
       if (CRM_Core_Permission::check('administer CiviCRM')) {
-        self::$_tabLinks[$cacheKey]['reminder'] =
-          array(
+        self::$_tabLinks[$cacheKey]['reminder']
+          array(
             'title' => ts('Schedule Reminders'),
             'url' => 'civicrm/event/manage/reminder',
             'field' => 'reminder',
           );
       }
-      self::$_tabLinks[$cacheKey]['conference'] =
-        array(
+      self::$_tabLinks[$cacheKey]['conference']
+        array(
           'title' => ts('Conference Slots'),
           'url' => 'civicrm/event/manage/conference',
           'field' => 'slot_label_id',
         );
-      self::$_tabLinks[$cacheKey]['friend'] =
-        array(
+      self::$_tabLinks[$cacheKey]['friend']
+        array(
           'title' => ts('Tell a Friend'),
           'url' => 'civicrm/event/manage/friend',
           'field' => 'friend',
         );
-      self::$_tabLinks[$cacheKey]['pcp'] =
-        array(
+      self::$_tabLinks[$cacheKey]['pcp']
+        array(
           'title' => ts('Personal Campaign Pages'),
           'url' => 'civicrm/event/manage/pcp',
           'field' => 'is_pcp_enabled',
         );
-      self::$_tabLinks[$cacheKey]['repeat'] =
-        array(
+      self::$_tabLinks[$cacheKey]['repeat']
+        array(
           'title' => ts('Repeat'),
           'url' => 'civicrm/event/manage/repeat',
           'field' => 'is_repeating_event',
         );
-
     }
 
     if (!$enableCart) {
@@ -418,8 +418,7 @@ ORDER BY start_date desc
   }
 
   public function search() {
-    if (isset($this->_action) &
-      (CRM_Core_Action::ADD |
+    if (isset($this->_action) & (CRM_Core_Action::ADD |
         CRM_Core_Action::UPDATE |
         CRM_Core_Action::DELETE
       )
index 00be8531225a7f90e0432eb59064da357b8b769a..3fcc187ae4b7dd90bb0e549ac439fcb4ff589011 100644 (file)
@@ -74,7 +74,7 @@ class CRM_Grant_Form_Task extends CRM_Core_Form {
    *
    * @return void
    */
-  function preProcess() {
+  public function preProcess() {
     self::preProcessCommon($this);
   }
 
index f9ce69dccca59aea7af4e246f3c7d124570ba7ec..3a6503deee2ebe09b41359d54c807850833323b3 100644 (file)
@@ -446,7 +446,7 @@ class CRM_Member_BAO_Membership extends CRM_Member_DAO_Membership {
    *
    * @param int $action
    *
-   * @return Array
+   * @return array
    *   array of contact_id of all related contacts.
    */
   public static function checkMembershipRelationship($membershipId, $contactId, $action = CRM_Core_Action::ADD) {
@@ -597,7 +597,7 @@ INNER JOIN  civicrm_membership_type type ON ( type.id = membership.membership_ty
    *   Membership id that needs to be deleted.
    *
    *
-   * @return $results   no of deleted Membership on success, false otherwise
+   * @return int $results  id of deleted Membership on success, false otherwise
    */
   public static function del($membershipId) {
     //delete related first and then delete parent.
@@ -612,7 +612,7 @@ INNER JOIN  civicrm_membership_type type ON ( type.id = membership.membership_ty
    *   Membership id that needs to be deleted.
    *
    *
-   * @return $results   no of deleted Membership on success, false otherwise
+   * @return int $results id of deleted Membership on success, false otherwise
    */
   public static function deleteMembership($membershipId) {
     // CRM-12147, retrieve membership data before we delete it for hooks
@@ -677,7 +677,7 @@ INNER JOIN  civicrm_membership_type type ON ( type.id = membership.membership_ty
    */
   public static function deleteRelatedMemberships($ownerMembershipId, $contactId = NULL) {
     if (!$ownerMembershipId && !$contactId) {
-      return;
+      return FALSE;
     }
 
     $membership = new CRM_Member_DAO_Membership();
@@ -748,7 +748,7 @@ INNER JOIN  civicrm_membership_type type ON ( type.id = membership.membership_ty
    *   Is this a separate membership payment
    *
    */
-  static function buildMembershipBlock(
+  public static function buildMembershipBlock(
     &$form,
     $pageID,
     $cid,
@@ -1519,7 +1519,7 @@ AND civicrm_membership.is_test = %2";
    * @throws CRM_Core_Exception
    *
    */
-  static function renewMembershipFormWrapper(
+  public static function renewMembershipFormWrapper(
     $contactID,
     $membershipTypeID,
     $is_test,
@@ -1762,7 +1762,7 @@ WHERE  civicrm_membership.contact_id = civicrm_contact.id
     if ($reset) {
       // not sure why a static var is in use here - we need a way to reset it from the test suite
       $relatedContactIds = array();
-      return;
+      return FALSE;
     }
 
     $membership = new CRM_Member_DAO_Membership();
@@ -1898,7 +1898,8 @@ WHERE  civicrm_membership.contact_id = civicrm_contact.id
               CRM_Member_BAO_Membership::create($params, $relMemIds);
               $available--;
             }
-            else { // we have run out of inherited memberships, so delete extras
+            else {
+              // we have run out of inherited memberships, so delete extras
               self::deleteMembership($params['id']);
             }
             // we need to first check if there will remain inherited memberships, so queue it up
@@ -1922,22 +1923,22 @@ WHERE  civicrm_membership.contact_id = civicrm_contact.id
    *
    * @param int $membershipId
    *
-   * @return boolean
-   *   true if deleted false otherwise
+   * @return object
+   *   $membershipPayment deleted membership payment object
    */
   public static function deleteMembershipPayment($membershipId) {
 
-    $membesrshipPayment = new CRM_Member_DAO_MembershipPayment();
-    $membesrshipPayment->membership_id = $membershipId;
-    $membesrshipPayment->find();
+    $membershipPayment = new CRM_Member_DAO_MembershipPayment();
+    $membershipPayment->membership_id = $membershipId;
+    $membershipPayment->find();
 
-    while ($membesrshipPayment->fetch()) {
-      CRM_Contribute_BAO_Contribution::deleteContribution($membesrshipPayment->contribution_id);
-      CRM_Utils_Hook::pre('delete', 'MembershipPayment', $membesrshipPayment->id, $membesrshipPayment);
+    while ($membershipPayment->fetch()) {
+      CRM_Contribute_BAO_Contribution::deleteContribution($membershipPayment->contribution_id);
+      CRM_Utils_Hook::pre('delete', 'MembershipPayment', $membershipPayment->id, $membershipPayment);
       $membesrshipPayment->delete();
-      CRM_Utils_Hook::post('delete', 'MembershipPayment', $membesrshipPayment->id, $membesrshipPayment);
+      CRM_Utils_Hook::post('delete', 'MembershipPayment', $membershipPayment->id, $membershipPayment);
     }
-    return $membesrshipPayment;
+    return $membershipPayment;
   }
 
   /**
@@ -2032,7 +2033,7 @@ FROM   civicrm_membership_type
    *
    * @param bool $isNotCancelled
    *
-   * @return boolean
+   * @return bool
    */
   public static function isCancelSubscriptionSupported($mid, $isNotCancelled = TRUE) {
     $cacheKeyString = "$mid";
index e34f6d1ab87f3e369da6f2e37c2247330858be1c..3cced1d432bba80148770f68436322190d4638d8 100644 (file)
@@ -62,7 +62,6 @@ class CRM_Member_BAO_MembershipLog extends CRM_Member_DAO_MembershipLog {
    *
    * @return mixed
    */
-
   public static function del($membershipID) {
     $membershipLog = new CRM_Member_DAO_MembershipLog();
     $membershipLog->membership_id = $membershipID;
index 62e5e829b052cbfceec30759d3641af195ed284d..9d3e48e71d64d375c7462878d5a2b66612d732b7 100644 (file)
@@ -139,7 +139,7 @@ class CRM_PCP_Page_PCPInfo extends CRM_Core_Page {
           'start_date',
           'end_date',
           'registration_start_date',
-          'registration_end_date'
+          'registration_end_date',
         )
       );
     }
index 3d808283dbcee994b1477e14e4a284924fd227c9..86a75fbceb6914583db7b2198d1c0fc9f590c9bd 100644 (file)
@@ -63,7 +63,7 @@ class CRM_Utils_Geocode_Yahoo {
    * @param bool $stateName
    *   This parameter currently has no function.
    *
-   * @return boolean
+   * @return bool
    *   true if we modified the address, false otherwise
    */
   public static function format(&$values, $stateName = FALSE) {
@@ -118,7 +118,7 @@ class CRM_Utils_Geocode_Yahoo {
 
     $add = 'q=' . urlencode('select * from geo.placefinder where ');
 
-    $add .= join(urlencode(' and '), $whereComponents);
+    $add .= implode(urlencode(' and '), $whereComponents);
 
     $add .= "&appid=" . urlencode($config->mapAPIKey);
 
index 012533eb9bb18244f97f8da7ea1da9ad3d1da2cc..f81cb4bfeb249ca3c93e80e411136b32929a0796 100644 (file)
@@ -94,7 +94,7 @@ class CRM_Utils_Recent {
    *
    * @return void
    */
-  static function add(
+  public static function add(
     $title,
     $url,
     $id,
index 511b8caa67c57572df549a0a8d9fce5391f52f00..5703bd978de9124a94b82cf6174298c89def280a 100644 (file)
@@ -51,8 +51,7 @@ class CRM_Utils_String {
    * Convert a display name into a potential variable
    * name that we could use in forms/code
    *
-   * @param name Name of the string
-   *
+   * @param $title title of the string
    * @param int $maxLength
    *
    * @return string
@@ -209,7 +208,7 @@ class CRM_Utils_String {
    * @param bool $utf8
    *   Attempt utf8 match on failure (default yes).
    *
-   * @return boolean
+   * @return bool
    *   true if string is ascii
    */
   public static function isAscii($str, $utf8 = TRUE) {
@@ -312,7 +311,7 @@ class CRM_Utils_String {
    * @param string $str
    *   Input string.
    *
-   * @return boolean
+   * @return bool
    */
   public static function isUtf8($str) {
     if (!function_exists(mb_detect_encoding)) {
@@ -342,7 +341,7 @@ class CRM_Utils_String {
    * @param string $url2
    *   The second url to be matched against.
    *
-   * @return boolean
+   * @return bool
    *   true if the urls match, else false
    */
   public static function match($url1, $url2) {
@@ -391,7 +390,7 @@ class CRM_Utils_String {
    * @param string $str
    *   The string to be translated.
    *
-   * @return boolean
+   * @return bool
    */
   public static function strtobool($str) {
     if (!is_scalar($str)) {
@@ -410,7 +409,7 @@ class CRM_Utils_String {
    * @param string $str
    *   The string to be translated.
    *
-   * @return boolean
+   * @return bool
    */
   public static function strtoboolstr($str) {
     if (!is_scalar($str)) {
@@ -571,7 +570,7 @@ class CRM_Utils_String {
    * @return string
    *   the sanitized string
    */
-  static function stripPathChars(
+  public static function stripPathChars(
     $string,
     $search = NULL,
     $replace = NULL
@@ -737,7 +736,7 @@ class CRM_Utils_String {
    * @param bool $case
    *   Boolean indicating whether you want the comparison to be case sensitive or not.
    *
-   * @return boolean
+   * @return bool
    *   TRUE (string are identical); FALSE (strings are not identical)
    */
   public static function compareStr($strOne, $strTwo, $case) {
@@ -771,7 +770,7 @@ class CRM_Utils_String {
    * (because most other odd characters are %-escaped in URLs; and %-escaped
    * strings don't need any extra escaping in HTML).
    *
-   * @param string $url
+   * @param string $htmlUrl
    *   URL with HTML entities.
    * @return string
    *   URL without HTML entities
index f958eb663af1eff6d5ef7374b049e4b54f1206c3..cc5d2dd21fe9cf66a64f98f657dd051249cb6d72 100644 (file)
@@ -84,7 +84,7 @@ class Analyzer {
       $activityStatusGroup = civicrm_api3('option_group', 'get', array('name' => 'activity_status'));
       $activityStatus = array(
         'name' => $status,
-        'option_group_id' => $activityStatusGroup['id']
+        'option_group_id' => $activityStatusGroup['id'],
       );
       $activityStatusID = civicrm_api3('option_value', 'get', $activityStatus);
       $activityStatusID = $activityStatusID['values'][$activityStatusID['id']]['value'];
index af6c14f63c24aa5960ab630308eaed78ed9892ca..0fc93821d6e46f071c9410826416a79b253cb40b 100644 (file)
@@ -63,7 +63,7 @@ function _civicrm_api3_job_create_spec(&$params) {
  *
  * @return array
  *
- * {@getfields Job_create}
+ *   {@getfields Job_create}
  */
 function civicrm_api3_job_create($params) {
   return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -76,7 +76,7 @@ function civicrm_api3_job_create($params) {
  *   input parameters
  * @return array
  *
- * {@getfields email_get}
+ *   {@getfields email_get}
  */
 function civicrm_api3_job_get($params) {
   return _civicrm_api3_basic_get(_civicrm_api3_get_BAO(__FUNCTION__), $params);
@@ -103,8 +103,9 @@ function civicrm_api3_job_delete($params) {
  * @return array
  *   API Result Array
  */
-function civicrm_api3_job_execute(/** @noinspection PhpUnusedParameterInspection */
-  $params) {
+function civicrm_api3_job_execute($params) {
+  // @noinspection PhpUnusedParameterInspection
+
   $facility = new CRM_Core_JobManager();
   $facility->execute(FALSE);
 
@@ -412,8 +413,7 @@ function civicrm_api3_job_process_participant($params) {
  * @return bool
  *   true if success, else false
  */
-function civicrm_api3_job_process_membership(/** @noinspection PhpUnusedParameterInspection */
-  $params) {
+function civicrm_api3_job_process_membership($params) {
   $lock = new CRM_Core_Lock('civimail.job.updateMembership');
   if (!$lock->isAcquired()) {
     return civicrm_api3_create_error('Could not acquire lock, another Membership Processing process is running');
@@ -491,9 +491,7 @@ function _civicrm_api3_job_process_batch_merge_spec(&$params) {
   );
   $params['mode'] = array(
     'title' => 'Mode',
-    'description' => 'helps decide how to behave when there are conflicts.
-       A \'safe\' value skips the merge if there are no conflicts.
-       Does a force merge otherwise.',
+    'description' => 'helps decide how to behave when there are conflicts. A \'safe\' value skips the merge if there are no conflicts. Does a force merge otherwise.',
     'type' => CRM_Utils_Type::T_STRING,
   );
   $params['auto_flip'] = array(
@@ -575,8 +573,8 @@ function civicrm_api3_job_cleanup($params) {
  * @return array
  * @throws \API_Exception
  */
-function civicrm_api3_job_disable_expired_relationships(/** @noinspection PhpUnusedParameterInspection */
-  $params) {
+function civicrm_api3_job_disable_expired_relationships($params) {
+  /** @noinspection PhpUnusedParameterInspection */
   $result = CRM_Contact_BAO_Relationship::disableExpiredRelationships();
   if (!$result) {
     throw new API_Exception('Failed to disable all expired relationships.');
index 7f7573040bf2b6edc0e336e4c53e9565bf092325..c64305b2d03d636aabacdd48aa68c283e4ebfcdf 100644 (file)
@@ -55,7 +55,7 @@ function _civicrm_api3_uf_group_create_spec(&$params) {
  *
  * @return array
  *   API result array
- * {@getfields UFGroup_create}
+ *   {@getfields UFGroup_create}
  * @example UFGroupCreate.php
  */
 function civicrm_api3_uf_group_create($params) {
@@ -72,7 +72,7 @@ function civicrm_api3_uf_group_create($params) {
  *
  * @return array
  *   Array of matching profiles
- * {@getfields UFGroup_get}
+ *   {@getfields UFGroup_get}
  * @example UFGroupGet.php
  */
 function civicrm_api3_uf_group_get($params) {
index c29eb0559c14faabaf0fb9a4df1bfde4aa5a2b85..481bb045ef76586bff418c0aa581c3b6ad946243 100644 (file)
@@ -30,6 +30,9 @@
  * in order to correct the condition where a contribution row is linked to > 1 membership.
  */
 
+/**
+ * Initialization
+ */
 function initialize() {
   session_start();
   if (!function_exists('drush_get_context')) {
index ac956e111210ac0bc74b324c4bdd222d9364132f..d4e37310a55abd741d2a4ca04d4d2253e0b8c5fb 100644 (file)
@@ -9,7 +9,7 @@ class test_extension_manager_searchtest extends CRM_Contact_Form_Search_Custom_B
   /**
    * @param $formValues
    */
-  function __construct(&$formValues) {
+  public function __construct(&$formValues) {
     parent::__construct($formValues);
   }
 
@@ -20,7 +20,7 @@ class test_extension_manager_searchtest extends CRM_Contact_Form_Search_Custom_B
    *   Modifiable.
    * @return void
    */
-  function buildForm(&$form) {
+  public function buildForm(&$form) {
     CRM_Utils_System::setTitle(ts('My Search Title'));
 
     $form->add('text',
@@ -52,7 +52,7 @@ class test_extension_manager_searchtest extends CRM_Contact_Form_Search_Custom_B
    *   - summary: string
    *   - total: numeric
    */
-  function summary() {
+  public function summary() {
     return NULL;
     // return array(
     //   'summary' => 'This is a summary',
@@ -65,7 +65,7 @@ class test_extension_manager_searchtest extends CRM_Contact_Form_Search_Custom_B
    *
    * @return array, keys are printable column headers and values are SQL column names
    */
-  function &columns() {
+  public function &columns() {
     // return by reference
     $columns = array(
       ts('Contact ID') => 'contact_id',
@@ -86,7 +86,7 @@ class test_extension_manager_searchtest extends CRM_Contact_Form_Search_Custom_B
    *
    * @return string, sql
    */
-  function all($offset = 0, $rowcount = 0, $sort = NULL, $includeContactIDs = FALSE) {
+  public function all($offset = 0, $rowcount = 0, $sort = NULL, $includeContactIDs = FALSE) {
     // delegate to $this->sql(), $this->select(), $this->from(), $this->where(), etc.
     return $this->sql($this->select(), $offset, $rowcount, $sort, $includeContactIDs, NULL);
   }
@@ -96,7 +96,7 @@ class test_extension_manager_searchtest extends CRM_Contact_Form_Search_Custom_B
    *
    * @return string, sql fragment with SELECT arguments
    */
-  function select() {
+  public function select() {
     return "
       contact_a.id           as contact_id  ,
       contact_a.contact_type as contact_type,
@@ -110,7 +110,7 @@ class test_extension_manager_searchtest extends CRM_Contact_Form_Search_Custom_B
    *
    * @return string, sql fragment with FROM and JOIN clauses
    */
-  function from() {
+  public function from() {
     return "
       FROM      civicrm_contact contact_a
       LEFT JOIN civicrm_address address ON ( address.contact_id       = contact_a.id AND
@@ -128,7 +128,7 @@ class test_extension_manager_searchtest extends CRM_Contact_Form_Search_Custom_B
    *
    * @return string, sql fragment with conditional expressions
    */
-  function where($includeContactIDs = FALSE) {
+  public function where($includeContactIDs = FALSE) {
     $params = array();
     $where = "contact_a.contact_type   = 'Household'";
 
@@ -172,7 +172,7 @@ class test_extension_manager_searchtest extends CRM_Contact_Form_Search_Custom_B
    *
    * @return string, template path (findable through Smarty template path)
    */
-  function templateFile() {
+  public function templateFile() {
     return 'CRM/Contact/Form/Search/Custom.tpl';
   }
 
@@ -183,7 +183,7 @@ class test_extension_manager_searchtest extends CRM_Contact_Form_Search_Custom_B
    *   Modifiable SQL result row.
    * @return void
    */
-  function alterRow(&$row) {
+  public function alterRow(&$row) {
     $row['sort_name'] .= ' ( altered )';
   }
 
index a3e808f1ee1e39a742d91e91b0e6f23affec5443..f7471db476722b1664cc92ac2106c7fef9bb47cf 100644 (file)
@@ -47,7 +47,7 @@ class CRM_Contacts_AllTests extends CiviTestSuite {
    */
   private static function getInstance() {
     if (is_null(self::$instance)) {
-      self::$instance = new self;
+      self::$instance = new CiviTestSuite();
     }
     return self::$instance;
   }
index 980d0ae60d72e48a6cb9cedf06b0100185023cb0..e0a204b820bc102fc9a285dce29d12a25a600612 100644 (file)
@@ -205,19 +205,19 @@ class CRM_Core_TransactionTest extends CiviUnitTestCase {
 
     CRM_Core_Transaction::addCallback(CRM_Core_Transaction::PHASE_PRE_COMMIT, array($this, '_preCommit'), array(
         'qwe',
-        'rty'
+        'rty',
       ));
     CRM_Core_Transaction::addCallback(CRM_Core_Transaction::PHASE_POST_COMMIT, array($this, '_postCommit'), array(
         'uio',
-        'p[]'
+        'p[]',
       ));
     CRM_Core_Transaction::addCallback(CRM_Core_Transaction::PHASE_PRE_ROLLBACK, array(
         $this,
-        '_preRollback'
+        '_preRollback',
       ), array('asd', 'fgh'));
     CRM_Core_Transaction::addCallback(CRM_Core_Transaction::PHASE_POST_ROLLBACK, array(
         $this,
-        '_postRollback'
+        '_postRollback',
       ), array('jkl', ';'));
 
     CRM_Core_DAO::executeQuery('UPDATE civicrm_contact SET id = 100 WHERE id = 100');
@@ -233,19 +233,19 @@ class CRM_Core_TransactionTest extends CiviUnitTestCase {
 
     CRM_Core_Transaction::addCallback(CRM_Core_Transaction::PHASE_PRE_COMMIT, array($this, '_preCommit'), array(
         'ewq',
-        'ytr'
+        'ytr',
       ));
     CRM_Core_Transaction::addCallback(CRM_Core_Transaction::PHASE_POST_COMMIT, array($this, '_postCommit'), array(
         'oiu',
-        '][p'
+        '][p',
       ));
     CRM_Core_Transaction::addCallback(CRM_Core_Transaction::PHASE_PRE_ROLLBACK, array(
         $this,
-        '_preRollback'
+        '_preRollback',
       ), array('dsa', 'hgf'));
     CRM_Core_Transaction::addCallback(CRM_Core_Transaction::PHASE_POST_ROLLBACK, array(
         $this,
-        '_postRollback'
+        '_postRollback',
       ), array('lkj', ';'));
 
     CRM_Core_DAO::executeQuery('UPDATE civicrm_contact SET id = 100 WHERE id = 100');
@@ -395,8 +395,7 @@ class CRM_Core_TransactionTest extends CiviUnitTestCase {
    *   See assertContactsMix.
    * @param string $outcome
    *   'rollback'|'implicit-commit'|'explicit-commit' how to finish transaction.
-   * @return int
-   *   cid
+   * @return void
    */
   public function runBatch($nesting, $callbacks, $existsByOffset, $outcome) {
     if ($nesting != 'reuse-tx' && $nesting != 'nest-tx') {
index 0052ed6483b073ce871b214062ea7ac91a3f3209..baf41b6a947caab5331ec284770810d0ea78152a 100644 (file)
@@ -11,7 +11,7 @@ class CRM_Mailing_BAO_QueryTest extends CiviUnitTestCase {
    * @return CRM_Mailing_BAO_QueryTestDataProvider
    */
   public function dataProvider() {
-    return new CRM_Mailing_BAO_QueryTestDataProvider;
+    return new CRM_Mailing_BAO_QueryTestDataProvider();
   }
 
   public function setUp() {
index dc1a56b1cf3df3303eba6286af133dd354468e82..56d135d3990fd75f26a534b0a508f79132e010d4 100644 (file)
@@ -47,7 +47,7 @@ class Civi_AllTests extends CiviTestSuite {
    */
   private static function getInstance() {
     if (is_null(self::$instance)) {
-      self::$instance = new self;
+      self::$instance = new CiviTestSuite();
     }
     return self::$instance;
   }