Bug 16621

Summary: Translatability: Issues in opac-user.tt (sentence splitting)
Product: Koha Reporter: Marc Véron <veron>
Component: I18N/L10NAssignee: Marc Véron <veron>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: f.demians, fridolin.somers, julian.maurice, katrin.fischer, kyle
Version: master   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16898
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12509
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19849
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Fix translatability issues due to sentence splitting in koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
Version(s) released in:
Attachments: Bug 16621: Translatability: Issues in opac-user.tt (sentence splitting)
Bug 16621: Translatability: Issues in opac-user.tt (sentence splitting)
[SIGNED-OFF] Bug 16621: Translatability: Issues in opac-user.tt (sentence splitting)
[PASSED QA] Bug 16621: Translatability: Issues in opac-user.tt (sentence splitting)

Description Marc Véron 2016-05-31 09:04:21 UTC
Fix translation issues by sentence splitting in koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt line 103:

<li id="renewal_blocked_fines"><strong>Please note: </strong> Since you have <a href="/cgi-bin/koha/opac-account.pl">[% IF renewal_blocked_fines != "0.00" %] more than <span id="renewal_blocked_fines_amount">[% renewal_blocked_fines %]</span> in [% END %] fines</a>, you cannot renew your books online. Please pay your fines if you wish to renew your books.</li>
Comment 1 Marc Véron 2016-06-10 08:49:42 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2016-07-10 19:02:56 UTC Comment hidden (obsolete)
Comment 3 Owen Leonard 2016-07-11 15:19:57 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2016-07-11 20:33:36 UTC
Hm, I think we could leave out the "Reason:" here:
+  <li id="renewal_blocked_fines"><strong>Please note: </strong>You cannot renew your books online. Reason: [% IF ( renewal_blocked_fines ) > 0  %...

But not a blocker :)
Comment 5 Katrin Fischer 2016-07-11 20:34:39 UTC
Created attachment 53280 [details] [review]
[PASSED QA] Bug 16621: Translatability: Issues in opac-user.tt (sentence splitting)

This patch fixes issues due to sentence splitting in
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt

Note: This is a string patch. It does not add or change functionallity.

To test:
- Examine code and verify that the text changes remove sentence splitting
  and that the changes make sense.
- Log in to OPAC and verify that messages look correct
  - with blocked /debarred user (with and without comment and date)
  - with user who has to much fines (needs syspref OpacRenewalAllowed
    and syspref OPACFineNoRenewals)
  - with user who has his card marked as lost
  - with user with an account that is about to expire
  - with user with expired account
  - with user with uncertain address information (with and without
    syspref OPACPatronDetails set to Allow)

Note: Amended to make patch apply 2016-07-10 mv
Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 6 Kyle M Hall 2016-07-15 15:09:19 UTC
Pushed to master for 16.11, thanks Marc!
Comment 7 Frédéric Demians 2016-08-01 13:41:08 UTC
Pushed in 16.05. Will be in 16.05.02.
Comment 8 Julian Maurice 2016-08-17 11:01:35 UTC
Patch pushed to 3.22.x, will be in 3.22.10