Bug 8385

Summary: searchengine/solr/indexes.tt fails 'prove ./xt/tt_valid.t' test
Product: Koha Reporter: Mason James <mtj>
Component: TemplatesAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P5 - low CC: chris, paul.poulain
Version: 3.10   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 8407    
Bug Blocks:    
Attachments: Bug 8385: FIX spaces to make tt_valid.t happy
Bug 8385: FIX spaces to make tt_valid.t happy
Bug 8385: FIX spaces to make tt_valid.t happy
Bug 8385: Dirty patch to make tt_valid.t happy

Description Mason James 2012-07-09 06:13:22 UTC
$ prove ./xt/tt_valid.t
./xt/tt_valid.t .. 1/1 
#   Failed test 'TT syntax: not using TT directive within HTML tag'
#   at ./xt/tt_valid.t line 55.
# Files list: 
# intranet-tmpl/prog/en/modules/admin/searchengine/solr/indexes.tt: 132, 133, 134, 135, 136, 137, 141, 144
# Looks like you failed 1 test of 1.
./xt/tt_valid.t .. Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/1 subtests 

Test Summary Report
-------------------
./xt/tt_valid.t (Wstat: 256 Tests: 1 Failed: 1)
  Failed test:  1
  Non-zero exit status: 1
Files=1, Tests=1,  3 wallclock secs ( 0.02 usr  0.00 sys +  2.62 cusr  0.04 csys =  2.68 CPU)
Result: FAIL
Comment 1 Jonathan Druart 2012-07-13 07:21:57 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2012-07-16 15:21:45 UTC Comment hidden (obsolete)
Comment 3 Mason James 2012-07-18 02:54:30 UTC Comment hidden (obsolete)
Comment 4 Mason James 2012-07-18 02:55:50 UTC
(In reply to comment #3)
> Created attachment 10916 [details] [review]
> Bug 8385: FIX spaces to make tt_valid.t happy
> 
> must be apply after Bug 8407
> Updated with last modification for Bug 8407
> 
> Signed-off-by: Mason James <mtj@kohaaloha.com>


patch looks good, signing-off
Comment 5 Jonathan Druart 2012-08-01 13:41:08 UTC
Created attachment 11257 [details] [review]
Bug 8385: Dirty patch to make tt_valid.t happy

A problem in routine text_replace_tag (tmpl_process3.pl) have to be
fixed to prevent this kind of dirty patch :)
Comment 6 Paul Poulain 2012-08-03 09:47:04 UTC
QA comment: this patch is needed because of the limits of our translation tools. Jonathan tried to fix it, but failed. So it's still required that there is no [% %] inside html, because the translation parser can't handle them at all.
Comment 7 Chris Cormack 2013-04-20 11:01:37 UTC
Released in 3.10.0