Bug 8911

Summary: docs/history.txt file missing for about.pl
Product: Koha Reporter: Mark Tompsett <mtompset>
Component: OPACAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact:
Severity: minor    
Priority: P5 - low CC: bgkriegel, gmcharlt, m.de.rooy, robin, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10915
GIT URL: Change sponsored?: ---
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 8911 - history.txt file missing for about.pl
Bug 8911 - history.txt file missing for about.pl
[PASSED QA] Bug 8911 - history.txt file missing for about.pl

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 (tcohen) 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 (tcohen) 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