|
Description
Baptiste Wojtkowski (bwoj)
2024-10-04 08:27:10 UTC
Bundle implementation in Bug 37996 does not implement marc representation of the relation between bundles and items they contain. Bug 29560 creates a link (773 in MARC21, 461 in UNIMARC) from the item in the bundle to the bundle itself. We also should also add a MARC link from the bundle to the item contained in the dedicated field (774 in MARC21, 462 in UNIMARC). Created attachment 172395 [details] [review] Bug 38093: Add unit tests Created attachment 172396 [details] [review] Bug 38093: Bundles: Add MARC link options should also add a link in 462 Bundle implementation in Bug 37996 does not implement marc representation of the relation between bundles and items they contain. Bug 29560 creates a link (773 in MARC21, 461 in UNIMARC) from the item in the bundle to the bundle itself. We also should also add a MARC link from the bundle to the item contained in the dedicated field (774 in MARC21, 462 in UNIMARC). TEST PLAN: 1 - Apply patches 2 - Create a bundle item (cf after for more info on this step if you are unfamiliar with bundles) 3 - Click on "manage bundle", fill with an existing barcode, click on Add marc link 4 - Click on MARC -> tab 7 (on marc21) and tab 4 on unimarc -> there should be a link in field 733 (marc21) or 462 (unimarc) 5 - Go back to the bundle and remove the item from the bundle. 6 - Click again on MARC -> precedent fields should now be empty Create a bundle : 1 - Create a new biblio/Edit an existing one, edit the field 000, the field 7 must be set to c-Collection. 2 - Go on the biblio and create a new item attached to the biblio 3 - You should see an icon "Manage bundle" on the item created Created attachment 172397 [details] [review] Bug 38093: Bundles: Remove the link created in 462 when removing from the bundle Tried both on KTD and a sandbox, ending with the same error : error: sha1 information is lacking or useless (Koha/Biblio.pm). error: could not build fake ancestor Patch failed at 0001 Bug 38093: Bundles: Remove the link created in 462 when removing from the bundle Created attachment 177573 [details] [review] Bug 38093: Bundles: Add MARC link options should also add a link in 462 Bundle implementation in Bug 37996 does not implement marc representation of the relation between bundles and items they contain. Bug 29560 creates a link (773 in MARC21, 461 in UNIMARC) from the item in the bundle to the bundle itself. We also should also add a MARC link from the bundle to the item contained in the dedicated field (774 in MARC21, 462 in UNIMARC). TEST PLAN: 1 - Apply patches 2 - Create a bundle item (cf after for more info on this step if you are unfamiliar with bundles) 3 - Click on "manage bundle", fill with an existing barcode, click on Add marc link 4 - Click on MARC -> tab 7 (on marc21) and tab 4 on unimarc -> there should be a link in field 733 (marc21) or 462 (unimarc) 5 - Go back to the bundle and remove the item from the bundle. 6 - Click again on MARC -> precedent fields should now be empty Create a bundle : 1 - Create a new biblio/Edit an existing one, edit the field 000, the field 7 must be set to c-Collection. 2 - Go on the biblio and create a new item attached to the biblio 3 - You should see an icon "Manage bundle" on the item created Created attachment 177574 [details] [review] Bug 38093: Bundles: Remove the link created in 462 when removing from the bundle blocked: following dependencies leads Bug 37996 If I understand correctly, it shouldn't be in the need signoff queue because changes to Bug 37996 would likely invalidate testing. Hi Victor,
I've made some work to get things moving, I think this patch can no longer be considered as "BLOCKED" since the patch it was relying upon has been resolved as duplicate.
I've removed the dependency to the duplicate and with minor changes to previous patches was able to apply that one.
There is a small typo in the first patch which should be fixed this way :
<pre>
kohadev-koha@kohadevbox:koha(malles)$ git diff
diff --git a/Koha/Biblio.pm b/Koha/Biblio.pm
index d8e378f47e4..579027d1b9b 100644
--- a/Koha/Biblio.pm
+++ b/Koha/Biblio.pm
@@ -2220,7 +2220,7 @@ sub link_marc_bundled_item {
if ( ref( $params->{bundled_item} ) eq 'Koha::Item' ) {
$bundled_item = $params->{bundled_item};
} else {
- my $bundled_item = Koha::Items->find( $params->{host} );
+ my $bundled_item = Koha::Items->find( $params->{bundled_item} );
}
my $record = $self->metadata->record;
</pre>
Also the test plan has another typo (says 733 in step 4, it should be 774).
Test plan may also document that 774 subfields 0,9 and t should be configured in the marc framework, and systempreference "EasyAnalyticalRecords" set to 1.
Otherwise it may not work.
I'm setting this patch to In Discussion because it needs some changes which I need to discuss a bit further with BibLibre's colleagues before submitting.
Arthur
Created attachment 190961 [details] [review] Bug 38093: Bundles: (QA follow-up) fix small error in parameter name This patch set is ready for signoff! |