Attachment 168822 Details for Bug 29560 – Bug 29560: Add option to create MARC Links when bundling items

[patch] Bug 29560: Add option to create MARC Links when bundling items

Bug-29560-Add-option-to-create-MARC-Links-when-bun.patch (text/plain), 4.77 KB, created by Martin Renvoize (ashimema) on 2024-07-11 13:34:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2024-07-11 13:34:54 UTC

Size: 4.77 KB

patch

obsolete

>From ffe87d67ad65f7fccbe8aa5006ed726726b7ac17 Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Date: Wed, 18 May 2022 08:30:48 +0100
>Subject: [PATCH] Bug 29560: Add option to create MARC Links when bundling
> items
>
>This patch add a checkbox to the 'Add to bundle' modal allowing the user
>to optionally add MARC links between the new child record and the
>current biblio. For MARC21 this will result in a 773 pointing to the
>host from the item record and for UNIMARC a 461 field will be added.
>
>Signed-off-by: Tuomas Kunttu <tuomas.kunttu@kouvola.fi>
>---
> Koha/REST/V1/Items.pm                                 |  5 +++++
> api/v1/swagger/definitions/bundle_link.yaml           |  3 +++
> .../intranet-tmpl/prog/en/modules/catalogue/detail.tt | 11 ++++++++---
> 3 files changed, 16 insertions(+), 3 deletions(-)
>
>diff --git a/Koha/REST/V1/Items.pm b/Koha/REST/V1/Items.pm
>index 4d147ed1953..2612cf326ee 100644
>--- a/Koha/REST/V1/Items.pm
>+++ b/Koha/REST/V1/Items.pm
>@@ -316,6 +316,7 @@ sub add_to_bundle {
>     return $c->render_resource_not_found("Bundle item")
>         unless $bundle_item;
> 
>+    my $add_link = $c->validation->param('body')->{'marc_link'} // 0;
>     return try {
>         my $options = {
>             force_checkin => $body->{force_checkin},
>@@ -323,6 +324,10 @@ sub add_to_bundle {
>         };
> 
>         my $link = $item->add_to_bundle($bundle_item, $options);
>+        if ($add_link) {
>+            $bundle_item->biblio->link_marc_host(
>+                { biblionumber => $item->biblio->biblionumber } );
>+        }
>         return $c->render(
>             status  => 201,
>             openapi => $bundle_item
>diff --git a/api/v1/swagger/definitions/bundle_link.yaml b/api/v1/swagger/definitions/bundle_link.yaml
>index b564d039ee6..82d042a16bc 100644
>--- a/api/v1/swagger/definitions/bundle_link.yaml
>+++ b/api/v1/swagger/definitions/bundle_link.yaml
>@@ -19,4 +19,7 @@ properties:
>     type:
>       - boolean
>       - "null"
>+  marc_link:
>+    type: boolean
>+    description: Is there a marc link for this item
> additionalProperties: false
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
>index c50b09225ad..a5ffff247d0 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
>@@ -1074,6 +1074,10 @@
>                                         <input type="text" id="external_id" name="external_id" required="required">
>                                         <span class="required">Required</span>
>                                     </li>
>+                                    <li>
>+                                        <label for="bundle_link">Add MARC link: </label>
>+                                        <input type="checkbox" name="bundle_link"/>
>+                                    </li>
>                                 </ol>
>                             </fieldset>
>                         </div>
>@@ -1712,6 +1716,7 @@
>                     });
> 
>                     const barcode = data.external_id;
>+                    const marc_link = data.marc_link;
> 
>                     /* Report the results */
>                     posting.done(function(data) {
>@@ -1731,7 +1736,7 @@
>                                     .addClass('btn btn-xs')
>                                     .text(_("Check in and add to bundle"))
>                                     .on('click', function () {
>-                                        addToBundle(url, { external_id: barcode, force_checkin: true });
>+                                        addToBundle(url, { external_id: barcode, force_checkin: true, marc_link: marc_link });
>                                     });
>                                 $('#addResult')
>                                     .empty()
>@@ -1748,7 +1753,7 @@
>                                     .addClass('btn btn-xs')
>                                     .text(_("Ignore holds and add to bundle"))
>                                     .on('click', function () {
>-                                        addToBundle(url, { external_id: barcode, ignore_holds: true });
>+                                        addToBundle(url, { external_id: barcode, ignore_holds: true, marc_link: marc_link });
>                                     });
>                                 $('#addResult')
>                                     .empty()
>@@ -1779,7 +1784,7 @@
>                     event.preventDefault();
> 
>                     const url = this.action;
>-                    const data = { external_id: this.elements.external_id.value };
>+                    const data = { external_id: this.elements.external_id.value, marc_link: this.elements.bundle_link.checked };
> 
>                     addToBundle(url, data);
>                 });
>-- 
>2.45.2

Actions: View | Diff | Splinter Review

Attachments on bug 29560: 135090 | 135091 | 137509 | 137510 | 156299 | 156300 | 159081 | 159082 | 168720 | 168721 | 168722 | 168723 | 168816 | 168817 | 168818 | 168819 | 168820 | 168821 | 168822 | 168823 | 168824 | 168825 | 168885 | 168886 | 168887 | 168888 | 168889 | 168968