Bug 24682

Summary: Clarify UsageStatsGeolocation syspref description and behaviour
Product: Koha Reporter: paxed <pasi.kallinen>
Component: System AdministrationAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: 1joynelson, gmcharlt, hayleypelham, jonathan.druart, katrin.fischer, lucas, martin.renvoize, maryse.simard
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
20.05.00, 19.11.05, 19.05.10
Attachments: Bug 24682: Make UsageStatsGeolocation readonly (and redirect to the config page)
Bug 24682: Make UsageStatsGeolocation readonly (and redirect to the config page)
Bug 24682: Make UsageStatsGeolocation readonly (and redirect to the config page)

Description paxed 2020-02-19 10:40:42 UTC
The syspref UsageStatsGeolocation needs to be improved, in couple ways:

Perform value format checking (eg. there should be no spaces, no letters)
OR
Make the value read-only.

AND

Change the syspref description to mention this is edited in <intranet>/cgi-bin/koha/admin/usage_statistics.pl (with a link to there, of course)
Comment 1 Jonathan Druart 2020-02-21 09:37:09 UTC
Created attachment 99361 [details] [review]
Bug 24682: Make UsageStatsGeolocation readonly (and redirect to the config page)

To prevent invalid values in this pref (and so on the server), we should
make this input readonly on the syspref page.
The sysprefs related to Hea should be edited from the dedicated page, so
also adding a note about that.

Test plan:
Search syspref with "UsageStats"
Notice the note about the admin page
Notice the UsageStatsGeolocation input is now readonly (and resized, to
display the whole value when filled)
Comment 2 Maryse Simard 2020-02-21 21:13:36 UTC
Created attachment 99452 [details] [review]
Bug 24682: Make UsageStatsGeolocation readonly (and redirect to the config page)

To prevent invalid values in this pref (and so on the server), we should
make this input readonly on the syspref page.
The sysprefs related to Hea should be edited from the dedicated page, so
also adding a note about that.

Test plan:
Search syspref with "UsageStats"
Notice the note about the admin page
Notice the UsageStatsGeolocation input is now readonly (and resized, to
display the whole value when filled)

Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Comment 3 Martin Renvoize 2020-02-24 12:15:56 UTC
Created attachment 99489 [details] [review]
Bug 24682: Make UsageStatsGeolocation readonly (and redirect to the config page)

To prevent invalid values in this pref (and so on the server), we should
make this input readonly on the syspref page.
The sysprefs related to Hea should be edited from the dedicated page, so
also adding a note about that.

Test plan:
Search syspref with "UsageStats"
Notice the note about the admin page
Notice the UsageStatsGeolocation input is now readonly (and resized, to
display the whole value when filled)

Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Martin Renvoize 2020-02-24 12:16:21 UTC
Useful simple improvement.. Passing QA
Comment 5 Martin Renvoize 2020-02-24 13:22:12 UTC
Nice work everyone!

Pushed to master for 20.05
Comment 6 Joy Nelson 2020-03-30 22:02:02 UTC
PUshed to 19.11.x branch for 19.11.05
Comment 7 Lucas Gass 2020-04-08 14:57:52 UTC
backported to 19.05.x for 19.05.10
Comment 8 Hayley Pelham 2020-04-14 02:50:24 UTC
Enhancement will not be backported to 18.11.x
Comment 9 Katrin Fischer 2020-04-19 01:47:06 UTC
*** Bug 20106 has been marked as a duplicate of this bug. ***