Bug 14427: Use Koha::version instead of C4::Context->KOHAVERSION
authorMark Tompsett <mtompset@hotmail.com>
Fri, 19 Jun 2015 18:00:13 +0000 (14:00 -0400)
committerTomas Cohen Arazi <tomascohen@unc.edu.ar>
Tue, 7 Jul 2015 18:16:43 +0000 (15:16 -0300)
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>
Signed-off-by: Tomas Cohen Arazi <tomascohen@unc.edu.ar>
installer/install.pl

index 056107e..cc67557 100755 (executable)
@@ -326,7 +326,7 @@ elsif ( $step && $step == 3 ) {
         my $now = POSIX::strftime( "%Y-%m-%dT%H:%M:%S", localtime() );
         my $logdir = C4::Context->config('logdir');
         my $dbversion = C4::Context->preference('Version');
-        my $kohaversion = C4::Context->KOHAVERSION;
+        my $kohaversion = Koha::version;
         $kohaversion =~ s/(.*\..*)\.(.*)\.(.*)/$1$2$3/;
 
         my $filename_suffix = join '_', $now, $dbversion, $kohaversion;