Bug 35686 - Case missing from installer step 3 template title
Summary: Case missing from installer step 3 template title
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Installation and upgrade (web-based installer) (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor (vote)
Assignee: Owen Leonard
QA Contact: Lucas Gass
URL: /cgi-bin/koha/installer/install.pl?st...
Keywords:
Depends on:
Blocks:
 
Reported: 2024-01-03 13:07 UTC by Owen Leonard
Modified: 2024-01-19 22:32 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
This fixes a web browser page title for the web installer - from " > Web installer > Koha" to "Updating database structure > Web installer > Koha".
Version(s) released in:
24.05.00,23.11.02


Attachments
Bug 35686: Case missing from installer step 3 template title (1.54 KB, patch)
2024-01-03 14:38 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 35686: Case missing from installer step 3 template title (1.59 KB, patch)
2024-01-05 20:22 UTC, David Nind
Details | Diff | Splinter Review
Bug 35686: (follow-up) Correct page heading (1.00 KB, patch)
2024-01-08 14:23 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 35686: Case missing from installer step 3 template title (1.64 KB, patch)
2024-01-08 23:51 UTC, Lucas Gass
Details | Diff | Splinter Review
Bug 35686: (follow-up) Correct page heading (1.06 KB, patch)
2024-01-08 23:51 UTC, Lucas Gass
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2024-01-03 13:07:34 UTC
There is a case missing from the conditionals around title display for the installer's step 3. When you're on the "Updating database structure" step the first part of the page title is empty: " > Web installer > Koha"
Comment 1 Owen Leonard 2024-01-03 14:38:32 UTC
Created attachment 160467 [details] [review]
Bug 35686: Case missing from installer step 3 template title

This patch adds a missing case to the display of the title on step 3 of
the web installer. This resulted in an empty first part of the title:
" > Web installer > Koha"

To test, apply the patch and reset the database version from the command
line in Koha Testing Docker:

> koha-mysql kohadev
> UPDATE systempreferences SET value = '23.1200002 WHERE variable = 'version';
> quit
> Restart all

- Go to the staff interface. You should be taken to the web installer.
- Log in to the web installer and proceed through until you get to the
  "Updating database structure" page.
- Confirm that the page title is correct.
Comment 2 David Nind 2024-01-05 20:22:42 UTC
Created attachment 160593 [details] [review]
Bug 35686: Case missing from installer step 3 template title

This patch adds a missing case to the display of the title on step 3 of
the web installer. This resulted in an empty first part of the title:
" > Web installer > Koha"

To test, apply the patch and reset the database version from the command
line in Koha Testing Docker:

> koha-mysql kohadev
> UPDATE systempreferences SET value = '23.1200002 WHERE variable = 'version';
> quit
> Restart all

- Go to the staff interface. You should be taken to the web installer.
- Log in to the web installer and proceed through until you get to the
  "Updating database structure" page.
- Confirm that the page title is correct.

Signed-off-by: David Nind <david@davidnind.com>
Comment 3 David Nind 2024-01-05 20:34:34 UTC
Does the title displayed on the page need updating as well?

Most pages have this title pattern, for example: Web installer › Update database

On the page that this bug fixes, the page title is just: Updating database structure

Should this be: Web installer › Updating database structure

Testing notes using KTD:

1. I used this statement to update the version: UPDATE systempreferences SET value = '23.1200000' WHERE variable = 'version';
2. Restart everything: restart_all
3. Go to the staff interface
4. Upgrade starts
   - Use koha_kohadev as the user and password as the password
   - Grant more privileges to the koha_kohadev user:
     . mysql -uroot -ppassword -hkoha-db-1
     . use mysql
     . grant all on koha_kohadev.* to koha_kohadev;
     . flush privileges;
Comment 4 Owen Leonard 2024-01-08 14:23:01 UTC
Created attachment 160627 [details] [review]
Bug 35686: (follow-up) Correct page heading
Comment 5 Lucas Gass 2024-01-08 23:51:04 UTC
Straightforward and simple fixes, no issues with the QA tool.
Comment 6 Lucas Gass 2024-01-08 23:51:39 UTC
Created attachment 160677 [details] [review]
Bug 35686: Case missing from installer step 3 template title

This patch adds a missing case to the display of the title on step 3 of
the web installer. This resulted in an empty first part of the title:
" > Web installer > Koha"

To test, apply the patch and reset the database version from the command
line in Koha Testing Docker:

> koha-mysql kohadev
> UPDATE systempreferences SET value = '23.1200002 WHERE variable = 'version';
> quit
> Restart all

- Go to the staff interface. You should be taken to the web installer.
- Log in to the web installer and proceed through until you get to the
  "Updating database structure" page.
- Confirm that the page title is correct.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Comment 7 Lucas Gass 2024-01-08 23:51:41 UTC
Created attachment 160678 [details] [review]
Bug 35686: (follow-up) Correct page heading

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Comment 8 Katrin Fischer 2024-01-11 14:50:18 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 9 Fridolin Somers 2024-01-15 14:14:29 UTC
Pushed to 23.11.x for 23.11.02
Comment 10 Lucas Gass 2024-01-19 22:32:25 UTC
Merge conflicts with 23.05.x, no backport.