Bug 35408: Fix import job link

After importing titles from a list there is a link to the background job
that is displayed.
This link points to /cgi-bin/koha/admin/background_jobs/:id
There is a route to this path in routes/erm.js, to redirect to the
correct URI /cgi-bin/koha/admin/background_jobs.pl?op=view&id=:id
However this link will not work if we are accessing it from outside the
ERM app/module, or if we are opening it in a new tab/window.

The idea was interesting, but does not work...

Test plan:
Create a new ERM local package, a new list (virtual shelf), add some
items to the list.
Go to E-resource management > eHoldings > Local > Titles > Import from a list
Click "import"
Notice the "see job" link and click it

=> Confirm that with this patch applied you can open it in a new tab
without getting a 404

Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
(cherry picked from commit 09d5355ffb)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit 32eccd8974)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
Jonathan Druart 2023-11-28 09:11:43 +01:00 committed by Lucas Gass
parent 4bb0f0db8d
commit 38ce604ab1
2 changed files with 5 additions and 11 deletions

View file

@ -2,9 +2,12 @@
<h2>{{ $__("Import from a list") }}</h2>
<div v-if="job_id" class="dialog message">
{{ $__("Import in progress,") }}
<router-link :to="`/cgi-bin/koha/admin/background_jobs/${job_id}`">
<a
:href="`/cgi-bin/koha/admin/background_jobs.pl?op=view&id=${job_id}`"
target="_blank"
>
{{ $__("see job #%s").format(job_id) }}
</router-link>
</a>
</div>
<fieldset id="package_list" class="rows">
{{ $__("To the following local package") }}:

View file

@ -24,15 +24,6 @@ import LicensesFormAdd from "../components/ERM/LicensesFormAdd.vue";
import { $__ } from "../i18n";
export const routes = [
{
path: "/cgi-bin/koha/admin/background_jobs/:id",
is_base: true,
beforeEnter(to, from, next) {
window.location.href =
"/cgi-bin/koha/admin/background_jobs.pl?op=view&id=" +
to.params.id;
},
},
{
path: "/cgi-bin/koha/erm/erm.pl",
is_default: true,