[Bug 15929] New: typo in explanation for MaxSearchResultsItemsPerRecordStatusCheck
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 Bug ID: 15929 Summary: typo in explanation for MaxSearchResultsItemsPerRecordStatusCheck Change sponsored?: --- Product: Koha Version: 3.22 Hardware: All OS: All Status: NEW Severity: trivial Priority: P5 - low Component: System Administration Assignee: koha-bugs@lists.koha-community.org Reporter: hbraum@nekls.org QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com There's a typo in the explanation for the new MaxSearchResultsItemsPerRecordStatusCheck sys pref that was added in 3.22.2. "If a record has more than this number of items, they availability statuses ..." they should be the -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 Claire Gravely <c.gravely@arts.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |c.gravely@arts.ac.uk |ity.org | CC| |c.gravely@arts.ac.uk -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 Claire Gravely <c.gravely@arts.ac.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 --- Comment #1 from Claire Gravely <c.gravely@arts.ac.uk> --- Created attachment 52343 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=52343&action=edit Bug 15929 - typo in explanation for MaxSearchResultsItemsPerRecordStatusCheck There's a typo in the explanation for the new MaxSearchResultsItemsPerRecordStatusCheck sys pref that was added in 3.22.2. "If a record has more than this number of items, they availability statuses ..." I have re-worded the description so it reads a bit better. Test plan: 1. Check the description for syspref MaxSearchResultsItemsPerRecordStatusCheck. 2. Apply patch 3. Check syspref MaxSearchResultsItemsPerRecordStatusCheck description to make sure it makes sense. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 Mirko Tietgen <mirko@abunchofthings.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 Mirko Tietgen <mirko@abunchofthings.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52343|0 |1 is obsolete| | --- Comment #2 from Mirko Tietgen <mirko@abunchofthings.net> --- Created attachment 52344 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=52344&action=edit Bug 15929 - typo in explanation for MaxSearchResultsItemsPerRecordStatusCheck There's a typo in the explanation for the new MaxSearchResultsItemsPerRecordStatusCheck sys pref that was added in 3.22.2. "If a record has more than this number of items, they availability statuses ..." I have re-worded the description so it reads a bit better. Test plan: 1. Check the description for syspref MaxSearchResultsItemsPerRecordStatusCheck. 2. Apply patch 3. Check syspref MaxSearchResultsItemsPerRecordStatusCheck description to make sure it makes sense. Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org, | |kyle@bywatersolutions.com Version|3.22 |unspecified Depends on| |13022 --- Comment #3 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Letting this one to Kyle as he is the author of the description. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13022 [Bug 13022] Hardcoded limit causes records with more than 20 items to show inaccurate statuses -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #52344|0 |1 is obsolete| | --- Comment #4 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 52818 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=52818&action=edit Bug 15929 - typo in explanation for MaxSearchResultsItemsPerRecordStatusCheck There's a typo in the explanation for the new MaxSearchResultsItemsPerRecordStatusCheck sys pref that was added in 3.22.2. "If a record has more than this number of items, they availability statuses ..." I have re-worded the description so it reads a bit better. Test plan: 1. Check the description for syspref MaxSearchResultsItemsPerRecordStatusCheck. 2. Apply patch 3. Check syspref MaxSearchResultsItemsPerRecordStatusCheck description to make sure it makes sense. Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 --- Comment #5 from Kyle M Hall <kyle@bywatersolutions.com> --- (In reply to Jonathan Druart from comment #3)
Letting this one to Kyle as he is the author of the description.
Looks good to me! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 --- Comment #6 from Kyle M Hall <kyle@bywatersolutions.com> --- Pushed to master for 16.11, thanks Claire! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 Frédéric Demians <frederic@tamil.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |frederic@tamil.fr Status|Pushed to Master |Pushed to Stable --- Comment #7 from Frédéric Demians <frederic@tamil.fr> --- Pushed in 16.05. Will be in 16.05.02. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15929 Julian Maurice <julian.maurice@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |julian.maurice@biblibre.com --- Comment #8 from Julian Maurice <julian.maurice@biblibre.com> --- Patch pushed to 3.22.x, will be in 3.22.10 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org