Attachment 170564 Details for Bug 36831 – Bug 36831: Allow for multiple file types

[patch] Bug 36831: Allow for multiple file types

Bug-36831-Allow-for-multiple-file-types.patch (text/plain), 2.68 KB, created by Matt Blenkinsop on 2024-08-21 12:52:33 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Matt Blenkinsop

Created: 2024-08-21 12:52:33 UTC

Size: 2.68 KB

patch

obsolete

>From 3c78f329c72aab6ca71dfce35b6ee41cc6567ed3 Mon Sep 17 00:00:00 2001
>From: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
>Date: Wed, 3 Jul 2024 09:52:45 +0000
>Subject: [PATCH] Bug 36831: Allow for multiple file types
>
>This patch allows the KBART import tool to accept different filetypes and then work out whether they are CSV or TSV files.
>
>Test plan:
>1) Try to import the file attached to the bug using the KBART import tool (E-resource management > eHoldings > Local > Titles > Import from a KBART file)
>2) It should return an error saying that the file must be .csv or .tsv
>3) Apply patch
>4) yarn build
>5) restart_all
>6) Repeat step 1
>7) The file should now successfully import
>---
> .../ERM/EHoldingsLocalTitlesKBARTImport.vue     | 17 +----------------
> 1 file changed, 1 insertion(+), 16 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalTitlesKBARTImport.vue b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalTitlesKBARTImport.vue
>index 5049bf9cb09..5cb0081f6e4 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalTitlesKBARTImport.vue
>+++ b/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalTitlesKBARTImport.vue
>@@ -2,21 +2,6 @@
>     <h2>{{ $__("Import from a KBART file") }}</h2>
>     <div class="page-section" id="files">
>         <form @submit="addDocument($event)" class="file_upload">
>-            <h3>{{ $__("Requirements:") }}</h3>
>-            <ul style="margin-bottom: 1.5em">
>-                <li>{{ $__("The file must be in TSV or CSV format") }}</li>
>-                <li>
>-                    {{
>-                        $__(
>-                            "The file should not contain any additional information / header rows, e.g. a file with a single title would be structured as follows:"
>-                        )
>-                    }}
>-                    <ol>
>-                        <li>{{ $__("Column headings row") }}</li>
>-                        <li>{{ $__("Title data row") }}</li>
>-                    </ol>
>-                </li>
>-            </ul>
>             <fieldset class="rows" id="package_list">
>                 <h3>{{ $__("Select file for upload") }}:</h3>
>                 <ol>
>@@ -173,7 +158,7 @@ export default {
>                     }
>                     if (success.invalid_filetype) {
>                         message += `<p>${this.$__(
>-                            "The file must be in .tsv or .csv format, please convert your file and try again."
>+                            "Could not detect whether the file is TSV or CSV, please check the file."
>                         )}</p>`
>                         setWarning(message)
>                     }
>-- 
>2.39.3 (Apple Git-146)

Actions: View | Diff | Splinter Review

Attachments on bug 36831: 166958 | 166959 | 166960 | 166961 | 166989 | 166990 | 166991 | 168431 | 168432 | 168433 | 168434 | 170549 | 170550 | 170551 | 170552 | 170562 | 170563 | 170564 | 170565