Bug 14427

Summary: Use Koha::version() instead of C4::Context->KOHAVERSION
Product: Koha Reporter: Mark Tompsett <mtompset>
Component: Architecture, internals, and plumbingAssignee: Mark Tompsett <mtompset>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: minor    
Priority: P5 - low CC: jonathan.druart, m.de.rooy, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9006
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13876
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 13758, 13876    
Bug Blocks:    
Attachments: Bug 14427: Use Koha::version instead of C4::Context->KOHAVERSION
Bug 14427: Use Koha::version instead of C4::Context->KOHAVERSION
Bug 14427: Use Koha::version instead of C4::Context->KOHAVERSION

Description Mark Tompsett 2015-06-19 17:45:06 UTC
While testing bug 9006 and reading bug 13876, KOHAVERSION was discovered in the installer/install.pl script.

Considering other sections use my $var = Koha::version; type code, this upgrade section needs fixing.
Comment 1 Mark Tompsett 2015-06-19 18:16:18 UTC Comment hidden (obsolete)
Comment 2 Mark Tompsett 2015-06-19 18:22:24 UTC
Also: git grep KOHAVERSION
Should only show release notes entry. :)
Comment 3 Jonathan Druart 2015-06-22 11:21:21 UTC
This is because 13876 and 13758 have been written tested more or less at the same time.
Comment 4 Jonathan Druart 2015-06-22 11:24:24 UTC
The only problem is that the generated log file is named
updatedatabase_2015-06-22T13:18:26_3.2100009_.log
when it should be
updatedatabase_2015-06-22T13:23:31_3.2100009_3.2100010.log
Comment 5 Jonathan Druart 2015-06-22 11:24:57 UTC Comment hidden (obsolete)
Comment 6 Marcel de Rooy 2015-07-03 07:32:47 UTC
Created attachment 40771 [details] [review]
Bug 14427: Use Koha::version instead of C4::Context->KOHAVERSION

While testing bug 9006, I discovered this gem in
installer/install.pl

TEST PLAN
---------
 1) back up DB
 2) drop db and create db
 3) git reset --hard origin/3.20.x
 4) run web installer and all sample data
 5) git reset --hard origin/master
 6) empty your koha intranet error log
 7) run web installer to upgrade
    -- there will be warnings regarding regex around
       installer/install.pl line 328 or so.
       grep "\$kohaversion" ~/koha-dev/var/logs/koha-error_log
 8) repeat steps 2-6
 9) apply this patch
10) run web installer to upgrade
    -- no regex warnings.
       grep "\$kohaversion" ~/koha-dev/var/logs/koha-error_log
11) koha qa test tools

Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 7 Tomás Cohen Arazi 2015-07-07 18:29:56 UTC
Patch pushed to master.

Thanks Mark!