Bug 18277

Summary: Koha::Biblio - Remove GetBiblionumberFromItemnumber
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, m.de.rooy
Version: unspecified   
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: 18276    
Bug Blocks: 17628, 18260    
Attachments: Bug 18277: Remove GetBiblionumberFromItemnumber - Easy ones
Bug 18277: Remove GetBiblionumberFromItemnumber - linkitem
Bug 18277: Remove GetBiblionumberFromItemnumber - create_analytical_rel.pl
Bug 18277: Remove the C4::Biblio::GetBiblionumberFromItemnumber
[SIGNED-OFF] Bug 18277: Remove GetBiblionumberFromItemnumber - Easy ones
[SIGNED-OFF] Bug 18277: Remove GetBiblionumberFromItemnumber - linkitem
[SIGNED-OFF] Bug 18277: Remove GetBiblionumberFromItemnumber - create_analytical_rel.pl
[SIGNED-OFF] Bug 18277: Remove the C4::Biblio::GetBiblionumberFromItemnumber
Bug 18277: Remove GetBiblionumberFromItemnumber - Easy ones
Bug 18277: Remove GetBiblionumberFromItemnumber - linkitem
Bug 18277: Remove GetBiblionumberFromItemnumber - create_analytical_rel.pl
Bug 18277: Remove the C4::Biblio::GetBiblionumberFromItemnumber
Bug 18277: Remove GetBiblionumberFromItemnumber - Easy ones
Bug 18277: Remove GetBiblionumberFromItemnumber - linkitem
Bug 18277: Remove GetBiblionumberFromItemnumber - create_analytical_rel.pl
Bug 18277: Remove the C4::Biblio::GetBiblionumberFromItemnumber
Bug 18277: [QA Follow-up] Additional polishing

Description Jonathan Druart 2017-03-15 19:08:46 UTC

    
Comment 1 Jonathan Druart 2017-03-15 19:53:31 UTC
Created attachment 61127 [details] [review]
Bug 18277: Remove GetBiblionumberFromItemnumber - Easy ones

To retrieve a biblionumber from an itemnumber, we can use:
  Koha::Item->biblio->biblionumber

This is only what this patchset does.
Doing that we will be able to get rid of the
C4::Biblio::GetBiblionumberFromItemnumber subroutine.

Test plan:
- Acquisition module: cancel a receipt
- Export a record to CSV
- Modify items in a batch

Item's info should be correct

Other changes with be checked by QA team, by reading the code.
Comment 2 Jonathan Druart 2017-03-15 19:53:37 UTC
Created attachment 61128 [details] [review]
Bug 18277: Remove GetBiblionumberFromItemnumber - linkitem

Test plan:
Enable EasyAnalyticalRecords and link an item to another one (Edit >
link to host item)
Comment 3 Jonathan Druart 2017-03-15 19:53:46 UTC
Created attachment 61129 [details] [review]
Bug 18277: Remove GetBiblionumberFromItemnumber - create_analytical_rel.pl

I have no idea how to test this patch, see bug 5528, or simply read the
code.
Comment 4 Jonathan Druart 2017-03-15 19:53:52 UTC
Created attachment 61130 [details] [review]
Bug 18277: Remove the C4::Biblio::GetBiblionumberFromItemnumber
Comment 5 Josef Moravec 2017-05-09 15:51:47 UTC
Created attachment 63301 [details] [review]
[SIGNED-OFF] Bug 18277: Remove GetBiblionumberFromItemnumber - Easy ones

To retrieve a biblionumber from an itemnumber, we can use:
  Koha::Item->biblio->biblionumber

This is only what this patchset does.
Doing that we will be able to get rid of the
C4::Biblio::GetBiblionumberFromItemnumber subroutine.

Test plan:
- Acquisition module: cancel a receipt
- Export a record to CSV
- Modify items in a batch

Item's info should be correct

Other changes with be checked by QA team, by reading the code.

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 6 Josef Moravec 2017-05-09 15:51:52 UTC
Created attachment 63302 [details] [review]
[SIGNED-OFF] Bug 18277: Remove GetBiblionumberFromItemnumber - linkitem

Test plan:
Enable EasyAnalyticalRecords and link an item to another one (Edit >
link to host item)

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 7 Josef Moravec 2017-05-09 15:51:56 UTC
Created attachment 63303 [details] [review]
[SIGNED-OFF] Bug 18277: Remove GetBiblionumberFromItemnumber - create_analytical_rel.pl

I have no idea how to test this patch, see bug 5528, or simply read the
code.

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 8 Josef Moravec 2017-05-09 15:52:02 UTC
Created attachment 63304 [details] [review]
[SIGNED-OFF] Bug 18277: Remove the C4::Biblio::GetBiblionumberFromItemnumber

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 9 Jonathan Druart 2017-07-05 18:42:33 UTC
Created attachment 64819 [details] [review]
Bug 18277: Remove GetBiblionumberFromItemnumber - Easy ones

To retrieve a biblionumber from an itemnumber, we can use:
  Koha::Item->biblio->biblionumber

This is only what this patchset does.
Doing that we will be able to get rid of the
C4::Biblio::GetBiblionumberFromItemnumber subroutine.

Test plan:
- Acquisition module: cancel a receipt
- Export a record to CSV
- Modify items in a batch

Item's info should be correct

Other changes with be checked by QA team, by reading the code.

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 10 Jonathan Druart 2017-07-05 18:42:37 UTC
Created attachment 64820 [details] [review]
Bug 18277: Remove GetBiblionumberFromItemnumber - linkitem

Test plan:
Enable EasyAnalyticalRecords and link an item to another one (Edit >
link to host item)

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 11 Jonathan Druart 2017-07-05 18:42:41 UTC
Created attachment 64821 [details] [review]
Bug 18277: Remove GetBiblionumberFromItemnumber - create_analytical_rel.pl

I have no idea how to test this patch, see bug 5528, or simply read the
code.

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 12 Jonathan Druart 2017-07-05 18:42:45 UTC
Created attachment 64822 [details] [review]
Bug 18277: Remove the C4::Biblio::GetBiblionumberFromItemnumber

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 13 Jonathan Druart 2017-07-05 18:43:09 UTC
Patches have been rebased.
Comment 14 Marcel de Rooy 2017-07-07 07:35:50 UTC
QA: Looking here now
Comment 15 Marcel de Rooy 2017-07-07 08:35:38 UTC
Created attachment 64892 [details] [review]
Bug 18277: Remove GetBiblionumberFromItemnumber - Easy ones

To retrieve a biblionumber from an itemnumber, we can use:
  Koha::Item->biblio->biblionumber

This is only what this patchset does.
Doing that we will be able to get rid of the
C4::Biblio::GetBiblionumberFromItemnumber subroutine.

Test plan:
- Acquisition module: cancel a receipt
- Export a record to CSV
- Modify items in a batch

Item's info should be correct

Other changes with be checked by QA team, by reading the code.

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 16 Marcel de Rooy 2017-07-07 08:35:43 UTC
Created attachment 64893 [details] [review]
Bug 18277: Remove GetBiblionumberFromItemnumber - linkitem

Test plan:
Enable EasyAnalyticalRecords and link an item to another one (Edit >
link to host item)

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 17 Marcel de Rooy 2017-07-07 08:35:47 UTC
Created attachment 64894 [details] [review]
Bug 18277: Remove GetBiblionumberFromItemnumber - create_analytical_rel.pl

I have no idea how to test this patch, see bug 5528, or simply read the
code.

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 18 Marcel de Rooy 2017-07-07 08:35:50 UTC
Created attachment 64895 [details] [review]
Bug 18277: Remove the C4::Biblio::GetBiblionumberFromItemnumber

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 19 Marcel de Rooy 2017-07-07 08:35:54 UTC
Created attachment 64896 [details] [review]
Bug 18277: [QA Follow-up] Additional polishing

Fix POD error in Biblio.pm, as reported by qa tools:
    Apparent command =cut not preceded by blank line

Resolve crash in t/db_dependent/Items/DelItem.t:
  Can't call method "biblio" on an undefined value at C4/Items.pm line 669.

Add find test in tools/batchMod.pl. Increase readability of map statement.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 20 Jonathan Druart 2017-07-10 17:53:59 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 21 Fridolin Somers 2017-07-18 11:52:43 UTC
Enhancement not pushed to 17.05.x