Fix plural strings
authorColeman Watts <coleman@civicrm.org>
Sun, 15 Mar 2015 02:12:31 +0000 (22:12 -0400)
committerColeman Watts <coleman@civicrm.org>
Wed, 18 Mar 2015 00:54:15 +0000 (20:54 -0400)
13 files changed:
CRM/Activity/Form/Task/AddToTag.php
CRM/Activity/Form/Task/Delete.php
CRM/Activity/Form/Task/RemoveFromTag.php
CRM/Campaign/Form/Task/Release.php
CRM/Campaign/Form/Task/Reserve.php
CRM/Campaign/Page/DashBoard.php
CRM/Case/Form/Task/Delete.php
CRM/Case/Form/Task/Restore.php
CRM/Contribute/Form/Task/Delete.php
CRM/Event/Form/Task/Delete.php
CRM/Grant/Form/Task/Delete.php
CRM/Member/Form/Task/Delete.php
CRM/Pledge/Form/Task/Delete.php

index 175a784aa18b32c626a7edd1d89c3fe0fba6c2a2..b2ebb54af314969541fb9e466fa78032197826df 100644 (file)
@@ -145,9 +145,9 @@ class CRM_Activity_Form_Task_AddToTag extends CRM_Activity_Form_Task {
 
       list($total, $added, $notAdded) = CRM_Core_BAO_EntityTag::addEntitiesToTag($this->_activityHolderIds, $key, 'civicrm_activity');
 
-      $status = array(ts('%count activities tagged', array('count' => $added, 'plural' => '%count activities tagged')));
+      $status = array(ts('Activity tagged', array('count' => $added, 'plural' => '%count activities tagged')));
       if ($notAdded) {
-        $status[] = ts('%count activities already had this tag', array(
+        $status[] = ts('1 activity already had this tag', array(
           'count' => $notAdded,
           'plural' => '%count activities already had this tag',
         ));
index 1a899e003b55346b601c40a4bc6f338d937a1292..aa9df7f625ad5da49a69fb0381f23a0e1ff553e0 100755 (executable)
@@ -74,16 +74,25 @@ class CRM_Activity_Form_Task_Delete extends CRM_Activity_Form_Task {
    * @return void
    */
   public function postProcess() {
-    $deletedActivities = 0;
+    $deleted = $failed = 0;
     foreach ($this->_activityHolderIds as $activityId['id']) {
       $moveToTrash = CRM_Case_BAO_Case::isCaseActivity($activityId['id']);
       if (CRM_Activity_BAO_Activity::deleteActivity($activityId, $moveToTrash)) {
-        $deletedActivities++;
+        $deleted++;
       }
+      else {
+        $failed++;
+      }
+    }
+
+    if ($deleted) {
+      $msg = ts('%count activity deleted.', array('plural' => '%count activities deleted.', 'count' => $deleted));
+      CRM_Core_Session::setStatus($msg, ts('Removed'), 'success');
     }
 
-    CRM_Core_Session::setStatus($deletedActivities, ts('Deleted Activities'), "success");
-    CRM_Core_Session::setStatus("", ts('Total Selected Activities: %1', array(1 => count($this->_activityHolderIds))), "info");
+    if ($failed) {
+      CRM_Core_Session::setStatus(ts('1 could not be deleted.', array('plural' => '%count could not be deleted.', 'count' => $failed)), ts('Error'), 'error');
+    }
   }
 
 }
index 26c0e26d3d422cb83695e3a8f9713a0150add779..37973570e6d89b2d5647883ab36fe11d47f33e29 100644 (file)
@@ -135,7 +135,7 @@ class CRM_Activity_Form_Task_RemoveFromTag extends CRM_Activity_Form_Task {
       list($total, $removed, $notRemoved) = CRM_Core_BAO_EntityTag::removeEntitiesFromTag($this->_activityHolderIds, $key, 'civicrm_activity');
 
       $status = array(
-        ts('%count activities un-tagged', array(
+        ts('%count activity un-tagged', array(
             'count' => $removed,
             'plural' => '%count activities un-tagged',
           )),
index 2fe884cb814b84a0bf49e13810ba74ed1effae15..d22dbd0374162702dabd5a7977088664cfe89255 100644 (file)
@@ -159,13 +159,23 @@ class CRM_Campaign_Form_Task_Release extends CRM_Campaign_Form_Task {
       $query = 'UPDATE civicrm_activity SET is_deleted = 1 WHERE id IN ( ' . implode(', ', $deleteActivityIds) . ' )';
       CRM_Core_DAO::executeQuery($query);
 
-      $status = array(ts("%1 respondent(s) have been released.", array(1 => count($deleteActivityIds))));
+      if ($deleteActivityIds) {
+        $status = ts("Respondent has been released.", array(
+          'count' => count($deleteActivityIds),
+          'plural' => '%count respondents have been released.',
+        ));
+        CRM_Core_Session::setStatus($status, ts('Released'), 'success');
+      }
+
       if (count($this->_contactIds) > count($deleteActivityIds)) {
-        $status[] = ts("%1 respondents did not release.",
-          array(1 => (count($this->_contactIds) - count($deleteActivityIds)))
+        $status = ts('1 respondent did not release.',
+          array(
+            'count' => (count($this->_contactIds) - count($deleteActivityIds)),
+            'plural' => '%count respondents did not release.',
+          )
         );
+        CRM_Core_Session::setStatus($status, ts('Notice'), 'alert');
       }
-      CRM_Core_Session::setStatus(implode('&nbsp;', $status), '', 'info');
     }
   }
 
index 852a3466d272f5ba981aafcbd7dec311aee483c5..3fd757bfda47a59dada782e01d39294b709fa71e 100644 (file)
@@ -132,16 +132,22 @@ class CRM_Campaign_Form_Task_Reserve extends CRM_Campaign_Form_Task {
         $errorMsg = ts('The maximum number of contacts is already reserved for this interviewer.');
       }
       elseif (count($this->_contactIds) > ($maxVoters - $this->_numVoters)) {
-        $errorMsg = ts('You can reserve a maximum of %1 contact(s) at a time for this survey.',
-          array(1 => $maxVoters - $this->_numVoters)
+        $errorMsg = ts('You can reserve a maximum of %count contact at a time for this survey.',
+          array(
+            'plural' => 'You can reserve a maximum of %count contacts at a time for this survey.',
+            'count' => $maxVoters - $this->_numVoters,
+          )
         );
       }
     }
 
     $defaultNum = CRM_Utils_Array::value('default_number_of_contacts', $this->_surveyDetails);
     if (!$errorMsg && $defaultNum && (count($this->_contactIds) > $defaultNum)) {
-      $errorMsg = ts('You can reserve a maximum of %1 contact(s) at a time for this survey.',
-        array(1 => $defaultNum)
+      $errorMsg = ts('You can reserve a maximum of %count contact at a time for this survey.',
+        array(
+          'plural' => 'You can reserve a maximum of %count contacts at a time for this survey.',
+          'count' => $defaultNum,
+        )
       );
     }
 
@@ -276,18 +282,21 @@ class CRM_Campaign_Form_Task_Reserve extends CRM_Campaign_Form_Task {
 
     // Success message
     if ($countVoters > 0) {
-      $status = '<p>' . ts("%1 Contact(s) have been reserved.", array(1 => $countVoters)) . '</p>';
+      $status = '<p>' . ts("%count contact has been reserved.", array('plural' => '%count contacts have been reserved.', 'count' => $countVoters)) . '</p>';
       if ($groupAdditions) {
-        $status .= '<p>' . ts('Respondent(s) has been added to %1 group(s).',
-            array(1 => implode(', ', $groupAdditions))
+        $status .= '<p>' . ts('They have been added to %1.',
+            array(1 => implode(' ' . ts('and') . ' ', $groupAdditions))
           ) . '</p>';
       }
       CRM_Core_Session::setStatus($status, ts('Reservation Added'), 'success');
     }
     // Error message
     if (count($this->_contactIds) > $countVoters) {
-      CRM_Core_Session::setStatus(ts('Reservation did not add for %1 contact(s).',
-        array(1 => (count($this->_contactIds) - $countVoters))
+      CRM_Core_Session::setStatus(ts('Reservation did not add for %count contact.',
+        array(
+          'plural' => 'Reservation did not add for %count contacts.',
+          'count' => (count($this->_contactIds) - $countVoters),
+        )
       ), ts('Notice'));
     }
 
index f51bdfc4e0667fcdfc3fbb06fb931bf51d31345e..fe64ba0328a90eeb8cd5c9879fbf9dc9ca32cff0 100644 (file)
@@ -314,7 +314,7 @@ class CRM_Campaign_Page_DashBoard extends CRM_Core_Page {
         $surveysData[$sid]['campaign'] = CRM_Utils_Array::value($campaignId, $campaigns);
         $surveysData[$sid]['activity_type'] = $surveyType[$survey['activity_type_id']];
         if (!empty($survey['release_frequency'])) {
-          $surveysData[$sid]['release_frequency'] = $survey['release_frequency'] . ' ' . ts('Day(s)');
+          $surveysData[$sid]['release_frequency'] = ts('1 Day', array('plural' => '%count Days', 'count' => $survey['release_frequency']));
         }
 
         $action = array_sum(array_keys(self::surveyActionLinks($surveysData[$sid]['activity_type'])));
index 0579065f9007b74175380ff4ca85f2eee0188998..f3d0f8cb3c4c034a0995618b90d296c0c0a7d648 100644 (file)
@@ -80,15 +80,29 @@ class CRM_Case_Form_Task_Delete extends CRM_Case_Form_Task {
    * @return void
    */
   public function postProcess() {
-    $deletedCases = 0;
+    $deleted = $failed = 0;
     foreach ($this->_caseIds as $caseId) {
       if (CRM_Case_BAO_Case::deleteCase($caseId, $this->_moveToTrash)) {
-        $deletedCases++;
+        $deleted++;
       }
+      else {
+        $failed++;
+      }
+    }
+
+    if ($deleted) {
+      if ($this->_moveToTrash) {
+        $msg = ts('%count case moved to trash.', array('plural' => '%count cases moved to trash.', 'count' => $deleted));
+      }
+      else {
+        $msg = ts('%count case permanently deleted.', array('plural' => '%count cases permanently deleted.', 'count' => $deleted));
+      }
+      CRM_Core_Session::setStatus($msg, ts('Removed'), 'success');
     }
 
-    CRM_Core_Session::setStatus($deletedCases, ts('Deleted Cases'), 'success');
-    CRM_Core_Session::setStatus('', ts('Total Selected Case(s): %1', array(1 => count($this->_caseIds))), 'info');
+    if ($failed) {
+      CRM_Core_Session::setStatus(ts('1 could not be deleted.', array('plural' => '%count could not be deleted.', 'count' => $failed)), ts('Error'), 'error');
+    }
   }
 
 }
index dfbfa985b13f4ce8b4179047c61913ae863e4025..8f773ceee97c654980f1ee689315f59ce0458ec1 100644 (file)
@@ -74,15 +74,27 @@ class CRM_Case_Form_Task_Restore extends CRM_Case_Form_Task {
    * @return void
    */
   public function postProcess() {
-    $restoredCases = 0;
+    $restoredCases = $failed = 0;
     foreach ($this->_caseIds as $caseId) {
       if (CRM_Case_BAO_Case::restoreCase($caseId)) {
         $restoredCases++;
       }
+      else {
+        $failed++;
+      }
+    }
+
+    if ($restoredCases) {
+      $msg = ts('%count case restored from trash.', array(
+        'plural' => '%count cases restored from trash.',
+        'count' => $restoredCases,
+      ));
+      CRM_Core_Session::setStatus($msg, ts('Restored'), 'success');
     }
 
-    CRM_Core_Session::setStatus($restoredCases, ts('Restored Cases'), 'success');
-    CRM_Core_Session::setStatus('', ts('Total Selected Case(s): %1', array(1 => count($this->_caseIds))), 'info');
+    if ($failed) {
+      CRM_Core_Session::setStatus(ts('1 could not be restored.', array('plural' => '%count could not be restored.', 'count' => $failed)), ts('Error'), 'error');
+    }
   }
 
 }
index 5b19e4cdbc362189a791bca2c179698e3287093d..06fc3eff07d661bf6eddc10989003790fed1be0f 100644 (file)
@@ -78,17 +78,24 @@ class CRM_Contribute_Form_Task_Delete extends CRM_Contribute_Form_Task {
    * @return void
    */
   public function postProcess() {
-    $deletedContributions = 0;
+    $deleted = $failed = 0;
     foreach ($this->_contributionIds as $contributionId) {
       if (CRM_Contribute_BAO_Contribution::deleteContribution($contributionId)) {
-        $deletedContributions++;
+        $deleted++;
       }
+      else {
+        $failed++;
+      }
+    }
+
+    if ($deleted) {
+      $msg = ts('%count contribution deleted.', array('plural' => '%count contributions deleted.', 'count' => $deleted));
+      CRM_Core_Session::setStatus($msg, ts('Removed'), 'success');
+    }
+
+    if ($failed) {
+      CRM_Core_Session::setStatus(ts('1 could not be deleted.', array('plural' => '%count could not be deleted.', 'count' => $failed)), ts('Error'), 'error');
     }
-    $status = ts('Deleted Contribution(s): %1 (Total Selected: %2) ', array(
-        1 => $deletedContributions,
-        2 => count($this->_contributionIds),
-      ));
-    CRM_Core_Session::setStatus($status, '', 'info');
   }
 
 }
index a2c11055d4e7e127263c65daa6ffb9d14f95e552..7dfc462beb749704ca0e03e538362fd36652c1b1 100644 (file)
@@ -123,18 +123,13 @@ class CRM_Event_Form_Task_Delete extends CRM_Event_Form_Task {
       $deletedParticipants += $additionalCount;
     }
 
-    $status = array(
-      ts('Participant(s) Deleted: %1 (Total Selected: %2)', array(
-          1 => $deletedParticipants,
-          2 => count($this->_participantIds),
-        )),
-    );
-    if (!empty($participantLinks)) {
-      $status[] = ts('The following participants no longer have an event fee recorded. You can edit their registration and record a replacement contribution by clicking the links below:') . '<br/>' . $participantLinks;
-    }
-    foreach ($status as $text) {
-      CRM_Core_Session::setStatus($text, '', 'info');
+    $status = ts('%count participant deleted.', array('plural' => '%count participants deleted.', 'count' => $deletedParticipants));
+
+    if ($participantLinks) {
+      $status .= '<p>' . ts('The following participants no longer have an event fee recorded. You can edit their registration and record a replacement contribution by clicking the links below:')
+        . '</p>' . $participantLinks;
     }
+    CRM_Core_Session::setStatus($status, ts('Removed'), 'info');
   }
 
 }
index 18703ab2cc6e67d11880a8833cb73154ae5a42cd..d3b4680c18eccea36b3b4a957b4f9d290e19ea66 100644 (file)
@@ -79,15 +79,24 @@ class CRM_Grant_Form_Task_Delete extends CRM_Grant_Form_Task {
    * @return void
    */
   public function postProcess() {
-    $deletedGrants = 0;
+    $deleted = $failed = 0;
     foreach ($this->_grantIds as $grantId) {
       if (CRM_Grant_BAO_Grant::del($grantId)) {
-        $deletedGrants++;
+        $deleted++;
       }
+      else {
+        $failed++;
+      }
+    }
+
+    if ($deleted) {
+      $msg = ts('%count grant deleted.', array('plural' => '%count grants deleted.', 'count' => $deleted));
+      CRM_Core_Session::setStatus($msg, ts('Removed'), 'success');
     }
 
-    CRM_Core_Session::setStatus(ts('Deleted Grant(s): %1', array(1 => $deletedGrants)), '', 'info');
-    CRM_Core_Session::setStatus(ts('Total Selected Grant(s): %1', array(1 => count($this->_grantIds))), '', 'info');
+    if ($failed) {
+      CRM_Core_Session::setStatus(ts('1 could not be deleted.', array('plural' => '%count could not be deleted.', 'count' => $failed)), ts('Error'), 'error');
+    }
   }
 
 }
index a661a50ba579822cd3f51ab6669d067292efe63a..cb6ab91efde911ed3ad231cdf6bb8e9116b93113 100644 (file)
@@ -78,15 +78,24 @@ class CRM_Member_Form_Task_Delete extends CRM_Member_Form_Task {
    * @return void
    */
   public function postProcess() {
-    $deletedMembers = 0;
+    $deleted = $failed = 0;
     foreach ($this->_memberIds as $memberId) {
       if (CRM_Member_BAO_Membership::del($memberId)) {
-        $deletedMembers++;
+        $deleted++;
       }
+      else {
+        $failed++;
+      }
+    }
+
+    if ($deleted) {
+      $msg = ts('%count membership deleted.', array('plural' => '%count memberships deleted.', 'count' => $deleted));
+      CRM_Core_Session::setStatus($msg, ts('Removed'), 'success');
     }
 
-    CRM_Core_Session::setStatus($deletedMembers, ts('Deleted Membership(s)'), 'success');
-    CRM_Core_Session::setStatus(ts('Total Selected Membership(s): %1', array(1 => count($this->_memberIds))), '', 'info');
+    if ($failed) {
+      CRM_Core_Session::setStatus(ts('1 could not be deleted.', array('plural' => '%count could not be deleted.', 'count' => $failed)), ts('Error'), 'error');
+    }
   }
 
 }
index 2bca8215a5ab51e0e02f1a0f1a95f4950aadbfc0..f67636aaf4253afda1eda20872f1e3400634d856 100644 (file)
@@ -76,18 +76,24 @@ class CRM_Pledge_Form_Task_Delete extends CRM_Pledge_Form_Task {
    * @return void
    */
   public function postProcess() {
-    $deletedPledges = 0;
+    $deleted = $failed = 0;
     foreach ($this->_pledgeIds as $pledgeId) {
       if (CRM_Pledge_BAO_Pledge::deletePledge($pledgeId)) {
-        $deletedPledges++;
+        $deleted++;
       }
+      else {
+        $failed++;
+      }
+    }
+
+    if ($deleted) {
+      $msg = ts('%count pledge deleted.', array('plural' => '%count pledges deleted.', 'count' => $deleted));
+      CRM_Core_Session::setStatus($msg, ts('Removed'), 'success');
     }
 
-    $status = ts('Deleted Pledge(s): %1 (Total Selected: %2) ', array(
-        1 => $deletedPledges,
-        2 => count($this->_pledgeIds),
-      ));
-    CRM_Core_Session::setStatus($status, '', 'info');
+    if ($failed) {
+      CRM_Core_Session::setStatus(ts('1 could not be deleted.', array('plural' => '%count could not be deleted.', 'count' => $failed)), ts('Error'), 'error');
+    }
   }
 
 }