Description
Lari Taskula
2019-03-16 00:00:22 UTC
Created attachment 86698 [details] [review] Bug 22527: Fix link to wrong database manual in web installer To reproduce: 1. Use MySQL or MariaDB for your Koha installation 2. Create an empty database and point $KOHA_CONF to this database name. 3. Grant SELECT privilege to your database user, e.g. grant select on koha3.* to 'koha'@'%' identified by 'koha'; 4. Go to step 2 of web installer where it checks database connection 5. Observe the following message User koha doesn't have enough privilege on database koha. Ask for or make a change in the user's privileges. User koha must have USAGE, INSERT, UPDATE, DELETE, DROP and CREATE privileges on database koha2. Need help? See manual for PostgreSQL 6. Note "See manual for PostgreSQL" is wrong. Should be "See manual for MySQL". 7. Apply patch 8. Refresh page and observe it now displays correct link Sponsored-by: Hypernova Oy Created attachment 86699 [details] [review] Bug 22527: Enhance visibility of error message Place database user privilege issue into a yellow box to enhance visilibty. Sponsored-by: Hypernova Oy Created attachment 87550 [details] [review] Bug 22527: Fix link to wrong database manual in web installer To reproduce: 1. Use MySQL or MariaDB for your Koha installation 2. Create an empty database and point $KOHA_CONF to this database name. 3. Grant SELECT privilege to your database user, e.g. grant select on koha3.* to 'koha'@'%' identified by 'koha'; 4. Go to step 2 of web installer where it checks database connection 5. Observe the following message User koha doesn't have enough privilege on database koha. Ask for or make a change in the user's privileges. User koha must have USAGE, INSERT, UPDATE, DELETE, DROP and CREATE privileges on database koha2. Need help? See manual for PostgreSQL 6. Note "See manual for PostgreSQL" is wrong. Should be "See manual for MySQL". 7. Apply patch 8. Refresh page and observe it now displays correct link Sponsored-by: Hypernova Oy Signed-off-by: Liz Rea <wizzyrea@gmail.com> Created attachment 87551 [details] [review] Bug 22527: Enhance visibility of error message Place database user privilege issue into a yellow box to enhance visilibty. Sponsored-by: Hypernova Oy Signed-off-by: Liz Rea <wizzyrea@gmail.com> *** Bug 22366 has been marked as a duplicate of this bug. *** (In reply to Katrin Fischer from comment #5) > *** Bug 22366 has been marked as a duplicate of this bug. *** Do we want the default of this link to be PostgreSQL? For instance, if the user is not using a MySQL database but for some reason is also not using a PostgreSQL database, they will still be pointed to the PostgreSQL manual. This could be confusing for first time users. Is this a case that could feasibly occur? I don't know enough about the types of databases that people use with Koha. (In reply to Hayley Mapley from comment #6) > (In reply to Katrin Fischer from comment #5) > > *** Bug 22366 has been marked as a duplicate of this bug. *** > > Do we want the default of this link to be PostgreSQL? For instance, if the > user is not using a MySQL database but for some reason is also not using a > PostgreSQL database, they will still be pointed to the PostgreSQL manual. > This could be confusing for first time users. Is this a case that could > feasibly occur? I don't know enough about the types of databases that people > use with Koha. I think the only databases Koha currently supports are MariaDB and MySQL - PostgreSQL support is not functional, but I think it was once an option or work had been started on it. Lari, could you comment too please? Will this work for MariaDB? MariaDB and MySQL are not significantly different in this regard. To be fair, we could omit the link entirely, and tell them what to search for for their chosen DBMS. For example: For help with granting permissions, please search for $DBMS manual grant permissions or something like. Or we could not say anything at all, and expect that working with DBMS systems is something lots of people should know how to do, or that they have an IT professional somewhere up the chain they can ask who knows what to search for. This is, I feel, a less nice way to go about it but it would keep us from having to maintain this external connection (I mean, we link to MySQL manual version 4.1 - that's really old!) I think Liz makes a good point about the link to the MySQL manual being quite old. Maybe having a generic message would be the best way to go here. Created attachment 88084 [details] [review] Bug 2527: web installer links to outdated DB manual To reproduce: 1. Use MySQL or MariaDB for your Koha installation 2. Create an empty database and point $KOHA_CONF to this database name. 3. Grant SELECT privilege to your database user, e.g. grant select on koha3.* to 'koha'@'%' identified by 'koha'; 4. Go to step 2 of web installer where it checks database connection 5. Observe the following message User koha doesn't have enough privilege on database koha. Ask for or make a change in the user's privileges. User koha must have USAGE, INSERT, UPDATE, DELETE, DROP and CREATE privileges on database koha2. Need help? See manual for PostgreSQL 6. Note "See manual for PostgreSQL" is wrong. Should now read Need help? For help with granting permissions, please search online for "[% dbms %] manual grant permissions" of course the dbms should be the dbms you are using. 7. Apply patch, restart all 8. Refresh page and observe it now displays correct link Test plan and original patchset by Lari Taskula <lari.taskula@hypernova.fi> Sponsored-by: Hypernova Oy Created attachment 89198 [details] [review] Bug 22527: web installer links to outdated DB manual To reproduce: 1. Use MySQL or MariaDB for your Koha installation 2. Create an empty database and point $KOHA_CONF to this database name. 3. Grant SELECT privilege to your database user, e.g. grant select on koha3.* to 'koha'@'%' identified by 'koha'; 4. Go to step 2 of web installer where it checks database connection 5. Observe the following message User koha doesn't have enough privilege on database koha. Ask for or make a change in the user's privileges. User koha must have USAGE, INSERT, UPDATE, DELETE, DROP and CREATE privileges on database koha2. Need help? See manual for PostgreSQL 6. Note "See manual for PostgreSQL" is wrong. Should now read Need help? For help with granting permissions, please search online for "[% dbms %] manual grant permissions" of course the dbms should be the dbms you are using. 7. Apply patch, restart all 8. Refresh page and observe it now displays correct link Test plan and original patchset by Lari Taskula <lari.taskula@hypernova.fi> Sponsored-by: Hypernova Oy Signed-off-by: Hayley Mapley <hayleymapley@catalyst.net.nz> Created attachment 89374 [details] [review] Bug 22527: web installer links to outdated DB manual To reproduce: 1. Use MySQL or MariaDB for your Koha installation 2. Create an empty database and point $KOHA_CONF to this database name. 3. Grant SELECT privilege to your database user, e.g. grant select on koha3.* to 'koha'@'%' identified by 'koha'; 4. Go to step 2 of web installer where it checks database connection 5. Observe the following message User koha doesn't have enough privilege on database koha. Ask for or make a change in the user's privileges. User koha must have USAGE, INSERT, UPDATE, DELETE, DROP and CREATE privileges on database koha2. Need help? See manual for PostgreSQL 6. Note "See manual for PostgreSQL" is wrong. Should now read Need help? For help with granting permissions, please search online for "[% dbms %] manual grant permissions" of course the dbms should be the dbms you are using. 7. Apply patch, restart all 8. Refresh page and observe it now displays correct link Test plan and original patchset by Lari Taskula <lari.taskula@hypernova.fi> Sponsored-by: Hypernova Oy Signed-off-by: Hayley Mapley <hayleymapley@catalyst.net.nz> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 89375 [details] [review] Bug 22527: Remove extra space after '?' Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Nick, not sure about the 2 spaces after '?'. I let you decide (and squash!) Awesome work all! Included second patch and squashed Pushed to master for 19.05 Created attachment 89452 [details] [review] Bug 22527: (RM follow-up) Add filter Pushed to 18.11.x for 18.11.06 backported to 18.05.x for 18.05.13 |