Translated function name in es-ES causes Internal Server Error on every search

Bug #1652098 reported by Jason Boyer
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
2.11
Fix Released
Medium
Unassigned

Bug Description

Eg 2.11

In build/i18n/po/tpac/es-ES.po a call to the quant() function has been translated to cuant(), which crashes EGWeb.pm when trying to view search results using that locale. Super simple branch incoming.

Revision history for this message
Jason Boyer (jboyer) wrote :
Jason Boyer (jboyer)
tags: added: i18n
Jason Boyer (jboyer)
tags: added: pullrequest
Revision history for this message
Adam Bowling (abowling) wrote :

Additionally modified the '%' signs that were misplaced in the translation.

Revision history for this message
Adam Bowling (abowling) wrote :
Revision history for this message
Adam Bowling (abowling) wrote :

Another...restored one other 'cuant()' back to 'quant()'.

Revision history for this message
Adam Bowling (abowling) wrote :
Jason Boyer (jboyer)
Changed in evergreen:
milestone: none → 2.12-beta
Revision history for this message
Ben Shum (bshum) wrote :

The latest PO sync pulled in the majority of the changes specified here from LP translations to the tpac es-ES.po file without the need for the branches listed here. Thanks to Jason and Adam though for making the necessary fixes to ensure working Spanish translation.

For backport support however, to rel_2_11 where we first included Spanish, I have created the following working branch just for rel_2_11 and updating the tpac file there for es-ES.po: user/bshum/spanish-tpac-rel_2_11

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/bshum/spanish-tpac-rel_2_11

We can consider merging just that file change for TPAC to address the bugs noted here, and then carry on with the future work in this area for master/2.12.

Changed in evergreen:
status: New → Fix Committed
importance: Undecided → Medium
Revision history for this message
Galen Charlton (gmc) wrote :

I've merged Ben's branch to rel_2_11. Thanks!

Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.