Bug 28236 - Selecting database columns for system preferences in standard and dev installs is broken
Summary: Selecting database columns for system preferences in standard and dev install...
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Staff interface (show other bugs)
Version: Main
Hardware: All All
: P5 - low critical
Assignee: Johanna Räisä
QA Contact: Tomás Cohen Arazi (tcohen)
URL:
Keywords:
Depends on: 9812 22844
Blocks:
  Show dependency tree / graph
 
Reported: 2021-04-27 11:58 UTC by Martin Renvoize (ashimema)
Modified: 2022-06-06 20:25 UTC (History)
10 users (show)

See Also:
Change sponsored?: Sponsored
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
This fixes Apache access to json files in koha-tmpl for non-package installs. This causes issues for system preferences where there is a pick list for database columns. Bug 22844 introduced a pick list of database columns for system preferences where such a list was required. This list is in a plain json file under the templates directory. This works fine for packages, but because of bug 9812 (which limited browser access to selected files) those directories are not accessible to the outside world for both standard and dev type installs using the make process.
Version(s) released in:
21.11.00
Circulation function:

Attachments
Bug 28236: Selecting database columns for system preferences in standard and dev installs is broken (1.10 KB, patch)
2021-09-02 10:00 UTC, Johanna Räisä 		Details | Diff | Splinter Review
Bug 28236: Selecting database columns for system preferences in standard and dev installs is broken (1.16 KB, patch)
2021-10-21 08:39 UTC, Martin Renvoize (ashimema) 		Details | Diff | Splinter Review
Bug 28236: Allow access to json files from apache config (1.19 KB, patch)
2021-11-03 12:26 UTC, Jonathan Druart 		Details | Diff | Splinter Review
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Renvoize (ashimema) 2021-04-27 11:58:04 UTC 
With bug 22844 we introduced a pick list of database columns for system preferences where such a list was required.

However, the list is in a plain json file under the templates directory.  This works fine for packages, but because of bug 9812 those directories are not accessible to the outside world for both standard and dev type installs using the make process.

We need to resolve this somehow.. and it raises the question as to whether templates should be available to view on packages at all?
Comment 1 Tomás Cohen Arazi (tcohen) 2021-04-27 12:17:54 UTC 
Should we just produce an API route for that...
Comment 2 Martin Renvoize (ashimema) 2021-04-27 12:46:52 UTC 
Or use the API specs that are already available perhaps?
Comment 3 Johanna Räisä 2021-09-02 08:22:54 UTC 
We have finally started our upgrading process to the newest Koha release and this problem came out when our superlibrarian was trying to set again systempreferences. 

This is a kind of show stopper. Maybe we could open those json-files from Apache?
Comment 4 Johanna Räisä 2021-09-02 10:00:23 UTC 
Created attachment 124424 [details] [review]
Bug 28236: Selecting database columns for system preferences  in standard and dev installs is broken

This patch grants json files from koha-tmpl. Otherwise systempreferences won't be able to fetch them.

TEST PLAN:

1) Try to set systempreference BorrowerUnwantedField.
2) The modal is empty.
3) Modify Apache configurations like in this patch.
4) Try again set the systempreference.
5) The modal should show list of parameters.

Sponsored-by: Koha-Suomi Oy
Comment 5 Martin Renvoize (ashimema) 2021-10-21 08:38:32 UTC 
Oops, I totally missed that you had added a patch here... looks good, signing off.
Comment 6 Martin Renvoize (ashimema) 2021-10-21 08:39:32 UTC 
Created attachment 126645 [details] [review]
Bug 28236: Selecting database columns for system preferences in standard and dev installs is broken

This patch grants json files from koha-tmpl. Otherwise systempreferences won't be able to fetch them.

TEST PLAN:

1) Try to set systempreference BorrowerUnwantedField.
2) The modal is empty.
3) Modify Apache configurations like in this patch.
4) Try again set the systempreference.
5) The modal should show list of parameters.

Sponsored-by: Koha-Suomi Oy
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 7 Fridolin Somers 2021-11-03 07:27:08 UTC 
From Martin request, I test this since we use dev install in production servers.
We usually use Nginx but we have some Apache installs (because of Sibboleth).

Works fine, I reproduce the HTTP 403 without this patch.

Passed QA
Comment 8 Jonathan Druart 2021-11-03 12:26:42 UTC 
Created attachment 127248 [details] [review]
Bug 28236: Allow access to json files from apache config

This patch grants json files from koha-tmpl. Otherwise systempreferences won't be able to fetch them.

TEST PLAN:

1) Try to set systempreference BorrowerUnwantedField.
2) The modal is empty.
3) Modify Apache configurations like in this patch.
4) Try again set the systempreference.
5) The modal should show list of parameters.

Sponsored-by: Koha-Suomi Oy
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

JD Amended patch: replace tab characters and reword commit title
Comment 9 Jonathan Druart 2021-11-03 14:30:40 UTC 
Pushed to master for 21.11, thanks to everybody involved!
Comment 10 Jonathan Druart 2021-11-03 14:35:32 UTC 
Congratulations for your first patch pushed, Johanna!
Comment 11 Johanna Räisä 2021-11-09 08:17:45 UTC 
(In reply to Jonathan Druart from comment #10)
> Congratulations for your first patch pushed, Johanna!

Nice, thank you very much. :)