Attachment 85353 Details for Bug 19417 – Bug 19417 - Display full error text when problem importing records

[patch] Bug 19417 - Display full error text when problem importing records

Bug-19417---Display-full-error-text-when-problem-i.patch (text/plain), 2.65 KB, created by Nick Clemens (kidclamp) on 2019-02-20 13:25:31 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2019-02-20 13:25:31 UTC

Size: 2.65 KB

patch

obsolete

>From 9a7f6c24a86de233de6a26276980606abba906db Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Thu, 5 Oct 2017 15:26:46 +0000
>Subject: [PATCH] Bug 19417 - Display full error text when problem importing
> records
>
>To test:
>1 - Upload a file to stage
>2 - Select your import options
>3 - Delete the file from the server, example:
>    sudo rm /tmp/koha_kohadev_uploads/
>4 - 'Click stage for import'
>5 - Get an alert 'Failed to submit form: error'
>6 - Apply patches
>7 - Repeat 1-4
>8 - Get a pretty error with additional info
>---
> koha-tmpl/intranet-tmpl/lib/jquery/plugins/humanmsg.js             | 1 -
> koha-tmpl/intranet-tmpl/prog/en/modules/tools/stage-marc-import.tt | 2 ++
> koha-tmpl/intranet-tmpl/prog/js/background-job-progressbar.js      | 2 +-
> 3 files changed, 3 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/lib/jquery/plugins/humanmsg.js b/koha-tmpl/intranet-tmpl/lib/jquery/plugins/humanmsg.js
>index 347d345230..a4c1ab9928 100644
>--- a/koha-tmpl/intranet-tmpl/lib/jquery/plugins/humanmsg.js
>+++ b/koha-tmpl/intranet-tmpl/lib/jquery/plugins/humanmsg.js
>@@ -75,7 +75,6 @@ var humanMsg = {
> 	bindEvents: function() {
> 	// Remove message if mouse is moved or key is pressed
> 		jQuery(document)
>-			.mousemove(humanMsg.removeMsg)
> 			.click(humanMsg.removeMsg)
> 			.keypress(humanMsg.removeMsg)
> 	},
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/stage-marc-import.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/stage-marc-import.tt
>index e169bcd12d..6ce1c675f9 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/stage-marc-import.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/stage-marc-import.tt
>@@ -8,6 +8,8 @@
>     #fileuploadstatus,#fileuploadfailed,#fileuploadcancel,#jobpanel,#jobstatus,#jobfailed { display : none; }
> </style>
> 
>+[% Asset.css("css/humanmsg.css") %]
>+
> </head>
> <body id="tools_stage-marc-import" class="tools">
> [% INCLUDE 'header.inc' %]
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/background-job-progressbar.js b/koha-tmpl/intranet-tmpl/prog/js/background-job-progressbar.js
>index cc08618d11..66dac87283 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/background-job-progressbar.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/background-job-progressbar.js
>@@ -69,7 +69,7 @@ function submitBackgroundJob(f) {
>                 backgroundJobProgressTimer = setInterval("updateJobProgress()", 500);
>             },
>             error: function(xml, textStatus) {
>-                alert('Failed to submit form: ' + textStatus);
>+                humanMsg.displayMsg( '<p>' + _('Failed to submit form: ') + textStatus + '</p></br>'+xml.responseText, { className: 'humanError' } );
>             }
> 
>         });
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 19417: 67648 | 85353 | 85367 | 86033 | 86037 | 86038 | 86039 | 86042 | 86528 | 86529 | 86530