diff --git l10n_client.js l10n_client.js index d4f99d4..b5bf40c 100644 --- l10n_client.js +++ l10n_client.js @@ -107,7 +107,7 @@ Drupal.behaviors.l10nClient.attach = function (context) { $('#l10n-client-string-editor .source-text').text(Drupal.l10nClient.getString(index, 'source')); $('#l10n-client-form #edit-target').val(Drupal.l10nClient.getString(index, 'target')); - + $('#l10n-client-form #edit-textgroup').val(Drupal.l10nClient.getString(index, 'textgroup')); Drupal.l10nClient.selected = index; }); @@ -155,6 +155,7 @@ Drupal.behaviors.l10nClient.attach = function (context) { data: { source: $('#l10n-client-string-editor .source-text').text(), target: $('#l10n-client-form #edit-target').val(), + textgroup: $('#l10n-client-form #edit-textgroup').val(), 'form_token': $('#l10n-client-form #edit-l10n-client-form-form-token').val() }, success: function (data) { diff --git l10n_client.module l10n_client.module index 4cdf09c..9cd47ad 100644 --- l10n_client.module +++ l10n_client.module @@ -168,13 +168,13 @@ function l10n_client_translate_page($display_translated = FALSE, $textgroup = 'd if ($display_translated) { $table[] = array(check_plain($data->source), check_plain($data->translation)); if ($allow_translation) { - l10_client_add_string_to_page($data->source, $data->translation); + l10_client_add_string_to_page($data->source, $data->translation, $textgroup); } } else { $table[] = array(check_plain($data->source)); if ($allow_translation) { - l10_client_add_string_to_page($data->source, TRUE); + l10_client_add_string_to_page($data->source, TRUE, $textgroup); } } } @@ -200,16 +200,20 @@ function l10n_client_page_alter(&$page) { if (user_access('use on-page translation') && ($page_strings = _l10n_client_page_strings())) { // If we have strings for the page language, restructure the data. $l10n_strings = array(); - foreach ($page_strings as $string => $translation) { - $l10n_strings[] = array($string, $translation); + foreach ($page_strings as $textgroup => $group_strings) { + foreach ($group_strings as $string => $translation) { + $l10n_strings[] = array($string, $translation, $textgroup); + } } array_multisort($l10n_strings); // Include string selector on page. $string_list = _l10n_client_string_list($l10n_strings); // Include editing form on page. - $l10n_form = drupal_render(drupal_get_form('l10n_client_form', $l10n_strings)); + $form = drupal_get_form('l10n_client_form', $l10n_strings); + $l10n_form = drupal_render($form); // Include search form on page. - $l10n_search = drupal_render(drupal_get_form('l10n_client_search_form')); + $form = drupal_get_form('l10n_client_search_form'); + $l10n_search = drupal_render($form); // Generate HTML wrapper with strings data. $l10n_dom = _l10n_client_dom_strings($l10n_strings); @@ -257,12 +261,14 @@ function l10n_client_page_alter(&$page) { * Source string or NULL if geting the list of strings specified. * @param $translation * Translation string. TRUE if untranslated. + * @param $textgroup + * Text group the string belongs to */ -function l10_client_add_string_to_page($source = NULL, $translation = NULL) { +function l10_client_add_string_to_page($source = NULL, $translation = NULL, $textgroup = 'default') { static $strings = array(); if (isset($source)) { - $strings[$source] = $translation; + $strings[$textgroup][$source] = $translation; } else { return $strings; @@ -278,24 +284,24 @@ function l10_client_add_string_to_page($source = NULL, $translation = NULL) { */ function _l10n_client_page_strings() { global $language; - - // Get the page strins stored by this or other modules. - $strings = l10_client_add_string_to_page(); + $strings = array(); // If this is not the module's translation page, merge all strings used on the page. if (arg(0) != 'locale' && is_array($locale = locale()) && isset($locale[$language->language])) { + // Get the page strings stored by this or other modules. + $strings += array('default' => array()); // @todo: add actual context support. - $strings = array_merge($strings, $locale[$language->language]['']); + $strings['default'] = array_merge($strings['default'], $locale[$language->language]['']); // Also select and add other strings for this path. Other users may have run // into these strings for the same page. This might be useful in some cases // but will not work reliably in all cases, since strings might have been // found on completely different paths first, or on a slightly different // path. - $result = db_query("SELECT s.source, t.translation FROM {locales_source} s LEFT JOIN {locales_target} t ON s.lid = t.lid AND t.language = :language WHERE s.location = :location", array(':language' => $language->language, ':location' => request_uri())); + $result = db_query("SELECT s.source, t.translation, s.textgroup FROM {locales_source} s LEFT JOIN {locales_target} t ON s.lid = t.lid AND t.language = :language WHERE s.location = :location", array(':language' => $language->language, ':location' => request_uri())); foreach ($result as $data) { - if (!array_key_exists($data->source, $strings)) { - $strings[$data->source] = (empty($data->translation) ? TRUE : $data->translation); + if (!array_key_exists($data->source, $strings[$data->textgroup])) { + $strings[$data->textgroup][$data->source] = (empty($data->translation) ? TRUE : $data->translation); } } } @@ -311,7 +317,9 @@ function _l10n_client_dom_strings($strings) { foreach ($strings as $values) { $source = $values[0] === TRUE ? '' : htmlspecialchars($values[0], ENT_NOQUOTES, 'UTF-8'); $target = $values[1] === TRUE ? '' : htmlspecialchars($values[1], ENT_NOQUOTES, 'UTF-8'); - $output .= "