Bug 8911 - docs/history.txt file missing for about.pl
Summary: docs/history.txt file missing for about.pl
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: master
Hardware: All All
: P5 - low minor (vote)
Assignee: Owen Leonard
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-12 07:05 UTC by Mark Tompsett
Modified: 2014-05-26 21:04 UTC (History)
6 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 8911 - history.txt file missing for about.pl (3.12 KB, patch)
2013-07-02 21:46 UTC, Tomás Cohen Arazi
Details | Diff | Splinter Review
Bug 8911 - history.txt file missing for about.pl (3.18 KB, patch)
2013-07-07 08:25 UTC, Chris Cormack
Details | Diff | Splinter Review
[PASSED QA] Bug 8911 - history.txt file missing for about.pl (3.26 KB, patch)
2013-07-21 08:54 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Tompsett 2012-10-12 07:05:33 UTC
When opening about, three error log entries were triggered. Two were silent warnings, and an entry caused by this file not being there.

Upon further research, I discovered that the entire docs directory was not in /usr/share/koha/intranet/cgi/docs as it is expected by the about.pl module. I copied the docs directory from the tarball, and the error disappeared.

I believe it is a packaging issue which should be fixed on a forward basis.
Comment 1 Mark Tompsett 2012-10-12 07:07:12 UTC
Assigned to Robin, as per his instructions.
Comment 2 Robin Sheat 2012-11-07 04:20:18 UTC
So, this appears to be a larger issue: if I do a makefile install, history.txt isn't installed anywhere.
Comment 3 Tomás Cohen Arazi 2013-07-02 21:46:00 UTC Comment hidden (obsolete)
Comment 4 Chris Cormack 2013-07-07 08:25:38 UTC Comment hidden (obsolete)
Comment 5 Katrin Fischer 2013-07-14 08:10:53 UTC
Robin, could you take a look at this please? :)
Comment 6 Katrin Fischer 2013-07-21 08:54:07 UTC
Created attachment 19827 [details] [review]
[PASSED QA] Bug 8911 - history.txt file missing for about.pl

This patch makes Makefile.PL put the history.txt file in the right places
depending on the chosen setup layout, adds a reference to that place in
koha-conf.xml (and debian template version), and finally tweaks about.pl to
use it.

To test, apply the patch and verify that perl Makefile.PL runs fine, and installing in
 - dev
 - single
 - standard
layouts works as expected. Then go to the about.pl page and see if Koha's history shows there.

Then, build your packages and test on your newly created instances.

Regards
To+

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Comment 7 Galen Charlton 2013-07-22 14:44:19 UTC
Pushed to master.  Thanks, Tomás!
Comment 8 Tomás Cohen Arazi 2013-07-22 15:05:31 UTC
This patch has been pushed to 3.12.x, will be in 3.12.2.

Added a followup that advertises the change for people updating to add the line to koha-conf.xml manually.
Comment 9 Bernardo Gonzalez Kriegel 2013-08-04 19:59:57 UTC
Pushed to 3.10.x, will be in 3.10.10
Comment 10 Marcel de Rooy 2013-09-19 11:40:34 UTC
When just upgrading your dev install, you now lose the contents of the timeline page on about.
With warnings like:

about.pl: Use of uninitialized value $docdir in string at /usr/share/koha/testclone/about.pl line 152.

about.pl: readline() on closed filehandle $file at /usr/share/koha/testclone/about.pl line 158.
Comment 11 Marcel de Rooy 2013-09-19 12:28:25 UTC
Please see bug 10915