Bug 17248

Summary: Koha::AuthorisedValues - Remove GetKohaAuthorisedValueLib
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: brendan, claire_gravely
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 17216    
Bug Blocks: 23912, 15799, 17249    
Attachments: Bug 17248 - Koha::AuthorisedValues - Remove GetKohaAuthorisedValueLib
Bug 17248 - Koha::AuthorisedValues - Remove GetKohaAuthorisedValueLib
Bug 17248 - Koha::AuthorisedValues - Remove GetKohaAuthorisedValueLib
Bug 17248 - Koha::AuthorisedValues - Remove GetKohaAuthorisedValueLib
[PASSED QA] Bug 17248 - Koha::AuthorisedValues - Remove GetKohaAuthorisedValueLib

Description Jonathan Druart 2016-09-05 14:00:54 UTC
The subroutine C4::Koha::GetKohaAuthorisedValueLib just retrieves a description (lib) for a given authorised value.

We can easily replace it using:
  Koha::AuthorisedValues->search({ category => $cat, authorised_value => $av })->lib
or
  Koha::AuthorisedValues->search({ category => $cat, authorised_value => $av })->opac_description
Comment 1 Jonathan Druart 2016-09-05 14:13:24 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2016-09-05 15:26:46 UTC Comment hidden (obsolete)
Comment 3 Claire Gravely 2016-09-05 22:37:43 UTC
could not apply bug 17216 (depends on)
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/admin/authorised_values.tt
Failed to merge in the changes.
Comment 4 Jonathan Druart 2016-09-06 07:21:47 UTC
(In reply to Claire Gravely from comment #3)
> could not apply bug 17216 (depends on)
> CONFLICT (content): Merge conflict in
> koha-tmpl/intranet-tmpl/prog/en/modules/admin/authorised_values.tt
> Failed to merge in the changes.

Rebased.
Comment 5 Claire Gravely 2016-09-06 20:47:27 UTC Comment hidden (obsolete)
Comment 6 Jonathan Druart 2016-09-08 17:47:20 UTC
Created attachment 55377 [details] [review]
Bug 17248 - Koha::AuthorisedValues - Remove GetKohaAuthorisedValueLib

The subroutine C4::Koha::GetKohaAuthorisedValueLib just retrieves a description
(lib) for a given authorised value.

We can easily replace it using:
  Koha::AuthorisedValues->search({ category => $cat, authorised_value => $av })->lib
or
  Koha::AuthorisedValues->search({ category => $cat, authorised_value => $av })->opac_description

Test plan:
- On the detail page of a bibliographic record, the description for notforloan,
restricted and stack (?) should be correctly displayed
- View a shelf, the location (LOC) description should be displayed
- On the search result page, the location description should be displayed in the
  facets
- Set AcqCreateItem=ordering and receiving items.
The description for notforloan, restricted, location, ccode, etc. field
should be displayed.
- When creating item in the acquisition module, the dropdown list for
  field linked to AV should display the AV' descriptions
- On the transfers page, the description of the location should be
  displayed.
- On the checkout list from the circulation.pl and returns.pl pages, the
  description for "materials" should be displayed
- Fill some OPAC_SUG AV and create a suggestion, the reason dropdown
  list should display the description of OPAC_SUG

Signed-off-by: Claire Gravely <claire_gravely@hotmail.com>
Comment 7 Jonathan Druart 2016-09-08 17:47:36 UTC
just a rebase.
Comment 8 Katrin Fischer 2016-10-12 12:36:23 UTC
Created attachment 56267 [details] [review]
[PASSED QA] Bug 17248 - Koha::AuthorisedValues - Remove GetKohaAuthorisedValueLib

The subroutine C4::Koha::GetKohaAuthorisedValueLib just retrieves a description
(lib) for a given authorised value.

We can easily replace it using:
  Koha::AuthorisedValues->search({ category => $cat, authorised_value => $av })->lib
or
  Koha::AuthorisedValues->search({ category => $cat, authorised_value => $av })->opac_description

Test plan:
- On the detail page of a bibliographic record, the description for notforloan,
restricted and stack (?) should be correctly displayed
- View a shelf, the location (LOC) description should be displayed
- On the search result page, the location description should be displayed in the
  facets
- Set AcqCreateItem=ordering and receiving items.
The description for notforloan, restricted, location, ccode, etc. field
should be displayed.
- When creating item in the acquisition module, the dropdown list for
  field linked to AV should display the AV' descriptions
- On the transfers page, the description of the location should be
  displayed.
- On the checkout list from the circulation.pl and returns.pl pages, the
  description for "materials" should be displayed
- Fill some OPAC_SUG AV and create a suggestion, the reason dropdown
  list should display the description of OPAC_SUG

Signed-off-by: Claire Gravely <claire_gravely@hotmail.com>

Signed-off-by: Katrin Fischer  <katrin.fischer@bsz-bw.de>
Comment 9 Brendan Gallagher 2016-10-13 15:23:49 UTC
Pushed to Master - Should be in the November 16.11 Release.  Thanks!