Bug 32781

Summary: CreateEHoldingsFromBiblios not dealing with non-existent package correcly
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: RESOLVED FIXED QA Contact: Kyle M Hall <kyle>
Severity: normal    
Priority: P5 - low CC: kyle, pedro.amorim
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00,22.11.04
Bug Depends on: 30943    
Bug Blocks:    
Attachments: Bug 32781: Prevent import from list to fail if package does not exist
Bug 32781: Prevent import from list to fail if package does not exist
Bug 32781: Prevent import from list to fail if package does not exist

Description Jonathan Druart 2023-02-01 15:12:29 UTC

    
Comment 1 Jonathan Druart 2023-02-01 15:25:33 UTC
Created attachment 145959 [details] [review]
Bug 32781: Prevent import from list to fail if package does not exist

This is theorical (caught when writting tests) but we need to deal with
it.

Can't call method "resources" on an undefined value at /kohadevbox/koha/Koha/BackgroundJob/CreateEHoldingsFromBiblios.pm line 94.

Test plan:
  prove t/db_dependent/Koha/BackgroundJob/CreateEHoldingsFromBiblios.t
must return green
Comment 2 Pedro Amorim 2023-02-01 17:30:34 UTC
This is not applying for me:
Applying: Bug 32781: Prevent import from list to fail if package does not exist
error: sha1 information is lacking or useless (Koha/BackgroundJob/CreateEHoldingsFromBiblios.pm).
error: could not build fake ancestor
Comment 3 Jonathan Druart 2023-02-01 20:34:00 UTC
(In reply to Pedro Amorim from comment #2)
> This is not applying for me:
> Applying: Bug 32781: Prevent import from list to fail if package does not
> exist
> error: sha1 information is lacking or useless
> (Koha/BackgroundJob/CreateEHoldingsFromBiblios.pm).
> error: could not build fake ancestor

I was missing  dependency. It needs bug 32779 then bug 30943, and finally this one.
Comment 4 Pedro Amorim 2023-02-02 09:21:39 UTC
Created attachment 145992 [details] [review]
Bug 32781: Prevent import from list to fail if package does not exist

This is theorical (caught when writting tests) but we need to deal with
it.

Can't call method "resources" on an undefined value at /kohadevbox/koha/Koha/BackgroundJob/CreateEHoldingsFromBiblios.pm line 94.

Test plan:
  prove t/db_dependent/Koha/BackgroundJob/CreateEHoldingsFromBiblios.t
must return green

Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Comment 5 Kyle M Hall 2023-03-10 15:47:37 UTC
Created attachment 148077 [details] [review]
Bug 32781: Prevent import from list to fail if package does not exist

This is theorical (caught when writting tests) but we need to deal with
it.

Can't call method "resources" on an undefined value at /kohadevbox/koha/Koha/BackgroundJob/CreateEHoldingsFromBiblios.pm line 94.

Test plan:
  prove t/db_dependent/Koha/BackgroundJob/CreateEHoldingsFromBiblios.t
must return green

Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 6 Tomás Cohen Arazi 2023-03-13 19:05:40 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 7 Matt Blenkinsop 2023-03-14 10:44:40 UTC
Nice work everyone!

Pushed to stable for 22.11.x