Created attachment 17423 [details] screenshot To reproduce: 1) Make sure comments/reviews are activated 2) Find a record with 0 comments/reviews in the opac 3) The tab shows: Comments ( 0 ) 4) Switch to any other languages. 5) Tab now shows: <translation of Comments> ( )
Created attachment 17424 [details] [review] Bug 10049: Comments tab shows ( ) instad of ( 0 ) for translated templates To test: 1) Make sure comments/reviews are activated 2) Find a record with 0 comments/reviews in the opac 3) The tab shows: Comments ( 0 ) 4) Switch to any other languages. 5) Tab now shows: <translation of Comments> ( ) 6) Apply patch 7) Check English templates stil work correctly 8) Rerun 'perl translate install <langcode> 9) Check display is now correct. 10) Also check that number shows correctly for records with comments.
Created attachment 17455 [details] [review] Bug 10049 [Alternate] Comments tab shows ( ) instead of ( 0 ) on translated templates This patch is an alternate solution to the problem of "0" not appearing on the comment tab in translated templates. It uses the Template Toolkit option of specifying a default value for a variable: [% reviews.size || 0 %] This syntax is already in use elsewhere in the template. Same test plan: 1) Make sure comments/reviews are activated 2) Find a record with 0 comments/reviews in the opac 3) The tab shows: Comments ( 0 ) 4) Switch to any other languages. 5) Tab now shows: <translation of Comments> ( ) 6) Apply patch 7) Check English templates stil work correctly 8) Rerun 'perl translate install <langcode> 9) Check display is now correct. 10) Also check that number shows correctly for records with comments.
Created attachment 17496 [details] [review] [SIGNED-OFF] Bug 10049 [Alternate] Comments tab shows ( ) instead of ( 0 ) on translated templates This patch is an alternate solution to the problem of "0" not appearing on the comment tab in translated templates. It uses the Template Toolkit option of specifying a default value for a variable: [% reviews.size || 0 %] This syntax is already in use elsewhere in the template. Same test plan: 1) Make sure comments/reviews are activated 2) Find a record with 0 comments/reviews in the opac 3) The tab shows: Comments ( 0 ) 4) Switch to any other languages. 5) Tab now shows: <translation of Comments> ( ) 6) Apply patch 7) Check English templates stil work correctly 8) Rerun 'perl translate install <langcode> 9) Check display is now correct. 10) Also check that number shows correctly for records with comments. Signed-off-by: Magnus Enger <magnus@enger.priv.no> Tested with nb-NO. English still works after the patch. Norwegian was showing "Comments" untranslated, which had me a bit unnerved. Turns out the string to be translated changed from "Comments ( %s%s%s%s )" to "Comments ( %s )", so to make sure all was well, I had to update the nb-NO translation, fix the number of %s's in the Norwegian string, make sure it was not marked as fuzzy and then install nb-NO. And it works!
QA comment: Same as Magnus here, translate install is not enough but update + translation + install works ! Marked as Passed QA.
Created attachment 17544 [details] [review] Bug 10049 [Alternate] Comments tab shows ( ) instead of ( 0 ) on translated templates This patch is an alternate solution to the problem of "0" not appearing on the comment tab in translated templates. It uses the Template Toolkit option of specifying a default value for a variable: [% reviews.size || 0 %] This syntax is already in use elsewhere in the template. Same test plan: 1) Make sure comments/reviews are activated 2) Find a record with 0 comments/reviews in the opac 3) The tab shows: Comments ( 0 ) 4) Switch to any other languages. 5) Tab now shows: <translation of Comments> ( ) 6) Apply patch 7) Check English templates stil work correctly 8) Rerun 'perl translate install <langcode> 9) Check display is now correct. 10) Also check that number shows correctly for records with comments. Signed-off-by: Magnus Enger <magnus@enger.priv.no> Tested with nb-NO. English still works after the patch. Norwegian was showing "Comments" untranslated, which had me a bit unnerved. Turns out the string to be translated changed from "Comments ( %s%s%s%s )" to "Comments ( %s )", so to make sure all was well, I had to update the nb-NO translation, fix the number of %s's in the Norwegian string, make sure it was not marked as fuzzy and then install nb-NO. And it works! Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
This patch has been pushed to master and 3.12.x.