From 5e51279ce9ace65057a707cf10a486283bfbf4d4 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 16 Nov 2022 07:40:59 +0100 Subject: [PATCH] Bug 31891: (bug 27421 follow-up) Remove one additional click From comment 0 """ Moving marc batch import actions to background jobs has introduced some confusing clicks to the process. Before this change, after staging a batch one was taken to a "MARC staging results" page with details of how the staging went and, most importantly, a link straight to the batch you'd just staged (at manage-marc-import.pl?import_batch_id). Now, however, one must click the status bar to get to details of the job and then click "View batch" from there. Not only is that an extra click and page load, but neither of those links are particularly large or obvious. """ Test plan: - Stage a record for import, notice that the "View batch" link appears when the job is finished - Add order to basket from a new file, select a file and import Notice that the "Add staged files to basket" link is displayed when the job is finished Note for QA: We should have a js_callback in the background_job include file instead. Signed-off-by: Katrin Fischer Signed-off-by: Marcel de Rooy Signed-off-by: Tomas Cohen Arazi --- .../prog/en/modules/tools/stage-marc-import.tt | 16 +++++++++++++--- koha-tmpl/intranet-tmpl/prog/js/job_progess.js | 5 +++-- 2 files changed, 16 insertions(+), 5 deletions(-) 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 6c6be6c43f..dfb1d05fe7 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 @@ -79,6 +79,7 @@

The job has been enqueued! It will be processed as soon as possible.

[% INCLUDE "job_progress.inc" job_id=job_id %]

View detail of the enqueued job +

[% ELSE %]

Stage MARC records for import

@@ -105,8 +106,7 @@
Upload progress: - - + 0%
@@ -428,7 +428,17 @@ }) }); [% IF job_enqueued %] - updateProgress([% job_id | html %]); + updateProgress([% job_id | html %], function(){ + $.getJSON('/api/v1/jobs/[% job_id | html %]', function(job){ + let import_batch_id = job.data.report.import_batch_id; + $('

%s

'.format(import_batch_id, _("View batch"))).appendTo("#job_callback"); + let basket_id = job.data.report.basket_id; + let vendor_id = job.data.report.vendor_id; + if ( basket_id && vendor_id ) { + $('

%s

'.format(import_batch_id, basket_id, vendor_id, _("Add staged files to basket"))).appendTo("#job_callback"); + } + }); + }); [% END %] }); diff --git a/koha-tmpl/intranet-tmpl/prog/js/job_progess.js b/koha-tmpl/intranet-tmpl/prog/js/job_progess.js index 5b91507596..8da44420a1 100644 --- a/koha-tmpl/intranet-tmpl/prog/js/job_progess.js +++ b/koha-tmpl/intranet-tmpl/prog/js/job_progess.js @@ -1,4 +1,4 @@ -function updateProgress(job_id) { +function updateProgress(job_id, callback) { $.getJSON('/api/v1/jobs/' + job_id, function(job){ let recheck = true; @@ -19,6 +19,7 @@ function updateProgress(job_id) { $('#progress-bar-' + job_id).addClass("progress-bar-success"); $('#progress-bar-' + job_id).attr('aria-valuenow', 100).css("width", "100%"); recheck = false; + callback(); } else if ( job.status == "failed" ) { $('#job-percent-' + job_id).text(0); $('#job-status-' + job_id).text(JOB_PROGRESS_FAILED); @@ -28,7 +29,7 @@ function updateProgress(job_id) { } if ( recheck ) { - setTimeout(function(){updateProgress(job_id)}, 1 * 1000); + setTimeout(function(){updateProgress(job_id, callback)}, 1 * 1000); } }); } -- 2.39.5