Misc ts fixes
authorMathieu Lutfy <mathieu@symbiotic.coop>
Sun, 18 Jun 2023 19:29:13 +0000 (15:29 -0400)
committerMathieu Lutfy <mathieu@bidon.ca>
Mon, 19 Jun 2023 12:52:09 +0000 (08:52 -0400)
12 files changed:
CRM/Admin/Form/OptionGroup.php
CRM/Contact/BAO/Query.php
CRM/Financial/BAO/EntityFinancialAccount.php
CRM/Member/BAO/Membership.php
CRM/Report/Form/Contribute/Detail.php
CRM/Upgrade/Incremental/php/FiveFiftyOne.php
CRM/Utils/Check/Component/AddressParsing.php
CRM/Utils/Check/Component/Case.php
CRM/Utils/Check/Component/PriceFields.php
CRM/Utils/Check/Component/Schema.php
CRM/Utils/Check/Component/Security.php
CRM/Utils/Type.php

index fbf87dfb07cc9b95f472f1e2def8b03221c39ddd..31876b79099431f86a24e3c1d10383c58a74860b 100644 (file)
@@ -108,7 +108,7 @@ class CRM_Admin_Form_OptionGroup extends CRM_Admin_Form {
       $name = CRM_Utils_String::titleToVar(strtolower($fields['title']));
     }
     if (!CRM_Core_DAO::objectExists($name, 'CRM_Core_DAO_OptionGroup', $self->_id)) {
-      $errors['title'] = ts('Option Group name ' . $name . ' already exists in the database. Option Group Names need to be unique');
+      $errors['title'] = ts("Option Group name '%1' already exists in the database. Option Group Names must be unique.", [1 => $name]);
     }
     return empty($errors) ? TRUE : $errors;
   }
index d6c875142a85042caed0fb66b9ce5595b81d5ba5..6ebc10a0eaf029c94eca9874cf07db8b9f75976c 100644 (file)
@@ -4261,7 +4261,7 @@ WHERE  $smartGroupClause
 
     // Description
     if (!empty($description[2]) && trim($description[2])) {
-      $this->_qill[$grouping][] = ts('Relationship description - ' . $description[2]);
+      $this->_qill[$grouping][] = ts('Relationship description - %1', [1 => $description[2]]);
       $description = CRM_Core_DAO::escapeString(trim($description[2]));
       $where[$grouping][] = "civicrm_relationship.description LIKE '%{$description}%'";
     }
index 283247a516833050db3d411e6a595089fc61ee16..f7ba3dd7cde40f7ee33f9935931270e2fc8f76ce 100644 (file)
@@ -267,10 +267,7 @@ class CRM_Financial_BAO_EntityFinancialAccount extends CRM_Financial_DAO_EntityF
     $financialAccountType = CRM_Core_DAO::getFieldValue('CRM_Financial_DAO_FinancialAccount', $financialTypeAccount->financial_account_id, 'financial_account_type_id');
     if (CRM_Utils_Array::value($financialTypeAccount->account_relationship, $financialAccountLinks) != $financialAccountType) {
       $accountRelationships = CRM_Core_PseudoConstant::get('CRM_Financial_DAO_EntityFinancialAccount', 'account_relationship');
-      $params = [
-        1 => $accountRelationships[$financialTypeAccount->account_relationship],
-      ];
-      throw new CRM_Core_Exception(ts("This financial account cannot have '%1' relationship.", $params));
+      throw new CRM_Core_Exception(ts("This financial account cannot have '%1' relationship.", [1 => $accountRelationships[$financialTypeAccount->account_relationship]]));
     }
   }
 
index 303af1714100c1df028bfd67dd6fe6597087524f..f5d2bf8999933442b3b843defd57338c483c9f29 100644 (file)
@@ -298,7 +298,7 @@ class CRM_Member_BAO_Membership extends CRM_Member_DAO_Membership {
           'now', $excludeIsAdmin, $params['membership_type_id'] ?? NULL, $params
         );
         if (empty($calcStatus)) {
-          throw new CRM_Core_Exception(ts("The membership cannot be saved because the status cannot be calculated for start_date: {$params['start_date']} end_date {$params['end_date']} join_date {$params['join_date']} as at " . CRM_Utils_Time::date('Y-m-d H:i:s')));
+          throw new CRM_Core_Exception(ts("The membership cannot be saved because the status cannot be calculated for start_date: %1, end_date: %2, join_date: %3. Current time is: %4.", [1 => $params['start_date'], 2 => $params['end_date'], 3 => $params['join_date'], 4 => CRM_Utils_Time::date('Y-m-d H:i:s')]));
         }
         $params['status_id'] = $calcStatus['id'];
       }
index 6019750b5ea58b0a3e9b7a6e3753e4d83f06a1ba..04667ec3c44272cfe249e366ce0bf5dd5c260bda 100644 (file)
@@ -935,14 +935,10 @@ WHERE  civicrm_contribution_contribution_id={$row['civicrm_contribution_contribu
         // ts exception to avoid having ts("%1 %2: %3")
         $title = '%1 contributions / soft-credits: %2';
 
-        if (CRM_Utils_Array::value('contribution_or_soft_value', $this->_params) ==
-          'contributions_only'
-        ) {
+        if (CRM_Utils_Array::value('contribution_or_soft_value', $this->_params) == 'contributions_only') {
           $title = '%1 contributions: %2';
         }
-        elseif (CRM_Utils_Array::value('contribution_or_soft_value', $this->_params) ==
-          'soft_credits_only'
-        ) {
+        elseif (CRM_Utils_Array::value('contribution_or_soft_value', $this->_params) == 'soft_credits_only') {
           $title = '%1 soft-credits: %2';
         }
         foreach ($totals as $key => $total) {
index 2c4a34dd0cb7b0a15d2edb3b94f0aa87c11ab9ee..0b31540361b97871ab372cdc592cbb0d53acfe40 100644 (file)
@@ -106,9 +106,9 @@ class CRM_Upgrade_Incremental_php_FiveFiftyOne extends CRM_Upgrade_Incremental_B
     }
     $fieldMap[ts('Soft Credit')] = 'soft_credit';
     $fieldMap[ts('Pledge Payment')] = 'pledge_payment';
-    $fieldMap[ts(ts('Pledge ID'))] = 'pledge_id';
-    $fieldMap[ts(ts('Financial Type'))] = 'financial_type_id';
-    $fieldMap[ts(ts('Payment Method'))] = 'payment_instrument_id';
+    $fieldMap[ts('Pledge ID')] = 'pledge_id';
+    $fieldMap[ts('Financial Type')] = 'financial_type_id';
+    $fieldMap[ts('Payment Method')] = 'payment_instrument_id';
     $fieldMap[ts('- do not import -')] = 'do_not_import';
 
     // Membership fields
index 0d42a1771c6ddc624dc458668b8531e2ebeaa426..02d63d94d90567ac4aac600d28a993ee9327f975 100644 (file)
@@ -31,16 +31,11 @@ class CRM_Utils_Check_Component_AddressParsing extends CRM_Utils_Check_Component
     if ($addressOptions['street_address_parsing']) {
       if (!CRM_Core_BAO_Address::isSupportedParsingLocale()) {
         $config = CRM_Core_Config::singleton();
+        $url_address = CRM_Utils_System::url('civicrm/admin/setting/preferences/address', 'reset=1');
+        $url_localization = CRM_Utils_System::url('civicrm/admin/setting/localization', 'reset=1');
         $messages[] = new CRM_Utils_Check_Message(
           __FUNCTION__,
-          ts(
-            '<a href=' .
-            CRM_Utils_System::url('civicrm/admin/setting/preferences/address', 'reset=1') .
-            '">Street address parsing</a> is enabled but not supported by <a href="' .
-            CRM_Utils_System::url('civicrm/admin/setting/localization', 'reset=1') .
-            '">your locale</a> (%1).',
-            [1 => $config->lcMessages]
-          ),
+          ts('<a %1>Street address parsing</a> is enabled but not supported by <a %2>your language settings</a> (%3).', [1 => "href='$url_address'", 2 => "href='$url_localization'", 3 => $config->lcMessages]),
           ts('Street address parsing'),
           \Psr\Log\LogLevel::WARNING,
           'fa-address-card'
index d6b4c1148824d86f66d1eaae0a8bc1e8d49b6ba0..bd9f757c35f75330871cac5e248df475791486fa 100644 (file)
@@ -361,17 +361,15 @@ class CRM_Utils_Check_Component_Case extends CRM_Utils_Check_Component {
       }
       $messages[] = new CRM_Utils_Check_Message(
         __FUNCTION__ . "missingroles",
-        ts("<p>The following roles listed in your case type definitions do not match any relationship type defined in the system: <em>%1</em>.</p>"
-          . "<p>This might be because of a mismatch if you are using external xml files to manage case types. If using xml files, then use either the name_a_b or name_b_a value from the following table. (Out of the box you would use name_b_a, which lists them on the case from the client perspective.) If you are not using xml files, you can edit your case types at Administer - CiviCase - Case Types.</p>"
-          . "<table>
-            <tr><th>ID</th><th>name_a_b</th><th>name_b_a</th><th>label_a_b</th><th>label_b_a</th></tr>"
+        '<p>' . ts("The following roles listed in your case type definitions do not match any relationship type defined in the system: <em>%1</em>.", [1 => htmlspecialchars(implode(', ', $missing))]) . '</p>'
+          . "<p>" . ts("This might be because of a mismatch if you are using external xml files to manage case types. If using xml files, then use either the name_a_b or name_b_a value from the following table. (Out of the box you would use name_b_a, which lists them on the case from the client perspective.) If you are not using xml files, you can edit your case types at Administer - CiviCase - Case Types.") . '</p>'
+          . '<table>'
+          . '<tr><th>ID</th><th>name_a_b</th><th>name_b_a</th><th>label_a_b</th><th>label_b_a</th></tr>'
           . implode("\n", $tableRows)
-          . "</table>", [
-            1 => htmlspecialchars(implode(', ', $missing)),
-          ]) .
-          '<br /><a href="' . CRM_Utils_System::docURL2('user/case-management/what-you-need-to-know#missing-roles', TRUE) . '">' .
-          ts('Read more about this warning') .
-          '</a>',
+          . '</table>'
+          . '<br /><a href="' . CRM_Utils_System::docURL2('user/case-management/what-you-need-to-know#missing-roles', TRUE) . '">'
+          . ts('Read more about this warning')
+          . '</a>',
         ts('Missing Roles'),
         \Psr\Log\LogLevel::ERROR,
         'fa-exclamation'
index 03816c98fe7c0ea37fb752b366a6d94a3f5f28c5..b73cc4839365e5ae4d2ff163b529420907b1dfe0 100644 (file)
@@ -39,17 +39,17 @@ class CRM_Utils_Check_Component_PriceFields extends CRM_Utils_Check_Component {
         'action' => 'browse',
         'sid' => $dao->ps_id,
       ]);
-      $html .= "<tr><td>$dao->ps_title</td><td>$dao->psf_label</td><td><a href='$url'>View Price Set Fields</a></td></tr>";
+      $html .= "<tr><td>$dao->ps_title</td><td>$dao->psf_label</td><td><a href='$url'>" . ts('View Price Set Fields') . '</a></td></tr>';
     }
     if ($count > 0) {
-      $msg = "<p>the following Price Set Fields use disabled or invalid financial types and need to be fixed if they are to still be used.<p>
-          <p><table><thead><tr><th>Price Set</th><th>Price Set Field</th><th>Action Link</th>
-          </tr></thead><tbody>
-          $html
-          </tbody></table></p>";
+      $msg = '<p>' . ts('The following Price Set Fields use disabled or invalid financial types and need to be fixed if they are to still be used.') . '<p>'
+        . '<p><table><thead><tr><th>' . ts('Price Set') . '</th><th>' . ts('Price Set Field') . '</th><th>' . ts('Action') . '</th>'
+        . '</tr></thead><tbody>'
+        . $html
+        . '</tbody></table></p>';
       $messages[] = new CRM_Utils_Check_Message(
         __FUNCTION__,
-       ts($msg),
+       $msg,
        ts('Invalid Price Fields'),
        \Psr\Log\LogLevel::WARNING,
        'fa-lock'
index da26ccc5e2a1d081352b543bdd646e150fdb2bd9..ac67ebb449a52cb21b515ddd85c91e0ba1abc97e 100644 (file)
@@ -38,14 +38,14 @@ class CRM_Utils_Check_Component_Schema extends CRM_Utils_Check_Component {
           $html .= "<tr><td>{$tableName}</td><td>{$index['name']}</td><td>$fields</td>";
         }
       }
-      $message = "<p>The following tables have missing indices. Click 'Update Indices' button to create them.<p>
-        <p><table><thead><tr><th>Table Name</th><th>Key Name</th><th>Expected Indices</th>
-        </tr></thead><tbody>
-        $html
-        </tbody></table></p>";
+      $message = '<p>' . ts("The following tables have missing indices. Click 'Update Indices' button to create them.") . '<p>'
+        . '<p><table><thead><tr><th>' . ts('Table Name') . '</th><th>' . ts('Key Name') . '</th><th>' . ts('Expected Indices') . '</th>'
+        . '</tr></thead><tbody>'
+        $html
+        . '</tbody></table></p>';
       $msg = new CRM_Utils_Check_Message(
         __FUNCTION__,
-        ts($message),
+        $message,
         ts('Performance warning: Missing indices'),
         \Psr\Log\LogLevel::WARNING,
         'fa-server'
index f4852db380fc446f406309b4b263121bd9b29f9c..90510c8e621324a0045adbd40931f3e49320aa91 100644 (file)
@@ -75,12 +75,12 @@ class CRM_Utils_Check_Component_Security extends CRM_Utils_Check_Component {
           $log_url = implode($filePathMarker, $url);
           if ($this->fileExists($log_url)) {
             $docs_url = $this->createDocUrl('the-log-file-should-not-be-accessible');
-            $msg = 'The <a href="%1">CiviCRM debug log</a> should not be downloadable.'
-              . '<br />' .
-              '<a href="%2">Read more about this warning</a>';
+            $msg = ts('The <a %1>CiviCRM debug log</a> should not be downloadable.', [1 => "href='$log_url'"])
+              . '<br />'
+              . '<a href="' . $docs_url . '">' . ts('Read more about this warning') . '</a>';
             $messages[] = new CRM_Utils_Check_Message(
               __FUNCTION__,
-              ts($msg, [1 => $log_url, 2 => $docs_url]),
+              $msg,
               ts('Security Warning'),
               \Psr\Log\LogLevel::WARNING,
               'fa-lock'
index 87087711d0828139e9174b7f23019503c432ecfb..510718060bc59599b1b1a2e8b11fbb46b21266ce 100644 (file)
@@ -391,7 +391,7 @@ class CRM_Utils_Type {
       'Color',
     ];
     if (!in_array($type, $possibleTypes)) {
-      throw new CRM_Core_Exception(ts('Invalid type, must be one of : ' . implode($possibleTypes)));
+      throw new CRM_Core_Exception('Invalid type, must be one of : ' . implode($possibleTypes));
     }
     switch ($type) {
       case 'Integer':