From 9d72834de3992699bc83c6aae402f1c56538595f Sat, 17 Mar 2012 16:18:30 +0100
From: hass <hass@85918.no-reply.drupal.org>
Date: Sat, 17 Mar 2012 16:17:55 +0100
Subject: [PATCH] 1487720: Translatable strings review

diff --git a/includes/workbench_access_handler_field_edit_node.inc b/includes/workbench_access_handler_field_edit_node.inc
index 47790f9..669cea8 100644
--- a/includes/workbench_access_handler_field_edit_node.inc
+++ b/includes/workbench_access_handler_field_edit_node.inc
@@ -22,7 +22,7 @@
     if (!node_access('update', $node)) {
       return '';
     }
-    $text = !empty($this->options['text']) ? $this->options['text'] : t('edit');
+    $text = !empty($this->options['text']) ? $this->options['text'] : t('Edit');
     return l($text, "node/$values->nid/edit", array('query' => drupal_get_destination()));
   }
 }
diff --git a/workbench_access.admin.inc b/workbench_access.admin.inc
index 0a2c675..51f95af 100644
--- a/workbench_access.admin.inc
+++ b/workbench_access.admin.inc
@@ -57,7 +57,7 @@
   if (!empty($types)) {
     $form['node_types'] = array(
       '#type' => 'fieldset',
-      '#title' => t('Content types enabled'),
+      '#title' => t('Enabled content types'),
       '#collapsible' => TRUE,
       '#collapsed' => TRUE,
     );
@@ -92,7 +92,7 @@
         if (empty($controlled)) {
           $form['field_elements'][$type] = array(
             '#type' => 'item',
-            '#title' => t('%type access control fields', array('%type' => $name)),
+            '#title' => t('%type content type access control fields', array('%type' => $name)),
             '#markup' => t('This content type is not under access control.'),
           );
         }
@@ -101,7 +101,7 @@
           drupal_set_message($message, 'warning', FALSE);
           $form['field_elements'][$type] = array(
             '#type' => 'item',
-            '#title' => t('%type access control fields', array('%type' => $name)),
+            '#title' => t('%type content type access control fields', array('%type' => $name)),
             '#markup' => $message,
           );
         }
@@ -116,7 +116,7 @@
           }
           $form['field_elements'][$type] = array(
             '#type' => 'select',
-            '#title' => t('%type access control fields', array('%type' => $name)),
+            '#title' => t('%type content type access control fields', array('%type' => $name)),
             '#options' => $field_options,
             '#default_value' => $default_options,
             '#multiple' => variable_get('workbench_access_allow_multiple', 0),
@@ -321,7 +321,7 @@
     drupal_set_message(workbench_access_sections_needed_message(), 'warning');
   }
   else {
-    drupal_set_message(t('Editorial sections saved.'));
+    drupal_set_message(t('Editorial sections have been saved.'));
   }
 }
 
@@ -450,7 +450,7 @@
   $uid = $form_state['values']['uid'];
   workbench_access_rebuild_user($uid, $sections);
   if (!empty($sections)) {
-    drupal_set_message(t('User permissions updated.'));
+    drupal_set_message(t('User permissions have been updated.'));
   }
 }
 
@@ -685,7 +685,7 @@
   $variables = array(
     'header' => $header,
     'rows' => $rows,
-    'empty' => t('No active editors.'),
+    'empty' => t('No active editors have been found.'),
   );
   $output .= theme('table', $variables);
   $output .= drupal_render_children($form);
@@ -784,7 +784,7 @@
       theme('item_list', array('items' => $item['roles'], 'type' => 'ul')),
     );
   }
-  $table = theme('table', array('header' => $header, 'rows' => $rows, 'empty' => t('No active roles.')));
+  $table = theme('table', array('header' => $header, 'rows' => $rows, 'empty' => t('No active roles have been found.')));
   $form['content'] = array(
     '#weight' => -5,
     '#markup' => $output,
@@ -800,7 +800,7 @@
     '#title' => t('Roles'),
     '#options' => $roles,
     '#default_value' => $default,
-    '#description' => empty($roles) ? t('There are no roles with the proper permissions.') : t('Select the role(s) that should have all users assigned to this section.'),
+    '#description' => empty($roles) ? t('There are no roles with the proper permissions.') : format_plural(count($roles), t('Select the role that should have all users assigned to this section.'), t('Select the roles that should have all users assigned to this section.')),
   );
   // TODO: replace with fancy jQuery.
   if (isset($roles[DRUPAL_AUTHENTICATED_RID])) {
@@ -860,7 +860,7 @@
   $exists = workbench_access_vocabulary_exists($vocabulary);
   // If the vocabulary exists, do not create any terms for it.
   if ($exists) {
-    drupal_set_message(t('Workbench Access vocabulary already installed.'));
+    drupal_set_message(t('Workbench Access vocabulary has already been installed.'));
     return;
   }
   // Save.
@@ -1003,7 +1003,7 @@
     '#type' => 'value',
     '#value' => $path,
   );
-  $question = t('Install test configuration');
+  $question = t('Are you sure you want to install the test configuration?');
   return confirm_form($form, $question, $path);
 }
 
diff --git a/workbench_access.api.php b/workbench_access.api.php
index 4593641..3c249ce 100644
--- a/workbench_access.api.php
+++ b/workbench_access.api.php
@@ -61,7 +61,7 @@
  *  - 'translatable'
  *    Boolean value that indicated the value is translatable via FieldAPI.
  *    Tells the module how to save language-sensitive data. Required.
- *    
+ *
  *
  * The remainder of the elements are used with Views to provide proper query
  * execution. They provide run-time alterations to Views handlers provided by
@@ -330,7 +330,7 @@
   $form['taxonomy_workbench_access_info']['workbench_access_taxonomy'] = array(
     '#type' => 'checkboxes',
     '#title' => t('Editorial vocabulary'),
-    '#description' => t('Select the vocabulary to be used for access control. <strong>Warning: changing this value in production may disrupt your workflow.</strong>'),
+    '#description' => t('Select the vocabulary to be used for access control. <strong>Warning: Changing this value in production may disrupt your workflow.</strong>'),
     '#options' => $options,
     '#default_value' => variable_get('workbench_access_taxonomy', array()),
     '#states' => array(
diff --git a/workbench_access.install b/workbench_access.install
index 5db0d44..0b1566b 100644
--- a/workbench_access.install
+++ b/workbench_access.install
@@ -166,7 +166,7 @@
     $description = array();
     $modules = module_implements('workbench_access_info');
     if (empty($modules) && !module_exists('taxonomy') && !module_exists('menu')) {
-      $description[] = t('No modules are enabled that support access heirarchies. Menu and Taxonomy are two options.');
+      $description[] = t('No modules are enabled that support access hierarchies. Menu and Taxonomy are two options.');
     }
     // If this variable has not been set, workbench access is not fully configured.
     if (!variable_get('workbench_access', FALSE)) {
@@ -236,7 +236,7 @@
       variable_set('workbench_access_taxonomy', array($taxonomy));
     }
   }
-  return t('Set access rules correctly.');
+  return t('Access rules have been corrected.');
 }
 
 /**
@@ -246,7 +246,7 @@
  */
 function workbench_access_update_7001() {
   if (db_table_exists('workbench_access_role')) {
-    return t('Update not required.');
+    return t('Update is not required.');
   }
   $schema = array(
     'description' => 'Maps role access to sections',
diff --git a/workbench_access.module b/workbench_access.module
index 3964d36..8bf39a1 100644
--- a/workbench_access.module
+++ b/workbench_access.module
@@ -35,7 +35,7 @@
   );
   $items['admin/config/workbench/access/editors'] = array(
     'title' => 'Editors',
-    'description' => 'Editor settings.',
+    'description' => 'The editor assignment settings.',
     'page callback' => 'workbench_access_editors',
     'access arguments' => array('assign workbench access'),
     'type' => MENU_DEFAULT_LOCAL_TASK,
@@ -1870,7 +1870,7 @@
     $node = (object) $data;
     workbench_access_node_update($node);
   }
-  drupal_set_message(t('Editorial sections updated'));
+  drupal_set_message(t('Editorial sections have been updated.'));
 }
 
 /**
diff --git a/workbench_access.views.inc b/workbench_access.views.inc
index 8604fdd..d92bd10 100644
--- a/workbench_access.views.inc
+++ b/workbench_access.views.inc
@@ -287,7 +287,7 @@
   );
   $data['menu_links']['plid'] = array(
     'title' => t('Parent menu link id'),
-    'help' => t('The unique menu link identifier for this item\'s parent.'),
+    'help' => t("The unique menu link identifier for this item's parent."),
     'field' => array(
       'click sortable' => FALSE,
     ),
