Bug 7127

Summary: Templates must be valid XHTML
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: chris, katrin.fischer, koha.sekjal, m.de.rooy, paul.poulain, smoreland
Version: Main   
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:    
Bug Blocks: 7452    
Attachments: Proposed fix
Bug 7127 - Templates must be valid XHTML
Revised for current master
Bug 7127 - Follow-up - Templates must be valid XHTML
Signed off Patch
Bug 7127 - Follow-up - Templates must be valid XHTML
[SIGNED OFF] Bug 7127 - Follow-up - Templates must be valid XHTML
Bug 7127 - Templates must be valid XHTML
Bug 7127 - Templates must be valid XHTML

Description Owen Leonard 2011-11-01 16:42:38 UTC
All templates should be valid XHTML. I'm finding many new instances of unescaped ampersands which need to be corrected.
Comment 1 Owen Leonard 2011-11-18 19:11:55 UTC Comment hidden (obsolete)
Comment 2 Chris Cormack 2011-12-11 00:26:43 UTC Comment hidden (obsolete)
Comment 3 Paul Poulain 2011-12-13 16:53:16 UTC
QA comment:
* how to prove the validity of XHTML ? Is there a jenkin test for that ?
Comment 4 Marcel de Rooy 2012-01-10 15:21:34 UTC
QA: Looks good. Updating status..
Comment 5 Paul Poulain 2012-01-13 10:49:47 UTC
QA comment

Chris = is there a unit test for XHTML validity ?

+ merge problem:
Applying: Bug 7127 - Templates must be valid XHTML
/home/paul/koha.dev/koha-community/.git/rebase-apply/patch:64: space before tab in indent.
                      		<a href="/cgi-bin/koha/catalogue/search.pl?idx=hi&amp;q=[% itemloo.itemnumber %]">[% itemloo.countanalytics %] analytics</a>
warning: 1 line adds whitespace errors.
Using index info to reconstruct a base tree...
<stdin>:64: space before tab in indent.
                      		<a href="/cgi-bin/koha/catalogue/search.pl?idx=hi&amp;q=[% itemloo.itemnumber %]">[% itemloo.countanalytics %] analytics</a>
warning: 1 line applied after fixing whitespace errors.
Falling back to patching base and 3-way merge...
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt
Failed to merge in the changes.
Patch failed at 0001 Bug 7127 - Templates must be valid XHTML
Comment 6 Owen Leonard 2012-01-13 16:04:55 UTC Comment hidden (obsolete)
Comment 7 Owen Leonard 2012-01-26 20:12:26 UTC Comment hidden (obsolete)
Comment 8 Owen Leonard 2012-01-26 20:14:06 UTC
Attachment 7339 [details] can be applied and tested independently of attachment 7140 [details] [review].
Comment 9 Owen Leonard 2012-01-26 20:27:46 UTC
Attachment 7339 [details] should also fix Bug 7452.
Comment 10 Ian Walls 2012-02-03 12:59:15 UTC Comment hidden (obsolete)
Comment 11 Ian Walls 2012-02-03 13:00:17 UTC
Second patch double |html filters on page-numbers.inc; while this may be benign, it won't pass QA.
Comment 12 Ian Walls 2012-02-03 13:01:09 UTC
Marking Passed QA for signed off patch only, since it's a safe template-only change.
Comment 13 Owen Leonard 2012-02-03 13:39:08 UTC Comment hidden (obsolete)
Comment 14 Paul Poulain 2012-02-03 13:54:56 UTC
i'm not sure what to do with those patches. Should I push the 1st and reset status to need signoff?
Comment 15 Ian Walls 2012-02-03 23:39:56 UTC Comment hidden (obsolete)
Comment 16 Ian Walls 2012-02-03 23:40:59 UTC
Signed off and Passed QA on followup: both attached patches can now be committed.
Comment 17 Owen Leonard 2012-02-17 15:08:24 UTC
*** Bug 7452 has been marked as a duplicate of this bug. ***
Comment 18 Owen Leonard 2012-05-24 14:20:20 UTC Comment hidden (obsolete)
Comment 19 Jared Camins-Esakov 2012-05-26 10:47:53 UTC Comment hidden (obsolete)
Comment 20 Paul Poulain 2012-05-28 15:50:20 UTC
QA comment:
 * this patch does not apply on new/bug_7127 branch, that is too old
 * template only change, for better XHTML compliance

passed QA, will push them on master branch directly
Comment 21 Paul Poulain 2012-05-28 15:50:40 UTC
Comment on attachment 9788 [details] [review]
Bug 7127 - Templates must be valid XHTML

this patch has been pushed
Comment 22 Katrin Fischer 2013-04-14 10:43:45 UTC
Should we leave this open or better have individual new bugs for new problems?
That templates should be valid XHTML is part of the coding guidelines.
Comment 23 Owen Leonard 2013-05-15 14:38:40 UTC
(In reply to comment #22)
> Should we leave this open or better have individual new bugs for new
> problems?

I think in general individual bugs work better, and since we're using an HTML5 doctype now this bug should be closed.