Bug 13676

Summary: OpacSuppression description says 'items' but means 'records'
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: System AdministrationAssignee: sherryn <sherryn>
Status: RESOLVED FIXED QA Contact: Josef Moravec <josef.moravec>
Severity: trivial    
Priority: P5 - low CC: chris, claire_gravely, gmcharlt, jens.weber, jonathan.druart, josef.moravec, mtj, nengard, sherryn, tomascohen, veron
Version: master   
Target Milestone: ---   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19833
Whiteboard:
Change sponsored?: --- Patch complexity: String patch
Bot Control: --- When did the bot last check this:
Who signed the patch off: Text to go in the release notes:
Version(s) released in:
Attachments: Bug 13676 - OpacSuppression description says 'items' but means 'records'
Bug 13676 - OpacSuppression description says 'items' but means 'records'
Bug 13676 - OpacSuppression description says 'items' but means 'records'
Bug: 13676: Fixing the OpacSuppression description
Bug: 13676: Fixing the OpacSuppression description
Bug: 13676: Fixing the OpacSuppression description

Description Katrin Fischer 2015-02-06 16:37:36 UTC
The OpacSuppression preference description is not clear:

... items marked as suppressed from OPAC search results. Note that you must have the <code>Suppress</code> index set up in Zebra and at least one suppressed item, or your searches will be broken. ...

The feature works on record level, not on item level.
Comment 1 Marc Véron 2015-02-10 17:36:20 UTC Comment hidden (obsolete)
Comment 2 Brendan Gallagher 2015-02-24 06:16:41 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2015-02-25 11:49:22 UTC
Created attachment 36164 [details] [review]
Bug 13676 - OpacSuppression description says 'items' but means 'records'

This patch changes the wording for the OpacSuppression preference description.

Without patch:
... items marked as suppressed from OPAC search results. Note that you must have the <code>Suppress</code> index set up in Zebra and at least one suppressed item, or your searches will be broken. ...

With patch:
... items marked as suppressed from OPAC search results. Note that you must have the <code>Suppress</code> index set up in Zebra and at least one suppressed biblio record, or your searches will be broken. ...

Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 4 Tomás Cohen Arazi 2015-03-02 15:30:15 UTC
Patch pushed to master.

Thanks Marc!
Comment 5 Chris Cormack 2015-03-12 07:07:31 UTC
Pushed to 3.18.x will be in 3.18.5
Comment 6 Mason James 2015-03-26 02:11:21 UTC
Contains string changes, holding for 3.16.10
Comment 7 Nicole C. Engard 2015-03-30 15:17:38 UTC
Shouldn't we then change the beginning of the preference description:

___ items marked as suppressed from OPAC search results. 

should be

___ biblios marked as suppressed from OPAC search results.
Comment 8 Jens Weber 2017-12-06 17:35:48 UTC
I agree with Nicole that this has to be changed in the preference description just the same!
(I hope it is okay I didn't file a new bug on this ...)
Comment 9 sherryn 2017-12-18 21:42:20 UTC
Created attachment 69862 [details] [review]
Bug: 13676: Fixing the OpacSuppression description

To Test
1/ Search for OpacSuppression system preference
2/ Notice that it says "Items marked as suppressed from OPAC search
results
3/ Noticre also that addresses is spelt wrong
4/ Apply patch
5/ Refresh the page, notice addresses is fixed and items has become
biblios
Comment 10 Katrin Fischer 2017-12-18 21:49:30 UTC
Created attachment 69863 [details] [review]
Bug: 13676: Fixing the OpacSuppression description

To Test
1/ Search for OpacSuppression system preference
2/ Notice that it says "Items marked as suppressed from OPAC search
results
3/ Noticre also that addresses is spelt wrong
4/ Apply patch
5/ Refresh the page, notice addresses is fixed and items has become
biblios

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 11 Katrin Fischer 2017-12-18 21:51:59 UTC
Thx for your first patch Sherryn!
Comment 12 Josef Moravec 2017-12-19 09:48:22 UTC
Created attachment 69900 [details] [review]
Bug: 13676: Fixing the OpacSuppression description

To Test
1/ Search for OpacSuppression system preference
2/ Notice that it says "Items marked as suppressed from OPAC search
results
3/ Noticre also that addresses is spelt wrong
4/ Apply patch
5/ Refresh the page, notice addresses is fixed and items has become
biblios

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 13 Jonathan Druart 2017-12-19 13:37:46 UTC
Sherryn, you did not specify your lastname in the commit message, is it expected?
Comment 14 Chris Cormack 2017-12-19 19:37:44 UTC
Hi Jonathan

It is Mak

Thank you
Comment 15 Jonathan Druart 2017-12-20 16:45:04 UTC
Patch pushed to master for 18.05, congrats Sherryn for your first patch in!