Clean up translatable strings in templates

[Pasi Kallinen]

Several translatable strings in templates include html, newlines, or extraneous spaces, resulting in ugly and hard-to-translate strings in po-files.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/paxed/nohtmli18nstrings

That branch has several commits that clean up the strings.

[Pasi Kallinen (paxed)]

As an example, here's some very messy translatable strings in po-files, before the cleanup:

---

msgid ""
"\n"
" <td>Page </td><td class=\"current_page\">%1</td><td> of </td><td class=\"total_pages\">%2</td>\n"
" "

---

msgid ""
"\n"
" Unable to request <a href=\"'\n"
"mkurl(ctx.kpac_root _ '/record/' _ ctx.bre_id, { }, kill_params)\n"
"'\">%1</a>.<br/>\n"
" Please see a librarian.\n"
" "

---

msgid ""
"Choose the Lead\n"
" Selection List:"

[Ben Shum (bshum)]

Needs to be rebased to more recent master. Code's drifted a bit and I had enough conflicts while attempting a rebase that I wasn't fully sure what changed and why at each turn.

[Pasi Kallinen (paxed)]

I've updated the branch to master, and squashed the commits into one.

[Ben Shum (bshum)]

Setting back to confirmed since paxed did as asked in updating his code for this bug. Probably won't get to finish this review before bumping it to the next 2.5 milestone though.

[Remington Steed (rjs7)]

I fixed a few bugs in Pasi's branch and added them as a new commit. Otherwise, I tested his changes as best as I could by using the interfaces involved, and things seem to work fine. Good work Pasi! Rebased to master and signed off:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/rsteed/nohtmli18nstrings

working/user/rsteed/nohtmli18nstrings

[Dan Scott (denials)]

I gave Remington's branch a further quick test in TPAC and KPAC and didn't see any obvious remaining problems, then squashed the two commits together as there's no reason to maintain a history with bugs. Pushed to master. Many thanks, Pasi and Remington!