Bug 10249: Fix untranslatable strings in OPAC detail page
Star ratings feature:
- Your CGI session cookie is not current. Please refresh
the page and try again.
- average rating:
- your rating:
Browse results feature:
- Click to rewind the list to
- See biblio
- with biblionumber
- by (between title and author)
- Click to forward the list to
- Go to detail
To test:
- Update the po file using 'perl translate update <langcode>'
- Translate all new strings, maybe mark them with XX to make it easier
to find them
- Install the updated translation file using 'perl translate install
<langcode>'
- Test the ratings and browse results features in the OPAC detail page
- Verify everything is still working/displaying like it should
- Check that you don't find any unstranslated strings/terms
Some hints where the changes are:
- Change an existing rating - the average and your rating will change
- Browse result list and check mouse over tooltips for all links
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Strings are now translated, and no emergent JS errors have revealed
themselves.
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 787a480626)
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
This commit is contained in:
parent
240f535e80
commit
a6cf34d164
|
|
@ -170,17 +170,17 @@ $(function () {
|
|||
}, function (data) {
|
||||
|
||||
if (data.auth_status != 'ok') {
|
||||
window.alert('Your CGI session cookie is not current. Please refresh the page and try again.');
|
||||
window.alert(_("Your CGI session cookie is not current. Please refresh the page and try again."));
|
||||
} else {
|
||||
$("#rating_value").val(data.rating_value);
|
||||
|
||||
if (data.rating_value) {
|
||||
$("#rating_value_text").text('your rating: ' + data.rating_value + ', ');
|
||||
$("#rating_value_text").text(_("your rating:") + ' ' + data.rating_value + ', ');
|
||||
} else {
|
||||
$("#rating_value_text").text('');
|
||||
}
|
||||
|
||||
$("#rating_text").text('average rating: ' + data.rating_avg_int + ' (' + data.rating_total + ' votes)');
|
||||
$("#rating_text").text(_("average rating:") + " " + data.rating_avg_int + ' (' + data.rating_total + ' votes)');
|
||||
|
||||
}
|
||||
}, "json");
|
||||
|
|
@ -303,7 +303,7 @@ function renderPagIndexList(index, ul)
|
|||
var ini = index - 1;
|
||||
var end = ini + totalPagItemList - 1;
|
||||
li = $("<li />");
|
||||
html = (index > pag_index_ini)?"<a href='#' id='rew_list_index' onclick='rewindList()' title='Click to rewind the list to " + ini + " - " + end + "'>«</a>":"«";
|
||||
html = (index > pag_index_ini)?"<a href='#' id='rew_list_index' onclick='rewindList()' title='" + _("Click to rewind the list to") + " " + ini + " - " + end + "'>«</a>":"«";
|
||||
li.html(html);
|
||||
ul.append(li);
|
||||
var title = "";
|
||||
|
|
@ -311,9 +311,9 @@ function renderPagIndexList(index, ul)
|
|||
if (arrPagination[i] == undefined) continue;
|
||||
var li = $("<li />");
|
||||
if (arrPagination[i].url != "") {
|
||||
title = "See biblio "" + arrPagination[i].title + "" ";
|
||||
if (arrPagination[i].author != "") title += " by "" + arrPagination[i].author + """;
|
||||
title += " with biblionumber " + arrPagination[i].biblionumber;
|
||||
title = _("See biblio") + " "" + arrPagination[i].title + "" ";
|
||||
if (arrPagination[i].author != "") title += " " + _("by") + """ + arrPagination[i].author + """;
|
||||
title += " " + _("with biblionumber") + " " + arrPagination[i].biblionumber;
|
||||
html = "<a href='" + arrPagination[i].url + "' title='" + title + "' class='a_pag' id='a_pag_" + i + "'";
|
||||
html += " onmouseover='renderPagination(" + i + ", " + (arrPagination.length - 1) + ", $(\"#ul_pagination_list\"), true)'";
|
||||
html += ">" + i + "</a>";
|
||||
|
|
@ -325,7 +325,7 @@ function renderPagIndexList(index, ul)
|
|||
li = $("<li />");
|
||||
ini = index + 1;
|
||||
end = (arrPagination.length > index + totalPagItemList)?index + totalPagItemList:arrPagination.length - 1;
|
||||
html = (end <= arrPagination.length - 1 && (end - index) >= totalPagItemList)?"<a href='#' id='fw_list_index' onclick='forwardList()' title='Click to forward the list to " + ini + " - " + end + "'>»</a>":"»";
|
||||
html = (end <= arrPagination.length - 1 && (end - index) >= totalPagItemList)?"<a href='#' id='fw_list_index' onclick='forwardList()' title='" + _("Click to forward the list to") + " " + ini + " - " + end + "'>»</a>":"»";
|
||||
li.html(html);
|
||||
ul.append(li);
|
||||
}//renderPagIndexList
|
||||
|
|
@ -340,7 +340,7 @@ function renderPagination(index, total, ul, highlIndex)
|
|||
var j = 0;
|
||||
for (var i = index; i <= total && j < totalPagItemList; i++) {
|
||||
if (arrPagination[i] == undefined || arrPagination[i].url == "") continue;
|
||||
var li = $("<li id='li_pag_" + i + "' " + ((j % 2 == 0)?"class='highlight'":"") + " title='Go to detail' />");
|
||||
var li = $("<li id='li_pag_" + i + "' " + ((j % 2 == 0)?"class='highlight'":"") + " title='" + _("Go to detail") + "' />");
|
||||
var html = "<span class='li_pag_index'>" + i + "</span><a href='" + arrPagination[i].url + "'>" + arrPagination[i].title + "</a>";
|
||||
if (arrPagination[i].author) html += "<br /> by " + arrPagination[i].author;
|
||||
li.html(html);
|
||||
|
|
@ -351,9 +351,9 @@ function renderPagination(index, total, ul, highlIndex)
|
|||
for (i = pag_index_ini; i < index && j < totalPagItemList; i++) {
|
||||
if (arrPagination[i] == undefined || arrPagination[i].url == "") continue;
|
||||
$("#li_pag_" + i).remove();
|
||||
var li = $("<li id='li_pag_" + i + "' " + ((j % 2 == 0)?"class='highlight'":"") + " title='Go to detail' />");
|
||||
var li = $("<li id='li_pag_" + i + "' " + ((j % 2 == 0)?"class='highlight'":"") + " title='" + _("Go to detail") + "' />");
|
||||
var html = "<span class='li_pag_index'>" + i + "</span><a href='" + arrPagination[i].url + "'>" + arrPagination[i].title + "</a>";
|
||||
if (arrPagination[i].author) html += "<br /> by " + arrPagination[i].author;
|
||||
if (arrPagination[i].author) html += "<br /> " + _("by") + " " + arrPagination[i].author;
|
||||
li.html(html);
|
||||
ul.append(li);
|
||||
j++;
|
||||
|
|
|
|||
Loading…
Reference in New Issue