Attachment 156300 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.53 KB, created by Martin Renvoize (ashimema) on 2023-09-27 16:07:39 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2023-09-27 16:07:39 UTC

Size: 4.53 KB

patch

obsolete

>From 12a767d4f8d6ce512ebef7543f6b164d9b2bbf6f 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.
>---
> 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 30791352864..81e31589c40 100644
>--- a/Koha/REST/V1/Items.pm
>+++ b/Koha/REST/V1/Items.pm
>@@ -307,6 +307,7 @@ sub add_to_bundle {
>         );
>     }
> 
>+    my $add_link = $c->validation->param('body')->{'marc_link'} // 0;
>     return try {
>         my $options = {
>             force_checkin => $body->{force_checkin},
>@@ -314,6 +315,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 a3c08aa693c..95a6b5b938f 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
>@@ -1299,6 +1299,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>
>@@ -1974,6 +1978,7 @@
>                   });
> 
>                   const barcode = data.external_id;
>+                  const marc_link = data.marc_link;
> 
>                   /* Report the results */
>                   posting.done(function(data) {
>@@ -1993,7 +1998,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()
>@@ -2010,7 +2015,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()
>@@ -2041,7 +2046,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.41.0

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