Bug 16225 - Extra closing quote in circulation home page template.
Summary: Extra closing quote in circulation home page template.
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL: /cgi-bin/koha/circ/circulation-home.pl
Keywords:
Depends on: 15640
Blocks:
  Show dependency treegraph
 
Reported: 2016-04-07 16:48 UTC by Owen Leonard
Modified: 2016-12-05 21:22 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 16225 - Extra closing quote in circulation home page template (1.75 KB, patch)
2016-04-07 16:54 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 16225 - Extra closing quote in circulation home page template (1.78 KB, patch)
2016-04-08 08:08 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2016-04-07 16:48:29 UTC
The changes made in Bug 15640 left behind an extra closing quote at line 42 of circulation-home.tt.
Comment 1 Owen Leonard 2016-04-07 16:54:16 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2016-04-08 08:08:09 UTC
Created attachment 50050 [details] [review]
Bug 16225 - Extra closing quote in circulation home page template

This patch removes an extra closing quote left behind by the changes
made in Bug 15640.

This patch also corrects the case of the "Circulation reports"
heading. (See coding guideline HTML4).

To test, apply the patch and go to the circulation home page. Validate
the page's HTML and confirm that there are no errors.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 3 Jonathan Druart 2016-04-08 08:08:25 UTC
Trivial, skipping QA.
Comment 4 Brendan Gallagher 2016-04-22 23:02:32 UTC
Pushed to Master - Should be in the May 2016 Release.  Thanks!
Comment 5 Julian Maurice 2016-05-16 11:18:00 UTC
Not needed in stable versions, status changed to RESOLVED