Bug 17911

Summary: Message and timeout mismatch at the end of the install process
Product: Koha Reporter: Mark Tompsett <mtompset>
Component: Installation and upgrade (command-line installer)Assignee: Mark Tompsett <mtompset>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: jonathan.druart, julian.maurice, katrin.fischer, kyle, mtj
Version: MainKeywords: Academy
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 17911: Message and timeout mismatch
Bug 17911: Message and timeout mismatch
Bug 17911: Message and timeout mismatch

Description Mark Tompsett 2017-01-16 04:25:54 UTC
Step 3 in the installer says "If this page does not redirect in 5 seconds..." but the actual number is 10 seconds (see the second line with the meta refresh content says 10).

Solution: fix the meta refresh, because changing the message could be a translation mess.
Comment 1 Mark Tompsett 2017-01-16 04:30:04 UTC
Created attachment 59000 [details] [review]
Bug 17911: Message and timeout mismatch

The installation step 3 has a meta refresh of 10 seconds,
but the message says 5 seconds. People could get impatient.

TEST PLAN
---------
1) Fresh install
   -- notice the discrepancy on the web installation.
2) drop and create the DB
3) apply the patch
4) run the web install
   -- notice it refreshes faster (5 seconds vs 10 seconds)
5) run koha qa test tools
Comment 2 Alex Buckley 2017-04-17 05:43:23 UTC
Created attachment 62195 [details] [review]
Bug 17911: Message and timeout mismatch

The installation step 3 has a meta refresh of 10 seconds,
but the message says 5 seconds. People could get impatient.

TEST PLAN
---------
1) Fresh install
   -- notice the discrepancy on the web installation.
2) drop and create the DB
3) apply the patch
4) run the web install
   -- notice it refreshes faster (5 seconds vs 10 seconds)
5) run koha qa test tools

Works as intended and passes qa test tool
 OK koha-tmpl/intranet-tmpl/prog/en/modules/installer/step3.tt
  OK    forbidden patterns
  OK     git manipulation
  OK      spelling
  OK   tt_valid
  OK    valid_template

Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>
Comment 3 Jonathan Druart 2017-04-18 18:53:47 UTC
Created attachment 62322 [details] [review]
Bug 17911: Message and timeout mismatch

The installation step 3 has a meta refresh of 10 seconds,
but the message says 5 seconds. People could get impatient.

TEST PLAN
---------
1) Fresh install
   -- notice the discrepancy on the web installation.
2) drop and create the DB
3) apply the patch
4) run the web install
   -- notice it refreshes faster (5 seconds vs 10 seconds)
5) run koha qa test tools

Works as intended and passes qa test tool
 OK koha-tmpl/intranet-tmpl/prog/en/modules/installer/step3.tt
  OK    forbidden patterns
  OK     git manipulation
  OK      spelling
  OK   tt_valid
  OK    valid_template

Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 4 Kyle M Hall 2017-04-21 11:39:29 UTC
Pushed to master for 17.05, thanks Mark!
Comment 5 Katrin Fischer 2017-04-21 22:26:04 UTC
Changing the timing instead of the message == no string change :)

This patch has been pushed to 16.11.x and will be in 16.11.07.
Comment 6 Mason James 2017-05-03 04:35:04 UTC
Pushed to 16.05.x, for 16.05.12 release
Comment 7 Julian Maurice 2017-05-10 09:31:05 UTC
Pushed to 3.22.x for 3.22.21