INFRA-132 - s/true/TRUE/
authorTim Otten <totten@civicrm.org>
Fri, 9 Jan 2015 02:06:41 +0000 (18:06 -0800)
committerTim Otten <totten@civicrm.org>
Fri, 9 Jan 2015 02:06:41 +0000 (18:06 -0800)
19 files changed:
CRM/Contact/BAO/Contact/Optimizer.php
CRM/Contact/BAO/Group.php
CRM/Contact/BAO/GroupContact.php
CRM/Contact/BAO/GroupContactCache.php
CRM/Contact/Form/Edit/Individual.php
CRM/Contact/Form/Inline/Website.php
CRM/Contact/Form/Search.php
CRM/Contact/Form/Search/Builder.php
CRM/Contact/Form/Search/Criteria.php
CRM/Contact/Form/Task/EmailCommon.php
CRM/Contact/Form/Task/PDFLetterCommon.php
CRM/Contribute/Form/UpdateSubscription.php
CRM/Core/BAO/FinancialTrxn.php
CRM/Core/BAO/MessageTemplate.php
CRM/Core/BAO/Navigation.php
CRM/Core/BAO/SchemaHandler.php
CRM/Core/Invoke.php
CRM/Event/Form/Registration/Register.php
CRM/Event/Page/EventInfo.php

index bcf8a33157eca3b8772bfa1c5446cae97df75aef..6cf962ba1b4d6e56000bc4884b72f47d0e44b4f5 100644 (file)
@@ -58,7 +58,7 @@ class CRM_Contact_BAO_Contact_Optimizer {
     }
 
     // check if we had a value in the old
-    $oldEmpty = $newEmpty = true;
+    $oldEmpty = $newEmpty = TRUE;
     $old = $new = array();
 
     foreach ($oldWebsiteValues as $idx => $value) {
@@ -93,7 +93,7 @@ class CRM_Contact_BAO_Contact_Optimizer {
           $old['website_type_id'] == $new['website_type_id'] &&
           $old['url'] == $new['url']
         ) {
-          $found = true;
+          $found = TRUE;
           unset($new[$newID]);
           break;
         }
@@ -121,7 +121,7 @@ class CRM_Contact_BAO_Contact_Optimizer {
     }
 
     // check if we had a value in the old
-    $oldEmpty = $newEmpty = true;
+    $oldEmpty = $newEmpty = TRUE;
     $old = $new = array();
 
     foreach ($oldEmailValues as $idx => $value) {
@@ -172,7 +172,7 @@ class CRM_Contact_BAO_Contact_Optimizer {
           $old['email_type_id'] == $new['email_type_id'] &&
           $old['url'] == $new['url']
         ) {
-          $found = true;
+          $found = TRUE;
           unset($new[$newID]);
           break;
         }
index 843fca7c4d34ba04bba63fbc480051f386b2bfb5..c60eff217f99ccfab335735a097703b26b052538 100644 (file)
@@ -941,7 +941,7 @@ class CRM_Contact_BAO_Group extends CRM_Contact_DAO_Group {
         $values[$object->id]['is_parent'] = false;
         if (array_key_exists('children', $values[$object->id])) {
           $values[$object->id]['class'][] = "crm-group-parent";
-          $values[$object->id]['is_parent'] = true;
+          $values[$object->id]['is_parent'] = TRUE;
         }
 
         // If group is a child, add child class
index 7a347b5a24701b3eb169fb3d7de31291950993f6..863ad13a4aadefa17efbf6e7f34c9f24bca7af7d 100644 (file)
@@ -332,7 +332,7 @@ 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
+   *                                 the total count when $count is TRUE
    *
    * $access public
    */
index 2ba68e681f3197f0179bdfdb324da00bb420b619..8948c33c977d79781a9c0a8fc5b59ca87f2091e5 100644 (file)
@@ -43,7 +43,7 @@ class CRM_Contact_BAO_GroupContactCache extends CRM_Contact_DAO_GroupContactCach
    * @param $groupIDs
    *   Of group that we are checking against.
    *
-   * @return boolean true if we did not regenerate, false if we did
+   * @return boolean TRUE if we did not regenerate, false if we did
    */
   public static function check($groupIDs) {
     if (empty($groupIDs)) {
@@ -121,7 +121,7 @@ AND     ( g.cache_date IS NULL OR
    * @param int $limit
    *   Limits the number of groups we evaluate.
    *
-   * @return boolean true if we did not regenerate, false if we did
+   * @return boolean TRUE if we did not regenerate, false if we did
    */
   public static function loadAll($groupIDs = NULL, $limit = 0) {
     // ensure that all the smart groups are loaded
@@ -291,7 +291,7 @@ WHERE  id IN ( $groupIDs )
     static $invoked = FALSE;
 
     // typically this needs to happy only once per instance
-    // this is especially true in import, where we dont need
+    // this is especially TRUE in import, where we dont need
     // to do this all the time
     // this optimization is done only when no groupID is passed
     // i.e. cache is reset for all groups
@@ -397,7 +397,8 @@ WHERE  id = %1
    * Removes one or more contacts from the smart group cache
    * @param int|array $cid
    * @param int $groupId
-   * @return bool - true if successful
+   * @return bool
+   *   TRUE if successful.
    */
   public static function removeContact($cid, $groupId = NULL) {
     $cids = array();
index 627af134b9ee088bf5e1d32bd5565c288681e98f..e3d52678e088dab17ea08163877cd00eb1b86019 100644 (file)
@@ -152,7 +152,8 @@ class CRM_Contact_Form_Edit_Individual {
    *   The uploaded files if any.
    * @param int $contactID
    *
-   * @return true if no errors, else array of errors
+   * @return bool
+   *   TRUE if no errors, else array of errors.
    * @static
    */
   public static function formRule($fields, $files, $contactID = NULL) {
index 5f9e9394f499e0fd4957a4cdd8839a2f0d76c514..f01c648a7502d555f6e9c519f944babda0e4f82f 100644 (file)
@@ -123,7 +123,7 @@ class CRM_Contact_Form_Inline_Website extends CRM_Contact_Form_Inline {
     $params = $this->exportValues();
 
     // Process / save websites
-    CRM_Core_BAO_Website::create($params['website'], $this->_contactId, true);
+    CRM_Core_BAO_Website::create($params['website'], $this->_contactId, TRUE);
 
     $this->log();
     $this->response();
index 66c096b2d8dad347f9d678ed888913c9a762c794..8c6cf410f533235eb82a3a93bd78b6d9382fe68e 100644 (file)
@@ -533,8 +533,8 @@ class CRM_Contact_Form_Search extends CRM_Core_Form_Search {
 
     // get user submitted values
     // get it from controller only if form has been submitted, else preProcess has set this
-    // $this->controller->isModal( ) returns true if page is
-    // valid, i.e all the validations are true
+    // $this->controller->isModal( ) returns TRUE if page is
+    // valid, i.e all the validations are TRUE
 
     if (!empty($_POST) && !$this->controller->isModal()) {
       $this->_formValues = $this->controller->exportValues($this->_name);
@@ -658,7 +658,7 @@ class CRM_Contact_Form_Search extends CRM_Core_Form_Search {
         $this->_params,
         $this->_returnProperties,
         $this->_action,
-        false, true,
+        false, TRUE,
         $this->_context,
         $this->_contextMenu
       );
index cfe0504ae4cb9046dfef3f4381f1d1f5dfaaefa4..e9850f164c32298d5094d39d2a0d10d9c0c0de32 100644 (file)
@@ -472,7 +472,7 @@ class CRM_Contact_Form_Search_Builder extends CRM_Contact_Form_Search {
    */
   public static function checkArrayKeyEmpty($val) {
     if (is_array($val)) {
-      $v2empty = true;
+      $v2empty = TRUE;
       foreach ($val as $vk => $vv) {
         if (!empty($vk)) {
           $v2empty = false;
index 767b2620423e890c9c2bedaede5b2a1702b4ba53..b491667cac7de9d1aae309c9caf50d947f7c0f76 100644 (file)
@@ -94,7 +94,7 @@ class CRM_Contact_Form_Search_Criteria {
         if (count($tags) && $key != 'civicrm_file' && $key != 'civicrm_contact') {
           //if tags exists then add type to display in adv search form help text
           $tagsTypes[] = ts($value);
-          $showAllTagTypes = true;
+          $showAllTagTypes = TRUE;
         }
       }
       $tagTypesText = implode(" or ", $tagsTypes);
index e715b320f98c3f7e62a01bed9fca20d469558a82..d1afbf7315b4054b3a6b9608f21ba7ebba0cec46 100644 (file)
@@ -324,7 +324,7 @@ class CRM_Contact_Form_Task_EmailCommon {
    * @param array $self
    *   Additional values form 'this'.
    *
-   * @return true if no errors, else array of errors
+   * @return TRUE if no errors, else array of errors
    *
    */
   public static function formRule($fields, $dontCare, $self) {
@@ -376,7 +376,7 @@ class CRM_Contact_Form_Task_EmailCommon {
         $allEmails = explode(',', $formValues[$element]);
         foreach ($allEmails as $value) {
           list($contactId, $email) = explode('::', $value);
-          $contactURL = CRM_Utils_System::url('civicrm/contact/view', "reset=1&force=1&cid={$contactId}", true);
+          $contactURL = CRM_Utils_System::url('civicrm/contact/view', "reset=1&force=1&cid={$contactId}", TRUE);
           switch ($element) {
             case 'cc_id':
               $ccValues['email'][] = '"' . $form->_contactDetails[$contactId]['sort_name'] . '" <' . $email . '>';
index 7ce4c846ec864bc4dbfa545667989bd6eb456059..2aaa42f5c90c4d6c0996a963f0423a3b24935acd 100644 (file)
@@ -209,7 +209,8 @@ class CRM_Contact_Form_Task_PDFLetterCommon {
    * @param array $self
    *   Additional values form 'this'.
    *
-   * @return true if no errors, else array of errors
+   * @return bool
+   *   TRUE if no errors, else array of errors.
    *
    */
   public static function formRule($fields, $dontCare, $self) {
index 7b934fbcec646a2759ab3f42e89f432526ad1f5b..3c47ffdc0832ed60fcfdecd3827941087927abbe 100644 (file)
@@ -199,7 +199,7 @@ class CRM_Contribute_Form_UpdateSubscription extends CRM_Core_Form {
     $message = '';
 
     $params['subscriptionId'] = $this->_subscriptionDetails->subscription_id;
-    $updateSubscription = true;
+    $updateSubscription = TRUE;
     if ($this->_paymentProcessorObj->isSupported('changeSubscriptionAmount')) {
         $updateSubscription = $this->_paymentProcessorObj->changeSubscriptionAmount($message, $params);
     }
index e59cea439cc252b47a1aeacb2170b30c5632780f..9edc2974c9115bcbb270cdb747f28124c803b79b 100644 (file)
@@ -133,7 +133,7 @@ class CRM_Core_BAO_FinancialTrxn extends CRM_Financial_DAO_FinancialTrxn {
   public static function retrieve(&$params, &$defaults) {
     $financialItem = new CRM_Financial_DAO_FinancialTrxn();
     $financialItem->copyValues($params);
-    if ($financialItem->find(true)) {
+    if ($financialItem->find(TRUE)) {
       CRM_Core_DAO::storeValues($financialItem, $defaults);
       return $financialItem;
     }
@@ -296,7 +296,8 @@ WHERE lt.entity_id = %1 ";
    * Delete financial transaction
    *
    * @param int $entity_id
-   * @return true on success, false otherwise
+   * @return bool
+   *   TRUE on success, false otherwise.
    * @static
    */
   public static function deleteFinancialTrxn($entity_id) {
index a205b8cfabacf134a504f3b0190c3cf1bf502645..f68ed7b97fef064ada49beffda30209f7eea2e07 100644 (file)
@@ -221,10 +221,10 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
         $bodyType = "body_{$value}";
         if ($$bodyType) {
           CRM_Utils_Token::replaceGreetingTokens($$bodyType, NULL, $contact['contact_id']);
-          $$bodyType = CRM_Utils_Token::replaceDomainTokens($$bodyType, $domain, true, $tokens, true);
-          $$bodyType = CRM_Utils_Token::replaceContactTokens($$bodyType, $contact, false, $tokens, false, true);
-          $$bodyType = CRM_Utils_Token::replaceComponentTokens($$bodyType, $contact, $tokens, true);
-          $$bodyType = CRM_Utils_Token::replaceHookTokens($$bodyType, $contact , $categories, true);
+          $$bodyType = CRM_Utils_Token::replaceDomainTokens($$bodyType, $domain, TRUE, $tokens, TRUE);
+          $$bodyType = CRM_Utils_Token::replaceContactTokens($$bodyType, $contact, false, $tokens, false, TRUE);
+          $$bodyType = CRM_Utils_Token::replaceComponentTokens($$bodyType, $contact, $tokens, TRUE);
+          $$bodyType = CRM_Utils_Token::replaceHookTokens($$bodyType, $contact , $categories, TRUE);
         }
       }
       $html = $body_html;
@@ -238,9 +238,9 @@ class CRM_Core_BAO_MessageTemplate extends CRM_Core_DAO_MessageTemplate {
 
       // do replacements in message subject
       $messageSubject = CRM_Utils_Token::replaceContactTokens($body_subject, $contact, false, $tokens);
-      $messageSubject = CRM_Utils_Token::replaceDomainTokens($messageSubject, $domain, true, $tokens);
-      $messageSubject = CRM_Utils_Token::replaceComponentTokens($messageSubject, $contact, $tokens, true);
-      $messageSubject = CRM_Utils_Token::replaceHookTokens($messageSubject, $contact, $categories, true);
+      $messageSubject = CRM_Utils_Token::replaceDomainTokens($messageSubject, $domain, TRUE, $tokens);
+      $messageSubject = CRM_Utils_Token::replaceComponentTokens($messageSubject, $contact, $tokens, TRUE);
+      $messageSubject = CRM_Utils_Token::replaceHookTokens($messageSubject, $contact, $categories, TRUE);
 
       $messageSubject = $smarty->fetch("string:{$messageSubject}");
 
index 227c9ac7436ea485ee270a345ebce9eafadd2eda..a608743a171a29993e9c6615bcc80facb85b2fd0 100644 (file)
@@ -739,7 +739,7 @@ ORDER BY parent_id, weight";
       $parentClause = 'parent_id IS NULL';
     }
 
-    $incrementOtherNodes = true;
+    $incrementOtherNodes = TRUE;
     $sql    = "SELECT weight from civicrm_navigation WHERE {$parentClause} ORDER BY weight LIMIT %1, 1";
     $params = array(1 => array($position, 'Positive'));
     $newWeight = CRM_Core_DAO::singleValueQuery($sql, $params);
index ada9528dbe6533580f6862b794c73f8f25e218e2..6ceae0068acb0091d66e33276a33bc70763b2e39 100644 (file)
@@ -51,7 +51,7 @@
  *                                             'class'         => FIELD_CLASS_TYPE,
  *                                             'primary'       => BOOLEAN,
  *                                             'required'      => BOOLEAN,
- *                                             'searchable'    => true,
+ *                                             'searchable'    => TRUE,
  *                                             'fk_table_name' => FOREIGN_KEY_TABLE_NAME,
  *                                             'fk_field_name' => FOREIGN_KEY_FIELD_NAME,
  *                                             'comment'       => COMMENT,
@@ -66,7 +66,7 @@ class CRM_Core_BAO_SchemaHandler {
    *
    * @param array $params
    *
-   * @return true if successfully created, false otherwise
+   * @return TRUE if successfully created, false otherwise
    *
    * @static
    */
@@ -366,7 +366,8 @@ ALTER TABLE {$tableName}
    * @param String $tableName
    *   Name of the table to be created.
    *
-   * @return true if successfully deleted, false otherwise
+   * @return bool
+   *   TRUE if successfully deleted, false otherwise.
    *
    * @static
    */
index 667b9c8ea4f26d4b827b9dec8902b818d4724e2f..73a35cc9a49d1620bb6cab437edc8bcecd473e43 100644 (file)
@@ -86,7 +86,7 @@ class CRM_Core_Invoke {
       // Symfony-based invocation path
       require_once CIVICRM_SYMFONY_PATH . '/app/bootstrap.php.cache';
       require_once CIVICRM_SYMFONY_PATH . '/app/AppKernel.php';
-      $kernel = new AppKernel('dev', true);
+      $kernel = new AppKernel('dev', TRUE);
       $kernel->loadClassCache();
       $response = $kernel->handle(Symfony\Component\HttpFoundation\Request::createFromGlobals());
       if (preg_match(':^text/html:', $response->headers->get('Content-Type'))) {
@@ -299,7 +299,7 @@ class CRM_Core_Invoke {
             $addSequence = $addSequence ? 'true' : 'false';
             unset($pageArgs['addSequence']);
           }
-          $object = new $item['page_callback'] ($title, true, $mode, NULL, $addSequence);
+          $object = new $item['page_callback'] ($title, TRUE, $mode, NULL, $addSequence);
         }
         else {
           CRM_Core_Error::fatal();
index f67dceeeecedd7a80e98ca34f18e38f31f213f21..ccad8f888bcec33689525fcf96b4b5eb2d5c4e96 100644 (file)
@@ -613,13 +613,13 @@ class CRM_Event_Form_Registration_Register extends CRM_Event_Form_Registration {
       // CRM-14492 Admin price fields should show up on event registration if user has 'administer CiviCRM' permissions
       $adminFieldVisible = false;
       if (CRM_Core_Permission::check('administer CiviCRM')) {
-        $adminFieldVisible = true;
+        $adminFieldVisible = TRUE;
       }
 
       foreach ($form->_feeBlock as $field) {
         // public AND admin visibility fields are included for back-office registration and back-office change selections
         if (CRM_Utils_Array::value('visibility', $field) == 'public' ||
-          (CRM_Utils_Array::value('visibility', $field) == 'admin' && $adminFieldVisible == true) ||
+          (CRM_Utils_Array::value('visibility', $field) == 'admin' && $adminFieldVisible == TRUE) ||
           $className == 'CRM_Event_Form_Participant' ||
           $className == 'CRM_Event_Form_ParticipantFeeSelection'
         ) {
@@ -803,7 +803,7 @@ class CRM_Event_Form_Registration_Register extends CRM_Event_Form_Registration {
    * @param $self
    *
    *
-   * @return true if no errors, else array of errors
+   * @return TRUE if no errors, else array of errors
    * @static
    */
   public static function formRule($fields, $files, $self) {
index 8d9583ba9a81ebcc5ad9617727aad70561812c53..3bdf94b7f9e76c8ed4a15dffc5238608b9e3bae4 100644 (file)
@@ -122,7 +122,7 @@ class CRM_Event_Page_EventInfo extends CRM_Core_Page {
           // CRM-14492 Admin price fields should show up on event registration if user has 'administer CiviCRM' permissions
           $adminFieldVisible = false;
           if (CRM_Core_Permission::check('administer CiviCRM')) {
-            $adminFieldVisible = true;
+            $adminFieldVisible = TRUE;
           }
 
           foreach ($priceSetFields as $fid => $fieldValues) {