Bug 22454

Summary: Add Koha::Item::hidden_in_opac method
Product: Koha Reporter: Tomás Cohen Arazi <tomascohen>
Component: Architecture, internals, and plumbingAssignee: Tomás Cohen Arazi <tomascohen>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: jonathan.druart, josef.moravec, katrin.fischer, martin.renvoize, mtompset, nick
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:
Bug Depends on:    
Bug Blocks: 22455, 22749    
Attachments: Bug 22454: Unit tests
Bug 22454: Add Koha::Item::hidden_in_opac method
Bug 22454: Add Koha::Item::hidden_in_opac method
Bug 22454: Unit tests
Bug 22454: Add Koha::Item::hidden_in_opac method

Description Tomás Cohen Arazi 2019-03-05 12:44:08 UTC
We need such a method to determine if an item needs to be hidden in OPAC. It should take the OpacHiddenItems hashref as (optional) parameter. The case for 'exceptions' is left out on purpose. This should be just calculated by the calling party.
Comment 1 Tomás Cohen Arazi 2019-03-05 12:47:48 UTC
Created attachment 86061 [details] [review]
Bug 22454: Unit tests
Comment 2 Tomás Cohen Arazi 2019-03-05 12:47:52 UTC
Created attachment 86062 [details] [review]
Bug 22454: Add Koha::Item::hidden_in_opac method

This patch adds a hidden_in_opac method that does the same calculation
done in GetHiddenItemnumbers, but for a single item, and doesn't get the
OpacHiddenItems syspref, but expects them to be passed as parameters (to
avoid multiple reads).

To test:
- Apply this patches
- Run:
  $ kshell
 k$ prove t/db_dependent/Koha/Item.t
=> SUCCESS: Tests pass!
Comment 3 Michal Denar 2019-03-12 08:55:10 UTC
Created attachment 86501 [details] [review]
Bug 22454: Add Koha::Item::hidden_in_opac method

This patch adds a hidden_in_opac method that does the same calculation
done in GetHiddenItemnumbers, but for a single item, and doesn't get the
OpacHiddenItems syspref, but expects them to be passed as parameters (to
avoid multiple reads).

To test:
- Apply this patches
- Run:
  $ kshell
 k$ prove t/db_dependent/Koha/Item.t
=> SUCCESS: Tests pass!

Signed-off-by: Michal Denar <black23@gmail.com>
Comment 4 Josef Moravec 2019-03-12 10:12:56 UTC
Created attachment 86509 [details] [review]
Bug 22454: Unit tests

Signed-off-by: Michal Denar <black23@gmail.com>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 5 Josef Moravec 2019-03-12 10:12:59 UTC
Created attachment 86510 [details] [review]
Bug 22454: Add Koha::Item::hidden_in_opac method

This patch adds a hidden_in_opac method that does the same calculation
done in GetHiddenItemnumbers, but for a single item, and doesn't get the
OpacHiddenItems syspref, but expects them to be passed as parameters (to
avoid multiple reads).

To test:
- Apply this patches
- Run:
  $ kshell
 k$ prove t/db_dependent/Koha/Item.t
=> SUCCESS: Tests pass!

Signed-off-by: Michal Denar <black23@gmail.com>

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 6 Nick Clemens (kidclamp) 2019-03-16 13:55:15 UTC
Awesome work all!

Pushed to master for 19.05
Comment 7 Martin Renvoize 2019-03-19 11:07:20 UTC
Enhancement will not be backported to 18.11.x series.